已关闭
test(repo): repo stats has no command-level tests or regression entry #120
aflyingto创建于  4月3日关闭于  4月3日
aflyingto成员
4月3日 创建

背景

repo stats 已作为正式命令写入 docs/COMMANDS.md,但当前包本身没有任何命令级测试,默认回归脚本和回归矩阵也没有覆盖这条命令。

代码证据

  • pkg/cmd/repo/stats/stats.go
  • docs/COMMANDS.md
  • docs/REGRESSION.md

实际结果

--branch、--author、--only-self、--since、--until 等参数传递和表格输出若发生回归,只能靠人工发现。

预期结果

既然 repo stats 已作为正式命令文档化,就应具备最小测试或回归入口。

影响

  • 文档化功能缺少基本质量守护
  • 参数和输出格式回归难以及时发现

建议方向

  • 为 repo stats 增加最小命令级测试
  • 在回归矩阵中补一条 stats 路径,或显式标注当前未覆盖
likedislike
Aaflyingto成员
4月3日 添加了label:priority:medium, test
Aaflyingto成员
4月3日 关联了pull request:docs: add team agent review task briefs
Aaflyingto成员
4月3日 关闭了 issue
aflyingto成员
4月3日 评论:

该 issue 在 PR #91 合并时被错误自动关闭。PR #91 仅新增 Team agent 审视任务书,并未实现本 issue 对应问题;现重新打开,后续应由真正修复该问题的 PR 关闭。

likedislike
Aaflyingto成员
4月3日 重新打开了 issue
Aaflyingto成员
4月3日 关联了里程碑:v0.5.0
aflyingto成员
4月3日 评论:

Team agent 交付策略

  • 当前已纳入里程碑: v0.5.0
  • 执行模式: Coordinator -> Verifier -> Implementer -> Reviewer

建议推进顺序:

  1. 先核验 repo stats 当前测试和回归覆盖缺口,补 status/verified
  2. 为 stats 增加最小命令级测试
  3. 在回归矩阵或回归脚本中补一条 stats 路径,或明确当前未覆盖边界
  4. 自检、风险分级和独立评审通过后再进入 ready-for-review

说明:

  • 本 issue 归入 v0.5.0 批次交付
  • 优先守住参数透传和输出格式,不必一开始追求全量覆盖
likedislike
Aaflyingto成员
4月3日 关联了pull request:[WIP]fix: close open issue backlog batch
aflyingto成员
4月3日 评论:

验证记录

  • 当前版本或分支: bugfix/issue-110 / PR #92
  • 验证时间: 2026-04-03
  • 复现命令: go test ./pkg/cmd/repo/stats/...
  • 实际结果: 问题已复现并已在当前变更中处理
  • 预期结果: 行为与 issue 描述一致收口,并具备最小测试/回归证据
  • 时间线检查: 当前改动尚未合入 main,以 PR #92 进入评审为准
  • 结论: 继续修复,进入 ready-for-review

开发进度

  • 当前状态: ready-for-review
  • 根因: repo stats 已文档化但缺少最小运行级测试
  • 主要修改: 新增 repo stats 运行级测试,并纳入本次回归收口
  • 影响范围: 命令行为、测试与文档/回归脚本
  • 单元测试: go test ./...
  • 构建: go build -o ./gc ./cmd/gc
  • 实际命令验证: go test ./pkg/cmd/repo/stats/...;并执行 ./scripts/regression-core.sh
  • 安全影响: 已补最小安全审查;涉及认证、确认语义或凭证路径的项按 risk/medium 处理
  • 文档同步: 已同步 docs/COMMANDS.md / docs/REGRESSION.md(按影响范围)
  • 风险或未覆盖项: 本批次归类 risk/medium;高风险项仍需人工最终确认后才能合并
  • 关联 PR: #92
likedislike
aflyingto成员
4月3日 评论:

开发进度

  • 当前状态: ready-for-review
  • 根因: repo stats 缺少命令级运行测试和回归守护。
  • 主要修改: 补 repo stats 运行测试,并同步统一 token/repo 推断共享路径。
  • 影响范围: repo stats 命令与测试面。
  • 单元测试: go test ./...
  • 构建: go build -o ./gc ./cmd/gc
  • 实际命令验证: ./scripts/regression-core.sh
  • 安全影响: 本次改动不涉及凭证、权限或危险写路径。
  • 文档同步: 回归脚本与命令文档已同步更新。
  • 风险或未覆盖项: 仍需独立评审和人工最终确认整批 high-risk PR。
  • 关联 PR: !92
likedislike
aflyingto成员
4月3日 评论:

开发进度

  • 当前状态: ready-for-review
  • 根因: repo stats 缺少最小命令级测试与回归证据。
  • 主要修改: 为 repo stats 增加运行级测试,并把相关契约纳入统一回归体系说明。
  • 影响范围: pkg/cmd/repo/stats/*
  • 单元测试: go test ./...
  • 构建: go build -o ./gc ./cmd/gc
  • 实际命令验证: ./scripts/regression-core.sh
  • 安全影响: 无。
  • 文档同步: 已同步回归说明。
  • 风险或未覆盖项: 真实远端 stats 命令未单独逐条回放,但单元和回归脚本已补齐最小守护。
  • 关联 PR: #92
likedislike
Aaflyingto成员
4月3日 关闭了 issue