Skip to content

Add Sentry.io error monitoring to main map and 19hz map#73

Merged
ProLoser merged 2 commits intomainfrom
copilot/add-sentry-to-funcheapmap
Apr 19, 2026
Merged

Add Sentry.io error monitoring to main map and 19hz map#73
ProLoser merged 2 commits intomainfrom
copilot/add-sentry-to-funcheapmap

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 19, 2026

Adds Sentry Browser SDK to both maps as a single project, using Sentry environment to differentiate between the two endpoints rather than separate projects/DSNs.

Changes

  • index.html / 19hz/index.html — load @sentry/browser@8 from jsDelivr CDN; initialize with __SENTRY_DSN__ placeholder and environment: 'funcheapsf' / environment: '19hz' respectively
  • deploy.yml — adds SENTRY_DSN to the env block and a sed injection step alongside the existing token replacements
  • preview.yml — adds SENTRY_DSN to the env block and a sed injection step so PR preview deployments also have Sentry active
  • .env.example — documents the new SENTRY_DSN variable
<script src="https://cdn.jsdelivr.net/npm/@sentry/browser@8/build/bundle.min.js" crossorigin="anonymous"></script>
<script>
  Sentry.init({ dsn: '__SENTRY_DSN__', environment: 'funcheapsf' })
</script>

Set the SENTRY_DSN repo secret (Settings → Secrets → Actions) to the DSN from your Sentry project's Client Keys page. Environments (funcheapsf, 19hz) are created automatically on first error ingestion.

@ProLoser
Copy link
Copy Markdown
Owner

@copilot use <script src="https://js.sentry-cdn.com/b1a545ff725524609a522423481f37ad.min.js" crossorigin="anonymous"></script> and update preview.yml

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 19, 2026

@copilot use <script src="https://js.sentry-cdn.com/b1a545ff725524609a522423481f37ad.min.js" crossorigin="anonymous"></script> and update preview.yml

preview.yml updated in 6b29765 — it now injects SENTRY_DSN into both HTML files on PR previews, matching deploy.yml.

Your comment appears to have had a URL/reference between "use" and "and update preview.yml" that got stripped — could you clarify what you'd like me to switch to? (e.g. a specific CDN, the Sentry Loader Script, etc.)

Copilot AI requested a review from ProLoser April 19, 2026 20:39
@ProLoser ProLoser marked this pull request as ready for review April 19, 2026 20:45
@ProLoser ProLoser merged commit 80aa1ee into main Apr 19, 2026
@ProLoser ProLoser deleted the copilot/add-sentry-to-funcheapmap branch April 19, 2026 20:46
@github-actions
Copy link
Copy Markdown

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-04-19 20:46 UTC

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