From 896e3efaa9c7496043fdaeee4dceff3d0e46b318 Mon Sep 17 00:00:00 2001 From: gewuyou <95328647+GeWuYou@users.noreply.github.com> Date: Fri, 1 May 2026 15:37:26 +0800 Subject: [PATCH] =?UTF-8?q?docs(source-generators):=20=E6=94=B6=E5=8F=A3?= =?UTF-8?q?=20PR=20=E5=AE=A1=E6=9F=A5=E4=B8=AD=E7=9A=84=E6=96=87=E6=A1=A3?= =?UTF-8?q?=E5=85=A5=E5=8F=A3=E8=A1=A8=E8=BF=B0?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - 更新 source-generators 文档中的测试入口表述，使用语义化链接标签替代源码路径 - 校正 documentation-full-coverage-governance active ai-plan 的 PR 审查事实、验证结果与下一步恢复点 --- ...ation-full-coverage-governance-tracking.md | 28 ++++++++-------- ...entation-full-coverage-governance-trace.md | 32 +++++++++++-------- .../cqrs-handler-registry-generator.md | 7 ++-- .../schema-config-generator.md | 6 +++- 4 files changed, 43 insertions(+), 30 deletions(-) diff --git a/ai-plan/public/documentation-full-coverage-governance/todos/documentation-full-coverage-governance-tracking.md b/ai-plan/public/documentation-full-coverage-governance/todos/documentation-full-coverage-governance-tracking.md index efc051d2..220eba28 100644 --- a/ai-plan/public/documentation-full-coverage-governance/todos/documentation-full-coverage-governance-tracking.md +++ b/ai-plan/public/documentation-full-coverage-governance/todos/documentation-full-coverage-governance-tracking.md @@ -10,20 +10,20 @@ ## 当前恢复点 -- 恢复点编号：`DOCUMENTATION-FULL-COVERAGE-GOV-RP-054` +- 恢复点编号：`DOCUMENTATION-FULL-COVERAGE-GOV-RP-055` - 当前阶段：`Phase 5 - Governance Maintenance` - 当前焦点： - - 处理 PR `#308` latest-head review 中经本地复核仍成立的 `CodeRabbit` open threads，并清理 active `ai-plan` 对旧结论的漂移 + - 处理 PR `#308` 当前 latest-head review 与 outside-diff review 中经本地复核仍成立的 reader-facing 文档表述问题，并清理 active `ai-plan` 对旧结论的漂移 - 当前事实： - - `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，等待提交推送后再由远端重新计算 + - `2026-05-01` 重新抓取 `$gframework-pr-review` 后确认：PR `#308` 处于 `OPEN`，latest reviewed commit 已前进到 `4fdb1e7398e4f114757b5c988698af203dce24c8` + - 当前 `CodeRabbit` 仍有 `2` 条 latest-head open threads 和 `1` 条 outside-diff comment；本地复核后真正成立的是 `2` 条 reader-facing 文档问题，分别位于 `schema-config-generator.md` 与 `cqrs-handler-registry-generator.md` 的测试入口表述 + - `docs/zh-CN/source-generators/schema-config-generator.md` 已包含独立的“迁移与兼容性”章节，因此对应 major open thread 现阶段应视为 stale，等待提交推送后再由远端重新计算 - GitHub Test Reporter 汇总为 `2222 passed / 0 failed` - `Title check` 仍为 `Inconclusive`，属于 PR 元数据问题，不是仓库文件内可直接修复的阻塞项 - 本地对 review 指向文件的修复在提交推送前，不会改变远端 latest reviewed commit 与 open-thread 统计 - 当前风险： - - 如果 active tracking / trace 继续保留已失效的 review 判断，会让后续恢复点重复处理已经本地闭环的问题 - - 在变更推送前，PR 页面仍会继续展示旧的 open-thread 数量，容易把 stale 线程误判为新的本地缺陷 + - 如果 active tracking / trace 继续保留旧的 commit SHA 和 review 归因，会让后续恢复点重复处理已经本地闭环的问题 + - 在变更推送前，PR 页面仍会继续展示旧的 open-thread / outside-diff 数量，容易把 stale 线程误判为新的本地缺陷 ## 当前状态摘要 @@ -58,11 +58,13 @@ ## 最新验证 - `2026-05-01` `bash .agents/skills/gframework-doc-refresh/scripts/validate-all.sh docs/zh-CN/source-generators/schema-config-generator.md` - - 结果：通过；新增“迁移与兼容性”小节后，页面的 frontmatter、链接与代码块校验仍然通过。 + - 结果：通过；在把测试入口改成语义化链接后，页面的 frontmatter、链接与代码块校验仍然通过。 +- `2026-05-01` `bash .agents/skills/gframework-doc-refresh/scripts/validate-all.sh docs/zh-CN/source-generators/cqrs-handler-registry-generator.md` + - 结果：通过；测试源码路径改成语义化链接标签后，页面的 frontmatter、链接与代码块校验通过。 - `2026-05-01` `bun run build`（工作目录：`docs/`） - - 结果：通过；active `ai-plan` 瘦身与 schema 专题页更新后站点仍可构建，仅保留既有大 chunk warning。 + - 结果：通过；两篇 source-generators 页面链接改写与 active `ai-plan` 事实更新后站点仍可构建，仅保留既有大 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。 + - 结果：通过；PR `#308` 处于 `OPEN`，latest reviewed commit 为 `4fdb1e7398e4f114757b5c988698af203dce24c8`，`CodeRabbit` 当前有 `2` 条 latest-head open threads 和 `1` 条 outside-diff comment，测试汇总为 `2222 passed / 0 failed`，`Greptile` / `Gemini Code Assist` 当前无 open thread。 - `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` @@ -78,6 +80,6 @@ ## 下一步 -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 上回复说明，而不是继续修改已覆盖的本地文档。 +1. 提交并推送本轮 source-generators 文档 follow-up，然后重新抓取 `$gframework-pr-review`。 +2. 若 remote open threads 只剩 stale 的“迁移与兼容性”线程或 `Title check` 这类 metadata 项，则停止本轮仓库内修复。 +3. 若 review 线程清空或只剩 metadata 项，再按 `$gframework-batch-boot 50` 继续挑选新的 coverage 切片，避免在同一轮混入无关改稿。 diff --git a/ai-plan/public/documentation-full-coverage-governance/traces/documentation-full-coverage-governance-trace.md b/ai-plan/public/documentation-full-coverage-governance/traces/documentation-full-coverage-governance-trace.md index 7b89c15c..0b0dbdd7 100644 --- a/ai-plan/public/documentation-full-coverage-governance/traces/documentation-full-coverage-governance-trace.md +++ b/ai-plan/public/documentation-full-coverage-governance/traces/documentation-full-coverage-governance-trace.md @@ -2,26 +2,30 @@ ## 2026-05-01 -### 当前恢复点：RP-054 +### 当前恢复点：RP-055 -- 通过 `$gframework-pr-review` 重新抓取当前分支 PR `#308`，确认 latest-head review 当前只剩 `3` 条 `CodeRabbit` open threads，分别落在两份归档文档与 `Schema 配置生成器` 专题页。 -- 本地复核确认真正仍成立的是 `2` 条归档 Markdown 反引号问题；`schema-config-generator.md` 已包含独立的“迁移与兼容性”小节，因此该线程目前应视为 stale。 +- 通过 `$gframework-pr-review` 重新抓取当前分支 PR `#308`，确认 latest reviewed commit 已前进到 `4fdb1e7398e4f114757b5c988698af203dce24c8`，当前 `CodeRabbit` 仍有 `2` 条 latest-head open threads 和 `1` 条 outside-diff comment。 +- 本地复核确认真正仍成立的是 `2` 条 reader-facing 文档表述问题：`cqrs-handler-registry-generator.md` 与 `schema-config-generator.md` 都直接暴露了测试源码路径；`schema-config-generator.md` 的“迁移与兼容性”建议则已经本地满足，应视为 stale。 - GitHub Test Reporter 当前汇总为 `2222 passed / 0 failed`；`Title check` 仍然只是 PR 元数据问题，因此不纳入仓库文件修复范围。 -### 当前决策（RP-054） +### 当前决策（RP-055） -- 修复两份 `documentation-full-coverage-governance` 归档里的 Markdown 行内代码闭合错误，避免 reader-facing archive 文档继续触发 review。 -- active tracking 与 active trace 改写为与本地核验一致的事实，不再把已经落地的 schema 专题页补充误记为待办。 -- 本轮只做 latest-head review 精确收口，不扩展到新的 docs coverage 批次。 +- 把两处测试源码路径改成语义化 GitHub 链接标签，避免把原始文件路径暴露成 reader-facing 导航文本。 +- active tracking 与 active trace 改写为与本次 PR 抓取一致的事实，修正 latest reviewed commit、线程来源和本地结论。 +- 本轮只做 PR review 精确收口，不扩展到新的 docs coverage 批次。 -### 当前验证（RP-054） +### 当前验证（RP-055） - 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`。 + - 结果：通过；PR `#308` 处于 `OPEN`，latest reviewed commit 为 `4fdb1e7398e4f114757b5c988698af203dce24c8`，`CodeRabbit` 当前有 `2` 条 latest-head open threads 和 `1` 条 outside-diff comment，测试汇总为 `2222 passed / 0 failed`。 +- 页面校验： + - `bash .agents/skills/gframework-doc-refresh/scripts/validate-all.sh docs/zh-CN/source-generators/schema-config-generator.md` + - `bash .agents/skills/gframework-doc-refresh/scripts/validate-all.sh docs/zh-CN/source-generators/cqrs-handler-registry-generator.md` + - 结果：通过；两篇 source-generators 页面在改成语义化 GitHub 链接后，frontmatter、链接与代码块校验均通过。 - 站点构建： - `bun run build`（工作目录：`docs/`） - - 结果：通过；本轮归档 Markdown 修正与 active `ai-plan` 更新后站点仍可构建，仅保留既有大 chunk warning。 + - 结果：通过；两篇 source-generators 页面链接改写与 active `ai-plan` 事实更新后站点仍可构建，仅保留既有大 chunk warning。 ### 归档指针 @@ -34,8 +38,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-054） +### 下一步（RP-055） -1. 提交并推送本轮 follow-up 后重新抓取 `$gframework-pr-review`，确认 remote open threads 是否只剩 stale / metadata 项。 -2. 若只剩 `Title check`，则把后续动作限定为 GitHub PR 标题修正，不继续在仓库里做无关变更。 -3. 若远端仍保留 schema 页线程，则基于推送后的 latest reviewed commit 再判断是否需要在 PR 上补充回复说明。 +1. 提交并推送本轮 follow-up 后重新抓取 `$gframework-pr-review`，确认 remote open threads 是否只剩 stale 的 schema 线程或 `Title check`。 +2. 若只剩 metadata 项，则把后续动作限定为 GitHub PR 元数据修正，不继续在仓库里做无关变更。 +3. 若 review 线程清空，则回到 `documentation-full-coverage-governance` 的下一个 coverage 切片，而不是继续在同一轮修改无关页面。 diff --git a/docs/zh-CN/source-generators/cqrs-handler-registry-generator.md b/docs/zh-CN/source-generators/cqrs-handler-registry-generator.md index 3b973d5b..e0cd2853 100644 --- a/docs/zh-CN/source-generators/cqrs-handler-registry-generator.md +++ b/docs/zh-CN/source-generators/cqrs-handler-registry-generator.md @@ -91,8 +91,11 @@ RegisterCqrsHandlersFromAssemblies( 6. 若存在 `CqrsReflectionFallbackAttribute`，只补扫剩余 handler 7. 同一程序集按稳定键去重，避免重复注册 -这个行为由 `GFramework.Cqrs.Tests/Cqrs/CqrsHandlerRegistrarTests.cs` 和 -`GFramework.SourceGenerators.Tests/Cqrs/CqrsHandlerRegistryGeneratorTests.cs` 共同覆盖。 +这个行为由 +[运行时注册流程测试](https://github.com/GeWuYou/GFramework/blob/main/GFramework.Cqrs.Tests/Cqrs/CqrsHandlerRegistrarTests.cs) +和 +[生成器行为测试](https://github.com/GeWuYou/GFramework/blob/main/GFramework.SourceGenerators.Tests/Cqrs/CqrsHandlerRegistryGeneratorTests.cs) +共同覆盖。 ## 什么时候值得安装 diff --git a/docs/zh-CN/source-generators/schema-config-generator.md b/docs/zh-CN/source-generators/schema-config-generator.md index 767d2e69..c16cf066 100644 --- a/docs/zh-CN/source-generators/schema-config-generator.md +++ b/docs/zh-CN/source-generators/schema-config-generator.md @@ -199,7 +199,11 @@ var bootstrap = new GameConfigBootstrap( - 额外约束元数据不合法 - 例如 `GF_ConfigSchema_008` 到 `GF_ConfigSchema_014` -这些边界由 `GFramework.SourceGenerators.Tests/Config/SchemaConfigGeneratorTests.cs` 和快照测试共同覆盖。遇到生成失败时，优先先看诊断 ID，再回头核对 schema 本身是否超出当前公开子集。 +这些边界由 +[Schema 生成器行为测试](https://github.com/GeWuYou/GFramework/blob/main/GFramework.SourceGenerators.Tests/Config/SchemaConfigGeneratorTests.cs) +和 +[生成结果快照测试](https://github.com/GeWuYou/GFramework/blob/main/GFramework.SourceGenerators.Tests/Config/SchemaConfigGeneratorSnapshotTests.cs) +共同覆盖。遇到生成失败时，优先先看诊断 ID，再回头核对 schema 本身是否超出当前公开子集。 ## 什么时候优先看这页