feat(remix): Add more missing @sentry/node re-exports (getsentry#10391)

Lms24 · web-flow · commit 2267493882d7 · 2024-01-30T09:37:32.000+01:00
Missed some exports in my manual pass in getsentry#10385. Test in getsentry#10389 discovered more missing exports which this PR adds or marks as unnecessary in the re-export test.
diff --git a/dev-packages/e2e-tests/test-applications/node-exports-test-app/scripts/consistentExports.ts b/dev-packages/e2e-tests/test-applications/node-exports-test-app/scripts/consistentExports.ts
@@ -22,8 +22,6 @@ const NODE_EXPORTS_IGNORE = [
   'DebugSession',
   'AnrIntegrationOptions',
   'LocalVariablesIntegrationOptions',
-  // deprecated
-  'spanStatusfromHttpCode',
 ];
 
 type Dependent = {
@@ -43,13 +41,10 @@ const DEPENDENTS: Dependent[] = [
     // Next.js doesn't require explicit exports, so we can just merge top level and `default` exports:
     // @ts-expect-error: `default` is not in the type definition but it's defined
     exports: Object.keys({ ...SentryNextJs, ...SentryNextJs.default }),
-    ignoreExports: ['withSentryConfig'],
   },
   {
     package: '@sentry/remix',
     exports: Object.keys(SentryRemix),
-    // TODO: Fix exports in remix
-    skip: true,
   },
   {
     package: '@sentry/serverless',
@@ -58,9 +53,9 @@ const DEPENDENTS: Dependent[] = [
       // Deprecated, no need to add this now to serverless
       'extractTraceparentData',
       'getModuleFromFilename',
+      'enableAnrDetection',
       // TODO: Should these be exported from serverless?
       'cron',
-      'enableAnrDetection',
       'runWithAsyncContext',
       'hapiErrorPlugin',
     ],
diff --git a/packages/remix/src/index.server.ts b/packages/remix/src/index.server.ts
@@ -39,6 +39,7 @@ export {
   // eslint-disable-next-line deprecation/deprecation
   makeMain,
   setCurrentClient,
+  NodeClient,
   Scope,
   // eslint-disable-next-line deprecation/deprecation
   startTransaction,
@@ -83,6 +84,18 @@ export {
   isInitialized,
   cron,
   parameterize,
+  metrics,
+  // eslint-disable-next-line deprecation/deprecation
+  getModuleFromFilename,
+  createGetModuleFromFilename,
+  functionToStringIntegration,
+  hapiErrorPlugin,
+  inboundFiltersIntegration,
+  linkedErrorsIntegration,
+  requestDataIntegration,
+  runWithAsyncContext,
+  // eslint-disable-next-line deprecation/deprecation
+  enableAnrDetection,
 } from '@sentry/node';
 
 // Keeping the `*` exports for backwards compatibility and types
