docs: update CHANGELOG and file new backlog items#59
Conversation
CHANGELOG: document invariants, legends, bearing, backlog triage, doctor fix, and rg→grep test fix under [Unreleased]. New backlog items filed: - task:backlog-doc-sweep (29 description-less quests need docs) - task:seed-scripts-cleanup (retire or keep one-shot scripts) - task:campaign-to-legend-migration (campaign→legend edge migration) - task:cycle-entity (cycle:* as first-class graph entity)
📝 WalkthroughWalkthroughCHANGELOG.md documents new graph-ontology primitives including 9 invariants, 5 legends, and 3 edge types; operational backlog triage reduction from 158 to 53 quests; diagnostic-control-flow restriction in Doctor service to PLANNED/READY/IN_PROGRESS states; and CI reliability improvements in diagnostic logging tests. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
There was a problem hiding this comment.
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 `@CHANGELOG.md`:
- Line 17: Update the CHANGELOG description to accurately reflect the differing
filters used by the two Doctor scans: state that the "readiness" scan excludes
BACKLOG, GRAVEYARD, DONE and therefore checks PLANNED, READY, IN_PROGRESS and
BLOCKED, while the "governed-completion" scan excludes only BACKLOG and
GRAVEYARD and therefore checks PLANNED, READY, IN_PROGRESS, BLOCKED and DONE;
reference the two checks by their names ("readiness checks" and
"governed-completion checks") and list the exact statuses (PLANNED, READY,
IN_PROGRESS, BLOCKED, DONE) so the changelog aligns with the actual code logic.
🪄 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: 28349a49-26db-46d1-adc5-42b7dc931eb4
📒 Files selected for processing (1)
CHANGELOG.md
| ### Changed | ||
|
|
||
| - **Backlog triaged from 158 open quests to 53** — 11 quests sealed as already completed, ~88 graveyarded with full "WHAT IT WAS" intent reconstructions, 24 stubs merged into 6 consolidated cycle-worthy items, 18 rethought (12 graveyarded, 1 reset, 5 reframed with new titles and descriptions reflecting METHOD adoption). | ||
| - **Doctor no longer flags BACKLOG, GRAVEYARD, or DONE quests for readiness and governed-completion gaps** — these scans now only check PLANNED/READY/IN_PROGRESS quests, eliminating 52 false diagnostics from graveyarded and sealed work. Doctor diagnostics went from 55 to 3 (all legitimate PLANNED quest work). |
There was a problem hiding this comment.
Incomplete description of Doctor filtering behavior.
The CHANGELOG states both checks "only check PLANNED/READY/IN_PROGRESS quests", but the actual code filters are different:
- Readiness checks (context snippet 1, line 970): exclude
BACKLOG,GRAVEYARD,DONE→ checks PLANNED, READY, IN_PROGRESS, BLOCKED (missing BLOCKED in the description) - Governed-completion checks (context snippet 2, line 1116-1117): exclude
BACKLOG,GRAVEYARDonly → checks PLANNED, READY, IN_PROGRESS, BLOCKED, DONE (incorrectly excludes DONE and BLOCKED in the description)
The two checks have different filter logic but are described identically.
📝 Suggested fix for accuracy
-**Doctor no longer flags BACKLOG, GRAVEYARD, or DONE quests for readiness and governed-completion gaps** — these scans now only check PLANNED/READY/IN_PROGRESS quests, eliminating 52 false diagnostics from graveyarded and sealed work. Doctor diagnostics went from 55 to 3 (all legitimate PLANNED quest work).
+**Doctor no longer flags BACKLOG and GRAVEYARD quests for readiness and governed-completion gaps** — readiness scans now check PLANNED/READY/IN_PROGRESS/BLOCKED quests (excluding DONE), while governed-completion scans check PLANNED/READY/IN_PROGRESS/BLOCKED/DONE quests. This change eliminated 52 false diagnostics from graveyarded and backlog work. Doctor diagnostics went from 55 to 3 (all legitimate PLANNED quest work).📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| - **Doctor no longer flags BACKLOG, GRAVEYARD, or DONE quests for readiness and governed-completion gaps** — these scans now only check PLANNED/READY/IN_PROGRESS quests, eliminating 52 false diagnostics from graveyarded and sealed work. Doctor diagnostics went from 55 to 3 (all legitimate PLANNED quest work). | |
| - **Doctor no longer flags BACKLOG and GRAVEYARD quests for readiness and governed-completion gaps** — readiness scans now check PLANNED/READY/IN_PROGRESS/BLOCKED quests (excluding DONE), while governed-completion scans check PLANNED/READY/IN_PROGRESS/BLOCKED/DONE quests. This change eliminated 52 false diagnostics from graveyarded and backlog work. Doctor diagnostics went from 55 to 3 (all legitimate PLANNED quest work). |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@CHANGELOG.md` at line 17, Update the CHANGELOG description to accurately
reflect the differing filters used by the two Doctor scans: state that the
"readiness" scan excludes BACKLOG, GRAVEYARD, DONE and therefore checks PLANNED,
READY, IN_PROGRESS and BLOCKED, while the "governed-completion" scan excludes
only BACKLOG and GRAVEYARD and therefore checks PLANNED, READY, IN_PROGRESS,
BLOCKED and DONE; reference the two checks by their names ("readiness checks"
and "governed-completion checks") and list the exact statuses (PLANNED, READY,
IN_PROGRESS, BLOCKED, DONE) so the changelog aligns with the actual code logic.
Summary
task:backlog-doc-sweep— 29 quests need METHOD-style docstask:seed-scripts-cleanup— decide fate of one-shot seed scriptstask:campaign-to-legend-migration— migrate campaign edges to legendstask:cycle-entity— make cycle:* first-class (the first real FLOW cycle)Test plan
Summary by CodeRabbit
New Features
Improvements
Chores