feat(tanstackstart-react): Add global sentry exception middlewares - #19330

Merged
nicohrubec merged 22 commits into
developfrom
nh/tss-error-middleware
Feb 17, 2026
Merged

feat(tanstackstart-react): Add global sentry exception middlewares#19330
nicohrubec merged 22 commits into
developfrom
nh/tss-error-middleware

Conversation

@nicohrubec

@nicohrubecnicohrubec commented Feb 16, 2026•
edited
Loading

Copy link
Copy Markdown
Member

This PR adds sentryGlobalRequestMiddleware and sentryGlobalFunctionMiddleware that capture unhandled errors from all HTTP requests and server function invocations. Users add these as the first entries in the requestMiddleware / functionMiddleware arrays of createStart(). These internal middlewares get marked with a __SENTRY_INTERNAL__, so that they can be easily skipped in the vite plugin to exclude them from middleware auto-instrumentation. Originally we wanted to do this in the server-entry-point, but since there haven't been any updates on this front in months I propose this as an alternative solution for now. This is probably slightly worse UX but in my case better than having nothing in place.

We could also think about auto-injecting this during the build, but maybe not worth the effort since this is a one-time setup step.

Limitations

Tanstack Start has three types of server-side errors that we care about. With these middlewares we can capture 2 of these (route, function exceptions). We cannot capture SSR exceptions like this, because the exceptions are serialized at a deeper layer and newer thrown.

Usage

import { sentryGlobalFunctionMiddleware, sentryGlobalRequestMiddleware } from '@sentry/tanstackstart-react';
import { createStart } from '@tanstack/react-start';
export const startInstance = createStart(() => ({
requestMiddleware: [sentryGlobalRequestMiddleware, ...otherMiddleware],
functionMiddleware: [sentryGlobalFunctionMiddleware, ...otherMiddleware],
}));

Tests

  • Updated E2E tests to verify server side function/route errors are being captured
  • Added an E2E test to document that SSR exceptions are NOT being captured

Closes#18283

@github-actions

github-actionsBot commented Feb 16, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor

Codecov Results 📊

Generated by Codecov Action

@github-actions

github-actionsBot commented Feb 16, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor

node-overhead report 🧳

Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.

ScenarioRequests/s% of BaselinePrev. Requests/sChange %
GET Baseline8,523-11,206-24%
GET With Sentry1,58019%1,856-15%
GET With Sentry (error only)5,97870%7,570-21%
POST Baseline1,135-1,166-3%
POST With Sentry56950%572-1%
POST With Sentry (error only)1,01890%1,025-1%
MYSQL Baseline3,246-3,984-19%
MYSQL With Sentry44514%515-14%
MYSQL With Sentry (error only)2,60880%3,288-21%

View base workflow run

@nicohrubecnicohrubec changed the title feat(tanstackstart-react): Add sentry exception middlewarefeat(tanstackstart-react): Add global sentry exception middlewaresFeb 16, 2026
nicohrubec
nicohrubec commented Feb 16, 2026
View reviewed changes
Comment threadpackages/tanstackstart-react/src/common/types.ts
// eslint-disable-next-line @typescript-eslint/no-explicit-any
'~types': any;
// eslint-disable-next-line @typescript-eslint/no-explicit-any
options: { server?: (...args: any[]) => any };

@nicohrubecnicohrubecFeb 16, 2026

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

typescript kept complaining in my sample app that the types of the new sentry middlewares clash with what is expected from tanstack (it was still running fine thought), had to change it to this to fix that

@nicohrubec
nicohrubec marked this pull request as ready for review February 16, 2026 15:22
cursor[bot]
cursorBot reviewed Feb 16, 2026
View reviewed changes
Comment threadpackages/tanstackstart-react/src/server/globalMiddleware.ts
andreiborza
andreiborza approved these changes Feb 16, 2026
View reviewed changes

@andreiborzaandreiborza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM overall! Nice workaround for the time being.

Comment threaddev-packages/e2e-tests/test-applications/tanstackstart-react/tests/errors.test.ts
Comment threadpackages/tanstackstart-react/src/server/globalMiddleware.ts
cursor[bot]
cursorBot reviewed Feb 17, 2026
View reviewed changes

@cursorcursorBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

Comment threadpackages/tanstackstart-react/src/common/types.ts
@github-actions

github-actionsBot commented Feb 17, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor

size-limit report 📦

PathSize% ChangeChange
@sentry/browser25.56 kB--
@sentry/browser - with treeshaking flags24.08 kB--
@sentry/browser (incl. Tracing)42.36 kB--
@sentry/browser (incl. Tracing, Profiling)47.03 kB--
@sentry/browser (incl. Tracing, Replay)81.18 kB--
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags70.8 kB--
@sentry/browser (incl. Tracing, Replay with Canvas)85.87 kB--
@sentry/browser (incl. Tracing, Replay, Feedback)98.03 kB--
@sentry/browser (incl. Feedback)42.29 kB--
@sentry/browser (incl. sendFeedback)30.23 kB--
@sentry/browser (incl. FeedbackAsync)35.22 kB--
@sentry/browser (incl. Metrics)26.74 kB--
@sentry/browser (incl. Logs)26.88 kB--
@sentry/browser (incl. Metrics & Logs)27.56 kB--
@sentry/react27.33 kB--
@sentry/react (incl. Tracing)44.72 kB--
@sentry/vue30.01 kB--
@sentry/vue (incl. Tracing)44.22 kB--
@sentry/svelte25.58 kB--
CDN Bundle28.11 kB--
CDN Bundle (incl. Tracing)43.2 kB--
CDN Bundle (incl. Logs, Metrics)28.95 kB--
CDN Bundle (incl. Tracing, Logs, Metrics)44.03 kB--
CDN Bundle (incl. Replay, Logs, Metrics)68.02 kB--
CDN Bundle (incl. Tracing, Replay)80.07 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)80.94 kB--
CDN Bundle (incl. Tracing, Replay, Feedback)85.5 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)86.4 kB--
CDN Bundle - uncompressed82.22 kB--
CDN Bundle (incl. Tracing) - uncompressed127.93 kB--
CDN Bundle (incl. Logs, Metrics) - uncompressed85.05 kB--
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed130.76 kB--
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed208.71 kB--
CDN Bundle (incl. Tracing, Replay) - uncompressed244.81 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed247.63 kB--
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed257.61 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed260.42 kB--
@sentry/nextjs (client)47.07 kB--
@sentry/sveltekit (client)42.81 kB--
@sentry/node-core52.15 kB+0.02%+9 B 🔺
@sentry/node166.53 kB+0.01%+5 B 🔺
@sentry/node - without tracing93.95 kB+0.02%+12 B 🔺
@sentry/aws-serverless109.45 kB+0.01%+9 B 🔺

View base workflow run

