mirror of
https://github.com/GeWuYou/GFramework.git
synced 2026-05-08 09:34:30 +08:00
fix(ai-plan): 修复PR评审涉及的归档跟踪文档问题
- 修复 analyzer warning reduction 归档 todo 中指向 RP-001 的相对链接 - 清理 rp002-rp041 trace 中误混入的 RP-001 历史段落 - 更新 active tracking 与 trace 的恢复点描述和验证结论
This commit is contained in:
gewuyou
parent
833a95f7f3
commit
136b139312
@ -303,8 +303,8 @@
|
||||
|
||||
## 活跃文档
|
||||
|
||||
- 历史跟踪归档：[analyzer-warning-reduction-history-rp001.md](../archive/todos/analyzer-warning-reduction-history-rp001.md)
|
||||
- 历史 trace 归档：[analyzer-warning-reduction-history-rp001.md](../archive/traces/analyzer-warning-reduction-history-rp001.md)
|
||||
- 历史跟踪归档：[analyzer-warning-reduction-history-rp001.md](analyzer-warning-reduction-history-rp001.md)
|
||||
- 历史 trace 归档：[analyzer-warning-reduction-history-rp001.md](../traces/analyzer-warning-reduction-history-rp001.md)
|
||||
|
||||
## 验证说明
|
||||
|
||||
|
||||
@ -1108,36 +1108,6 @@
|
||||
folder 并导致 `ResolvePackageAssets` 失败
|
||||
- sandbox 内运行 `dotnet` 会因 MSBuild named-pipe 限制失败；需要在提权上下文中执行 .NET 验证
|
||||
|
||||
## 2026-04-19
|
||||
|
||||
### 阶段：local-plan 迁移收口（RP-001）
|
||||
|
||||
- 复核当前工作树后确认：`local-plan/` 仅保存 analyzer warning reduction 主题的 durable recovery state，不应继续作为
|
||||
worktree-root 遗留目录存在
|
||||
- 按 `ai-plan` 治理规则建立 `ai-plan/public/analyzer-warning-reduction/` 主题目录，并补齐：
|
||||
- `todos/`
|
||||
- `traces/`
|
||||
- `archive/todos/`
|
||||
- `archive/traces/`
|
||||
- 将旧 `local-plan` 中的详细 tracking / trace 迁入主题内历史归档，保留 `RP-001` 的完整实现与验证上下文
|
||||
- 新建精简版 active tracking / trace 入口，并在 `ai-plan/public/README.md` 中建立
|
||||
`fix/analyzer-warning-reduction-batch` -> `analyzer-warning-reduction` 的 worktree 映射
|
||||
- 删除旧 `local-plan` 文件，避免 `boot` 或后续协作者继续从过时目录恢复
|
||||
- 验证通过：
|
||||
- `find ai-plan/public/analyzer-warning-reduction -maxdepth 3 -type f | sort`
|
||||
- `dotnet build GFramework.Core.Abstractions/GFramework.Core.Abstractions.csproj -c Release -p:RestoreFallbackFolders=`
|
||||
|
||||
### Archive Context
|
||||
|
||||
- 历史跟踪归档：
|
||||
- `ai-plan/public/analyzer-warning-reduction/archive/todos/analyzer-warning-reduction-history-rp001.md`
|
||||
- 历史 trace 归档：
|
||||
- `ai-plan/public/analyzer-warning-reduction/archive/traces/analyzer-warning-reduction-history-rp001.md`
|
||||
|
||||
### 下一步
|
||||
|
||||
1. 若继续 analyzer warning reduction，优先回到 `GFramework.Core` 剩余 `MA0051` 热点，并继续保持“单 warning family、单切入点”的节奏
|
||||
2. 后续所有 WSL 下的 .NET 定向验证命令继续显式附带 `-p:RestoreFallbackFolders=`，避免把环境问题误判成代码回归
|
||||
# 2026-04-23
|
||||
|
||||
- RP-034 / PR #273 review follow-up：
|
||||
|
||||
@ -10,22 +10,23 @@
|
||||
- 当前阶段：`Phase 42`
|
||||
- 当前焦点：
|
||||
- 已于 `2026-04-24` 使用 `gframework-pr-review` 复核当前分支 PR #280，latest-head review 仍有 `3` 条 open threads
|
||||
- 本地确认仍成立的项只有两类：`SchemaConfigGeneratorTests` 的冗余 `global::` 返回类型，以及 active tracking/trace 需要归档压缩
|
||||
- `Greptile` 指出的返回类型冗余不影响行为；本轮按最小改动收口，避免扩大测试写集
|
||||
- 本地确认这 `3` 条 open threads 均指向 `ai-plan` 文档：错误归档链接、`rp002-rp041` trace 混入 `RP-001` 段落，以及 active trace 的恢复信息失真
|
||||
- 本轮按最小写集直接修正文档恢复入口，不再扩大 `GFramework.SourceGenerators.Tests` 的代码写集
|
||||
- `RP-041` 验证完成时，分支相对 `origin/main` 的唯一变更文件数为 `4`；这说明继续只处理同一热点文件时，该指标增长会很慢
|
||||
- `GFramework.SourceGenerators.Tests` 在 `RP-042` 的 `net10.0` Release build 中仍为 `10` 条 `MA0051` warning、`0` error；剩余热点继续集中在 `CqrsHandlerRegistryGeneratorTests.cs`
|
||||
|
||||
## 当前状态摘要
|
||||
|
||||
- 已将旧 active tracking / trace 的详细阶段历史归档到主题内 `archive/`，避免 `boot` 默认入口继续承载 `RP-002` 到 `RP-041` 的长历史
|
||||
- 当前 active 文档仅保留恢复点、活跃事实、风险、验证结论与下一步建议
|
||||
- 已修正 `archive/todos/analyzer-warning-reduction-history-rp002-rp041.md` 中指向 `RP-001` 归档的相对链接，恢复历史入口可点击性
|
||||
- 已从 `archive/traces/analyzer-warning-reduction-history-rp002-rp041.md` 中移除误混入的 `RP-001` 段落，确保文件名与内容范围一致
|
||||
- 已刷新 active tracking / trace 的恢复点描述，使其反映当前仍待远端收敛的是文档类 review threads，而不是已处理过的代码项
|
||||
- PR #280 的 MegaLinter 仍显示 `dotnet-format` warning，但测试报告为 `2156 passed / 0 failed`；该 warning 目前更像 CI 环境 restore / SDK 噪音，而不是本地代码行为回归
|
||||
|
||||
## 当前活跃事实
|
||||
|
||||
- 当前主题仍保持 active，因为 `GFramework.SourceGenerators.Tests` 尚有剩余 `MA0051` warning 需要决定是否继续推进
|
||||
- 继续按“单文件单方法”节奏处理 `CqrsHandlerRegistryGeneratorTests.cs` 可以稳定消除 warning，但不利于快速提高唯一变更文件数
|
||||
- 当前 PR review 已没有新的 failed-test 信号；后续优先级应回到本地仍成立的 review thread 和剩余 warning 热点
|
||||
- 当前 PR review 已没有新的 failed-test 信号；当前优先级是提交这轮 `ai-plan` 修正并等待远端 PR threads 收敛
|
||||
|
||||
## 当前风险
|
||||
|
||||
|
||||
@ -2,27 +2,24 @@
|
||||
|
||||
## 2026-04-24 — RP-042
|
||||
|
||||
### 阶段：PR #280 review follow-up 与 active 文档归档压缩
|
||||
### 阶段：PR #280 review follow-up 与 ai-plan 恢复入口修正
|
||||
|
||||
- 启动复核：
|
||||
- 使用 `gframework-pr-review` 抓取当前分支 PR #280 的 latest-head review threads、MegaLinter 摘要与测试报告
|
||||
- 本地核对后确认 `3` 条 open threads 中仍成立的是 `SchemaConfigGeneratorTests` 的冗余 `global::` 返回类型，以及 active tracking 入口过长的问题
|
||||
- 本地核对后确认 `3` 条 open threads 均仍成立，但全部集中在 `ai-plan` 文档恢复入口，而不是新的代码行为问题
|
||||
- 决策：
|
||||
- 对测试代码仅做最小行为无关修正，避免扩大 `GFramework.SourceGenerators.Tests` 写集
|
||||
- 将旧 active tracking / trace 的 `RP-002` 到 `RP-041` 详细历史整体迁入 `archive/`，重新建立精简版恢复入口
|
||||
- 不再继续扩大 `GFramework.SourceGenerators.Tests` 的写集，先把远端 latest-head review 中仍成立的文档问题全部收口
|
||||
- 保持 `RP-042` 作为 active recovery point，仅刷新其事实描述、归档链接和 trace 范围边界
|
||||
- 实施调整：
|
||||
- 将 `RunAndCollectGeneratedSources(...)` 的返回类型从 `global::System.Collections.Generic.IReadOnlyDictionary<string, string>` 收口为 `IReadOnlyDictionary<string, string>`
|
||||
- 归档旧 tracking 到 `archive/todos/analyzer-warning-reduction-history-rp002-rp041.md`
|
||||
- 归档旧 trace 到 `archive/traces/analyzer-warning-reduction-history-rp002-rp041.md`
|
||||
- 重建 active tracking / trace，只保留当前恢复点、活跃事实、风险、验证与下一步
|
||||
- 修正 `archive/todos/analyzer-warning-reduction-history-rp002-rp041.md` 中两条指向 `RP-001` 归档的相对链接
|
||||
- 从 `archive/traces/analyzer-warning-reduction-history-rp002-rp041.md` 中移除误混入的 `RP-001` 段落，使文件只保留 `RP-002` 到 `RP-041`
|
||||
- 刷新 active tracking / trace 的恢复点描述，明确当前 open threads 已收敛为文档问题，并记录本轮 follow-up 的事实与下一步
|
||||
- 验证结果：
|
||||
- `DOTNET_CLI_HOME=/tmp/dotnet-home dotnet restore GFramework.SourceGenerators.Tests/GFramework.SourceGenerators.Tests.csproj -p:RestoreFallbackFolders=""`
|
||||
- 结果：通过；测试项目资产已摆脱失效的 Windows fallback package folder
|
||||
- `DOTNET_CLI_HOME=/tmp/dotnet-home dotnet build GFramework.SourceGenerators.Tests/GFramework.SourceGenerators.Tests.csproj -c Release -t:Rebuild --no-restore --disable-build-servers -m:1 -p:UseSharedCompilation=false -p:RestoreFallbackFolders="" -nologo -clp:"Summary;WarningsOnly"`
|
||||
- 结果：`10 Warning(s)`，`0 Error(s)`；warning 继续全部落在 `CqrsHandlerRegistryGeneratorTests.cs` 的既有 `MA0051` 热点
|
||||
- 结果：`10 Warning(s)`，`0 Error(s)`；warning 仍全部来自 `CqrsHandlerRegistryGeneratorTests.cs` 的既有 `MA0051` 基线
|
||||
- 当前结论：
|
||||
- PR #280 当前没有 failed-test 回归信号；本轮主要是收口 latest-head review thread 中仍成立的低风险项
|
||||
- active 恢复入口已回到可读规模，后续 `boot` 不必再扫描完整阶段历史
|
||||
- PR #280 当前没有 failed-test 回归信号；latest-head review 剩余项可以全部在 `ai-plan` 范围内处理
|
||||
- active 恢复入口与历史归档范围已重新对齐，后续 `boot` 不会再从 `rp002-rp041` 误读 `RP-001`
|
||||
- 下一步建议：
|
||||
- 提交后重新抓取 PR #280 review，确认 open threads 是否收敛
|
||||
- 若 threads 收敛，则回到 `CqrsHandlerRegistryGeneratorTests.cs` 剩余 `MA0051`，或根据目标改切新的 warning 写集
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user