Skip to content

docs(ai-plan): 迁移 coroutine 早期计划#257

Merged
GeWuYou merged 4 commits into
mainfrom
feat/coroutine-optimization
Apr 19, 2026
Merged

docs(ai-plan): 迁移 coroutine 早期计划#257
GeWuYou merged 4 commits into
mainfrom
feat/coroutine-optimization

Conversation

@GeWuYou

@GeWuYou GeWuYou commented Apr 19, 2026

Copy link
Copy Markdown
Owner
  • 新增 coroutine-optimization 主题并整合 legacy local-plan todo,补写缺失 trace 的恢复边界
  • 更新 ai-plan 公共索引与治理跟踪,建立 feat/coroutine-optimization 的 topic 映射
  • 删除 worktree 根目录 legacy local-plan 入口,统一从 ai-plan/public 恢复

Summary by CodeRabbit

  • 文档
    • 新增“协程优化”主题的目录、跟踪、追踪、todos 与 archive 文档,包含历史快照、迁移验证、治理阶段与后续推进选项。
  • 杂项
    • 将早期仅含 todo 的协程计划迁入新主题并补写基于 todo 的历史 trace;更新分支到主题的映射以包含 feat/coroutine-optimization,并增加相应的验证与清理步骤。

- 新增 coroutine-optimization 主题并整合 legacy local-plan todo,补写缺失 trace 的恢复边界\n- 更新 ai-plan 公共索引与治理跟踪,建立 feat/coroutine-optimization 的 topic 映射\n- 删除 worktree 根目录 legacy local-plan 入口,统一从 ai-plan/public 恢复
@coderabbitai

coderabbitai Bot commented Apr 19, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 49fa883f-1c35-4ca8-85f4-4e4a79559539

📥 Commits

Reviewing files that changed from the base of the PR and between da7d03d and 2381e72.

📒 Files selected for processing (1)
  • ai-plan/public/README.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • ai-plan/public/README.md
