From 30ddb841a916c71117814409303d6abbf68d4011 Mon Sep 17 00:00:00 2001 From: gewuyou <95328647+GeWuYou@users.noreply.github.com> Date: Wed, 6 May 2026 21:12:42 +0800 Subject: [PATCH 1/2] =?UTF-8?q?fix(release):=20=E5=89=8D=E7=A7=BB=E5=8F=91?= =?UTF-8?q?=E5=B8=83=E5=8C=85=E6=B8=85=E5=8D=95=E6=A0=A1=E9=AA=8C?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - 修复 benchmark 项目误入发布面的风险,明确 GFramework.Cqrs.Benchmarks 保持不可打包。 - 新增共享 packed modules 校验脚本,并让 publish 与 CI 工作流复用同一份发布包名单规则。 - 更新 CQRS active tracking 与 trace,记录本轮发布校验前移的恢复点与验证结果。 --- .github/workflows/ci.yml | 12 +++++ .github/workflows/publish.yml | 36 +------------ .../GFramework.Cqrs.Benchmarks.csproj | 3 ++ .../todos/cqrs-rewrite-migration-tracking.md | 29 ++++++++--- .../traces/cqrs-rewrite-migration-trace.md | 16 ++++++ scripts/validate-packed-modules.sh | 51 +++++++++++++++++++ 6 files changed, 105 insertions(+), 42 deletions(-) create mode 100644 scripts/validate-packed-modules.sh diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 644e5cd4..c4b80392 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -155,6 +155,18 @@ jobs: - name: Build run: dotnet build GFramework.sln -c Release --no-restore + - name: Pack published modules + run: | + rm -rf ./packages + dotnet pack GFramework.sln \ + -c Release \ + --no-restore \ + -o ./packages \ + -p:IncludeSymbols=false + + - name: Validate packed modules + run: bash scripts/validate-packed-modules.sh ./packages + # 运行单元测试,输出TRX格式结果到TestResults目录 # 顺序执行各测试项目,避免并发 dotnet test 进程导致“TRX 全绿但 step 仍返回失败”的假红状态 - name: Test All Projects diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml index 500ac980..a576594c 100644 --- a/.github/workflows/publish.yml +++ b/.github/workflows/publish.yml @@ -82,41 +82,7 @@ jobs: -p:IncludeSymbols=false - name: Validate packed modules - run: | - set -euo pipefail - - expected_packages=( - "GeWuYou.GFramework" - "GeWuYou.GFramework.Core" - "GeWuYou.GFramework.Core.Abstractions" - "GeWuYou.GFramework.Core.SourceGenerators" - "GeWuYou.GFramework.Cqrs" - "GeWuYou.GFramework.Cqrs.Abstractions" - "GeWuYou.GFramework.Cqrs.SourceGenerators" - "GeWuYou.GFramework.Ecs.Arch" - "GeWuYou.GFramework.Ecs.Arch.Abstractions" - "GeWuYou.GFramework.Game" - "GeWuYou.GFramework.Game.Abstractions" - "GeWuYou.GFramework.Game.SourceGenerators" - "GeWuYou.GFramework.Godot" - "GeWuYou.GFramework.Godot.SourceGenerators" - ) - - mapfile -t actual_packages < <( - find ./packages -maxdepth 1 -type f -name '*.nupkg' -printf '%f\n' \ - | sed -E 's/\.[0-9][0-9A-Za-z.-]*\.nupkg$//' \ - | sort -u - ) - - printf '%s\n' "${expected_packages[@]}" | sort > expected-packages.txt - printf '%s\n' "${actual_packages[@]}" | sort > actual-packages.txt - - echo "Expected packages:" - cat expected-packages.txt - echo "Actual packages:" - cat actual-packages.txt - - diff -u expected-packages.txt actual-packages.txt + run: bash scripts/validate-packed-modules.sh ./packages - name: Validate runtime-generator package boundaries run: python3 scripts/validate-runtime-generator-boundaries.py --package-dir ./packages diff --git a/GFramework.Cqrs.Benchmarks/GFramework.Cqrs.Benchmarks.csproj b/GFramework.Cqrs.Benchmarks/GFramework.Cqrs.Benchmarks.csproj index c7d764fa..dbb621e5 100644 --- a/GFramework.Cqrs.Benchmarks/GFramework.Cqrs.Benchmarks.csproj +++ b/GFramework.Cqrs.Benchmarks/GFramework.Cqrs.Benchmarks.csproj @@ -11,6 +11,9 @@ disable enable false + + false + false diff --git a/ai-plan/public/cqrs-rewrite/todos/cqrs-rewrite-migration-tracking.md b/ai-plan/public/cqrs-rewrite/todos/cqrs-rewrite-migration-tracking.md index f7b6f7b9..93ac148e 100644 --- a/ai-plan/public/cqrs-rewrite/todos/cqrs-rewrite-migration-tracking.md +++ b/ai-plan/public/cqrs-rewrite/todos/cqrs-rewrite-migration-tracking.md @@ -7,9 +7,9 @@ CQRS 迁移与收敛。 ## 当前恢复点 -- 恢复点编号:`CQRS-REWRITE-RP-090` +- 恢复点编号:`CQRS-REWRITE-RP-091` - 当前阶段:`Phase 8` -- 当前 PR 锚点:`PR #326` +- 当前 PR 锚点:`待创建` - 当前结论: - `GFramework.Cqrs` 已完成对外部 `Mediator` 的生产级替代,当前主线已从“是否可替代”转向“仓库内部收口与能力深化顺序” - `dispatch/invoker` 生成前移已扩展到 request / stream 路径,`RP-077` 已补齐 request invoker provider gate 与 stream gate 对称的 descriptor / descriptor entry runtime 合同回归 @@ -26,11 +26,15 @@ CQRS 迁移与收敛。 - 当前 `RP-088` 已补齐 request invoker reflection / generated-provider 对照,开始直接量化 dispatcher 预热 generated descriptor 的收益 - 当前 `RP-089` 已补齐 stream invoker reflection / generated-provider 对照,使 generated descriptor 预热收益从 request 扩展到 stream 路径 - 当前 `RP-090` 已收敛 `PR #326` benchmark review:统一 benchmark 最小宿主构建、冻结 GFramework 容器、限制 MediatR 扫描范围,并恢复 request startup cold-start 对照 - - `ai-plan` active 入口现以 `PR #326` 和 `RP-090` 为唯一权威恢复锚点;`PR #323`、`PR #307` 与其他更早阶段细节均以下方归档或说明为准 + - 当前 `RP-091` 已把 benchmark 项目发布面隔离与包清单校验前移到 PR:`GFramework.Cqrs.Benchmarks` 明确保持不可打包,`publish.yml` 与 `ci.yml` 复用同一份 packed-modules 校验脚本 + - `ai-plan` active 入口现以 `RP-091` 为最新恢复锚点;`PR #326`、`PR #323`、`PR #307` 与其他更早阶段细节均以下方归档或说明为准 ## 当前活跃事实 -- 当前分支对应 `PR #326`,状态为 `OPEN` +- 当前分支为 `fix/package-validation-guard` +- `GFramework.Cqrs.Benchmarks` 作为 benchmark 基础设施项目,必须持续排除在 NuGet / GitHub Packages 发布集合之外 +- 发布工作流已有 packed modules 校验,但 PR 工作流此前没有等价的 solution pack 产物名单校验 +- 本地 `dotnet pack GFramework.sln -c Release --no-restore -o ` 当前只产出 14 个预期包,未复现 benchmark `.nupkg` - latest-head review 现仍有少量 open thread,但本地复核后,仍成立的问题已收敛到 benchmark 对照公平性、workflow 输入安全性与 active 文档压缩 - benchmark 场景现统一通过 `BenchmarkHostFactory` 构建最小宿主:GFramework 侧在 runtime 分发前显式 `Freeze()` 容器,MediatR 侧只扫描当前场景需要的 handler / behavior 类型 - `RequestStartupBenchmarks` 已恢复 `ColdStart_GFrameworkCqrs` 结果产出,不再命中 `No CQRS request handler registered` @@ -41,12 +45,23 @@ CQRS 迁移与收敛。 ## 当前风险 - 顶层 `GFramework.sln` / `GFramework.csproj` 在 WSL 下仍可能受 Windows NuGet fallback 配置影响,完整 solution 级验证成本高于模块级验证 +- 若后续新增 benchmark / example / tooling 项目但未同步校验发布面,solution 级 `dotnet pack` 仍可能在 tag 发布前才暴露异常包 - `RequestStartupBenchmarks` 为了量化真正的单次 cold-start,引入了 `InvocationCount=1` / `UnrollFactor=1` 的专用 job;该配置会触发 BenchmarkDotNet 的 `MinIterationTime` 提示,后续若要做稳定基线比较,还需要决定是否引入批量外层循环或自定义 cold-start harness - 仓库内部仍保留旧 `Command` / `Query` API、`LegacyICqrsRuntime` alias 与部分历史命名语义,后续若不继续分批收口,容易混淆“对外替代已完成”与“内部收口未完成” - 若继续扩大 generated invoker 覆盖面,需要持续区分“可静态表达的合同”与 `PreciseReflectedRegistrationSpec` 等仍需保守回退的场景 ## 最近权威验证 +- `dotnet pack GFramework.sln -c Release --no-restore -o /tmp/gframework-pack-validation -p:IncludeSymbols=false` + - 结果:通过 + - 备注:当前本地产物仅包含 14 个预期发布包,未生成 `GFramework.Cqrs.Benchmarks.*.nupkg` +- `bash scripts/validate-packed-modules.sh /tmp/gframework-pack-validation` + - 结果:通过 + - 备注:共享脚本确认 actual package set 与预期 14 个发布包完全一致 +- `dotnet build GFramework.Cqrs.Benchmarks/GFramework.Cqrs.Benchmarks.csproj -c Release` + - 结果:通过,`0 warning / 0 error` +- `python3 scripts/license-header.py --check` + - 结果:通过 - `dotnet run --project GFramework.Cqrs.Benchmarks/GFramework.Cqrs.Benchmarks.csproj -c Release -- --filter "*RequestStartupBenchmarks*" --job short --warmupCount 1 --iterationCount 1 --launchCount 1` - 结果:通过 - 备注:`ColdStart_GFrameworkCqrs` 已恢复出数,最新本地输出约 `220-292 us`,MediatR 对照约 `575-616 us`;当前仅剩 BenchmarkDotNet 对单次 cold-start 场景的 `MinIterationTime` 提示 @@ -64,9 +79,9 @@ CQRS 迁移与收敛。 ## 下一推荐步骤 -1. 重新运行 `$gframework-pr-review`,确认本轮 workflow / benchmark / active 文档修复是否已消化当前 latest-head open threads -2. 若 `PR #326` 仍剩基准语义类反馈,优先判断它们属于真实对照偏差还是有意保留的 benchmark 设计取舍 -3. 若需要在 CI 中手动复核 benchmark,继续使用 workflow 的 `benchmark_filter` 输入按场景筛选,避免默认运行整个 benchmark 矩阵 +1. 运行 `dotnet pack` 与新的 `scripts/validate-packed-modules.sh`,确认本轮共享校验脚本与 PR workflow 步骤在本地一致通过 +2. 运行受影响的 Release build / 头部校验,确认 workflow 与脚本改动未引入新的命名、文件头或 shell 语法问题 +3. 创建修复 PR 时,将重点放在“发布面保护前移到 PR”而不是“扩充 expected package 列表” ## 活跃文档 diff --git a/ai-plan/public/cqrs-rewrite/traces/cqrs-rewrite-migration-trace.md b/ai-plan/public/cqrs-rewrite/traces/cqrs-rewrite-migration-trace.md index 4c0fb380..76bf5572 100644 --- a/ai-plan/public/cqrs-rewrite/traces/cqrs-rewrite-migration-trace.md +++ b/ai-plan/public/cqrs-rewrite/traces/cqrs-rewrite-migration-trace.md @@ -2,6 +2,22 @@ ## 2026-05-06 +### 阶段:benchmark 发布面隔离与包清单校验前移(CQRS-REWRITE-RP-091) + +- 针对 tag 发布中出现的 `GFramework.Cqrs.Benchmarks` 异常包名单,本轮先复核 benchmark 项目与 solution pack 的本地事实: + - `GFramework.Cqrs.Benchmarks.csproj` 已包含 `IsPackable=false` 与 `GeneratePackageOnBuild=false` + - 本地执行 `dotnet pack GFramework.sln -c Release --no-restore -o /tmp/gframework-sln-pack-probe -p:IncludeSymbols=false` 时,产物仅包含 14 个预期发布包 + - 因此本轮不把 benchmark 包加入发布白名单,而是把“benchmark 永不发布”与“PR 前置完整包名单校验”同时固化 +- 本轮决策: + - 为 `GFramework.Cqrs.Benchmarks` 补充注释,明确其 benchmark-only 的发布边界 + - 新增 `scripts/validate-packed-modules.sh`,集中维护预期包集合与实际 `.nupkg` diff 逻辑 + - `publish.yml` 改为调用共享脚本,避免发布工作流与 PR 工作流各自维护一份包名单 + - `ci.yml` 新增 solution `dotnet pack` 与 packed modules 校验,把异常发布包从 tag 发布前移到普通 PR 阶段 +- 预期结果: + - benchmark / example / tooling 一类新项目若意外进入发布面,会先在 PR 失败,而不是等到 tag 发布 + - 发布与 PR 使用同一份包名单规则,减少后续名单漂移 + - `GFramework.Cqrs.Benchmarks` 继续只服务于 benchmark workflow,不进入 NuGet / GitHub Packages + ### 阶段:benchmark 对照宿主收敛与 startup cold-start 恢复(CQRS-REWRITE-RP-090) - 使用 `$gframework-pr-review` 拉取 `PR #326` latest-head review 后,主线程确认仍有效的 benchmark 反馈集中在三类问题: diff --git a/scripts/validate-packed-modules.sh b/scripts/validate-packed-modules.sh new file mode 100644 index 00000000..74afaab1 --- /dev/null +++ b/scripts/validate-packed-modules.sh @@ -0,0 +1,51 @@ +#!/usr/bin/env bash +# Copyright (c) 2025-2026 GeWuYou +# SPDX-License-Identifier: Apache-2.0 + +set -euo pipefail + +package_dir="${1:-./packages}" + +if [ ! -d "$package_dir" ]; then + echo "Package directory not found: $package_dir" >&2 + exit 1 +fi + +expected_packages=( + "GeWuYou.GFramework" + "GeWuYou.GFramework.Core" + "GeWuYou.GFramework.Core.Abstractions" + "GeWuYou.GFramework.Core.SourceGenerators" + "GeWuYou.GFramework.Cqrs" + "GeWuYou.GFramework.Cqrs.Abstractions" + "GeWuYou.GFramework.Cqrs.SourceGenerators" + "GeWuYou.GFramework.Ecs.Arch" + "GeWuYou.GFramework.Ecs.Arch.Abstractions" + "GeWuYou.GFramework.Game" + "GeWuYou.GFramework.Game.Abstractions" + "GeWuYou.GFramework.Game.SourceGenerators" + "GeWuYou.GFramework.Godot" + "GeWuYou.GFramework.Godot.SourceGenerators" +) + +work_dir="$(mktemp -d)" +trap 'rm -rf "$work_dir"' EXIT + +expected_file="$work_dir/expected-packages.txt" +actual_file="$work_dir/actual-packages.txt" + +mapfile -t actual_packages < <( + find "$package_dir" -maxdepth 1 -type f -name '*.nupkg' -printf '%f\n' \ + | sed -E 's/\.[0-9][0-9A-Za-z.-]*\.nupkg$//' \ + | sort -u +) + +printf '%s\n' "${expected_packages[@]}" | sort > "$expected_file" +printf '%s\n' "${actual_packages[@]}" > "$actual_file" + +echo "Expected packages:" +cat "$expected_file" +echo "Actual packages:" +cat "$actual_file" + +diff -u "$expected_file" "$actual_file" From e3d6aa51115a1c18a9fcc86d6c5e05e7d64247a2 Mon Sep 17 00:00:00 2001 From: gewuyou <95328647+GeWuYou@users.noreply.github.com> Date: Wed, 6 May 2026 21:27:21 +0800 Subject: [PATCH 2/2] =?UTF-8?q?fix(release):=20=E4=BF=AE=E5=A4=8D=E5=8F=91?= =?UTF-8?q?=E5=B8=83=E6=A0=A1=E9=AA=8C=E9=93=BE=E8=B7=AF=E7=9A=84=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 - 修复 PR workflow 中 dotnet pack 重复构建整个 solution 的问题 - 优化 packed modules 校验脚本的 find 实现以兼容 BSD 环境 - 更新 cqrs-rewrite 活跃跟踪与追踪文档中的当前 PR 锚点和审查结论 --- .github/workflows/ci.yml | 1 + .../todos/cqrs-rewrite-migration-tracking.md | 6 +++--- .../traces/cqrs-rewrite-migration-trace.md | 15 +++++++++++++++ scripts/validate-packed-modules.sh | 2 +- 4 files changed, 20 insertions(+), 4 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index c4b80392..6e2f6988 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -160,6 +160,7 @@ jobs: rm -rf ./packages dotnet pack GFramework.sln \ -c Release \ + --no-build \ --no-restore \ -o ./packages \ -p:IncludeSymbols=false diff --git a/ai-plan/public/cqrs-rewrite/todos/cqrs-rewrite-migration-tracking.md b/ai-plan/public/cqrs-rewrite/todos/cqrs-rewrite-migration-tracking.md index 93ac148e..2583af84 100644 --- a/ai-plan/public/cqrs-rewrite/todos/cqrs-rewrite-migration-tracking.md +++ b/ai-plan/public/cqrs-rewrite/todos/cqrs-rewrite-migration-tracking.md @@ -9,7 +9,7 @@ CQRS 迁移与收敛。 - 恢复点编号:`CQRS-REWRITE-RP-091` - 当前阶段:`Phase 8` -- 当前 PR 锚点:`待创建` +- 当前 PR 锚点:`PR #331` - 当前结论: - `GFramework.Cqrs` 已完成对外部 `Mediator` 的生产级替代,当前主线已从“是否可替代”转向“仓库内部收口与能力深化顺序” - `dispatch/invoker` 生成前移已扩展到 request / stream 路径,`RP-077` 已补齐 request invoker provider gate 与 stream gate 对称的 descriptor / descriptor entry runtime 合同回归 @@ -27,7 +27,7 @@ CQRS 迁移与收敛。 - 当前 `RP-089` 已补齐 stream invoker reflection / generated-provider 对照,使 generated descriptor 预热收益从 request 扩展到 stream 路径 - 当前 `RP-090` 已收敛 `PR #326` benchmark review:统一 benchmark 最小宿主构建、冻结 GFramework 容器、限制 MediatR 扫描范围,并恢复 request startup cold-start 对照 - 当前 `RP-091` 已把 benchmark 项目发布面隔离与包清单校验前移到 PR:`GFramework.Cqrs.Benchmarks` 明确保持不可打包,`publish.yml` 与 `ci.yml` 复用同一份 packed-modules 校验脚本 - - `ai-plan` active 入口现以 `RP-091` 为最新恢复锚点;`PR #326`、`PR #323`、`PR #307` 与其他更早阶段细节均以下方归档或说明为准 +- `ai-plan` active 入口现以 `RP-091` 为最新恢复锚点;`PR #331`、`PR #326`、`PR #323`、`PR #307` 与其他更早阶段细节均以下方归档或说明为准 ## 当前活跃事实 @@ -70,7 +70,7 @@ CQRS 迁移与收敛。 - 备注:用于验证本轮 request invoker / pipeline / stream invoker 调整与 benchmark workflow 改动后的 Release 编译结果 - `python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --format json --json-output ` - 结果:通过 - - 备注:确认当前分支对应 `PR #326`,本轮剩余 open AI feedback 以 workflow 输入安全、benchmark 对照公平性与 active 文档压缩为主 + - 备注:确认当前分支对应 `PR #331`,本轮 latest-head open AI feedback 已收敛到 `dotnet pack --no-build`、共享包校验脚本跨平台兼容性与 active 文档 PR 锚点同步 - `python3 scripts/license-header.py --check` - 结果:通过 - 备注:当前 WSL worktree 需要显式绑定 `GIT_DIR` / `GIT_WORK_TREE` 后运行 diff --git a/ai-plan/public/cqrs-rewrite/traces/cqrs-rewrite-migration-trace.md b/ai-plan/public/cqrs-rewrite/traces/cqrs-rewrite-migration-trace.md index 76bf5572..6bf3ce36 100644 --- a/ai-plan/public/cqrs-rewrite/traces/cqrs-rewrite-migration-trace.md +++ b/ai-plan/public/cqrs-rewrite/traces/cqrs-rewrite-migration-trace.md @@ -2,6 +2,21 @@ ## 2026-05-06 +### 阶段:PR #331 review 收尾补丁(CQRS-REWRITE-RP-091) + +- 使用 `$gframework-pr-review` 拉取当前分支 `fix/package-validation-guard` 对应的 `PR #331` latest-head review 后,主线程只保留本地复核仍成立的问题: + - `.github/workflows/ci.yml` 的 `dotnet pack` 步骤缺少 `--no-build`,会在已完成 solution `Build` 后重复编译整仓库 + - `scripts/validate-packed-modules.sh` 使用 GNU `find -printf`,在 macOS / BSD `find` 下无法运行 + - `ai-plan/public/cqrs-rewrite/todos/cqrs-rewrite-migration-tracking.md` 的 active PR 锚点仍写成 `待创建`,与当前公开 PR 状态不一致 +- 本轮决策: + - `ci.yml` 的 pack 步骤显式补上 `--no-build`,使其与前置 `Build` 步骤形成单次编译链路 + - 共享包校验脚本改为使用 `find ... -exec basename {} \;`,避免依赖 GNU-only 选项 + - active tracking 同步到 `PR #331`,并把这轮 PR review 的剩余问题描述更新为当前已核验的真实范围 +- 预期结果: + - PR workflow 的 pack 阶段不再对同一 solution 重复编译 + - `validate-packed-modules.sh` 可在 GNU / BSD `find` 环境下保持相同行为 + - `cqrs-rewrite` active 恢复入口继续与当前公开 PR 保持一致 + ### 阶段:benchmark 发布面隔离与包清单校验前移(CQRS-REWRITE-RP-091) - 针对 tag 发布中出现的 `GFramework.Cqrs.Benchmarks` 异常包名单,本轮先复核 benchmark 项目与 solution pack 的本地事实: diff --git a/scripts/validate-packed-modules.sh b/scripts/validate-packed-modules.sh index 74afaab1..b7f421b1 100644 --- a/scripts/validate-packed-modules.sh +++ b/scripts/validate-packed-modules.sh @@ -35,7 +35,7 @@ expected_file="$work_dir/expected-packages.txt" actual_file="$work_dir/actual-packages.txt" mapfile -t actual_packages < <( - find "$package_dir" -maxdepth 1 -type f -name '*.nupkg' -printf '%f\n' \ + find "$package_dir" -maxdepth 1 -type f -name '*.nupkg' -exec basename {} \; \ | sed -E 's/\.[0-9][0-9A-Za-z.-]*\.nupkg$//' \ | sort -u )