@pylonsync/sync 0.3.227 → 0.3.229

package/src/test-harness/server.ts

@@ -196,6 +196,42 @@ export class TestServer {
     return s;
   }
 
+  /** Inject a one-shot outcome for the NEXT push: a `network` failure
+   *  (the fetch rejects with no status — simulating offline / connection
+   *  reset) or an HTTP `status` (e.g. 403 permanent rejection, 503
+   *  transient). Lets tests exercise the transient-vs-permanent
+   *  classification in pushInner. */
+  private nextPushOutcome:
+    | { kind: "network" }
+    | { kind: "status"; status: number }
+    | null = null;
+  primeNextPushOutcome(
+    o: { kind: "network" } | { kind: "status"; status: number },
+  ): void {
+    this.nextPushOutcome = o;
+  }
+  consumeNextPushOutcome():
+    | { kind: "network" }
+    | { kind: "status"; status: number }
+    | null {
+    const o = this.nextPushOutcome;
+    this.nextPushOutcome = null;
+    return o;
+  }
+
+  /** Make the NEXT delta pull report `has_more: true` once, exercising
+   *  the change-log tail-pull recursion in pullInner (the path that
+   *  self-deadlocked before the pullInner() fix). */
+  private nextPullHasMore = false;
+  primeNextPullHasMore(): void {
+    this.nextPullHasMore = true;
+  }
+  consumeNextPullHasMore(): boolean {
+    const v = this.nextPullHasMore;
+    this.nextPullHasMore = false;
+    return v;
+  }
+
   // ---- Entity data --------------------------------------------------------
 
   /** Bulk-seed rows for an entity AND emit insert events into the

package/src/test-harness/transport.ts

@@ -259,6 +259,10 @@ async function handle(
       };
     }
     const resp = await server.pull(token, since);
+    // One-shot has_more on a delta pull → drives the tail-pull recursion.
+    if (since > 0 && server.consumeNextPullHasMore()) {
+      return { status: 200, body: { ...resp, has_more: true } };
+    }
     return { status: 200, body: resp };
   }
 
@@ -275,6 +279,18 @@ async function handle(
 
   // /api/sync/push — accept ops from optimistic mutations.
   if (url.endsWith("/api/sync/push") && method === "POST") {
+    const outcome = server.consumeNextPushOutcome();
+    if (outcome?.kind === "network") {
+      // Reject like a real offline fetch: no HTTP status → the engine
+      // classifies it as TRANSIENT (keep pending, retry).
+      throw new Error("simulated network failure (offline)");
+    }
+    if (outcome?.kind === "status") {
+      return {
+        status: outcome.status,
+        body: { error: { code: "PUSH_REJECTED" } },
+      };
+    }
     return { status: 200, body: { ops: [] } };
   }