GFramework/ai-plan/public/analyzer-warning-reduction/traces/analyzer-warning-reduction-trace.md

Analyzer Warning Reduction 追踪

2026-04-25 — RP-067

阶段：收口 Game runtime 单文件长方法切片，并继续压低根构建 warning 基线

• 触发背景：
  • RP-066 收尾后，当前分支已通过 be26640 把 YamlConfigLoaderTests.cs 的 4 个 MA0051 落地，仓库根基线降到 652 Warning(s)
  • 主线程随后切到 GFramework.Game/Internal/VersionedMigrationRunner.cs，继续挑选单文件、低风险、可独立验证的 runtime warning 切片
• 主线程实施：
  • 将 MigrateToTargetVersion 中的运行时版本校验、迁移解析、单步应用与结果一致性校验拆分为具名 helper
  • 为新增 helper 补齐 XML 注释，保持该共享迁移执行器的职责边界可读，并避免仅靠机械拆分留下语义不清的私有方法
  • 保持外部行为不变，只收敛长方法 warning，不扩展到存储或日志相关调用方
• 验证里程碑：
  • dotnet clean
    • 结果：成功
  • dotnet build
    • 结果：成功；649 Warning(s)、0 Error(s)，相较 RP-066 的 652 再下降 3
  • dotnet build GFramework.Game/GFramework.Game.csproj -c Release
    • 结果：成功；0 Warning(s)、0 Error(s)
• 当前结论：
  • VersionedMigrationRunner.cs 这个 runtime 单文件批次已被主线程收口，并继续压低仓库根 warning 基线
  • 本轮只新增 1 个源码唯一文件，branch diff 仍显著低于 $gframework-batch-boot 50 的主停止阈值
  • 下一轮可以继续挑选 GFramework.Cqrs.Tests 或 GFramework.Game 的单文件轻量切片，并保持主线程验证、subagent 并行探索的节奏

2026-04-25 — RP-066

阶段：主线程回收停滞的单文件批次，并继续压低根构建 warning 基线

• 触发背景：
  • RP-065 收尾后，fix/analyzer-warning-reduction-batch 已通过 6a704f3 把 AGENTS / active ai-plan 真值修正和 4 文件测试噪音批次提交到分支
  • 原先负责 YamlConfigLoaderTests.cs 的 worker 长时间无结果，主线程收回该单文件批次以避免继续阻塞
• 主线程实施：
  • 关闭停滞 worker，直接重构 GFramework.Game.Tests/Config/YamlConfigLoaderTests.cs
  • 通过提取固定夹具内容、热重载接线 helper 与共享断言，收敛以下 4 个长方法 warning：
    • EnableHotReload_Should_Keep_Previous_State_When_Contains_Reference_Dependency_Breaks
    • EnableHotReload_Should_Support_Options_Object
    • EnableHotReload_Should_Keep_Previous_Table_When_Schema_Change_Makes_Reload_Fail
    • EnableHotReload_Should_Keep_Previous_State_When_Dependency_Table_Breaks_Cross_Table_Reference
  • 在第一次仓库根重建中命中了两个 CS0411 泛型推断错误，主线程随即补上显式类型参数并重新建立 clean/build 基线
• 验证里程碑：
  • dotnet clean
    • 结果：成功
  • dotnet build
    • 结果：成功；652 Warning(s)、0 Error(s)，相较 RP-065 的 656 再下降 4
  • dotnet build GFramework.Game.Tests/GFramework.Game.Tests.csproj -c Release
    • 结果：成功；0 Warning(s)、0 Error(s)
• 当前结论：
  • YamlConfigLoaderTests.cs 这 4 个根构建直接确认的 MA0051 已被消化
  • 当前分支在 6a704f3 之后的下一提交只会新增 1 个唯一文件，因此 branch diff 仍明显低于 $gframework-batch-boot 50 阈值
  • 下一轮可继续选择新的单文件或小写集热点，而不必暂停当前 batch loop

2026-04-25 — RP-065

阶段：确认 .NET 验证噪音来自沙箱，并把无沙箱直跑写成仓库规则

