fix(cloudflare): Bind pass-through Queue producer methods to the binding

dev-packages/cloudflare-integration-tests/suites/queue/index.ts

@@ -35,6 +35,10 @@ export default Sentry.withSentry(
         return new Response('enqueued batch');
       }
 
+      if (url.pathname === '/metrics') {
+        return Response.json(await env.MY_QUEUE.metrics());
+      }
+
       return new Response('not found', { status: 404 });
     },
     async queue(batch: MessageBatch<{ trigger?: 'error'; payload?: string }>) {

dev-packages/cloudflare-integration-tests/suites/queue/test.ts

@@ -126,3 +126,19 @@ it('emits a queue.publish span with batch attributes on env.MY_QUEUE.sendBatch',
   await runner.makeRequest('post', '/enqueue/batch');
   await runner.completed();
 });
+
+it('calls non-instrumented Queue methods like metrics() on the underlying binding', async ({ signal }) => {
+  const runner = createRunner(__dirname)
+    .expect((envelope: Envelope) => {
+      expect(envelopeItemType(envelope)).toBe('transaction');
+      expect(envelopeItem(envelope)).toMatchObject({
+        transaction: 'GET /metrics',
+        contexts: { trace: { status: 'ok' } },
+      });
+    })
+    .start(signal);
+
+  const metrics = await runner.makeRequest('get', '/metrics');
+  expect(metrics).toMatchObject({ backlogCount: expect.any(Number), backlogBytes: expect.any(Number) });
+  await runner.completed();
+});

packages/cloudflare/src/instrumentations/worker/instrumentQueueProducer.ts

@@ -99,7 +99,9 @@ export function instrumentQueueProducer<T extends Queue>(queue: T, bindingName:
         };
       }
 
-      return Reflect.get(target, prop, receiver);
+      const value = Reflect.get(target, prop) as unknown;
+
+      return typeof value === 'function' ? value.bind(target) : value;
     },
   });
 }

packages/cloudflare/test/instrumentations/worker/instrumentQueueProducer.test.ts

@@ -178,4 +178,19 @@ describe('instrumentQueueProducer', () => {
     const wrapped = instrumentQueueProducer(queue, 'MY_QUEUE') as Queue & { customMethod: () => string };
     expect(wrapped.customMethod()).toBe('hi');
   });
+
+  test('calls pass-through methods with the underlying queue as `this`', async () => {
+    const metrics = { backlogCount: 3, backlogBytes: 42 };
+    const queue = Object.assign(createMockQueue(), {
+      metrics(this: unknown) {
+        if (this !== queue) {
+          throw new TypeError('Illegal invocation: function called with incorrect `this` reference.');
+        }
+        return Promise.resolve(metrics);
+      },
+    }) as unknown as Queue & { metrics: () => Promise<typeof metrics> };
+    const wrapped = instrumentQueueProducer(queue, 'MY_QUEUE') as typeof queue;
+
+    await expect(wrapped.metrics()).resolves.toEqual(metrics);
+  });
 });