gewuyou/GFramework
1
0
Fork 0
mirror of https://github.com/GeWuYou/GFramework.git synced 2026-05-12 22:03:30 +08:00
Code Issues Packages Projects Releases Wiki Activity
GFramework/ai-plan/public/analyzer-warning-reduction/todos/analyzer-warning-reduction-tracking.md
gewuyou 067d72fada fix(tooling): 收口PR评审遗留nitpick 
- 修复 PR review 脚本对 failed-test 额外列表格的解析容错

- 清理 AsyncExtensionsTests 中多余的等待并保留参数名断言

- 补充脚本回归测试并同步 analyzer-warning-reduction 恢复点
2026-04-27 10:22:45 +08:00

6.4 KiB
Raw Blame History

Analyzer Warning Reduction 跟踪

目标

继续以“直接看构建输出、直接修构建 warning”为原则推进当前分支，并保持 active recovery 文档只保留当前真值。

当前恢复点

  • 恢复点编号：ANALYZER-WARNING-REDUCTION-RP-081
  • 当前阶段：Phase 81
  • 当前焦点：
    • 2026-04-27 已复核 PR #295 的 latest-head review，确认 ThrowShouldNotRetry 的 ParamName open thread 属于 stale finding，本地代码已经使用传入值而非 nameof(parameterName)
    • 已清理 AsyncExtensionsTests.WithRetry_Should_Respect_ShouldRetry_Predicate 中的冗余 Task.Delay(50)，保留 ParamName == nameof(taskFactory) 断言锁定契约
    • 已增强 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py 的 failed-test 表格解析，允许 Name / Failure Message 后出现尾随额外列
    • 已新增 Python unittest 回归用例覆盖“尾随额外列不影响前两列提取”的场景
    • 当前剩余 warning 热点仍集中在 YamlConfigSchemaValidator*、YamlConfigLoader.cs 与大批量 MA0048 文件名拆分；这些 slice 仍高于本轮 PR review follow-up 的低风险边界

当前活跃事实

  • 当前 origin/main 基线提交为 617e0bf（2026-04-26T12:17:15+08:00）。
  • 当前 PR review 真值：
    • python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --json-output <current-pr-review-json>
      • 最新结果：成功；当前分支对应 PR 为 #295
      • 当前测试报告输出已能显示 Summary 统计、失败测试名称，以及 Name / Failure Message 表格中的关键信息
      • 当前 GitHub latest-head review 仍显示 1 条 open thread，但该线程指向的 nameof(parameterName) 问题已不在本地代码中成立，属于 stale finding
      • 当前 latest review 中仍有 2 条与本地工作树一致的 nitpick：AsyncExtensionsTests 冗余等待，以及 failed-test 表格解析对尾随列不鲁棒
  • 当前直接验证结果：
    • python3 .agents/skills/gframework-pr-review/scripts/test_fetch_current_pr_review.py
      • 最新结果：成功；Ran 1 test in 0.000s, OK
    • python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --section tests --json-output /tmp/current-pr-review-postfix.json
      • 最新结果：成功；真实 PR 评论抓取仍能输出 2 份测试报告，失败用例详情保持可见
    • dotnet test GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release --filter "FullyQualifiedName~WithRetry_Should_Respect_ShouldRetry_Predicate"
      • 最新结果：成功；Failed: 0, Passed: 1, Skipped: 0, Total: 1
    • dotnet test GFramework.Game.Tests/GFramework.Game.Tests.csproj -c Release --filter "FullyQualifiedName~RegisterMigration_During_Cache_Rebuild_Should_Not_Leave_Stale_Type_Cache"
      • 最新结果：成功；Failed: 0, Passed: 1, Skipped: 0, Total: 1
  • 当前分支 stop-condition 指标：
    • git diff --name-only refs/remotes/origin/main...HEAD | wc -l
      • 最新结果：35
    • git diff --numstat refs/remotes/origin/main...HEAD
      • 最新结果：642 changed lines
  • 当前批次摘要：
    • 三轮低风险 warning 清理已在此前验证中将仓库根 warning 从 639 降到 397
    • 当前批次的已完成 slice 明细已迁移到归档，active todo 仅保留恢复真值
    • 本轮新增内容为 PR review nitpick 收口与脚本回归测试补齐，不扩展 warning reduction 的热点清理边界
  • 当前建议保留到下一波次的候选：
    • GFramework.Game/Config/YamlConfigLoader.cs 的 MA0158（单点可修，但文件本身同时承载其他高耦合 warning）
    • 测试项目中的 MA0048 文件名拆分波次（会显著增加 changed-file 数）

当前风险

  • GFramework.Game/Config/YamlConfigSchemaValidator*.cs 仍然聚集多类高耦合 warning。
    • 缓解措施：本轮先避开该热点，只清理低风险且 ownership 清晰的文件集合。
  • MA0158 迁移涉及 net8.0 / net9.0 / net10.0 多目标兼容。
    • 缓解措施：复用 StoreSelection.cs 已存在的 #if NET9_0_OR_GREATER 专用锁模式，不在 net8.0 引入不兼容 API。
  • 当前 PR open thread 与 CI 失败信号仍依赖新提交进入远端 PR head 才能复核。
    • 缓解措施：本轮提交并推送后重新执行 $gframework-pr-review，确认 stale open thread 是否被 GitHub 收口，以及两条 nitpick 是否从 latest review 中消失。

活跃文档

验证说明

  • 权威验证结果统一维护在“当前活跃事实”。
  • GFramework.Core.Tests 当前仍有既有 analyzer / nullable warning 基线，因此本轮验证只证明 PR review 修复未引入构建错误，未将该项目 warning 清零。
  • 后续若刷新构建或 PR review 真值，只更新上述权威区块，不在本节重复抄录。

下一步建议

  1. 提交本轮 AsyncExtensionsTests / $gframework-pr-review nitpick 修复、Python 回归测试与 ai-plan 同步。
  2. 推送后重新执行 $gframework-pr-review，确认 PR #295 的 stale open thread、nitpick 与测试报告是否已刷新为新 head 真值。
  3. 若后续继续推进 warning reduction，建议另开下一波次处理 YamlConfigLoader.cs 热点或测试项目 MA0048 拆分波次。