From 1635a03a99a9367d3f4b1767303ef0590b3608a2 Mon Sep 17 00:00:00 2001 From: gewuyou <95328647+GeWuYou@users.noreply.github.com> Date: Tue, 28 Apr 2026 10:11:05 +0800 Subject: [PATCH] =?UTF-8?q?fix(tests):=20=E6=94=B6=E6=95=9BPR300=E5=AE=A1?= =?UTF-8?q?=E6=9F=A5=E9=81=97=E7=95=99=E9=97=AE=E9=A2=98?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - 修复 TestArchitectureContextV3 的共享事件总线语义并补充对应回归测试 - 优化 CapturingLoggerFactoryProvider 的最小级别同步策略并补充验证 - 更新 RegistryInitializationHookBase 异常文档与 analyzer warning reduction 的恢复文档归档 --- .../TestArchitectureContextBehaviorTests.cs | 15 ++ .../TestArchitectureContextV3.cs | 12 +- .../RegistryInitializationHookBase.cs | 1 + .../Cqrs/CapturingLoggerFactoryProvider.cs | 28 ++- .../Cqrs/CqrsHandlerRegistrarTests.cs | 21 ++ ...r-warning-reduction-history-rp083-rp088.md | 55 +++++ .../analyzer-warning-reduction-tracking.md | 29 +-- .../analyzer-warning-reduction-trace.md | 222 ++---------------- 8 files changed, 159 insertions(+), 224 deletions(-) create mode 100644 ai-plan/public/analyzer-warning-reduction/archive/traces/analyzer-warning-reduction-history-rp083-rp088.md diff --git a/GFramework.Core.Tests/Architectures/TestArchitectureContextBehaviorTests.cs b/GFramework.Core.Tests/Architectures/TestArchitectureContextBehaviorTests.cs index a4532a0f..4c0889a1 100644 --- a/GFramework.Core.Tests/Architectures/TestArchitectureContextBehaviorTests.cs +++ b/GFramework.Core.Tests/Architectures/TestArchitectureContextBehaviorTests.cs @@ -28,6 +28,21 @@ public class TestArchitectureContextBehaviorTests Assert.That(eventReceived, Is.True); } + /// + /// 验证用于 ArchitectureServices 的上下文替身也会把事件注册与发送委托到同一个事件总线实例。 + /// + [Test] + public void RegisterEvent_And_SendEvent_On_TestArchitectureContextV3_Should_Use_Shared_EventBus() + { + var context = new TestArchitectureContextV3(); + var eventReceived = false; + + context.RegisterEvent(_ => eventReceived = true); + context.SendEvent(); + + Assert.That(eventReceived, Is.True); + } + /// /// 验证测试上下文的旧版命令与查询入口会显式抛出未支持异常。 /// diff --git a/GFramework.Core.Tests/Architectures/TestArchitectureContextV3.cs b/GFramework.Core.Tests/Architectures/TestArchitectureContextV3.cs index 7cdd8a57..313864cd 100644 --- a/GFramework.Core.Tests/Architectures/TestArchitectureContextV3.cs +++ b/GFramework.Core.Tests/Architectures/TestArchitectureContextV3.cs @@ -31,6 +31,7 @@ public class TestArchitectureContextV3 : IArchitectureContext { private readonly MicrosoftDiContainer _container = new(); private readonly DefaultEnvironment _environment = new(); + private readonly EventBus _eventBus = new(); /// /// 获取或初始化用于区分测试上下文实例的标识。 @@ -167,6 +168,7 @@ public class TestArchitectureContextV3 : IArchitectureContext /// 事件类型。 public void SendEvent() where TEvent : new() { + _eventBus.Send(); } /// @@ -174,8 +176,11 @@ public class TestArchitectureContextV3 : IArchitectureContext /// /// 事件类型。 /// 事件实例。 + /// 为 。 public void SendEvent(TEvent e) where TEvent : class { + ArgumentNullException.ThrowIfNull(e); + _eventBus.Send(e); } /// @@ -184,9 +189,11 @@ public class TestArchitectureContextV3 : IArchitectureContext /// 事件类型。 /// 事件处理回调。 /// 用于注销回调的句柄。 + /// 为 。 public IUnRegister RegisterEvent(Action handler) { - return new DefaultUnRegister(() => { }); + ArgumentNullException.ThrowIfNull(handler); + return _eventBus.Register(handler); } /// @@ -194,8 +201,11 @@ public class TestArchitectureContextV3 : IArchitectureContext /// /// 事件类型。 /// 要取消的事件回调。 + /// 为 。 public void UnRegisterEvent(Action onEvent) { + ArgumentNullException.ThrowIfNull(onEvent); + _eventBus.UnRegister(onEvent); } /// diff --git a/GFramework.Core/Architectures/RegistryInitializationHookBase.cs b/GFramework.Core/Architectures/RegistryInitializationHookBase.cs index 49b99c18..277dc132 100644 --- a/GFramework.Core/Architectures/RegistryInitializationHookBase.cs +++ b/GFramework.Core/Architectures/RegistryInitializationHookBase.cs @@ -34,6 +34,7 @@ public abstract class RegistryInitializationHookBase : IArch /// /// 当前的架构阶段 /// 相关的架构实例 + /// 为 。 /// /// 当目标注册表未被装入当前架构上下文时，该钩子会保持 no-op， /// 以便同一组配置可以安全复用于不包含该注册表的测试或裁剪场景。 diff --git a/GFramework.Cqrs.Tests/Cqrs/CapturingLoggerFactoryProvider.cs b/GFramework.Cqrs.Tests/Cqrs/CapturingLoggerFactoryProvider.cs index dfb8dff4..01b825a9 100644 --- a/GFramework.Cqrs.Tests/Cqrs/CapturingLoggerFactoryProvider.cs +++ b/GFramework.Cqrs.Tests/Cqrs/CapturingLoggerFactoryProvider.cs @@ -17,6 +17,7 @@ namespace GFramework.Cqrs.Tests.Cqrs; internal sealed class CapturingLoggerFactoryProvider : ILoggerFactoryProvider { private readonly List _loggers = []; + private LogLevel _minLevel; private readonly Lock _sync = new(); /// @@ -25,7 +26,7 @@ internal sealed class CapturingLoggerFactoryProvider : ILoggerFactoryProvider /// 要应用到新建测试日志器的最小日志级别。 public CapturingLoggerFactoryProvider(LogLevel minLevel = LogLevel.Info) { - MinLevel = minLevel; + _minLevel = minLevel; } /// @@ -45,7 +46,24 @@ internal sealed class CapturingLoggerFactoryProvider : ILoggerFactoryProvider /// /// 获取或设置新建测试日志器的最小日志级别。 /// - public LogLevel MinLevel { get; set; } + public LogLevel MinLevel + { + get + { + lock (_sync) + { + return _minLevel; + } + } + + set + { + lock (_sync) + { + _minLevel = value; + } + } + } /// /// 创建一个测试日志器并将其纳入捕获集合。 @@ -54,13 +72,11 @@ internal sealed class CapturingLoggerFactoryProvider : ILoggerFactoryProvider /// 用于后续断言的测试日志器。 public ILogger CreateLogger(string name) { - var logger = new TestLogger(name, MinLevel); - lock (_sync) { + var logger = new TestLogger(name, _minLevel); _loggers.Add(logger); + return logger; } - - return logger; } } diff --git a/GFramework.Cqrs.Tests/Cqrs/CqrsHandlerRegistrarTests.cs b/GFramework.Cqrs.Tests/Cqrs/CqrsHandlerRegistrarTests.cs index 76a2b922..7e5afc7d 100644 --- a/GFramework.Cqrs.Tests/Cqrs/CqrsHandlerRegistrarTests.cs +++ b/GFramework.Cqrs.Tests/Cqrs/CqrsHandlerRegistrarTests.cs @@ -216,6 +216,27 @@ internal sealed class CqrsHandlerRegistrarTests } } + /// + /// 验证捕获型日志工厂在更新最小日志级别后，会将新值应用到后续创建的日志器。 + /// + [Test] + public void CapturingLoggerFactoryProvider_Should_Apply_Updated_MinLevel_To_Subsequent_Loggers() + { + var provider = new CapturingLoggerFactoryProvider(LogLevel.Warning); + var warningLogger = (TestLogger)provider.CreateLogger("warning"); + + provider.MinLevel = LogLevel.Debug; + + var debugLogger = (TestLogger)provider.CreateLogger("debug"); + + Assert.Multiple(() => + { + Assert.That(warningLogger.IsDebugEnabled(), Is.False); + Assert.That(debugLogger.IsDebugEnabled(), Is.True); + Assert.That(provider.Loggers, Has.Count.EqualTo(2)); + }); + } + /// /// 验证当生成注册器提供精确 fallback 类型名时，运行时会定向补扫剩余 handlers， /// 而不是重新枚举整个程序集的类型列表。 diff --git a/ai-plan/public/analyzer-warning-reduction/archive/traces/analyzer-warning-reduction-history-rp083-rp088.md b/ai-plan/public/analyzer-warning-reduction/archive/traces/analyzer-warning-reduction-history-rp083-rp088.md new file mode 100644 index 00000000..8ea52800 --- /dev/null +++ b/ai-plan/public/analyzer-warning-reduction/archive/traces/analyzer-warning-reduction-history-rp083-rp088.md @@ -0,0 +1,55 @@ +# Analyzer Warning Reduction 历史归档（RP-083 ~ RP-088） + +## 范围说明 + +- 归档区间：`RP-083` 到 `RP-088` +- 归档原因：active trace 已累计多个已完成阶段，不再适合作为默认恢复入口 +- 当前活跃恢复点：返回 `ai-plan/public/analyzer-warning-reduction/traces/analyzer-warning-reduction-trace.md` + +## RP-088 + +- 阶段：收敛 `PR #300` 的 open review threads 与 failed-test follow-up +- 主结论： + - 核对 `TestArchitectureContext*`、`RegistryInitializationHookBase`、`TestResourceLoader`、`CapturingLoggerFactoryProvider`、`PartialGeneratedNotificationHandlerRegistry` 等 review 位点 + - 新增 `TestArchitectureContextBehaviorTests.cs`，覆盖共享事件总线、旧入口失败契约与 `RegisterLifecycleHook` 接口行为 + - 受影响项目验证通过，`GFramework.Cqrs.Tests` 仍保留既有 `Mediator/*` warning 基线 + +## RP-087 + +- 阶段：按 `$gframework-batch-boot 50` 并行收敛 `Core.Tests` / `Cqrs.Tests` 低风险切片 +- 主结论： + - 建立仓库根 non-incremental warning 基线后，并行消化 `Core.Tests` 与 `Cqrs.Tests` 的低风险 warning + - 仓库根 warning 从 `288` 下降到 `236` + - 剩余热点开始集中到 `Mediator/*` 与 `YamlConfigSchemaValidator*` + +## RP-086 + +- 阶段：收敛 `PR #298` 的 CodeRabbit nitpick follow-up +- 主结论： + - 修复测试辅助类型的可维护性 nitpick + - `GFramework.Core.Tests` 定向验证通过 + - 剩余 warning 仍集中在既有热点文件 + +## RP-085 + +- 阶段：按 `$gframework-batch-boot 100` 并行消化 `GFramework.Core.Tests` 低风险 `MA0048` +- 主结论： + - 四波次并行拆分 `GFramework.Core.Tests` 测试辅助类型 + - 仓库根 warning 从 `353` 下降到 `288` + - active footprint 接近阈值后主动收口 + +## RP-084 + +- 阶段：收敛 `PR #297` 的 CodeRabbit follow-up +- 主结论： + - 校正 `YamlConfigLoader` 取消语义与若干 XML 文档问题 + - 新增定向回归测试覆盖取消异常路径 + - 相关构建与测试全部通过 + +## RP-083 + +- 阶段：修复 `YamlConfigLoader` 单文件 warning，并拆分 `MicrosoftDiContainerTests` 的辅助类型 +- 主结论： + - 从仓库根基线出发完成单文件 warning 修复与两组测试辅助类型拆分 + - 仓库根 warning 从 `397` 下降到 `353` + - 后续工作切入点转向 `ArchitectureContextTests.cs` / `AsyncQueryExecutorTests.cs` diff --git a/ai-plan/public/analyzer-warning-reduction/todos/analyzer-warning-reduction-tracking.md b/ai-plan/public/analyzer-warning-reduction/todos/analyzer-warning-reduction-tracking.md index 441e2b3e..f6f40fd8 100644 --- a/ai-plan/public/analyzer-warning-reduction/todos/analyzer-warning-reduction-tracking.md +++ b/ai-plan/public/analyzer-warning-reduction/todos/analyzer-warning-reduction-tracking.md @@ -6,42 +6,43 @@ ## 当前恢复点 -- 恢复点编号：`ANALYZER-WARNING-REDUCTION-RP-088` -- 当前阶段：`Phase 88` +- 恢复点编号：`ANALYZER-WARNING-REDUCTION-RP-089` +- 当前阶段：`Phase 89` - 当前焦点： - - `2026-04-28` 已执行 `$gframework-pr-review`，确认 `PR #300` 最新 head 上仍有 `8` 条 CodeRabbit open threads、`1` 个 failed test，以及 `dotnet-format restore failed` 的 CI 噪音 - - 本轮已核对并收敛仍然成立的 review comments：`TestArchitectureContext*` 旧入口显式失败、共享事件总线、`RegisterLifecycleHook` 语义统一、`TestResourceLoader` 契约、`TestLogger` / `CapturingLoggerFactoryProvider` 快照访问、`DeterministicNotificationHandlerState` 并发说明与 `PartialGeneratedNotificationHandlerRegistry` XML 异常文档 - - 已新增 `TestArchitectureContextBehaviorTests.cs`，直接覆盖共享事件总线、旧入口失败契约与接口视角生命周期钩子行为 - - `RegistryInitializationHookBase` 现已在注册表缺失时保持 no-op，修复了 PR 上报的失败测试 `OnPhase_Should_Not_Throw_When_Registry_Not_Found` + - `2026-04-28` 重新执行 `$gframework-pr-review`，确认 `PR #300` 最新 head 上仍显示 `6` 条 CodeRabbit open threads；其中 `Task.CompletedTask` 强转与 failed-test 信号已是 stale + - 本轮继续补齐仍然成立的差异：`TestArchitectureContextV3` 共享事件总线、`RegistryInitializationHookBase.OnPhase` XML 异常文档、`CapturingLoggerFactoryProvider.MinLevel` 同步，以及 active trace 归档瘦身 + - 已扩展 `TestArchitectureContextBehaviorTests.cs` 覆盖 `TestArchitectureContextV3` 的共享事件总线行为 + - 已新增 `CapturingLoggerFactoryProvider_Should_Apply_Updated_MinLevel_To_Subsequent_Loggers`，验证后续 logger 会读取更新后的最小级别 ## 当前活跃事实 - 当前 `origin/main` 基线提交为 `6cc87a9`（`2026-04-27T20:28:50+08:00`）。 - 当前直接验证结果： + - `dotnet build GFramework.Core/GFramework.Core.csproj -c Release` + - 最新结果：成功；`0 Warning(s)`、`0 Error(s)` - `dotnet build GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release` - 最新结果：成功；`0 Warning(s)`、`0 Error(s)` - `dotnet build GFramework.Cqrs.Tests/GFramework.Cqrs.Tests.csproj -c Release` - - 最新结果：成功；`125 Warning(s)`、`0 Error(s)`；warning 仍集中在既有 `Mediator/*` 文件，不在本轮 PR review 修复写集内 - - `dotnet build GFramework.Core/GFramework.Core.csproj -c Release` - - 最新结果：成功；`0 Warning(s)`、`0 Error(s)` - - `dotnet test GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~RegistryInitializationHookBaseTests|FullyQualifiedName~WaitForMultipleEventsTests|FullyQualifiedName~ResourceManagerTests|FullyQualifiedName~LoggerTests|FullyQualifiedName~TestArchitectureContextBehaviorTests"` - - 最新结果：成功；`97` 通过、`0` 失败 - - `dotnet test GFramework.Cqrs.Tests/GFramework.Cqrs.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~CqrsHandlerRegistrarTests"` + - 最新结果：成功；`125 Warning(s)`、`0 Error(s)`；warning 仍集中在既有 `Mediator/*` 文件，不在本轮 PR review follow-up 写集内 + - `dotnet test GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~TestArchitectureContextBehaviorTests|FullyQualifiedName~RegistryInitializationHookBaseTests"` - 最新结果：成功；`11` 通过、`0` 失败 + - `dotnet test GFramework.Cqrs.Tests/GFramework.Cqrs.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~CqrsHandlerRegistrarTests"` + - 最新结果：成功；`12` 通过、`0` 失败 - 当前批次摘要： - - 当前工作树包含 `11` 个已修改文件和 `1` 个新增测试文件，全部来自 `Core` / `Core.Tests` / `Cqrs.Tests` 的 PR review follow-up + - 当前工作树包含 `8` 个已修改文件和 `1` 个新增归档 trace 文件，全部来自 `Core` / `Core.Tests` / `Cqrs.Tests` / `ai-plan` 的 `PR #300` follow-up - 本轮没有触碰 `Mediator/*` 或 `YamlConfigSchemaValidator*` 的高耦合 warning 波次 ## 当前风险 - `GFramework.Cqrs.Tests` 当前项目级 Release 构建仍有 `125` 条既有 warning，主要集中在 `MediatorArchitectureIntegrationTests.cs`、`MediatorAdvancedFeaturesTests.cs` 与 `MediatorComprehensiveTests.cs`。 - - 缓解措施：本轮仅记录为现存 blocker，不在 PR #300 的 review follow-up 里扩展到 `Mediator/*` warning reduction 波次。 + - 缓解措施：本轮仅记录为现存 blocker，不在 `PR #300` 的 review follow-up 里扩展到 `Mediator/*` warning reduction 波次。 - `GFramework.Game/Config/YamlConfigSchemaValidator*` 仍然是仓库根 warning 热点，但与本轮 review 修复无交集。 - 缓解措施：继续保持为独立高耦合波次。 ## 活跃文档 - 当前轮次归档： + - [analyzer-warning-reduction-history-rp083-rp088.md](../archive/traces/analyzer-warning-reduction-history-rp083-rp088.md) - [analyzer-warning-reduction-history-rp074-rp078.md](../archive/todos/analyzer-warning-reduction-history-rp074-rp078.md) - [analyzer-warning-reduction-history-rp042-rp048.md](../archive/todos/analyzer-warning-reduction-history-rp042-rp048.md) - 历史跟踪归档： diff --git a/ai-plan/public/analyzer-warning-reduction/traces/analyzer-warning-reduction-trace.md b/ai-plan/public/analyzer-warning-reduction/traces/analyzer-warning-reduction-trace.md index 334eb204..ce044c93 100644 --- a/ai-plan/public/analyzer-warning-reduction/traces/analyzer-warning-reduction-trace.md +++ b/ai-plan/public/analyzer-warning-reduction/traces/analyzer-warning-reduction-trace.md @@ -1,229 +1,45 @@ # Analyzer Warning Reduction 追踪 -## 2026-04-28 — RP-088 +## 2026-04-28 — RP-089 -### 阶段：收敛 PR #300 的 open review threads 与 failed-test follow-up +### 阶段：复核 `PR #300` 最新 review 真值并收敛仍然成立的本地差异 - 触发背景： - - 用户执行 `$gframework-pr-review`，要求以当前分支 PR 真值为准核对 AI review / failed-test / linter 信号 - - `fetch_current_pr_review.py --json-output /tmp/current-pr-review.json` 返回 `PR #300`，latest head 上仍有 `8` 条 CodeRabbit open threads、`1` 个失败测试 `RegistryInitializationHookBaseTests.OnPhase_Should_Not_Throw_When_Registry_Not_Found`，以及 `dotnet-format` restore failed 的 CI 噪音 + - 用户再次执行 `$gframework-pr-review` + - `fetch_current_pr_review.py --json-output /tmp/current-pr-review.json` 返回 `PR #300`，latest head 仍显示 `6` 条 CodeRabbit open threads；其中 `Task.CompletedTask` 强转与 failed-test 信号已是 stale，本地仍需处理的是 `TestArchitectureContextV3` 事件语义、`RegistryInitializationHookBase.OnPhase` XML 异常文档、`CapturingLoggerFactoryProvider.MinLevel` 同步，以及 active trace 过长问题 - 主线程实施： - - 核对 `TestArchitectureContext` / `TestArchitectureContextV3` 后，修复共享事件总线与旧版命令/查询入口的静默成功问题，统一改为显式 `NotSupportedException` 或 faulted task - - 校正 `TestArchitectureWithRegistry` / `TestArchitectureWithoutRegistry` 的显式接口 `RegisterLifecycleHook`，使接口视角与公开 no-op 语义一致 - - 修复 `RegistryInitializationHookBase` 在注册表缺失场景下的 no-op 行为，并保持有注册表路径继续使用单实例 `GetUtility()` - - 收敛 `TestResourceLoader`、`TestLogger`、`CapturingLoggerFactoryProvider`、`DeterministicNotificationHandlerState`、`PartialGeneratedNotificationHandlerRegistry` 的判空、XML 文档、快照访问与并发语义说明 - - 新增 `TestArchitectureContextBehaviorTests.cs`，覆盖共享事件总线、旧入口失败契约与 `RegisterLifecycleHook` 接口行为 + - 将 `TestArchitectureContextV3` 的事件发送/注册/注销统一接入共享 `EventBus`，避免静默 no-op + - 为 `RegistryInitializationHookBase.OnPhase` 补充 `ArgumentNullException` 异常契约 + - 让 `CapturingLoggerFactoryProvider.MinLevel` 与 `CreateLogger` 共享同一把锁，并新增回归测试覆盖更新后的最小级别行为 + - 将 active trace 的 `RP-083` ~ `RP-088` 迁移到 [analyzer-warning-reduction-history-rp083-rp088.md](../archive/traces/analyzer-warning-reduction-history-rp083-rp088.md)，恢复单一恢复入口 - 验证里程碑： - `dotnet build GFramework.Core/GFramework.Core.csproj -c Release` - 结果：成功；`0 Warning(s)`、`0 Error(s)` - `dotnet build GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release` - 结果：成功；`0 Warning(s)`、`0 Error(s)` - - `dotnet test GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~RegistryInitializationHookBaseTests|FullyQualifiedName~WaitForMultipleEventsTests|FullyQualifiedName~ResourceManagerTests|FullyQualifiedName~LoggerTests|FullyQualifiedName~TestArchitectureContextBehaviorTests"` - - 结果：成功；`97` 通过、`0` 失败 - `dotnet build GFramework.Cqrs.Tests/GFramework.Cqrs.Tests.csproj -c Release` - - 结果：成功；`125 Warning(s)`、`0 Error(s)`；warning 全部来自既有 `Mediator/*` 文件，当前 helper 改动未新增 warning - - `dotnet test GFramework.Cqrs.Tests/GFramework.Cqrs.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~CqrsHandlerRegistrarTests"` + - 结果：成功；`125 Warning(s)`、`0 Error(s)`；warning 仍全部集中在既有 `Mediator/*` 文件 + - `dotnet test GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~TestArchitectureContextBehaviorTests|FullyQualifiedName~RegistryInitializationHookBaseTests"` - 结果：成功；`11` 通过、`0` 失败 -- 当前结论： - - `PR #300` 当前仍然成立的 open review threads 已在本地收敛，且 failed-test 信号已被现有测试回归覆盖 - - `Core` / `Core.Tests` 的本轮受影响项目均保持 `0 Warning(s)`，新增测试也覆盖了此前没有直接回归的测试替身行为 - - `GFramework.Cqrs.Tests` 仍保留 `125` 条既有 `Mediator/*` warning；这属于下一轮 warning reduction 波次，而不是本轮 PR review follow-up 的直接写集 -- 下一步： - 1. 提交本轮 PR review follow-up 与 `ai-plan` 同步。 - 2. 若继续处理 `Cqrs.Tests` warning，以下一轮单独规划 `Mediator/*` 波次为起点。 - -## 2026-04-28 — RP-087 - -### 阶段：按 `$gframework-batch-boot 50` 并行收敛 `Core.Tests` / `Cqrs.Tests` 低风险切片 - -- 触发背景： - - 用户再次要求先拿仓库根构建 warning，再把可切分的 warning 批次分派给多个 subagent，以降低主线程上下文压力 - - 当前分支与 `origin/main@6cc87a9` 无提交差异，适合从单文件和小型混合 warning 切片重新起步 -- 主线程实施： - - 先执行仓库规则要求的 non-incremental 基线：`dotnet restore GFramework.sln -p:RestoreFallbackFolders=` 以修复当前 WSL 环境里的旧 fallback 资产，再执行仓库根 `dotnet clean` + `dotnet build` - - 基线结果：`288 Warning(s)`、唯一位点 `214` - - 并行下发四个 disjoint worker： - - `GameContextTests.cs` - - `ArchitectureServicesTests.cs` - - `RegistryInitializationHookBaseTests.cs` - - `CqrsDispatcherCacheTests.cs` - - 主线程补齐不与 worker 重叠的零散切片： - - `ResourceManagerTests.cs` - - `TestEvent.cs` - - `EventListenerScopeTests.cs` - - `TestArchitectureBase.cs` - - `ContextProviderTests.cs` - - `LoggerTests.cs` - - `LoggingConfigurationTests.cs` - - `WaitForMultipleEventsTests.cs` - - `CommandCoroutineExtensionsTests.cs` - - 集成 `GameContextTests`、`CqrsDispatcherCacheTests` 与 `CqrsHandlerRegistrarTests` worker 产出的 commit `a7be413` / `9098490` / `98afcbf`，并接受已直接落到共享工作树的 `ArchitectureServicesTests` / `RegistryInitializationHookBaseTests` 切片 - - 重新执行仓库根 `dotnet clean` + `dotnet build` - - final 结果：`236 Warning(s)`、唯一位点 `162` -- 验证里程碑： - - `dotnet build GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release` - - 结果：成功；`0 Warning(s)`、`0 Error(s)` - - `dotnet build GFramework.Cqrs.Tests/GFramework.Cqrs.Tests.csproj -c Release` - - 结果：成功；`0 Warning(s)`、`0 Error(s)` - - `dotnet clean` - - 结果：成功 - - `dotnet build` - - 结果：成功；`236 Warning(s)`、`0 Error(s)`，唯一位点 `162` - - `git diff --name-only refs/remotes/origin/main...HEAD | wc -l` - - 结果：`21` - - 已提交 diff 与当前工作树变更并集文件数 - - 结果：`45 / 50` -- 当前结论： - - 本轮已经把剩余 `Core.Tests` warning 全部清空，并把 `CqrsDispatcherCacheTests.cs` 与 `CqrsHandlerRegistrarTests.cs` 这两处 `Cqrs.Tests` 单文件 `MA0048` 热点从仓库根中移除 - - 仓库根权威 warning 从 `288` 下降到 `236`，唯一位点从 `214` 下降到 `162` - - 在 `45 / 50` footprint 下，继续扩批将明显压缩 review 余量；剩余切片也只剩 `Mediator*` 与 `YamlConfigSchemaValidator*` 这两组高风险热点，因此本轮应在这里收口 -- 下一步： - 1. 提交本轮已验证的 warning reduction 与 `ai-plan` 同步。 - 2. 下一轮在新提交基础上单独规划 `Mediator*` 波次。 - 3. 将 `YamlConfigSchemaValidator*` 保持为独立高耦合波次，再决定是否需要新的并行切法。 - -## 2026-04-27 — RP-086 - -### 阶段：收敛 PR #298 的 CodeRabbit nitpick follow-up - -- 触发背景： - - 用户再次执行 `$gframework-pr-review` 后，要求按 `PR #298` 的 nitpick comments 收敛仍然适用的问题 - - 复核 PR 真值后确认当前分支无 failed checks、无 open review threads，但仍有一批测试辅助类型的可维护性 nitpick 值得本地落地 -- 主线程实施： - - 校验 `TestStateMachineSystemV5`、`ComplexQuery`、`TrackingPipelineBehavior`、`TestEnvironment`、`TestContextUtilityV1/V2` 等改动后，分别修复可变内部状态暴露、`_context!` 空抑制、静态计数器非原子递增、`new Register(...)` 测试辅助入口和生命周期标记公开 setter 问题 - - 统一更新 `TestQueryV2`、`TestCommandWithResultV2`、`TestAsyncQueryInput`、`TestAsyncQueryResult*` 的 XML 文档，使 `init` 属性语义与文档一致 - - 将三倍结果属性从 `DoubleValue` 更名为 `TripleValue`，同步更新 `TestAsyncComplexQuery*` 与相关断言，避免名称与 `* 3` 的行为不一致 - - 精简 active tracking，移除重复的 `GFramework.Core.Tests` Release build 记录，并把该项目的当前真值修正为 `28 Warning(s)` -- 验证里程碑： - - `dotnet build GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release` - - 结果：成功；`28 Warning(s)`、`0 Error(s)` - - `dotnet test GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release --no-build` - - 结果：成功；`1610` 通过、`0` 失败 -- 当前结论： - - `PR #298` 中仍然适用的低风险 nitpick 已完成收敛，且没有为当前 touched files 引入新的构建 warning 或测试回归 - - `GFramework.Core.Tests` 的剩余 warning 仍集中在 `GameContextTests.cs`、`ArchitectureServicesTests.cs`、`RegistryInitializationHookBaseTests.cs` 等既有热点，不属于本轮 nitpick follow-up 新引入问题 -- 下一步： - 1. 提交本轮 `PR #298` nitpick follow-up 与 `ai-plan` 同步。 - 2. 回到 `GameContextTests.cs` / `ArchitectureServicesTests.cs` 的 `CS8766` warning reduction 主线。 - -## 2026-04-27 — RP-085 - -### 阶段：按 `$gframework-batch-boot 100` 并行消化 `GFramework.Core.Tests` 低风险 `MA0048` - -- 触发背景： - - 用户要求以仓库根 non-incremental 构建 warning 为准，并在上下文可控前提下把小切片分派给多个 subagent 并行处理 - - 本轮开始时，当前分支与 `origin/main@7cfdd2c` 无提交差异，适合从纯 `MA0048` 单文件切片起步 -- 主线程实施： - - 执行权威基线：`dotnet clean` + 仓库根 `dotnet build` - - 初始结果：`353 Warning(s)`、唯一位点 `279` - - 分四波次并行下发 `GFramework.Core.Tests` 小切片，累计完成 `20+` 个文件的测试辅助类型拆分 - - 主线程持续复核共享工作树、处理并发编译阻断，并在每一轮后复跑 `GFramework.Core.Tests` Release 构建 - - 在工作树达到 `61` 个变更条目时主动停止扩批，保留对 `100` 文件停止线的充分余量 -- 代表性已落地切片： - - `ArchitectureContextTests.cs` - - `AsyncQueryExecutorTests.cs` - - `CommandExecutorTests.cs` - - `StateTests.cs` - - `StateMachineTests.cs` - - `StateMachineSystemTests.cs` - - `ArchitectureModulesBehaviorTests.cs` - - `ArchitectureAdditionalCqrsHandlersTests.cs` - - `QueryCoroutineExtensionsTests.cs` - - `ObjectPoolTests.cs` - - `AbstractContextUtilityTests.cs` - - `EnvironmentTests.cs` - - `EventBusTests.cs` - - `ContextAwareTests.cs` -- 验证里程碑： - - `dotnet build GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release` - - 结果：成功；`0 Warning(s)`、`0 Error(s)` - - `dotnet clean` - - 结果：成功 - - `dotnet build` - - 结果：成功；`288 Warning(s)`、`0 Error(s)`，唯一位点 `214` -- 当前结论： - - 本轮主要收益来自 `GFramework.Core.Tests` 内的纯 `MA0048` 大范围收敛 - - 仓库根权威 warning 已从 `353` 降到 `288`，唯一位点从 `279` 降到 `214` - - 下一波不再适合继续盲目平铺纯拆分，因为剩余 `GFramework.Core.Tests` 热点已开始混入 `CS8766` / `MA0016` -- 下一步： - 1. 提交本轮 warning reduction 与 `ai-plan` 同步。 - 2. 下一波优先由主线程处理 `GameContextTests.cs` / `ArchitectureServicesTests.cs` 的混合 warning。 - 3. 保持 `YamlConfigSchemaValidator*` 与 `GFramework.Cqrs.Tests/Mediator/*` 为独立高风险波次。 - -## 2026-04-27 — RP-084 - -### 阶段：收敛 PR #297 的 CodeRabbit follow-up - -- 触发背景： - - 用户执行 `$gframework-pr-review`，要求以当前分支对应 PR 为准，提取并核对 AI review / check 信号 - - `fetch_current_pr_review.py` 返回 PR `#297` 的最新 head review 中仍有 `3` 个 open threads，另有 `2` 个 folded nitpick 仍然适用 -- 主线程实施： - - 校验 `GFramework.Game/Config/YamlConfigLoader.cs` 后，保留 `ReadYamlAsync` 的原始取消语义，并把 `IntegerTryParseDelegate` 第一个参数改为 `string?` - - 校验 `GFramework.Core.Tests/Ioc/*` 与 `Query/TestAsyncQueryWithExceptionV4.cs` 后，补齐缺失 XML 文档，让 `IPrioritizedService` 继承 `IMixedService` 复用 `Name` 契约，并补上 `` 文档 - - 新增 `YamlConfigLoaderTests.ReadYamlAsync_Should_Preserve_OperationCanceledException_When_Cancellation_Is_Requested`，用反射直接命中私有读取路径，稳定回归本次取消语义修复 - - 用 `dotnet format --verify-no-changes --include ...` 清理并验证本次改动文件的格式状态 -- 验证里程碑： - - `dotnet test GFramework.Game.Tests/GFramework.Game.Tests.csproj -c Release --filter "FullyQualifiedName~YamlConfigLoaderTests.ReadYamlAsync_Should_Preserve_OperationCanceledException_When_Cancellation_Is_Requested"` - - 结果：成功；`1` 通过、`0` 失败 - - `dotnet test GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release --filter "FullyQualifiedName~MicrosoftDiContainerTests.GetAllByPriority_Should_Sort_By_Priority_Ascending"` - - 结果：成功；`1` 通过、`0` 失败 - - `dotnet build GFramework.Game/GFramework.Game.csproj -c Release` - - 结果：成功；`0 Warning(s)`、`0 Error(s)` - - `dotnet build GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release` - - 结果：成功；`0 Warning(s)`、`0 Error(s)` - - `dotnet format GFramework.sln --verify-no-changes --include GFramework.Game/Config/YamlConfigLoader.cs GFramework.Game.Tests/Config/YamlConfigLoaderTests.cs GFramework.Core.Tests/Ioc/IMixedService.cs GFramework.Core.Tests/Ioc/IPrioritizedService.cs GFramework.Core.Tests/Ioc/PrioritizedService.cs GFramework.Core.Tests/Query/TestAsyncQueryWithExceptionV4.cs` - - 结果：成功 -- 当前结论： - - PR `#297` 当前仍然有效的 CodeRabbit open threads 与 folded nitpick 已在本地全部核对并收敛 - - 当前恢复点完成后，分支可以回到 `ArchitectureContextTests.cs` / `AsyncQueryExecutorTests.cs` / `YamlConfigSchemaValidator*` 的 warning reduction 主线 -- 下一步： - 1. 提交本轮 PR review follow-up。 - 2. 继续执行下一波 `MA0048` 小切片，优先避免一次性进入 `Mediator*` 的高 changed-file 风险波次。 - -## 2026-04-27 — RP-083 - -### 阶段：修复 `YamlConfigLoader` 单文件 warning，并拆分 `MicrosoftDiContainerTests` 的辅助类型 - -- 触发背景： - - 用户执行 `$gframework-batch-boot 50`，要求先拿仓库根构建 warning，再按 bounded slice 分派给不同 subagent 并持续推进 - - 当前分支在本轮开始时与 `origin/main@b6a9fef` 零提交差异，适合从低风险 warning slice 起步 -- 主线程实施： - - 先执行 non-incremental 仓库根基线：`dotnet clean` + `dotnet build`，得到 `397 Warning(s)` / `316` 个唯一位点 - - 主线程修复 `GFramework.Game/Config/YamlConfigLoader.cs` 的 `MA0051`、`MA0002` 与 `MA0158` - - 接受一个 worker batch：将 `GFramework.Core.Tests/Ioc/MicrosoftDiContainerTests.cs` 末尾的 `10` 个测试辅助接口/类拆分到 `Ioc/` 同目录独立文件 - - 接受第二波 worker 的已落地结果：将 `GFramework.Core.Tests/Query/AbstractAsyncQueryTests.cs` 末尾的 `7` 个测试辅助类型拆分到 `Query/` 同目录独立文件 - - 启动 `ArchitectureContextTests.cs` 候选 worker，但在共享工作树落地前主动停止，以避免本轮上下文与 review 面积继续膨胀 -- 验证里程碑： - - `dotnet build GFramework.Game/GFramework.Game.csproj -c Release` - - 结果：成功；`111 Warning(s)`、`0 Error(s)` - - 观察：构建输出未再报告 `GFramework.Game/Config/YamlConfigLoader.cs` - - `dotnet build GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release` - - 结果：成功；`0 Warning(s)`、`0 Error(s)` - - `dotnet clean` - - 结果：成功；刷新最终 non-incremental 仓库根 warning 基线 - - `dotnet build` - - 结果：成功；`353 Warning(s)`、`0 Error(s)`，唯一位点 `279` - - 观察：构建输出未再报告 `GFramework.Game/Config/YamlConfigLoader.cs`、`GFramework.Core.Tests/Ioc/MicrosoftDiContainerTests.cs` 与 `GFramework.Core.Tests/Query/AbstractAsyncQueryTests.cs` -- 当前结论： - - 本轮已完成一个主线程单文件 slice 和两个 worker 拆分 slice；仓库根 non-incremental warning 从 `397` 降到 `353` - - 当前共享工作树 footprint 为 `22` 个 changed files，仍低于 `$gframework-batch-boot 50` 的停止线 - - 下一波更适合继续处理 `7` 个 `MA0048` 的小文件，而不是立即进入 `Mediator*` 或 `YamlConfigSchemaValidator*` 的高耦合热点 + - `dotnet test GFramework.Cqrs.Tests/GFramework.Cqrs.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~CqrsHandlerRegistrarTests"` + - 结果：成功；`12` 通过、`0` 失败 ## 活跃风险 -- `GFramework.Cqrs.Tests/Mediator/*` 的 `MA0048` 位点密度很高，一次性拆分会迅速推高 changed-file 数。 - - 缓解措施：下一波优先继续拿 `7` warning 级别的小切片。 -- `YamlConfigSchemaValidator*` 仍然聚集多类高耦合 warning。 - - 缓解措施：继续维持为独立波次，不与测试项目拆分混提。 +- `GFramework.Cqrs.Tests/Mediator/*` 仍保留 `125` 条既有 warning。 + - 缓解措施：保持为下一轮独立 warning reduction 波次，不把当前 PR review follow-up 扩展到 `Mediator/*` 写集。 +- GitHub PR 上的 open threads 在本地提交前仍可能显示为未关闭。 + - 缓解措施：以当前工作树和定向验证作为真值，推送后再让 PR 线程重新比对最新 head。 ## 下一步 -1. 完成本轮 `YamlConfigLoader.cs`、`MicrosoftDiContainerTests.cs` 与 `ai-plan` 的提交。 -2. 下一波优先从 `ArchitectureContextTests.cs` 或 `AsyncQueryExecutorTests.cs` 继续拆分纯 `MA0048`。 +1. 提交本轮 `PR #300` follow-up 与 `ai-plan` 同步。 +2. 若继续推进 warning reduction，下一轮单独规划 `Mediator/*` 切片。 ## 历史归档指针 - 最新 trace 归档： + - [analyzer-warning-reduction-history-rp083-rp088.md](../archive/traces/analyzer-warning-reduction-history-rp083-rp088.md) - [analyzer-warning-reduction-history-rp073-rp078.md](../archive/traces/analyzer-warning-reduction-history-rp073-rp078.md) - [analyzer-warning-reduction-history-rp062-rp071.md](../archive/traces/analyzer-warning-reduction-history-rp062-rp071.md) - 历史 todo 归档：