Skip to content

docs: correct post-merge evidence records - #20

Merged
YanZhiwei merged 3 commits into
mainfrom
docs/post-merge-corrections
Oct 31, 2026
Merged

YanZhiwei merged 3 commits into
mainfrom
docs/post-merge-corrections

Conversation

@YanZhiwei

@YanZhiwei YanZhiwei commented Oct 31, 2026 •

Copy link
Copy Markdown
Owner

摘要

修正 PR #19 合并后复审发现的记录准确性问题:squash 合并的可达性表述、tree 等价声明、
“未 push/merge”残留陈述,以及被误删的 S3 fingerprint 定义。

变更类型

  • 文档或治理

当前设计(变更前)

  • delivery-v021-upgrade.md 写成材料提交 “became reachable in main”,与 squash 合并事实
    不符:内容进入了 main,提交本身仍不可达。
  • delivery-v022-upgrade.md 声称 “the merge preserved the recorded tree”,但
    tree(079ccc2)=c57792d 与 tree(353af49)=fe42f62 不同。
  • v0.2.2 权限段和 008 S3 limitations 仍无条件声明 “no push/merge/commit”,与同一记录的合并
    陈述矛盾。
  • 008 evidence 的 S3 Fingerprints 段在上一 PR 中被意外删除,而 task-verdicts.yml
    T018–T020 仍引用 769a3b85…。

变更内容

位置 变更 原因
harness/evidence/delivery-v021-upgrade.md 改为“内容随 PR #17 squash 提交进入 main,源提交不可达”;补充共享范围与分支专属 SHA 说明 恢复事实准确性
harness/evidence/delivery-v022-upgrade.md 改为“与 079ccc2 相比,合并树只差 harness/build-log.md 与本记录”;安装器段落限定为安装器行为 去掉不可验证的 tree 声明,消除自相矛盾
.specify/delivery/008-*/evidence.md 恢复 Fingerprints 段并注明绑定 3088ce8、不描述合并树;S3 limitations 改为“本地验证离线,但分支已推送、PR #18 已创建,远端 Quality 覆盖 3088ce8/b514a3c”;re-anchor 说明补充分支专属 SHA 不可达 保留可追溯的指纹定义,修正 S3 时间线
harness/build-log.md v0.2.2 状态改为已合并,Base/head 改为 9cdb10a → 353af49;v0.2.1 条目同步为“内容随 PR #17 合并”,并把“未提交/未推送”限定为当次升级验证 与证据和实际合并一致,消除条目自相矛盾

影响

维度 影响 证据
公共 API、命令、配置 无 仅 Markdown
持久化或迁移 无 —
兼容性与升级路径 无 —
安全与权限 无 —
性能与资源 无 —
依赖与供应链 无 —

设计

N/A:只修正记录文字和状态,不改变结构。

架构决策

  • 没有架构决策变化。

行为与验收

需求 结果 证据
不再声称 squash 使源提交可达 通过 v021 尾部说明
tree 声明可验证 通过 比较 079ccc2 与 353af49 的 diff 仅两个记录文件
无自相矛盾的“未合并”陈述 通过 v022 权限段、008 limitations
S3 指纹定义可追溯 通过 恢复的 Fingerprints 段 + scope note
build-log 与合并事实一致 通过 状态与 Base/head 行

验证

检查 结果 证据
构建 不适用 无源码变更
测试 不适用 无行为变更
治理检查 pass(0 error,1 warning) 既有 v0.2.2 新鲜度信号
文档检查 pass scripts/quality.ps1 -Check docs
空白与 diff pass git diff --check

文档

  • 源文档已更新。
  • 生成文档已同步。
  • 不需要文档变更。

回滚

revert 提交 14c9af6;只涉及 Markdown 记录。

AI 辅助

  • 未使用 AI 辅助。
  • 使用了 AI 辅助,且每一行变更都已审阅和验证。

评审说明

  • Lane:Lightweight(记录修正,无行为变化)。
  • 修正来源:PR docs: close out post-merge evidence #19 合并后独立复审的阻断项 R1、R3、R4,以及 follow-up R5 中的 build-log 与
    共享范围说明;第二轮复审指出 S3 时间线和 build-log 自相矛盾,已在第二个提交修正。
  • 文档检查仍需临时移出本地忽略的 specs/007-llm-protocol-abstraction,检查后已恢复。
  • 未执行 merge。

检查清单

  • 验收条件已覆盖。
  • 阻断性评审问题已解决。
  • ADR 与 HEAD 的实现一致。
  • 已附相关验证证据。
  • 外发 diff 与已审阅范围一致。

@YanZhiwei
YanZhiwei merged commit 8d804f6 into main Oct 31, 2026
14 checks passed
@YanZhiwei
YanZhiwei deleted the docs/post-merge-corrections branch October 31, 2026 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant