gewuyou/GFramework
1
0
Fork 0
mirror of https://github.com/GeWuYou/GFramework.git synced 2026-05-12 22:03:30 +08:00
Code Issues Packages Projects Releases Wiki Activity
GFramework/ai-plan/public/ai-first-config-system/traces/ai-first-config-system-trace.md
GeWuYou ed53f9c68c fix(ai-first-config): 收口PR评审解析与Tooling校验 
- 修复 gframework-pr-review 对 outside-diff 评论与 Python nitpick 卡片的解析，并补充结构化输出
- 优化 review section 解析边界，避免 latest review body 区块串读并消化 marker 查找 nitpick
- 收紧 config tool 对条件分支 schema 坏形状的拒绝规则，并新增 JS 回归测试
- 更新 ai-plan 跟踪与 trace，记录本轮 PR #262 follow-up 验证结果
2026-04-20 19:20:47 +08:00

6.8 KiB
Raw Blame History

AI-First Config System 执行 Trace

2026-04-19

阶段：active 入口归档收口（AI-FIRST-CONFIG-RP-002）

  • 已将截至 2026-04-17 的详细实现历史从默认 trace 入口移到主题内归档
  • active trace 现在只保留当前恢复点和下一步，避免 boot 每次恢复都重新读取已完成的长历史
  • 当前功能主线不变，仍是：
    • C# Runtime + Source Generator + Consumer DX
    • 下一批共享 JSON Schema 关键字评估
    • 优先看 if / then / else

Archive Context

  • 历史跟踪归档：
    • ai-plan/public/ai-first-config-system/archive/todos/ai-first-config-system-history-through-2026-04-17.md
  • 历史 trace 归档：
    • ai-plan/public/ai-first-config-system/archive/traces/ai-first-config-system-history-through-2026-04-17.md

验证

  • 2026-04-19：入口归档收口验证
    • 执行命令：wc -l ai-plan/public/ai-first-config-system/todos/ai-first-config-system-tracking.md ai-plan/public/ai-first-config-system/traces/ai-first-config-system-trace.md
    • 结果：通过
    • 备注：active 入口文件行数显著减少，已完成阶段详细历史已移至归档
  • 2026-04-17 之前：详细实现与定向验证命令
    • 参考：ai-plan/public/ai-first-config-system/archive/todos/ai-first-config-system-history-through-2026-04-17.md
    • 备注：包含 Runtime / Generator / Tooling 三端同步落地的每日验证记录与具体测试命令

下一步

  1. 从 ai-first-config-system-csharp-experience-next.md 读取当前 backlog，而不是继续翻已完成历史
  2. 先判断 if / then / else 是否满足“三端一致且不改变生成形状”的前提
  3. 若不满足，直接回退到下一批收益更明确的共享关键字评估

2026-04-20

阶段：object-focused if / then / else 收口（AI-FIRST-CONFIG-RP-003）

  • 已在 Runtime、Source Generator 与 VS Code Tooling 三端落地 object-focused if / then / else
  • 本轮采用的约束边界：
    • 仅允许 object 节点上的 object-typed inline schema
    • if 必填，且必须至少存在 then 或 else 之一
    • then / else 只能约束父对象已声明字段，不做属性合并
    • 条件匹配沿用 dependentSchemas / allOf 的 focused matcher 语义，允许未在条件块中声明的额外同级字段继续存在
  • 生成器新增 GF_ConfigSchema_013，在生成阶段提前拒绝坏形状的条件元数据，并把条件摘要写入 XML 文档
  • VS Code 工具同步补齐 schema 解析、校验消息、本地化文本与表单 hint 元数据显示

验证

  • 2026-04-20：bun run test（tools/gframework-config-tool）
    • 结果：通过
  • 2026-04-20：dotnet test GFramework.SourceGenerators.Tests/GFramework.SourceGenerators.Tests.csproj -c Release --filter "FullyQualifiedName~SchemaConfigGeneratorTests"
    • 结果：通过
  • 2026-04-20：dotnet test GFramework.Game.Tests/GFramework.Game.Tests.csproj -c Release --filter "FullyQualifiedName~YamlConfigLoaderIfThenElseTests"
    • 结果：通过
    • 备注：修正断言路径后，运行时诊断显示路径与 reward[if] / reward[then] 的约定保持一致
  • 2026-04-20：dotnet build GFramework.sln -c Release
    • 结果：通过
    • 备注：解决方案构建成功；输出包含仓库既有 analyzer warning，但无新增错误

