Skip to content

Add sentry#170

Merged
wfelliss merged 5 commits into
mainfrom
add-sentry
Jul 6, 2026
Merged

Add sentry#170
wfelliss merged 5 commits into
mainfrom
add-sentry

Conversation

@wfelliss

@wfelliss wfelliss commented Jul 6, 2026

Copy link
Copy Markdown
Owner

No description provided.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request integrates Sentry SDKs into both the NestJS backend and the React Router frontend to enable error tracking and performance monitoring. Key changes include initializing Sentry in both environments, registering the Sentry NestJS module, and capturing unhandled exceptions in the backend's error filter. The review feedback highlights several important improvements: Sentry's debug mode should not be hardcoded to true in production, active Sentry DSNs should be replaced with placeholders in the .env.example files, and the server-rendered HTML stream in the frontend should be wrapped with Sentry.injectTraceMetaTags to ensure distributed tracing propagates correctly to the client.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread apps/backend/src/instrument.ts Outdated
Comment thread apps/backend/.env.example Outdated
Comment thread apps/frontend/.env.example Outdated
Comment thread apps/frontend/app/entry.server.tsx
wfelliss and others added 3 commits July 6, 2026 15:12
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@wfelliss
wfelliss merged commit f2384dc into main Jul 6, 2026
2 checks passed
@wfelliss
wfelliss deleted the add-sentry branch July 6, 2026 14:22
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.

1 participant