From 77db2675a75af4e9a542f091f711807eb60e435a Mon Sep 17 00:00:00 2001 From: Nicolas Hrubec Date: Wed, 26 Nov 2025 18:15:44 +0100 Subject: [PATCH 1/2] chore(tanstack): Remove unnecessary placeholder functions --- .../tanstackstart-react/src/common/index.ts | 24 +------------------ .../tanstackstart-react/src/config/index.ts | 17 +------------ .../tanstackstart-react/src/server/index.ts | 14 ----------- 3 files changed, 2 insertions(+), 53 deletions(-) diff --git a/packages/tanstackstart-react/src/common/index.ts b/packages/tanstackstart-react/src/common/index.ts index d83c684d1523..cb0ff5c3b541 100644 --- a/packages/tanstackstart-react/src/common/index.ts +++ b/packages/tanstackstart-react/src/common/index.ts @@ -1,23 +1 @@ -/** - * A middleware handler that can be passed to TanStack Start's `createMiddleware().server(...)` method as [global middleware](https://tanstack.com/start/latest/docs/framework/react/middleware#global-middleware) for instrumenting server functions. - */ -export function sentryGlobalServerMiddlewareHandler() { - // eslint-disable-next-line @typescript-eslint/no-explicit-any - return function (server: { next: (...args: any[]) => T }): T { - return server.next(); - }; -} - -/** - * Wraps a TanStack Start stream handler with Sentry instrumentation that can be passed to `createStartHandler(...)`. - */ -export function wrapStreamHandlerWithSentry(handler: H): H { - return handler; -} - -/** - * Wraps the create root route function with Sentry for server-client tracing with SSR. - */ -export function wrapCreateRootRouteWithSentry(createRootRoute: F): F { - return createRootRoute; -} +export {}; diff --git a/packages/tanstackstart-react/src/config/index.ts b/packages/tanstackstart-react/src/config/index.ts index e41b60609c43..cb0ff5c3b541 100644 --- a/packages/tanstackstart-react/src/config/index.ts +++ b/packages/tanstackstart-react/src/config/index.ts @@ -1,16 +1 @@ -/** - * Wraps a TanStack Start config. - */ -export function wrapVinxiConfigWithSentry( - config: C, - // TODO: Expand this type in the future. Right now it is just so that TS doesn't complain for our users when they copy paste from the docs. - // eslint-disable-next-line @typescript-eslint/no-unused-vars - sentryBuildOptions: { - org?: string; - project?: string; - silent?: boolean; - authToken?: string; - } = {}, -): C { - return config; -} +export {}; diff --git a/packages/tanstackstart-react/src/server/index.ts b/packages/tanstackstart-react/src/server/index.ts index 91f80547f143..c79572a7f318 100644 --- a/packages/tanstackstart-react/src/server/index.ts +++ b/packages/tanstackstart-react/src/server/index.ts @@ -16,13 +16,6 @@ export const ErrorBoundary = (props: React.PropsWithChildren): React.Re return props.children; }; -/** - * A passthrough redux enhancer for the server that doesn't depend on anything from the `@sentry/react` package. - */ -export function createReduxEnhancer() { - return (createStore: unknown) => createStore; -} - /** * A passthrough error boundary wrapper for the server that doesn't depend on any react. Error boundaries don't catch * SSR errors so they should simply be a passthrough. @@ -33,10 +26,3 @@ export function withErrorBoundary

>( ): React.FC

{ return WrappedComponent as React.FC

; } - -/** - * Just a passthrough since we're on the server and showing the report dialog on the server doesn't make any sense. - */ -export function showReportDialog(): void { - return; -} From 3c402fe127239df4280d909ed45d547093432ef4 Mon Sep 17 00:00:00 2001 From: Nicolas Hrubec Date: Wed, 26 Nov 2025 18:34:58 +0100 Subject: [PATCH 2/2] fix lint --- packages/tanstackstart-react/src/index.server.ts | 2 ++ 1 file changed, 2 insertions(+) diff --git a/packages/tanstackstart-react/src/index.server.ts b/packages/tanstackstart-react/src/index.server.ts index 3295df71604b..6b8974481131 100644 --- a/packages/tanstackstart-react/src/index.server.ts +++ b/packages/tanstackstart-react/src/index.server.ts @@ -1,3 +1,5 @@ +/* eslint-disable import/export */ + export * from './config'; export * from './server'; export * from './common';