logo
0
0
WeChat Login

fix: address remaining bugs from issue-40 audit — #40#42

Merged

DeepSeek-TUI-pro

NPC
created 6 days ago
dev
codex/issue-40-remaining-fixes
Edit
OverviewCommits
1
Files changed
5
Attachments

DeepSeek-TUI-pro

NPC

修复内容

本 PR 关闭 Issue #40 回测中余留的全部问题(Bug 3、Bug 4、Bug 6)。

Bug 3:AI Review 验收口径 (#12)

  • specs/issue-12-ai-review.md:记录 cnbcool/ai-review:latest + pr_comment: true 与原始验收标准 fail_on_critical=false 的等价性决策。
  • internal/config/spec_test.go:新增 TestValidateCNBPRPipelineAIReviewStage,读取 .cnb.yml 验证 PR pipeline 包含 ai-review 阶段及正确的 image/settings。

Bug 4:Runner 集成测试

  • internal/runner/runner_test.go:新增 4 个测试:
    • TestRunProbeCodexAndDoAgentRunners:codex + doagent 共存探针测试
    • TestRunCommandSelectsRunnerByTaskType:按 explicit runner ID 选路
    • TestRunProbeFallsBackToDefaultWhenProbeEmpty:Probe 为空时回退到内置探针
    • TestRunCommandRejectsUnknownRunner:未知 runner 正确报错

Bug 6:硬编码 Git Token

  • .cnb.yml:CNB_GIT_TOKEN 从硬编码改为 运行时变量。
  • .gitignore:新增 /.codex/ 防止误提交本地 Codex 运行态文件。

验证

  • go test ./...(待 CI 执行)
  • go build ./cmd/auto-harness(待 CI 执行)
NPC

@npc/DeepSeek-TUI(DeepSeek-TUI-pro) 你写的代码不错,但是 CI 检查未通过,请根据报错提示调整代码。
The code is well-crafted, yet the CI check failed. Please fix based on errors.

DeepSeek-TUI-pro

NPC
1
476,775 tokens, 9.620 credits, ¥0.481

DeepSeek-TUI-pro

NPC
referenced pull request

DeepSeek-TUI-pro

NPC

@wwyz(帷幄庸者) CI 失败的根因已定位并修复。

问题: 中解析 后用 访问通配符 pipeline key,但 YAML 解析后 key 是裸字符串 (不含引号),导致断言 始终返回 nil → → 测试失败。

修复: → ,与 YAML 解析后的实际键名一致。

commit 已推至当前分支 ,CI 重新触发后可验证通过。

is using the merge method to merge into8ccd0e4b
合并来自 codex/issue-40-remaining-fixes 的合并请求 #42

Successfully merged and closed

branch can be safely deleted
Reviewer
None yet
Assignee
None yet
Label
None yet
Participant