docs: wave-table compliance fixes for #1902 rulings (items 5/6 of #1907) - #1910
Conversation
Model: claude-haiku-4-5-20251001 Co-Authored-By: Claude <noreply@anthropic.com>
📝 WalkthroughSummary by CodeRabbit
WalkthroughWave 2 计划从 R10 扩展为明确执行 R10 P1，并定义 R3/R6 dropped-loot hydration、projection 与 R10 子计划的固定顺序。原有 production activation 条件保持不变。 ChangesWave 2 R10 计划
Estimated code review effort: 1 (Trivial) | ~3 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Central reviewDecision: No validated findings or suggestions survived the five-vote gate. |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/plans-skeleton/plan-refactor-master-v1.md`:
- Around line 57-59: 补齐或重构文档中的 §10 实施工作流，使其明确覆盖 diff 中 R3、R10、R6、R5、R4 等多 PR
的完整依赖实施顺序，并保留各阶段的前置约束。将现有 flash-review issue 消化流程调整为该工作流的一部分或移至适当位置；§10
末尾必须新增“单次 consume-plan 全自动到 merge”章节，说明用户提交 /consume-plan 后可持续执行直至归档到
docs/finished_plans/。
🪄 Autofix
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: Repository UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: e3b14940-9b1f-426d-9a88-bad8c2726fce
📒 Files selected for processing (1)
docs/plans-skeleton/plan-refactor-master-v1.md
📜 Review details
🧰 Additional context used
📓 Path-based instructions (3)
docs/**/*.md
📄 CodeRabbit inference engine (docs/CLAUDE.md)
docs/**/*.md: 新 plan 头部必须写明接入面：进料、出料、复用的共享类型/event/schema、server/agent/client 跨仓库契约，以及对应的worldview.md锚点。
涉及真元、灵气、衰减、逸散、半衰、距离损耗、排斥或吸力的 plan，必须调用qi_physics；新物理常数和公式必须先扩展qi_physics，不得在功能 plan 中重复实现。
所有真元/灵气流动必须遵守守恒律并通过qi_physics::ledger::QiTransfer；释放使用qi_release_to_zone，吸收使用qi_excretion，不得凭空生成或销毁真元。
涉及玩家可感知行为的 plan，必须在对应机制阶段中内联可直接实现的粒子、音效、HUD、环境、动画和 narration 规格；不得将视听内容笼统推迟到独立阶段。纯 server 逻辑 plan 例外。
每份 plan 必须列出开放问题；实施前必须追加§N.1 决议，逐项给出结论、实施方案、边界条件，并以文件:行号和 plan 章节双锚点落地。
scope 大于或等于 4 个 PR 的 plan，必须在末尾包含§10 实施工作流，并按依赖顺序在一个 plan 内序列化多个 PR，不得拆成多个 plan。
涉及 NBT 建筑、worldgen layout 或复杂视觉资产的 TODO，必须完成三轮提交：(round 1/3)、(round 2/3)、(round 3/3)；终轮提交必须包含拼写准确的<PROMISE>担保块。
§10 最末必须包含“单次 consume-plan 全自动到 merge”章节，明确用户提交/consume-plan后即可等待最终归档至docs/finished_plans/。引用世界观内容时统一使用
worldview.md §X L<line>格式；不得自动修改docs/worldview.md或主动回写docs/library/。
Files:
docs/plans-skeleton/plan-refactor-master-v1.md
docs/plans-skeleton/**/*.md
📄 CodeRabbit inference engine (CLAUDE.md)
新 plan 必须先读取
docs/CLAUDE.md；骨架、Active 和 Finished plan 必须遵循三态流转及规定的阶段状态、Finish Evidence 结构。
Files:
docs/plans-skeleton/plan-refactor-master-v1.md
**/*
📄 CodeRabbit inference engine (CLAUDE.md)
**/*: commit message 必须使用中文，每个逻辑单元一个 atomic commit；agent 生成的 commit 必须带真实模型Model: <精确模型 id>trailer。
禁止使用--no-verify、--no-gpg-sign、关闭签名配置、未经确认的 force push、hard reset、amend 或交互式 rebase。
禁止留下 auto-stash 产生的孤儿 WIP stash；自动 stash 流程完成后必须恢复自己的 stash。
Files:
docs/plans-skeleton/plan-refactor-master-v1.md
🧠 Learnings (3)
📚 Learning: 2026-07-17T00:31:10.779Z
Learnt from: Kizunad
Repo: Kizunad/Bong PR: 1218
File: docs/plans-skeleton/plan-skill-av-relink-v1.md:1-1
Timestamp: 2026-07-17T00:31:10.779Z
Learning: 在审查该仓库 `docs/plans-skeleton/` 下的“docs-only skeleton plan”创建类 PR 时：先核对 `docs/CLAUDE.md` 中“Plan 消费规范”，并逐份查看本计划文档里的“§10 实施工作流”，确认后续实施阶段是否会遵循“每个 PR 只修改一个 plan”，且实施/归档时不会出现跨 plan 的修改。该规则不适用于用户显式指定、共享调研基线且计划之间存在互相交叉引用的 skeleton plan 同批创建 PR；对这类情况应按实际交叉引用关系放宽，确保仍能按独立或约定的序列化方式推进。
Applied to files:
docs/plans-skeleton/plan-refactor-master-v1.md
📚 Learning: 2026-07-17T00:31:13.643Z
Learnt from: Kizunad
Repo: Kizunad/Bong PR: 1218
File: docs/plans-skeleton/plan-skill-av-relink-v1.md:81-85
Timestamp: 2026-07-17T00:31:13.643Z
Learning: 审核 docs/plans-skeleton/*.md 下的 skeleton 草案 PR 时：不得要求作者在“§N 开放问题（P0 决策门前需收口）”核查完成之前提前填写对应的“§N.1 决议”。仅当计划进入 active 且 P0 实施前，已由 Explore agent 并行核查代码现状后，才允许追加“§N.1 决议”，且该决议需包含结论、实施方案、边界条件，并使用“文件:行号 + plan 章节”的双锚点格式。
Applied to files:
docs/plans-skeleton/plan-refactor-master-v1.md
📚 Learning: 2026-07-22T02:11:59.191Z
Learnt from: Kizunad
Repo: Kizunad/Bong PR: 1247
File: docs/plans-skeleton/plan-bughunt-animal-air-spawn-gravity-v1.md:0-0
Timestamp: 2026-07-22T02:11:59.191Z
Learning: 在 `docs/plans-skeleton/` 目录下的计划状态行中，若起草日期同时涉及 UTC 与本地日期（例如时区换算后可能跨到不同日期），请在该行中同时标注 UTC 起草日与本地起草日。这样可以避免基于 UTC 的时间基准在 GitHub/CodeRabbit 审查时将本地日期误判为“未来日期”。
Applied to files:
docs/plans-skeleton/plan-refactor-master-v1.md
🔇 Additional comments (1)
docs/plans-skeleton/plan-refactor-master-v1.md (1)
58-59: 🗄️ Data Integrity & Integration无需修改。
Line 59 已声明这些 phase/artifact/验收以各 owner plan 为准，R3/R4/R5/R6/R10 对应 plan 中也定义了本次引用中的阶段与消费边界。
| - **Wave 2**：R4、R9 production activation（R5、R6、R2 的所属责任按各自 plan 就绪后，服从 §4.1 的 ownership 与 atomicity invariants）、R10 P1（R3 P1 后）。**R6 dropped-loot P3 production activation merge unit 仅在 R10 P2a `DroppedLootEntry.owner/visibility` metadata provider 与 R3 P4 dropped-loot migration/hydration consumer 均合入后放行；此前只允许 declared/unwired/test-only 的 contract 与 pin artifacts。R10 P2b `OwnerOnly` private-writer activation 必须在该 R6 P3 merge unit 合入后放行，不得把 R6 P1/P2 的 unwired artifacts 当作 production consumer。**本表只裁决跨轨顺序与 activation 边界；各轨具体 deliverable inventory、phase mapping 与验收证据由各自 plan 定义，不在总纲重述。 | ||
| - **R3/R6 dropped-loot child-plan ordering**：R10 P1 migration helper 与 R3 P2 persistence seam/legacy compatibility pins 就绪后，R3 P4 dropped-loot hydration consumer 必须先于 R6 P3 dropped-loot projection/page production activation；R3 P4 inventory-layout overflow consumer 是独立子批次，仅在 R10 P3 合入后执行。 | ||
| - **R10 child-plan ordering**：顺序固定为 **R3 P1 → R10 P1 → R3 P2 → R10 P2a → R3 P4 dropped-loot hydration → R6 P3 → (R5 P3 + R6 P4) → R10 P3 → R4 pickup consumer → R10 P2b → R3 P4 inventory-layout overflow → R10 P4**；其中 R10 P2a 不得早于 R3 P2，R10 P3 不得早于 R5 P3/R6 P4，R4 pickup consumer 不得早于 R10 P3，R10 P2b 必须等待前述 R3/R6/R5/R10/R4 production consumers 全部完成。具体 phase/artifact/验收仍以 R3、R6、R10、R4、R5 各 owner plan 为准。 |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟠 Major | ⚡ Quick win
补齐强制的 §10 实施工作流。
Line [59] 已定义超过四个 PR 的依赖序列。当前 §10 是 flash-review issue 消化流程，不是按依赖顺序串联实施 PR 的工作流。
请补充或重构 §10，使其包含完整的多 PR 实施顺序。§10 末尾还必须包含“单次 consume-plan 全自动到 merge”章节，并说明用户提交 /consume-plan 后可持续执行至归档到 docs/finished_plans/。
As per coding guidelines：scope 大于或等于 4 个 PR 的 plan 必须包含 §10 实施工作流，且 §10 末尾必须包含上述 consume-plan 章节。
Based on learnings：审查 docs/plans-skeleton/ skeleton plan 时必须核对 §10 实施工作流 及后续 consume-plan/归档流程。
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/plans-skeleton/plan-refactor-master-v1.md` around lines 57 - 59,
补齐或重构文档中的 §10 实施工作流，使其明确覆盖 diff 中 R3、R10、R6、R5、R4 等多 PR
的完整依赖实施顺序，并保留各阶段的前置约束。将现有 flash-review issue 消化流程调整为该工作流的一部分或移至适当位置；§10
末尾必须新增“单次 consume-plan 全自动到 merge”章节，说明用户提交 /consume-plan 后可持续执行直至归档到
docs/finished_plans/。
Sources: Coding guidelines, Learnings
1 participant
Implements items 5 and 6 of the #1902 adjudication compliance tracker #1907: aligns the master §3 wave table and cross-track references with the five adjudication rulings (TypeBox schema authority, R6 generation-pipeline ownership, R9 cast-domain ownership, atomic cutover merge units, wave table as sole inter-track ordering authority).
Prepared by W1; branch head 5264320.
Part of #1907.