Cloudflare D1 instrumentation prevents deployment to workers

[depsimon]

Is there an existing issue for this?

• I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
• I have reviewed the documentation https://docs.sentry.io/
• I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/cloudflare - hono

SDK Version

9.34.0

Framework Version

Hono 4.8.3

Link to Sentry event

No response

Reproduction Example/SDK Setup

// lib/db.ts 
import { env } from "cloudflare:workers";

export const db = Sentry.instrumentD1WithSentry(env.DB);

// index.ts
import { Hono } from "hono";
import { db } from "~/lib/db";

type Env = {
	Bindings: CloudflareEnv;
};

const app = new Hono<Env>();

app.get("/", async (c) => {
	const result = await db.exec("SELECT 1");
	
	return c.json({
		message: "Hello Hono!",
		result,
	});
});

export default Sentry.withSentry((env: CloudflareEnv) => {
	return {
		dsn: env.SENTRY_DSN,

		// Adds request headers & IP for users
		sendDefaultPii: true,

		// Enable logs to be sent to Sentry
		_experiments: { enableLogs: true },

		tracesSampleRate: 1.0,
	};
}, app);

Steps to Reproduce

1. Setup D1 instrumentation as per the docs
2. Use that db somewhere in your app
3. Deploy the worker

Expected Result

Without the instrumentation, it works fine both locally & in production.

With the instrumentation, it works locally, but the deployment fails due to env.DB being undefined.

I would expect the behavior to be consistent with the non-instrumented version.

Actual Result

CloudflareApiError: Error 400 uploading worker script worker 'backend': Uncaught TypeError: Cannot read properties of undefined (reading 'prepare')
  at worker.js:59795:31 in instrumentD1WithSentry

[AbhiPrasad]

Hey @depsimon thanks for writing in.

if env.DB is undefined wouldn't await db.exec fail as well? I guess a workaround for now is to call instrumentD1WithSentry conditionally based on if env.DB is defined.

[depsimon]

Hey @depsimon thanks for writing in.

if env.DB is undefined wouldn't await db.exec fail as well? I guess a workaround for now is to call instrumentD1WithSentry conditionally based on if env.DB is defined.

Hi @AbhiPrasad thanks for your quick response.

I didn't think about that, it might very well be an issue only during the deployment (where the env.DB is not used anyway) so I guess it can be fine to use that workaround.

It let me deploy at least!

Now I just think there might still be something weird I tested two queries and only one of them was captured.

export const db = env.DB ? Sentry.instrumentD1WithSentry(env.DB) : env.DB;

import { db } from "~/db";

app.get("/", async (c) => {
	const result = await db.exec("SELECT 1"); // This is not captured
	const result = await Sentry.instrumentD1WithSentry(db).exec("SELECT 1"); // This is not captured either

	return c.json({
		message: "Hello Hono!",
		result,
	});
});

What gets captured though is the queries run by betterAuth + drizzle adapter.

import { env } from "cloudflare:workers";
import { betterAuth, type SecondaryStorage } from "better-auth";
import { drizzleAdapter } from "better-auth/adapters/drizzle";
import { drizzle } from "drizzle-orm/d1";
import { db } from "~/db";

export const auth = betterAuth({
	database: drizzleAdapter(drizzle(db), {
		provider: "sqlite",
		usePlural: true,
		schema: {
			...schema,
			account: schema.accounts,
			session: schema.sessions,
			user: schema.users,
			verification: schema.verifications,
		},
	}),

	basePath: "/auth",

	emailAndPassword: {
		enabled: true,
	},

	baseURL: env.BETTER_AUTH_URL,
	secret: env.BETTER_AUTH_SECRET,
	secondaryStorage,
});

Am I missing something like only some queries are captured?

[AbhiPrasad]

So both of these should be captured! There's something else going on.

We can take a look!