ref(skills): move superseded skills to skills-legacy#265
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit f2c5900. Configure here.
f2c5900 to
52f3ad7
Compare
52f3ad7 to
383a5d9
Compare
| | Create Sentry alerts using the workflow engine API | [`sentry-create-alert`](skills/sentry-create-alert/SKILL.md) | | ||
| | Instruments structured Sentry logs in a new or existing application | [`sentry-instrument-logging`](skills/sentry-instrument-logging/SKILL.md) | | ||
| | Decide which Sentry signal to reach for when instrumenting code — error, span, span attribute, log, or metric | [`sentry-instrumentation-guide`](skills/sentry-instrumentation-guide/SKILL.md) | | ||
| | Configure the OpenTelemetry Collector with Sentry Exporter for multi-project routing and automatic project creation | [`sentry-otel-exporter-setup`](skills/sentry-otel-exporter-setup/SKILL.md) | | ||
| | Setup Sentry AI Agent Monitoring in any project | [`sentry-setup-ai-monitoring`](skills/sentry-setup-ai-monitoring/SKILL.md) | | ||
| | Full Sentry Snapshots setup for Apple/Cocoa projects | [`sentry-snapshots-cocoa`](skills/sentry-snapshots-cocoa/SKILL.md) | | ||
| | Migrate Dart/Flutter SDK to Sentry span streaming (span-first trace lifecycle) | [`sentry-span-streaming-dart`](skills/sentry-span-streaming-dart/SKILL.md) | | ||
| | Migrate JavaScript SDK to Sentry span streaming (span-first trace lifecycle) | [`sentry-span-streaming-js`](skills/sentry-span-streaming-js/SKILL.md) | | ||
| | Migrate Python SDK to Sentry span streaming (span-first trace lifecycle) | [`sentry-span-streaming-python`](skills/sentry-span-streaming-python/SKILL.md) | | ||
|
|
||
| ## Quick Lookup | ||
|
|
||
| Match your project to a skill by keywords. | ||
|
|
||
| | Keywords | Skill | | ||
| |---|---| |
There was a problem hiding this comment.
Bug: Agent definition files in .github/agents/ contain hardcoded paths to skills that were moved to skills-legacy/. The agents will fail to load their knowledge base.
Severity: MEDIUM
Suggested Fix
Update all hardcoded file paths in .github/agents/skill-updater.agent.md and .github/agents/skill-creator.agent.md. Specifically, change references from skills/sentry-sdk-skill-creator/, skills/sentry-go-sdk/, and skills/sentry-nextjs-sdk/ to their corresponding skills-legacy/ paths.
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: SKILL_TREE.md#L60-L72
Potential issue: The agent definition files `.github/agents/skill-updater.agent.md` and
`.github/agents/skill-creator.agent.md` contain hardcoded file paths referencing skills
in the `skills/` directory. However, this pull request moves those skills to the
`skills-legacy/` directory. When the `skill-updater` or `skill-creator` agents are
invoked, they will attempt to read their knowledge base from these outdated paths. The
read operations will fail, preventing the agents from accessing their source material
and causing them to produce incorrect or nonsensical outputs.
Also affects:
.github/agents/skill-updater.agent.md:30~43.github/agents/skill-creator.agent.md:31~44
Did we get this right? 👍 / 👎 to inform future reviews.
Moves the staged next-generation content into its canonical location: skills-next/skills/* -> skills/ (the flat, self-contained skills sentry-get-started, sentry-instrument, sentry-debug-issue) and skills-next/references/ -> references/ (the shared library the plugin hydrate step and the skills.sentry.dev proxy already expect at repo root). Teaches build-skill-tree.sh about standalone (flat) skills: they need only a name + description and are surfaced in a new "Standalone Skills" section at the top of SKILL_TREE.md. The SDK-setup router section (and its keyword Quick Lookup) is dropped from the index — sentry-instrument now covers SDK setup, and the per-SDK skills move to skills-legacy/ (#265). The router/leaf machinery is kept for the workflow and feature skills still to be migrated.
Moves the staged next-generation content into its canonical location: skills-next/skills/* -> skills/ (the flat, self-contained skills sentry-get-started, sentry-instrument, sentry-debug-issue) and skills-next/references/ -> references/ (the shared library the plugin hydrate step and the skills.sentry.dev proxy already expect at repo root). Teaches build-skill-tree.sh about standalone (flat) skills: they need only a name + description and are surfaced in a new "Standalone Skills" section at the top of SKILL_TREE.md. The SDK-setup router section (and its keyword Quick Lookup) is dropped from the index — sentry-instrument now covers SDK setup, and the per-SDK skills move to skills-legacy/ (#265). The router/leaf machinery is kept for the workflow and feature skills still to be migrated.
These per-SDK skills, plus a handful of setup/workflow skills, are superseded by the sentry-instrument skill and the references/sdks/ trees. Move them under skills-legacy/ so they are excluded from the plugin build and skipped by the drift automation, while skills.sentry.dev can still serve them as a fallback for existing URL consumers. Moves 27 skills: the 19 per-SDK skills plus sentry-sdk-setup, sentry-sdk-skill-creator, sentry-fix-issues, the three sentry-span-streaming-* skills, sentry-instrument-logging, and sentry-instrumentation-guide. Regenerates SKILL_TREE.md and repaths the moved entries in CODEOWNERS. Also trims the two remaining routers so they no longer point at moved skills: sentry-workflow drops the production-issue / sentry-fix-issues path (now the standalone sentry-debug-issue skill), and sentry-feature-setup drops the span-streaming, sentry-instrumentation-guide, and sentry-instrument-logging entries (now covered by sentry-instrument). sentry-sdk is intentionally left in place; its removal is handled in a separate PR.
383a5d9 to
a7a0e4a
Compare

Per the decision to retire the per-SDK skills in favor of the
sentry-instrumentskill and thereferences/sdks/trees, this moves the superseded skills underskills-legacy/. Living there keeps them out of the plugin build (which readsskills/) and out of the nightly drift automation, whileskills.sentry.devcan still serve them as a fallback so existing/sentry-*-sdk/SKILL.mdURLs keep resolving.Moves 26 skills: the 19 per-SDK skills plus
sentry-sdk-skill-creator,sentry-fix-issues, the threesentry-span-streaming-*skills,sentry-instrument-logging, andsentry-instrumentation-guide. RegeneratesSKILL_TREE.mdand repaths the moved entries in.github/CODEOWNERS.sentry-sdkis intentionally left inskills/— its removal is handled in a separate PR.Sequencing note: because the SDK skills are gone from
skills/, the regeneratedSKILL_TREE.mdnow has an empty "SDK Setup" table and empty "Quick Lookup", and thesentry-sdk-setuprouter still links to the moved skills. That is expected — this should land together with (or just before) theskills-next→ root merge, which replaces the index and routers wholesale withsentry-get-started/sentry-instrument. It should not be deployed standalone. Theskills.sentry.devskills-legacy/fallback and the CODEOWNERS ownership forreferences/sdks/are deliberately out of scope here and land separately.