@peerbit/shared-log 16.0.27 → 16.0.28

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@peerbit/shared-log",
-  "version": "16.0.27",
+  "version": "16.0.28",
   "description": "Shared log",
   "sideEffects": false,
   "type": "module",
@@ -62,22 +62,22 @@
     "pidusage": "^4.0.1",
     "pino": "^9.4.0",
     "uint8arrays": "^5.1.0",
+    "@peerbit/cache": "3.1.1",
     "@peerbit/blocks": "4.3.0",
     "@peerbit/blocks-interface": "2.2.0",
-    "@peerbit/cache": "3.1.1",
     "@peerbit/crypto": "3.1.6",
-    "@peerbit/any-store": "2.2.17",
     "@peerbit/diagnostics": "0.0.1",
     "@peerbit/indexer-interface": "3.0.13",
+    "@peerbit/any-store": "2.2.17",
     "@peerbit/indexer-sqlite3": "3.0.20",
     "@peerbit/log": "6.2.32",
     "@peerbit/logger": "2.0.2",
     "@peerbit/program": "6.0.59",
     "@peerbit/pubsub": "5.4.5",
-    "@peerbit/riblt": "1.2.0",
     "@peerbit/pubsub-interface": "5.2.2",
-    "@peerbit/rpc": "6.1.27",
+    "@peerbit/riblt": "1.2.0",
     "@peerbit/time": "3.0.1",
+    "@peerbit/rpc": "6.1.27",
     "@peerbit/stream-interface": "6.0.16"
   },
   "optionalDependencies": {
@@ -88,11 +88,11 @@
     "@types/libsodium-wrappers": "^0.7.14",
     "@types/pidusage": "^2.0.5",
     "uuid": "^11.1.1",
+    "@peerbit/indexer-simple": "1.2.17",
     "@peerbit/any-store-rust": "0.1.7",
+    "peerbit": "5.4.0",
     "@peerbit/indexer-rust": "1.0.14",
-    "@peerbit/indexer-simple": "1.2.17",
-    "@peerbit/test-utils": "3.1.39",
-    "peerbit": "5.4.0"
+    "@peerbit/test-utils": "3.1.39"
   },
   "repository": {
     "type": "git",

package/src/index.ts

@@ -2723,6 +2723,10 @@ export class SharedLog<
 	// The epoch tokens are PeerSession objects (src/peer-session.ts); the map
 	// itself lives on the registry. Stage-2 of the fence refactor.
 	private _peerSessions!: PeerSessionRegistry;
+	// An exact successor session gets at most one unchanged-Full repair. Weak
+	// ownership keeps retired transport generations collectible without an
+	// identity-keyed cleanup path.
+	private _successorFullRepairSessions!: WeakSet<PeerSession>;
 	// A superseded removal may be the queue item that actually observed an active
 	// replicator. Carry that leave obligation to the transition that ultimately
 	// wins, while a winning reconnect clears it without emitting a stale leave.
@@ -4307,6 +4311,7 @@ export class SharedLog<
 		// replication-info blocked set (fence B5) alongside the session maps —
 		// the legacy inline `new Set()` that sat above moved there.
 		this._peerSessions = this.createPeerSessionRegistry();
+		this._successorFullRepairSessions = new WeakSet();
 		this._pendingReplicatorLeaveByPeer = new Set();
 		this._activeReceiveHandlersByPeer = new Map();
 		this._receiveHandlerDrainByPeer = new Map();
@@ -17959,6 +17964,7 @@ export class SharedLog<
 			})) > 0;
 
 		this._gidPeersHistory = new Map();
+		this._successorFullRepairSessions = new WeakSet();
 		this.resetGidPeerHistoryCleanupState();
 		const replicationChangeOwnershipLifecycleController =
 			this.captureReplicationOwnershipLifecycle();
@@ -20758,6 +20764,7 @@ export class SharedLog<
 			this._receiveHandlerDrainByPeer?.clear();
 			this._openingSyncCapabilitiesByPeer?.clear();
 			this._gidPeersHistory?.clear();
+			this._successorFullRepairSessions = new WeakSet();
 			this._peerSyncCapabilities?.clear();
 			this._peerSyncCapabilitySessions?.clear();
 			this._peerSyncCapabilityTimestamps?.clear();
@@ -24026,6 +24033,10 @@ export class SharedLog<
 		) {
 			return;
 		}
