docs(analyzer-warning-reduction): 同步PR审查恢复记录

- 更新 tracking 文档中的变更文件计数与验证口径,使其与当前 PR head 和 trace 记录一致。

- 补充 RP-092 trace,记录当前 open threads 复核结论、定向测试结果与后续恢复步骤。
This commit is contained in:
gewuyou 2026-04-28 15:49:26 +08:00
parent f33a176570
commit 5c4f2df15b
2 changed files with 31 additions and 10 deletions

View File

@ -6,12 +6,12 @@
## 当前恢复点
- 恢复点编号:`ANALYZER-WARNING-REDUCTION-RP-091`
- 当前阶段:`Phase 91`
- 恢复点编号:`ANALYZER-WARNING-REDUCTION-RP-092`
- 当前阶段:`Phase 92`
- 当前焦点:
- `2026-04-28` 继续收口 `PR #300` 的剩余 nitpick已把 `TestArchitectureContext` / `TestArchitectureContextV3` 的重复 `IArchitectureContext` 测试实现抽到共享基类 `TestArchitectureContextBase`
- 已将 “本地验证后仍然成立的 nitpick 不得默认降级为可选项” 写入 `.agents/skills/gframework-pr-review/SKILL.md`
- `dotnet format --verify-no-changes``GFramework.Core.Tests` 既有 `FINALNEWLINE``CHARSET``WHITESPACE` 基线仍保持独立,不与本轮共享基类重构混提
- `2026-04-28` 复核 `PR #300` 最新 open threads代码类线程已与当前工作树对齐仅剩 `ai-plan/public/analyzer-warning-reduction/todos/analyzer-warning-reduction-tracking.md` 的文件计数与验证口径漂移仍然成立
- 已将 tracking 文档修正为与 `6cc87a9...HEAD` 的实际变更规模一致,并与 trace 中记录的 `dotnet build`、定向 `dotnet test``git diff --check` 验证口径保持一致
- `dotnet format --verify-no-changes``GFramework.Core.Tests` 既有 `FINALNEWLINE``CHARSET``WHITESPACE` 基线仍保持独立,不与当前 `ai-plan` 同步修复混提
## 当前活跃事实
@ -22,11 +22,13 @@
- `dotnet test GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~ArchitectureServicesTests|FullyQualifiedName~ContextAwareServiceExtensionsTests|FullyQualifiedName~TestArchitectureContextBehaviorTests|FullyQualifiedName~RegistryInitializationHookBaseTests|FullyQualifiedName~ArchitectureContextTests"`
- 最新结果:成功;`67` 通过、`0` 失败
- 当前批次摘要:
- 当前工作树包含 `6` 个已修改文件与 `1` 个新增文件,分别位于 `GFramework.Core.Tests`、`.agents/skills/gframework-pr-review/``ai-plan/public/analyzer-warning-reduction`
- 当前分支相对 `6cc87a9...HEAD` 包含 `18` 个已修改文件与 `38` 个新增文件(合计 `56` 个变更文件),分别位于 `GFramework.Core.Tests``GFramework.Cqrs.Tests``GFramework.Core`、`.agents/skills/gframework-pr-review/``ai-plan/public/analyzer-warning-reduction`
- 本轮没有触碰 `Mediator/*``YamlConfigSchemaValidator*``GFramework.Core.Tests` 的整项目格式基线波次
## 当前风险
- GitHub PR 上的 open threads 可能仍显示为未关闭,因为当前只同步了 `ai-plan` 文档,尚未推送新的 head 供审查机器人重新折叠线程。
- 缓解措施:推送本次 `ai-plan` 同步提交后重新执行 `$gframework-pr-review`,以最新 head 再核对 thread 状态。
- `dotnet format GFramework.Core.Tests/GFramework.Core.Tests.csproj --verify-no-changes` 当前会命中项目内大量历史格式诊断。
- 缓解措施:本轮只记录为现存基线,不把 `PR #300` 的 review follow-up 扩展成整项目格式清理。
- `GFramework.Game/Config/YamlConfigSchemaValidator*` 仍然是仓库根 warning 热点,但与本轮 review 修复无交集。
@ -53,10 +55,10 @@
- 权威验证结果统一维护在“当前活跃事实”。
- `GFramework.Core.Tests` 的当前受影响项目 Release 构建已清零,并通过对应定向测试回归。
- `git diff --check` 结果为空,说明本轮新增改动没有引入新的尾随空格或冲突标记。
- warning reduction 的仓库级真值只以同轮 `dotnet clean` 后的 `dotnet build` 为准
- warning reduction 的仓库级真值以同轮 `dotnet build`、定向 `dotnet test``git diff --check` 为准,并与 trace 中的验证里程碑保持一致
## 下一步建议
1. 提交本轮 `PR #300` nitpick follow-up、技能规则更新与 `ai-plan` 同步。
2. 推送后重新执行 `$gframework-pr-review`,确认该 nitpick thread 是否随最新 head 自动收口。
2. 推送后重新执行 `$gframework-pr-review`,确认 `ai-plan` 相关 thread 是否随最新 head 自动收口。
3. 若要清理 `dotnet format` 基线,另开 `GFramework.Core.Tests` 格式治理切片,不与当前 PR review 修复混提。

