Skip to content

ci: 对依赖分支上的 PR 执行完整验证 - #32

Closed
ChenM0M wants to merge 2 commits into
mainfrom
codex/agent-config-ci-trigger
Closed

ChenM0M wants to merge 2 commits into
mainfrom
codex/agent-config-ci-trigger

Conversation

@ChenM0M

@ChenM0M ChenM0M commented Sep 22, 2026 •

Copy link
Copy Markdown
Owner

此前 pull_request 只匹配 main/dev,导致本批 15 个以其他 PR 分支为目标的依赖 PR 没有运行 CI。删除目标分支过滤,让所有 PR 执行相同的 macOS/Linux/Windows 门槛。

全部现有测试、打包步骤、平台矩阵、push 规则与权限保持原样。用 YAML 解析对比确认只改变 PR 触发范围。当前已有 PR 的补验将使用指向准确待测提交的临时 feature 验证分支,不等待此修复合并。

2026-09-23 repair verification

ChenM0M added a commit that referenced this pull request Sep 23, 2026
…dation

Integrates PRs #6-#32 (Agent configuration reliability, contract and model modality work) as one reviewable candidate, including the stacked-branch CI trigger fix and cross-platform test hardening.

- JSONC comments and unmanaged fields preserved when editing managed fields; failed discovery keeps the file and a structured error
- XDG_CONFIG_HOME / OPENCODE_CONFIG / OPENCODE_CONFIG_DIR discovery with absolute-path validation and WSL environment inspection
- Empty OpenCode config initialization; the selected config is bound to the launch environment
- Draft save/discard/cancel guards on context switches; save before launch and before setting defaults
- Model defaults keep provider identity; disabled models retained and controlled through native filters
- Undeclared reasoning/tool/modality capabilities stay unknown; modalities, token limits, variant JSON and thinking parameters are editable
- Effective config preview shows managed values and unknown sources and excludes credentials
- Windows launcher keeps native argv boundaries and reports real launch results; interactive agents get a visible console while background probes stay silent
- build.yml runs the full gates for PRs on dependent branches; MCP cold-start budget is separated from warm latency

Verification: three-platform CI green at 8841ad8 (run 35838494343); local production build, agent-profile behavior checks, 828 contract assertions, three-locale parity, release metadata, Rust formatting and workspace tests passed.

Not a release: Windows host/WSL native acceptance (A07/E07/F07-F10/G05) still requires the target environment against the Release artifact/hash.
@ChenM0M

ChenM0M commented Sep 23, 2026

Copy link
Copy Markdown
Owner Author

内容已随整合 PR #33 合入 main (e3cb573)。此 PR 为被取代的分支/栈式 PR,按用户指示关闭;分支保留,未删除。

@ChenM0M ChenM0M closed this Sep 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant