mirror of
https://github.com/GeWuYou/GFramework.git
synced 2026-05-08 17:44:29 +08:00
docs(ai-plan): 脱敏 analyzer warning 跟踪路径记录
- 更新 analyzer-warning-reduction active todo/trace 的 PR review 真值与恢复点 - 脱敏 archive trace 中的临时输出路径与 Windows fallback package folder 记录 - 补充 dotnet build 验证结果并保持 639 Warning(s) 基线说明
This commit is contained in:
gewuyou
parent
c39cb5c9dc
commit
3a3359b495
@ -25,7 +25,7 @@
|
||||
- 更新 active todo 恢复点为 `RP-071`，将 `.gitignore` 纳入“已提交的低风险批次文件”，并移除已过时的 `.codex` 活跃风险描述
|
||||
- 在 active todo 中明确保留当前仍未采纳的两条 non-blocking nitpick：`VersionedMigrationRunner.cs` 的上下文一致性建议，以及 active trace 归档 RP-062 ~ RP-064 的建议
|
||||
- 验证里程碑：
|
||||
- `python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --json-output /tmp/current-pr-review.json`
|
||||
- `python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --json-output <current-pr-review-json>`
|
||||
- 结果：成功；确认 PR `#291` latest-head open review threads 为 `1`，唯一路径为 `ai-plan/public/analyzer-warning-reduction/todos/analyzer-warning-reduction-tracking.md:54`
|
||||
- `dotnet build`
|
||||
- 结果：成功；`0 Warning(s)`、`0 Error(s)`；该次为增量 Debug 构建，只作为本轮文档同步的完成校验，warning 权威基线仍保持 `RP-070` 记录的 `639 Warning(s)`
|
||||
@ -48,7 +48,7 @@
|
||||
- 在 `GFramework.Game/Internal/VersionedMigrationRunner.cs` 内引入私有 `MigrationExecutionContext<TData, TMigration>`，把多处 helper 共享的不变迁移上下文收口为参数对象，并同步补齐新增泛型 helper 的 XML 文档
|
||||
- 明确拒绝把 `TestLogger` 重复实现与 `YamlConfigLoaderTests.cs` 常量位置这类“可选整理”混入本轮 warning 收敛批次
|
||||
- 验证里程碑：
|
||||
- `python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --json-output /tmp/current-pr-review.json`
|
||||
- `python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --json-output <current-pr-review-json>`
|
||||
- 结果：成功；确认 PR `#291` latest-head open review threads 为 `1`，MegaLinter 仅有 `dotnet-format` restore 失败，tests 为 `2156 passed`
|
||||
- `dotnet build GFramework.Game/GFramework.Game.csproj -c Release`
|
||||
- 结果：成功；`326 Warning(s)`、`0 Error(s)`
|
||||
@ -258,13 +258,13 @@
|
||||
- `NumericExtensions.cs`、`StringExtensions.cs`、`StoreBuilder.cs` 的 Allman 花括号残留
|
||||
- `StoreSelection.cs` 在 `net9.0+` 下切到 `System.Threading.Lock`，同时保留 `net8.0` 兼容分支
|
||||
- 验证里程碑：
|
||||
- `python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --json-output /tmp/current-pr-review.json`
|
||||
- `python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --json-output <current-pr-review-json>`
|
||||
- 结果：成功；确认 PR `#288` 的 latest-head unresolved AI review threads 共 `9` 个，其中 `AsyncExtensionsTests.cs:126` 为 critical 编译错误
|
||||
- `DOTNET_CLI_HOME=/tmp/dotnet-home MSBuildEnableWorkloadResolver=false dotnet build GFramework.Core/GFramework.Core.csproj -c Release --no-restore -p:TargetFramework=net8.0 -p:RestoreFallbackFolders="" -v minimal`
|
||||
- 结果：失败；`MSB4018`，`ResolvePackageAssets` 仍读取失效 Windows fallback package folder `D:\Tool\Development Tools\Microsoft Visual Studio\Shared\NuGetPackages`
|
||||
- `DOTNET_CLI_HOME=/tmp/dotnet-home MSBuildEnableWorkloadResolver=false dotnet build GFramework.Core/GFramework.Core.csproj -c Release --no-restore -p:TargetFramework=net9.0 -p:RestoreFallbackFolders="" -v minimal`
|
||||
- `DOTNET_CLI_HOME=<dotnet-cli-home> MSBuildEnableWorkloadResolver=false dotnet build GFramework.Core/GFramework.Core.csproj -c Release --no-restore -p:TargetFramework=net8.0 -p:RestoreFallbackFolders="" -v minimal`
|
||||
- 结果：失败；`MSB4018`，`ResolvePackageAssets` 仍读取失效 Windows fallback package folder `<stale-windows-fallback-package-folder>`
|
||||
- `DOTNET_CLI_HOME=<dotnet-cli-home> MSBuildEnableWorkloadResolver=false dotnet build GFramework.Core/GFramework.Core.csproj -c Release --no-restore -p:TargetFramework=net9.0 -p:RestoreFallbackFolders="" -v minimal`
|
||||
- 结果：失败；原因同上
|
||||
- `DOTNET_CLI_HOME=/tmp/dotnet-home MSBuildEnableWorkloadResolver=false dotnet build GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release --no-restore -p:TargetFramework=net10.0 -p:RestoreFallbackFolders="" -v minimal`
|
||||
- `DOTNET_CLI_HOME=<dotnet-cli-home> MSBuildEnableWorkloadResolver=false dotnet build GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release --no-restore -p:TargetFramework=net10.0 -p:RestoreFallbackFolders="" -v minimal`
|
||||
- 结果：失败；原因同上
|
||||
- 当前结论：
|
||||
- 用户点名的 `AsyncExtensionsTests.cs` 编译错误已在源码层修复
|
||||
|
||||
@ -6,11 +6,11 @@
|
||||
|
||||
## 当前恢复点
|
||||
|
||||
- 恢复点编号：`ANALYZER-WARNING-REDUCTION-RP-072`
|
||||
- 当前阶段：`Phase 72`
|
||||
- 恢复点编号：`ANALYZER-WARNING-REDUCTION-RP-073`
|
||||
- 当前阶段：`Phase 73`
|
||||
- 当前焦点：
|
||||
- `2026-04-25` 主线程再次按 `$gframework-pr-review` 复核当前分支 PR `#291`，确认 latest-head 仍剩 `1` 条 open review thread 与 `1` 条 nitpick，均指向 active `ai-plan` 文档
|
||||
- 当前批次只同步 active todo/trace 到最新 review 真值：压缩 active trace，并把重复维护的验证结果改回单一权威区块
|
||||
- `2026-04-26` 主线程再次按 `$gframework-pr-review` 复核当前分支 PR `#291`，确认 latest-head 仍剩 `2` 条 open review thread，均指向 `ai-plan` 文档中的绝对路径记录
|
||||
- 当前批次同步 active todo/trace 与相关 archive trace：把 PR review 输出路径、临时 `dotnet` home 和失效 Windows fallback package folder 改写为仓库安全占位符
|
||||
- `dotnet clean` + `dotnet build` 的直接仓库根基线仍为 `639 Warning(s)`、`0 Error(s)`，因此本轮属于文档真值收口，而不是新的 warning 清理批次
|
||||
|
||||
## 当前活跃事实
|
||||
@ -22,33 +22,14 @@
|
||||
- `dotnet build`
|
||||
- 最新结果：成功；`639 Warning(s)`、`0 Error(s)`
|
||||
- 当前分支低风险批次文件：
|
||||
- `AGENTS.md`
|
||||
- `.gitignore`
|
||||
- `GFramework.Core.Tests/Logging/LogContextTests.cs`
|
||||
- `GFramework.Core.Tests/Logging/LoggerTests.cs`
|
||||
- `GFramework.Cqrs.Tests/Cqrs/CqrsHandlerRegistrarTests.cs`
|
||||
- `GFramework.Cqrs.Tests/Logging/TestLogger.cs`
|
||||
- `GFramework.Cqrs.Tests/Mediator/MediatorAdvancedFeaturesTests.cs`
|
||||
- `GFramework.Ecs.Arch.Tests/Ecs/EcsAdvancedTests.cs`
|
||||
- `GFramework.Game.Tests/Config/GameConfigBootstrapTests.cs`
|
||||
- `GFramework.Game.Tests/Config/GeneratedConfigConsumerIntegrationTests.cs`
|
||||
- `GFramework.Game.Tests/Config/YamlConfigLoaderTests.cs`
|
||||
- `GFramework.Game.Tests/Config/YamlConfigTextValidatorTests.cs`
|
||||
- `GFramework.Game/Internal/VersionedMigrationRunner.cs`
|
||||
- `ai-plan/public/analyzer-warning-reduction/todos/analyzer-warning-reduction-tracking.md`
|
||||
- `ai-plan/public/analyzer-warning-reduction/traces/analyzer-warning-reduction-trace.md`
|
||||
- `ai-plan/public/analyzer-warning-reduction/archive/traces/analyzer-warning-reduction-history-rp062-rp071.md`
|
||||
- 当前批次验证结果：
|
||||
- `python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --json-output /tmp/current-pr-review.json`
|
||||
- 最新主线程结果：成功；确认 PR `#291` latest-head open review thread 为 `1`、CodeRabbit nitpick 为 `1`，两者都指向 active `ai-plan` 文档入口
|
||||
- `python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --json-output <current-pr-review-json>`
|
||||
- 最新主线程结果：成功；确认 PR `#291` latest-head open review thread 为 `2`，两者都指向 `ai-plan` 文档中的绝对路径记录
|
||||
- `dotnet build`
|
||||
- 最新主线程结果：成功；`0 Warning(s)`、`0 Error(s)`；该次为增量 Debug 构建，只作为本轮文档同步完成校验，warning 权威基线仍以 `dotnet clean` 后的 `639 Warning(s)` 为准
|
||||
- `dotnet build GFramework.Game/GFramework.Game.csproj -c Release`
|
||||
- 最新主线程结果：成功；`326 Warning(s)`、`0 Error(s)`
|
||||
- `dotnet build GFramework.Cqrs.Tests/GFramework.Cqrs.Tests.csproj -c Release`
|
||||
- 最新主线程结果：成功；`149 Warning(s)`、`0 Error(s)`
|
||||
- `dotnet build GFramework.Core.Tests/GFramework.Core.Tests.csproj -c Release`
|
||||
- 上一轮主线程结果：成功；`0 Warning(s)`、`0 Error(s)`
|
||||
- 最新主线程结果：成功；`639 Warning(s)`、`0 Error(s)`；与当前权威仓库根基线一致
|
||||
|
||||
## 当前风险
|
||||
|
||||
@ -77,6 +58,6 @@
|
||||
|
||||
## 下一步建议
|
||||
|
||||
1. 推送包含本轮 active todo/trace 同步的提交后，重新执行 `$gframework-pr-review`，确认 PR `#291` 的 latest-head open thread 与 nitpick 是否已自动收口。
|
||||
1. 推送包含本轮 absolute-path 脱敏的提交后，重新执行 `$gframework-pr-review`，确认 PR `#291` 的 latest-head open thread 是否已自动收口。
|
||||
2. 若 PR `#291` 已清零，继续以当前 `639 Warning(s)` 根基线为恢复点，按 `$gframework-batch-boot 50` 规则挑选下一个 1-3 文件的低风险热点。
|
||||
3. 若 GitHub 仍保留 review 信号，先确认它们是否指向新 head，再决定是否需要继续细化 active `ai-plan` 入口。
|
||||
3. 若 GitHub 仍保留 review 信号，先确认它们是否仍指向新 head，再决定是否需要继续清理同主题下的其它历史 `ai-plan` 记录。
|
||||
|
||||
@ -1,26 +1,26 @@
|
||||
# Analyzer Warning Reduction 追踪
|
||||
|
||||
## 2026-04-25 — RP-072
|
||||
## 2026-04-26 — RP-073
|
||||
|
||||
### 阶段：收口 PR #291 剩余的 active ai-plan 文档 review
|
||||
### 阶段：脱敏 analyzer-warning-reduction 文档中的绝对路径记录
|
||||
|
||||
- 触发背景：
|
||||
- 用户再次显式要求执行 `$gframework-pr-review`，当前分支仍对应 PR `#291`
|
||||
- 最新抓取结果确认 latest-head 还剩 `1` 条 open review thread，指向 active trace 入口过长；同时 CodeRabbit 还有 `1` 条 nitpick，指向 active todo 中重复维护验证结果
|
||||
- 这两项问题都仍然成立，且都属于 active `ai-plan` 入口维护问题，不涉及新的源码 warning 修复
|
||||
- 最新抓取结果确认 latest-head 还剩 `2` 条 open review thread，分别指向 active todo 与 archive trace 中记录的绝对路径
|
||||
- active trace 当前也保留了同类 `/tmp` 路径记录；虽然这次 review 没直接点名，但继续保留会留下同一类治理缺口
|
||||
- 主线程实施：
|
||||
- 将 `ai-plan/public/analyzer-warning-reduction/traces/analyzer-warning-reduction-trace.md` 压缩为只保留当前恢复点与归档指针
|
||||
- 将 `RP-062` 至 `RP-071` 的详细 trace 迁入 [analyzer-warning-reduction-history-rp062-rp071.md](../archive/traces/analyzer-warning-reduction-history-rp062-rp071.md)
|
||||
- 同步更新 active todo，移除重复的验证结果抄录，改为只在权威区块维护构建与 PR review 真值
|
||||
- 将 active todo 与 active trace 中的 PR review 输出路径改写为 `--json-output <current-pr-review-json>`
|
||||
- 将 [analyzer-warning-reduction-history-rp062-rp071.md](../archive/traces/analyzer-warning-reduction-history-rp062-rp071.md) 里的临时 `dotnet` home、PR review 输出路径和失效 Windows fallback package folder 改写为仓库安全占位符
|
||||
- 同步刷新 active todo 中的 review 真值，把当前恢复点更新到 `RP-073`
|
||||
- 验证里程碑：
|
||||
- `python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --json-output /tmp/current-pr-review.json`
|
||||
- 结果：成功；确认 PR `#291` latest-head open review thread 为 `1`，CodeRabbit nitpick 为 `1`，两者都指向 active `ai-plan` 文档
|
||||
- `python3 .agents/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --json-output <current-pr-review-json>`
|
||||
- 结果：成功；确认 PR `#291` latest-head open review thread 为 `2`，两者都指向 `ai-plan` 文档中的绝对路径记录
|
||||
- `dotnet build`
|
||||
- 结果：成功；`0 Warning(s)`、`0 Error(s)`；该次为增量 Debug 构建，只作为本轮文档同步完成校验，warning 权威基线仍保持 `639 Warning(s)`、`0 Error(s)`
|
||||
- 结果：成功；`639 Warning(s)`、`0 Error(s)`；与当前权威仓库根基线一致
|
||||
- 当前结论：
|
||||
- 本轮只吸收当前仍成立的 PR review 文档项，不扩展到新的 warning 清理切片
|
||||
- 当前仓库根 warning 权威基线仍保持 `639 Warning(s)`、`0 Error(s)`；本轮目标是让 active 恢复入口重新变短且不重复维护真值
|
||||
- 下一轮默认先推送本轮同步并重新执行 `$gframework-pr-review`，确认 PR `#291` 的 open thread 与 nitpick 是否已自动收口
|
||||
- 当前仓库根 warning 权威基线仍保持 `639 Warning(s)`、`0 Error(s)`；本轮目标是让 analyzer-warning-reduction 主题下当前入口不再记录绝对路径
|
||||
- 下一轮默认先推送本轮同步并重新执行 `$gframework-pr-review`，确认 PR `#291` 的 open thread 是否已自动收口
|
||||
|
||||
## 历史归档指针
|
||||
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user