Skip to content

chore: cleanup proposals, plans, and reports folder structure#218

Merged
HumanBean17 merged 3 commits into
masterfrom
chore/cleanup-proposals-plans-reports
May 24, 2026
Merged

chore: cleanup proposals, plans, and reports folder structure#218
HumanBean17 merged 3 commits into
masterfrom
chore/cleanup-proposals-plans-reports

Conversation

@HumanBean17
Copy link
Copy Markdown
Owner

Summary

  • Move completed proposes (HINTS-STRUCTURED-LABEL) to propose/completed/
  • Move stale proposes to propose/stale/ (ENHANCED-ROLE-RECOGNITION, INDEX-AUTO-MODE, TIER2-INCREMENTAL-REBUILD, RANKING-MICROSERVICE)
  • Move completed plans to plans/completed/ (PLAN-DESCRIBE-HINTS-STRUCTURAL, AGENT-PROMPTS-DESCRIBE-HINTS-STRUCTURAL)
  • Create propose/active/ and plans/active/ for future active items
  • Move PRODUCT-VISION.md to docs/
  • Move CODEBASE_REQUIREMENTS.md to docs/
  • Remove reports/ folder (call-graph-review.md, what-to-borrow-from-cmm.md, review/)

Changes

  • Folder structure cleanup to better organize completed, active, and stale work
  • Documentation files moved to docs/ for better discoverability
  • Removed obsolete reports/ folder

🤖 Generated with Claude Code

- Move completed proposes (HINTS-STRUCTURED-LABEL) to propose/completed/
- Move stale proposes to propose/stale/ (ENHANCED-ROLE-RECOGNITION, INDEX-AUTO-MODE, TIER2-INCREMENTAL-REBUILD, RANKING-MICROSERVICE)
- Move completed plans to plans/completed/ (PLAN-DESCRIBE-HINTS-STRUCTURAL, AGENT-PROMPTS-DESCRIBE-HINTS-STRUCTURAL)
- Create propose/active/ and plans/active/ for future active items
- Move PRODUCT-VISION.md to docs/
- Move CODEBASE_REQUIREMENTS.md to docs/
- Remove reports/ folder (call-graph-review.md, what-to-borrow-from-cmm.md, review/)

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@HumanBean17
Copy link
Copy Markdown
Owner Author

Review: broken references from moves and deletions

The reorganization is reasonable, but this PR introduces dead links that need to be resolved before merging.

Deleted files still referenced

reports/what-to-borrow-from-cmm.md is referenced by:

  • propose/completed/TIER1B-HTTP-ASYNC-EDGES-PROPOSE.md
  • propose/completed/TIER1-COMPLETION-PROPOSE.md

reports/call-graph-review.md is referenced by:

  • propose/completed/TIER1B-HTTP-ASYNC-EDGES-PROPOSE.md
  • propose/completed/TIER1-COMPLETION-PROPOSE.md
  • plans/completed/PLAN-TIER1B-COMPLETION.md
  • plans/completed/AGENT-PROMPTS-TIER1B.md

reports/review/completed/PLAN-BROWNFIELD-ROLE-OVERRIDES-design-issues.md is referenced by:

  • plans/completed/PLAN-BROWNFIELD-ROLE-OVERRIDES-design-fixes.md

These reviews contain actionable bug lists and design invariants that the proposals explicitly cite — deleting them breaks the paper trail.

Moved files with stale links

CODEBASE_REQUIREMENTS.mddocs/CODEBASE_REQUIREMENTS.md leaves broken references in:

  • README.md, AGENTS.md, and 15+ proposal/plan files

propose/PRODUCT-VISION.mddocs/PRODUCT-VISION.md leaves broken references in:

  • README.md and 8+ proposal/plan files

Suggested fix

Either:

  1. Don't delete reports/ — move it to docs/reports/ or reports/completed/ instead, or
  2. Update all referencing files to point to the new locations (and confirm nothing of value is lost by the deletions).

At minimum, README.md and AGENTS.md links need updating for the docs/ moves.

🤖 Generated with Claude Code

- Move reports/ to docs/reports/ to preserve technical content
- Update all references to CODEBASE_REQUIREMENTS.md → docs/CODEBASE_REQUIREMENTS.md
- Update all references to PRODUCT-VISION.md → docs/PRODUCT-VISION.md
- Update reports/ references to docs/reports/
- Fix broken links in README.md, AGENTS.md, and 21 proposal/plan files

The deleted reports contained valuable technical content:
- what-to-borrow-from-cmm.md: borrowing patterns guide from Codebase-Memory MCP
- call-graph-review.md: code review with bugs and design invariants
- PLAN-BROWNFIELD-ROLE-OVERRIDES-design-issues.md: design issues and gaps

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@HumanBean17
Copy link
Copy Markdown
Owner Author

Re-review: fixes applied

Good improvements since the first review — reports/ is now moved to docs/reports/ (not deleted), and most references are updated. A few gaps remain.

What's fixed

  • reports/docs/reports/ (preserved, not deleted)
  • README.md, AGENTS.md, docs/JAVA-CODEBASE-RAG-CLI.md updated for docs/ moves
  • Most plans/completed/ and propose/completed/ files updated for both CODEBASE_REQUIREMENTS.md and PRODUCT-VISION.md moves
  • 3 unreferenced implementation-review reports deleted cleanly

Remaining stale references

reports/call-graph-review.md not updated to docs/reports/... in:

  • plans/completed/AGENT-PROMPTS-TIER1B.md (lines 483, 558)
  • plans/completed/PLAN-TIER1B-COMPLETION.md (line 830)

CODEBASE_REQUIREMENTS.md not updated to docs/CODEBASE_REQUIREMENTS.md in:

  • .agents/skills/plan-project-scope/SKILL.md (line 34)
  • .agents/skills/propose/SKILL.md (line 30)
  • plans/completed/AGENT-PROMPTS-CLI-SCENARIOS.md (lines 287, 329, 353)
  • propose/completed/BROWNFIELD-ANNOTATIONS-V2-PROPOSE.md (lines 356, 395)
  • propose/completed/HTTP-ROUTE-METHOD-ENUM-PROPOSE.md (lines 304, 425)

(docs/CONFIGURATION.md and docs/MANUAL-VERIFICATION-CHECKLIST.md also reference bare CODEBASE_REQUIREMENTS.md but resolve correctly as sibling files — these are fine.)

Verdict

Almost there. The 3 stale reports/ references and 7 stale CODEBASE_REQUIREMENTS.md references in non-docs/ files need updating, then this is ready to merge.

🤖 Generated with Claude Code

The link was pointing to ./docs/CONFIGURATION.md which is incorrect from
within the docs/ directory. Changed to ./CONFIGURATION.md.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@HumanBean17 HumanBean17 merged commit c217421 into master May 24, 2026
1 check passed
@HumanBean17 HumanBean17 deleted the chore/cleanup-proposals-plans-reports branch May 24, 2026 12:08
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