Skip to content

Build and Verify 检查数量断言阻断新增检查 #43

Description

@liuli195

问题

tests/test_skill_layout.py 使用精确的 len(checks) 断言验证 Build and Verify(构建与验证)配置。新增 verify.worktree-setup 后,完整 CI(持续集成)因检查数量从 17 变为 18 而失败;把常量改成 18 只修复了本次症状,下次新增合法检查仍会再次阻断。

证据

  • PR fix/share worktree dependencies #41 首轮 Full Verify(完整验证)失败:AssertionError: assert 18 == 17
  • 失败位置:tests/test_skill_layout.py::test_build_and_verify_covers_local_quant_research_without_local_data
  • 修改常量并重跑后 CI 通过,说明失败与检查行为无关,仅由全局数量变化触发。

建议解决方案

删除精确总数断言,改为验证必需检查 ID(标识)集合、ID 唯一性及确有语义要求的顺序。

验收标准

  • 新增独立且合法的 Build and Verify 检查时,不需要修改无关的全局数量常量。
  • 缺失必需检查、重复 ID 或关键顺序变化仍会使测试失败。
  • 现有 Build and Verify 配置测试和完整验证通过。

补充信息

优先级:P2 → P2(本次复盘确认 1 次,无动态调整)。

Metadata

Metadata

Assignees

No one assigned

    Labels

    P2Medium priority / non-blocking retained issuebugSomething isn't workingready-for-agentReady for an implementation agent

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions