chore: Remove completed planning docs (CLEANUP_PLAN.md, FEATURE_INVENTORY.md)#43
Merged
Merged
Conversation
CLEANUP_PLAN.md (PR #27) drove the 12-PR cleanup effort: dead-code deletion, test reorg, coverage to 80%, feature discoverability, the new Workflows submenu, and the help-overlay rebuild. All tasks in the plan landed (PRs #28-42). docs/FEATURE_INVENTORY.md (PR #30) was a point-in-time snapshot of every show* flag's reachability status; it informed C2.a/b and B4. The flags it lists have since shifted: some were promoted into submenus, 7 were dropped, and the live README + ? overlay now reflect the current feature surface. Both docs remain in git history if needed for reference.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Removes two planning documents that drove the cleanup effort. Both are completed; their content is preserved in git history and superseded by the live state of the project.
CLEANUP_PLAN.md(added in Add CLEANUP_PLAN.md #27) — the three-workstream plan that drove PRs A1: Remove dead test file and checked-in coverage artifacts #28–42 (dead-code deletion, test reorg, coverage push to 80%, discoverability fixes, new Workflows submenu, rebuilt?overlay). Every task in the plan landed.docs/FEATURE_INVENTORY.md(added in C1: Add docs/FEATURE_INVENTORY.md #30) — the per-flag reachability snapshot. It was the input for C2.a/b and B4. The flags it lists have since shifted: some were promoted into submenus, 7 were dropped, and the live README +?overlay are now the source of truth for what's actually shipped.Both docs are searchable in git history if anyone wants the historical context.
Test plan
git diff --statconfirms only the two docs were removedgrep -rn 'CLEANUP_PLAN\|FEATURE_INVENTORY' .should come up empty after this lands)https://claude.ai/code/session_01LtDEmp1JiMecLNhq8YqKP2
Generated by Claude Code