mirror of
https://github.com/GeWuYou/GFramework.git
synced 2026-05-06 16:16:44 +08:00
docs(source-generators): 收口最新 PR 审查跟进
- 更新 schema-config-generator 文案并同步 PR review 最新建议 - 修复 documentation-full-coverage-governance active tracking 的 RP-055 验证引用与审查事实 - 更新 active trace 的 latest reviewed commit、线程结论与验证摘要
This commit is contained in:
gewuyou
parent
00ecf6fb10
commit
6983b7ee84
@ -15,16 +15,17 @@
|
||||
- 当前焦点：
|
||||
- 处理 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 已前进到 `896e3efaa9c7496043fdaeee4dceff3d0e46b318`
|
||||
- 当前 `CodeRabbit` 仍有 `3` 条 latest-head open threads 和 `1` 条 outside-diff comment；`Greptile` / `Gemini Code Assist` 当前无 open thread
|
||||
- 本地复核后真正仍成立的 `4` 条问题已经全部在当前工作树完成修复：`cqrs-handler-registry-generator.md` 的 fallback 条件说明、`schema-config-generator.md` 的 `configRootPath` 示例、active trace 的显式风险段，以及 active tracking 的验证摘要压缩
|
||||
- `docs/zh-CN/source-generators/schema-config-generator.md` 已包含独立的“迁移与兼容性”章节，因此对应 major open thread 现阶段应视为 stale，等待提交推送后再由远端重新计算
|
||||
- `2026-05-01` 重新抓取 `$gframework-pr-review` 后确认：PR `#308` 处于 `OPEN`，latest reviewed commit 已前进到 `00ecf6fb1083e9039c9dc544a3265c38e1ba9117`
|
||||
- 当前 `CodeRabbit` 仍有 `5` 条 latest-head open threads；其中 `2` 条是本轮 latest review 针对 active tracking 与 `schema-config-generator.md` 措辞提出的新 actionable comments，另有 `3` 条为已被当前文件内容满足但尚未由远端重新计算关闭的 stale 线程
|
||||
- `Greptile` / `Gemini Code Assist` 当前无 open thread，CodeRabbit latest review 状态为 `CHANGES_REQUESTED`
|
||||
- 本地复核后，最新 review 中真正仍成立的 `2` 条问题分别是：active tracking 的 RP-055 验证引用仍指向 `RP-049` 到 `RP-052` 归档，以及 `schema-config-generator.md` 的“更稳妥地回退”措辞未同步
|
||||
- `docs/zh-CN/source-generators/schema-config-generator.md` 已包含独立的“迁移与兼容性”章节，且运行时示例已补齐 `configRootPath` 定义，因此对应的 major / minor open threads 现阶段应视为 stale，等待提交推送后再由远端重新计算
|
||||
- GitHub Test Reporter 汇总为 `2247 passed / 0 failed`
|
||||
- `Title check` 仍为 `Inconclusive`，属于 PR 元数据问题，不是仓库文件内可直接修复的阻塞项
|
||||
- 本地修复尚未推送前，不会改变远端 latest reviewed commit 与 open-thread 统计
|
||||
- 当前风险：
|
||||
- 如果 active tracking / trace 继续保留旧的 commit SHA 和 review 归因，会让后续恢复点重复处理已经本地闭环的问题
|
||||
- 在变更推送前，PR 页面仍会继续展示旧的 open-thread / outside-diff 数量，容易把 stale 线程误判为新的本地缺陷
|
||||
- 在变更推送前，PR 页面仍会继续展示 latest reviewed commit `00ecf6fb...` 下的 open-thread 数量，容易把 stale 线程误判为新的本地缺陷
|
||||
|
||||
## 当前状态摘要
|
||||
|
||||
@ -58,10 +59,9 @@
|
||||
|
||||
## 最新验证
|
||||
|
||||
- `2026-05-01` 本轮关键验证结论：`$gframework-pr-review` 重新抓取通过，确认 PR `#308` 的 latest reviewed commit 为 `896e3efaa9c7496043fdaeee4dceff3d0e46b318`，`CodeRabbit` 当前仍有 `3` 条 latest-head open threads 与 `1` 条 outside-diff comment；两篇 source-generators 页面校验通过，`docs/` 站点构建通过并仅保留既有大 chunk warning。
|
||||
- 逐条命令与详细结果见：
|
||||
- `ai-plan/public/documentation-full-coverage-governance/archive/todos/documentation-full-coverage-governance-validation-history-rp-049-to-rp-052-2026-05-01.md`
|
||||
- `ai-plan/public/documentation-full-coverage-governance/archive/traces/documentation-full-coverage-governance-trace-history-rp-049-to-rp-052-2026-05-01.md`
|
||||
- `2026-05-01` 本轮关键验证结论：`$gframework-pr-review` 重新抓取通过，确认 PR `#308` 的 latest reviewed commit 为 `00ecf6fb1083e9039c9dc544a3265c38e1ba9117`，`CodeRabbit` 当前仍有 `5` 条 latest-head open threads；其中最新 review 的 `2` 条 actionable comments 已在当前工作树完成修复，其余 `3` 条因当前文件内容已满足而应视为 stale。两篇 source-generators 页面校验通过，`docs/` 站点构建通过并仅保留既有大 chunk warning。
|
||||
- 本轮逐条命令与详细结果见 active trace 的 `RP-055` 验证段：
|
||||
- `ai-plan/public/documentation-full-coverage-governance/traces/documentation-full-coverage-governance-trace.md`
|
||||
|
||||
## 下一步
|
||||
|
||||
|
||||
@ -4,27 +4,29 @@
|
||||
|
||||
### 当前恢复点：RP-055
|
||||
|
||||
- 通过 `$gframework-pr-review` 重新抓取当前分支 PR `#308`，确认 latest reviewed commit 已前进到 `896e3efaa9c7496043fdaeee4dceff3d0e46b318`，当前 `CodeRabbit` 仍有 `3` 条 latest-head open threads 和 `1` 条 outside-diff comment。
|
||||
- 本地复核确认真正仍成立的 `4` 条问题已经全部在当前工作树完成修复：`cqrs-handler-registry-generator.md` 的 fallback 条件说明、`schema-config-generator.md` 的 `configRootPath` 示例、active trace 的显式风险段，以及 active tracking 的“最新验证”压缩；`schema-config-generator.md` 的“迁移与兼容性”建议则已经本地满足，应视为 stale。
|
||||
- 通过 `$gframework-pr-review` 重新抓取当前分支 PR `#308`，确认 latest reviewed commit 已前进到 `00ecf6fb1083e9039c9dc544a3265c38e1ba9117`，当前 `CodeRabbit` 仍有 `5` 条 latest-head open threads。
|
||||
- 本地复核确认最新 review 中真正仍成立的 `2` 条问题已经在当前工作树完成修复：active tracking 的 RP-055 验证引用改为指向 active trace，以及 `schema-config-generator.md` 的“更稳妥地回退”措辞同步。
|
||||
- `schema-config-generator.md` 的“迁移与兼容性”章节与自包含运行时示例已在当前文件内容中满足 earlier open threads，对应 `3` 条历史线程现阶段应视为 stale，等待提交推送后由远端重新计算。
|
||||
- GitHub Test Reporter 当前汇总为 `2247 passed / 0 failed`；`Title check` 仍然只是 PR 元数据问题，因此不纳入仓库文件修复范围。
|
||||
|
||||
### 当前决策（RP-055）
|
||||
|
||||
- 补齐 `cqrs-handler-registry-generator.md` 的 fallback 条件说明，以及 `schema-config-generator.md` 的自包含运行时示例。
|
||||
- active tracking 与 active trace 改写为与本次 PR 抓取一致的事实，修正 latest reviewed commit、线程数量、本地结论与验证摘要。
|
||||
- 只修仍然成立的 latest review 问题，不重复处理已被当前内容满足但尚未关闭的 stale 线程。
|
||||
- 本轮只做 PR review 精确收口，不扩展到新的 docs coverage 批次。
|
||||
|
||||
### 当前风险（RP-055）
|
||||
|
||||
- 在本轮提交推送前，PR 页面仍会继续展示基于旧 diff 的 open threads，因此远端线程数量短时间内不会反映本地已修复状态。
|
||||
- `schema-config-generator.md` 的“迁移与兼容性”线程已经由当前文件内容满足，但仍需依赖远端下一次 review 重新计算后才能从 open thread 列表中消失。
|
||||
- 在本轮提交推送前，PR 页面仍会继续展示 latest reviewed commit `00ecf6fb...` 下的 open threads，因此远端线程数量短时间内不会反映本地已修复状态。
|
||||
- `schema-config-generator.md` 的“迁移与兼容性”线程和 `configRootPath` 线程已经由当前文件内容满足，但仍需依赖远端下一次 review 重新计算后才能从 open thread 列表中消失。
|
||||
- `Title check` 仍是 PR 元数据层面的 `Inconclusive` 项；即使仓库文件全部修完，也不能仅凭本地改动让该检查转绿。
|
||||
|
||||
### 当前验证（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 reviewed commit 为 `896e3efaa9c7496043fdaeee4dceff3d0e46b318`，`CodeRabbit` 当前有 `3` 条 latest-head open threads 和 `1` 条 outside-diff comment，测试汇总为 `2247 passed / 0 failed`。
|
||||
- 结果：通过；PR `#308` 处于 `OPEN`，latest reviewed commit 为 `00ecf6fb1083e9039c9dc544a3265c38e1ba9117`，`CodeRabbit` 当前有 `5` 条 latest-head open threads，其中最新 review 的 `2` 条 actionable comments 已本地修复，其余 `3` 条应视为 stale；测试汇总为 `2247 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`
|
||||
|
||||
@ -182,7 +182,7 @@ var bootstrap = new GameConfigBootstrap(
|
||||
- `x-gframework-index`、路径元数据和标识符映射都要求满足安全约束；旧项目里依赖宽松命名或不安全路径的配置，需要先整理 schema，再切换到聚合注册入口。
|
||||
- 迁移到生成链路后，运行时读取模型保持不变，但“表类型定义”和“注册目录”会改为编译期输出，因此自定义扩展更适合挂在单表绑定或 `ConfigureLoader` 阶段，而不是继续复制生成器会维护的样板代码。
|
||||
|
||||
如果你需要分批迁移，当前更稳妥的回退方式是只回退注册入口，而不是同时回退 schema 结构：
|
||||
如果你需要分批迁移，当前更稳妥地回退是只回退注册入口，而不是同时回退 schema 结构：
|
||||
|
||||
- 可以先让一部分表继续沿手写注册保留，另一部分切到生成的单表绑定或聚合注册。
|
||||
- 如果某张表在迁移后触发诊断，优先根据诊断 ID 收敛 schema，再决定是否临时回到手写注册。
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user