chore: Improve Claude setup#548
Merged
Merged
Conversation
Member
Patrick Lafrance (patricklafrance)
commented
Mar 2, 2026
Patrick Lafrance (patricklafrance)
commented
- Deleted JavaScript performance rules from js-rules.md
- Deleted rendering performance rules from rendering-rules.md
- Deleted re-render optimization rules from rerender-rules.md
- Updated CLAUDE.md to reflect changes in documentation structure
- Updated agent-skills.md to clarify skill directories
- Updated progressive disclosure design decision to reflect Memory Index terminology
- Updated ODR index to reference Memory Index instead of Documentation Index
- Updated skills-lock.json with new computed hash for turborepo
…rences - Deleted JavaScript performance rules from js-rules.md - Deleted rendering performance rules from rendering-rules.md - Deleted re-render optimization rules from rerender-rules.md - Updated CLAUDE.md to reflect changes in documentation structure - Updated agent-skills.md to clarify skill directories - Updated progressive disclosure design decision to reflect Memory Index terminology - Updated ODR index to reference Memory Index instead of Documentation Index - Updated skills-lock.json with new computed hash for turborepo
Copilot started reviewing on behalf of
Patrick Lafrance (patricklafrance)
March 2, 2026 16:46
View session
@squide/core
@squide/env-vars
@squide/fakes
@squide/firefly
@squide/firefly-module-federation
@squide/firefly-rsbuild-configs
@squide/firefly-rsbuild-storybook
@squide/firefly-webpack-configs
@squide/i18next
@squide/launch-darkly
@squide/msw
@squide/react-router
commit: |
Patrick Lafrance (patricklafrance)
merged commit 68bfcc2
into
main
9 of 10 checks passed
There was a problem hiding this comment.
Pull request overview
This PR refactors the agent/Claude documentation and skill layout by removing several bundled skill reference documents, updating terminology from “Documentation Index” to “Memory Index”, and refreshing the locked skill hash for Turborepo.
Changes:
- Rename CLAUDE.md “Documentation Index” to “Memory Index” and update related ODR references/terminology.
- Update agent skill documentation to describe the new skill directory structure.
- Remove multiple skill reference documents (React best practices, Chromatic best practices, Turborepo docs, pnpm docs, agent-browser templates/docs, etc.) and update
skills-lock.jsonTurborepo hash.
Reviewed changes
Copilot reviewed 79 out of 79 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
skills-lock.json |
Updates the locked computedHash for the turborepo skill. |
agent-docs/odr/index.md |
Updates ODR summary wording to “Memory Index”. |
agent-docs/odr/0010-claude-md-progressive-disclosure-design.md |
Marks ODR-0010 as accepted and replaces “Documentation Index” with “Memory Index”. |
agent-docs/docs/references/agent-skills.md |
Updates skill directory documentation (now “three” directories, adds .claude/skills/). |
CLAUDE.md |
Renames “Documentation Index” section to “Memory Index” and trims the intro line. |
Multiple files under .claude/skills/** and .agents/skills/** |
Removes a large set of skill/reference documents and templates. |
| |---|---| | ||
| | Skill SKILL.md body keeps only critical patterns; everything else lives in reference files. Body should not grow past ~250 lines. | [0008](./0008-skill-body-reference-split.md) | | ||
| | CLAUDE.md uses a single Documentation Index with keyword summaries — never a separate routing table. Section anchors are added only for files over ~80 lines with distinct, high-frequency sub-tasks. Never add folder links to category headings. Keep the file under ~55 lines. | [0010](./0010-claude-md-progressive-disclosure-design.md) | | ||
| | CLAUDE.md uses a single Memory Index with keyword summaries — never a separate routing table. Section anchors are added only for files over ~80 lines with distinct, high-frequency sub-tasks. Never add foler links to category headings. Keep the file under ~55 lines. | [0010](./0010-claude-md-progressive-disclosure-design.md) | |
There was a problem hiding this comment.
Typo in the ODR table row: "Never add foler links" should be "Never add folder links".
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.