Skip to content

tests:独立评审 watch 夹具的评论补上可信作者(delivery-harness T702 前置 D5) - #76

Merged
SnowsonZ merged 1 commit into
mainfrom
test/d5-review-fixture-author
Oct 3, 2026
Merged

SnowsonZ merged 1 commit into
mainfrom
test/d5-review-fixture-author

Conversation

@Snowson

@Snowson Snowson commented Oct 3, 2026

Copy link
Copy Markdown
Collaborator

需要你决定的

批准合并。合并后,delivery-harness 才能派发 T702。原因是那边的消费方契约 CI 会检出本仓库 main 来跑这组测试。

内容

只改 tests/test_harness_review_independent.py 的 BackgroundTest 夹具:给两条带评审标记的评论补上 "author": {"login": "Snowson"},即 checks.toml [identity] agent_login。

  • 为什么要改:delivery-harness 的 T702 上线后,watch 只认 agent_login 写的评审标记(防第三方评论伪造,见该仓库自治试验设计第 6 节)。这个夹具里的评论没有 author,在新引擎下 Rename bundle to Agent Notification, Dock unread badge, official CLI icons #3 会被当成未评审。
  • 新旧引擎结果一致:旧引擎的 reviewed_heads 不看作者;新引擎认得出可信作者。两边的待评审集合都是 [1, 4]。

已核对

  • 当前引擎下,python3 -m unittest tests.test_harness_review_independent 通过;推送前的 bin/verify 也通过。
  • 新引擎还要等 T702 实现,届时由设计方在 T702 的 G2 中核对。

🤖 Generated with Claude Code

delivery-harness 的自治试验(B81)中,T702 上线后 watch 只认 checks.toml [identity]
agent_login 写的评审标记。BackgroundTest 夹具里的两条评论原来没有 author,新引擎下 #3 会被
当成未评审,待评审集合会从 [1, 4] 变成 [1, 3, 4]。补上 author 后,新旧引擎下结果都是 [1, 4]:
旧引擎的 reviewed_heads 不看作者,结果不变。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added class:K5 needs-independent-review 待非设计方独立评审(试行) labels Oct 3, 2026
@github-actions
github-actions Bot requested a review from SnowsonZ October 3, 2026 06:00

@SnowsonZ SnowsonZ left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

独立评审 watch 夹具评论补可信作者(delivery-harness T702 前置 D5),harness 与 build 全绿,按用户决定批准合并。

@SnowsonZ
SnowsonZ merged commit e67c030 into main Oct 3, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants