Skip to content

Comments

Trim CLAUDE.md from 792 to 160 lines#175

Merged
igerber merged 1 commit intomainfrom
claude-md-cleanup
Feb 20, 2026
Merged

Trim CLAUDE.md from 792 to 160 lines#175
igerber merged 1 commit intomainfrom
claude-md-cleanup

Conversation

@igerber
Copy link
Owner

@igerber igerber commented Feb 19, 2026

Summary

  • Rewrite CLAUDE.md from 792 → 160 lines (80% reduction) to improve instruction-following quality
  • Extract 6 development checklists + grep patterns to .claude/commands/dev-checklists.md (accessible via /dev-checklists)
  • Extract documentation requirements + test writing guidelines to CONTRIBUTING.md
  • Update cross-references in review-plan.md, docs-check.md, and check-plan-review.sh
  • Remove discoverable content: module structure listing, performance architecture, benchmark docs, test structure listing

Methodology references (required if estimator / math changes)

  • N/A — no methodology changes

Validation

  • Tests added/updated: No test changes (documentation-only refactor)
  • No runtime code modified — CLAUDE.md, command files, and hook are agent instructions only

Security / privacy

  • Confirm no secrets/PII in this PR: Yes

Generated with Claude Code

Extract development checklists to .claude/commands/dev-checklists.md (accessible
via /dev-checklists) and documentation requirements + test guidelines to
CONTRIBUTING.md. Remove module structure listing, performance architecture,
benchmark docs, and test structure listing — all discoverable from the codebase.
Update cross-references in review-plan.md, docs-check.md, and
check-plan-review.sh.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link

Overall assessment: ✅ Looks good

Executive Summary

  • Documentation-only refactor; no estimator, math, inference, or runtime code paths changed in the diff.
  • References updated to point documentation requirements to CONTRIBUTING.md and development checklists to .claude/commands/dev-checklists.md, with review-plan and hook messaging aligned (.claude/commands/docs-check.md:L198-L205, .claude/commands/review-plan.md:L74-L86, .claude/hooks/check-plan-review.sh:L16-L33).
  • No tests needed for a doc-only change; new contributor guidance centralized (CONTRIBUTING.md:L1-L77).

Methodology

  • No findings. No estimator/math/SE/inference changes in this PR, so docs/methodology/REGISTRY.md cross-check is not applicable.

Code Quality

  • No findings.

Performance

  • No findings.

Maintainability

  • No findings.

Tech Debt

  • No findings.

Security

  • No findings.

Documentation/Tests

  • No findings. Doc changes are internally consistent across updated command docs and contributor guidance.

@igerber igerber merged commit 4ea2f25 into main Feb 20, 2026
1 check passed
@igerber igerber deleted the claude-md-cleanup branch February 20, 2026 00:18
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