@spooky-sync/core 0.0.1-canary.206 → 0.0.1-canary.208

package/dist/index.d.ts

@@ -537,15 +537,29 @@ declare class CacheModule implements StreamUpdateReceiver {
   private versionLookups;
   /** Shared-tabs leader: fan every committed ingest out to follower circuits.
    *  Fired AFTER the local tx (the rows are already in the shared store, so a
-   *  follower only needs the circuit feed). Never set on followers. */
+   *  follower only needs the circuit feed). A follower relays its own
+   *  mutations to the leader the same way, see {@link setIngestRelay}. */
   private ingestRelay;
+  /** See {@link setIngestRelay}. */
+  private relayLocalWritesOnly;
   constructor(local: LocalStore, streamProcessor: StreamProcessorService, streamUpdateCallback: (update: StreamUpdate) => void, logger: Logger$1);
   /**
    * Implements StreamUpdateReceiver interface
    * Called directly by StreamProcessor when views change
    */
   onStreamUpdate(update: StreamUpdate): void;
-  setIngestRelay(cb: ((tuples: CacheIngestTuple[]) => void) | null): void;
+  /**
+   * Fan every committed ingest out to the other tabs. The leader relays
+   * everything (its sync fetches are the only copy the followers get). A
+   * follower relays with `localWritesOnly`: just the mutation path, which is
+   * the only thing it knows that the leader does not. Its sync-fetched
+   * batches are the leader's data coming back and must not be re-broadcast,
+   * or every follower registration would fan its whole working set to
+   * every tab.
+   */
+  setIngestRelay(cb: ((tuples: CacheIngestTuple[]) => void) | null, opts?: {
+    localWritesOnly?: boolean;
+  }): void;
   /**
    * Shared-tabs follower: feed relayed tuples into THIS tab's circuit only.
    * The rows are already in the shared store (the leader wrote them), so no
@@ -1035,6 +1049,15 @@ declare class DataModule<S extends SchemaStructure> {
    * Rollback a failed optimistic update by restoring the previous record state
    */
   rollbackUpdate(recordId: RecordId, tableName: string, beforeRecord: Record<string, unknown>): Promise<void>;
+  /**
+   * Force a re-materialize + notify of every active query on `tableName`.
+   * Used after a DELETE landed in the local store (this tab's own, or one
+   * relayed from another tab): the SSP may not emit a view update for a
+   * DELETE ingest, and the re-materialize reads the store, which already
+   * excludes the row. Each query is isolated so one failing re-materialize
+   * can't stop the others.
+   */
+  notifyTableQueries(tableName: string): Promise<void>;
   /**
    * Remove a record from all active query states and notify subscribers
    */
