Skip to content

harness:补上 #63 漏合的提交;评审方只读锁定与后台自动评审(T007) - #64

Merged
SnowsonZ merged 2 commits into
mainfrom
task/harness-review-background
Sep 28, 2026
Merged

SnowsonZ merged 2 commits into
mainfrom
task/harness-review-background

Conversation

@Snowson

@Snowson Snowson commented Sep 28, 2026

Copy link
Copy Markdown
Collaborator

任务

  • 任务书:docs/plans/task-007-review-background.md(T007,K7,随本 PR 提交:设计方自行实现,方案 B)
  • 目标终态:补上 harness:独立评审试行与评审校准(阶段二 P7,T006) #63 漏合的提交;OpenCode 评审方真正只读;评审在本机后台自动运行,不占用用户等待时间。
  • 对应验收编号:不挂规格(护栏);验收表见任务书
  • 非目标:不把评审放进 GitHub Actions;不安装 launchd 常驻(持久的本机配置,需另行决定)
  • 待办(docs/plans/backlog.md):无新增或关闭(B27 试行继续)

需要注意

证据

  • CI 运行(当前 head):见本 PR checks
  • 风险等级与修复证据:见 harness job summary(机器生成)

需要人工验收的部分

  • 合并后在本机终端运行 bin/dispatch review --watch,下一个 R2 以上 PR 的 CI 通过后几分钟内应出现独立评审评论。

🤖 Generated with Claude Code

Snowson and others added 2 commits September 29, 2026 00:25
- OpenCode 评审方(plan 代理只读、--pure),设为默认;此前卡住的原因是标准输入未关闭,
  所有评审方现关闭标准输入
- 已合并 PR 以合并提交的第一个父提交为基点(#62 首次试行拿到空材料,被评审方指出)
- 评审材料加入 CI 检查结论;提示词要求材料为空时不得判通过
- 试行 1/3 记入规范与待办(B27);#62 评审发现的 Swift 解析限制登记为 B33

补上 PR #63 合并时未包含的提交(原 87acfcf)。

Task: T006
Task: T007
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- OpenCode 评审方以配置关掉 bash、子代理、网页与写入类工具(plan 代理只禁止编辑,#62 试行中它运行了
  PR 的测试与 gh)
- bin/dispatch review --pending / --watch:逐个评审带 needs-independent-review、CI 已通过、当前 head
  尚无结论的 PR;停机标记存在即退出;不并行(多个 OpenCode 同时运行会冲突)
- 材料写进提示词的试验未见加快,撤回;任务书 T007 随本 PR(方案 B)
- 规范、harness README 同步

Task: T007
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added class:K7 needs-independent-review 待非设计方独立评审(试行) labels Sep 28, 2026
@github-actions
github-actions Bot requested a review from SnowsonZ September 28, 2026 16:31
@SnowsonZ
SnowsonZ merged commit b2e4fa8 into main Sep 28, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

class:K7 needs-independent-review 待非设计方独立评审(试行)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants