fix(cloudflare): Add hono transaction name when error is thrown (#18529)

If the error is caught with the hono error handler, no `trace` data was
sent alongside the event. This PR fixed this.

Before:
<img width="216" height="80" alt="image"
src="https://github.com/user-attachments/assets/35bc99b7-4a59-40e3-aad8-10130f816d68"
/>


After:
<img width="248" height="85" alt="image"
src="https://github.com/user-attachments/assets/dbc89577-0832-4166-8532-5210dc5e5d99"
/>

Author: s1gr1d

dev-packages/cloudflare-integration-tests/expect.ts

@@ -58,7 +58,7 @@ export function expectedEvent(event: Event): Event {
   });
 }
 
-export function eventEnvelope(event: Event): Envelope {
+export function eventEnvelope(event: Event, includeSampleRand = false): Envelope {
   return [
     {
       event_id: UUID_MATCHER,
@@ -69,6 +69,7 @@ export function eventEnvelope(event: Event): Envelope {
         public_key: 'public',
         trace_id: UUID_MATCHER,
         sample_rate: expect.any(String),
+        ...(includeSampleRand && { sample_rand: expect.stringMatching(/^[01](\.\d+)?$/) }),
         sampled: expect.any(String),
         transaction: expect.any(String),
       },

dev-packages/cloudflare-integration-tests/package.json

@@ -14,7 +14,8 @@
   },
   "dependencies": {
     "@langchain/langgraph": "^1.0.1",
-    "@sentry/cloudflare": "10.32.0"
+    "@sentry/cloudflare": "10.32.0",
+    "hono": "^4.0.0"
   },
   "devDependencies": {
     "@cloudflare/workers-types": "^4.20250922.0",

dev-packages/cloudflare-integration-tests/suites/hono/basic/index.ts

@@ -0,0 +1,33 @@
+import * as Sentry from '@sentry/cloudflare';
+import { Hono } from 'hono';
+
+interface Env {
+  SENTRY_DSN: string;
+}
+
+const app = new Hono<{ Bindings: Env }>();
+
+app.get('/', c => {
+  return c.text('Hello from Hono on Cloudflare!');
+});
+
+app.get('/json', c => {
+  return c.json({ message: 'Hello from Hono', framework: 'hono', platform: 'cloudflare' });
+});
+
+app.get('/error', () => {
+  throw new Error('Test error from Hono app');
+});
+
+app.get('/hello/:name', c => {
+  const name = c.req.param('name');
+  return c.text(`Hello, ${name}!`);
+});
+
+export default Sentry.withSentry(
+  (env: Env) => ({
+    dsn: env.SENTRY_DSN,
+    tracesSampleRate: 1.0,
+  }),
+  app,
+);

dev-packages/cloudflare-integration-tests/suites/hono/basic/test.ts

@@ -0,0 +1,53 @@
+import { expect, it } from 'vitest';
+import { eventEnvelope } from '../../../expect';
+import { createRunner } from '../../../runner';
+
+it('Hono app captures errors', async ({ signal }) => {
+  const runner = createRunner(__dirname)
+    // First envelope: error event from Hono error handler
+    .expect(
+      eventEnvelope(
+        {
+          level: 'error',
+          transaction: 'GET /error',
+          exception: {
+            values: [
+              {
+                type: 'Error',
+                value: 'Test error from Hono app',
+                stacktrace: {
+                  frames: expect.any(Array),
+                },
+                mechanism: { type: 'auto.faas.hono.error_handler', handled: false },
+              },
+            ],
+          },
+          request: {
+            headers: expect.any(Object),
+            method: 'GET',
+            url: expect.any(String),
+          },
+        },
+        true,
+      ),
+    )
+    // Second envelope: transaction event
+    .expect(envelope => {
+      const transactionEvent = envelope[1]?.[0]?.[1];
+      expect(transactionEvent).toEqual(
+        expect.objectContaining({
+          type: 'transaction',
+          transaction: 'GET /error',
+          contexts: expect.objectContaining({
+            trace: expect.objectContaining({
+              op: 'http.server',
+              status: 'internal_error',
+            }),
+          }),
+        }),
+      );
+    })
+    .start(signal);
+  await runner.makeRequest('get', '/error', { expectError: true });
+  await runner.completed();
+});

dev-packages/cloudflare-integration-tests/suites/hono/basic/wrangler.jsonc

@@ -0,0 +1,7 @@
+{
+  "name": "hono-basic-worker",
+  "compatibility_date": "2025-06-17",
+  "main": "index.ts",
+  "compatibility_flags": ["nodejs_compat"]
+}
+

packages/cloudflare/src/handler.ts

@@ -67,9 +67,9 @@ export function withSentry<
     ) {
       handler.errorHandler = new Proxy(handler.errorHandler, {
         apply(target, thisArg, args) {
-          const [err] = args;
+          const [err, context] = args;
 
-          getHonoIntegration()?.handleHonoException(err);
+          getHonoIntegration()?.handleHonoException(err, context);
 
           return Reflect.apply(target, thisArg, args);
         },

packages/cloudflare/src/integrations/hono.ts

@@ -1,5 +1,14 @@
 import type { IntegrationFn } from '@sentry/core';
-import { captureException, debug, defineIntegration, getClient } from '@sentry/core';
+import {
+  captureException,
+  debug,
+  defineIntegration,
+  getActiveSpan,
+  getClient,
+  getIsolationScope,
+  getRootSpan,
+  updateSpanName,
+} from '@sentry/core';
 import { DEBUG_BUILD } from '../debug-build';
 
 const INTEGRATION_NAME = 'Hono';
@@ -8,6 +17,11 @@ interface HonoError extends Error {
   status?: number;
 }
 
+// Minimal type - only exported for tests
+export interface HonoContext {
+  req: { method: string; path?: string };
+}
+
 export interface Options {
   /**
    * Callback method deciding whether error should be captured and sent to Sentry
@@ -28,10 +42,14 @@ function isHonoError(err: unknown): err is HonoError {
   return typeof err === 'object' && err !== null && 'status' in (err as Record<string, unknown>);
 }
 
+// Vendored from https://github.com/honojs/hono/blob/d3abeb1f801aaa1b334285c73da5f5f022dbcadb/src/helper/route/index.ts#L58-L59
+const routePath = (c: HonoContext): string => c.req?.path ?? '';
+
 const _honoIntegration = ((options: Partial<Options> = {}) => {
   return {
     name: INTEGRATION_NAME,
-    handleHonoException(err: HonoError): void {
+    // Hono error handler: https://github.com/honojs/hono/blob/d3abeb1f801aaa1b334285c73da5f5f022dbcadb/src/hono-base.ts#L35
+    handleHonoException(err: HonoError, context: HonoContext): void {
       const shouldHandleError = options.shouldHandleError || defaultShouldHandleError;
 
       if (!isHonoError(err)) {
@@ -40,6 +58,18 @@ const _honoIntegration = ((options: Partial<Options> = {}) => {
       }
 
       if (shouldHandleError(err)) {
+        if (context) {
+          const activeSpan = getActiveSpan();
+          const spanName = `${context.req.method} ${routePath(context)}`;
+
+          if (activeSpan) {
+            activeSpan.updateName(spanName);
+            updateSpanName(getRootSpan(activeSpan), spanName);
+          }
+
+          getIsolationScope().setTransactionName(spanName);
+        }
+
         captureException(err, { mechanism: { handled: false, type: 'auto.faas.hono.error_handler' } });
       } else {
         DEBUG_BUILD && debug.log('[Hono] Not capturing exception because `shouldHandleError` returned `false`.', err);

packages/cloudflare/test/handler.test.ts

@@ -1108,7 +1108,8 @@ describe('withSentry', () => {
       const errorHandlerResponse = honoApp.errorHandler?.(error);
 
       expect(handleHonoException).toHaveBeenCalledTimes(1);
-      expect(handleHonoException).toHaveBeenLastCalledWith(error);
+      // 2nd param is context, which is undefined here
+      expect(handleHonoException).toHaveBeenLastCalledWith(error, undefined);
       expect(errorHandlerResponse?.status).toBe(500);
     });
 

packages/cloudflare/test/integrations/hono.test.ts

@@ -2,6 +2,7 @@ import * as sentryCore from '@sentry/core';
 import { type Client, createStackParser } from '@sentry/core';
 import { beforeEach, describe, expect, it, vi } from 'vitest';
 import { CloudflareClient } from '../../src/client';
+import type { HonoContext } from '../../src/integrations/hono';
 import { honoIntegration } from '../../src/integrations/hono';
 
 class FakeClient extends CloudflareClient {
@@ -10,7 +11,11 @@ class FakeClient extends CloudflareClient {
   }
 }
 
-type MockHonoIntegrationType = { handleHonoException: (err: Error) => void };
+type MockHonoIntegrationType = { handleHonoException: (err: Error, ctx: HonoContext) => void };
+
+const sampleContext: HonoContext = {
+  req: { method: 'GET', path: '/vitest-sample' },
+};
 
 describe('Hono integration', () => {
   let client: FakeClient;
@@ -34,7 +39,7 @@ describe('Hono integration', () => {
 
     const error = new Error('hono boom');
     // simulate withSentry wrapping of errorHandler calling back into integration
-    (integration as unknown as MockHonoIntegrationType).handleHonoException(error);
+    (integration as unknown as MockHonoIntegrationType).handleHonoException(error, sampleContext);
 
     expect(captureExceptionSpy).toHaveBeenCalledTimes(1);
     expect(captureExceptionSpy).toHaveBeenLastCalledWith(error, {
@@ -49,6 +54,7 @@ describe('Hono integration', () => {
 
     (integration as unknown as MockHonoIntegrationType).handleHonoException(
       Object.assign(new Error('client err'), { status: 404 }),
+      sampleContext,
     );
     expect(captureExceptionSpy).not.toHaveBeenCalled();
   });
@@ -60,6 +66,7 @@ describe('Hono integration', () => {
 
     (integration as unknown as MockHonoIntegrationType).handleHonoException(
       Object.assign(new Error('redirect'), { status: 302 }),
+      sampleContext,
     );
     expect(captureExceptionSpy).not.toHaveBeenCalled();
   });
@@ -70,7 +77,7 @@ describe('Hono integration', () => {
     integration.setupOnce?.();
 
     const err = Object.assign(new Error('server err'), { status: 500 });
-    (integration as unknown as MockHonoIntegrationType).handleHonoException(err);
+    (integration as unknown as MockHonoIntegrationType).handleHonoException(err, sampleContext);
     expect(captureExceptionSpy).toHaveBeenCalledTimes(1);
   });
 
@@ -79,7 +86,7 @@ describe('Hono integration', () => {
     const integration = honoIntegration();
     integration.setupOnce?.();
 
-    (integration as unknown as MockHonoIntegrationType).handleHonoException(new Error('no status'));
+    (integration as unknown as MockHonoIntegrationType).handleHonoException(new Error('no status'), sampleContext);
     expect(captureExceptionSpy).toHaveBeenCalledTimes(1);
   });
 
@@ -88,7 +95,17 @@ describe('Hono integration', () => {
     const integration = honoIntegration({ shouldHandleError: () => false });
     integration.setupOnce?.();
 
-    (integration as unknown as MockHonoIntegrationType).handleHonoException(new Error('blocked'));
+    (integration as unknown as MockHonoIntegrationType).handleHonoException(new Error('blocked'), sampleContext);
     expect(captureExceptionSpy).not.toHaveBeenCalled();
   });
+
+  it('does not throw error without passed context and still captures', () => {
+    const captureExceptionSpy = vi.spyOn(sentryCore, 'captureException');
+    const integration = honoIntegration();
+    integration.setupOnce?.();
+
+    // @ts-expect-error context is not passed
+    (integration as unknown as MockHonoIntegrationType).handleHonoException(new Error());
+    expect(captureExceptionSpy).toHaveBeenCalledTimes(1);
+  });
 });

yarn.lock

@@ -18848,10 +18848,10 @@ homedir-polyfill@^1.0.1:
   dependencies:
     parse-passwd "^1.0.0"
 
-hono@^4.9.8:
-  version "4.9.8"
-  resolved "https://registry.yarnpkg.com/hono/-/hono-4.9.8.tgz#1710981135ec775fe26fab5ea6535b403e92bcc3"
-  integrity sha512-JW8Bb4RFWD9iOKxg5PbUarBYGM99IcxFl2FPBo2gSJO11jjUDqlP1Bmfyqt8Z/dGhIQ63PMA9LdcLefXyIasyg==
+hono@^4.0.0, hono@^4.9.8:
+  version "4.11.1"
+  resolved "https://registry.yarnpkg.com/hono/-/hono-4.11.1.tgz#cb1b0c045fc74a96c693927234c95a45fb46ab0b"
+  integrity sha512-KsFcH0xxHes0J4zaQgWbYwmz3UPOOskdqZmItstUG93+Wk1ePBLkLGwbP9zlmh1BFUiL8Qp+Xfu9P7feJWpGNg==
 
 hookable@^5.5.3:
   version "5.5.3"