ref(skills): move superseded skills to skills-legacy - #265
#265Merged
evanpurkhiser merged 1 commit intoJul 10, 2026
maingetsentry/sentry-for-ai:mainfrom
evanpurkhiser/ref-skills-move-superseded-skills-to-skills-legacygetsentry/sentry-for-ai:evanpurkhiser/ref-skills-move-superseded-skills-to-skills-legacyCopy head branch name to clipboard
Merged
ref(skills): move superseded skills to skills-legacy#265evanpurkhiser merged 1 commit intomaingetsentry/sentry-for-ai:mainfrom evanpurkhiser/ref-skills-move-superseded-skills-to-skills-legacygetsentry/sentry-for-ai:evanpurkhiser/ref-skills-move-superseded-skills-to-skills-legacyCopy head branch name to clipboard
evanpurkhiser merged 1 commit into
maingetsentry/sentry-for-ai:mainfrom
evanpurkhiser/ref-skills-move-superseded-skills-to-skills-legacygetsentry/sentry-for-ai:evanpurkhiser/ref-skills-move-superseded-skills-to-skills-legacyCopy head branch name to clipboard
Conversation
evanpurkhiser
requested review from
JPeer264,
Lms24,
andreiborza,
chargome,
logaretm,
mydea and
nicohrubec
and removed request for
a team
July 9, 2026 15:36
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.
evanpurkhiser
force-pushed
the
evanpurkhiser/ref-skills-move-superseded-skills-to-skills-legacy
branch
from
July 9, 2026 15:40
f2c5900 to
52f3ad7
Compare
evanpurkhiser
force-pushed
the
evanpurkhiser/ref-skills-move-superseded-skills-to-skills-legacy
branch
from
July 9, 2026 19:33
52f3ad7 to
383a5d9
Compare
Comment on lines
62
to
72
| | 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.
evanpurkhiser
added a commit
that referenced
this pull request
Jul 9, 2026
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.
evanpurkhiser
added a commit
that referenced
this pull request
Jul 9, 2026
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.
evanpurkhiser
force-pushed
the
evanpurkhiser/ref-skills-move-superseded-skills-to-skills-legacy
branch
from
July 9, 2026 21:09
383a5d9 to
a7a0e4a
Compare
evanpurkhiser
added a commit
that referenced
this pull request
Jul 10, 2026
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 26 skills: the 19 per-SDK skills plus 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. sentry-sdk is intentionally left in place; its removal is handled in a separate PR.
evanpurkhiser
force-pushed
the
evanpurkhiser/ref-skills-move-superseded-skills-to-skills-legacy
branch
from
July 10, 2026 15:44
a7a0e4a to
582df66
Compare
evanpurkhiser
deleted the
evanpurkhiser/ref-skills-move-superseded-skills-to-skills-legacy
branch
July 10, 2026 15:51
This was referenced Jul 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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.