fix(node): Read syncBuiltinESMExports off the module namespace so Deno can load @sentry/node

packages/node/src/integrations/fs/vendored/instrumentation.ts

@@ -26,7 +26,7 @@ import {
   withActiveSpan,
 } from '@sentry/core';
 import * as fsNamespace from 'fs';
-import { syncBuiltinESMExports } from 'module';
+import * as moduleNamespace from 'module';
 import { promisify } from 'util';
 import { CALLBACK_FUNCTIONS, PROMISE_FUNCTIONS, SYNC_FUNCTIONS } from './constants';
 import type { FMember, FPMember, FsInstrumentationConfig, GenericFunction } from './types';
@@ -36,6 +36,11 @@ import { indexFs } from './utils';
 // behind its default export instead. The CJS build has no `.default` and already gets the mutable object.
 const fs = (fsNamespace as { default?: typeof fsNamespace }).default ?? fsNamespace;
 
+// Deno before 2.8.2 has `syncBuiltinESMExports` only on the default export of `node:module`, so a named import fails to link.
+type SyncBuiltinESMExports = { syncBuiltinESMExports?: () => void };
+const moduleExports = moduleNamespace as unknown as SyncBuiltinESMExports & { default?: SyncBuiltinESMExports };
+const syncBuiltinESMExports = moduleExports.default?.syncBuiltinESMExports ?? moduleExports.syncBuiltinESMExports;
+
 const SPAN_ORIGIN = 'auto.file.fs';
 const SPAN_OP = FILE;
 
@@ -312,7 +317,7 @@ export function enableFsInstrumentation(config: FsInstrumentationConfig = {}): v
 
   // Named ESM imports (`import { readFile } from 'fs'`) are snapshots of the CJS exports until synced.
   try {
-    syncBuiltinESMExports();
+    syncBuiltinESMExports?.();
   } catch {
     // Cloudflare workerd exports `syncBuiltinESMExports` but throws `ERR_METHOD_NOT_IMPLEMENTED` when it is called.
   }

packages/node/test/integrations/fs.test.ts

@@ -0,0 +1,17 @@
+import { describe, expect, it, vi } from 'vitest';
+
+const { syncBuiltinESMExports } = vi.hoisted(() => ({ syncBuiltinESMExports: vi.fn() }));
+
+// Deno before 2.8.2 has `syncBuiltinESMExports` only on the default export of `node:module`, so a named import of it fails to link.
+vi.mock('module', () => ({ default: { syncBuiltinESMExports } }));
+vi.mock('fs', () => ({ default: { promises: {}, realpath: {}, realpathSync: {} } }));
+
+describe('enableFsInstrumentation', () => {
+  it('syncs the builtin ESM exports when `module` has no named `syncBuiltinESMExports` export', async () => {
+    const { enableFsInstrumentation } = await import('../../src/integrations/fs/vendored/instrumentation');
+
+    enableFsInstrumentation();
+
+    expect(syncBuiltinESMExports).toHaveBeenCalledTimes(1);
+  });
+});