@@ -1085,6 +1108,14 @@ type FollowerToLeaderMessage = {
   mutationId: string;
 } | {
   type: 'request-poll';
+}
+/** An optimistic write this follower committed to the SHARED store and
+ *  ingested into its own circuit. The leader ingests it (no DB write, the
+ *  row is already there) and fans it out to every OTHER follower as
+ *  `ingest-relay`, so a follower's write lands in every tab in one hop
+ *  instead of after the server round-trip. */ | {
+  type: 'ingest';
+  tuples: IngestTuple[];
 };
 type LeaderToFollowerMessage = {
   type: 'db-ready';
@@ -1114,6 +1145,15 @@ type LeaderToFollowerMessage = {
   recordId: string;
   eventType: 'create' | 'update' | 'delete';
   error: string;
+}
+/** The leader's drain pushed a mutation and deleted its outbox row from the
+ *  SHARED store. Every follower starts its settled-write grace so a row it
+ *  was rendering as a pending write does not blink out before its
+ *  `_00_list_ref` membership arrives. */ | {
+  type: 'mutation-settled';
+  mutationId: string;
+  recordId: string;
+  eventType: 'create' | 'update' | 'delete';
 };
 //#endregion
 //#region src/services/tabs/coordinator.d.ts
@@ -1150,6 +1190,11 @@ declare class SyncForwarder {
   unbind(): void;
   private post;
   mutationEnqueued(mutationId: string): void;
+  /** An optimistic write this tab just ingested. Deliberately NOT queued while
+   *  detached: a new leader primes its circuit from the shared store, which
+   *  already holds the row, and replaying a stale tuple at it later would put
+   *  an older `_00_rv` in its version memo. */
+  ingest(tuples: IngestTuple[]): void;
   requestPoll(): void;
 }
 //#endregion
@@ -1381,6 +1426,13 @@ declare class Sp00kySync<S extends SchemaStructure> {
   private onMutationDropped;
   /** A forwarded outbox row from a follower: load + drain it. Idempotent. */
   enqueueForwardedMutation(mutationId: string): Promise<void>;
+  /**
+   * Tuples another tab already committed to the shared store: feed them to
+   * THIS tab's circuit (no local write). A DELETE additionally forces a
+   * re-materialize of the table's queries, exactly as the writing tab does
+   * for itself, because the SSP may not emit a view update for it.
+   */
+  private applyRelayedIngest;
   /** A relayed `_00_list_ref` LIVE event: resolve against THIS tab's queries
    *  and run the exact same handling the LIVE subscription would have. */
   private applyRelayedListRefChange;

package/dist/index.js

@@ -1,6 +1,6 @@
 import { a as renderOrderSql, c as serializeRow, i as projectedDataSql, l as resolveRelations, o as renderWhereSql, r as project, s as reviveRow, t as PROMOTION_OPEN_OPTIONS, u as stableKey } from "./sqlite-open.js";
 import { DateTime, Duration, RecordId, Surreal, Uuid, applyDiagnostics, createRemoteEngines } from "surrealdb";
-import { QueryBuilder, RecordId as RecordId$1, cyrb53 } from "@spooky-sync/query-builder";
+import { QueryBuilder, RecordId as RecordId$1, baseFieldOfParam, cyrb53 } from "@spooky-sync/query-builder";
 import pino from "pino";
 import { applyPatch } from "fast-json-patch";
 import init, { Sp00kyProcessor } from "@spooky-sync/ssp-wasm";
@@ -84,6 +84,10 @@ function cleanRecord(tableSchema, record) {
 	for (const [key, value] of Object.entries(record)) if (key === "id" || key.startsWith("_00_") || key in tableSchema) cleaned[key] = value;
 	return cleaned;
 }
+/**
+* Parse a RECORD's fields against the table schema. Anything the schema does not
+* know is dropped, which is what keeps a stray field out of a write.
+*/
 function parseParams(tableSchema, params) {
 	const parsedParams = {};
 	for (const [key, value] of Object.entries(params)) {
@@ -92,6 +96,26 @@ function parseParams(tableSchema, params) {
 	}
 	return parsedParams;
 }
+/**
+* Parse a QUERY's params. Unlike a record's fields, a param name is not always a
+* column: an `_or` branch binds under a synthetic `white__or0` (see
+* `orParamName` in @spooky-sync/query-builder), and the surql that references it
+* is already written. So resolve the column through the synthetic name, and keep
+* a param we cannot type rather than dropping it.
+*
+* Dropping was the bug: `parseParams` kept only column-named params, so every
+* `_or` query registered with `$or0`/`$or1` unbound and matched no rows at all,
+* silently (rowCount 0, errorCount 0).
+*/
+function parseQueryParams(tableSchema, params) {
+	const parsedParams = {};
+	for (const [key, value] of Object.entries(params)) {
+		if (value === void 0) continue;
+		const column = tableSchema[key] ?? tableSchema[baseFieldOfParam(key)];
+		parsedParams[key] = column ? parseValue(key, column, value) : value;
+	}
+	return parsedParams;
+}
 function parseValue(name, column, value) {
 	if (column.recordId) {
 		if (value instanceof RecordId$1) return value;
@@ -4531,15 +4555,7 @@ var DataModule = class DataModule {
 				Category: "sp00ky-client::DataModule::delete"
 			}, "SSP delete-ingest failed; relying on query re-materialize to reflect the delete");
 		}
-		for (const [queryHash, queryState] of this.activeQueries) if (queryState.config.tableName === tableName) try {
-			await this.notifyQuerySynced(queryHash);
-		} catch (err) {
-			this.logger.error({
-				err,
-				queryHash,
-				Category: "sp00ky-client::DataModule::delete"
-			}, "notifyQuerySynced failed after delete");
-		}
+		await this.notifyTableQueries(tableName);
 		const mutationEvent = {
 			type: "delete",
 			mutation_id: mutationId,
@@ -4610,6 +4626,26 @@ var DataModule = class DataModule {
 		}
 	}
 	/**
+	* Force a re-materialize + notify of every active query on `tableName`.
+	* Used after a DELETE landed in the local store (this tab's own, or one
+	* relayed from another tab): the SSP may not emit a view update for a
+	* DELETE ingest, and the re-materialize reads the store, which already
+	* excludes the row. Each query is isolated so one failing re-materialize
+	* can't stop the others.
+	*/
+	async notifyTableQueries(tableName) {
+		for (const [queryHash, queryState] of this.activeQueries) if (queryState.config.tableName === tableName) try {
+			await this.notifyQuerySynced(queryHash);
+		} catch (err) {
+			this.logger.error({
+				err,
+				queryHash,
+				tableName,
+				Category: "sp00ky-client::DataModule::notifyTableQueries"
+			}, "notifyQuerySynced failed after delete");
+		}
+	}
+	/**
 	* Remove a record from all active query states and notify subscribers
 	*/
 	removeRecordFromQueries(recordId) {
@@ -4723,7 +4759,7 @@ var DataModule = class DataModule {
 			...configRecord,
 			id: recordId,
 			plan,
-			params: parseParams(tableSchema.columns, configRecord.params),
+			params: parseQueryParams(tableSchema.columns, params ?? configRecord.params),
 			membershipKey
 		};
 		if (membershipKey && !config.remoteArray?.length) {
@@ -6134,8 +6170,14 @@ var Sp00kySync = class Sp00kySync {
 			switch (msg.type) {
 				case "sync-hello": break;
 				case "mutation-enqueued":
+					this.listRefIdleStreak = 0;
 					this.enqueueForwardedMutation(msg.mutationId);
 					break;
+				case "ingest":
+					this.applyRelayedIngest(msg.tuples);
+					hub.relayIngest(msg.tuples, tabId);
+					this.listRefIdleStreak = 0;
+					break;
 				case "request-poll":
 					this.listRefIdleStreak = 0;
 					break;
@@ -6180,6 +6222,12 @@ var Sp00kySync = class Sp00kySync {
 		this.killRefLiveQuery();
 		forwarder.onLeaderMessage = (msg) => {
 			switch (msg.type) {
+				case "ingest-relay":
+					this.applyRelayedIngest(msg.tuples);
+					break;
+				case "mutation-settled":
+					this.dataModule.noteWriteSettled(msg.recordId, msg.eventType);
+					break;
 				case "list-ref-change":
 					this.applyRelayedListRefChange(msg).catch((err) => {
 						this.logger.error({
@@ -6225,6 +6273,24 @@ var Sp00kySync = class Sp00kySync {
 		if (this.tabRole !== "leader") return;
 		await this.upQueue.enqueueFromDatabase(mutationId);
 	}
+	/**
+	* Tuples another tab already committed to the shared store: feed them to
+	* THIS tab's circuit (no local write). A DELETE additionally forces a
+	* re-materialize of the table's queries, exactly as the writing tab does
+	* for itself, because the SSP may not emit a view update for it.
+	*/
+	applyRelayedIngest(tuples) {
+		this.cache.applyRelayedIngest(tuples);
+		const deletedTables = /* @__PURE__ */ new Set();
+		for (const t of tuples) if (t.op === "DELETE") deletedTables.add(t.table);
+		for (const table of deletedTables) this.dataModule.notifyTableQueries(table).catch((err) => {
+			this.logger.warn({
+				err,
+				table,
+				Category: "sp00ky-client::Sp00kySync::applyRelayedIngest"
+			}, "Re-materialize after relayed delete failed");
+		});
+	}
 	/** A relayed `_00_list_ref` LIVE event: resolve against THIS tab's queries
 	*  and run the exact same handling the LIVE subscription would have. */
 	async applyRelayedListRefChange(msg) {
@@ -6617,8 +6683,20 @@ var Sp00kySync = class Sp00kySync {
 		}, "Live update is being processed");
 		const diff = createDiffFromDbOp(action, recordId, version, localArray);
 		const hash = extractIdPart(existing.config.id);
-		if (existing.config.membershipKnown && (diff.removed.length || diff.added.length)) {
-			const next = applyRecordVersionDiff(existing.config.remoteArray ?? [], diff);
+		if (existing.config.membershipKnown) {
+			const membershipDiff = action === "DELETE" ? {
+				added: [],
+				updated: [],
+				removed: [recordId]
+			} : {
+				added: [{
+					id: recordId,
+					version
+				}],
+				updated: [],
+				removed: []
+			};
+			const next = applyRecordVersionDiff(existing.config.remoteArray ?? [], membershipDiff);
 			if (!recordVersionArraysEqual(next, existing.config.remoteArray ?? [])) await this.dataModule.updateQueryRemoteArray(hash, next);
 		}
 		await this.runSyncForQuery(hash, diff);
@@ -6737,7 +6815,14 @@ var Sp00kySync = class Sp00kySync {
 	* vanish, and return, while every other client showed it throughout.
 	*/
 	handleMutationSettled(event) {
-		this.dataModule.noteWriteSettled(encodeRecordId(event.record_id), event.type);
+		const recordId = encodeRecordId(event.record_id);
+		this.dataModule.noteWriteSettled(recordId, event.type);
+		this.hub?.broadcast({
+			type: "mutation-settled",
+			mutationId: encodeRecordId(event.mutation_id),
+			recordId,
+			eventType: event.type
+		});
 	}
 	async handleRollback(event, error) {
 		const recordId = encodeRecordId(event.record_id);
@@ -7353,8 +7438,8 @@ function selfAllowlistedVariant(flag, userId) {
 
 //#endregion
 //#region src/modules/devtools/index.ts
-const CORE_VERSION = "0.0.1-canary.206";
-const WASM_VERSION = "0.0.1-canary.206";
+const CORE_VERSION = "0.0.1-canary.208";
+const WASM_VERSION = "0.0.1-canary.208";
 const SURREAL_VERSION = "3.0.3";
 var DevToolsService = class DevToolsService {
 	eventsHistory = [];
@@ -8883,8 +8968,11 @@ var CacheModule = class {
 	versionLookups = {};
 	/** Shared-tabs leader: fan every committed ingest out to follower circuits.
 	*  Fired AFTER the local tx (the rows are already in the shared store, so a
-	*  follower only needs the circuit feed). Never set on followers. */
+	*  follower only needs the circuit feed). A follower relays its own
+	*  mutations to the leader the same way, see {@link setIngestRelay}. */
 	ingestRelay = null;
+	/** See {@link setIngestRelay}. */
+	relayLocalWritesOnly = false;
 	constructor(local, streamProcessor, streamUpdateCallback, logger) {
 		this.local = local;
 		this.streamProcessor = streamProcessor;
@@ -8904,8 +8992,18 @@ var CacheModule = class {
 		}, "Stream update received");
 		this.streamUpdateCallback(update);
 	}
-	setIngestRelay(cb) {
+	/**
+	* Fan every committed ingest out to the other tabs. The leader relays
+	* everything (its sync fetches are the only copy the followers get). A
+	* follower relays with `localWritesOnly`: just the mutation path, which is
+	* the only thing it knows that the leader does not. Its sync-fetched
+	* batches are the leader's data coming back and must not be re-broadcast,
+	* or every follower registration would fan its whole working set to
+	* every tab.
+	*/
+	setIngestRelay(cb, opts = {}) {
 		this.ingestRelay = cb;
+		this.relayLocalWritesOnly = opts.localWritesOnly === true;
 	}
 	/**
 	* Shared-tabs follower: feed relayed tuples into THIS tab's circuit only.
@@ -8996,7 +9094,7 @@ var CacheModule = class {
 			});
 			const ingested = this.streamProcessor.ingestMany(bulk);
 			for (const t of ingested) this.versionLookups[t.id] = versionOf.get(t.id) ?? 0;
-			if (ingested.length > 0) this.ingestRelay?.(ingested);
+			if (ingested.length > 0 && (!this.relayLocalWritesOnly || skipDbInsert)) this.ingestRelay?.(ingested);
 			this.logger.debug({
 				count: records.length,
 				Category: "sp00ky-client::CacheModule::saveBatch"
@@ -9037,7 +9135,7 @@ var CacheModule = class {
 				id,
 				record: recordData
 			}]);
-			this.ingestRelay?.([{
+			if (!this.relayLocalWritesOnly || skipDbDelete) this.ingestRelay?.([{
 				table,
 				op: "DELETE",
 				id,
@@ -10617,6 +10715,19 @@ var SyncForwarder = class {
 			mutationId
 		});
 	}
+	/** An optimistic write this tab just ingested. Deliberately NOT queued while
+	*  detached: a new leader primes its circuit from the shared store, which
+	*  already holds the row, and replaying a stale tuple at it later would put
+	*  an older `_00_rv` in its version memo. */
+	ingest(tuples) {
+		if (!this.port) return;
+		try {
+			this.port.postMessage({
+				type: "ingest",
+				tuples
+			});
+		} catch {}
+	}
 	requestPoll() {
 		this.post({ type: "request-poll" });
 	}
@@ -10865,6 +10976,8 @@ var TabsCoordinator = class {
 		const forwarder = this.forwarder;
 		await new Promise((resolve) => {
 			let adopted = false;
+			let attached = false;
+			const pending = [];
 			const previousHandler = forwarder.onLeaderMessage;
 			forwarder.onLeaderMessage = (msg) => {
 				if (msg.type === "db-ready" && !adopted) {
@@ -10875,11 +10988,18 @@ var TabsCoordinator = class {
 						leadershipId: msg.leadershipId
 					}).then(() => {
 						this.deps.hooks.becomeSyncFollower(forwarder);
+						attached = true;
+						const backlog = pending.splice(0);
+						for (const m of backlog) forwarder.onLeaderMessage?.(m);
 						this.setRole("follower");
 						resolve();
 					});
 					return;
 				}
+				if (!attached) {
+					pending.push(msg);
+					return;
+				}
 				previousHandler?.(msg);
 			};
 			forwarder.rebind(syncPort);
@@ -12303,7 +12423,7 @@ var Sp00kyClient = class {
 		return new TabsCoordinator({
 			tabId,
 			fingerprint: computeTabsFingerprint({
-				coreVersion: "0.0.1-canary.206",
+				coreVersion: "0.0.1-canary.208",
 				schemaHash: hash53(this.config.schemaSurql),
 				endpoint: this.config.database.endpoint ?? "",
 				namespace: this.config.database.namespace,
@@ -12330,18 +12450,10 @@ var Sp00kyClient = class {
 				resumeSyncLeaderDuties: () => this.sync.resumeLeaderDuties(),
 				becomeSyncFollower: (forwarder) => {
 					this.streamProcessor.setPersistenceEnabled(false);
-					this.cache.setIngestRelay(null);
+					this.cache.setIngestRelay((tuples) => forwarder.ingest(tuples), { localWritesOnly: true });
 					this.sync.setTabContext("follower", tabId);
 					this.dataModule.setTabId(tabId);
 					this.sync.demoteToFollower(forwarder);
-					const inner = forwarder.onLeaderMessage;
-					forwarder.onLeaderMessage = (msg) => {
-						if (msg.type === "ingest-relay") {
-							this.cache.applyRelayedIngest(msg.tuples);
-							return;
-						}
-						inner?.(msg);
-					};
 				},
 				becomeSyncSolo: () => {
 					this.streamProcessor.setPersistenceEnabled(true);
@@ -12748,7 +12860,7 @@ var Sp00kyClient = class {
 	async initQuery(table, q, ttl) {
 		const tableSchema = this.config.schema.tables.find((t) => t.name === table);
 		if (!tableSchema) throw new Error(`Table ${table} not found`);
-		const params = parseParams(tableSchema.columns, q.selectQuery.vars ?? {});
+		const params = parseQueryParams(tableSchema.columns, q.selectQuery.vars ?? {});
 		const hash = await this.dataModule.query(table, q.selectQuery.query, params, ttl, q.selectQuery.plan);
 		if (!this.pendingQueryInits.has(hash)) {
 			const chain = this.finishQueryInit(hash, q, params).finally(() => {
@@ -12820,7 +12932,7 @@ var Sp00kyClient = class {
 		const tableName = q.tableName;
 		const tableSchema = this.config.schema.tables.find((t) => t.name === tableName);
 		if (!tableSchema) throw new Error(`Table ${tableName} not found`);
-		const params = parseParams(tableSchema.columns, q.selectQuery.vars ?? {});
+		const params = parseQueryParams(tableSchema.columns, q.selectQuery.vars ?? {});
 		const hashKey = String(q.hash);
 		const marker = await this.dataModule.getPreloadMarker(hashKey);
 		if (!marker) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@spooky-sync/core",
-  "version": "0.0.1-canary.206",
+  "version": "0.0.1-canary.208",
   "type": "module",
   "sideEffects": false,
   "main": "./dist/index.js",
@@ -60,8 +60,8 @@
     }
   },
   "dependencies": {
-    "@spooky-sync/query-builder": "0.0.1-canary.206",
-    "@spooky-sync/ssp-wasm": "0.0.1-canary.206",
+    "@spooky-sync/query-builder": "0.0.1-canary.208",
+    "@spooky-sync/ssp-wasm": "0.0.1-canary.208",
     "@sqlite.org/sqlite-wasm": "3.53.0-build1",
     "@surrealdb/wasm": "^3.0.3",
     "blurhash": "^2.0.5",

package/src/modules/cache/cache.relay.test.ts

@@ -0,0 +1,95 @@
+import { describe, it, expect, vi } from 'vitest';
+import { RecordId } from 'surrealdb';
+import { CacheModule } from './index';
+
+// The ingest relay is how one tab's circuit learns what another tab wrote.
+// The leader relays every ingest (its sync fetches are the only copy the
+// followers get); a follower relays with `localWritesOnly`, just its mutation
+// path, because its sync-fetched batches are the leader's data coming back.
+
+function makeLogger(): any {
+  const logger: any = { debug: () => {}, info: () => {}, warn: () => {}, error: () => {}, trace: () => {} };
+  logger.child = () => logger;
+  return logger;
+}
+
+function setup() {
+  const local: any = { epoch: 1, execute: vi.fn(async () => {}), query: vi.fn(async () => []) };
+  const ssp: any = {
+    addReceiver: vi.fn(),
+    ingestMany: vi.fn((records: unknown[]) => records),
+  };
+  const cache = new CacheModule(local, ssp, () => {}, makeLogger());
+  const relay = vi.fn();
+  return { cache, local, ssp, relay };
+}
+
+const rec = (id: string, version: number) => ({
+  table: 'thread',
+  op: 'CREATE' as const,
+  record: { id: new RecordId('thread', id), title: 't' },
+  version,
+});
+
+describe('CacheModule ingest relay', () => {
+  it('applyRelayedIngest updates the version memo and never re-relays', () => {
+    const { cache, ssp, relay } = setup();
+    cache.setIngestRelay(relay);
+
+    cache.applyRelayedIngest([
+      { table: 'thread', op: 'CREATE', id: 'thread:a', record: { id: 'thread:a', _00_rv: 4 } },
+    ]);
+
+    expect(ssp.ingestMany).toHaveBeenCalledTimes(1);
+    expect(cache.lookup('thread:a')).toBe(4);
+    expect(relay).not.toHaveBeenCalled();
+  });
+
+  it('localWritesOnly relays the mutation path (save + delete)', async () => {
+    const { cache, local, relay } = setup();
+    cache.setIngestRelay(relay, { localWritesOnly: true });
+
+    await cache.save(rec('a', 1), true);
+    expect(local.execute).not.toHaveBeenCalled();
+    expect(relay).toHaveBeenCalledTimes(1);
+    expect(relay.mock.calls[0][0][0]).toMatchObject({ table: 'thread', op: 'CREATE', id: 'thread:a' });
+
+    const before = { id: 'thread:a', title: 't' };
+    await cache.delete('thread', 'thread:a', true, before);
+    expect(relay).toHaveBeenCalledTimes(2);
+    expect(relay.mock.calls[1][0]).toEqual([
+      { table: 'thread', op: 'DELETE', id: 'thread:a', record: before },
+    ]);
+  });
+
+  it('localWritesOnly stays silent for sync-fetched batches', async () => {
+    const { cache, local, relay } = setup();
+    cache.setIngestRelay(relay, { localWritesOnly: true });
+
+    await cache.saveBatch([rec('a', 1), rec('b', 1)], false);
+    expect(local.execute).toHaveBeenCalledTimes(1);
+    expect(relay).not.toHaveBeenCalled();
+
+    await cache.delete('thread', 'thread:a', false, {});
+    expect(relay).not.toHaveBeenCalled();
+  });
+
+  it('an unscoped relay fires for every ingest', async () => {
+    const { cache, relay } = setup();
+    cache.setIngestRelay(relay);
+
+    await cache.saveBatch([rec('a', 1)], false);
+    await cache.save(rec('b', 1), true);
+    await cache.delete('thread', 'thread:a', false, {});
+
+    expect(relay).toHaveBeenCalledTimes(3);
+  });
+
+  it('setIngestRelay(null) stops relaying', async () => {
+    const { cache, relay } = setup();
+    cache.setIngestRelay(relay, { localWritesOnly: true });
+    cache.setIngestRelay(null);
+    await cache.save(rec('a', 1), true);
+    expect(relay).not.toHaveBeenCalled();
+  });
+});

package/src/modules/cache/index.ts

@@ -33,8 +33,11 @@ export class CacheModule implements StreamUpdateReceiver {
   private versionLookups: Record<string, number> = {};
   /** Shared-tabs leader: fan every committed ingest out to follower circuits.
    *  Fired AFTER the local tx (the rows are already in the shared store, so a
-   *  follower only needs the circuit feed). Never set on followers. */
+   *  follower only needs the circuit feed). A follower relays its own
+   *  mutations to the leader the same way, see {@link setIngestRelay}. */
   private ingestRelay: ((tuples: CacheIngestTuple[]) => void) | null = null;
+  /** See {@link setIngestRelay}. */
+  private relayLocalWritesOnly = false;
 
   constructor(
     private local: LocalStore,
@@ -64,8 +67,21 @@ export class CacheModule implements StreamUpdateReceiver {
     this.streamUpdateCallback(update);
   }
 
-  setIngestRelay(cb: ((tuples: CacheIngestTuple[]) => void) | null): void {
+  /**
+   * Fan every committed ingest out to the other tabs. The leader relays
+   * everything (its sync fetches are the only copy the followers get). A
+   * follower relays with `localWritesOnly`: just the mutation path, which is
+   * the only thing it knows that the leader does not. Its sync-fetched
+   * batches are the leader's data coming back and must not be re-broadcast,
+   * or every follower registration would fan its whole working set to
+   * every tab.
+   */
+  setIngestRelay(
+    cb: ((tuples: CacheIngestTuple[]) => void) | null,
+    opts: { localWritesOnly?: boolean } = {}
+  ): void {
     this.ingestRelay = cb;
+    this.relayLocalWritesOnly = opts.localWritesOnly === true;
   }
 
   /**
@@ -197,7 +213,11 @@ export class CacheModule implements StreamUpdateReceiver {
       // failed mid-batch is neither "known" here nor fanned out to followers.
       const ingested = this.streamProcessor.ingestMany(bulk);
       for (const t of ingested) this.versionLookups[t.id] = versionOf.get(t.id) ?? 0;
-      if (ingested.length > 0) this.ingestRelay?.(ingested as CacheIngestTuple[]);
+      // `skipDbInsert` is exactly the mutation path (the tx already wrote the
+      // row); sync-fetched batches pass false.
+      if (ingested.length > 0 && (!this.relayLocalWritesOnly || skipDbInsert)) {
+        this.ingestRelay?.(ingested as CacheIngestTuple[]);
+      }
 
       this.logger.debug(
         { count: records.length, Category: 'sp00ky-client::CacheModule::saveBatch' },
@@ -239,7 +259,9 @@ export class CacheModule implements StreamUpdateReceiver {
       // 2. Ingest deletion into DBSP (pass record data so predicates can be matched)
       delete this.versionLookups[id];
       this.streamProcessor.ingestMany([{ table, op: 'DELETE', id, record: recordData }]);
-      this.ingestRelay?.([{ table, op: 'DELETE', id, record: recordData }]);
+      if (!this.relayLocalWritesOnly || skipDbDelete) {
+        this.ingestRelay?.([{ table, op: 'DELETE', id, record: recordData }]);
+      }
 
       this.logger.debug(
         { table, id, Category: 'sp00ky-client::CacheModule::delete' },

package/src/modules/data/data.notify-table.test.ts

@@ -0,0 +1,41 @@
+import { describe, it, expect, vi } from 'vitest';
+import { RecordId } from 'surrealdb';
+import { DataModule } from './index';
+
+// A DELETE that landed in the local store (this tab's own, or one relayed
+// from another tab) needs a forced re-materialize of the table's queries: the
+// SSP may not emit a view update for it.
+
+function makeLogger(): any {
+  const logger: any = { debug: () => {}, info: () => {}, warn: () => {}, error: () => {}, trace: () => {} };
+  logger.child = () => logger;
+  return logger;
+}
+
+const schema = { tables: [{ name: 'comment', columns: {} }, { name: 'thread', columns: {} }] } as any;
+
+function state(hash: string, tableName: string): any {
+  return {
+    config: { id: new RecordId('_00_query', hash), tableName, localArray: [], remoteArray: [] },
+    records: [],
+  };
+}
+
+describe('DataModule.notifyTableQueries', () => {
+  it('re-materializes only the queries on that table, isolating failures', async () => {
+    const local: any = { epoch: 1, query: vi.fn(async () => [[]]) };
+    const dm = new DataModule({ saveBatch: async () => {} } as any, local, schema, makeLogger(), 100);
+    (dm as any).activeQueries.set('c1', state('c1', 'comment'));
+    (dm as any).activeQueries.set('c2', state('c2', 'comment'));
+    (dm as any).activeQueries.set('t1', state('t1', 'thread'));
+    const notified: string[] = [];
+    dm.notifyQuerySynced = vi.fn(async (hash: string) => {
+      notified.push(hash);
+      if (hash === 'c1') throw new Error('boom');
+    }) as any;
+
+    await dm.notifyTableQueries('comment');
+
+    expect(notified.sort()).toEqual(['c1', 'c2']);
+  });
+});