Skip to content

chore: adapt CLAUDE.md to Opus 5 with /doctor - #7517

Merged
diegolmello merged 5 commits into
developfrom
quartz-tuck
Jul 25, 2026
Merged

chore: adapt CLAUDE.md to Opus 5 with /doctor#7517
diegolmello merged 5 commits into
developfrom
quartz-tuck

Conversation

@diegolmello

@diegolmello diegolmello commented Jul 25, 2026

Copy link
Copy Markdown
Member

Proposed changes

CLAUDE.md is loaded into context at the start of every Claude Code session in this repo. Most of its 101 lines restated facts a session can read directly from the repo, so they cost context on every run without adding information — and one line was factually wrong.

Removed:

  • Stack and version bullets (React, React Native, Expo, TypeScript config, Node engines) — already in package.json and tsconfig.json, and they go stale silently.
  • The commands block, apart from corepack enable and the pod-install ordering note — every other entry is a standard package.json script.
  • Prettier and ESLint config restatement — already in .prettierrc.js and .eslintrc.js.
  • The Architecture section. The Redux / WatermelonDB / React Navigation bullets are all in package.json and the directory names match the concepts one-to-one. The app/ee/ Enterprise line and the "VoIP and VideoConf are separate features" warning are both covered better by CONTEXT.md, which is a ubiquitous-language glossary. What survives is two gotchas that nothing else states: the local-first data flow, and the fact that Redux and Zustand coexist so global state is not all in one place.
  • The Entry Points section.
  • The claim that pre-commit hooks enforce lint and tests. There is no .husky directory and no pre-commit config in the repo; CI is the only gate. Replaced with an accurate note.

Kept everything not derivable from the codebase: the CONTEXT.md pointer, platform floors, corepack enable, pod-install before iOS builds, the before-committing directive, the offline-first data-flow rationale, the note that VideoConf may be removed, and the warning that VoIP and VideoConf are separate features.

101 lines down to 31, roughly 830 fewer tokens of context per session.

Issue(s)

https://rocketchat.atlassian.net/browse/NATIVE-1444

How to test or reproduce

Read CLAUDE.md and confirm nothing removed is unavailable from the repo itself, and that nothing non-derivable was lost. Documentation only — no source code, build config, or test touched.

Screenshots

N/A

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • Improvement (non-breaking change which improves a current function)
  • New feature (non-breaking change which adds functionality)
  • Documentation update (if none of the other choices apply)

Checklist

  • I have read the CONTRIBUTING doc
  • I have signed the CLA
  • Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works (if applicable)
  • I have added necessary documentation (if applicable)
  • Any dependent changes have been merged and published in downstream modules

Further comments

Lint and unit tests were not run: this PR changes one Markdown file and touches no source, build config, or test.

The removed content is recoverable from this PR's diff if any of it turns out to be worth keeping. The Pre-commit hooks enforce these checks line was verified false: no .husky directory, no pre-commit config, and package.json has no prepare hook wiring one up.

Summary by CodeRabbit

  • Documentation
    • Simplified project guidance for a more focused development experience.
    • Reduced setup, testing, linting, and formatting instructions to essential commands.
    • Added concise notes about local-first data behavior and state management patterns.
    • Removed outdated or overly detailed architecture and tooling information.

@coderabbitai

coderabbitai Bot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

CLAUDE.md was shortened to retain project context, essential setup and validation commands, and concise notes about local-first synchronization and mixed state management.

Changes

CLAUDE.md guidance

Layer / File(s) Summary
Commands and code style guidance
CLAUDE.md
Project overview, setup instructions, and pre-commit commands are reduced to essential guidance, with other commands delegated to package.json scripts.
Architecture overview
CLAUDE.md
Detailed architecture documentation is replaced with notes describing WatermelonDB and saga synchronization, plus Redux global/server state and feature-local Zustand stores.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Suggested labels: type: chore

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title is specific and clearly related to the CLAUDE.md rewrite and guidance update.

Warning

Review ran into problems

🔥 Problems

Errors were encountered while retrieving linked issues.

Errors (1)
  • NATIVE-1444: Request failed with status code 401

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@CLAUDE.md`:
- Line 22: Update the “Before committing” guidance in CLAUDE.md to state that
pnpm prettier-lint formats the entire repository and may rewrite unrelated
files, while TZ=UTC pnpm test runs the full Jest suite. Remove the implication
that either command is limited to modified files, and retain the note that CI is
the only enforced gate.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: eddd4c1c-e64c-423b-b592-688a728f6498

📥 Commits

Reviewing files that changed from the base of the PR and between 1d9b5cc and c7a4a0b.

📒 Files selected for processing (1)
  • CLAUDE.md
📜 Review details
⏰ Context from checks skipped due to timeout. (2)
  • GitHub Check: ESLint and Test / run-eslint-and-test
  • GitHub Check: format
🔇 Additional comments (1)
CLAUDE.md (1)

7-19: LGTM!

Also applies to: 24-31

Comment thread CLAUDE.md
@diegolmello
diegolmello had a problem deploying to approve_e2e_testing July 25, 2026 14:47 — with GitHub Actions Failure
@diegolmello diegolmello changed the title docs: trim derivable content from CLAUDE.md chore: adapt CLAUDE.md to Opus 5 with /doctor Jul 25, 2026
@diegolmello
diegolmello merged commit fbd4243 into develop Jul 25, 2026
6 of 10 checks passed
@diegolmello
diegolmello deleted the quartz-tuck branch July 25, 2026 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants