Skip to content

Commit 258d20b

Browse files
isaacsclaude
andauthored
chore(agents): trim rules to save context overhead (#22923)
Found by `claude doctor` as a significant source of context usage, leading to more frequent compaction. Trim details out of the AGENTS.md file to only include what agents can't already infer. This saves a few thousand tokens of context used in every session. The package script table, package layout listing, and skill index all restated things agents read anyway: root `package.json`, the `packages/` tree, and `.claude/skills/`. Each one costs context on every session and goes stale on every rename. Keep only the parts that aren't discoverable: `packages/types` is deprecated, and an AI provider integration spans three packages. Everything else points at the source of truth. Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
1 parent ab4eb93 commit 258d20b

1 file changed

Lines changed: 10 additions & 82 deletions

File tree

AGENTS.md

Lines changed: 10 additions & 82 deletions
Original file line numberDiff line numberDiff line change
@@ -24,21 +24,8 @@ After writing or editing code, check LSP diagnostics and fix errors before proce
2424

2525
## Package Manager
2626

27-
Use **yarn**: `yarn install`, `yarn build:dev`, `yarn test`, `yarn lint`
28-
29-
| Command | Purpose |
30-
| ------------------------------------- | ----------------------------- |
31-
| `yarn build` | Full production build |
32-
| `yarn build:dev` | Dev build (transpile + types) |
33-
| `yarn build:dev:filter @sentry/<pkg>` | Build one package + deps |
34-
| `yarn build:bundle` | Browser bundles only |
35-
| `yarn test` | All unit tests |
36-
| `yarn verify` | Lint + format check |
37-
| `yarn fix` | Format + lint fix |
38-
| `yarn lint` | Lint (Oxlint) |
39-
| `yarn lint:fix` | Lint + auto-fix (Oxlint) |
40-
| `yarn format` | Format files (Oxfmt) |
41-
| `yarn format:check` | Check formatting (Oxfmt) |
27+
Use **yarn**, never npm or pnpm. Scripts live in the root `package.json`.
28+
`yarn build:dev:filter @sentry/<pkg>` builds one package and its deps.
4229

4330
Single package: `cd packages/<name> && yarn test`
4431

@@ -81,42 +68,12 @@ Uses **Git Flow** (see `docs/gitflow.md`).
8168

8269
## Architecture
8370

84-
### Core
85-
86-
- `packages/core/` — Base SDK: interfaces, types, core functionality
87-
- `packages/types/` — Shared types (**deprecated, never modify – instead find types in packages/core**)
88-
- `packages/browser-utils/` — Browser utilities and instrumentation
89-
90-
### Platform SDKs
91-
92-
- `packages/browser/` — Browser SDK + CDN bundles
93-
- `packages/node/` — Node.js SDK (client, transports, non-OTel integrations, and OTel instrumentation)
94-
- `packages/bun/`, `packages/deno/`, `packages/cloudflare/`
95-
96-
### Framework Integrations
97-
98-
- `packages/{framework}/` — React, Vue, Angular, Next.js, Nuxt, SvelteKit, Remix, etc.
99-
- Some have client/server entry points (nextjs, nuxt, sveltekit)
100-
101-
### AI Integrations
102-
103-
- `packages/core/src/tracing/{provider}/` — Core instrumentation
104-
- `packages/node/src/integrations/tracing/{provider}/` — Node.js integration + OTel
105-
- `packages/cloudflare/src/integrations/tracing/{provider}.ts` — Edge runtime
106-
- Use `/add-ai-integration` skill when adding or modifying integrations
107-
108-
### User Experience
109-
110-
- `packages/replay-internal/`, `packages/replay-canvas/`, `packages/replay-worker/` — Session replay
111-
- `packages/feedback/` — User feedback
112-
113-
### Dev Packages (`dev-packages/`)
114-
115-
- `browser-integration-tests/` — Playwright browser tests
116-
- `e2e-tests/` — E2E tests (70+ framework combos)
117-
- `node-integration-tests/` — Node.js integration tests
118-
- `test-utils/` — Shared test utilities
119-
- `rollup-utils/` — Build utilities
71+
- `packages/types/` is **deprecated — never modify it**. Types live in
72+
`packages/core/`.
73+
- An AI provider integration spans three places: core instrumentation in
74+
`packages/core/src/tracing/{provider}/`, the Node integration in
75+
`packages/node/src/integrations/tracing/{provider}/`, and the edge
76+
runtime in `packages/cloudflare/src/integrations/tracing/{provider}.ts`.
12077

12178
## Linting & Formatting
12279

@@ -154,34 +111,5 @@ Before even proposing lazy loading:
154111

155112
## Skills
156113

157-
### E2E Testing
158-
159-
Use `/e2e` skill to run E2E tests. See `.claude/skills/e2e/SKILL.md`
160-
161-
### Security Vulnerabilities
162-
163-
Use `/fix-security-vulnerability` skill for Dependabot alerts. See `.claude/skills/fix-security-vulnerability/SKILL.md`
164-
165-
### Issue Triage
166-
167-
Use `/triage-issue` skill. See `.claude/skills/triage-issue/SKILL.md`
168-
169-
### CDN Bundles
170-
171-
Use `/add-cdn-bundle` skill. See `.claude/skills/add-cdn-bundle/SKILL.md`
172-
173-
### Publishing a Release
174-
175-
Use `/release` skill. See `.claude/skills/release/SKILL.md`
176-
177-
### Dependency Upgrades
178-
179-
Use `/upgrade-dep` skill. See `.claude/skills/upgrade-dep/SKILL.md`
180-
181-
### Vendor OpenTelemetry Instrumentation
182-
183-
Use `/vendor-otel` skill. See `.claude/skills/vendor-otel/SKILL.md`
184-
185-
### AI Integration
186-
187-
Use `/add-ai-integration` skill. See `.claude/skills/add-ai-integration/SKILL.md`
114+
Task-specific instructions live in `.claude/skills/`. Each skill lists its
115+
own trigger, so consult that directory rather than this file.

0 commit comments

Comments
 (0)