mirror of
https://github.com/GeWuYou/GFramework.git
synced 2026-05-08 01:24:31 +08:00
docs(ai-plan): 收口 PR 审查遗留文档问题
- 修复 documentation-full-coverage-governance 归档中的 Markdown 行内代码闭合错误 - 更新 active tracking 与 trace,使其反映 PR #308 review 的本地核验结论 - 补充本轮文档构建验证结果与后续恢复步骤
This commit is contained in:
parent
241c9ffeb3
commit
4fdb1e7398
@ -9,14 +9,14 @@
|
||||
## 2026-04-29 / RP-049
|
||||
|
||||
- 重新进入时确认当前分支仍为 `docs/sdk-update-documentation`,但上游恢复路径不再依赖旧 PR 线程,而是改用 `origin/main` 作为 batch stop-condition baseline。
|
||||
- 本轮工作树在 reader-facing 文案收口后相对 `origin/main` 来到 `13` files / `132` lines`,仍明显低于 `$gframework-batch-boot 50` 的阈值。
|
||||
- 本轮工作树在 reader-facing 文案收口后相对 `origin/main` 来到 `13` files / `132` lines,仍明显低于 `$gframework-batch-boot 50` 的阈值。
|
||||
- 主线程接受的变更范围限定在 `docs/zh-CN/godot/*`、`game/*` 和少量 README 标签,不扩展到导航重写或大型结构改稿。
|
||||
|
||||
## 2026-04-29 / RP-050
|
||||
|
||||
- 在 `RP-049` 的基础上继续做第 2 批低风险 reader-facing 收口,触达 `game/data.md`、`game/storage.md`、`godot/ui.md` 与 `GFramework.Cqrs.Abstractions/README.md`、`GFramework.SourceGenerators.Common/README.md`。
|
||||
- 决策上只接受“改句子即可闭环”的问题,不把 README 子系统地图或结构级重写混入同一轮。
|
||||
- 当轮工作树相对 `origin/main` 为 `18` files / `225` lines`,仍保留充足余量。
|
||||
- 当轮工作树相对 `origin/main` 为 `18` files / `225` lines,仍保留充足余量。
|
||||
|
||||
## 2026-04-29 / RP-051
|
||||
|
||||
@ -27,5 +27,5 @@
|
||||
## 2026-04-30 / RP-052
|
||||
|
||||
- 该批次提交为 `f88f96c3`(`docs(source-generators): 补充生成器专题覆盖并更新进度`)。
|
||||
- 提交后重新计算确认 committed branch diff vs `origin/main` 已回落到 `8` files / `337` lines`,说明提交前的 `39` files / `2555` lines` 只是临时工作树峰值,不应继续作为默认恢复指标。
|
||||
- 提交后重新计算确认 committed branch diff vs `origin/main` 已回落到 `8` files / `337` lines,说明提交前的 `39` files / `2555` lines 只是临时工作树峰值,不应继续作为默认恢复指标。
|
||||
- active topic 后续仍可以继续按 `$gframework-batch-boot 50` 推进,但应优先挑“已有 package README、但站内专题仍不足”的覆盖切片。
|
||||
|
||||
@ -26,6 +26,6 @@
|
||||
|
||||
## 2026-04-30 / RP-052
|
||||
|
||||
- 这一轮 coverage 扩展提交为 `f88f96c3`,提交后 branch diff 相对 `origin/main` 回落到 `8` files / `337` lines`,重新释放了 stop-condition 余量。
|
||||
- 这一轮 coverage 扩展提交为 `f88f96c3`,提交后 branch diff 相对 `origin/main` 回落到 `8` files / `337` lines,重新释放了 stop-condition 余量。
|
||||
- active topic 后续仍可继续扩批,但更适合选择“已有 package README、但站内 docs 仍缺 reader-facing 专题”的切片,而不是继续给共享支撑层单开页面。
|
||||
- 该状态也为后续用 `$gframework-pr-review` 精确跟进最新 review 线程提供了更清晰的恢复入口。
|
||||
|
||||
@ -10,19 +10,20 @@
|
||||
|
||||
## 当前恢复点
|
||||
|
||||
- 恢复点编号:`DOCUMENTATION-FULL-COVERAGE-GOV-RP-053`
|
||||
- 恢复点编号:`DOCUMENTATION-FULL-COVERAGE-GOV-RP-054`
|
||||
- 当前阶段:`Phase 5 - Governance Maintenance`
|
||||
- 当前焦点:
|
||||
- 处理 PR `#308` latest-head review 中仍成立的 3 条 `CodeRabbit` open threads:继续瘦身 active `ai-plan` 文档,并为 `Schema 配置生成器` 专题补集中式迁移与兼容性说明
|
||||
- 处理 PR `#308` latest-head review 中经本地复核仍成立的 `CodeRabbit` open threads,并清理 active `ai-plan` 对旧结论的漂移
|
||||
- 当前事实:
|
||||
- `2026-05-01` 重新抓取 `$gframework-pr-review` 后确认:PR `#308` 处于 `OPEN`,latest reviewed commit 为 `097e97bcd66c89c79b4dafc30e24e8b650c7db63`
|
||||
- 当前 latest-head review 只剩 `CodeRabbit` `3` 条 open threads,分别指向本 tracking、active trace 和 `docs/zh-CN/source-generators/schema-config-generator.md`
|
||||
- `2026-05-01` 重新抓取 `$gframework-pr-review` 后确认:PR `#308` 处于 `OPEN`,latest reviewed commit 为 `241c9ffeb3dbd11872b23ecf4a80971feeca475b`
|
||||
- 当前 latest-head review 只剩 `CodeRabbit` `3` 条 open threads,但本地复核后只有 `2` 条仍然成立,均指向 `documentation-full-coverage-governance` 的归档 Markdown 反引号问题
|
||||
- `docs/zh-CN/source-generators/schema-config-generator.md` 已包含独立的“迁移与兼容性”章节,因此对应 open thread 现阶段应视为 stale,等待提交推送后再由远端重新计算
|
||||
- GitHub Test Reporter 汇总为 `2222 passed / 0 failed`
|
||||
- `Title check` 仍为 `Inconclusive`,属于 PR 元数据问题,不是仓库文件内可直接修复的阻塞项
|
||||
- 本地已完成 review 指向文件的收口,但在变更尚未提交推送前,重新抓取的 PR review 仍会继续显示旧的 latest reviewed commit 与同一批 open threads
|
||||
- 本地对 review 指向文件的修复在提交推送前,不会改变远端 latest reviewed commit 与 open-thread 统计
|
||||
- 当前风险:
|
||||
- active tracking / trace 若继续保留阶段性细节与逐项验证,会再次偏离“快速恢复入口”的用途
|
||||
- `Schema 配置生成器` 页已经说明边界,但迁移步骤、兼容边界和回退方式仍分散在多个段落中
|
||||
- 如果 active tracking / trace 继续保留已失效的 review 判断,会让后续恢复点重复处理已经本地闭环的问题
|
||||
- 在变更推送前,PR 页面仍会继续展示旧的 open-thread 数量,容易把 stale 线程误判为新的本地缺陷
|
||||
|
||||
## 当前状态摘要
|
||||
|
||||
@ -62,8 +63,6 @@
|
||||
- 结果:通过;active `ai-plan` 瘦身与 schema 专题页更新后站点仍可构建,仅保留既有大 chunk warning。
|
||||
- `2026-05-01` `python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --json-output /tmp/current-pr-review.json`
|
||||
- 结果:通过;PR `#308` 处于 `OPEN`,latest-head review 当前只剩 `3` 条 `CodeRabbit` open threads,测试汇总为 `2222 passed / 0 failed`,`Greptile` / `Gemini Code Assist` 当前无 open thread。
|
||||
- `2026-05-01` `python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --json-output /tmp/current-pr-review-after-fix.json`
|
||||
- 结果:通过;remote latest reviewed commit 仍是 `097e97bcd66c89c79b4dafc30e24e8b650c7db63`,因此在本地改动尚未提交推送前,PR 页面仍显示同一批 `3` 条 open threads。
|
||||
- `2026-04-30` `bash .agents/skills/gframework-doc-refresh/scripts/validate-all.sh docs/zh-CN/source-generators/schema-config-generator.md`
|
||||
- 结果:通过;`Schema 配置生成器` 专题页的 frontmatter、链接与代码块校验通过。
|
||||
- `2026-04-30` `bash .agents/skills/gframework-doc-refresh/scripts/validate-all.sh docs/zh-CN/source-generators/index.md`
|
||||
@ -79,6 +78,6 @@
|
||||
|
||||
## 下一步
|
||||
|
||||
1. 完成本轮 3 个 latest-head review follow-up 后,重新运行最小文档验证,并重新抓取 `$gframework-pr-review`,确认 remote open threads 是否清空。
|
||||
2. 若 review 线程清空,再继续按 `$gframework-batch-boot 50` 挑选“已有 package README、但站内专题仍不足”的 coverage 切片,而不是继续扩写共享支撑层。
|
||||
3. 若 `Title check` 仍保留,则单独修改 GitHub PR 标题,不把它和仓库文件内容混为同一类修复项。
|
||||
1. 提交并推送本轮归档修正与 active `ai-plan` 事实校正,然后重新抓取 `$gframework-pr-review`,确认 remote open threads 是否只剩 stale / metadata 项。
|
||||
2. 若 review 线程清空或只剩 `Title check`,再按 `$gframework-batch-boot 50` 继续挑选新的 coverage 切片,避免在同一轮混入无关改稿。
|
||||
3. 若远端仍保留 schema 页线程,则基于推送后的 latest reviewed commit 再判断是否需要单独在 PR 上回复说明,而不是继续修改已覆盖的本地文档。
|
||||
|
||||
@ -2,32 +2,26 @@
|
||||
|
||||
## 2026-05-01
|
||||
|
||||
### 当前恢复点:RP-053
|
||||
### 当前恢复点:RP-054
|
||||
|
||||
- 通过 `$gframework-pr-review` 重新抓取当前分支 PR `#308`,确认 latest-head review 当前只剩 `3` 条 `CodeRabbit` open threads,且都落在 active tracking、active trace 与 `Schema 配置生成器` 专题页。
|
||||
- 本地复核确认这 3 条评论都仍成立:active `ai-plan` 文档累积了过多阶段性细节,而 `schema-config-generator.md` 缺少集中式的迁移与兼容性小节。
|
||||
- 通过 `$gframework-pr-review` 重新抓取当前分支 PR `#308`,确认 latest-head review 当前只剩 `3` 条 `CodeRabbit` open threads,分别落在两份归档文档与 `Schema 配置生成器` 专题页。
|
||||
- 本地复核确认真正仍成立的是 `2` 条归档 Markdown 反引号问题;`schema-config-generator.md` 已包含独立的“迁移与兼容性”小节,因此该线程目前应视为 stale。
|
||||
- GitHub Test Reporter 当前汇总为 `2222 passed / 0 failed`;`Title check` 仍然只是 PR 元数据问题,因此不纳入仓库文件修复范围。
|
||||
|
||||
### 当前决策(RP-053)
|
||||
### 当前决策(RP-054)
|
||||
|
||||
- active tracking 与 active trace 只保留当前恢复点、关键事实、风险、验证结论与下一步;`RP-049` 到 `RP-052` 的阶段细节迁入新的 archive 文件。
|
||||
- `Schema 配置生成器` 页新增独立的“迁移与兼容性”小节,集中说明从手写注册迁移的最小步骤、当前兼容边界与回退做法。
|
||||
- 修复两份 `documentation-full-coverage-governance` 归档里的 Markdown 行内代码闭合错误,避免 reader-facing archive 文档继续触发 review。
|
||||
- active tracking 与 active trace 改写为与本地核验一致的事实,不再把已经落地的 schema 专题页补充误记为待办。
|
||||
- 本轮只做 latest-head review 精确收口,不扩展到新的 docs coverage 批次。
|
||||
|
||||
### 当前验证(RP-053)
|
||||
### 当前验证(RP-054)
|
||||
|
||||
- PR review 抓取:
|
||||
- `python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --json-output /tmp/current-pr-review.json`
|
||||
- 结果:通过;PR `#308` 处于 `OPEN`,latest-head review 当前只剩 `3` 条 `CodeRabbit` open threads,测试汇总为 `2222 passed / 0 failed`。
|
||||
- 页面校验:
|
||||
- `bash .agents/skills/gframework-doc-refresh/scripts/validate-all.sh docs/zh-CN/source-generators/schema-config-generator.md`
|
||||
- 结果:通过;新增“迁移与兼容性”小节后页面校验通过。
|
||||
- 站点构建:
|
||||
- `bun run build`(工作目录:`docs/`)
|
||||
- 结果:通过;active `ai-plan` 瘦身与 schema 专题页更新后站点仍可构建,仅保留既有大 chunk warning。
|
||||
- 复核抓取:
|
||||
- `python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --json-output /tmp/current-pr-review-after-fix.json`
|
||||
- 结果:通过;由于 remote latest reviewed commit 仍未变化,本地未提交推送时 PR 页面仍显示同一批 `3` 条 open threads。
|
||||
- 结果:通过;本轮归档 Markdown 修正与 active `ai-plan` 更新后站点仍可构建,仅保留既有大 chunk warning。
|
||||
|
||||
### 归档指针
|
||||
|
||||
@ -40,8 +34,8 @@
|
||||
- `RP-049` 到 `RP-052` 的验证明细:
|
||||
`ai-plan/public/documentation-full-coverage-governance/archive/todos/documentation-full-coverage-governance-validation-history-rp-049-to-rp-052-2026-05-01.md`
|
||||
|
||||
### 下一步(RP-053)
|
||||
### 下一步(RP-054)
|
||||
|
||||
1. 提交本轮 active `ai-plan` 瘦身与 `Schema 配置生成器` 迁移说明补充。
|
||||
2. 运行最小文档验证并重新抓取 `$gframework-pr-review`,确认 latest-head review 是否已清空。
|
||||
3. 若只剩 `Title check`,则把后续动作限定为 GitHub PR 标题修正,不继续在仓库里做无关变更。
|
||||
1. 提交并推送本轮 follow-up 后重新抓取 `$gframework-pr-review`,确认 remote open threads 是否只剩 stale / metadata 项。
|
||||
2. 若只剩 `Title check`,则把后续动作限定为 GitHub PR 标题修正,不继续在仓库里做无关变更。
|
||||
3. 若远端仍保留 schema 页线程,则基于推送后的 latest reviewed commit 再判断是否需要在 PR 上补充回复说明。
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user