fix: trim CLAUDE.md from 42.7k to 1.6k via ruler rule compression - #826
Conversation
|
You do not have enough credits to review this pull request. Please purchase more credits to continue. |
Summary of ChangesHello @shunkakinoki, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request addresses a performance warning related to the size of the Highlights
Changelog
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
📝 WalkthroughWalkthroughThe pull request advances the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 3 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
Mesa DescriptionTL;DRTrimmed What changed?
Description generated by Mesa. Update settings |
There was a problem hiding this comment.
Pull request overview
This PR updates the dotagents submodule to reduce the size of the generated ~/.claude/CLAUDE.md file from 42.7k to 37.9k characters, bringing it below the 40k performance threshold. The reduction is achieved by condensing the tool-search.md rule from 6.2k to 1.3k characters using a compact table format.
Changes:
- Update dotagents submodule commit reference to include the trimmed
tool-search.mdfile
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Code Review
This pull request updates the dotagents submodule to a new commit. As described in the pull request, this change is intended to reduce the size of a generated file to improve performance by trimming a dependency. The change itself is a straightforward submodule pointer update and appears correct given the clear context provided. No issues were found in this change.
Entire-Checkpoint: d6bdd7a3c150
1308f42 to
0f42a25
Compare
Changes
tool-search.md(6.2k → 1.3k)~/.claude/CLAUDE.mdfrom 42.7k to 37.9k chars (under 40k performance threshold)Context
Recent dotagents updates caused Ruler's generated
CLAUDE.mdto exceed the 40k char limit, triggering the "Large CLAUDE.md will impact performance" warning. Thetool-search.mdrule was the largest verbatim source (6.2k) — condensed to a compact table format while preserving all essential guidance.Generated with Claude Code
Summary by cubic
Shrink the generated ~/.claude/CLAUDE.md from 42.7k to ~1.6k by updating dotagents. This removes the performance warning and keeps us well under the 40k limit.
Written for commit 0f42a25. Summary will update on new commits.