+		const isFirstSuccessorFull =
+			msg instanceof FullReplicationInfoV2Message &&
+			receiveSession.hasPredecessor &&
+			!this._successorFullRepairSessions.has(receiveSession);
 		const admission = this._v2Receive.reserve(msg, {
 			from,
 			peerSession: receiveSession,
@@ -24059,6 +24070,8 @@ export class SharedLog<
 				const exactGate = () =>
 					hostGate() && this._v2Receive.isAdmissionCurrent(admission);
 				let durableCommitted = false;
+				let committedSuccessorFull = false;
+				let repairUnchangedSuccessorFull = false;
 
 				try {
 					if (
@@ -24102,6 +24115,12 @@ export class SharedLog<
 						) {
 							return;
 						}
+						committedSuccessorFull =
+							isFirstSuccessorFull &&
+							msg instanceof FullReplicationInfoV2Message &&
+							msg.segments.length > 0;
+						repairUnchangedSuccessorFull =
+							committedSuccessorFull && result.length === 0;
 					} else {
 						if (!exactGate()) {
 							return;
@@ -24175,6 +24194,30 @@ export class SharedLog<
 				if (!hostGate()) {
 					return;
 				}
+				if (committedSuccessorFull) {
+					if (!exactGate()) {
+						return;
+					}
+					this._successorFullRepairSessions.add(receiveSession);
+					if (repairUnchangedSuccessorFull) {
+						// The unchanged range rows belong to the public-key principal, but
+						// delivery evidence may belong to the superseded process. Reuse the
+						// receipt-driven authoritative repair path and bypass only those stale
+						// hints; do not synthesize public membership or range-change events.
+						this._repairFrontierBypassKnownPeersByMode
+							.get("join-authoritative")
+							?.add(fromHash);
+						const peers = new Set([fromHash]);
+						this.scheduleRepairSweep(
+							{ mode: "join-authoritative", peers },
+							lane.ownershipLifecycleController,
+						);
+						this.scheduleJoinAuthoritativeRepair(
+							peers,
+							lane.ownershipLifecycleController,
+						);
+					}
+				}
 				this._liveness.markReplicatorActivity(fromHash);
 				// A committed V2 announcement is applied progress: the peer answers,
 				// so recovery re-solicitation may restart from the base interval.
@@ -25141,6 +25184,11 @@ export class SharedLog<
 			timeout?: number;
 		},
 	) {
+		if (this.closed) {
+			throw new ClosedError();
+		}
+		const closeSignal = this._closeController.signal;
+		if (closeSignal.aborted) throw new ClosedError();
 		if (options?.signal?.aborted) {
 			throw new AbortError();
 		}
@@ -25149,6 +25197,9 @@ export class SharedLog<
 		const resolvedRoleAge = options?.eager
 			? undefined
 			: (options?.roleAge ?? (await this.getDefaultMinRoleAge()));
+		if (this.closed || closeSignal.aborted) {
+			throw new ClosedError();
+		}
 		if (options?.signal?.aborted) {
 			throw new AbortError();
 		}
@@ -25164,6 +25215,7 @@ export class SharedLog<
 			this.events.removeEventListener("replicator:mature", runCheck);
 			this.events.removeEventListener("replication:change", runCheck);
 			options?.signal?.removeEventListener("abort", onAbort);
+			closeSignal.removeEventListener("abort", onClose);
 			if (timer != null) {
 				clearTimeout(timer);
 				timer = undefined;
@@ -25203,9 +25255,11 @@ export class SharedLog<
 		};
 
 		const onAbort = () => reject(new AbortError());
+		const onClose = () => reject(new ClosedError());
 		if (options?.signal) {
-			options.signal.addEventListener("abort", onAbort);
+			options.signal.addEventListener("abort", onAbort, { once: true });
 		}
+		closeSignal.addEventListener("abort", onClose, { once: true });
 
 		timer = setTimeout(() => {
 			reject(
@@ -25248,31 +25302,32 @@ export class SharedLog<
 				return;
 			}
 
-			if (requestAttempts >= maxRequestAttempts) {
-				return;
-			}
-
-			requestAttempts++;
+			if (requestAttempts < maxRequestAttempts) {
+				requestAttempts++;
 
-			const peerHash = key.hashcode();
-			const peerSession = this._peerSessions.current(peerHash);
-			if (peerSession?.phase === "open") {
-				this._v2Receive.ensureRequestProgress({
-					peerHash,
-					peerSession,
-					receiveEpoch: this._peerSessions.receiveEpoch(peerHash),
-				});
-			} else if (peerSession === null || peerSession.phase === "departing") {
-				// A peer can be known to routing before its SharedLog topic
-				// subscription has been observed. Legacy requests used to bootstrap
-				// that case directly; V2 needs an authoritative Subscribe snapshot
-				// before it can create a fenced PeerSession and request a Full.
-				requestSubscriberSnapshot();
+				const peerHash = key.hashcode();
+				const peerSession = this._peerSessions.current(peerHash);
+				if (peerSession?.phase === "open") {
+					this._v2Receive.ensureRequestProgress({
+						peerHash,
+						peerSession,
+						receiveEpoch: this._peerSessions.receiveEpoch(peerHash),
+					});
+				} else if (peerSession === null || peerSession.phase === "departing") {
+					// A peer can be known to routing before its SharedLog topic
+					// subscription has been observed. Legacy requests used to bootstrap
+					// that case directly; V2 needs an authoritative Subscribe snapshot
+					// before it can create a fenced PeerSession and request a Full.
+					requestSubscriberSnapshot();
+				}
 			}
 
-			if (requestAttempts < maxRequestAttempts) {
-				requestTimer = setTimeout(requestReplicationInfo, requestIntervalMs);
-			}
+			// Replication change events are wake hints, not replayable state. Keep
+			// checking the authoritative index after the bounded network recovery
+			// attempts are exhausted so a transiently empty transition read cannot
+			// strand an otherwise-ready waiter until its outer timeout.
+			runCheck();
+			requestTimer = setTimeout(requestReplicationInfo, requestIntervalMs);
 		};
 
 		const check = async () => {
@@ -25324,8 +25379,9 @@ export class SharedLog<
 		// replay a maturity/change event that fires while that read is in flight.
 		this.events.addEventListener("replicator:mature", runCheck);
 		this.events.addEventListener("replication:change", runCheck);
-		requestReplicationInfo();
-		runCheck();
+		if (closeSignal.aborted) onClose();
+		else if (options?.signal?.aborted) onAbort();
+		else requestReplicationInfo();
 
 		return deferred.promise.finally(clear);
 	}