code fix

src/lib/telemetry/auth-events.server.test.ts

@@ -0,0 +1,47 @@
+import { describe, it, expect, vi } from "vitest";
+import OverrideableBuilder from "supertokens-js-override";
+import { withPasswordlessTelemetry } from "./auth-events.server";
+
+vi.mock("@/lib/pocketbase/client", () => ({
+  pbAdmin: {
+    authPromise: Promise.resolve(),
+    getPlayerByAuthId: async () => undefined,
+    createActivity: async () => undefined,
+  },
+}));
+
+const makeOriginalImpl = () => ({
+  createCode: async (input: any) => ({ status: "OK", input }),
+  consumeCode: async (input: any) => ({ status: "OK", user: { id: "u1" }, input }),
+  listCodes: async () => [],
+});
+
+describe("withPasswordlessTelemetry", () => {
+  it("composes with an inner override that spreads the proxy (twilio recipe shape)", async () => {
+    const inner = (originalImplementation: any) => ({
+      ...originalImplementation,
+      consumeCode: async (input: any) => originalImplementation.consumeCode(input),
+    });
+
+    const builder = new OverrideableBuilder(makeOriginalImpl() as any);
+    builder.override(withPasswordlessTelemetry(inner) as any);
+    const impl = builder.build() as any;
+
+    await expect(impl.createCode({ phoneNumber: "+15555550100" })).resolves.toMatchObject({
+      status: "OK",
+    });
+    await expect(impl.consumeCode({ preAuthSessionId: "x" })).resolves.toMatchObject({
+      status: "OK",
+    });
+  });
+
+  it("works without an inner override", async () => {
+    const builder = new OverrideableBuilder(makeOriginalImpl() as any);
+    builder.override(withPasswordlessTelemetry() as any);
+    const impl = builder.build() as any;
+
+    await expect(impl.createCode({ phoneNumber: "+15555550100" })).resolves.toMatchObject({
+      status: "OK",
+    });
+  });
+});

src/lib/telemetry/auth-events.server.ts

@@ -60,11 +60,14 @@ export const withPasswordlessTelemetry =
   (originalImplementation: any) => {
     const base = existing ? existing(originalImplementation) : originalImplementation;
 
+    const callBase = (name: string, input: any) =>
+      (base[name] ?? originalImplementation[name]).call(originalImplementation, input);
+
     return {
       ...base,
       createCode: async (input: any) => {
         try {
-          const response = await base.createCode(input);
+          const response = await callBase("createCode", input);
           recordAuthEvent("auth.otp_sent", {
             phone: input.phoneNumber,
             success: response.status === "OK",
@@ -82,7 +85,7 @@ export const withPasswordlessTelemetry =
       },
       consumeCode: async (input: any) => {
         try {
-          const response = await base.consumeCode(input);
+          const response = await callBase("consumeCode", input);
           if (response.status === "OK") {
             recordAuthEvent("auth.otp_consumed", {
               authUserId: response.user?.id,