fix(webapp): reach SentryContextManager through the default export

The webapp server bundle is ESM and @sentry/remix is CommonJS. Node's
loader derives named exports by static analysis, and it does not see
SentryContextManager because that name is re-exported transitively from
@sentry/node-core. A named import type-checks and bundles, then throws
SyntaxError when the server boots. The property is reachable on the
default export, which for a CommonJS module is module.exports.

Vitest resolves the named import fine, so this only shows up when the
built server actually starts.

Author: ericallam

apps/webapp/app/v3/tracer.server.ts

@@ -15,7 +15,7 @@ import {
   metrics,
   type Meter,
 } from "@opentelemetry/api";
-import { SentryContextManager } from "@sentry/remix";
+import sentryRemix from "@sentry/remix";
 import { logs, SeverityNumber } from "@opentelemetry/api-logs";
 import { OTLPTraceExporter } from "@opentelemetry/exporter-trace-otlp-http";
 import { OTLPLogExporter } from "@opentelemetry/exporter-logs-otlp-http";
@@ -220,9 +220,13 @@ function getResource() {
  * the request attributed to each Sentry event) from leaking between concurrent
  * requests. It extends `AsyncLocalStorageContextManager`, so OTel behaviour is
  * unchanged.
+ *
+ * Reached through the default export because `@sentry/remix` is CommonJS and
+ * Node's ESM loader does not detect this transitively re-exported name, so a
+ * named import resolves at build time and then fails when the server boots.
  */
 function createContextManager() {
-  return new SentryContextManager();
+  return new sentryRemix.SentryContextManager();
 }
 
 function setupTelemetry() {

apps/webapp/test/sentryRequestIsolation.test.ts

@@ -1,7 +1,7 @@
 import { context } from "@opentelemetry/api";
 import { NodeTracerProvider } from "@opentelemetry/sdk-trace-node";
 import * as Sentry from "@sentry/remix";
-import { SentryContextManager } from "@sentry/remix";
+import sentryRemix from "@sentry/remix";
 import { afterEach, beforeAll, describe, expect, it } from "vitest";
 
 /**
@@ -42,7 +42,7 @@ describe("Sentry request isolation", () => {
   });
 
   it("keeps each request's isolation scope separate with SentryContextManager", async () => {
-    new NodeTracerProvider().register({ contextManager: new SentryContextManager() });
+    new NodeTracerProvider().register({ contextManager: new sentryRemix.SentryContextManager() });
 
     const observed = await raceTwoRequests();