Skip to content

fix: filter Epic Stack Sentry healthcheck and bot noise#1093

Merged
kody-bot merged 2 commits into
mainfrom
cursor/sentry-s1-epic-stack-0d40
Jul 24, 2026
Merged

fix: filter Epic Stack Sentry healthcheck and bot noise#1093
kody-bot merged 2 commits into
mainfrom
cursor/sentry-s1-epic-stack-0d40

Conversation

@kentcdodds

@kentcdodds kentcdodds commented Jul 24, 2026

Copy link
Copy Markdown
Member

Summary

Sentry project epic-stack was dominated by non-actionable noise on the public template demo:

  1. Slow DB Query (EPIC-STACK-2A) — Fly healthchecks calling prisma.user.count(). Orphaned Prisma spans still reached Sentry as prisma:client:operation.
  2. getInternalRouterError family (EPIC-STACK-22 / 28 / 21) — bots/scanners hitting POST / without an action, GET /resources/theme-switch without a loader, and OPTIONS /.
  3. TypeError: Load failed (EPIC-STACK-2B) — one-off Mobile Safari network blip; ignore in Sentry (no code change).

Changes

  • Healthcheck DB probe: prisma.user.count()prisma.$queryRaw\SELECT 1``
  • Server Sentry filters for expected React Router method/handler errors and orphaned healthcheck Prisma transactions
  • handleError skips capturing those expected router errors
  • Unit tests for the pure filter helpers

Risk

Low — monitoring/filter + lighter healthcheck query only.

Test Plan

  • vitest run app/utils/sentry-event-filters.test.ts
  • npm run lint / npm run typecheck
  • CI green on this PR

Checklist

  • Tests updated
  • Docs updated

Screenshots

N/A (Sentry/monitoring + healthcheck probe only)

Open in Web Open in Cursor 

cursoragent and others added 2 commits July 24, 2026 20:02
Fly healthchecks were generating Slow DB Query Sentry insights from
prisma.user.count() on the tiny demo VM without indicating app issues.

Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Filter React Router missing loader/action/method errors and orphaned
healthcheck Prisma transactions so the public demo issue stream stays
actionable.

Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@cursor
cursor Bot marked this pull request as ready for review July 24, 2026 20:02
@kody-bot
kody-bot merged commit f1da831 into main Jul 24, 2026
7 checks passed
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