@nicohrubec
nicohrubec merged commit 766eb98 into developFeb 17, 2026
37 checks passed
@nicohrubec
nicohrubec deleted the nh/tss-error-middleware branch February 17, 2026 16:12
This was referenced Feb 17, 2026
Closed
Merged
Merged
@logaretmlogaretm mentioned this pull request Feb 20, 2026
Closed
@github-actionsgithub-actionsBot mentioned this pull request Feb 24, 2026
Closed
nicohrubec added a commit to getsentry/sentry-docs that referenced this pull request Feb 25, 2026
@nicohrubec@coolguyzone
feat(tanstackstart-react): Add global sentry exception middlewares (#…
e11daf6
…16411)
We added global sentry middlewares to capture server-side errors in the
Tanstack Start SDK.
[PR](getsentry/sentry-javascript#19330)
This will ship with the `10.40.0` release of the javascript SDKs.
Updating the docs accordingly.
---------
Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
nicohrubec added a commit to getsentry/sentry that referenced this pull request Feb 25, 2026
@nicohrubec
feat(onboarding): Add global sentry exception middlewares to tanstack…
1c633ad
… start onboarding (#108351)
We added global sentry middlewares to capture server-side errors in the
Tanstack Start SDK.
[PR](getsentry/sentry-javascript#19330) [Docs
PR](getsentry/sentry-docs#16411)
This will ship with the `10.40.0` release of the javascript SDKs.
Updating the onboarding accordingly.
@jrhizorjrhizor mentioned this pull request Mar 10, 2026
Closed
@terijakiterijaki mentioned this pull request Jun 10, 2026
Closed
3 tasks
@coderabbitaicoderabbitaiBot mentioned this pull request Aug 1, 2026
Merged
@coderabbitaicoderabbitaiBot mentioned this pull request Aug 11, 2026
Merged
7 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments
@andreiborzaandreiborzaandreiborza approved these changes
@s1gr1ds1gr1dAwaiting requested review from s1gr1d
@JPeer264JPeer264Awaiting requested review from JPeer264

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Capture server-side errors

2 participants

@nicohrubec@andreiborza
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all 
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '

Skip to content
feat(tanstackstart-react): Add global sentry exception middlewares - #19330
Merged
nicohrubec merged 22 commits into
developfrom 
nh/tss-error-middleware
Feb 17, 2026
Merged
feat(tanstackstart-react): Add global sentry exception middlewares#19330
nicohrubec merged 22 commits into
developfrom 
nh/tss-error-middleware
Conversation
@nicohrubec
@nicohrubecnicohrubec
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This PR adds sentryGlobalRequestMiddleware and sentryGlobalFunctionMiddleware that capture unhandled errors from all HTTP requests and server function invocations. Users add these as the first entries in the requestMiddleware / functionMiddleware arrays of createStart(). These internal middlewares get marked with a __SENTRY_INTERNAL__, so that they can be easily skipped in the vite plugin to exclude them from middleware auto-instrumentation. Originally we wanted to do this in the server-entry-point, but since there haven't been any updates on this front in months I propose this as an alternative solution for now. This is probably slightly worse UX but in my case better than having nothing in place.
We could also think about auto-injecting this during the build, but maybe not worth the effort since this is a one-time setup step.
Limitations
Tanstack Start has three types of server-side errors that we care about. With these middlewares we can capture 2 of these (route, function exceptions). We cannot capture SSR exceptions like this, because the exceptions are serialized at a deeper layer and newer thrown.
Usage
import { sentryGlobalFunctionMiddleware, sentryGlobalRequestMiddleware } from '@sentry/tanstackstart-react';
import { createStart } from '@tanstack/react-start';
export const startInstance = createStart(() => ({
requestMiddleware: [sentryGlobalRequestMiddleware, ...otherMiddleware],
functionMiddleware: [sentryGlobalFunctionMiddleware, ...otherMiddleware],
}));

Tests
  • Updated E2E tests to verify server side function/route errors are being captured
  • Added an E2E test to document that SSR exceptions are NOT being captured
Closes#18283
@github-actions
github-actionsBot
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Codecov Results 📊
Generated by Codecov Action
@github-actions
github-actionsBot
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
node-overhead report 🧳
Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.
ScenarioRequests/s% of BaselinePrev. Requests/sChange %
GET Baseline8,523-11,206-24%
GET With Sentry1,58019%1,856-15%
GET With Sentry (error only)5,97870%7,570-21%
POST Baseline1,135-1,166-3%
POST With Sentry56950%572-1%
POST With Sentry (error only)1,01890%1,025-1%
MYSQL Baseline3,246-3,984-19%
MYSQL With Sentry44514%515-14%
MYSQL With Sentry (error only)2,60880%3,288-21%
View base workflow run
@nicohrubecnicohrubec
changed the title
feat(tanstackstart-react): Add sentry exception middlewarefeat(tanstackstart-react): Add global sentry exception middlewaresFeb 16, 2026
nicohrubec
nicohrubec
commented
Feb 16, 2026
View reviewed changes
Comment threadpackages/tanstackstart-react/src/common/types.ts
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 '~types': any;
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 options: { server?: (...args: any[]) => any };
@nicohrubecnicohrubecFeb 16, 2026

Copy link


Copy Markdown

MemberAuthor
Choose a reason for hiding this comment

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

typescript kept complaining in my sample app that the types of the new sentry middlewares clash with what is expected from tanstack (it was still running fine thought), had to change it to this to fix that
@nicohrubec
nicohrubec
marked this pull request as ready for review
February 16, 2026 15:22
cursor[bot]
cursorBot
reviewed
Feb 16, 2026
View reviewed changes
Comment threadpackages/tanstackstart-react/src/server/globalMiddleware.ts
andreiborza
andreiborza
approved these changes
Feb 16, 2026
View reviewed changes
@andreiborzaandreiborza
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

LGTM overall! Nice workaround for the time being.
Comment threaddev-packages/e2e-tests/test-applications/tanstackstart-react/tests/errors.test.ts
Comment threadpackages/tanstackstart-react/src/server/globalMiddleware.ts
cursor[bot]
cursorBot
reviewed
Feb 17, 2026
View reviewed changes
@cursorcursorBot
left a comment


Copy link


Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
Comment threadpackages/tanstackstart-react/src/common/types.ts
@github-actions
github-actionsBot
commented
Feb 17, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
size-limit report 📦
PathSize% ChangeChange
@sentry/browser25.56 kB--
@sentry/browser - with treeshaking flags24.08 kB--
@sentry/browser (incl. Tracing)42.36 kB--
@sentry/browser (incl. Tracing, Profiling)47.03 kB--
@sentry/browser (incl. Tracing, Replay)81.18 kB--
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags70.8 kB--
@sentry/browser (incl. Tracing, Replay with Canvas)85.87 kB--
@sentry/browser (incl. Tracing, Replay, Feedback)98.03 kB--
@sentry/browser (incl. Feedback)42.29 kB--
@sentry/browser (incl. sendFeedback)30.23 kB--
@sentry/browser (incl. FeedbackAsync)35.22 kB--
@sentry/browser (incl. Metrics)26.74 kB--
@sentry/browser (incl. Logs)26.88 kB--
@sentry/browser (incl. Metrics & Logs)27.56 kB--
@sentry/react27.33 kB--
@sentry/react (incl. Tracing)44.72 kB--
@sentry/vue30.01 kB--
@sentry/vue (incl. Tracing)44.22 kB--
@sentry/svelte25.58 kB--
CDN Bundle28.11 kB--
CDN Bundle (incl. Tracing)43.2 kB--
CDN Bundle (incl. Logs, Metrics)28.95 kB--
CDN Bundle (incl. Tracing, Logs, Metrics)44.03 kB--
CDN Bundle (incl. Replay, Logs, Metrics)68.02 kB--
CDN Bundle (incl. Tracing, Replay)80.07 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)80.94 kB--
CDN Bundle (incl. Tracing, Replay, Feedback)85.5 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)86.4 kB--
CDN Bundle - uncompressed82.22 kB--
CDN Bundle (incl. Tracing) - uncompressed127.93 kB--
CDN Bundle (incl. Logs, Metrics) - uncompressed85.05 kB--
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed130.76 kB--
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed208.71 kB--
CDN Bundle (incl. Tracing, Replay) - uncompressed244.81 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed247.63 kB--
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed257.61 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed260.42 kB--
@sentry/nextjs (client)47.07 kB--
@sentry/sveltekit (client)42.81 kB--
@sentry/node-core52.15 kB+0.02%+9 B 🔺
@sentry/node166.53 kB+0.01%+5 B 🔺
@sentry/node - without tracing93.95 kB+0.02%+12 B 🔺
@sentry/aws-serverless109.45 kB+0.01%+9 B 🔺
View base workflow run
@nicohrubec
nicohrubec
merged commit 766eb98
into
developFeb 17, 2026

37 checks passed

@nicohrubec
nicohrubec
deleted the

nh/tss-error-middleware

branch
February 17, 2026 16:12

This was referenced Feb 17, 2026
 Closed

 Merged

 Merged

@logaretmlogaretm
mentioned this pull request
Feb 20, 2026
 Closed

@github-actionsgithub-actionsBot
mentioned this pull request
Feb 24, 2026

Closed

nicohrubec added a commit
to getsentry/sentry-docs
that referenced
this pull request
Feb 25, 2026
@nicohrubec@coolguyzone
feat(tanstackstart-react): Add global sentry exception middlewares (#…
e11daf6
…16411)
We added global sentry middlewares to capture server-side errors in the
Tanstack Start SDK.
[PR](getsentry/sentry-javascript#19330)
This will ship with the `10.40.0` release of the javascript SDKs.
Updating the docs accordingly.
---------
Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
nicohrubec added a commit
to getsentry/sentry
that referenced
this pull request
Feb 25, 2026
@nicohrubec
feat(onboarding): Add global sentry exception middlewares to tanstack…
1c633ad
… start onboarding (#108351)
We added global sentry middlewares to capture server-side errors in the
Tanstack Start SDK.
[PR](getsentry/sentry-javascript#19330) [Docs
PR](getsentry/sentry-docs#16411)
This will ship with the `10.40.0` release of the javascript SDKs.
Updating the onboarding accordingly.
@jrhizorjrhizor
mentioned this pull request
Mar 10, 2026

Closed

@terijakiterijaki
mentioned this pull request
Jun 10, 2026

Closed

3 tasks
@coderabbitaicoderabbitaiBot
mentioned this pull request
Aug 1, 2026
 Merged

@coderabbitaicoderabbitaiBot
mentioned this pull request
Aug 11, 2026
 Merged

7 tasks
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments
@andreiborzaandreiborzaandreiborza approved these changes
@s1gr1ds1gr1dAwaiting requested review from s1gr1d
@JPeer264JPeer264Awaiting requested review from JPeer264

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

Capture server-side errors


2 participants

@nicohrubec@andreiborza

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix");
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
feat(tanstackstart-react): Add global sentry exception middlewares - #19330
Merged
nicohrubec merged 22 commits into
developfrom 
nh/tss-error-middleware
Feb 17, 2026
Merged
feat(tanstackstart-react): Add global sentry exception middlewares#19330
nicohrubec merged 22 commits into
developfrom 
nh/tss-error-middleware
Conversation
@nicohrubec
@nicohrubecnicohrubec
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This PR adds sentryGlobalRequestMiddleware and sentryGlobalFunctionMiddleware that capture unhandled errors from all HTTP requests and server function invocations. Users add these as the first entries in the requestMiddleware / functionMiddleware arrays of createStart(). These internal middlewares get marked with a __SENTRY_INTERNAL__, so that they can be easily skipped in the vite plugin to exclude them from middleware auto-instrumentation. Originally we wanted to do this in the server-entry-point, but since there haven't been any updates on this front in months I propose this as an alternative solution for now. This is probably slightly worse UX but in my case better than having nothing in place.
We could also think about auto-injecting this during the build, but maybe not worth the effort since this is a one-time setup step.
Limitations
Tanstack Start has three types of server-side errors that we care about. With these middlewares we can capture 2 of these (route, function exceptions). We cannot capture SSR exceptions like this, because the exceptions are serialized at a deeper layer and newer thrown.
Usage
import { sentryGlobalFunctionMiddleware, sentryGlobalRequestMiddleware } from '@sentry/tanstackstart-react';
import { createStart } from '@tanstack/react-start';
export const startInstance = createStart(() => ({
requestMiddleware: [sentryGlobalRequestMiddleware, ...otherMiddleware],
functionMiddleware: [sentryGlobalFunctionMiddleware, ...otherMiddleware],
}));

Tests
  • Updated E2E tests to verify server side function/route errors are being captured
  • Added an E2E test to document that SSR exceptions are NOT being captured
Closes#18283
@github-actions
github-actionsBot
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Codecov Results 📊
Generated by Codecov Action
@github-actions
github-actionsBot
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
node-overhead report 🧳
Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.
ScenarioRequests/s% of BaselinePrev. Requests/sChange %
GET Baseline8,523-11,206-24%
GET With Sentry1,58019%1,856-15%
GET With Sentry (error only)5,97870%7,570-21%
POST Baseline1,135-1,166-3%
POST With Sentry56950%572-1%
POST With Sentry (error only)1,01890%1,025-1%
MYSQL Baseline3,246-3,984-19%
MYSQL With Sentry44514%515-14%
MYSQL With Sentry (error only)2,60880%3,288-21%
View base workflow run
@nicohrubecnicohrubec
changed the title
feat(tanstackstart-react): Add sentry exception middlewarefeat(tanstackstart-react): Add global sentry exception middlewaresFeb 16, 2026
nicohrubec
nicohrubec
commented
Feb 16, 2026
View reviewed changes
Comment threadpackages/tanstackstart-react/src/common/types.ts
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 '~types': any;
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 options: { server?: (...args: any[]) => any };
@nicohrubecnicohrubecFeb 16, 2026

Copy link


Copy Markdown

MemberAuthor
Choose a reason for hiding this comment

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

typescript kept complaining in my sample app that the types of the new sentry middlewares clash with what is expected from tanstack (it was still running fine thought), had to change it to this to fix that
@nicohrubec
nicohrubec
marked this pull request as ready for review
February 16, 2026 15:22
cursor[bot]
cursorBot
reviewed
Feb 16, 2026
View reviewed changes
Comment threadpackages/tanstackstart-react/src/server/globalMiddleware.ts
andreiborza
andreiborza
approved these changes
Feb 16, 2026
View reviewed changes
@andreiborzaandreiborza
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

LGTM overall! Nice workaround for the time being.
Comment threaddev-packages/e2e-tests/test-applications/tanstackstart-react/tests/errors.test.ts
Comment threadpackages/tanstackstart-react/src/server/globalMiddleware.ts
cursor[bot]
cursorBot
reviewed
Feb 17, 2026
View reviewed changes
@cursorcursorBot
left a comment


Copy link


Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
Comment threadpackages/tanstackstart-react/src/common/types.ts
@github-actions
github-actionsBot
commented
Feb 17, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
size-limit report 📦
PathSize% ChangeChange
@sentry/browser25.56 kB--
@sentry/browser - with treeshaking flags24.08 kB--
@sentry/browser (incl. Tracing)42.36 kB--
@sentry/browser (incl. Tracing, Profiling)47.03 kB--
@sentry/browser (incl. Tracing, Replay)81.18 kB--
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags70.8 kB--
@sentry/browser (incl. Tracing, Replay with Canvas)85.87 kB--
@sentry/browser (incl. Tracing, Replay, Feedback)98.03 kB--
@sentry/browser (incl. Feedback)42.29 kB--
@sentry/browser (incl. sendFeedback)30.23 kB--
@sentry/browser (incl. FeedbackAsync)35.22 kB--
@sentry/browser (incl. Metrics)26.74 kB--
@sentry/browser (incl. Logs)26.88 kB--
@sentry/browser (incl. Metrics & Logs)27.56 kB--
@sentry/react27.33 kB--
@sentry/react (incl. Tracing)44.72 kB--
@sentry/vue30.01 kB--
@sentry/vue (incl. Tracing)44.22 kB--
@sentry/svelte25.58 kB--
CDN Bundle28.11 kB--
CDN Bundle (incl. Tracing)43.2 kB--
CDN Bundle (incl. Logs, Metrics)28.95 kB--
CDN Bundle (incl. Tracing, Logs, Metrics)44.03 kB--
CDN Bundle (incl. Replay, Logs, Metrics)68.02 kB--
CDN Bundle (incl. Tracing, Replay)80.07 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)80.94 kB--
CDN Bundle (incl. Tracing, Replay, Feedback)85.5 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)86.4 kB--
CDN Bundle - uncompressed82.22 kB--
CDN Bundle (incl. Tracing) - uncompressed127.93 kB--
CDN Bundle (incl. Logs, Metrics) - uncompressed85.05 kB--
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed130.76 kB--
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed208.71 kB--
CDN Bundle (incl. Tracing, Replay) - uncompressed244.81 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed247.63 kB--
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed257.61 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed260.42 kB--
@sentry/nextjs (client)47.07 kB--
@sentry/sveltekit (client)42.81 kB--
@sentry/node-core52.15 kB+0.02%+9 B 🔺
@sentry/node166.53 kB+0.01%+5 B 🔺
@sentry/node - without tracing93.95 kB+0.02%+12 B 🔺
@sentry/aws-serverless109.45 kB+0.01%+9 B 🔺
View base workflow run
@nicohrubec
nicohrubec
merged commit 766eb98
into
developFeb 17, 2026

37 checks passed

@nicohrubec
nicohrubec
deleted the

nh/tss-error-middleware

branch
February 17, 2026 16:12

This was referenced Feb 17, 2026
 Closed

 Merged

 Merged

@logaretmlogaretm
mentioned this pull request
Feb 20, 2026
 Closed

@github-actionsgithub-actionsBot
mentioned this pull request
Feb 24, 2026

Closed

nicohrubec added a commit
to getsentry/sentry-docs
that referenced
this pull request
Feb 25, 2026
@nicohrubec@coolguyzone
feat(tanstackstart-react): Add global sentry exception middlewares (#…
e11daf6
…16411)
We added global sentry middlewares to capture server-side errors in the
Tanstack Start SDK.
[PR](getsentry/sentry-javascript#19330)
This will ship with the `10.40.0` release of the javascript SDKs.
Updating the docs accordingly.
---------
Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
nicohrubec added a commit
to getsentry/sentry
that referenced
this pull request
Feb 25, 2026
@nicohrubec
feat(onboarding): Add global sentry exception middlewares to tanstack…
1c633ad
… start onboarding (#108351)
We added global sentry middlewares to capture server-side errors in the
Tanstack Start SDK.
[PR](getsentry/sentry-javascript#19330) [Docs
PR](getsentry/sentry-docs#16411)
This will ship with the `10.40.0` release of the javascript SDKs.
Updating the onboarding accordingly.
@jrhizorjrhizor
mentioned this pull request
Mar 10, 2026

Closed

@terijakiterijaki
mentioned this pull request
Jun 10, 2026

Closed

3 tasks
@coderabbitaicoderabbitaiBot
mentioned this pull request
Aug 1, 2026
 Merged

@coderabbitaicoderabbitaiBot
mentioned this pull request
Aug 11, 2026
 Merged

7 tasks
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments
@andreiborzaandreiborzaandreiborza approved these changes
@s1gr1ds1gr1dAwaiting requested review from s1gr1d
@JPeer264JPeer264Awaiting requested review from JPeer264

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

Capture server-side errors


2 participants

@nicohrubec@andreiborza

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms");
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
feat(tanstackstart-react): Add global sentry exception middlewares - #19330
Merged
nicohrubec merged 22 commits into
developfrom 
nh/tss-error-middleware
Feb 17, 2026
Merged
feat(tanstackstart-react): Add global sentry exception middlewares#19330
nicohrubec merged 22 commits into
developfrom 
nh/tss-error-middleware
Conversation
@nicohrubec
@nicohrubecnicohrubec
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This PR adds sentryGlobalRequestMiddleware and sentryGlobalFunctionMiddleware that capture unhandled errors from all HTTP requests and server function invocations. Users add these as the first entries in the requestMiddleware / functionMiddleware arrays of createStart(). These internal middlewares get marked with a __SENTRY_INTERNAL__, so that they can be easily skipped in the vite plugin to exclude them from middleware auto-instrumentation. Originally we wanted to do this in the server-entry-point, but since there haven't been any updates on this front in months I propose this as an alternative solution for now. This is probably slightly worse UX but in my case better than having nothing in place.
We could also think about auto-injecting this during the build, but maybe not worth the effort since this is a one-time setup step.
Limitations
Tanstack Start has three types of server-side errors that we care about. With these middlewares we can capture 2 of these (route, function exceptions). We cannot capture SSR exceptions like this, because the exceptions are serialized at a deeper layer and newer thrown.
Usage
import { sentryGlobalFunctionMiddleware, sentryGlobalRequestMiddleware } from '@sentry/tanstackstart-react';
import { createStart } from '@tanstack/react-start';
export const startInstance = createStart(() => ({
requestMiddleware: [sentryGlobalRequestMiddleware, ...otherMiddleware],
functionMiddleware: [sentryGlobalFunctionMiddleware, ...otherMiddleware],
}));

Tests
  • Updated E2E tests to verify server side function/route errors are being captured
  • Added an E2E test to document that SSR exceptions are NOT being captured
Closes#18283
@github-actions
github-actionsBot
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Codecov Results 📊
Generated by Codecov Action
@github-actions
github-actionsBot
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
node-overhead report 🧳
Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.
ScenarioRequests/s% of BaselinePrev. Requests/sChange %
GET Baseline8,523-11,206-24%
GET With Sentry1,58019%1,856-15%
GET With Sentry (error only)5,97870%7,570-21%
POST Baseline1,135-1,166-3%
POST With Sentry56950%572-1%
POST With Sentry (error only)1,01890%1,025-1%
MYSQL Baseline3,246-3,984-19%
MYSQL With Sentry44514%515-14%
MYSQL With Sentry (error only)2,60880%3,288-21%
View base workflow run
@nicohrubecnicohrubec
changed the title
feat(tanstackstart-react): Add sentry exception middlewarefeat(tanstackstart-react): Add global sentry exception middlewaresFeb 16, 2026
nicohrubec
nicohrubec
commented
Feb 16, 2026
View reviewed changes
Comment threadpackages/tanstackstart-react/src/common/types.ts
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 '~types': any;
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 options: { server?: (...args: any[]) => any };
@nicohrubecnicohrubecFeb 16, 2026

Copy link


Copy Markdown

MemberAuthor
Choose a reason for hiding this comment

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

typescript kept complaining in my sample app that the types of the new sentry middlewares clash with what is expected from tanstack (it was still running fine thought), had to change it to this to fix that
@nicohrubec
nicohrubec
marked this pull request as ready for review
February 16, 2026 15:22
cursor[bot]
cursorBot
reviewed
Feb 16, 2026
View reviewed changes
Comment threadpackages/tanstackstart-react/src/server/globalMiddleware.ts
andreiborza
andreiborza
approved these changes
Feb 16, 2026
View reviewed changes
@andreiborzaandreiborza
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

LGTM overall! Nice workaround for the time being.
Comment threaddev-packages/e2e-tests/test-applications/tanstackstart-react/tests/errors.test.ts
Comment threadpackages/tanstackstart-react/src/server/globalMiddleware.ts
cursor[bot]
cursorBot
reviewed
Feb 17, 2026
View reviewed changes
@cursorcursorBot
left a comment


Copy link


Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
Comment threadpackages/tanstackstart-react/src/common/types.ts
@github-actions
github-actionsBot
commented
Feb 17, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
size-limit report 📦
PathSize% ChangeChange
@sentry/browser25.56 kB--
@sentry/browser - with treeshaking flags24.08 kB--
@sentry/browser (incl. Tracing)42.36 kB--
@sentry/browser (incl. Tracing, Profiling)47.03 kB--
@sentry/browser (incl. Tracing, Replay)81.18 kB--
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags70.8 kB--
@sentry/browser (incl. Tracing, Replay with Canvas)85.87 kB--
@sentry/browser (incl. Tracing, Replay, Feedback)98.03 kB--
@sentry/browser (incl. Feedback)42.29 kB--
@sentry/browser (incl. sendFeedback)30.23 kB--
@sentry/browser (incl. FeedbackAsync)35.22 kB--
@sentry/browser (incl. Metrics)26.74 kB--
@sentry/browser (incl. Logs)26.88 kB--
@sentry/browser (incl. Metrics & Logs)27.56 kB--
@sentry/react27.33 kB--
@sentry/react (incl. Tracing)44.72 kB--
@sentry/vue30.01 kB--
@sentry/vue (incl. Tracing)44.22 kB--
@sentry/svelte25.58 kB--
CDN Bundle28.11 kB--
CDN Bundle (incl. Tracing)43.2 kB--
CDN Bundle (incl. Logs, Metrics)28.95 kB--
CDN Bundle (incl. Tracing, Logs, Metrics)44.03 kB--
CDN Bundle (incl. Replay, Logs, Metrics)68.02 kB--
CDN Bundle (incl. Tracing, Replay)80.07 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)80.94 kB--
CDN Bundle (incl. Tracing, Replay, Feedback)85.5 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)86.4 kB--
CDN Bundle - uncompressed82.22 kB--
CDN Bundle (incl. Tracing) - uncompressed127.93 kB--
CDN Bundle (incl. Logs, Metrics) - uncompressed85.05 kB--
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed130.76 kB--
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed208.71 kB--
CDN Bundle (incl. Tracing, Replay) - uncompressed244.81 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed247.63 kB--
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed257.61 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed260.42 kB--
@sentry/nextjs (client)47.07 kB--
@sentry/sveltekit (client)42.81 kB--
@sentry/node-core52.15 kB+0.02%+9 B 🔺
@sentry/node166.53 kB+0.01%+5 B 🔺
@sentry/node - without tracing93.95 kB+0.02%+12 B 🔺
@sentry/aws-serverless109.45 kB+0.01%+9 B 🔺
View base workflow run
@nicohrubec
nicohrubec
merged commit 766eb98
into
developFeb 17, 2026

37 checks passed

@nicohrubec
nicohrubec
deleted the

nh/tss-error-middleware

branch
February 17, 2026 16:12

This was referenced Feb 17, 2026
 Closed

 Merged

 Merged

@logaretmlogaretm
mentioned this pull request
Feb 20, 2026
 Closed

@github-actionsgithub-actionsBot
mentioned this pull request
Feb 24, 2026

Closed

nicohrubec added a commit
to getsentry/sentry-docs
that referenced
this pull request
Feb 25, 2026
@nicohrubec@coolguyzone
feat(tanstackstart-react): Add global sentry exception middlewares (#…
e11daf6
…16411)
We added global sentry middlewares to capture server-side errors in the
Tanstack Start SDK.
[PR](getsentry/sentry-javascript#19330)
This will ship with the `10.40.0` release of the javascript SDKs.
Updating the docs accordingly.
---------
Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
nicohrubec added a commit
to getsentry/sentry
that referenced
this pull request
Feb 25, 2026
@nicohrubec
feat(onboarding): Add global sentry exception middlewares to tanstack…
1c633ad
… start onboarding (#108351)
We added global sentry middlewares to capture server-side errors in the
Tanstack Start SDK.
[PR](getsentry/sentry-javascript#19330) [Docs
PR](getsentry/sentry-docs#16411)
This will ship with the `10.40.0` release of the javascript SDKs.
Updating the onboarding accordingly.
@jrhizorjrhizor
mentioned this pull request
Mar 10, 2026

Closed

@terijakiterijaki
mentioned this pull request
Jun 10, 2026

Closed

3 tasks
@coderabbitaicoderabbitaiBot
mentioned this pull request
Aug 1, 2026
 Merged

@coderabbitaicoderabbitaiBot
mentioned this pull request
Aug 11, 2026
 Merged

7 tasks
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments
@andreiborzaandreiborzaandreiborza approved these changes
@s1gr1ds1gr1dAwaiting requested review from s1gr1d
@JPeer264JPeer264Awaiting requested review from JPeer264

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

Capture server-side errors


2 participants

@nicohrubec@andreiborza

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links");
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '

Skip to content
feat(tanstackstart-react): Add global sentry exception middlewares - #19330
Merged
nicohrubec merged 22 commits into
developfrom 
nh/tss-error-middleware
Feb 17, 2026
Merged
feat(tanstackstart-react): Add global sentry exception middlewares#19330
nicohrubec merged 22 commits into
developfrom 
nh/tss-error-middleware
Conversation
@nicohrubec
@nicohrubecnicohrubec
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This PR adds sentryGlobalRequestMiddleware and sentryGlobalFunctionMiddleware that capture unhandled errors from all HTTP requests and server function invocations. Users add these as the first entries in the requestMiddleware / functionMiddleware arrays of createStart(). These internal middlewares get marked with a __SENTRY_INTERNAL__, so that they can be easily skipped in the vite plugin to exclude them from middleware auto-instrumentation. Originally we wanted to do this in the server-entry-point, but since there haven't been any updates on this front in months I propose this as an alternative solution for now. This is probably slightly worse UX but in my case better than having nothing in place.
We could also think about auto-injecting this during the build, but maybe not worth the effort since this is a one-time setup step.
Limitations
Tanstack Start has three types of server-side errors that we care about. With these middlewares we can capture 2 of these (route, function exceptions). We cannot capture SSR exceptions like this, because the exceptions are serialized at a deeper layer and newer thrown.
Usage
import { sentryGlobalFunctionMiddleware, sentryGlobalRequestMiddleware } from '@sentry/tanstackstart-react';
import { createStart } from '@tanstack/react-start';
export const startInstance = createStart(() => ({
requestMiddleware: [sentryGlobalRequestMiddleware, ...otherMiddleware],
functionMiddleware: [sentryGlobalFunctionMiddleware, ...otherMiddleware],
}));

Tests
  • Updated E2E tests to verify server side function/route errors are being captured
  • Added an E2E test to document that SSR exceptions are NOT being captured
Closes#18283
@github-actions
github-actionsBot
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Codecov Results 📊
Generated by Codecov Action
@github-actions
github-actionsBot
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
node-overhead report 🧳
Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.
ScenarioRequests/s% of BaselinePrev. Requests/sChange %
GET Baseline8,523-11,206-24%
GET With Sentry1,58019%1,856-15%
GET With Sentry (error only)5,97870%7,570-21%
POST Baseline1,135-1,166-3%
POST With Sentry56950%572-1%
POST With Sentry (error only)1,01890%1,025-1%
MYSQL Baseline3,246-3,984-19%
MYSQL With Sentry44514%515-14%
MYSQL With Sentry (error only)2,60880%3,288-21%
View base workflow run
@nicohrubecnicohrubec
changed the title
feat(tanstackstart-react): Add sentry exception middlewarefeat(tanstackstart-react): Add global sentry exception middlewaresFeb 16, 2026
nicohrubec
nicohrubec
commented
Feb 16, 2026
View reviewed changes
Comment threadpackages/tanstackstart-react/src/common/types.ts
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 '~types': any;
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 options: { server?: (...args: any[]) => any };
@nicohrubecnicohrubecFeb 16, 2026

Copy link


Copy Markdown

MemberAuthor
Choose a reason for hiding this comment

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

typescript kept complaining in my sample app that the types of the new sentry middlewares clash with what is expected from tanstack (it was still running fine thought), had to change it to this to fix that
@nicohrubec
nicohrubec
marked this pull request as ready for review
February 16, 2026 15:22
cursor[bot]
cursorBot
reviewed
Feb 16, 2026
View reviewed changes
Comment threadpackages/tanstackstart-react/src/server/globalMiddleware.ts
andreiborza
andreiborza
approved these changes
Feb 16, 2026
View reviewed changes
@andreiborzaandreiborza
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

LGTM overall! Nice workaround for the time being.
Comment threaddev-packages/e2e-tests/test-applications/tanstackstart-react/tests/errors.test.ts
Comment threadpackages/tanstackstart-react/src/server/globalMiddleware.ts
cursor[bot]
cursorBot
reviewed
Feb 17, 2026
View reviewed changes
@cursorcursorBot
left a comment


Copy link


Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
Comment threadpackages/tanstackstart-react/src/common/types.ts
@github-actions
github-actionsBot
commented
Feb 17, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
size-limit report 📦
PathSize% ChangeChange
@sentry/browser25.56 kB--
@sentry/browser - with treeshaking flags24.08 kB--
@sentry/browser (incl. Tracing)42.36 kB--
@sentry/browser (incl. Tracing, Profiling)47.03 kB--
@sentry/browser (incl. Tracing, Replay)81.18 kB--
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags70.8 kB--
@sentry/browser (incl. Tracing, Replay with Canvas)85.87 kB--
@sentry/browser (incl. Tracing, Replay, Feedback)98.03 kB--
@sentry/browser (incl. Feedback)42.29 kB--
@sentry/browser (incl. sendFeedback)30.23 kB--
@sentry/browser (incl. FeedbackAsync)35.22 kB--
@sentry/browser (incl. Metrics)26.74 kB--
@sentry/browser (incl. Logs)26.88 kB--
@sentry/browser (incl. Metrics & Logs)27.56 kB--
@sentry/react27.33 kB--
@sentry/react (incl. Tracing)44.72 kB--
@sentry/vue30.01 kB--
@sentry/vue (incl. Tracing)44.22 kB--
@sentry/svelte25.58 kB--
CDN Bundle28.11 kB--
CDN Bundle (incl. Tracing)43.2 kB--
CDN Bundle (incl. Logs, Metrics)28.95 kB--
CDN Bundle (incl. Tracing, Logs, Metrics)44.03 kB--
CDN Bundle (incl. Replay, Logs, Metrics)68.02 kB--
CDN Bundle (incl. Tracing, Replay)80.07 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)80.94 kB--
CDN Bundle (incl. Tracing, Replay, Feedback)85.5 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)86.4 kB--
CDN Bundle - uncompressed82.22 kB--
CDN Bundle (incl. Tracing) - uncompressed127.93 kB--
CDN Bundle (incl. Logs, Metrics) - uncompressed85.05 kB--
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed130.76 kB--
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed208.71 kB--
CDN Bundle (incl. Tracing, Replay) - uncompressed244.81 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed247.63 kB--
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed257.61 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed260.42 kB--
@sentry/nextjs (client)47.07 kB--
@sentry/sveltekit (client)42.81 kB--
@sentry/node-core52.15 kB+0.02%+9 B 🔺
@sentry/node166.53 kB+0.01%+5 B 🔺
@sentry/node - without tracing93.95 kB+0.02%+12 B 🔺
@sentry/aws-serverless109.45 kB+0.01%+9 B 🔺
View base workflow run
@nicohrubec
nicohrubec
merged commit 766eb98
into
developFeb 17, 2026

37 checks passed

@nicohrubec
nicohrubec
deleted the

nh/tss-error-middleware

branch
February 17, 2026 16:12

This was referenced Feb 17, 2026
 Closed

 Merged

 Merged

@logaretmlogaretm
mentioned this pull request
Feb 20, 2026
 Closed

@github-actionsgithub-actionsBot
mentioned this pull request
Feb 24, 2026

Closed

nicohrubec added a commit
to getsentry/sentry-docs
that referenced
this pull request
Feb 25, 2026
@nicohrubec@coolguyzone
feat(tanstackstart-react): Add global sentry exception middlewares (#…
e11daf6
…16411)
We added global sentry middlewares to capture server-side errors in the
Tanstack Start SDK.
[PR](getsentry/sentry-javascript#19330)
This will ship with the `10.40.0` release of the javascript SDKs.
Updating the docs accordingly.
---------
Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
nicohrubec added a commit
to getsentry/sentry
that referenced
this pull request
Feb 25, 2026
@nicohrubec
feat(onboarding): Add global sentry exception middlewares to tanstack…
1c633ad
… start onboarding (#108351)
We added global sentry middlewares to capture server-side errors in the
Tanstack Start SDK.
[PR](getsentry/sentry-javascript#19330) [Docs
PR](getsentry/sentry-docs#16411)
This will ship with the `10.40.0` release of the javascript SDKs.
Updating the onboarding accordingly.
@jrhizorjrhizor
mentioned this pull request
Mar 10, 2026

Closed

@terijakiterijaki
mentioned this pull request
Jun 10, 2026

Closed

3 tasks
@coderabbitaicoderabbitaiBot
mentioned this pull request
Aug 1, 2026
 Merged

@coderabbitaicoderabbitaiBot
mentioned this pull request
Aug 11, 2026
 Merged

7 tasks
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments
@andreiborzaandreiborzaandreiborza approved these changes
@s1gr1ds1gr1dAwaiting requested review from s1gr1d
@JPeer264JPeer264Awaiting requested review from JPeer264

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

Capture server-side errors


2 participants

@nicohrubec@andreiborza

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default");
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
feat(tanstackstart-react): Add global sentry exception middlewares - #19330
Merged
nicohrubec merged 22 commits into
developfrom 
nh/tss-error-middleware
Feb 17, 2026
Merged
feat(tanstackstart-react): Add global sentry exception middlewares#19330
nicohrubec merged 22 commits into
developfrom 
nh/tss-error-middleware
Conversation
@nicohrubec
@nicohrubecnicohrubec
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This PR adds sentryGlobalRequestMiddleware and sentryGlobalFunctionMiddleware that capture unhandled errors from all HTTP requests and server function invocations. Users add these as the first entries in the requestMiddleware / functionMiddleware arrays of createStart(). These internal middlewares get marked with a __SENTRY_INTERNAL__, so that they can be easily skipped in the vite plugin to exclude them from middleware auto-instrumentation. Originally we wanted to do this in the server-entry-point, but since there haven't been any updates on this front in months I propose this as an alternative solution for now. This is probably slightly worse UX but in my case better than having nothing in place.
We could also think about auto-injecting this during the build, but maybe not worth the effort since this is a one-time setup step.
Limitations
Tanstack Start has three types of server-side errors that we care about. With these middlewares we can capture 2 of these (route, function exceptions). We cannot capture SSR exceptions like this, because the exceptions are serialized at a deeper layer and newer thrown.
Usage
import { sentryGlobalFunctionMiddleware, sentryGlobalRequestMiddleware } from '@sentry/tanstackstart-react';
import { createStart } from '@tanstack/react-start';
export const startInstance = createStart(() => ({
requestMiddleware: [sentryGlobalRequestMiddleware, ...otherMiddleware],
functionMiddleware: [sentryGlobalFunctionMiddleware, ...otherMiddleware],
}));

Tests
  • Updated E2E tests to verify server side function/route errors are being captured
  • Added an E2E test to document that SSR exceptions are NOT being captured
Closes#18283
@github-actions
github-actionsBot
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Codecov Results 📊
Generated by Codecov Action
@github-actions
github-actionsBot
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
node-overhead report 🧳
Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.
ScenarioRequests/s% of BaselinePrev. Requests/sChange %
GET Baseline8,523-11,206-24%
GET With Sentry1,58019%1,856-15%
GET With Sentry (error only)5,97870%7,570-21%
POST Baseline1,135-1,166-3%
POST With Sentry56950%572-1%
POST With Sentry (error only)1,01890%1,025-1%
MYSQL Baseline3,246-3,984-19%
MYSQL With Sentry44514%515-14%
MYSQL With Sentry (error only)2,60880%3,288-21%
View base workflow run
@nicohrubecnicohrubec
changed the title
feat(tanstackstart-react): Add sentry exception middlewarefeat(tanstackstart-react): Add global sentry exception middlewaresFeb 16, 2026
nicohrubec
nicohrubec
commented
Feb 16, 2026
View reviewed changes
Comment threadpackages/tanstackstart-react/src/common/types.ts
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 '~types': any;
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 options: { server?: (...args: any[]) => any };
@nicohrubecnicohrubecFeb 16, 2026

Copy link


Copy Markdown

MemberAuthor
Choose a reason for hiding this comment

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

typescript kept complaining in my sample app that the types of the new sentry middlewares clash with what is expected from tanstack (it was still running fine thought), had to change it to this to fix that
@nicohrubec
nicohrubec
marked this pull request as ready for review
February 16, 2026 15:22
cursor[bot]
cursorBot
reviewed
Feb 16, 2026
View reviewed changes
Comment threadpackages/tanstackstart-react/src/server/globalMiddleware.ts
andreiborza
andreiborza
approved these changes
Feb 16, 2026
View reviewed changes
@andreiborzaandreiborza
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

LGTM overall! Nice workaround for the time being.
Comment threaddev-packages/e2e-tests/test-applications/tanstackstart-react/tests/errors.test.ts
Comment threadpackages/tanstackstart-react/src/server/globalMiddleware.ts
cursor[bot]
cursorBot
reviewed
Feb 17, 2026
View reviewed changes
@cursorcursorBot
left a comment


Copy link


Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
Comment threadpackages/tanstackstart-react/src/common/types.ts
@github-actions
github-actionsBot
commented
Feb 17, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
size-limit report 📦
PathSize% ChangeChange
@sentry/browser25.56 kB--
@sentry/browser - with treeshaking flags24.08 kB--
@sentry/browser (incl. Tracing)42.36 kB--
@sentry/browser (incl. Tracing, Profiling)47.03 kB--
@sentry/browser (incl. Tracing, Replay)81.18 kB--
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags70.8 kB--
@sentry/browser (incl. Tracing, Replay with Canvas)85.87 kB--
@sentry/browser (incl. Tracing, Replay, Feedback)98.03 kB--
@sentry/browser (incl. Feedback)42.29 kB--
@sentry/browser (incl. sendFeedback)30.23 kB--
@sentry/browser (incl. FeedbackAsync)35.22 kB--
@sentry/browser (incl. Metrics)26.74 kB--
@sentry/browser (incl. Logs)26.88 kB--
@sentry/browser (incl. Metrics & Logs)27.56 kB--
@sentry/react27.33 kB--
@sentry/react (incl. Tracing)44.72 kB--
@sentry/vue30.01 kB--
@sentry/vue (incl. Tracing)44.22 kB--
@sentry/svelte25.58 kB--
CDN Bundle28.11 kB--
CDN Bundle (incl. Tracing)43.2 kB--
CDN Bundle (incl. Logs, Metrics)28.95 kB--
CDN Bundle (incl. Tracing, Logs, Metrics)44.03 kB--
CDN Bundle (incl. Replay, Logs, Metrics)68.02 kB--
CDN Bundle (incl. Tracing, Replay)80.07 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)80.94 kB--
CDN Bundle (incl. Tracing, Replay, Feedback)85.5 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)86.4 kB--
CDN Bundle - uncompressed82.22 kB--
CDN Bundle (incl. Tracing) - uncompressed127.93 kB--
CDN Bundle (incl. Logs, Metrics) - uncompressed85.05 kB--
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed130.76 kB--
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed208.71 kB--
CDN Bundle (incl. Tracing, Replay) - uncompressed244.81 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed247.63 kB--
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed257.61 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed260.42 kB--
@sentry/nextjs (client)47.07 kB--
@sentry/sveltekit (client)42.81 kB--
@sentry/node-core52.15 kB+0.02%+9 B 🔺
@sentry/node166.53 kB+0.01%+5 B 🔺
@sentry/node - without tracing93.95 kB+0.02%+12 B 🔺
@sentry/aws-serverless109.45 kB+0.01%+9 B 🔺
View base workflow run
@nicohrubec
nicohrubec
merged commit 766eb98
into
developFeb 17, 2026

37 checks passed

@nicohrubec
nicohrubec
deleted the

nh/tss-error-middleware

branch
February 17, 2026 16:12

This was referenced Feb 17, 2026
 Closed

 Merged

 Merged

@logaretmlogaretm
mentioned this pull request
Feb 20, 2026
 Closed

@github-actionsgithub-actionsBot
mentioned this pull request
Feb 24, 2026

Closed

nicohrubec added a commit
to getsentry/sentry-docs
that referenced
this pull request
Feb 25, 2026
@nicohrubec@coolguyzone
feat(tanstackstart-react): Add global sentry exception middlewares (#…
e11daf6
…16411)
We added global sentry middlewares to capture server-side errors in the
Tanstack Start SDK.
[PR](getsentry/sentry-javascript#19330)
This will ship with the `10.40.0` release of the javascript SDKs.
Updating the docs accordingly.
---------
Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
nicohrubec added a commit
to getsentry/sentry
that referenced
this pull request
Feb 25, 2026
@nicohrubec
feat(onboarding): Add global sentry exception middlewares to tanstack…
1c633ad
… start onboarding (#108351)
We added global sentry middlewares to capture server-side errors in the
Tanstack Start SDK.
[PR](getsentry/sentry-javascript#19330) [Docs
PR](getsentry/sentry-docs#16411)
This will ship with the `10.40.0` release of the javascript SDKs.
Updating the onboarding accordingly.
@jrhizorjrhizor
mentioned this pull request
Mar 10, 2026

Closed

@terijakiterijaki
mentioned this pull request
Jun 10, 2026

Closed

3 tasks
@coderabbitaicoderabbitaiBot
mentioned this pull request
Aug 1, 2026
 Merged

@coderabbitaicoderabbitaiBot
mentioned this pull request
Aug 11, 2026
 Merged

7 tasks
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments
@andreiborzaandreiborzaandreiborza approved these changes
@s1gr1ds1gr1dAwaiting requested review from s1gr1d
@JPeer264JPeer264Awaiting requested review from JPeer264

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

Capture server-side errors


2 participants

@nicohrubec@andreiborza

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers");
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
feat(tanstackstart-react): Add global sentry exception middlewares - #19330
Merged
nicohrubec merged 22 commits into
developfrom 
nh/tss-error-middleware
Feb 17, 2026
Merged
feat(tanstackstart-react): Add global sentry exception middlewares#19330
nicohrubec merged 22 commits into
developfrom 
nh/tss-error-middleware
Conversation
@nicohrubec
@nicohrubecnicohrubec
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This PR adds sentryGlobalRequestMiddleware and sentryGlobalFunctionMiddleware that capture unhandled errors from all HTTP requests and server function invocations. Users add these as the first entries in the requestMiddleware / functionMiddleware arrays of createStart(). These internal middlewares get marked with a __SENTRY_INTERNAL__, so that they can be easily skipped in the vite plugin to exclude them from middleware auto-instrumentation. Originally we wanted to do this in the server-entry-point, but since there haven't been any updates on this front in months I propose this as an alternative solution for now. This is probably slightly worse UX but in my case better than having nothing in place.
We could also think about auto-injecting this during the build, but maybe not worth the effort since this is a one-time setup step.
Limitations
Tanstack Start has three types of server-side errors that we care about. With these middlewares we can capture 2 of these (route, function exceptions). We cannot capture SSR exceptions like this, because the exceptions are serialized at a deeper layer and newer thrown.
Usage
import { sentryGlobalFunctionMiddleware, sentryGlobalRequestMiddleware } from '@sentry/tanstackstart-react';
import { createStart } from '@tanstack/react-start';
export const startInstance = createStart(() => ({
requestMiddleware: [sentryGlobalRequestMiddleware, ...otherMiddleware],
functionMiddleware: [sentryGlobalFunctionMiddleware, ...otherMiddleware],
}));

Tests
  • Updated E2E tests to verify server side function/route errors are being captured
  • Added an E2E test to document that SSR exceptions are NOT being captured
Closes#18283
@github-actions
github-actionsBot
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Codecov Results 📊
Generated by Codecov Action
@github-actions
github-actionsBot
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
node-overhead report 🧳
Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.
ScenarioRequests/s% of BaselinePrev. Requests/sChange %
GET Baseline8,523-11,206-24%
GET With Sentry1,58019%1,856-15%
GET With Sentry (error only)5,97870%7,570-21%
POST Baseline1,135-1,166-3%
POST With Sentry56950%572-1%
POST With Sentry (error only)1,01890%1,025-1%
MYSQL Baseline3,246-3,984-19%
MYSQL With Sentry44514%515-14%
MYSQL With Sentry (error only)2,60880%3,288-21%
View base workflow run
@nicohrubecnicohrubec
changed the title
feat(tanstackstart-react): Add sentry exception middlewarefeat(tanstackstart-react): Add global sentry exception middlewaresFeb 16, 2026
nicohrubec
nicohrubec
commented
Feb 16, 2026
View reviewed changes
Comment threadpackages/tanstackstart-react/src/common/types.ts
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 '~types': any;
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 options: { server?: (...args: any[]) => any };
@nicohrubecnicohrubecFeb 16, 2026

Copy link


Copy Markdown

MemberAuthor
Choose a reason for hiding this comment

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

typescript kept complaining in my sample app that the types of the new sentry middlewares clash with what is expected from tanstack (it was still running fine thought), had to change it to this to fix that
@nicohrubec
nicohrubec
marked this pull request as ready for review
February 16, 2026 15:22
cursor[bot]
cursorBot
reviewed
Feb 16, 2026
View reviewed changes
Comment threadpackages/tanstackstart-react/src/server/globalMiddleware.ts
andreiborza
andreiborza
approved these changes
Feb 16, 2026
View reviewed changes
@andreiborzaandreiborza
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

LGTM overall! Nice workaround for the time being.
Comment threaddev-packages/e2e-tests/test-applications/tanstackstart-react/tests/errors.test.ts
Comment threadpackages/tanstackstart-react/src/server/globalMiddleware.ts
cursor[bot]
cursorBot
reviewed
Feb 17, 2026
View reviewed changes
@cursorcursorBot
left a comment


Copy link


Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
Comment threadpackages/tanstackstart-react/src/common/types.ts
@github-actions
github-actionsBot
commented
Feb 17, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
size-limit report 📦
PathSize% ChangeChange
@sentry/browser25.56 kB--
@sentry/browser - with treeshaking flags24.08 kB--
@sentry/browser (incl. Tracing)42.36 kB--
@sentry/browser (incl. Tracing, Profiling)47.03 kB--
@sentry/browser (incl. Tracing, Replay)81.18 kB--
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags70.8 kB--
@sentry/browser (incl. Tracing, Replay with Canvas)85.87 kB--
@sentry/browser (incl. Tracing, Replay, Feedback)98.03 kB--
@sentry/browser (incl. Feedback)42.29 kB--
@sentry/browser (incl. sendFeedback)30.23 kB--
@sentry/browser (incl. FeedbackAsync)35.22 kB--
@sentry/browser (incl. Metrics)26.74 kB--
@sentry/browser (incl. Logs)26.88 kB--
@sentry/browser (incl. Metrics & Logs)27.56 kB--
@sentry/react27.33 kB--
@sentry/react (incl. Tracing)44.72 kB--
@sentry/vue30.01 kB--
@sentry/vue (incl. Tracing)44.22 kB--
@sentry/svelte25.58 kB--
CDN Bundle28.11 kB--
CDN Bundle (incl. Tracing)43.2 kB--
CDN Bundle (incl. Logs, Metrics)28.95 kB--
CDN Bundle (incl. Tracing, Logs, Metrics)44.03 kB--
CDN Bundle (incl. Replay, Logs, Metrics)68.02 kB--
CDN Bundle (incl. Tracing, Replay)80.07 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)80.94 kB--
CDN Bundle (incl. Tracing, Replay, Feedback)85.5 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)86.4 kB--
CDN Bundle - uncompressed82.22 kB--
CDN Bundle (incl. Tracing) - uncompressed127.93 kB--
CDN Bundle (incl. Logs, Metrics) - uncompressed85.05 kB--
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed130.76 kB--
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed208.71 kB--
CDN Bundle (incl. Tracing, Replay) - uncompressed244.81 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed247.63 kB--
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed257.61 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed260.42 kB--
@sentry/nextjs (client)47.07 kB--
@sentry/sveltekit (client)42.81 kB--
@sentry/node-core52.15 kB+0.02%+9 B 🔺
@sentry/node166.53 kB+0.01%+5 B 🔺
@sentry/node - without tracing93.95 kB+0.02%+12 B 🔺
@sentry/aws-serverless109.45 kB+0.01%+9 B 🔺
View base workflow run
@nicohrubec
nicohrubec
merged commit 766eb98
into
developFeb 17, 2026

37 checks passed

@nicohrubec
nicohrubec
deleted the

nh/tss-error-middleware

branch
February 17, 2026 16:12

This was referenced Feb 17, 2026
 Closed

 Merged

 Merged

@logaretmlogaretm
mentioned this pull request
Feb 20, 2026
 Closed

@github-actionsgithub-actionsBot
mentioned this pull request
Feb 24, 2026

Closed

nicohrubec added a commit
to getsentry/sentry-docs
that referenced
this pull request
Feb 25, 2026
@nicohrubec@coolguyzone
feat(tanstackstart-react): Add global sentry exception middlewares (#…
e11daf6
…16411)
We added global sentry middlewares to capture server-side errors in the
Tanstack Start SDK.
[PR](getsentry/sentry-javascript#19330)
This will ship with the `10.40.0` release of the javascript SDKs.
Updating the docs accordingly.
---------
Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
nicohrubec added a commit
to getsentry/sentry
that referenced
this pull request
Feb 25, 2026
@nicohrubec
feat(onboarding): Add global sentry exception middlewares to tanstack…
1c633ad
… start onboarding (#108351)
We added global sentry middlewares to capture server-side errors in the
Tanstack Start SDK.
[PR](getsentry/sentry-javascript#19330) [Docs
PR](getsentry/sentry-docs#16411)
This will ship with the `10.40.0` release of the javascript SDKs.
Updating the onboarding accordingly.
@jrhizorjrhizor
mentioned this pull request
Mar 10, 2026

Closed

@terijakiterijaki
mentioned this pull request
Jun 10, 2026

Closed

3 tasks
@coderabbitaicoderabbitaiBot
mentioned this pull request
Aug 1, 2026
 Merged

@coderabbitaicoderabbitaiBot
mentioned this pull request
Aug 11, 2026
 Merged

7 tasks
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments
@andreiborzaandreiborzaandreiborza approved these changes
@s1gr1ds1gr1dAwaiting requested review from s1gr1d
@JPeer264JPeer264Awaiting requested review from JPeer264

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

Capture server-side errors


2 participants

@nicohrubec@andreiborza

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode");
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();

Skip to content
feat(tanstackstart-react): Add global sentry exception middlewares - #19330
Merged
nicohrubec merged 22 commits into
developfrom 
nh/tss-error-middleware
Feb 17, 2026
Merged
feat(tanstackstart-react): Add global sentry exception middlewares#19330
nicohrubec merged 22 commits into
developfrom 
nh/tss-error-middleware
Conversation
@nicohrubec
@nicohrubecnicohrubec
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This PR adds sentryGlobalRequestMiddleware and sentryGlobalFunctionMiddleware that capture unhandled errors from all HTTP requests and server function invocations. Users add these as the first entries in the requestMiddleware / functionMiddleware arrays of createStart(). These internal middlewares get marked with a __SENTRY_INTERNAL__, so that they can be easily skipped in the vite plugin to exclude them from middleware auto-instrumentation. Originally we wanted to do this in the server-entry-point, but since there haven't been any updates on this front in months I propose this as an alternative solution for now. This is probably slightly worse UX but in my case better than having nothing in place.
We could also think about auto-injecting this during the build, but maybe not worth the effort since this is a one-time setup step.
Limitations
Tanstack Start has three types of server-side errors that we care about. With these middlewares we can capture 2 of these (route, function exceptions). We cannot capture SSR exceptions like this, because the exceptions are serialized at a deeper layer and newer thrown.
Usage
import { sentryGlobalFunctionMiddleware, sentryGlobalRequestMiddleware } from '@sentry/tanstackstart-react';
import { createStart } from '@tanstack/react-start';
export const startInstance = createStart(() => ({
requestMiddleware: [sentryGlobalRequestMiddleware, ...otherMiddleware],
functionMiddleware: [sentryGlobalFunctionMiddleware, ...otherMiddleware],
}));

Tests
  • Updated E2E tests to verify server side function/route errors are being captured
  • Added an E2E test to document that SSR exceptions are NOT being captured
Closes#18283
@github-actions
github-actionsBot
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Codecov Results 📊
Generated by Codecov Action
@github-actions
github-actionsBot
commented
Feb 16, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
node-overhead report 🧳
Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.
ScenarioRequests/s% of BaselinePrev. Requests/sChange %
GET Baseline8,523-11,206-24%
GET With Sentry1,58019%1,856-15%
GET With Sentry (error only)5,97870%7,570-21%
POST Baseline1,135-1,166-3%
POST With Sentry56950%572-1%
POST With Sentry (error only)1,01890%1,025-1%
MYSQL Baseline3,246-3,984-19%
MYSQL With Sentry44514%515-14%
MYSQL With Sentry (error only)2,60880%3,288-21%
View base workflow run
@nicohrubecnicohrubec
changed the title
feat(tanstackstart-react): Add sentry exception middlewarefeat(tanstackstart-react): Add global sentry exception middlewaresFeb 16, 2026
nicohrubec
nicohrubec
commented
Feb 16, 2026
View reviewed changes
Comment threadpackages/tanstackstart-react/src/common/types.ts
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 '~types': any;
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 options: { server?: (...args: any[]) => any };
@nicohrubecnicohrubecFeb 16, 2026

Copy link


Copy Markdown

MemberAuthor
Choose a reason for hiding this comment

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

typescript kept complaining in my sample app that the types of the new sentry middlewares clash with what is expected from tanstack (it was still running fine thought), had to change it to this to fix that
@nicohrubec
nicohrubec
marked this pull request as ready for review
February 16, 2026 15:22
cursor[bot]
cursorBot
reviewed
Feb 16, 2026
View reviewed changes
Comment threadpackages/tanstackstart-react/src/server/globalMiddleware.ts
andreiborza
andreiborza
approved these changes
Feb 16, 2026
View reviewed changes
@andreiborzaandreiborza
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

LGTM overall! Nice workaround for the time being.
Comment threaddev-packages/e2e-tests/test-applications/tanstackstart-react/tests/errors.test.ts
Comment threadpackages/tanstackstart-react/src/server/globalMiddleware.ts
cursor[bot]
cursorBot
reviewed
Feb 17, 2026
View reviewed changes
@cursorcursorBot
left a comment


Copy link


Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
Comment threadpackages/tanstackstart-react/src/common/types.ts
@github-actions
github-actionsBot
commented
Feb 17, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
size-limit report 📦
PathSize% ChangeChange
@sentry/browser25.56 kB--
@sentry/browser - with treeshaking flags24.08 kB--
@sentry/browser (incl. Tracing)42.36 kB--
@sentry/browser (incl. Tracing, Profiling)47.03 kB--
@sentry/browser (incl. Tracing, Replay)81.18 kB--
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags70.8 kB--
@sentry/browser (incl. Tracing, Replay with Canvas)85.87 kB--
@sentry/browser (incl. Tracing, Replay, Feedback)98.03 kB--
@sentry/browser (incl. Feedback)42.29 kB--
@sentry/browser (incl. sendFeedback)30.23 kB--
@sentry/browser (incl. FeedbackAsync)35.22 kB--
@sentry/browser (incl. Metrics)26.74 kB--
@sentry/browser (incl. Logs)26.88 kB--
@sentry/browser (incl. Metrics & Logs)27.56 kB--
@sentry/react27.33 kB--
@sentry/react (incl. Tracing)44.72 kB--
@sentry/vue30.01 kB--
@sentry/vue (incl. Tracing)44.22 kB--
@sentry/svelte25.58 kB--
CDN Bundle28.11 kB--
CDN Bundle (incl. Tracing)43.2 kB--
CDN Bundle (incl. Logs, Metrics)28.95 kB--
CDN Bundle (incl. Tracing, Logs, Metrics)44.03 kB--
CDN Bundle (incl. Replay, Logs, Metrics)68.02 kB--
CDN Bundle (incl. Tracing, Replay)80.07 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)80.94 kB--
CDN Bundle (incl. Tracing, Replay, Feedback)85.5 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)86.4 kB--
CDN Bundle - uncompressed82.22 kB--
CDN Bundle (incl. Tracing) - uncompressed127.93 kB--
CDN Bundle (incl. Logs, Metrics) - uncompressed85.05 kB--
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed130.76 kB--
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed208.71 kB--
CDN Bundle (incl. Tracing, Replay) - uncompressed244.81 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed247.63 kB--
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed257.61 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed260.42 kB--
@sentry/nextjs (client)47.07 kB--
@sentry/sveltekit (client)42.81 kB--
@sentry/node-core52.15 kB+0.02%+9 B 🔺
@sentry/node166.53 kB+0.01%+5 B 🔺
@sentry/node - without tracing93.95 kB+0.02%+12 B 🔺
@sentry/aws-serverless109.45 kB+0.01%+9 B 🔺
View base workflow run
@nicohrubec
nicohrubec
merged commit 766eb98
into
developFeb 17, 2026

37 checks passed

@nicohrubec
nicohrubec
deleted the

nh/tss-error-middleware

branch
February 17, 2026 16:12

This was referenced Feb 17, 2026
 Closed

 Merged

 Merged

@logaretmlogaretm
mentioned this pull request
Feb 20, 2026
 Closed

@github-actionsgithub-actionsBot
mentioned this pull request
Feb 24, 2026

Closed

nicohrubec added a commit
to getsentry/sentry-docs
that referenced
this pull request
Feb 25, 2026
@nicohrubec@coolguyzone
feat(tanstackstart-react): Add global sentry exception middlewares (#…
e11daf6
…16411)
We added global sentry middlewares to capture server-side errors in the
Tanstack Start SDK.
[PR](getsentry/sentry-javascript#19330)
This will ship with the `10.40.0` release of the javascript SDKs.
Updating the docs accordingly.
---------
Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
nicohrubec added a commit
to getsentry/sentry
that referenced
this pull request
Feb 25, 2026
@nicohrubec
feat(onboarding): Add global sentry exception middlewares to tanstack…
1c633ad
… start onboarding (#108351)
We added global sentry middlewares to capture server-side errors in the
Tanstack Start SDK.
[PR](getsentry/sentry-javascript#19330) [Docs
PR](getsentry/sentry-docs#16411)
This will ship with the `10.40.0` release of the javascript SDKs.
Updating the onboarding accordingly.
@jrhizorjrhizor
mentioned this pull request
Mar 10, 2026

Closed

@terijakiterijaki
mentioned this pull request
Jun 10, 2026

Closed

3 tasks
@coderabbitaicoderabbitaiBot
mentioned this pull request
Aug 1, 2026
 Merged

@coderabbitaicoderabbitaiBot
mentioned this pull request
Aug 11, 2026
 Merged

7 tasks
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments
@andreiborzaandreiborzaandreiborza approved these changes
@s1gr1ds1gr1dAwaiting requested review from s1gr1d
@JPeer264JPeer264Awaiting requested review from JPeer264

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

Capture server-side errors


2 participants

@nicohrubec@andreiborza