docs(elysia): Add @sentry/elysia SDK documentation#17107
Open
docs(elysia): Add @sentry/elysia SDK documentation#17107
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Member
Author
|
Currently failing because of the icon, I already have a PR in flight that will resolve that. |
Add getting started guide for the new @sentry/elysia package covering Bun and Node.js runtimes, error capturing, automatic tracing, and distributed trace propagation. Wire up the Elysia platform icon. Refs getsentry/sentry-javascript#19509 Co-Authored-By: Claude <noreply@anthropic.com>
1019251 to
7b47799
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
Adds initial documentation for the new @sentry/elysia JavaScript SDK and wires up the Elysia platform icon so the guide can appear alongside other JS frameworks.
Changes:
- Adds a new Elysia getting-started guide covering Bun and Node.js runtimes.
- Adds Elysia icons (small/large) and maps
javascript-elysiato the new icon key. - Bumps
platformiconsto a version which includes the Elysia icon assets.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
src/components/platformIcon.tsx |
Imports Elysia SVGs and adds icon mappings for the new platform. |
docs/platforms/javascript/guides/elysia/index.mdx |
New getting-started guide and feature overview for @sentry/elysia. |
package.json |
Updates platformicons dependency to include Elysia assets. |
pnpm-lock.yaml |
Lockfile updates corresponding to the platformicons bump. |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (1)
src/components/platformIcon.tsx:500
formatToSVGappears to be kept in alphabetical order by key in this file, but the newelysiaentry is inserted beforeelixir. Please reorder the entries to keep the mapping sorted (helps scanability and reduces merge conflicts).
electron: {
sm: ElectronSVG,
lg: ElectronSVGLarge,
},
elysia: {
sm: ElysiaSVG,
lg: ElysiaSVGLarge,
},
elixir: {
sm: ElixirSVG,
lg: ElixirSVGLarge,
},
Contributor
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.
- Use correct sentry.javascript.elysia SDK identifier instead of bun - Fix inconsistent tab title casing (node → Node.js) - Clarify shouldHandleError example to avoid double-wrap confusion - Fix alphabetical ordering of elysia/elixir in icon map Co-Authored-By: Claude <noreply@anthropic.com>
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
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.

DESCRIBE YOUR PR
Add getting started documentation for the new
@sentry/elysiapackage, a Sentry SDK for the Elysia web framework.docs/platforms/javascript/guides/elysia/covering Bun and Node.js runtimesplatformIcon.tsxBlocked on
platformiconsversion here)IS YOUR CHANGE URGENT?
Help us prioritize incoming PRs by letting us know when the change needs to go live.
SLA
Thanks in advance for your help!
PRE-MERGE CHECKLIST
Make sure you've checked the following before merging your changes:
🤖 Generated with Claude Code