Reconcile roadmap with implemented workspace#82
Conversation
cosentinode
left a comment
There was a problem hiding this comment.
Review findings
-
[P1] Phase 13 status is already stale.
README.mdlines 40-43 says the Phase 13 and 14 umbrella issues remain open, anddocs/mvp-phases.mdlines 252-258 specifically says #15 remains open. Issue #15 is closed, with its closing comment explicitly declaring Phase 13 complete. This contradicts #58 acceptance criteria that phase status match issue state; mark Phase 13 complete/closed and avoid grouping its umbrella status with still-open #16. -
[P1] The planner documentation still says the merged Phase 11 behavior is unavailable.
docs/command-planner.mdlines 45-49 labelsopen prand the combined PR sequence as not completed, and lines 83-85 still tell users they must wait for #13. Issue #13 is closed, the implementation is present, and this PR now declares Phase 11 merged elsewhere. Update this surrounding workflow document too so the reconciled docs do not directly contradict each other.
|
Addressed both review concerns in f9cbd62:
Re-checked current GitHub state: #13 and #15 are closed; #16 remains open. Verification:
|
|
Fresh independent review of current head f9cbd62: zero concerns. I compared the complete diff with issue #58, live issue/PR/release state, Cargo metadata, the tracked tree, CI and release workflows, and surrounding documentation. The six-crate workspace and tracked layout are complete, roadmap statuses and links match the tracker, release status is accurately qualified, both repository validation scripts pass, |
Summary
Verification
cargo metadata --locked --no-deps --format-version 1git diff --checkbash scripts/test-release-workflow.shbash scripts/test-installation-docs.shghFixes #58