Skip to content

docs: Trim CLAUDE.md 64% (166 → 70 lines)#51

Merged
evansenter merged 1 commit into
mainfrom
docs/trim-claude-md
Jan 4, 2026
Merged

docs: Trim CLAUDE.md 64% (166 → 70 lines)#51
evansenter merged 1 commit into
mainfrom
docs/trim-claude-md

Conversation

@evansenter

Copy link
Copy Markdown
Owner

Summary

  • Trim CLAUDE.md from 166 to 70 lines (64% reduction)
  • Move reference material to guide.md where it belongs
  • Keep critical content: DB protection, design philosophy, commands, key patterns
  • Cut: key files table, naming conventions, data model, architecture diagram, verbose examples

Test plan

  • make check passes (289 tests)
  • All essential guidance retained
  • Reference material already exists in guide.md

🤖 Generated with Claude Code

Move reference material to guide.md where it belongs. Keep:
- Database protection (critical safety)
- Design philosophy (condensed)
- Commands + restart guidance
- Key patterns
- Adding endpoints checklist (condensed)

Cut: key files table, naming conventions, data model, architecture
diagram, verbose code examples, RFC reference.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@claude

claude Bot commented Jan 4, 2026

Copy link
Copy Markdown

Prompt: evansenter/dotfiles/.../claude-review.md

Code Review

Summary

This PR reduces CLAUDE.md by 64% (166→70 lines) by condensing verbose explanations and removing reference material that either exists in guide.md or is easily discoverable through code exploration.

Issues Found

Critical

None

Important

None

Suggestions

  • The naming conventions table (list_*, get_*, search_*, analyze_* prefixes and standard arg names like session_id, limit, days, project) was removed but isn't explicitly documented in guide.md. While the prefixes are mentioned in the condensed "Adding Endpoints" section, the standard argument naming conventions are not. Consider adding a brief note about standard args to guide.md to ensure this knowledge isn't lost.

Verdict

REQUEST_CHANGES - One minor suggestion about preserving naming conventions documentation.


Automated review by Claude Code

@evansenter evansenter merged commit 5526bb8 into main Jan 4, 2026
3 checks passed
@evansenter evansenter deleted the docs/trim-claude-md branch January 4, 2026 03:51
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