Skip to content

feat(skills): Add span streaming migration skill for JavaScript#130

Merged
chargome merged 6 commits into
mainfrom
cg/span-streaming-skill
Jun 5, 2026
Merged

feat(skills): Add span streaming migration skill for JavaScript#130
chargome merged 6 commits into
mainfrom
cg/span-streaming-skill

Conversation

@chargome
Copy link
Copy Markdown
Member

Adds a skill for migrating sentry setups from transaction based tracing to span streaming.

This PR only covers JS but has placeholders for other languages.

@chargome chargome self-assigned this May 15, 2026
@chargome chargome marked this pull request as draft May 15, 2026 12:43
Comment thread skills/sentry-span-streaming/SKILL.md
Comment thread skills/sentry-span-streaming/SKILL.md
String matching in ignoreSpans uses substring includes, not glob
patterns. The `*` was treated as a literal character.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@chargome chargome marked this pull request as ready for review May 18, 2026 12:18
@chargome chargome requested review from Lms24 and nicohrubec May 18, 2026 12:18
Comment thread skills/sentry-span-streaming/SKILL.md
Copy link
Copy Markdown
Member

@Lms24 Lms24 left a comment

Choose a reason for hiding this comment

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

Some comments but otherwise LGTM!

Comment thread skills/sentry-span-streaming/SKILL.md Outdated
Comment thread skills/sentry-span-streaming/SKILL.md Outdated
Comment thread skills/sentry-span-streaming/SKILL.md Outdated
Comment thread skills/sentry-span-streaming/SKILL.md
Comment thread skills/sentry-span-streaming/SKILL.md Outdated
@chargome chargome requested a review from Lms24 June 2, 2026 11:59
Copy link
Copy Markdown
Member

@Lms24 Lms24 left a comment

Choose a reason for hiding this comment

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

thanks!

Comment thread skills/sentry-span-streaming/SKILL.md Outdated
chargome and others added 2 commits June 5, 2026 10:16
Co-authored-by: Lukas Stracke <lukas.stracke@sentry.io>
# Conflicts:
#	AGENTS.md
#	skills/sentry-feature-setup/SKILL.md
Comment thread skills/sentry-span-streaming/SKILL.md
| Platform | Status |
|---|---|
| JavaScript (Browser, Node.js, Bun, Deno, Cloudflare) | Supported |
| Python | Not yet available |
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

m: I read something on slack about splitting the skills per language, are we doing that?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

happens in #156

});
```

### Full Migration Checklist
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

q: do you know if such checklists are actually helpful for agents?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I have seen these around as final verification gate yes, do you have other opinions?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

no not really, was just wondering fine with me to align this with other skills

Copy link
Copy Markdown
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.

nice

@chargome chargome merged commit 030b01f into main Jun 5, 2026
12 checks passed
@chargome chargome deleted the cg/span-streaming-skill branch June 5, 2026 09:04
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.

3 participants