Skip to content

feat(agent-monitoring): refer to sentry ai skills instead of hardcoding agent instructions#113049

Merged
shellmayr merged 3 commits intomasterfrom
shellmayr/feat/refer-to-sentry-ai-skills-instead-of-hardcoding-agent-monitoring
Apr 16, 2026
Merged

feat(agent-monitoring): refer to sentry ai skills instead of hardcoding agent instructions#113049
shellmayr merged 3 commits intomasterfrom
shellmayr/feat/refer-to-sentry-ai-skills-instead-of-hardcoding-agent-monitoring

Conversation

@shellmayr
Copy link
Copy Markdown
Member

  • We have a separate repository with agent skills for setting up sentry - we should unite around that so we don't have drift between the different repos and skills
  • Remove the hard-coded instructions and instruct the agent to retrieve instructions from skills.sentry.dev

Contributes to TET-2169

@linear-code
Copy link
Copy Markdown

linear-code bot commented Apr 15, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 15, 2026
@sentry
Copy link
Copy Markdown
Contributor

sentry bot commented Apr 15, 2026

Sentry Snapshot Testing

Name Added Removed Modified Renamed Unchanged Status
sentry-frontend
sentry-frontend
0 0 0 0 204 ✅ Unchanged

⚙️ sentry-frontend Snapshot Settings

@shellmayr shellmayr force-pushed the shellmayr/feat/refer-to-sentry-ai-skills-instead-of-hardcoding-agent-monitoring branch from 079027d to b8f1577 Compare April 16, 2026 08:07
@shellmayr shellmayr marked this pull request as ready for review April 16, 2026 13:46
@shellmayr shellmayr requested a review from a team as a code owner April 16, 2026 13:46
@shellmayr shellmayr requested a review from obostjancic April 16, 2026 13:46
Copy link
Copy Markdown
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit db55ba7. Configure here.

organization,
});
copy(LLM_ONBOARDING_INSTRUCTIONS, {
copy(LLM_ONBOARDING_COPY_MARKDOWN, {
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.

Preamble about "setup steps above" leaks into wrong contexts

Medium Severity

Merging the preamble and instructions into a single LLM_ONBOARDING_COPY_MARKDOWN constant means the text referencing "The setup steps above" is now copied in contexts where no setup steps exist. Previously, CopyLLMPromptButton and the CopyMarkdownButton in CopyInstructionsButton only copied LLM_ONBOARDING_INSTRUCTIONS (without the preamble), while the preamble was exclusively added via the postamble prop in the onboarding flow. Now all three code paths — including UnsupportedPlatformOnboarding and NoDocsOnboarding — copy text that tells the LLM to "complete setup steps above" that don't exist.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit db55ba7. Configure here.

> Then use the guide below for additional instrumentation and agent naming.`;

export const LLM_ONBOARDING_INSTRUCTIONS = `# Instrument Sentry AI Agent Monitoring
export const LLM_ONBOARDING_COPY_MARKDOWN = `
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.

Leading newline in markdown constant causes extra whitespace

Low Severity

The template literal for LLM_ONBOARDING_COPY_MARKDOWN starts with a newline character (the backtick is immediately followed by a line break). When copied to clipboard via CopyLLMPromptButton, the content starts with a blank line. When used as postamble, it produces …\n\n---\n\n\n> (an extra blank line after the horizontal rule) since OnboardingCopyMarkdownButton already joins with \n\n---\n\n.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit db55ba7. Configure here.

Copy link
Copy Markdown
Member

@obostjancic obostjancic left a comment

Choose a reason for hiding this comment

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

🚢

@shellmayr shellmayr merged commit 7c752c0 into master Apr 16, 2026
65 checks passed
@shellmayr shellmayr deleted the shellmayr/feat/refer-to-sentry-ai-skills-instead-of-hardcoding-agent-monitoring branch April 16, 2026 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants