Skip to content

chore(agents): Consolidate SDK dev rules into AGENTS.md#19521

Merged
chargome merged 5 commits intodevelopfrom
cg/agents.md
Feb 26, 2026
Merged

chore(agents): Consolidate SDK dev rules into AGENTS.md#19521
chargome merged 5 commits intodevelopfrom
cg/agents.md

Conversation

@chargome
Copy link
Member

@chargome chargome commented Feb 25, 2026

  • Create AGENTS.md as the single source of truth for SDK development rules
  • Remove .cursor/rules/sdk_development.mdc (all content now in AGENTS.md)
  • Symlink CLAUDE.md to AGENTS.md

Both CLAUDE.md and .cursor/rules/sdk_development.mdc contained duplicated SDK development rules that could drift out of sync. AGENTS.md is the standard cross-tool format — Cursor, Claude Code, and other agents all read it automatically, so a single shared file eliminates duplication.

Closes #19522 (added automatically)

@github-actions
Copy link
Contributor

github-actions bot commented Feb 25, 2026

size-limit report 📦

Path Size % Change Change
@sentry/browser 25.62 kB - -
@sentry/browser - with treeshaking flags 24.12 kB - -
@sentry/browser (incl. Tracing) 42.42 kB - -
@sentry/browser (incl. Tracing, Profiling) 47.09 kB - -
@sentry/browser (incl. Tracing, Replay) 81.24 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 70.86 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 85.94 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 98.2 kB - -
@sentry/browser (incl. Feedback) 42.43 kB - -
@sentry/browser (incl. sendFeedback) 30.29 kB - -
@sentry/browser (incl. FeedbackAsync) 35.34 kB - -
@sentry/browser (incl. Metrics) 26.79 kB - -
@sentry/browser (incl. Logs) 26.93 kB - -
@sentry/browser (incl. Metrics & Logs) 27.61 kB - -
@sentry/react 27.37 kB - -
@sentry/react (incl. Tracing) 44.76 kB - -
@sentry/vue 30.07 kB - -
@sentry/vue (incl. Tracing) 44.27 kB - -
@sentry/svelte 25.64 kB - -
CDN Bundle 28.16 kB - -
CDN Bundle (incl. Tracing) 43.25 kB - -
CDN Bundle (incl. Logs, Metrics) 29 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 44.09 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 68.08 kB - -
CDN Bundle (incl. Tracing, Replay) 80.13 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 80.99 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 85.64 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 86.53 kB - -
CDN Bundle - uncompressed 82.34 kB - -
CDN Bundle (incl. Tracing) - uncompressed 128.06 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 85.18 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 130.89 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 208.84 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 244.94 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 247.76 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 257.85 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 260.66 kB - -
@sentry/nextjs (client) 47.17 kB - -
@sentry/sveltekit (client) 42.88 kB - -
@sentry/node-core 52.18 kB +0.02% +10 B 🔺
@sentry/node 173.46 kB +0.01% +10 B 🔺
@sentry/node - without tracing 97.33 kB +0.02% +10 B 🔺
@sentry/aws-serverless 113.13 kB +0.01% +8 B 🔺

View base workflow run

@chargome chargome marked this pull request as draft February 25, 2026 14:31
@chargome chargome marked this pull request as ready for review February 25, 2026 14:50
Comment on lines +109 to +117
Use `/fix-security-vulnerability` skill for Dependabot alerts. See `.claude/skills/fix-security-vulnerability/SKILL.md`

### Issue Triage

Use `/triage-issue` skill. See `.claude/skills/triage-issue/SKILL.md`

### CDN Bundles

Use `/add-cdn-bundle` skill. See `.claude/skills/add-cdn-bundle/SKILL.md`
Copy link
Member

Choose a reason for hiding this comment

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

This could be more specific (like above "use /e2e skill to run E2E test). We could add a short description like "Use /triage-issue skill to get a in inspection and fix recommendation for a GitHub issue".

Copy link
Member Author

Choose a reason for hiding this comment

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

these are intentionally kept short, the agent knows how to and when to handle these skills

Copy link
Member Author

Choose a reason for hiding this comment

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

in the sense of reference, don't duplicate

### Framework Integrations

- `packages/{framework}/` — React, Vue, Angular, Next.js, Nuxt, SvelteKit, Remix, etc.
- Some have client/server entry points (nextjs, nuxt, sveltekit)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- Some have client/server entry points (nextjs, nuxt, sveltekit)
- Some are Meta-Framwork integrations and have client/server entry points (nextjs, nuxt, sveltekit)

Copy link
Member Author

Choose a reason for hiding this comment

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

that would be obvious context which we should avoid

chargome and others added 2 commits February 26, 2026 10:08
Co-authored-by: Sigrid <32902192+s1gr1d@users.noreply.github.com>
@chargome chargome requested a review from s1gr1d February 26, 2026 09:20
@github-actions
Copy link
Contributor

node-overhead report 🧳

Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.

Scenario Requests/s % of Baseline Prev. Requests/s Change %
GET Baseline 9,178 - 9,098 +1%
GET With Sentry 1,615 18% 1,661 -3%
GET With Sentry (error only) 6,088 66% 5,963 +2%
POST Baseline 1,192 - 1,188 +0%
POST With Sentry 562 47% 596 -6%
POST With Sentry (error only) 1,055 89% 1,052 +0%
MYSQL Baseline 3,246 - 3,224 +1%
MYSQL With Sentry 416 13% 479 -13%
MYSQL With Sentry (error only) 2,659 82% 2,620 +1%

View base workflow run

Copy link
Member

@nicohrubec nicohrubec left a comment

Choose a reason for hiding this comment

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

lgtm, thanks!

@chargome chargome merged commit d245097 into develop Feb 26, 2026
436 of 439 checks passed
@chargome chargome deleted the cg/agents.md branch February 26, 2026 10:23
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.

chore(agents): Consolidate SDK dev rules into AGENTS.md

3 participants