Skip to content

Condense CLAUDE.md guidance for clarity and brevity - #865

Merged
sroussey merged 1 commit into
mainfrom
claude/audit-claude-md-files-q4672a
Aug 24, 2026
Merged

Condense CLAUDE.md guidance for clarity and brevity#865
sroussey merged 1 commit into
mainfrom
claude/audit-claude-md-files-q4672a

Conversation

@sroussey

Copy link
Copy Markdown
Collaborator

Streamline the CLAUDE.md documentation by removing redundancy, condensing verbose explanations, and improving readability without losing essential guidance.

Summary

This PR refactors CLAUDE.md to be more concise and scannable while preserving all critical information. The file is reorganized to prioritize the most important guidance and uses tighter language throughout.

Key changes

  • Condensed command descriptions — removed parenthetical elaborations in favor of inline brevity (e.g., "Full build (all packages + integrations + examples, via Turbo)" → "Full build (packages + integrations + examples, via Turbo)")
  • Simplified Node.js requirement — replaced multi-sentence explanation with a single bold statement and bullet point
  • Restructured monorepo section — moved PRD guidance and spec-reference rules into subsections; simplified dependency graph with arrow notation
  • Tightened per-package build explanation — condensed the "bun" export condition rules into a single focused paragraph with the key constraint highlighted
  • Consolidated key packages — removed redundant descriptions and reorganized @workglow/util named exports as a compact list
  • Streamlined code style rules — removed elaboration from TypeScript rules (e.g., "always named exports (except framework-required)" → "named exports only (except framework-required)")
  • Condensed testing section — removed verbose test runner explanation and reorganized vitest projects guidance
  • Simplified development workflow — tightened use-source / use-dist / link-all descriptions

Notable details

  • All substantive guidance is preserved; only redundant phrasing and verbose elaborations are removed
  • The dependency graph now uses arrow notation () for clearer visual flow
  • The "bun" export condition rules remain pinned to the test fixture as before
  • License header guidance and code style rules remain unchanged in substance

https://claude.ai/code/session_01AZS3ko2CqaJY9tcWdKfakg

Cut 453 lines to 304 by removing prose that duplicates JSDoc already on the
code it describes:

- the `skipPreamble` last-complete-wins rules and the `push()` live-root
  aliasing contract are documented in full on `PartialJsonStreamOptions` /
  `PartialJsonStream`
- the three decode-usage bullets (prompt length as input count, 250ms throttle,
  `cached`/`cacheWrite` left undefined) are the JSDoc on
  `createDecodeUsageReporter`

Both are now one-line rules pointing at the module. The provider run-fn
conventions are reorganized as a rule list rather than a sequence of essays,
and the web-console and dev-without-building sections keep their invariants
without the surrounding narrative.

No rule was dropped: every identifier referenced by the old file still appears,
except four internals whose own JSDoc covers them.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AZS3ko2CqaJY9tcWdKfakg
@sroussey
sroussey merged commit 2c912b7 into main Aug 24, 2026
13 checks passed
@sroussey
sroussey deleted the claude/audit-claude-md-files-q4672a branch August 24, 2026 22:28
@github-actions

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines Unknown% 0 / 0
🔵 Statements Unknown% 0 / 0
🔵 Functions Unknown% 0 / 0
🔵 Branches Unknown% 0 / 0
File CoverageNo changed files found.
Generated in workflow #3291 for commit c8152d6 by the Vitest Coverage Report Action

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