• 触发背景：
  • 用户明确指出“之前很多清理、构建、测试报错像是环境问题，需要申请权限在沙箱外执行”，并要求把该解决方案写入 AGENTS.md
  • 主线程随后在同一 worktree 中对比了沙箱内与提权后直接 shell 的 dotnet clean / dotnet build
• 主线程实施：
  • 在沙箱内直接运行 dotnet clean 时再次复现“Build FAILED but 0 errors”的无诊断噪音
  • 申请提权后重新执行同一条 dotnet clean，确认命令可正常完成，说明先前 clean 失败并非仓库真值
  • 在同一提权上下文直接执行 dotnet build，拿到当前仓库根权威基线：656 Warning(s)、0 Error(s)
  • 关闭正在运行的 warning-reduction worker，把工作重心切到仓库治理与 active recovery 文档净化
  • 更新 AGENTS.md，新增规则：当沙箱内 dotnet clean / dotnet build / dotnet test 产生缺少诊断、权限错误或其他环境噪音时，必须申请沙箱外重跑同一条直接命令，并以该结果为准
  • 刷新 active todo/trace，把“环境阻塞”从默认恢复入口中降级为历史噪音，不再作为当前真值
• 并行工作：
  • worker 收敛了 4 个低风险测试噪音文件：
    • GFramework.Game.Tests/Config/GameConfigBootstrapTests.cs
    • GFramework.Game.Tests/Config/GeneratedConfigConsumerIntegrationTests.cs
    • GFramework.Game.Tests/Config/YamlConfigTextValidatorTests.cs
    • GFramework.Ecs.Arch.Tests/Ecs/EcsAdvancedTests.cs
  • worker 验证：
    • dotnet build GFramework.Game.Tests/GFramework.Game.Tests.csproj -c Release
      • worker 初次结果：成功；随后主线程在同一提权环境复核后确认当前为 0 Warning(s)、0 Error(s)
    • dotnet build GFramework.Ecs.Arch.Tests/GFramework.Ecs.Arch.Tests.csproj -c Release
      • 主线程复核：成功；0 Warning(s)、0 Error(s)
• 当前结论：
  • 本仓库当前在 agent 沙箱内执行 .NET 验证时，确实可能出现假失败或缺失诊断
  • 当前应把“提权后的直接 dotnet 命令输出”视为仓库真值，而不是继续围绕沙箱噪音扩展 workaround 命令形态
  • fix/analyzer-warning-reduction-batch 当前 HEAD 已与 origin/main 对齐；新的 $gframework-batch-boot 50 轮次从 0 files / 0 lines committed diff 开始
  • 下一轮低风险热点仍是 GFramework.Game.Tests/Config/YamlConfigLoaderTests.cs 的 4 个 MA0051

2026-04-25 — RP-064

阶段：按标准 WSL build 路径复核 PR #288 建议并完成本轮收口

• 触发背景：
  • 用户指出“在 WSL 里直接执行 dotnet build 可以成功”，要求主线程按普通路径重新验证，而不是继续使用带 MSBuildEnableWorkloadResolver=false、--no-restore、手工 TargetFramework 的 workaround 命令
  • 当前任务仍属于 PR #288 review follow-up，因此本轮重点改为“区分哪些 AI 建议值得采纳”以及“用真实 WSL build 结果验证”
• 主线程实施：
  • 重新抓取 PR #288 review，确认 latest-head open threads 为 CodeRabbit 6 + Greptile 2
  • 复核 outside diff + nitpick 的 19 条建议，只采纳本地仍成立的建议；拒绝把“评论总数”机械等同于“必须全改”
  • 完成以下高信号修复：
    • ContextAware* / AsyncExtensions / NumericExtensions / StringExtensions / StoreBuilder：回退为 ArgumentNullException.ThrowIfNull(...)
    • ArchitectureServicesTests / GameContextTests：同步 XML <exception> 到 NotSupportedException
    • RegistryInitializationHookBaseTests：修复 override 可空签名实现，避免再次引入编译错误
    • RollingFileAppenderTests / TaskCoroutineExtensionsTests / WaitForTaskTests / ScopedStorage：移除无收益噪音代码
    • FileStorage：通过 leaveOpen: true 修正 FileStream 的双重释放语义
    • SceneRouterBase：统一显式 ConfigureAwait(true) 并补齐引擎线程亲和说明
    • StoreSelection：保留 net9.0+ 的 System.Threading.Lock，同时修正条件编译旁的注释写法，避免 CS1587
