test(approvals): pin the #10153 harness fake engine to ObjectQL's write dispatch

check:engine-double-contract flagged the harness double's delete()/update() as
looser than the engine they stand in for. Route both through
assertEngineDeleteDispatch / assertEngineUpdateDispatch and record the new
pinned coverage in the retained ledger, as the gate's own remedy prescribes.

Part of #10153

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PnJHU45vPJj5UQrxe946Bx

Author: os-warren

packages/plugins/plugin-approvals/src/manager-approver-org-screen.premise.test.ts

@@ -22,6 +22,7 @@
  *           therefore moves that input from accepted to refused.
  */
 import { describe, it, expect, beforeEach } from 'vitest';
+import { assertEngineDeleteDispatch, assertEngineUpdateDispatch } from '@objectstack/objectql';
 import { ApprovalService } from './approval-service.js';
 
 function makeFakeEngine() {
@@ -50,14 +51,35 @@ function makeFakeEngine() {
       return rows.slice(0, options?.limit ?? 1000);
     },
     async insert(object: string, data: any) { ensure(object).push({ ...data }); return { ...data }; },
-    async update(object: string, idOrData: any, _opts?: any) {
-      const data = typeof idOrData === 'object' ? idOrData : _opts;
-      const id = typeof idOrData === 'object' ? idOrData.id : idOrData;
-      const t = ensure(object); const i = t.findIndex(r => r.id === id);
+    async update(object: string, data: any, options?: any) {
+      // Pinned to ObjectQL.update's OWN dispatch predicate — a double looser
+      // than the engine it stands in for turns a green suite into no suite.
+      const dispatch = assertEngineUpdateDispatch(data, options);
+      const t = ensure(object);
+      if (dispatch.kind === 'multi') {
+        let n = 0;
+        for (let i = 0; i < t.length; i++) {
+          if (matches(t[i], options?.where)) { t[i] = { ...t[i], ...data }; n++; }
+        }
+        return { updated: n };
+      }
+      const i = t.findIndex(r => r.id === dispatch.id);
       if (i >= 0) t[i] = { ...t[i], ...data };
       return t[i];
     },
-    async delete() { return {}; },
+    async delete(object: string, options?: any) {
+      const dispatch = assertEngineDeleteDispatch(options);
+      const t = ensure(object);
+      if (dispatch.kind === 'multi') {
+        const survivors = t.filter(r => !matches(r, options?.where));
+        const deleted = t.length - survivors.length;
+        t.splice(0, t.length, ...survivors);
+        return { deleted };
+      }
+      const i = t.findIndex(r => r.id === dispatch.id);
+      if (i >= 0) t.splice(i, 1);
+      return { id: dispatch.id };
+    },
     registerHook() {}, unregisterHooksByPackage() { return 0; }, async fire() {},
   };
 }

scripts/engine-double-contract.pinned.json

@@ -1016,6 +1016,16 @@
       "verb": "delete",
       "pinned": 1
     },
+    {
+      "file": "packages/plugins/plugin-approvals/src/manager-approver-org-screen.premise.test.ts",
+      "verb": "delete",
+      "pinned": 1
+    },
+    {
+      "file": "packages/plugins/plugin-approvals/src/manager-approver-org-screen.premise.test.ts",
+      "verb": "update",
+      "pinned": 1
+    },
     {
       "file": "packages/plugins/plugin-approvals/src/record-reader-visibility.test.ts",
       "verb": "delete",