阶段：PR #262 review follow-up 与分支同步

  • 已使用 gframework-pr-review 复核 PR #262，并确认 latest CodeRabbit review body 的第一行下方存在 folded 🧹 Nitpick comments (5)
  • 已修复 fetch_current_pr_review.py 的 follow-up 盲区：
    • 不再只依赖 issue comments，而会解析 latest review body 中的 folded nitpick cards
    • parse_comment_cards 现已覆盖 .js/.ts 等工具文件路径
    • text 输出会同时显示 declared / parsed 数量，避免 future drift 时静默少报
  • 已按 5 条 nitpick 收口代码：
    • VS Code tooling 的 ifElse hint 现会显示 condition
    • extension.js 已抽出可复用的 InlineObjectSchemaHint typedef
    • configValidation.js 已抽取共享 target reference 校验 helper
    • Source Generator tests 已补齐对称分支覆盖
    • Runtime test cleanup 已从 catch (Exception) 收窄到 IO / 权限异常
  • 已处理本地分支与远端分支差异：
    • 本地 feat/ai-first-config 已 rebase 到 origin/feat/ai-first-config
    • rebase 过程中 Git 跳过了远端已具备的 commit 76488dc
    • 当前分支已不再 behind 远端，仅保留本地领先提交

PR #262 review follow-up 验证

  • 2026-04-20：python3 .codex/skills/gframework-pr-review/scripts/fetch_current_pr_review.py
    • 结果：通过
    • 备注：输出 CodeRabbit actionable comments: 2、CodeRabbit nitpick comments: 2 declared, 1 parsed，并暴露剩余 review follow-up
  • 2026-04-20：skill parser follow-up
    • 结果：已补齐
    • 备注：gframework-pr-review 现可解析 latest review body 中的 Outside diff range comments，并且不再遗漏 .codex/.../*.py nitpick cards
  • 2026-04-20：python3 .codex/skills/gframework-pr-review/scripts/fetch_current_pr_review.py --pr 262 --format json
    • 结果：通过
    • 备注：输出 CodeRabbit outside-diff comments: 1 declared, 1 parsed、CodeRabbit nitpick comments: 2 declared, 2 parsed，parser warning 清零
  • 2026-04-20：运行时条件分支 follow-up
    • 结果：已补齐
    • 备注：YamlConfigSchemaValidator 现对非 object 的 if / then / else 使用分支级诊断路径；运行时测试新增 else 缺失 if 回归
  • 2026-04-20：bun run test（tools/gframework-config-tool）
    • 结果：通过（122 tests）
    • 备注：新增条件分支坏形状回归后，tooling 现在会拒绝缺失 type: "object"、坏形状 properties、坏形状 required 与空白 required 成员
  • 2026-04-20：dotnet test GFramework.SourceGenerators.Tests/GFramework.SourceGenerators.Tests.csproj -c Release --filter "FullyQualifiedName~SchemaConfigGeneratorTests"
    • 结果：通过（46 tests）
  • 2026-04-20：dotnet test GFramework.Game.Tests/GFramework.Game.Tests.csproj -c Release --filter "FullyQualifiedName~YamlConfigLoaderIfThenElseTests"
    • 结果：通过（8 tests）
    • 备注：新增 LoadAsync_Should_Throw_When_Else_Is_Declared_Without_If 后，运行时回归覆盖保持对称
  • 2026-04-20：dotnet build GFramework.sln -c Release
    • 结果：通过（历史记录）
    • 备注：存在仓库既有 analyzer warning，但无新增错误；本轮只需重新验证受影响测试切片

下一步

  1. 评估 oneOf / anyOf 是否值得继续沿用 object-focused 子集；若仍会造成生成形状漂移，就直接跳过
  2. 若继续扩共享关键字，先在 Runtime / Generator / Tooling 三端同时定义一致边界，再进入实现
  3. 继续把 active 入口保持精简，只记录当前恢复点、验证与下一步