fixup! fix(server-utils): Ensure orchestrion instrumentation lazy registers on Node (#22518)

Author: isaacs

packages/server-utils/src/orchestrion/detect.ts

@@ -10,7 +10,8 @@ import { debug, GLOBAL_OBJ } from '@sentry/core';
  * will ever publish to those channels.
  */
 export function isOrchestrionInjected(): boolean {
-  return !!GLOBAL_OBJ.__SENTRY_ORCHESTRION__;
+  const marker = GLOBAL_OBJ.__SENTRY_ORCHESTRION__;
+  return !!(marker?.runtime || marker?.bundler || marker?.integrations);
 }
 
 /**

packages/server-utils/test/orchestrion/detect.test.ts

@@ -0,0 +1,37 @@
+import { GLOBAL_OBJ } from '@sentry/core';
+import { afterEach, beforeEach, describe, expect, it } from 'vitest';
+import { isOrchestrionInjected } from '../../src/orchestrion/detect';
+
+describe('isOrchestrionInjected', () => {
+  beforeEach(() => {
+    delete GLOBAL_OBJ.__SENTRY_ORCHESTRION__;
+  });
+
+  afterEach(() => {
+    delete GLOBAL_OBJ.__SENTRY_ORCHESTRION__;
+  });
+
+  it('is false when no marker exists', () => {
+    expect(isOrchestrionInjected()).toBe(false);
+  });
+
+  it.each([
+    ['runtime', { runtime: [] }],
+    ['bundler array', { bundler: ['mysql'] }],
+    ['bundler true', { bundler: true }],
+    ['integrations', { integrations: new Map() }],
+  ] as const)('is true when %s injection is present', (_label, marker) => {
+    // Cast through `unknown`: rows are `as const` (readonly) and `bundler: true`
+    // is a valid runtime shape the marker type doesn't spell out.
+    GLOBAL_OBJ.__SENTRY_ORCHESTRION__ = marker as unknown as typeof GLOBAL_OBJ.__SENTRY_ORCHESTRION__;
+    expect(isOrchestrionInjected()).toBe(true);
+  });
+
+  // The bridge is installed before hook registration succeeds, so a marker
+  // carrying only `onInject` means nothing will publish channels. Opt-in paths
+  // (knex, dataloader, Nest) must still fall back to OTel in that case.
+  it('is false when only the on-inject bridge is installed', () => {
+    GLOBAL_OBJ.__SENTRY_ORCHESTRION__ = { onInject: () => {} };
+    expect(isOrchestrionInjected()).toBe(false);
+  });
+});