View File

@ -1,5 +1,24 @@
# Analyzer Warning Reduction 追踪
## 2026-04-28 — RP-092
### 阶段:复核 `PR #300` 的 open threads并只修正当前分支仍然成立的 `ai-plan` 漂移
- 触发背景:
- 用户要求恢复当前 `$gframework-pr-review` 任务,继续以 PR head 上的开放线程为准做 triage
- 主线程实施:
- 重新读取 `fetch_current_pr_review.py --json-output /tmp/current-pr-review.json` 的 latest head open threads
- 逐条对照本地文件后确认:`TestArchitectureContextBehaviorTests``TestArchitectureWithRegistry``TestResourceLoader``PartialGeneratedNotificationHandlerRegistry` 相关 CodeRabbit 线程在当前工作树上都已匹配修复,仅线程状态尚未随新 head 折叠
- 继续核对 `RegistryInitializationHookBaseTests.OnPhase_Should_Not_Throw_When_Registry_Not_Found`,确认当前实现 `RegistryInitializationHookBase.OnPhase` 已在缺少注册表时保持 no-op定向回归测试通过
- 修正 `analyzer-warning-reduction-tracking.md` 中仍然成立的两处漂移:
- 将文件计数更新为相对 `6cc87a9...HEAD` 的实际规模:`18` 个已修改文件、`38` 个新增文件、合计 `56` 个变更文件
- 将验证口径统一为 trace 已记录的 `dotnet build`、定向 `dotnet test``git diff --check`
- 验证里程碑:
- `dotnet test GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release --filter "FullyQualifiedName~RegistryInitializationHookBaseTests.OnPhase_Should_Not_Throw_When_Registry_Not_Found|FullyQualifiedName~TestArchitectureContextBehaviorTests"`
- 结果:成功;`10` 通过、`0` 失败
- `git diff --check`
- 结果:成功;无新增 whitespace / conflict-marker 问题
## 2026-04-28 — RP-091
### 阶段:收口 `PR #300` 的共享测试基础设施 nitpick并升级 PR-review triage 规则
@ -28,8 +47,8 @@
## 下一步
1. 提交本轮共享基类重构、技能规则更新与 `ai-plan` 同步。
2. 推送后重新执行 `$gframework-pr-review`,确认剩余 PR 线程是否已经下降。
1. 提交本轮 `ai-plan` 同步修复,使 PR head 能重新折叠文档相关线程
2. 推送后重新执行 `$gframework-pr-review`,确认剩余 open threads 是否已经下降。
## 历史归档指针