Skip to content

Clean up Sentry noise#607

Merged
kentcdodds merged 3 commits into
mainfrom
cleanup-sentry-noise
May 17, 2026
Merged

Clean up Sentry noise#607
kentcdodds merged 3 commits into
mainfrom
cleanup-sentry-noise

Conversation

@kentcdodds
Copy link
Copy Markdown
Member

@kentcdodds kentcdodds commented May 17, 2026

Summary

  • Add a root catch-all action so scanner POSTs to unmatched splat paths return a normal 404 instead of React Router's missing-action internal error path (EPICSHOP-F0).
  • Add a narrow client Sentry filter for exactly NotAllowedError: The video element is processing a Picture-in-Picture request. (EPICSHOP-FP).
  • Add focused Vitest coverage for the catch-all response and PiP predicate.

Validation

  • npx vitest run --project app-sentry-noise — 2 files / 4 tests passed.
  • npm run validate — build, typecheck, lint, and tests passed for 6 projects. Lint reported existing warnings in packages/workshop-mcp/src/quiz-me.test.ts but 0 errors.
  • Runtime dev-server check: POST /connectors/resource/index.php returned HTTP 404 and logged POST /connectors/resource/index.php 404.
Open in Web Open in Cursor 

Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@nx-cloud
Copy link
Copy Markdown

nx-cloud Bot commented May 17, 2026

View your CI Pipeline Execution ↗ for commit b783be8

Command Status Duration Result
nx run-many --target typecheck ✅ Succeeded 12s View ↗
nx run-many --target build ✅ Succeeded 57s View ↗
nx lint ✅ Succeeded 26s View ↗

☁️ Nx Cloud last updated this comment at 2026-05-17 15:15:37 UTC

cursoragent and others added 2 commits May 17, 2026 15:11
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@kentcdodds kentcdodds marked this pull request as ready for review May 17, 2026 15:23
@kentcdodds kentcdodds merged commit b2ec840 into main May 17, 2026
10 checks passed
@kentcdodds kentcdodds deleted the cleanup-sentry-noise branch May 17, 2026 15:31
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