Skip to content

chore(docs): Add Sentry Astro SDK - #563

Open
Lms24 wants to merge 3 commits into
mainfrom
lms/chore-add-sentry
Open

chore(docs): Add Sentry Astro SDK#563
Lms24 wants to merge 3 commits into
mainfrom
lms/chore-add-sentry

Conversation

@Lms24

@Lms24 Lms24 commented Aug 10, 2026

Copy link
Copy Markdown
Member

The dogs want their food :)

Adding Sentry SDK with 11.0.0-alpha.0. The SDK is only added client-side because this is purely a static site. No dynamic SSR or server routes. All Sentry features are enabled, otherwise default config. Source maps upload happens via Astro integration -> Sentry vite plugin. Auth token was added to the repo's secrets.

@github-actions

github-actions Bot commented Aug 10, 2026

Copy link
Copy Markdown

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


Internal Changes 🔧

Deps

  • Bump dompurify from 3.4.12 to 3.4.13 by dependabot in #559
  • Bump js-yaml from 4.3.0 to 4.3.1 by dependabot in #560

Other

  • (docs) Add Sentry Astro SDK by Lms24 in #563
  • Unstale yarn.lock by Lms24 in #561

🤖 This preview updates automatically when you update the PR.

Comment thread yarn.lock Outdated
@Lms24 Lms24 self-assigned this Aug 10, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 3d05b67. Configure here.

Comment thread docs/package.json Outdated
@Lms24
Lms24 marked this pull request as ready for review August 10, 2026 09:52
@Lms24
Lms24 requested review from a team, cleptric, mjq and nsdeschenes as code owners August 10, 2026 09:52
enabled: import.meta.env.PROD,
environment: import.meta.env.PROD ? 'production' : 'development',
integrations: [Sentry.browserTracingIntegration(), Sentry.replayIntegration()],
enableLogs: true,

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.

l: No need anymore 🥳

Suggested change
enableLogs: true,

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.

2 participants