chore(run-store,run-engine): fix formatting and bind forwarded prisma methods in test proxies

oxfmt the wrapped method signature, and make the lagging-replica test proxies
bind any forwarded method to the real client. Prisma delegates are proxy-based
and not pre-bound, so an unbound forwarded method could trip a this/private-field
brand check when called.

Author: d-cs

internal-packages/run-engine/src/engine/tests/startRunAttemptReadResidency.test.ts

@@ -137,12 +137,17 @@ function laggingLockReplica<C extends RunOpsPrismaClient>(
           return row ? { ...row, lockedById: null } : row;
         };
       }
-      return (target as any)[prop];
+      // Bind forwarded methods to the real client: Prisma delegates are proxy-based and not
+      // pre-bound, so an unbound method would trip a this/private-field brand check when called.
+      const forwarded = (target as any)[prop];
+      return typeof forwarded === "function" ? forwarded.bind(target) : forwarded;
     },
   });
   const client = new Proxy(real as any, {
     get(target, prop) {
-      return prop === "taskRun" ? laggingTaskRun : (target as any)[prop];
+      if (prop === "taskRun") return laggingTaskRun;
+      const forwarded = (target as any)[prop];
+      return typeof forwarded === "function" ? forwarded.bind(target) : forwarded;
     },
   }) as C;
   return { client, wasHit: () => hit };

internal-packages/run-engine/src/engine/tests/waitpointReplicaLag.test.ts

@@ -199,7 +199,9 @@ function laggingPendingReplica<C extends RunOpsPrismaClient>(
           return (target as any).$queryRaw(strings, ...values);
         };
       }
-      return (target as any)[prop];
+      // Bind forwarded methods to the real client (Prisma delegates are proxy-based, not pre-bound).
+      const forwarded = (target as any)[prop];
+      return typeof forwarded === "function" ? forwarded.bind(target) : forwarded;
     },
   }) as C;
   return { client, wasHit: () => hit };

internal-packages/run-store/src/runOpsStore.ts

@@ -833,7 +833,10 @@ export class RoutingRunStore implements RunStore {
   // Snapshot ids are cuids (they always classify LEGACY), and a snapshot's CompletedWaitpoint join
   // co-locates with its run, which may live on either store, so fan out to BOTH and merge (like
   // findWaitpointCompletedSnapshotIds) rather than route by the un-classifiable snapshot id.
-  async findSnapshotCompletedWaitpointIds(snapshotId: string, client?: ReadClient): Promise<string[]> {
+  async findSnapshotCompletedWaitpointIds(
+    snapshotId: string,
+    client?: ReadClient
+  ): Promise<string[]> {
     const [fromNew, fromLegacy] = await Promise.all([
       this.#new.findSnapshotCompletedWaitpointIds(
         snapshotId,