feat(hono): Emit warning if @sentry/node was imported instead of @sentry/hono/node

Author: s1gr1d

packages/hono/src/node/middleware.ts

@@ -1,4 +1,4 @@
-import { type BaseTransportOptions, debug, type Options, getClient } from '@sentry/core';
+import { type BaseTransportOptions, consoleSandbox, debug, getClient, type Options } from '@sentry/core';
 import type { Env, Hono, MiddlewareHandler } from 'hono';
 import { requestHandler, responseHandler } from '../shared/middlewareHandlers';
 import { applyPatches } from '../shared/applyPatches';
@@ -21,8 +21,19 @@ export const sentry = <E extends Env>(app: Hono<E>, options?: SentryHonoMiddlewa
       'Sentry is not initialized. Call `init()` from @sentry/hono/node in an `instrument.ts` file loaded via `--import` to set up Sentry for your application.',
     );
   } else {
-    sentryClient.getOptions().debug &&
-      debug.log('Sentry is initialized, proceeding to set up Hono `sentry` middleware.');
+    const isInitializedWithHonoSdk = sentryClient.getOptions()._metadata?.sdk?.name === 'sentry.javascript.hono';
+
+    if (!isInitializedWithHonoSdk) {
+      consoleSandbox(() => {
+        // eslint-disable-next-line no-console
+        console.warn(
+          '[Sentry] Detected `Sentry.init` call from `@sentry/node` instead of `@sentry/hono/node`. Import from `@sentry/hono/node` to ensure Hono-specific instrumentation is applied correctly.',
+        );
+      });
+    } else {
+      sentryClient.getOptions().debug &&
+        debug.log('Sentry is initialized, proceeding to set up Hono `sentry` middleware.');
+    }
   }
 
   applyPatches(app);

packages/hono/test/node/middleware.test.ts

@@ -72,15 +72,52 @@ describe('Hono Node Middleware', () => {
       expect(warnSpy).toHaveBeenCalledWith(expect.stringContaining('Sentry is not initialized'));
     });
 
-    it('does not emit a warning when Sentry is already initialized', () => {
+    it('does not emit a warning when Sentry is already initialized with @sentry/hono/node', () => {
       const warnSpy = vi.spyOn(SentryCore.debug, 'warn');
-      const fakeClient = { getOptions: () => ({ debug: false }) };
+      const consoleWarnSpy = vi.spyOn(console, 'warn').mockImplementation(() => undefined);
+      const fakeClient = {
+        getOptions: () => ({
+          debug: false,
+          _metadata: { sdk: { name: 'sentry.javascript.hono' } },
+        }),
+      };
       vi.spyOn(SentryCore, 'getClient').mockReturnValue(fakeClient as unknown as SentryCore.Client);
 
       const app = new Hono();
       sentry(app);
 
       expect(warnSpy).not.toHaveBeenCalled();
+      expect(consoleWarnSpy).not.toHaveBeenCalled();
+      consoleWarnSpy.mockRestore();
+    });
+
+    it('emits a console.warn when Sentry is initialized with @sentry/node instead of @sentry/hono/node', () => {
+      const consoleWarnSpy = vi.spyOn(console, 'warn').mockImplementation(() => undefined);
+      const fakeClient = {
+        getOptions: () => ({
+          debug: false,
+          _metadata: { sdk: { name: 'sentry.javascript.node' } },
+        }),
+      };
+      vi.spyOn(SentryCore, 'getClient').mockReturnValue(fakeClient as unknown as SentryCore.Client);
+
+      const app = new Hono();
+      sentry(app);
+
+      expect(consoleWarnSpy).toHaveBeenCalledWith(expect.stringContaining('@sentry/hono/node'));
+      consoleWarnSpy.mockRestore();
+    });
+
+    it('emits a console.warn when Sentry is initialized without any SDK metadata', () => {
+      const consoleWarnSpy = vi.spyOn(console, 'warn').mockImplementation(() => undefined);
+      const fakeClient = { getOptions: () => ({ debug: false }) };
+      vi.spyOn(SentryCore, 'getClient').mockReturnValue(fakeClient as unknown as SentryCore.Client);
+
+      const app = new Hono();
+      sentry(app);
+
+      expect(consoleWarnSpy).toHaveBeenCalledWith(expect.stringContaining('@sentry/hono/node'));
+      consoleWarnSpy.mockRestore();
     });
   });
 
@@ -145,16 +182,24 @@ describe('Hono Node Middleware', () => {
       expect(warnSpy).toHaveBeenCalledWith(expect.stringContaining('Sentry is not initialized'));
     });
 
-    it('does not emit a warning when Sentry is already initialized', () => {
+    it('does not emit a warning when Sentry is already initialized with @sentry/hono/node', () => {
       const warnSpy = vi.spyOn(SentryCore.debug, 'warn');
-      const fakeClient = { getOptions: () => ({ debug: false }) };
+      const consoleWarnSpy = vi.spyOn(console, 'warn').mockImplementation(() => undefined);
+      const fakeClient = {
+        getOptions: () => ({
+          debug: false,
+          _metadata: { sdk: { name: 'sentry.javascript.hono' } },
+        }),
+      };
       vi.spyOn(SentryCore, 'getClient').mockReturnValue(fakeClient as unknown as SentryCore.Client);
 
       const app = new Hono();
       const middleware = sentry(app);
 
       expect(warnSpy).not.toHaveBeenCalled();
+      expect(consoleWarnSpy).not.toHaveBeenCalled();
       expect(middleware.constructor.name).toBe('AsyncFunction');
+      consoleWarnSpy.mockRestore();
     });
   });