Skip to content

ref(skills): remove sentry-sdk-upgrade and dissolve the workflow router #287

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
evanpurkhiser wants to merge 1 commit into
base: main
Choose a base branch
from
Open

ref(skills): remove sentry-sdk-upgrade and dissolve the workflow router #287

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 1 addition & 4 deletions AGENTS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,6 @@ Skills use YAML frontmatter with `allowed-tools` — this is required by Cursor
| Skill | Description |
|-------|-------------|
| `sentry-fix-issues` | Find and fix Sentry issues using MCP |
| `sentry-sdk-upgrade` | Upgrade the Sentry JavaScript SDK across major versions |
| `sentry-create-alert` | Create Sentry alerts using the workflow engine API |

### Authoring Skills
Expand All @@ -87,15 +86,13 @@ Sentry MCP server configured at `https://mcp.sentry.dev/mcp`. The source of trut
## Skill Tree Navigation

**How it works:**
- 3 router skills (always visible in agent metadata): `sentry-sdk-setup`, `sentry-workflow`, `sentry-feature-setup`
- Router skills (always visible in agent metadata) group related skills; `sentry-feature-setup` is the last remaining one as the library migrates to flat standalone skills
- All other skills are hidden with `disable-model-invocation: true` — loaded on-demand when a router points to them
- `src/SKILL_TREE.md` is the flat sitemap listing every skill
- This keeps startup metadata at ~300 tokens instead of ~1,600+ as the library grows
- Tools that don't support `disable-model-invocation` simply see all skills (same as before)

**Categories:**
- `sdk-setup` — platform/language SDK setup wizards (router: `sentry-sdk-setup`)
- `workflow` — debugging, code review, issue management (router: `sentry-workflow`)
- `feature-setup` — specific feature configuration (router: `sentry-feature-setup`)
- `internal` — contributor tools, no router

Expand Down
8 changes: 1 addition & 7 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -99,12 +99,6 @@ To build any target locally, run `src/plugins/<agent>/build.sh <output-dir>`
| `sentry-instrument` | Add Sentry to a project, or wire up any signal — error monitoring, tracing, logging, metrics, profiling, session replay, user feedback, cron check-ins, and AI/LLM monitoring. Detects your platform and pulls the code from the reference library. |
| `sentry-debug-issue` | Find a Sentry issue, pull full context, optionally run Seer root-cause / autofix, apply the fix, and resolve it |

### Workflow

| Skill | What it does |
|-------|--------------|
| `sentry-sdk-upgrade` | Upgrade the Sentry JavaScript SDK across major versions |

### Feature Setup

| Skill | What it does |
Expand All @@ -127,7 +121,7 @@ To build any target locally, run `src/plugins/<agent>/build.sh <output-dir>`

The plugin configures the [Sentry MCP server](https://mcp.sentry.dev) automatically on install. No extra setup needed.

Some workflow skills require the [GitHub CLI](https://cli.github.com/):
Some skills require the [GitHub CLI](https://cli.github.com/):

```bash
brew install gh # macOS
Expand Down
1 change: 0 additions & 1 deletion src/SKILL_TREE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@ Debug production issues and maintain code quality with Sentry context.

| Use when | Skill |
|---|---|
| Upgrade the Sentry JavaScript SDK across major versions | [`sentry-sdk-upgrade`](skills/sentry-sdk-upgrade/SKILL.md) |

@cursor cursor Bot Jul 13, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Upgrade prompt lacks skill

Medium Severity

After removing sentry-sdk-upgrade, the regenerated SKILL_TREE.md still tells agents to offer Upgrade Sentry SDK in “Start Here,” but the Workflows table below has no matching skill row. Users who pick that option get no skill to load from the index.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit ef57592. Configure here.


## Feature Setup

Comment on lines 37 to 42

@sentry sentry Bot Jul 13, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bug: The "Upgrade Sentry SDK" menu option in src/SKILL_TREE.md points to the sentry-sdk-upgrade skill, which has been deleted. The option is hardcoded in scripts/build-skill-tree.sh.
Severity: MEDIUM

Suggested Fix

Remove the hardcoded "Upgrade Sentry SDK" list item from the HEADER heredoc in scripts/build-skill-tree.sh (around line 228). After updating the script, regenerate the src/SKILL_TREE.md file to reflect the change.

Prompt for AI Agent 
Review the code at the location below. A potential bug has been identified by an AI
agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's
not valid.

Location: src/SKILL_TREE.md#L37-L42

Potential issue: The `SKILL_TREE.md` file, which is used by AI agents to guide users,
contains a menu option for "Upgrade Sentry SDK". However, the corresponding
`sentry-sdk-upgrade` skill has been deleted in this pull request. The menu option is
hardcoded in the generation script `scripts/build-skill-tree.sh` and was not removed.
When a user selects this option via an agent, the agent will be unable to find the
skill, resulting in a broken user flow. The script's validation checks do not cover
these hardcoded menu options.

Also affects:

  • scripts/build-skill-tree.sh:228

Did we get this right? 👍 / 👎 to inform future reviews.

Expand Down
238 changes: 0 additions & 238 deletions src/skills/sentry-sdk-upgrade/SKILL.md
View file
Open in desktop

This file was deleted.

Loading
Loading