• 验证里程碑：
  • dotnet restore GFramework.sln -p:RestoreFallbackFolders="" -v minimal
    • 结果：成功；证明先前 MSB4018 来自 stale restore 元数据，而不是当前 WSL 默认 build 路径本身不可用
  • dotnet build GFramework.Core/GFramework.Core.csproj -c Release
    • 结果：成功；28 Warning(s)、0 Error(s)
  • dotnet build GFramework.Game/GFramework.Game.csproj -c Release
    • 结果：成功；329 Warning(s)、0 Error(s)
  • dotnet build GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release
    • 结果：成功；137 Warning(s)、0 Error(s)
• 当前结论：
  • 用户关于“WSL 里直接 dotnet build 可行”的判断正确
  • 前一轮失败的核心原因不是仓库不可构建，而是主线程附加的 workaround 参数改变了 MSBuild 行为
  • 本轮已完成 PR #288 中一组仍成立的建议修复，并重新拿到标准 WSL 路径下的 Release build 验证
  • 剩余 review 线程需要在新 head 上重新抓取后再决定是否逐条 resolve

2026-04-25 — RP-063

阶段：先收口 PR #288 latest-head 编译错误，再暂停在环境阻塞点并准备提交

• 触发背景：
  • 用户显式要求先执行 $gframework-pr-review，并指出 AsyncExtensionsTests.cs(126,23) 当前存在 CS0029 / CS1662 构建错误
  • 当前 worktree 仍是 fix/analyzer-warning-reduction-batch，因此本 turn 继续沿用 analyzer-warning-reduction 的 active recovery 文档
• 主线程实施：
  • 运行 PR review 抓取脚本，确认当前分支对应 PR #288
  • 核对 latest-head unresolved review threads 后，优先修复 AsyncExtensionsTests.cs 中 ct => Task.Delay(...).ConfigureAwait(false) 错误返回 ConfiguredTaskAwaitable 的问题
  • 顺手收敛多处已被 latest review 点名且本地仍成立的低风险残留：
    • 测试中的 async 无 await
    • ValueTask 断言包装
    • RegistryInitializationHookBaseTests.cs 的可空返回签名
    • NumericExtensions.cs、StringExtensions.cs、StoreBuilder.cs 的 Allman 花括号残留
    • StoreSelection.cs 在 net9.0+ 下切到 System.Threading.Lock，同时保留 net8.0 兼容分支
• 验证里程碑：
  • python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --json-output /tmp/current-pr-review.json
    • 结果：成功；确认 PR #288 的 latest-head unresolved AI review threads 共 9 个，其中 AsyncExtensionsTests.cs:126 为 critical 编译错误
  • DOTNET_CLI_HOME=/tmp/dotnet-home MSBuildEnableWorkloadResolver=false dotnet build GFramework.Core/GFramework.Core.csproj -c Release --no-restore -p:TargetFramework=net8.0 -p:RestoreFallbackFolders="" -v minimal
    • 结果：失败；MSB4018，ResolvePackageAssets 仍读取失效 Windows fallback package folder D:\Tool\Development Tools\Microsoft Visual Studio\Shared\NuGetPackages
  • DOTNET_CLI_HOME=/tmp/dotnet-home MSBuildEnableWorkloadResolver=false dotnet build GFramework.Core/GFramework.Core.csproj -c Release --no-restore -p:TargetFramework=net9.0 -p:RestoreFallbackFolders="" -v minimal
    • 结果：失败；原因同上
  • DOTNET_CLI_HOME=/tmp/dotnet-home MSBuildEnableWorkloadResolver=false dotnet build GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release --no-restore -p:TargetFramework=net10.0 -p:RestoreFallbackFolders="" -v minimal
    • 结果：失败；原因同上
