mirror of
https://github.com/GeWuYou/GFramework.git
synced 2026-05-08 17:44:29 +08:00
fix(cqrs): 收口 PR339 流式管道评审问题
- 修复 MicrosoftDiContainer 中 request 与 stream 行为注册逻辑的重复实现并统一校验路径 - 补充流式管道注册入口与 continuation 缓存的 XML 契约说明,明确并发与冻结前调用约束 - 更新 cqrs-rewrite 跟踪文档并修正 ICqrsRequestInvokerProvider 的 XML 缩进格式问题
This commit is contained in:
parent
aebf1e974d
commit
e44c56fb46
@ -90,6 +90,11 @@ public interface IArchitecture : IAsyncInitializable, IAsyncDestroyable, IInitia
|
||||
/// 也支持绑定到单一流式请求/响应对的封闭行为类型。
|
||||
/// </summary>
|
||||
/// <typeparam name="TBehavior">行为类型,必须是引用类型</typeparam>
|
||||
/// <exception cref="InvalidOperationException">当前架构的底层容器已冻结,无法继续注册流式管道行为。</exception>
|
||||
/// <exception cref="ObjectDisposedException">当前架构的底层容器已释放,无法继续注册流式管道行为。</exception>
|
||||
/// <remarks>
|
||||
/// 该入口应在架构初始化冻结容器之前调用;具体开放泛型或封闭行为类型的校验逻辑由底层容器负责。
|
||||
/// </remarks>
|
||||
void RegisterCqrsStreamPipelineBehavior<TBehavior>()
|
||||
where TBehavior : class;
|
||||
|
||||
|
||||
@ -109,6 +109,13 @@ public interface IIocContainer : IContextAware, IDisposable
|
||||
/// 注册 CQRS 流式请求管道行为。
|
||||
/// </summary>
|
||||
/// <typeparam name="TBehavior">行为类型,必须是引用类型</typeparam>
|
||||
/// <exception cref="InvalidOperationException">容器已冻结,无法继续注册流式管道行为。</exception>
|
||||
/// <exception cref="ObjectDisposedException">容器已释放,无法继续注册流式管道行为。</exception>
|
||||
/// <remarks>
|
||||
/// 该入口既支持实现 <c>IStreamPipelineBehavior<,></c> 的开放泛型行为类型,
|
||||
/// 也支持绑定到单一流式请求/响应对的封闭行为类型。
|
||||
/// 应在容器冻结前的注册阶段调用;具体可注册形态由实现容器负责校验。
|
||||
/// </remarks>
|
||||
void RegisterCqrsStreamPipelineBehavior<TBehavior>()
|
||||
where TBehavior : class;
|
||||
|
||||
|
||||
@ -182,6 +182,11 @@ public abstract class Architecture : IArchitecture
|
||||
/// 可以传入开放泛型行为类型,也可以传入绑定到特定流式请求的封闭行为类型。
|
||||
/// </summary>
|
||||
/// <typeparam name="TBehavior">行为类型,必须是引用类型</typeparam>
|
||||
/// <exception cref="InvalidOperationException">当前架构的底层容器已冻结,无法继续注册流式管道行为。</exception>
|
||||
/// <exception cref="ObjectDisposedException">当前架构的底层容器已释放,无法继续注册流式管道行为。</exception>
|
||||
/// <remarks>
|
||||
/// 该调用会委托到底层容器完成校验与注册,因此应在初始化冻结前完成所有流式行为接线。
|
||||
/// </remarks>
|
||||
public void RegisterCqrsStreamPipelineBehavior<TBehavior>() where TBehavior : class
|
||||
{
|
||||
_modules.RegisterCqrsStreamPipelineBehavior<TBehavior>();
|
||||
|
||||
@ -32,6 +32,8 @@ internal sealed class ArchitectureModules(
|
||||
/// 支持开放泛型行为类型和针对单一流式请求的封闭行为类型。
|
||||
/// </summary>
|
||||
/// <typeparam name="TBehavior">行为类型,必须是引用类型</typeparam>
|
||||
/// <exception cref="InvalidOperationException">底层容器已冻结,无法继续注册流式管道行为。</exception>
|
||||
/// <exception cref="ObjectDisposedException">底层容器已释放,无法继续注册流式管道行为。</exception>
|
||||
public void RegisterCqrsStreamPipelineBehavior<TBehavior>() where TBehavior : class
|
||||
{
|
||||
logger.Debug($"Registering CQRS stream pipeline behavior: {typeof(TBehavior).Name}");
|
||||
|
||||
@ -485,36 +485,11 @@ public class MicrosoftDiContainer(IServiceCollection? serviceCollection = null)
|
||||
try
|
||||
{
|
||||
ThrowIfFrozen();
|
||||
|
||||
var behaviorType = typeof(TBehavior);
|
||||
|
||||
if (behaviorType.IsGenericTypeDefinition)
|
||||
{
|
||||
GetServicesUnsafe.AddSingleton(typeof(IPipelineBehavior<,>), behaviorType);
|
||||
}
|
||||
else
|
||||
{
|
||||
var pipelineInterfaces = behaviorType
|
||||
.GetInterfaces()
|
||||
.Where(type => type.IsGenericType &&
|
||||
type.GetGenericTypeDefinition() == typeof(IPipelineBehavior<,>))
|
||||
.ToList();
|
||||
|
||||
if (pipelineInterfaces.Count == 0)
|
||||
{
|
||||
var errorMessage = $"{behaviorType.Name} does not implement IPipelineBehavior<,>";
|
||||
_logger.Error(errorMessage);
|
||||
throw new InvalidOperationException(errorMessage);
|
||||
}
|
||||
|
||||
// 为每个已闭合的管道接口建立显式映射,支持针对特定请求/响应的专用行为。
|
||||
foreach (var pipelineInterface in pipelineInterfaces)
|
||||
{
|
||||
GetServicesUnsafe.AddSingleton(pipelineInterface, behaviorType);
|
||||
}
|
||||
}
|
||||
|
||||
_logger.Debug($"CQRS pipeline behavior registered: {behaviorType.Name}");
|
||||
RegisterCqrsPipelineBehaviorCore(
|
||||
typeof(TBehavior),
|
||||
typeof(IPipelineBehavior<,>),
|
||||
"IPipelineBehavior<,>",
|
||||
"pipeline behavior");
|
||||
}
|
||||
finally
|
||||
{
|
||||
@ -535,36 +510,11 @@ public class MicrosoftDiContainer(IServiceCollection? serviceCollection = null)
|
||||
try
|
||||
{
|
||||
ThrowIfFrozen();
|
||||
|
||||
var behaviorType = typeof(TBehavior);
|
||||
|
||||
if (behaviorType.IsGenericTypeDefinition)
|
||||
{
|
||||
GetServicesUnsafe.AddSingleton(typeof(IStreamPipelineBehavior<,>), behaviorType);
|
||||
}
|
||||
else
|
||||
{
|
||||
var pipelineInterfaces = behaviorType
|
||||
.GetInterfaces()
|
||||
.Where(type => type.IsGenericType &&
|
||||
type.GetGenericTypeDefinition() == typeof(IStreamPipelineBehavior<,>))
|
||||
.ToList();
|
||||
|
||||
if (pipelineInterfaces.Count == 0)
|
||||
{
|
||||
var errorMessage = $"{behaviorType.Name} does not implement IStreamPipelineBehavior<,>";
|
||||
_logger.Error(errorMessage);
|
||||
throw new InvalidOperationException(errorMessage);
|
||||
}
|
||||
|
||||
// 为每个已闭合的流式管道接口建立显式映射,支持针对特定流式请求/响应的专用行为。
|
||||
foreach (var pipelineInterface in pipelineInterfaces)
|
||||
{
|
||||
GetServicesUnsafe.AddSingleton(pipelineInterface, behaviorType);
|
||||
}
|
||||
}
|
||||
|
||||
_logger.Debug($"CQRS stream pipeline behavior registered: {behaviorType.Name}");
|
||||
RegisterCqrsPipelineBehaviorCore(
|
||||
typeof(TBehavior),
|
||||
typeof(IStreamPipelineBehavior<,>),
|
||||
"IStreamPipelineBehavior<,>",
|
||||
"stream pipeline behavior");
|
||||
}
|
||||
finally
|
||||
{
|
||||
@ -572,6 +522,50 @@ public class MicrosoftDiContainer(IServiceCollection? serviceCollection = null)
|
||||
}
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// 复用 CQRS 行为注册的开放泛型/封闭接口校验逻辑,
|
||||
/// 让 request 与 stream 两条入口保持一致的容器注册语义。
|
||||
/// </summary>
|
||||
/// <param name="behaviorType">待注册的行为运行时类型。</param>
|
||||
/// <param name="openGenericInterfaceType">行为必须实现的开放泛型接口类型。</param>
|
||||
/// <param name="interfaceTypeDisplayName">用于日志与异常的接口显示名称。</param>
|
||||
/// <param name="registrationLabel">用于日志的注册类别名称。</param>
|
||||
/// <exception cref="InvalidOperationException"><paramref name="behaviorType" /> 未实现目标行为接口。</exception>
|
||||
private void RegisterCqrsPipelineBehaviorCore(
|
||||
Type behaviorType,
|
||||
Type openGenericInterfaceType,
|
||||
string interfaceTypeDisplayName,
|
||||
string registrationLabel)
|
||||
{
|
||||
if (behaviorType.IsGenericTypeDefinition)
|
||||
{
|
||||
GetServicesUnsafe.AddSingleton(openGenericInterfaceType, behaviorType);
|
||||
}
|
||||
else
|
||||
{
|
||||
var pipelineInterfaces = behaviorType
|
||||
.GetInterfaces()
|
||||
.Where(type => type.IsGenericType &&
|
||||
type.GetGenericTypeDefinition() == openGenericInterfaceType)
|
||||
.ToList();
|
||||
|
||||
if (pipelineInterfaces.Count == 0)
|
||||
{
|
||||
var errorMessage = $"{behaviorType.Name} does not implement {interfaceTypeDisplayName}";
|
||||
_logger.Error(errorMessage);
|
||||
throw new InvalidOperationException(errorMessage);
|
||||
}
|
||||
|
||||
// 为每个已闭合的行为接口建立显式映射,支持针对特定请求/响应对的专用行为。
|
||||
foreach (var pipelineInterface in pipelineInterfaces)
|
||||
{
|
||||
GetServicesUnsafe.AddSingleton(pipelineInterface, behaviorType);
|
||||
}
|
||||
}
|
||||
|
||||
_logger.Debug($"CQRS {registrationLabel} registered: {behaviorType.Name}");
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// 从指定程序集显式注册 CQRS 处理器。
|
||||
/// </summary>
|
||||
|
||||
@ -19,8 +19,8 @@ namespace GFramework.Cqrs;
|
||||
public interface ICqrsRequestInvokerProvider
|
||||
{
|
||||
/// <summary>
|
||||
/// 尝试为指定请求/响应类型对提供运行时元数据。
|
||||
/// </summary>
|
||||
/// 尝试为指定请求/响应类型对提供运行时元数据。
|
||||
/// </summary>
|
||||
/// <param name="requestType">请求运行时类型。</param>
|
||||
/// <param name="responseType">响应运行时类型。</param>
|
||||
/// <param name="descriptor">命中时返回的 request invoker 元数据。</param>
|
||||
|
||||
@ -1035,6 +1035,8 @@ internal sealed class CqrsDispatcher(
|
||||
/// <summary>
|
||||
/// 获取指定阶段的 continuation,并在首次请求时为该阶段绑定一次不可变调用入口。
|
||||
/// 同一行为多次调用 <c>next</c> 时会命中相同 continuation,保持与 request pipeline 一致的链式语义。
|
||||
/// 线程模型上,该缓存仅假定单次建流链按顺序推进;若某个 behavior 并发调用多个 <c>next</c>,
|
||||
/// 这里可能重复创建等价 continuation,但不会跨建流共享,也不会缓存容器解析出的实例。
|
||||
/// </summary>
|
||||
private StreamMessageHandlerDelegate<TRequest, TResponse> GetContinuation(int index)
|
||||
{
|
||||
|
||||
@ -7,9 +7,9 @@ CQRS 迁移与收敛。
|
||||
|
||||
## 当前恢复点
|
||||
|
||||
- 恢复点编号:`CQRS-REWRITE-RP-099`
|
||||
- 恢复点编号:`CQRS-REWRITE-RP-100`
|
||||
- 当前阶段:`Phase 8`
|
||||
- 当前 PR 锚点:`PR #334`
|
||||
- 当前 PR 锚点:`PR #339`
|
||||
- 当前结论:
|
||||
- `GFramework.Cqrs` 已完成对外部 `Mediator` 的生产级替代,当前主线已从“是否可替代”转向“仓库内部收口与能力深化顺序”
|
||||
- `dispatch/invoker` 生成前移已扩展到 request / stream 路径,`RP-077` 已补齐 request invoker provider gate 与 stream gate 对称的 descriptor / descriptor entry runtime 合同回归
|
||||
@ -34,14 +34,15 @@ CQRS 迁移与收敛。
|
||||
- `RP-096` 已再次使用 `$gframework-pr-review` 复核 `PR #334` latest-head review,确认仍显示为 open 的 AI threads 在本地代码中已无新增仍成立的运行时 / 测试 / 文档缺陷,剩余差异主要是 GitHub thread 未 resolve 的状态滞后
|
||||
- `RP-097` 已继续收口 `PR #334` latest-head nitpick:为 `AsyncQueryExecutorTests` / `CommandExecutorTests` 补齐可观察的上下文保留断言,并让 `RecordingCqrsRuntime` 在测试替身返回错误响应类型时抛出带请求/类型信息的诊断异常
|
||||
- 当前 `RP-098` 已再次使用 `$gframework-pr-review` 复核 `PR #334` latest-head review,并收口 `LegacyCqrsDispatchHelper.TryResolveDispatchContext(...)` 过宽吞掉 `InvalidOperationException` 的真实运行时诊断退化问题;现在仅把“上下文尚未就绪”视为允许 fallback 的信号,并为 fallback / 异常冒泡分别补齐回归测试
|
||||
- 当前 `RP-099` 已补齐 `GFramework.Cqrs` 的最小 stream pipeline seam:新增 `IStreamPipelineBehavior<,>` / `StreamMessageHandlerDelegate<,>`、`RegisterCqrsStreamPipelineBehavior<TBehavior>()`、dispatcher 侧 stream pipeline executor 缓存与 generated stream invoker 兼容回归,以及 `Architecture` 公开注册入口与对应文档说明
|
||||
- `ai-plan` active 入口现以 `RP-099` 为最新恢复锚点;`PR #334`、`PR #331`、`PR #326`、`PR #323`、`PR #307` 与其他更早阶段细节均以下方归档或说明为准
|
||||
- `RP-099` 已补齐 `GFramework.Cqrs` 的最小 stream pipeline seam:新增 `IStreamPipelineBehavior<,>` / `StreamMessageHandlerDelegate<,>`、`RegisterCqrsStreamPipelineBehavior<TBehavior>()`、dispatcher 侧 stream pipeline executor 缓存与 generated stream invoker 兼容回归,以及 `Architecture` 公开注册入口与对应文档说明
|
||||
- 当前 `RP-100` 已使用 `$gframework-pr-review` 复核 `PR #339` latest-head review:收口 `RegisterCqrsStreamPipelineBehavior<TBehavior>()` 的异常契约文档、为 `StreamPipelineInvocation.GetContinuation(...)` 补齐并发 continuation 缓存说明、抽取 `MicrosoftDiContainer` 的 CQRS 行为注册公共逻辑,并顺手修复当前 branch diff 内 `ICqrsRequestInvokerProvider.cs` 的 XML 缩进格式问题
|
||||
- `ai-plan` active 入口现以 `RP-100` 为最新恢复锚点;`PR #339`、`PR #334`、`PR #331`、`PR #326`、`PR #323`、`PR #307` 与其他更早阶段细节均以下方归档或说明为准
|
||||
|
||||
## 当前活跃事实
|
||||
|
||||
- 当前分支为 `feat/cqrs-optimization`
|
||||
- 本轮 `$gframework-batch-boot 50` 以 `origin/main` (`2c58d8b6`, 2026-05-07 13:24:46 +0800) 为基线;本地 `main` (`c2d22285`) 已落后,不作为 branch diff 基线
|
||||
- 当前分支相对 `origin/main` 的累计 branch diff 为 `0 files / 0 lines`,仍明显低于 `$gframework-batch-boot 50` 的文件 / 行数阈值
|
||||
- 当前分支相对 `origin/main` 的累计 branch diff 为 `31 files changed, 1176 insertions(+), 15 deletions(-)`,仍明显低于 `$gframework-batch-boot 50` 的文件 / 行数阈值
|
||||
- `GFramework.Cqrs.Benchmarks` 作为 benchmark 基础设施项目,必须持续排除在 NuGet / GitHub Packages 发布集合之外
|
||||
- `GFramework.Cqrs.Benchmarks` 现已覆盖 request steady-state、pipeline 数量矩阵、startup、request/stream generated invoker,以及 request handler `Singleton / Transient` 生命周期矩阵
|
||||
- `GFramework.Core` 当前已通过内部 bridge request / handler 把 legacy `ICommand`、`IAsyncCommand`、`IQuery`、`IAsyncQuery` 接到统一 `ICqrsRuntime`
|
||||
@ -69,6 +70,12 @@ CQRS 迁移与收敛。
|
||||
- `PR #334` 当前 latest-head open AI feedback 经过本轮本地复核与修复后,应主要剩余待 GitHub 重新索引的状态差异或已实质关闭但未 resolve 的 thread
|
||||
- `GFramework.Core.Tests` 中 legacy bridge 的“保留上下文”回归现在同时断言 bridge request 类型与目标对象执行期观察到的 `IArchitectureContext`
|
||||
- `RecordingCqrsRuntime` 对非 `Unit` 响应已显式校验返回值类型;若测试工厂返回了 `null` 或错误装箱类型,异常会直接指出 request 类型与期望/实际响应类型
|
||||
- `PR #339` 当前 latest-head review 仍显示 `2` 个 CodeRabbit open thread 与 `2` 个 nitpick;本轮本地复核后确认:
|
||||
- `GFramework.Cqrs.Tests/Cqrs/CqrsDispatcherCacheTests.cs` 的 `Per_Behavior_Count` 拼写已在当前 head 修正,属于 stale thread
|
||||
- `GFramework.Core.Abstractions/Ioc/IIocContainer.cs` 的流式行为注册入口此前确实缺少 `<exception>` / `<remarks>` 契约说明,现已补齐并同步到 `IArchitecture` / `Architecture` / `ArchitectureModules`
|
||||
- `GFramework.Cqrs/Internal/CqrsDispatcher.cs` 的 `StreamPipelineInvocation.GetContinuation(...)` 线程模型说明此前少于 request 对称路径,现已补齐并发 `next()` 时 continuation 缓存的语义边界
|
||||
- `GFramework.Core/Ioc/MicrosoftDiContainer.cs` 的 request / stream 行为注册逻辑此前存在重复实现,现已抽取共享私有 helper 以避免后续生命周期或校验逻辑漂移
|
||||
- 本地 `dotnet format GFramework.Cqrs/GFramework.Cqrs.csproj --verify-no-changes` 显示当前 diff 内仍有 `GFramework.Cqrs/ICqrsRequestInvokerProvider.cs` 的空白格式问题,本轮已修复;同一次命令报出的多条 `CHARSET` 提示集中在未触达的历史文件,不视为 `PR #339` 本轮新增 triage 结论
|
||||
|
||||
## 当前风险
|
||||
|
||||
@ -81,6 +88,7 @@ CQRS 迁移与收敛。
|
||||
- legacy bridge 当前只为已有 `Command` / `Query` 兼容入口接到统一 request pipeline;若后续要继续对齐 `Mediator`,仍需要单独设计 stream pipeline、telemetry 与 facade 公开面,而不是把这次 bridge 当成“全部收口完成”
|
||||
- `LegacyBridgePipelineTracker` 仍是进程级静态测试辅助;虽然现在已在相关 fixture 清理阶段重置并补充线程安全说明,但若将来扩大并行 bridge fixture 数量,仍要继续控制共享状态扩散
|
||||
- stream pipeline 当前只在“单次建流”层面包裹 handler 调用;若后续需要 per-item 拦截、元素级重试或流内 metrics 聚合,仍需额外设计更细粒度 contract,而不是把本轮 seam 直接等同于元素级 middleware
|
||||
- `PR #339` 在 GitHub 上仍有 1 个已本地失效但未 resolve 的 stale test-thread;若后续 head 再次变化,需要重新抓取 latest-head review 确认未解决线程是否收敛
|
||||
|
||||
## 最近权威验证
|
||||
|
||||
@ -105,6 +113,25 @@ CQRS 迁移与收敛。
|
||||
- 备注:确认当前分支对应 `PR #334`;`CodeRabbit` latest review 已 `APPROVED`,但 latest-head 仍显示 `10` 个 open thread、`Greptile` 仍显示 `3` 个 open thread;本地逐项复核后未发现新的仍成立缺陷,最新 CI 测试汇总为 `2311/2311` passed,`MegaLinter` 仅剩 `dotnet-format` restore 环境噪音
|
||||
- `dotnet build GFramework.Core/GFramework.Core.csproj -c Release`
|
||||
- 结果:通过,`0 warning / 0 error`
|
||||
- `python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --json-output /tmp/current-pr-review.json`
|
||||
- 结果:通过
|
||||
- 备注:确认当前分支对应 `PR #339`;latest-head 显示 `2` 个 CodeRabbit open thread 与 `2` 个 nitpick;本轮本地接受并修复的问题集中在流式行为注册入口 XML 契约、stream continuation 线程说明与 `MicrosoftDiContainer` 的重复注册逻辑,测试方法拼写线程已在当前 head 失效
|
||||
- `dotnet format GFramework.Cqrs/GFramework.Cqrs.csproj --verify-no-changes`
|
||||
- 结果:发现当前 diff 内 `GFramework.Cqrs/ICqrsRequestInvokerProvider.cs` 的空白格式问题;其余 `CHARSET` 提示集中在未触达的历史文件
|
||||
- `dotnet build GFramework.Cqrs/GFramework.Cqrs.csproj -c Release`
|
||||
- 结果:通过,`0 warning / 0 error`
|
||||
- `dotnet build GFramework.Core.Abstractions/GFramework.Core.Abstractions.csproj -c Release`
|
||||
- 结果:通过,`0 warning / 0 error`
|
||||
- `dotnet build GFramework.Core/GFramework.Core.csproj -c Release`
|
||||
- 结果:通过,`0 warning / 0 error`
|
||||
- `dotnet test GFramework.Cqrs.Tests/GFramework.Cqrs.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~CqrsDispatcherCacheTests"`
|
||||
- 结果:通过,`10/10` passed
|
||||
- `dotnet test GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~ArchitectureModulesBehaviorTests"`
|
||||
- 结果:通过,`4/4` passed
|
||||
- `env GIT_DIR=... GIT_WORK_TREE=... python3 scripts/license-header.py --check`
|
||||
- 结果:通过
|
||||
- `git diff --check`
|
||||
- 结果:通过
|
||||
- `dotnet build GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release`
|
||||
- 结果:通过,`0 warning / 0 error`
|
||||
- `dotnet test GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release --filter "FullyQualifiedName~CommandExecutorTests|FullyQualifiedName~AsyncQueryExecutorTests"`
|
||||
@ -177,9 +204,9 @@ CQRS 迁移与收敛。
|
||||
|
||||
## 下一推荐步骤
|
||||
|
||||
1. 若继续沿用 `$gframework-batch-boot 50` 且优先处理 `Mediator` 能力吸收,下一批建议从 `notification publisher` 策略或 facade 公开入口中选择一个独立切片推进
|
||||
2. 若后续要增强 stream observability,优先评估是否需要元素级 hook,而不是直接复用当前建流级 seam 承载更多语义
|
||||
3. 在 GitHub 上 resolve / reply 已被当前分支实质吸收的 `PR #334` stale review threads;若 head 更新后线程数量继续变化,再用 `$gframework-pr-review` 复核
|
||||
1. 在 GitHub 上 resolve / reply 已被当前分支实质吸收的 `PR #339` stale review threads;若 head 更新后线程数量继续变化,再用 `$gframework-pr-review` 复核
|
||||
2. 若继续沿用 `$gframework-batch-boot 50` 且优先处理 `Mediator` 能力吸收,下一批建议从 `notification publisher` 策略或 facade 公开入口中选择一个独立切片推进
|
||||
3. 若后续要增强 stream observability,优先评估是否需要元素级 hook,而不是直接复用当前建流级 seam 承载更多语义
|
||||
|
||||
## 活跃文档
|
||||
|
||||
|
||||
@ -2,6 +2,40 @@
|
||||
|
||||
## 2026-05-07
|
||||
|
||||
### 阶段:PR #339 stream pipeline seam review 收口(CQRS-REWRITE-RP-100)
|
||||
|
||||
- 使用 `$gframework-pr-review` 抓取 `feat/cqrs-optimization` 当前公开 PR,并确认已从历史 `PR #334` 进入新的 `PR #339`
|
||||
- 本轮 latest-head review 结论:
|
||||
- `CodeRabbit` 当前显示 `2` 个 open thread 与 `2` 个 nitpick,失败检查为 `0`,GitHub Test Reporter 汇总仍为全绿
|
||||
- `GFramework.Cqrs.Tests/Cqrs/CqrsDispatcherCacheTests.cs` 的 test-name 拼写 thread 已在当前 head 失效,本地代码已经是 `Per_Behavior_Count`
|
||||
- `GFramework.Core.Abstractions/Ioc/IIocContainer.cs` 的 `RegisterCqrsStreamPipelineBehavior<TBehavior>()` 仍缺 `<exception>` / `<remarks>` 契约说明,属于仍成立的文档缺口
|
||||
- `GFramework.Cqrs/Internal/CqrsDispatcher.cs` 的 `StreamPipelineInvocation.GetContinuation(...)` 缺少 request 对称路径已有的线程模型说明,属于仍成立的并发语义文档缺口
|
||||
- `GFramework.Core/Ioc/MicrosoftDiContainer.cs` 的 request / stream CQRS 行为注册逻辑完全重复,属于仍成立的维护性问题
|
||||
- 本轮主线程决策:
|
||||
- 在 `IIocContainer` 补齐流式行为注册入口的 `<exception>` / `<remarks>`,并把相同契约补到 `IArchitecture`、`Architecture` 与 `ArchitectureModules`,避免公开入口文档漂移
|
||||
- 为 `StreamPipelineInvocation.GetContinuation(...)` 补齐“仅假定单次建流链顺序推进;并发 `next()` 时可能重复创建等价 continuation,但不会跨建流共享实例”的说明
|
||||
- 在 `MicrosoftDiContainer` 抽取 `RegisterCqrsPipelineBehaviorCore(...)`,统一 request / stream 行为注册的开放泛型、封闭接口枚举、错误消息与日志路径
|
||||
- 顺手修复 `dotnet format` 在当前 branch diff 内实际命中的 `GFramework.Cqrs/ICqrsRequestInvokerProvider.cs` XML 缩进问题;不处理未触达历史文件上的 `CHARSET` 提示
|
||||
- 本轮权威验证:
|
||||
- `python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --json-output /tmp/current-pr-review.json`
|
||||
- 结果:通过
|
||||
- `dotnet format GFramework.Cqrs/GFramework.Cqrs.csproj --verify-no-changes`
|
||||
- 结果:发现当前 diff 内 `GFramework.Cqrs/ICqrsRequestInvokerProvider.cs` 的空白格式问题;其余 `CHARSET` 提示集中在未触达的历史文件
|
||||
- `dotnet build GFramework.Cqrs/GFramework.Cqrs.csproj -c Release`
|
||||
- 结果:通过,`0 warning / 0 error`
|
||||
- `dotnet build GFramework.Core.Abstractions/GFramework.Core.Abstractions.csproj -c Release`
|
||||
- 结果:通过,`0 warning / 0 error`
|
||||
- `dotnet build GFramework.Core/GFramework.Core.csproj -c Release`
|
||||
- 结果:通过,`0 warning / 0 error`
|
||||
- `dotnet test GFramework.Cqrs.Tests/GFramework.Cqrs.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~CqrsDispatcherCacheTests"`
|
||||
- 结果:通过,`10/10` passed
|
||||
- `dotnet test GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~ArchitectureModulesBehaviorTests"`
|
||||
- 结果:通过,`4/4` passed
|
||||
- `env GIT_DIR=... GIT_WORK_TREE=... python3 scripts/license-header.py --check`
|
||||
- 结果:通过
|
||||
- `git diff --check`
|
||||
- 结果:通过
|
||||
|
||||
### 阶段:stream pipeline seam 收口(CQRS-REWRITE-RP-099)
|
||||
|
||||
- 延续 `$gframework-batch-boot 50`,主线程先按 `origin/main` 评估 branch diff 容量,并在 `stream pipeline` 与 `notification publisher` 两个独立切片中选择更贴近 active gap 的下一批目标
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user