fix(metadata-protocol): realign the SysMetadataEngine test doubles' transaction signature with the contract (#5696)

CI's `TypeScript Type Check` went red where a scoped `pnpm --filter …
typecheck` could not: @objectstack/metadata-protocol has no `typecheck`
script, so its errors are only ever seen by the DEBT ledger's `--re-measure`,
which reported 63 -> 70 (+7).

All seven were one shape. Six test doubles stand in for `SysMetadataEngine`,
whose `transaction?` member is typed FROM the contract
(`IObjectQLEngine['transaction']`), and each declared its callback as
`(ctx: any) => Promise<T>` — one parameter, from before the callback gained
its `info` argument. A double may be narrower than the producer; it may not
contradict it.

Each now declares `(ctx: any, info: { owned: boolean })` and passes
`{ owned: true }` at the call — the honest value, since every one of these
doubles OPENS the transaction it stands in for, which is exactly what
`ObjectQL.transaction` reports on its own open branch.

Re-measured: 63 raw errors, the frozen count, with zero transaction-related
errors remaining. `check-type-check-coverage --re-measure` reports no upward
drift. metadata-protocol: 49 files / 502 tests pass.

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

Author: claude

packages/metadata-protocol/src/protocol-publish-drafts-endpoint-gate.test.ts

@@ -169,8 +169,8 @@ function makeStubEngine(namespace?: string) {
             rows.delete(found.key);
             return { deleted: 1 };
         },
-        async transaction<T>(cb: (ctx: any) => Promise<T>): Promise<T> {
-            return cb(undefined);
+        async transaction<T>(cb: (ctx: any, info: { owned: boolean }) => Promise<T>): Promise<T> {
+            return cb(undefined, { owned: true });
         },
         registry: {
             registerItem: () => {},

packages/metadata-protocol/src/protocol-publish-drafts-org-scope.test.ts

@@ -166,8 +166,8 @@ function makeStubEngine() {
             rows.delete(found.key);
             return { deleted: 1 };
         },
-        async transaction<T>(cb: (ctx: any) => Promise<T>): Promise<T> {
-            return cb(undefined);
+        async transaction<T>(cb: (ctx: any, info: { owned: boolean }) => Promise<T>): Promise<T> {
+            return cb(undefined, { owned: true });
         },
         registry: {
             registerItem: () => {},

packages/metadata-protocol/src/protocol.read-decorations.test.ts

@@ -92,7 +92,7 @@ function makeStubEngine() {
             assertEngineDeleteDispatch(opts);
             return { deleted: 0 };
         },
-        async transaction<T>(cb: (ctx: any) => Promise<T>): Promise<T> { return cb(undefined); },
+        async transaction<T>(cb: (ctx: any, info: { owned: boolean }) => Promise<T>): Promise<T> { return cb(undefined, { owned: true }); },
         async syncObjectSchema() { /* no DDL in this stub */ },
         registry: {
             listItems: () => [],

packages/metadata-protocol/src/sys-metadata-repository.draft-drain.test.ts

@@ -180,13 +180,13 @@ function makeFakeEngine() {
       rows.delete(found.key);
       return { deleted: 1 };
     },
-    async transaction<T>(cb: (ctx: any) => Promise<T>): Promise<T> {
+    async transaction<T>(cb: (ctx: any, info: { owned: boolean }) => Promise<T>): Promise<T> {
       const rowsSnapshot = new Map(Array.from(rows, ([k, r]) => [k, { ...r }] as const));
       const historySnapshot = historyRows.map((h) => ({ ...h }));
       const outer = pendingRollback;
       pendingRollback = rowsSnapshot;
       try {
-        return await cb({ txn: true });
+        return await cb({ txn: true }, { owned: true });
       } catch (err) {
         rows.clear();
         for (const [k, r] of rowsSnapshot) rows.set(k, r);

packages/metadata-protocol/src/sys-metadata-repository.history-counters.test.ts

@@ -158,11 +158,11 @@ function makeFakeEngine() {
       rows.delete(found.key);
       return { deleted: 1 };
     },
-    async transaction<T>(cb: (ctx: any) => Promise<T>): Promise<T> {
+    async transaction<T>(cb: (ctx: any, info: { owned: boolean }) => Promise<T>): Promise<T> {
       const rowsSnapshot = new Map(Array.from(rows, ([k, r]) => [k, { ...r }] as const));
       const historySnapshot = historyRows.map((h) => ({ ...h }));
       try {
-        return await cb({ txn: true });
+        return await cb({ txn: true }, { owned: true });
       } catch (err) {
         // ACID: a txn body that throws commits nothing at all.
         rows.clear();

packages/metadata-protocol/src/sys-metadata-repository.recorded-by.test.ts

@@ -96,7 +96,7 @@ function makeFakeEngine() {
       rows.delete(found.key);
       return { deleted: 1 };
     },
-    async transaction<T>(cb: (ctx: any) => Promise<T>): Promise<T> { return cb(undefined); },
+    async transaction<T>(cb: (ctx: any, info: { owned: boolean }) => Promise<T>): Promise<T> { return cb(undefined, { owned: true }); },
   };
 }