• 当前结论：
  • 用户点名的 AsyncExtensionsTests.cs 编译错误已在源码层修复
  • 本 turn 未能拿到新的可通过 Release build，阻塞点已从先前记录的 MSB4276 收敛为当前 obj/*.csproj.nuget.g.props 中 stale Windows fallback package folder 导致的 MSB4018
  • 用户随后要求“先不管这个了，先提交吧”，因此本 turn 在记录环境阻塞后先执行提交收口

2026-04-25 — RP-062

阶段：触达 $gframework-batch-boot 75 停止阈值并收口到 75 files / 2098 lines

• 触发背景：
  • RP-061 收尾时分支相对 origin/main 仍只有 48 个已提交文件，距离本轮 75 files 停止条件还有明显空间
  • 用户明确允许继续委派 subagent，因此主线程继续把低风险机械型写集拆成互不重叠的 test / runtime 小批次
  • 本轮主目标不是继续深挖单个高上下文热点，而是用新的低风险文件精确把 branch diff 推到阈值后停止
• 主线程实施：
  • 先接受并提交 7 文件 Core.Tests 收尾批次为 03c73a8 test(core-tests): 收敛测试桩与辅助类型 warning
  • 随后主线程与多个 worker 并行收口以下新增文件：
    • ArchitectureAdditionalCqrsHandlersTests.cs
    • RegistryInitializationHookBaseTests.cs
    • CommandCoroutineExtensionsTests.cs
    • TaskCoroutineExtensionsTests.cs
    • WaitForTaskTTests.cs
    • AsyncExtensionsTests.cs
    • LogContextTests.cs
    • PauseStackManagerTests.cs
    • AsyncExtensions.cs
    • CollectionExtensions.cs
    • ContextAwareCommandExtensions.cs
    • ContextAwareEnvironmentExtensions.cs
    • ContextAwareEventExtensions.cs
    • ContextAwareQueryExtensions.cs
    • ContextAwareServiceExtensions.cs
    • GuardExtensions.cs
    • NumericExtensions.cs
    • StoreEventBusExtensions.cs
    • StringExtensions.cs
    • StoreBuilder.cs
    • StoreSelection.cs
  • 将上述 22 文件批次收口为 9ce1fa6 refactor(core): 收敛 Core 扩展与测试的机械 warning
• 验证里程碑：
  • dotnet build GFramework.Core/GFramework.Core.csproj -c Release --no-restore -p:TargetFramework=net8.0 -p:RestoreFallbackFolders="" -v minimal
    • 结果：成功；0 Warning(s)、0 Error(s)
  • dotnet build GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release --no-incremental --no-restore -p:RestoreFallbackFolders= -v:diag
    • 结果：失败；MSB4276，默认 SDK resolver 缺少 Microsoft.NET.SDK.WorkloadAutoImportPropsLocator
  • dotnet restore GFramework.Core.Tests/GFramework.Core.Tests.csproj -p:TestTargetFrameworks=net8.0 -p:RestoreFallbackFolders="" -v minimal
    • 结果：失败；NU1201，GFramework.Tests.Common 仅支持 net10.0，不能作为 Core.Tests 的 net8 旁路验证
  • git diff --name-only origin/main...HEAD | wc -l
    • 结果：75
  • git diff --numstat origin/main...HEAD
    • 结果：累计 1083 added、1015 deleted，即 2098 changed lines
• 当前结论：
  • 本轮 $gframework-batch-boot 75 已精确达到主停止条件，默认恢复点应停止在 9ce1fa6
  • Core runtime 的本轮机械型改动已有可通过的最小 Release build 验证
  • Core.Tests 的继续推进当前首先受 MSB4276 环境阻塞影响；下一轮若要继续，应先修复构建环境，再重新建立 warning 基线

历史归档指针

• 早期 trace 归档：
  • analyzer-warning-reduction-history-rp001.md
  • analyzer-warning-reduction-history-rp002-rp041.md
  • analyzer-warning-reduction-history-rp042-rp048.md