mirror of
https://github.com/GeWuYou/GFramework.git
synced 2026-05-10 19:27:14 +08:00
docs(ai-plan): 更新 analyzer-warning-reduction 恢复点
- 更新 analyzer-warning-reduction 的 RP-060 跟踪与 trace - 记录并行 warning-reduction 批次的验证结果与当前分支体积 - 补充下一轮继续朝 75 文件阈值推进的恢复建议
This commit is contained in:
parent
b27bcb5832
commit
425c22d98f
@ -6,14 +6,14 @@
|
||||
|
||||
## 当前恢复点
|
||||
|
||||
- 恢复点编号:`ANALYZER-WARNING-REDUCTION-RP-059`
|
||||
- 当前阶段:`Phase 59`
|
||||
- 恢复点编号:`ANALYZER-WARNING-REDUCTION-RP-060`
|
||||
- 当前阶段:`Phase 60`
|
||||
- 当前焦点:
|
||||
- `2026-04-25` 按 `$gframework-batch-boot 75` 继续 warning reduction,基线使用本地现有 `origin/main`
|
||||
- 当前 `HEAD` 与 `origin/main` 同为 `9964962`,已提交 branch diff 为 `0` 个文件;本轮工作树投影为 `1` 个文件、`92` 行
|
||||
- 已将 `GFramework.Game.Tests/Config/YamlConfigLoaderTests.cs` 中 `44` 处 `Assert.ThrowsAsync(... async () => await ...)` 改为直接返回 `Task`,并为 `WaitForTaskWithinAsync` 补齐 `ConfigureAwait(false)`
|
||||
- `dotnet build GFramework.Game.Tests/GFramework.Game.Tests.csproj -c Release --no-incremental` 通过,结果从 `249 Warning(s)` 降到 `203 Warning(s)`;该文件不再出现在 `MA0004` 输出中
|
||||
- `dotnet test GFramework.Game.Tests/GFramework.Game.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~YamlConfigLoaderTests"` 已通过,结果为 `Passed: 74`
|
||||
- `2026-04-25` 继续按 `$gframework-batch-boot 75` 自动推进,并明确允许使用 subagent 处理互不重叠的写集
|
||||
- 当前 `HEAD` 为 `b27bcb5`,基线 `origin/main` 仍为 `9964962`
|
||||
- 当前累计 branch diff 相对 `origin/main` 为 `9` 个文件、`480` 行,仍远低于 `75 files` 停止阈值
|
||||
- 本轮已并行完成 4 个批次:`YamlConfigLoaderTests.cs` 的 4 个纯加载 `MA0051`、`SceneTransitionPipeline.cs` / `UiTransitionPipeline.cs` 的 `MA0004`、3 个配置测试文件的机械型 `MA0004`、以及 `AbstractArchitectureModuleInstallationTests.cs` 的机械型 `MA0004`
|
||||
- `GFramework.Game.Tests` 当前 `--no-incremental Release build` 已进一步降到 `189 Warning(s)`;`YamlConfigLoaderTests.cs` 当前只剩热重载相关 `MA0051`
|
||||
|
||||
## 当前活跃事实
|
||||
|
||||
@ -22,10 +22,12 @@
|
||||
- 当前整仓最近一次直接观测值仍是 `dotnet build GFramework.sln -c Release` 的 `116 warning(s)`
|
||||
- `RP-056` 已验证 `GeneratedConfigConsumerIntegrationTests.cs` 不再出现在项目 build warning 输出中
|
||||
- `RP-057` 已验证 `PersistenceTests.cs` 不再出现在 `dotnet build GFramework.Game.Tests/GFramework.Game.Tests.csproj -c Release --no-incremental` 的 warning 输出中
|
||||
- `RP-059` 已验证 `YamlConfigLoaderTests.cs` 中的 `MA0004` 已清零,当前该文件在项目 build 输出里只剩 `MA0051`
|
||||
- `GFramework.Game.Tests` 当前 `--no-incremental Release build` 结果为 `203 Warning(s)`、`0 Error(s)`
|
||||
- 当前 `origin/main` 基线提交为 `9964962`(`2026-04-24T23:05:53+08:00`),与本地 `HEAD` 相同
|
||||
- 本轮 batch 的主停止条件仍为相对 `origin/main` 的 changed files `< 75`;当前工作树投影仅 `1` 个文件,因此停止原因不是体积,而是剩余切片不再低风险
|
||||
- `RP-059` 已验证 `YamlConfigLoaderTests.cs` 中的 `MA0004` 已清零,本轮继续把该文件 4 个纯加载 `MA0051` 热点也降掉了
|
||||
- `GFramework.Game.Tests` 当前 `--no-incremental Release build` 结果为 `189 Warning(s)`、`0 Error(s)`
|
||||
- `GFramework.Game/GFramework.Game.csproj -c Release` 当前结果为 `519 Warning(s)`、`0 Error(s)`;本轮新增 touched files `SceneTransitionPipeline.cs` 与 `UiTransitionPipeline.cs` 未在可见 warning 输出中残留
|
||||
- `GFramework.Godot.Tests/GFramework.Godot.Tests.csproj -c Release` 当前结果为 `0 Warning(s)`、`0 Error(s)`;`AbstractArchitectureModuleInstallationTests.cs` 已通过单测复验
|
||||
- 当前 `origin/main` 基线提交为 `9964962`(`2026-04-24T23:05:53+08:00`)
|
||||
- 当前累计 branch diff 相对 `origin/main` 为 `9` 个文件、`480` 行;主停止条件仍然是 `75 changed files`
|
||||
|
||||
## 当前风险
|
||||
|
||||
@ -35,8 +37,10 @@
|
||||
- 缓解措施:后续若继续整仓 warning reduction,需要单独定位 clean 失败原因,或明确继续沿用 direct build 观测值作为临时真值
|
||||
- 当前 worktree 仍存在未跟踪的 `.codex` 目录
|
||||
- 缓解措施:提交当前批次时只暂存 analyzer-warning-reduction 相关源码与 `ai-plan` 文件,避免把工作目录辅助文件混入提交
|
||||
- `YamlConfigLoaderTests.cs` 剩余切片已经从机械性的 `MA0004` 进入 `MA0051` 长方法重构,继续自动推进的风险明显高于前几轮
|
||||
- 缓解措施:下一轮若继续处理该文件,只处理一到两个长方法,并以 helper 抽取为主,不与其他文件混批
|
||||
- `YamlConfigLoaderTests.cs` 剩余切片已经收敛到热重载相关 `MA0051`,继续处理它的单文件收益不再能明显提升 branch diff 文件数
|
||||
- 缓解措施:后续优先切回新的单文件热点,只有在缺少低风险新文件时再回到该文件的热重载方法
|
||||
- 并行 subagent 已经证明能加快批次落地,但主线程仍需逐批复核并统一记录,否则容易让恢复点失真
|
||||
- 缓解措施:每轮并行批次完成后先更新 active tracking / trace,再继续下一批
|
||||
|
||||
## 活跃文档
|
||||
|
||||
@ -65,6 +69,12 @@
|
||||
- `RP-057` 热点重排前:成功;`253 Warning(s)`、`0 Error(s)`
|
||||
- `RP-057` 当前结果:成功;`249 Warning(s)`、`0 Error(s)`
|
||||
- `RP-059` 当前结果:成功;`203 Warning(s)`、`0 Error(s)`
|
||||
- `dotnet build GFramework.Game.Tests/GFramework.Game.Tests.csproj -c Release --no-incremental`
|
||||
- `RP-060` 当前结果:成功;`189 Warning(s)`、`0 Error(s)`
|
||||
- `dotnet build GFramework.Game/GFramework.Game.csproj -c Release`
|
||||
- `RP-060` 当前结果:成功;`519 Warning(s)`、`0 Error(s)`
|
||||
- `dotnet build GFramework.Godot.Tests/GFramework.Godot.Tests.csproj -c Release`
|
||||
- `RP-060` 当前结果:成功;`0 Warning(s)`、`0 Error(s)`
|
||||
- `dotnet test GFramework.Game.Tests/GFramework.Game.Tests.csproj -c Release --filter "FullyQualifiedName~ArchitectureConfigIntegrationTests|FullyQualifiedName~GameConfigBootstrapTests|FullyQualifiedName~JsonSerializerTests"`
|
||||
- 结果:成功;`Passed: 19`、`Failed: 0`
|
||||
- `dotnet test GFramework.Game.Tests/GFramework.Game.Tests.csproj -c Release --filter "FullyQualifiedName~GeneratedConfigConsumerIntegrationTests"`
|
||||
@ -73,8 +83,12 @@
|
||||
- 结果:成功;`Passed: 2`、`Failed: 0`
|
||||
- `dotnet test GFramework.Game.Tests/GFramework.Game.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~YamlConfigLoaderTests"`
|
||||
- 结果:成功;`Passed: 74`、`Failed: 0`
|
||||
- `dotnet test GFramework.Game.Tests/GFramework.Game.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~YamlConfigTextValidatorTests|FullyQualifiedName~GameConfigBootstrapTests|FullyQualifiedName~YamlConfigLoaderDependentRequiredTests"`
|
||||
- 结果:成功;`Passed: 15`、`Failed: 0`
|
||||
- `dotnet test GFramework.Godot.Tests/GFramework.Godot.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~InstallGodotModuleAsync_ShouldThrowBeforeInvokingModuleInstall_WhenAnchorIsMissing"`
|
||||
- 结果:成功;`Passed: 1`、`Failed: 0`
|
||||
|
||||
## 下一步建议
|
||||
|
||||
1. 提交 `YamlConfigLoaderTests.cs` 与 `RP-059` tracking/trace 更新,保留这一轮单文件 `MA0004` 清理的独立边界
|
||||
2. 若继续 warning reduction 主线,优先把 `YamlConfigLoaderTests.cs` 的 `MA0051` 作为新的高上下文批次单独处理;若希望继续保持低风险节奏,则改选其它单文件小热点
|
||||
1. 提交 `RP-060` tracking/trace 更新,固定本轮 4 个并行批次的恢复点与当前 `9 files / 480 lines` 体积
|
||||
2. 下一轮优先挑新的单文件热点来提升 branch diff 文件数;只有在新文件候选不够低风险时,再回到 `YamlConfigLoaderTests.cs` 的热重载 `MA0051`
|
||||
|
||||
@ -2,6 +2,39 @@
|
||||
|
||||
# Analyzer Warning Reduction 追踪
|
||||
|
||||
## 2026-04-25 — RP-060
|
||||
|
||||
### 阶段:并行子批次推进到 `9 files / 480 lines`
|
||||
|
||||
- 触发背景:
|
||||
- 用户明确要求“循环继续下一轮,直到 75 阈值”,并允许委派 subagent
|
||||
- `RP-059` 结束后,相对 `origin/main` 的累计 branch diff 仅 `3` 个文件,继续只做单文件深挖无法有效推进主停止条件
|
||||
- 主线程因此将任务拆成 4 个互不重叠的写集:1 个 `YamlConfigLoaderTests.cs` `MA0051` 批次、2 个 runtime `MA0004` 文件、3 个配置测试文件、1 个 Godot 测试文件
|
||||
- 主线程实施:
|
||||
- 本地复核 `YamlConfigLoaderTests.cs` 的 `MA0051` 热点边界,确认前四个纯加载测试比热重载测试更适合作为 helper 抽取起点
|
||||
- 用 explorer 只读排序下一批候选,再把 4 个 disjoint 写集交给 worker 并持续复核主工作树状态
|
||||
- 接受并保留以下提交进入当前分支:
|
||||
- `877d1f3` `fix(godot-tests): 清理模块安装测试异步断言包装`
|
||||
- `1dae0b1` `test(game-tests): 清理配置测试中的机械型 MA0004 包装`
|
||||
- `27f5a2f` `fix(game): 清理切换管道中的低风险 MA0004`
|
||||
- `b27bcb5` `refactor(game-tests): 清理指定加载测试的 MA0051`
|
||||
- 验证里程碑:
|
||||
- `dotnet build GFramework.Godot.Tests/GFramework.Godot.Tests.csproj -c Release`
|
||||
- 结果:成功;`0 Warning(s)`、`0 Error(s)`
|
||||
- `dotnet test GFramework.Godot.Tests/GFramework.Godot.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~InstallGodotModuleAsync_ShouldThrowBeforeInvokingModuleInstall_WhenAnchorIsMissing"`
|
||||
- 结果:成功;`Passed: 1`
|
||||
- `dotnet build GFramework.Game.Tests/GFramework.Game.Tests.csproj -c Release --no-incremental`
|
||||
- 结果:成功;`189 Warning(s)`、`0 Error(s)`
|
||||
- 结论:`YamlConfigLoaderTests.cs` 的四个纯加载 `MA0051` 热点已清零,剩余热点集中到热重载测试
|
||||
- `dotnet test GFramework.Game.Tests/GFramework.Game.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~YamlConfigTextValidatorTests|FullyQualifiedName~GameConfigBootstrapTests|FullyQualifiedName~YamlConfigLoaderDependentRequiredTests"`
|
||||
- 结果:成功;`Passed: 15`
|
||||
- `dotnet build GFramework.Game/GFramework.Game.csproj -c Release`
|
||||
- 结果:成功;`519 Warning(s)`、`0 Error(s)`
|
||||
- 当前结论:
|
||||
- 本轮并行批次把相对 `origin/main` 的累计 branch diff 推进到 `9` 个文件、`480` 行
|
||||
- 当前剩余最自然的高上下文热点仍在 `YamlConfigLoaderTests.cs` 的热重载 `MA0051`,但从 branch-size 目标看,下一轮更应该优先挑新的单文件热点
|
||||
- 只要低风险新文件仍存在,就应继续沿“新文件优先、同文件深挖次之”的策略推进 toward `75 files`
|
||||
|
||||
## 2026-04-25 — RP-059
|
||||
|
||||
### 阶段:`YamlConfigLoaderTests.cs` 单文件 `MA0004` 清理
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user