📜 Recent review details
⏰ Context from checks skipped due to timeout of 900000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
  • GitHub Check: Analyze (C#)
  • GitHub Check: Build and Test
  • GitHub Check: Code Quality & Security
  • GitHub Check: submit-nuget

📝 Walkthrough

Walkthrough

新增 ai-plan 主题目录 coroutine-optimization(含 todos/traces/archive/...),将 legacy local-plan 的早期 coroutine todo 基线迁入并归档;在治理文档中补充迁移规则与验证步骤,并在 ai-plan/public/README.md 的 Worktree→Active Topic 映射中添加 feat/coroutine-optimizationcoroutine-optimization

Changes

Cohort / File(s) Summary
顶层映射与目录说明
ai-plan/public/README.md
新增 coroutine-optimization 活跃主题条目;在 Worktree→Active Topic 映射中添加 feat/coroutine-optimizationcoroutine-optimization(保留 ai-plan-governance 为次要)。
治理追踪与任务清单
ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md, ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
补充已完成项并新增治理阶段/迁移规则(含 RP-005),明确仅含 todo 且无 trace 的 legacy local-plan 迁入并归档,调整验证命令(加入 find、移除基于行计数的断言)。
新主题:活跃跟踪与追踪文档
ai-plan/public/coroutine-optimization/todos/coroutine-optimization-tracking.md, ai-plan/public/coroutine-optimization/traces/coroutine-optimization-trace.md
新增 RP-001 的活跃跟踪与 trace 文档,记录当前焦点(语义、调度、Godot 集成等)、活跃入口范围、后续路径与验证要点。
新主题:历史归档(Pre-RP-001)
ai-plan/public/coroutine-optimization/archive/todos/coroutine-optimization-history-pre-rp001.md, ai-plan/public/coroutine-optimization/archive/traces/coroutine-optimization-history-pre-rp001.md
新增历史 todo 快照与历史 trace 基线,按阶段整合原 local-plan 五个文档的要点、缺口与可核对条目,声明这些为归档快照非可复现的 durable trace。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 分钟

Possibly related PRs

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 标题准确反映了主要变更:将 coroutine 早期计划从 local-plan 迁移到 ai-plan/public/coroutine-optimization/ 主题目录,与提交内容高度一致。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/coroutine-optimization

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

Copy link
Copy Markdown

Summary

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Other ❓ Flaky 🍂 Duration ⏱️
2103 2103 0 0 0 0 1ms

Test Results

passed 2103 passed

Details

tests 2103 tests
clock 36.0s
tool nunit
build CI - Build & Test arrow-right build-and-test link #889
pull-request docs(ai-plan): 迁移 coroutine 早期计划 link #257

Insights

Average Tests per Run Total Flaky Tests Total Failed Slowest Test (p95)
2103 0 0 4.2s

build-and-test: Run #889

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Pending ⏳ Other ❓ Flaky 🍂 Duration ⏱️
2103 2103 0 0 0 0 0 36.0s

🎉 All tests passed!

Slowest Tests

Test 📝 Results 📊 Duration (avg) ⏱️ Duration (p95) ⏱️
SendRequestAsync_Should_ResolveCqrsRuntime_OnlyOnce_When_AccessedConcurrently 1 4.2s 4.2s
Does_Not_Report_When_FieldInjectedModel_Is_Registered 1 2.1s 2.1s
Generates_Scene_Behavior_Boilerplate 1 1.9s 1.9s
CleanupDuringAcquire_Should_NotCauseRaceCondition 1 1.1s 1.1s
Append_ShouldNotBlock 1 1.0s 1.0s
Context_Caching_Should_Improve_Performance 1 781ms 781ms
PendingCount_ShouldReflectQueuedEntries 1 501ms 501ms
Cleanup_Should_NotRemoveActiveLocks 1 404ms 404ms
Cleanup_Should_RemoveUnusedLocks 1 402ms 402ms
Generates_Precise_Assembly_Type_Lookups_For_Inaccessible_External_Protected_Types 1 334ms 334ms

🎉 No failed tests in this run. | 🍂 No flaky tests in this run.

Github Test Reporter by CTRF 💚

@github-actions

github-actions Bot commented Apr 19, 2026

Copy link
Copy Markdown

MegaLinter analysis: Success

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ REPOSITORY gitleaks yes no no 2.5s
✅ REPOSITORY trufflehog yes no no 2.79s

See detailed reports in MegaLinter artifacts
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security
Show us your support by starring ⭐ the repository

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md`:
- Line 92: 将重复的同级标题 "### 下一步"(触发 MD024)替换为一个带阶段或上下文限定的唯一标题,例如在原始 "### 下一步"
后添加阶段标识、时间点或模块名(如 "下一步 — 验证阶段" 或 "下一步(迭代 1)"),以确保标题唯一并消除 markdownlint 的 MD024
警告;在文档中寻找所有出现相同文本的标题并统一采用该唯一化命名约定,更新章节标题处的 "### 下一步" 即可(参考标识符: "### 下一步")。
🪄 Autofix (Beta)

✅ Autofix completed


ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c2416faf-c709-411d-968b-02afd073554b

📥 Commits

Reviewing files that changed from the base of the PR and between 84b40a2 and e1e32b2.

📒 Files selected for processing (7)
  • ai-plan/public/README.md
  • ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md
  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
  • ai-plan/public/coroutine-optimization/archive/todos/coroutine-optimization-history-pre-rp001.md
  • ai-plan/public/coroutine-optimization/archive/traces/coroutine-optimization-history-pre-rp001.md
  • ai-plan/public/coroutine-optimization/todos/coroutine-optimization-tracking.md
  • ai-plan/public/coroutine-optimization/traces/coroutine-optimization-trace.md
📜 Review details
⏰ Context from checks skipped due to timeout of 900000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: Analyze (C#)
  • GitHub Check: Code Quality & Security
🧰 Additional context used
📓 Path-based instructions (5)
**/README.md

📄 CodeRabbit inference engine (AGENTS.md)

Every user-facing package or module directory that contains a *.csproj intended for direct consumption MUST have a sibling README.md with canonical filename README.md. A module README MUST describe the module's purpose, relationship to adjacent runtime/abstractions/generator packages, major subdirectories/subsystems, minimum adoption path, and corresponding docs/zh-CN/ entry points.

Files:

  • ai-plan/public/README.md
ai-plan/public/**/*

📄 CodeRabbit inference engine (AGENTS.md)

ai-plan/public/**/*: Contributors MUST keep committed ai-plan/public/** content safe to publish in Git history. Never write secrets, tokens, credentials, private keys, machine usernames, home-directory paths, hostnames, IP addresses, proprietary URLs, or other sensitive environment details into any ai-plan/** file.
Never record absolute file-system paths in ai-plan/**; use repository-relative paths, branch names, PR numbers, or stable document identifiers instead.

Files:

  • ai-plan/public/README.md
  • ai-plan/public/coroutine-optimization/archive/todos/coroutine-optimization-history-pre-rp001.md
  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
  • ai-plan/public/coroutine-optimization/traces/coroutine-optimization-trace.md
  • ai-plan/public/coroutine-optimization/archive/traces/coroutine-optimization-history-pre-rp001.md
  • ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md
  • ai-plan/public/coroutine-optimization/todos/coroutine-optimization-tracking.md
ai-plan/public/README.md

📄 CodeRabbit inference engine (AGENTS.md)

ai-plan/public/README.md MUST list only active topics. Do not add ai-plan/public/archive/** content to the default boot index. When a worktree-to-topic mapping changes or when a topic becomes active/inactive, contributors MUST update ai-plan/public/README.md in the same change.

Files:

  • ai-plan/public/README.md
ai-plan/public/*/traces/**/*

📄 CodeRabbit inference engine (AGENTS.md)

Contributors MUST maintain a matching execution trace under ai-plan/public/<topic>/traces/ for complex work. The trace should record the current date, key decisions, validation milestones, and the immediate next step. Update traces at each meaningful milestone before pausing or handing work off.

Files:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
  • ai-plan/public/coroutine-optimization/traces/coroutine-optimization-trace.md
ai-plan/public/*/todos/**/*

📄 CodeRabbit inference engine (AGENTS.md)

When working from a tracked implementation plan, contributors MUST update the corresponding tracking document under ai-plan/public/<topic>/todos/ in the same change. Updates MUST reflect completed work, newly discovered issues, validation results, and the next recommended recovery point. Tracking files are recovery entrypoints, not append-only changelogs, and MUST stay concise.

Files:

  • ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md
  • ai-plan/public/coroutine-optimization/todos/coroutine-optimization-tracking.md
🧠 Learnings (14)
📓 Common learnings
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: For any multi-step refactor, migration, or cross-module task, contributors MUST create or adopt a dedicated recovery document under `ai-plan/public/<topic>/todos/` before making substantive code changes. Recovery documents MUST record the current phase, the active recovery point identifier, known risks, and the next recommended resume step so another contributor or subagent can continue the work safely.
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to ai-plan/public/*/todos/**/* : When working from a tracked implementation plan, contributors MUST update the corresponding tracking document under `ai-plan/public/<topic>/todos/` in the same change. Updates MUST reflect completed work, newly discovered issues, validation results, and the next recommended recovery point. Tracking files are recovery entrypoints, not append-only changelogs, and MUST stay concise.
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to ai-plan/public/README.md : `ai-plan/public/README.md` MUST list only active topics. Do not add `ai-plan/public/archive/**` content to the default boot index. When a worktree-to-topic mapping changes or when a topic becomes active/inactive, contributors MUST update `ai-plan/public/README.md` in the same change.

Applied to files:

  • ai-plan/public/README.md
  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
  • ai-plan/public/coroutine-optimization/traces/coroutine-optimization-trace.md
  • ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to ai-plan/public/*/todos/**/* : When working from a tracked implementation plan, contributors MUST update the corresponding tracking document under `ai-plan/public/<topic>/todos/` in the same change. Updates MUST reflect completed work, newly discovered issues, validation results, and the next recommended recovery point. Tracking files are recovery entrypoints, not append-only changelogs, and MUST stay concise.

Applied to files:

  • ai-plan/public/README.md
  • ai-plan/public/coroutine-optimization/archive/todos/coroutine-optimization-history-pre-rp001.md
  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
  • ai-plan/public/coroutine-optimization/traces/coroutine-optimization-trace.md
  • ai-plan/public/coroutine-optimization/archive/traces/coroutine-optimization-history-pre-rp001.md
  • ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md
  • ai-plan/public/coroutine-optimization/todos/coroutine-optimization-tracking.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: When a topic is fully complete, move the entire topic directory under `ai-plan/public/archive/<topic>/` and remove it from `ai-plan/public/README.md` in the same change.

Applied to files:

  • ai-plan/public/README.md
  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
  • ai-plan/public/coroutine-optimization/traces/coroutine-optimization-trace.md
  • ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: For any multi-step refactor, migration, or cross-module task, contributors MUST create or adopt a dedicated recovery document under `ai-plan/public/<topic>/todos/` before making substantive code changes. Recovery documents MUST record the current phase, the active recovery point identifier, known risks, and the next recommended resume step so another contributor or subagent can continue the work safely.

Applied to files:

  • ai-plan/public/README.md
  • ai-plan/public/coroutine-optimization/archive/todos/coroutine-optimization-history-pre-rp001.md
  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
  • ai-plan/public/coroutine-optimization/traces/coroutine-optimization-trace.md
  • ai-plan/public/coroutine-optimization/archive/traces/coroutine-optimization-history-pre-rp001.md
  • ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md
  • ai-plan/public/coroutine-optimization/todos/coroutine-optimization-tracking.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to ai-plan/public/*/traces/**/* : Contributors MUST maintain a matching execution trace under `ai-plan/public/<topic>/traces/` for complex work. The trace should record the current date, key decisions, validation milestones, and the immediate next step. Update traces at each meaningful milestone before pausing or handing work off.

Applied to files:

  • ai-plan/public/README.md
  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
  • ai-plan/public/coroutine-optimization/traces/coroutine-optimization-trace.md
  • ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: If a task depends on observations from `ai-libs/**`, record the referenced path and conclusion in the active plan or trace when the work is multi-step or complex, or when an active tracking document already exists, rather than editing the third-party reference copy.

Applied to files:

  • ai-plan/public/README.md
  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
  • ai-plan/public/coroutine-optimization/traces/coroutine-optimization-trace.md
  • ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Use `ai-plan/public/**` only for durable, handoff-safe task state. Put temporary notes, local experiments, or worktree-specific scratch recovery data under `ai-plan/private/`.

Applied to files:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
  • ai-plan/public/coroutine-optimization/traces/coroutine-optimization-trace.md
  • ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to ai-plan/public/**/* : Never record absolute file-system paths in `ai-plan/**`; use repository-relative paths, branch names, PR numbers, or stable document identifiers instead.

Applied to files:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
  • ai-plan/public/coroutine-optimization/traces/coroutine-optimization-trace.md
  • ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: When a stage inside an active topic is fully complete, move the finished artifacts into that topic's `archive/` directory instead of leaving every completed step in the default boot path. When completed and validated stages begin to accumulate, archive their detailed history out of the active `todos/` and `traces/` entry files. Keep only the current recovery point, active facts, active risks, immediate next step, and pointers to relevant archive files in the default boot path.

Applied to files:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
  • ai-plan/public/coroutine-optimization/traces/coroutine-optimization-trace.md
  • ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to docs/zh-CN/**/*.md : When a feature is added, removed, renamed, or substantially refactored, contributors MUST update or create the corresponding user-facing integration documentation in `docs/zh-CN/` in the same change. For integration-oriented features, documentation MUST cover project directory layout, file conventions, required project/package wiring, minimal working usage example, and migration/compatibility notes.

Applied to files:

  • ai-plan/public/coroutine-optimization/traces/coroutine-optimization-trace.md
  • ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Before considering work complete, confirm: required comments and XML docs are present, code follows repository style and naming rules, relevant tests were added or updated, sensitive or unsafe behavior was not introduced, user-facing documentation is updated when needed, and feature adoption docs under `docs/zh-CN/` were added or updated when functionality was added, removed, or refactored.

Applied to files:

  • ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to docs/zh-CN/**/*.md : Code samples, package names, and command examples in documentation MUST be aligned with the current repository state. When an existing documentation page no longer reflects the current implementation, fix the code without fixing the documentation is considered incomplete work.

Applied to files:

  • ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: When a task spans multiple commits or is likely to exceed a single agent context window, update both the recovery document and the trace at each meaningful milestone before pausing or handing work off.

Applied to files:

  • ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md
🪛 LanguageTool
ai-plan/public/coroutine-optimization/archive/todos/coroutine-optimization-history-pre-rp001.md

[uncategorized] ~40-~40: 您的意思是“"不"更”?
Context: ...eWaitOneFrame的命名差异是否值得保留 - 为阶段型等待补更多跨宿主说明与样例 - 审视WaitForCoroutine` 在父子调度...

(BU)


[uncategorized] ~45-~45: 您的意思是“"不"统一”?
Context: ...utine` 在父子调度器不同阶段时的语义 ### 可选增强 - 为等待指令补统一的语义类别元数据 - 支持宿主自定义阶段映射 ### 风险与验收 - 风...

(BU)


[uncategorized] ~77-~77: 您的意思是“"不"更”?
Context: ...路 ### 后续必做项 - 评估是否需要完成历史的上限或清理策略 - 为快照补更多可观测字段时保持分配与遍历成本可控 - 审查取消、终止、异常三种完成路径的外...

(BU)


[uncategorized] ~101-~101: 您的意思是“源"于"”吗?
Context: ...ration ### 已有基础 - Timing 已为各段提供缩放时间源与真实时间源 - PhysicsProcess / `DeferredProc...

(YU7_YU8)


[uncategorized] ~148-~148: 您的意思是“"不"异常”?
Context: ...odot 运行时级测试,覆盖节点归属、退树、暂停和各 segment 差异 - 补异常传播、完成历史与快照字段的更多边界测试 - 评估是否需要把 `GFramew...

(BU)


[uncategorized] ~190-~190: 您的意思是“"不"更”?
Context: ...FixedUpdate/WaitForEndOfFrame` 的宿主前提补更多页面说明 ### 可选增强 - 增加 FAQ 和排障章节 - 增加 Go...

(BU)


[uncategorized] ~205-~205: 您的意思是“"不"迁移”?
Context: ...状态 - 主文档与教程已对齐当前实现 - 下一步原计划:继续清理其他文档残留并补迁移说明 ## 历史结论 - 旧 local-plan 记录的不是“待从...

(BU)


[uncategorized] ~211-~211: 您的意思是“"不"回”?
Context: ...续推进时,应基于 active tracking 重新选择一个最小切入点,并补回当轮 trace 与验证记录

(BU)

ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md

[uncategorized] ~76-~76: 您的意思是“"不"齐”?
Context: ...i-plan/public/coroutine-optimization/,并补齐: - todos/ -traces/ -archi...

(BU)


[uncategorized] ~81-~81: 您的意思是“"不"写”?
Context: ... local-plan 中分散的五个阶段计划整合进主题内历史跟踪归档,并额外补写一份基于 todo 基线整理出的历史 trace, 显式记录“缺少原始 t...

(BU)


[uncategorized] ~95-~95: 您的意思是“"不"写”?
Context: ...,并明确标注推导边界 2. 保持新 topic 的 active 入口精简,不把补写 trace 变成伪造逐日执行日志

(BU)

ai-plan/public/coroutine-optimization/archive/traces/coroutine-optimization-history-pre-rp001.md

[uncategorized] ~10-~10: 您的意思是“"不"录”?
Context: ... ## 2026-04-19 ### 阶段:早期 coroutine 计划基线补录(RP-001) - 复核 `local-plan/todos/corout...

(BU)


[uncategorized] ~29-~29: 您的意思是“"不"齐”?
Context: ...free` 等行为仍需更强验证 - 更贴近运行时的集成测试与迁移对照文档仍待补齐 - 已显式记录信息缺口: - 没有原始执行 trace - 没有与每...

(BU)


[uncategorized] ~40-~40: 您的意思是“"不"证”?
Context: ...点编号 3. 如果需要更强历史精度,只能再从 Git 历史、测试记录或代码差异中补证,不能把旧 todo 当作完整 trace 使用

(BU)

ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md

[uncategorized] ~82-~82: 您的意思是“"不"齐”?
Context: ...期、只有 todo 没有 trace 的 coroutine 计划也已按治理规则补齐 active 入口与 archive - `test ! -e local-...

(BU)

ai-plan/public/coroutine-optimization/todos/coroutine-optimization-tracking.md

[uncategorized] ~5-~5: 您的意思是“"不"宿主”?
Context: ...e Optimization 跟踪 ## 目标 继续以“先收敛语义一致性,再补宿主验证和迁移文档”为原则推进当前协程体系,避免 Core 与 Godot 两侧...

(BU)


[uncategorized] ~47-~47: 动词的修饰一般为‘形容词(副词)+地+动词’。您的意思是否是:新"地"追加
Context: ... 跟踪文件只保留当前恢复点、活跃事实、风险与下一步,避免把更早期计划直接平移成新的追加式日志 ## 下一步 1. 若继续该主题,先在 `Core Semant...

(wb4)


[uncategorized] ~52-~52: 您的意思是“"不"验证”?
Context: ...Docs And Migration` 中只选一个切入点推进 2. 若优先补验证,先规划 Godot 集成测试宿主与节点归属/退树/暂停场景,再扩运行时诊断...

(BU)


[uncategorized] ~53-~53: 您的意思是“"不"文档”?
Context: ...t 集成测试宿主与节点归属/退树/暂停场景,再扩运行时诊断 API 3. 若优先补文档与迁移说明,先清理其余 `StartCoroutine()/StopCoro...

(BU)


[uncategorized] ~53-~53: 您的意思是“"不"统一”?
Context: ...routine()/StopCoroutine()` 残留,再为阶段等待和新入口补统一对照说明

(BU)

🪛 markdownlint-cli2 (0.22.0)
ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md

[warning] 92-92: Multiple headings with the same content

(MD024, no-duplicate-heading)

🔇 Additional comments (7)
ai-plan/public/coroutine-optimization/todos/coroutine-optimization-tracking.md (1)

10-53: 结构完整,符合恢复入口约束。

当前 tracking 已覆盖恢复点、活跃事实、风险、验证说明与下一步,且保持为可交接的精简入口,方向正确。

As per coding guidelines ai-plan/public/*/todos/**/*: “Updates MUST reflect completed work, newly discovered issues, validation results, and the next recommended recovery point.”

ai-plan/public/coroutine-optimization/archive/todos/coroutine-optimization-history-pre-rp001.md (1)

5-211: 历史基线归档边界定义清晰。

“来源—阶段—风险—验收—恢复边界”的组织方式可读性很好,便于后续在 active 文档中保持精简。

ai-plan/public/README.md (1)

28-32: 索引与分支映射更新到位。

新增 active topic 与 worktree 映射同步,符合 boot 入口定位预期。

As per coding guidelines ai-plan/public/README.md: “When a worktree-to-topic mapping changes ... contributors MUST update ai-plan/public/README.md in the same change.”

Also applies to: 54-57

ai-plan/public/coroutine-optimization/archive/traces/coroutine-optimization-history-pre-rp001.md (1)

8-40: trace 归档补录方式合理。

对“可恢复结论”与“不可伪造逐日流水”的边界写得明确,能有效降低后续误用风险。

ai-plan/public/coroutine-optimization/traces/coroutine-optimization-trace.md (1)

3-34: active trace 信息完整,适合交接恢复。

当前条目已覆盖关键决策、验证里程碑与立即下一步,符合复杂任务 trace 入口要求。

As per coding guidelines ai-plan/public/*/traces/**/*: “The trace should record the current date, key decisions, validation milestones, and the immediate next step.”

ai-plan/public/ai-plan-governance/todos/ai-plan-governance-tracking.md (1)

57-67: 治理 tracking 与本次迁移闭环一致。

完成项、验证项、后续扫描策略三者衔接自然,作为 RP-005 的恢复入口是合格的。

Also applies to: 80-83, 97-97

ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md (1)

72-90: RP-005 的 coroutine 迁移追踪记录完整。

该段对“todo-only 历史”的迁移边界、验证命令与后续约束描述清楚,治理可执行性较好。

Comment thread ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md Outdated
@coderabbitai

coderabbitai Bot commented Apr 19, 2026

Copy link
Copy Markdown
Contributor

Note

Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it.

Fixes Applied Successfully

Fixed 1 file(s) based on 1 unresolved review comment.

Files modified:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md

Commit: c3614b5b5d53f6dcbe79f5fe415c4c202bdd324c

The changes have been pushed to the feat/coroutine-optimization branch.

Time taken: 3m 30s

Fixed 1 file(s) based on 1 unresolved review comment.

Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
@github-actions

Copy link
Copy Markdown

Summary

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Other ❓ Flaky 🍂 Duration ⏱️
2103 2103 0 0 0 0 35.7s    ↓252ms

Test Results

passed 2103 passed

Details

tests 2103 tests
clock 35.7s ↓252ms
tool nunit
build CI - Build & Test arrow-right build-and-test link #890
pull-request docs(ai-plan): 迁移 coroutine 早期计划 link #257

Insights

Average Tests per Run Total Flaky Tests Total Failed Slowest Test (p95)
2103 0 0 4.7s

build-and-test: Run #890

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Pending ⏳ Other ❓ Flaky 🍂 Duration ⏱️
2103 2103 0 0 0 0 0 35.7s

🎉 All tests passed!

Slowest Tests

Test 📝 Results 📊 Duration (avg) ⏱️ Duration (p95) ⏱️
SendRequestAsync_Should_ResolveCqrsRuntime_OnlyOnce_When_AccessedConcurrently 2 4.4s 4.7s
Does_Not_Report_When_FieldInjectedModel_Is_Registered 2 2.1s 2.1s
Generates_Scene_Behavior_Boilerplate 2 1.9s 1.9s
CleanupDuringAcquire_Should_NotCauseRaceCondition 2 1.1s 1.1s
Append_ShouldNotBlock 2 1.0s 1.0s
Context_Caching_Should_Improve_Performance 2 780ms 781ms
PendingCount_ShouldReflectQueuedEntries 2 501ms 501ms
Cleanup_Should_NotRemoveActiveLocks 2 404ms 404ms
Cleanup_Should_RemoveUnusedLocks 2 402ms 402ms
Generates_Precise_Assembly_Type_Lookups_For_Inaccessible_External_Protected_Types 2 333ms 334ms

± Comparison with run #889 at 271e018 | 🎉 No failed tests detected across all runs. | 🍂 No flaky tests detected across all runs. | ⏱️ Measured over 2 runs.

Github Test Reporter by CTRF 💚

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md`:
- Around line 84-90: Update the ai-plan/public/README.md to include the
mandatory worktree mapping entry for feat/coroutine-optimization ->
coroutine-optimization as described in the trace; locate the README and add a
clear mapping line or section that documents the new worktree
(feat/coroutine-optimization) pointing to the coroutine-optimization directory
so the governance requirement referenced in ai-plan-governance-trace.md is
satisfied and the repo’s worktree mappings remain consistent with the created
coroutine-optimization directory and its todos/traces/archive structure.
🪄 Autofix (Beta)

❌ Autofix failed (check again to retry)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: a2b3abf2-6500-4b4b-86b9-8992d00de727

📥 Commits

Reviewing files that changed from the base of the PR and between e1e32b2 and c3614b5.

📒 Files selected for processing (1)
  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
📜 Review details
⏰ Context from checks skipped due to timeout of 900000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: Analyze (C#)
  • GitHub Check: Build and Test
  • GitHub Check: Code Quality & Security
🧰 Additional context used
📓 Path-based instructions (2)
ai-plan/public/**/*

📄 CodeRabbit inference engine (AGENTS.md)

ai-plan/public/**/*: Contributors MUST keep committed ai-plan/public/** content safe to publish in Git history. Never write secrets, tokens, credentials, private keys, machine usernames, home-directory paths, hostnames, IP addresses, proprietary URLs, or other sensitive environment details into any ai-plan/** file.
Never record absolute file-system paths in ai-plan/**; use repository-relative paths, branch names, PR numbers, or stable document identifiers instead.

Files:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
ai-plan/public/*/traces/**/*

📄 CodeRabbit inference engine (AGENTS.md)

Contributors MUST maintain a matching execution trace under ai-plan/public/<topic>/traces/ for complex work. The trace should record the current date, key decisions, validation milestones, and the immediate next step. Update traces at each meaningful milestone before pausing or handing work off.

Files:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
🧠 Learnings (13)
📓 Common learnings
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: For any multi-step refactor, migration, or cross-module task, contributors MUST create or adopt a dedicated recovery document under `ai-plan/public/<topic>/todos/` before making substantive code changes. Recovery documents MUST record the current phase, the active recovery point identifier, known risks, and the next recommended resume step so another contributor or subagent can continue the work safely.
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Commit messages MUST use Conventional Commits format: `<type>(<scope>): <summary>`. The commit `summary` MUST use simplified Chinese and briefly describe the main change. The commit `body` MUST use unordered list items, each starting with a verb such as `新增`、`修复`、`优化`、`更新`、`补充`、`重构`. Each commit body bullet MUST describe one independent change point. Keep technical terms in English when they are established project terms, such as `API`、`Model`、`System`.
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: When a topic is fully complete, move the entire topic directory under `ai-plan/public/archive/<topic>/` and remove it from `ai-plan/public/README.md` in the same change.
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to ai-plan/public/*/todos/**/* : When working from a tracked implementation plan, contributors MUST update the corresponding tracking document under `ai-plan/public/<topic>/todos/` in the same change. Updates MUST reflect completed work, newly discovered issues, validation results, and the next recommended recovery point. Tracking files are recovery entrypoints, not append-only changelogs, and MUST stay concise.
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to ai-plan/public/README.md : `ai-plan/public/README.md` MUST list only active topics. Do not add `ai-plan/public/archive/**` content to the default boot index. When a worktree-to-topic mapping changes or when a topic becomes active/inactive, contributors MUST update `ai-plan/public/README.md` in the same change.
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Use `ai-plan/public/**` only for durable, handoff-safe task state. Put temporary notes, local experiments, or worktree-specific scratch recovery data under `ai-plan/private/`.
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: When a stage inside an active topic is fully complete, move the finished artifacts into that topic's `archive/` directory instead of leaving every completed step in the default boot path. When completed and validated stages begin to accumulate, archive their detailed history out of the active `todos/` and `traces/` entry files. Keep only the current recovery point, active facts, active risks, immediate next step, and pointers to relevant archive files in the default boot path.
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to ai-plan/public/*/traces/**/* : Contributors MUST maintain a matching execution trace under `ai-plan/public/<topic>/traces/` for complex work. The trace should record the current date, key decisions, validation milestones, and the immediate next step. Update traces at each meaningful milestone before pausing or handing work off.
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to ai-plan/public/*/todos/**/* : When working from a tracked implementation plan, contributors MUST update the corresponding tracking document under `ai-plan/public/<topic>/todos/` in the same change. Updates MUST reflect completed work, newly discovered issues, validation results, and the next recommended recovery point. Tracking files are recovery entrypoints, not append-only changelogs, and MUST stay concise.

Applied to files:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to ai-plan/public/*/traces/**/* : Contributors MUST maintain a matching execution trace under `ai-plan/public/<topic>/traces/` for complex work. The trace should record the current date, key decisions, validation milestones, and the immediate next step. Update traces at each meaningful milestone before pausing or handing work off.

Applied to files:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: For any multi-step refactor, migration, or cross-module task, contributors MUST create or adopt a dedicated recovery document under `ai-plan/public/<topic>/todos/` before making substantive code changes. Recovery documents MUST record the current phase, the active recovery point identifier, known risks, and the next recommended resume step so another contributor or subagent can continue the work safely.

Applied to files:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: If a task depends on observations from `ai-libs/**`, record the referenced path and conclusion in the active plan or trace when the work is multi-step or complex, or when an active tracking document already exists, rather than editing the third-party reference copy.

Applied to files:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to ai-plan/public/README.md : `ai-plan/public/README.md` MUST list only active topics. Do not add `ai-plan/public/archive/**` content to the default boot index. When a worktree-to-topic mapping changes or when a topic becomes active/inactive, contributors MUST update `ai-plan/public/README.md` in the same change.

Applied to files:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Treat source code, `*.csproj`, tests, generated snapshots, and packaging metadata as the primary evidence for documentation updates. Treat `CoreGrid` as a secondary evidence source for real project adoption patterns, directory layouts, and end-to-end usage examples. Treat existing `README.md` files and `docs/zh-CN/` pages as editable outputs, not authoritative truth.

Applied to files:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to ai-plan/public/**/* : Never record absolute file-system paths in `ai-plan/**`; use repository-relative paths, branch names, PR numbers, or stable document identifiers instead.

Applied to files:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to docs/zh-CN/**/*.md : When a feature is added, removed, renamed, or substantially refactored, contributors MUST update or create the corresponding user-facing integration documentation in `docs/zh-CN/` in the same change. For integration-oriented features, documentation MUST cover project directory layout, file conventions, required project/package wiring, minimal working usage example, and migration/compatibility notes.

Applied to files:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Before considering work complete, confirm: required comments and XML docs are present, code follows repository style and naming rules, relevant tests were added or updated, sensitive or unsafe behavior was not introduced, user-facing documentation is updated when needed, and feature adoption docs under `docs/zh-CN/` were added or updated when functionality was added, removed, or refactored.

Applied to files:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: When a stage inside an active topic is fully complete, move the finished artifacts into that topic's `archive/` directory instead of leaving every completed step in the default boot path. When completed and validated stages begin to accumulate, archive their detailed history out of the active `todos/` and `traces/` entry files. Keep only the current recovery point, active facts, active risks, immediate next step, and pointers to relevant archive files in the default boot path.

Applied to files:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: When a topic is fully complete, move the entire topic directory under `ai-plan/public/archive/<topic>/` and remove it from `ai-plan/public/README.md` in the same change.

Applied to files:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Use `ai-plan/public/**` only for durable, handoff-safe task state. Put temporary notes, local experiments, or worktree-specific scratch recovery data under `ai-plan/private/`.

Applied to files:

  • ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md
🪛 LanguageTool
ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md

[uncategorized] ~76-~76: 您的意思是“"不"齐”?
Context: ...i-plan/public/coroutine-optimization/,并补齐: - todos/ -traces/ -archi...

(BU)


[uncategorized] ~81-~81: 您的意思是“"不"写”?
Context: ... local-plan 中分散的五个阶段计划整合进主题内历史跟踪归档,并额外补写一份基于 todo 基线整理出的历史 trace, 显式记录"缺少原始 t...

(BU)


[uncategorized] ~95-~95: 您的意思是“"不"写”?
Context: ...,并明确标注推导边界 2. 保持新 topic 的 active 入口精简,不把补写 trace 变成伪造逐日执行日志

(BU)

🔇 Additional comments (2)
ai-plan/public/ai-plan-governance/traces/ai-plan-governance-trace.md (2)

81-82: 良好的边界标注实践。

Line 81-82 明确记录"补写基于 todo 基线的历史 trace"并显式标注"缺少原始 trace,只能恢复稳定结论"的边界,这是处理遗留早期计划的最佳实践,既保证了追踪的完整性,又避免伪造详细执行历史。


92-95: "下一步"指导明确且可操作。

Line 92-95 为后续遇到类似"只有 todo、没有 trace"的早期计划提供了清晰的处理模式,强调保持 active 入口精简并明确推导边界,符合治理规则对可恢复性与精简性的双重要求。

@coderabbitai

coderabbitai Bot commented Apr 19, 2026

Copy link
Copy Markdown
Contributor

Note

Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it.

The agent ran but didn't make any changes. The issues may already be fixed or require manual intervention.

- 添加协程优化分支信息到文档
- 更新工作树提示为 GFramework 协程优化
- 添加协程优化优先级标记
@github-actions

Copy link
Copy Markdown

Summary

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Other ❓ Flaky 🍂 Duration ⏱️
2103 2103 0 0 0 0 35.5s    ↓235ms

Test Results

passed 2103 passed

Details

tests 2103 tests
clock 35.5s ↓235ms
tool nunit
build CI - Build & Test arrow-right build-and-test link #891
pull-request docs(ai-plan): 迁移 coroutine 早期计划 link #257

Insights

Average Tests per Run Total Flaky Tests Total Failed Slowest Test (p95)
2103 0 0 4.7s

build-and-test: Run #891

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Pending ⏳ Other ❓ Flaky 🍂 Duration ⏱️
2103 2103 0 0 0 0 0 35.5s

🎉 All tests passed!

Slowest Tests

Test 📝 Results 📊 Duration (avg) ⏱️ Duration (p95) ⏱️
SendRequestAsync_Should_ResolveCqrsRuntime_OnlyOnce_When_AccessedConcurrently 3 4.4s 4.7s
Does_Not_Report_When_FieldInjectedModel_Is_Registered 3 2.1s 2.1s
Generates_Scene_Behavior_Boilerplate 3 1.9s 1.9s
CleanupDuringAcquire_Should_NotCauseRaceCondition 3 1.1s 1.1s
Append_ShouldNotBlock 3 1.0s 1.0s
Context_Caching_Should_Improve_Performance 3 781ms 783ms
PendingCount_ShouldReflectQueuedEntries 3 501ms 501ms
Cleanup_Should_NotRemoveActiveLocks 3 404ms 404ms
Cleanup_Should_RemoveUnusedLocks 3 401ms 402ms
Generates_Precise_Assembly_Type_Lookups_For_Inaccessible_External_Protected_Types 3 331ms 334ms

± Comparison with run #890 at 47292e2 | 🎉 No failed tests detected across all runs. | 🍂 No flaky tests detected across all runs. | ⏱️ Measured over 3 runs.

Github Test Reporter by CTRF 💚

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@ai-plan/public/README.md`:
- Around line 61-64: 删除 README 中重复的映射块:移除第二处关于分支名 `feat/coroutine-optimization`
的映射定义(当前出现在行 61-64 的块),仅保留首个声明(行 54-57)并确保该保留块的缩进与周围映射一致以避免 boot
读取顺序或优先级歧义;确认最终文件只包含一处 `feat/coroutine-optimization` 映射且索引/优先级字段(如 `Worktree
hint`、`Priority`)在该唯一块中正确无误。
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d0164e22-f72b-4625-bafe-67ab00f1ac63

📥 Commits

Reviewing files that changed from the base of the PR and between c3614b5 and da7d03d.

📒 Files selected for processing (1)
  • ai-plan/public/README.md
📜 Review details
⏰ Context from checks skipped due to timeout of 900000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: Code Quality & Security
  • GitHub Check: Build and Test
  • GitHub Check: Analyze (C#)
🧰 Additional context used
📓 Path-based instructions (3)
**/README.md

📄 CodeRabbit inference engine (AGENTS.md)

Every user-facing package or module directory that contains a *.csproj intended for direct consumption MUST have a sibling README.md with canonical filename README.md. A module README MUST describe the module's purpose, relationship to adjacent runtime/abstractions/generator packages, major subdirectories/subsystems, minimum adoption path, and corresponding docs/zh-CN/ entry points.

Files:

  • ai-plan/public/README.md
ai-plan/public/**/*

📄 CodeRabbit inference engine (AGENTS.md)

ai-plan/public/**/*: Contributors MUST keep committed ai-plan/public/** content safe to publish in Git history. Never write secrets, tokens, credentials, private keys, machine usernames, home-directory paths, hostnames, IP addresses, proprietary URLs, or other sensitive environment details into any ai-plan/** file.
Never record absolute file-system paths in ai-plan/**; use repository-relative paths, branch names, PR numbers, or stable document identifiers instead.

Files:

  • ai-plan/public/README.md
ai-plan/public/README.md

📄 CodeRabbit inference engine (AGENTS.md)

ai-plan/public/README.md MUST list only active topics. Do not add ai-plan/public/archive/** content to the default boot index. When a worktree-to-topic mapping changes or when a topic becomes active/inactive, contributors MUST update ai-plan/public/README.md in the same change.

Files:

  • ai-plan/public/README.md
🧠 Learnings (7)
📓 Common learnings
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to docs/zh-CN/**/*.md : When a feature is added, removed, renamed, or substantially refactored, contributors MUST update or create the corresponding user-facing integration documentation in `docs/zh-CN/` in the same change. For integration-oriented features, documentation MUST cover project directory layout, file conventions, required project/package wiring, minimal working usage example, and migration/compatibility notes.
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: For any multi-step refactor, migration, or cross-module task, contributors MUST create or adopt a dedicated recovery document under `ai-plan/public/<topic>/todos/` before making substantive code changes. Recovery documents MUST record the current phase, the active recovery point identifier, known risks, and the next recommended resume step so another contributor or subagent can continue the work safely.
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to ai-plan/public/*/todos/**/* : When working from a tracked implementation plan, contributors MUST update the corresponding tracking document under `ai-plan/public/<topic>/todos/` in the same change. Updates MUST reflect completed work, newly discovered issues, validation results, and the next recommended recovery point. Tracking files are recovery entrypoints, not append-only changelogs, and MUST stay concise.
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Commit messages MUST use Conventional Commits format: `<type>(<scope>): <summary>`. The commit `summary` MUST use simplified Chinese and briefly describe the main change. The commit `body` MUST use unordered list items, each starting with a verb such as `新增`、`修复`、`优化`、`更新`、`补充`、`重构`. Each commit body bullet MUST describe one independent change point. Keep technical terms in English when they are established project terms, such as `API`、`Model`、`System`.
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: When a topic is fully complete, move the entire topic directory under `ai-plan/public/archive/<topic>/` and remove it from `ai-plan/public/README.md` in the same change.
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to ai-plan/public/README.md : `ai-plan/public/README.md` MUST list only active topics. Do not add `ai-plan/public/archive/**` content to the default boot index. When a worktree-to-topic mapping changes or when a topic becomes active/inactive, contributors MUST update `ai-plan/public/README.md` in the same change.
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Use `ai-plan/public/**` only for durable, handoff-safe task state. Put temporary notes, local experiments, or worktree-specific scratch recovery data under `ai-plan/private/`.
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to ai-plan/public/*/traces/**/* : Contributors MUST maintain a matching execution trace under `ai-plan/public/<topic>/traces/` for complex work. The trace should record the current date, key decisions, validation milestones, and the immediate next step. Update traces at each meaningful milestone before pausing or handing work off.
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to ai-plan/public/README.md : `ai-plan/public/README.md` MUST list only active topics. Do not add `ai-plan/public/archive/**` content to the default boot index. When a worktree-to-topic mapping changes or when a topic becomes active/inactive, contributors MUST update `ai-plan/public/README.md` in the same change.

Applied to files:

  • ai-plan/public/README.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to ai-plan/public/*/todos/**/* : When working from a tracked implementation plan, contributors MUST update the corresponding tracking document under `ai-plan/public/<topic>/todos/` in the same change. Updates MUST reflect completed work, newly discovered issues, validation results, and the next recommended recovery point. Tracking files are recovery entrypoints, not append-only changelogs, and MUST stay concise.

Applied to files:

  • ai-plan/public/README.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: When a topic is fully complete, move the entire topic directory under `ai-plan/public/archive/<topic>/` and remove it from `ai-plan/public/README.md` in the same change.

Applied to files:

  • ai-plan/public/README.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: Applies to ai-plan/public/*/traces/**/* : Contributors MUST maintain a matching execution trace under `ai-plan/public/<topic>/traces/` for complex work. The trace should record the current date, key decisions, validation milestones, and the immediate next step. Update traces at each meaningful milestone before pausing or handing work off.

Applied to files:

  • ai-plan/public/README.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: For any multi-step refactor, migration, or cross-module task, contributors MUST create or adopt a dedicated recovery document under `ai-plan/public/<topic>/todos/` before making substantive code changes. Recovery documents MUST record the current phase, the active recovery point identifier, known risks, and the next recommended resume step so another contributor or subagent can continue the work safely.

Applied to files:

  • ai-plan/public/README.md
📚 Learning: 2026-04-19T07:58:40.569Z
Learnt from: CR
Repo: GeWuYou/GFramework PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-04-19T07:58:40.569Z
Learning: If a task depends on observations from `ai-libs/**`, record the referenced path and conclusion in the active plan or trace when the work is multi-step or complex, or when an active tracking document already exists, rather than editing the third-party reference copy.

Applied to files:

  • ai-plan/public/README.md
🔇 Additional comments (1)
ai-plan/public/README.md (1)

28-32: 这两处索引更新方向正确,和本次 topic 激活目标一致。

coroutine-optimization 的 Active Topic 条目及其 Worktree 映射都已补齐,结构清晰,可作为恢复入口使用。

Based on learnings: ai-plan/public/README.md 在 worktree-to-topic 映射变化时必须在同一变更内更新。

Also applies to: 54-57

Comment thread ai-plan/public/README.md Outdated
- 移除了关于协程优化分支的文档条目
- 清理了已归档主题的相关工作树提示信息
- 更新了 SDK 使用说明以反映当前状态
@GeWuYou GeWuYou merged commit 58f3253 into main Apr 19, 2026
5 checks passed
@GeWuYou GeWuYou deleted the feat/coroutine-optimization branch April 19, 2026 13:59
@github-actions

Copy link
Copy Markdown

Summary

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Other ❓ Flaky 🍂 Duration ⏱️
2103 2103 0 0 0 0 1ms

Test Results

passed 2103 passed

Details

tests 2103 tests
clock 35.5s
tool nunit
build CI - Build & Test arrow-right build-and-test link #892
pull-request docs(ai-plan): 迁移 coroutine 早期计划 link #257

Insights

Average Tests per Run Total Flaky Tests Total Failed Slowest Test (p95)
2103 0 0 4.7s

build-and-test: Run #892

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Pending ⏳ Other ❓ Flaky 🍂 Duration ⏱️
2103 2103 0 0 0 0 0 35.5s

🎉 All tests passed!

Slowest Tests

Test 📝 Results 📊 Duration (avg) ⏱️ Duration (p95) ⏱️
SendRequestAsync_Should_ResolveCqrsRuntime_OnlyOnce_When_AccessedConcurrently 1 4.7s 4.7s
Does_Not_Report_When_FieldInjectedModel_Is_Registered 1 2.2s 2.2s
Generates_Scene_Behavior_Boilerplate 1 1.7s 1.7s
CleanupDuringAcquire_Should_NotCauseRaceCondition 1 1.1s 1.1s
Append_ShouldNotBlock 1 1.0s 1.0s
Context_Caching_Should_Improve_Performance 1 774ms 774ms
PendingCount_ShouldReflectQueuedEntries 1 501ms 501ms
Cleanup_Should_NotRemoveActiveLocks 1 405ms 405ms
Cleanup_Should_RemoveUnusedLocks 1 401ms 401ms
Generates_Precise_Assembly_Type_Lookups_For_Inaccessible_External_Protected_Types 1 304ms 304ms

🎉 No failed tests in this run. | 🍂 No flaky tests in this run.

Github Test Reporter by CTRF 💚

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant