Skip to content

feat: opt-in error tracking - #20

Merged
NotAFlightRisk merged 2 commits into
mainfrom
feat/error-tracking
Oct 5, 2026
Merged

NotAFlightRisk merged 2 commits into
mainfrom
feat/error-tracking

Conversation

@NotAFlightRisk

@NotAFlightRisk NotAFlightRisk commented Oct 5, 2026 •

Copy link
Copy Markdown
Owner

Category

Feature

Overview

Sends crash reports to our Bugsink at oops.peng.ly, but only when the build has PUBLIC_SENTRY_DSN set. Deploy passes it in from the repo variable (already set), so forks and the Docker image build without it, and nothing Sentry ends up in dist/ at all.

With it on, the layout gets one 283-byte script with two error listeners, and the Sentry SDK only downloads the first time something breaks. Sourcemaps upload during the deploy when the token and URL are set, and are deleted before anything's published.

Also groups @sentry/* in Dependabot.

Issue Number

N/A

Additional Info

Tested against a local Bugsink in a browser: nothing extra loads on a normal visit, and a thrown error pulled in the SDK and turned up as an issue. A build with upload on sent the maps and left none in dist/.

npm run validate and npm test pass.

@vercel

vercel Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
android-rom-compat Ready Ready Preview Oct 5, 2026 1:46pm UTC

@NotAFlightRisk
NotAFlightRisk merged commit aab2995 into main Oct 5, 2026
8 checks passed
@NotAFlightRisk
NotAFlightRisk deleted the feat/error-tracking branch October 5, 2026 13:47

This branch was successfully deployed

1 active deployment
Preview — ed62c306 Deployed Oct 5, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant