@tomflow/proflow-execution-browser-extension 0.1.49 → 0.1.50

package/CHANGELOG.md

@@ -0,0 +1,7 @@
+# @tomflow/proflow-execution-browser-extension
+
+## 0.1.50
+
+### Patch Changes
+
+- Preserve Observer recovery demand while a recovery pass is already in flight by coalescing reconnect, retry, and recovery requests into one trailing pass, without introducing concurrent recovery or stale retry replay.

package/dist/deployment/adapter.d.ts

@@ -53,7 +53,7 @@ export declare const behaviorAdapter: {
             ok: true;
             status: "SUCCEEDED";
             moduleRef: "execution-browser-extension";
-            moduleVersion: "0.1.49";
+            moduleVersion: "0.1.50";
             data: {
                 loadDir: string;
             };
@@ -66,7 +66,7 @@ export declare const behaviorAdapter: {
             readonly ok: true;
             readonly status: "SUCCEEDED";
             readonly moduleRef: "execution-browser-extension";
-            readonly moduleVersion: "0.1.49";
+            readonly moduleVersion: "0.1.50";
         };
         observedEffects: never[];
     }>;
@@ -76,7 +76,7 @@ export declare const behaviorAdapter: {
             ok: true;
             status: "SUCCEEDED";
             moduleRef: "execution-browser-extension";
-            moduleVersion: "0.1.49";
+            moduleVersion: "0.1.50";
             data: {
                 setupStatus: "ACTION_REQUIRED" | "READY";
                 runtimeStatus: "NOT_APPLICABLE";
@@ -101,7 +101,7 @@ export declare const behaviorAdapter: {
         result: {
             contract: "deployment.result.v1";
             moduleRef: "execution-browser-extension";
-            moduleVersion: "0.1.49";
+            moduleVersion: "0.1.50";
             ok: false;
             status: "ACTION_REQUIRED";
             data: {
@@ -133,14 +133,14 @@ export declare const behaviorAdapter: {
             readonly ok: true;
             readonly status: "SUCCEEDED";
             readonly moduleRef: "execution-browser-extension";
-            readonly moduleVersion: "0.1.49";
+            readonly moduleVersion: "0.1.50";
         };
         observedEffects: string[];
     } | {
         result: {
             contract: "deployment.result.v1";
             moduleRef: "execution-browser-extension";
-            moduleVersion: "0.1.49";
+            moduleVersion: "0.1.50";
             ok: false;
             status: "FAILED";
             error: {
@@ -173,7 +173,7 @@ export declare const behaviorAdapter: {
             ok: true;
             status: "SUCCEEDED";
             moduleRef: "execution-browser-extension";
-            moduleVersion: "0.1.49";
+            moduleVersion: "0.1.50";
             data: {
                 docs: string;
             };
@@ -186,11 +186,11 @@ export declare const behaviorAdapter: {
             readonly ok: true;
             readonly status: "SUCCEEDED";
             readonly moduleRef: "execution-browser-extension";
-            readonly moduleVersion: "0.1.49";
+            readonly moduleVersion: "0.1.50";
         } | {
             contract: "deployment.result.v1";
             moduleRef: "execution-browser-extension";
-            moduleVersion: "0.1.49";
+            moduleVersion: "0.1.50";
             ok: false;
             status: "FAILED";
             error: {
@@ -207,7 +207,7 @@ export declare const behaviorAdapter: {
             readonly ok: true;
             readonly status: "SUCCEEDED";
             readonly moduleRef: "execution-browser-extension";
-            readonly moduleVersion: "0.1.49";
+            readonly moduleVersion: "0.1.50";
         };
         observedEffects: never[];
     }>;

package/dist/deployment/descriptor.d.ts

@@ -3,7 +3,7 @@ export declare const descriptor: {
     readonly contractVersion: "1.0.0";
     readonly moduleRef: "execution-browser-extension";
     readonly packageName: "@tomflow/proflow-execution-browser-extension";
-    readonly moduleVersion: "0.1.49";
+    readonly moduleVersion: "0.1.50";
     readonly kind: "browser-extension";
     readonly templateVersion: "1.0.0";
     readonly platformCompatibility: ">=1.0.0 <2.0.0";

package/dist/deployment/descriptor.js

@@ -3,7 +3,7 @@ export const descriptor = {
     contractVersion: "1.0.0",
     moduleRef: "execution-browser-extension",
     packageName: "@tomflow/proflow-execution-browser-extension",
-    moduleVersion: "0.1.49",
+    moduleVersion: "0.1.50",
     kind: "browser-extension",
     templateVersion: "1.0.0",
     platformCompatibility: ">=1.0.0 <2.0.0",

package/dist/extension/background.js

@@ -14435,6 +14435,16 @@ async function emitStructuredLog(entry) {
     body: JSON.stringify({ timestamp: (/* @__PURE__ */ new Date()).toISOString(), ...entry })
   }).catch(() => void 0);
 }
+function emitObserverRecoveryDiagnostic(status, attemptNo, operationRef) {
+  void emitStructuredLog({
+    level: "INFO",
+    component: "browser-observer-recovery",
+    operation: "observer.recovery",
+    status,
+    attemptNo,
+    operationRef
+  }).catch(() => void 0);
+}
 async function invokeObserverApplication(operation, input) {
   const config2 = await taskApplicationConfig();
   if (!config2) throw new Error("OBSERVER_APPLICATION_NOT_CONFIGURED");
@@ -14535,9 +14545,23 @@ var collaborationCarrier = createCollaborationCarrierApplication({
   },
   agent: {
     async listPendingMessages(limit) {
-      return await invokeObserverApplication("collaboration.listPending", {
-        limit
-      });
+      const recoveryTriggerRef = `bridge-session:${bridgeSessionEpoch}`;
+      emitObserverRecoveryDiagnostic(
+        "COLLABORATION_LIST_PENDING_BEGIN",
+        observerRecoveryAttemptNo,
+        recoveryTriggerRef
+      );
+      try {
+        return await invokeObserverApplication("collaboration.listPending", {
+          limit
+        });
+      } finally {
+        emitObserverRecoveryDiagnostic(
+          "COLLABORATION_LIST_PENDING_SETTLED",
+          observerRecoveryAttemptNo,
+          recoveryTriggerRef
+        );
+      }
     },
     async getPendingMessage(messageRef) {
       return await invokeObserverApplication("collaboration.getPending", {
@@ -14612,18 +14636,46 @@ async function persistSystemObserverState(result) {
   });
 }
 var observerRecoveryInFlight = null;
+var observerRecoveryTrailingRequested = false;
 var observerRecoveryRetryCount = 0;
+var observerRecoveryAttemptNo = 0;
 var nextRecoverySuppressions = [];
 function suppressNextObserverRecovery(denials) {
   nextRecoverySuppressions = denials;
 }
 function runObserverRecovery() {
-  if (observerRecoveryInFlight) return observerRecoveryInFlight;
+  const recoveryTriggerRef = `bridge-session:${bridgeSessionEpoch}`;
+  if (observerRecoveryInFlight) {
+    observerRecoveryTrailingRequested = true;
+    emitObserverRecoveryDiagnostic(
+      "REUSED_IN_FLIGHT",
+      observerRecoveryAttemptNo,
+      recoveryTriggerRef
+    );
+    return observerRecoveryInFlight;
+  }
+  observerRecoveryAttemptNo += 1;
+  const recoveryAttemptNo = observerRecoveryAttemptNo;
+  emitObserverRecoveryDiagnostic(
+    "STARTED",
+    recoveryAttemptNo,
+    recoveryTriggerRef
+  );
   const suppressedContinuations = nextRecoverySuppressions;
   nextRecoverySuppressions = [];
   observerRecoveryInFlight = (async () => {
     let recoveryNeedsRetry = false;
+    emitObserverRecoveryDiagnostic(
+      "COLLABORATION_RECOVERY_BEGIN",
+      recoveryAttemptNo,
+      recoveryTriggerRef
+    );
     await collaborationCarrier.recoverPending(50).catch(() => void 0);
+    emitObserverRecoveryDiagnostic(
+      "COLLABORATION_RECOVERY_SETTLED",
+      recoveryAttemptNo,
+      recoveryTriggerRef
+    );
     const signalBatch = await invokeObserverApplication(
       "execution.listSignals",
       { limit: 50 }
@@ -14699,7 +14751,11 @@ function runObserverRecovery() {
     }
     if (recoveryNeedsRetry && observerRecoveryRetryCount < 6) {
       observerRecoveryRetryCount += 1;
-      setTimeout(() => void runObserverRecovery(), 2e3);
+      const retryScheduledFromAttemptNo = recoveryAttemptNo;
+      setTimeout(() => {
+        if (observerRecoveryAttemptNo !== retryScheduledFromAttemptNo) return;
+        void runObserverRecovery();
+      }, 2e3);
     } else if (!recoveryNeedsRetry) {
       observerRecoveryRetryCount = 0;
     }
@@ -14715,10 +14771,18 @@ function runObserverRecovery() {
     }
   })().finally(() => {
     observerRecoveryInFlight = null;
+    if (!observerRecoveryTrailingRequested) return;
+    observerRecoveryTrailingRequested = false;
+    void runObserverRecovery();
   });
   return observerRecoveryInFlight;
 }
 var observerRecoveryRearm = createObserverRecoveryRearm(() => {
+  emitObserverRecoveryDiagnostic(
+    "REARM_CALLBACK_ENTERED",
+    bridgeSessionEpoch,
+    `bridge-session:${bridgeSessionEpoch}`
+  );
   void runObserverRecovery();
 });
 function observationFor(tabId) {
@@ -15322,6 +15386,11 @@ async function runBridgeLoop() {
       });
       if (!hello.ok) throw new Error("BRIDGE_HELLO_REJECTED");
       bridgeSessionEpoch += 1;
+      emitObserverRecoveryDiagnostic(
+        "BRIDGE_EPOCH_ACCEPTED",
+        bridgeSessionEpoch,
+        `bridge-session:${bridgeSessionEpoch}`
+      );
       observerRecoveryRearm.bridgeSessionEstablished(bridgeSessionEpoch);
       await publishCarrierAttentions();
       let lastHeartbeatAt = Date.now();

package/extension/background.ts

@@ -473,6 +473,21 @@ async function emitStructuredLog(entry: BrowserStructuredLogEntry) {
 	}).catch(() => undefined);
 }
 
+function emitObserverRecoveryDiagnostic(
+	status: string,
+	attemptNo: number,
+	operationRef: string,
+): void {
+	void emitStructuredLog({
+		level: "INFO",
+		component: "browser-observer-recovery",
+		operation: "observer.recovery",
+		status,
+		attemptNo,
+		operationRef,
+	}).catch(() => undefined);
+}
+
 async function invokeObserverApplication(
 	operation: string,
 	input: Record<string, unknown>,
@@ -599,15 +614,29 @@ const collaborationCarrier = createCollaborationCarrierApplication({
 	},
 	agent: {
 		async listPendingMessages(limit) {
-			return (await invokeObserverApplication("collaboration.listPending", {
-				limit,
-			})) as Awaited<
-				ReturnType<
-					Parameters<
-						typeof createCollaborationCarrierApplication
-					>[0]["agent"]["listPendingMessages"]
-				>
-			>;
+			const recoveryTriggerRef = `bridge-session:${bridgeSessionEpoch}`;
+			emitObserverRecoveryDiagnostic(
+				"COLLABORATION_LIST_PENDING_BEGIN",
+				observerRecoveryAttemptNo,
+				recoveryTriggerRef,
+			);
+			try {
+				return (await invokeObserverApplication("collaboration.listPending", {
+					limit,
+				})) as Awaited<
+					ReturnType<
+						Parameters<
+							typeof createCollaborationCarrierApplication
+						>[0]["agent"]["listPendingMessages"]
+					>
+				>;
+			} finally {
+				emitObserverRecoveryDiagnostic(
+					"COLLABORATION_LIST_PENDING_SETTLED",
+					observerRecoveryAttemptNo,
+					recoveryTriggerRef,
+				);
+			}
 		},
 		async getPendingMessage(messageRef) {
 			return (await invokeObserverApplication("collaboration.getPending", {
@@ -721,7 +750,9 @@ async function persistSystemObserverState(
 }
 
 let observerRecoveryInFlight: Promise<void> | null = null;
+let observerRecoveryTrailingRequested = false;
 let observerRecoveryRetryCount = 0;
+let observerRecoveryAttemptNo = 0;
 let nextRecoverySuppressions: readonly CarrierContinuationDenial[] = [];
 function suppressNextObserverRecovery(
 	denials: readonly CarrierContinuationDenial[],
@@ -729,12 +760,38 @@ function suppressNextObserverRecovery(
 	nextRecoverySuppressions = denials;
 }
 function runObserverRecovery() {
-	if (observerRecoveryInFlight) return observerRecoveryInFlight;
+	const recoveryTriggerRef = `bridge-session:${bridgeSessionEpoch}`;
+	if (observerRecoveryInFlight) {
+		observerRecoveryTrailingRequested = true;
+		emitObserverRecoveryDiagnostic(
+			"REUSED_IN_FLIGHT",
+			observerRecoveryAttemptNo,
+			recoveryTriggerRef,
+		);
+		return observerRecoveryInFlight;
+	}
+	observerRecoveryAttemptNo += 1;
+	const recoveryAttemptNo = observerRecoveryAttemptNo;
+	emitObserverRecoveryDiagnostic(
+		"STARTED",
+		recoveryAttemptNo,
+		recoveryTriggerRef,
+	);
 	const suppressedContinuations = nextRecoverySuppressions;
 	nextRecoverySuppressions = [];
 	observerRecoveryInFlight = (async () => {
 		let recoveryNeedsRetry = false;
+		emitObserverRecoveryDiagnostic(
+			"COLLABORATION_RECOVERY_BEGIN",
+			recoveryAttemptNo,
+			recoveryTriggerRef,
+		);
 		await collaborationCarrier.recoverPending(50).catch(() => undefined);
+		emitObserverRecoveryDiagnostic(
+			"COLLABORATION_RECOVERY_SETTLED",
+			recoveryAttemptNo,
+			recoveryTriggerRef,
+		);
 		const signalBatch = await invokeObserverApplication(
 			"execution.listSignals",
 			{ limit: 50 },
@@ -846,7 +903,11 @@ function runObserverRecovery() {
 		// remains the no-blind-replay authority for APPLIED/NOT_APPLIED/UNKNOWN.
 		if (recoveryNeedsRetry && observerRecoveryRetryCount < 6) {
 			observerRecoveryRetryCount += 1;
-			setTimeout(() => void runObserverRecovery(), 2_000);
+			const retryScheduledFromAttemptNo = recoveryAttemptNo;
+			setTimeout(() => {
+				if (observerRecoveryAttemptNo !== retryScheduledFromAttemptNo) return;
+				void runObserverRecovery();
+			}, 2_000);
 		} else if (!recoveryNeedsRetry) {
 			observerRecoveryRetryCount = 0;
 		}
@@ -864,11 +925,19 @@ function runObserverRecovery() {
 		}
 	})().finally(() => {
 		observerRecoveryInFlight = null;
+		if (!observerRecoveryTrailingRequested) return;
+		observerRecoveryTrailingRequested = false;
+		void runObserverRecovery();
 	});
 	return observerRecoveryInFlight;
 }
 
 const observerRecoveryRearm = createObserverRecoveryRearm(() => {
+	emitObserverRecoveryDiagnostic(
+		"REARM_CALLBACK_ENTERED",
+		bridgeSessionEpoch,
+		`bridge-session:${bridgeSessionEpoch}`,
+	);
 	void runObserverRecovery();
 });
 
@@ -1650,6 +1719,11 @@ async function runBridgeLoop() {
 			});
 			if (!hello.ok) throw new Error("BRIDGE_HELLO_REJECTED");
 			bridgeSessionEpoch += 1;
+			emitObserverRecoveryDiagnostic(
+				"BRIDGE_EPOCH_ACCEPTED",
+				bridgeSessionEpoch,
+				`bridge-session:${bridgeSessionEpoch}`,
+			);
 			observerRecoveryRearm.bridgeSessionEstablished(bridgeSessionEpoch);
 			await publishCarrierAttentions();
 			// Hello only establishes a session. The first inner-loop action must be

package/manifest.json

@@ -1,7 +1,7 @@
 {
 	"manifest_version": 3,
 	"name": "ProFlow Execution Browser",
-	"version": "0.1.49",
+	"version": "0.1.50",
 	"permissions": [
 		"tabs",
 		"storage"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tomflow/proflow-execution-browser-extension",
-  "version": "0.1.49",
+  "version": "0.1.50",
   "description": "Execution-owned MV3 Browser executor, evidence provider and browser application surface.",
   "type": "module",
   "publishConfig": {
@@ -27,8 +27,8 @@
     "SETUP.md"
   ],
   "dependencies": {
-    "@tomflow/proflow-module-contract": "^0.1.13",
-    "@tomflow/proflow-execution-contracts": "^0.1.10"
+    "@tomflow/proflow-execution-contracts": "^0.1.10",
+    "@tomflow/proflow-module-contract": "^0.1.13"
   },
   "devDependencies": {
     "@tomflow/proflow-deployment-conformance": "^0.1.13",

package/proflow.module.json

@@ -3,7 +3,7 @@
 	"contractVersion": "1.0.0",
 	"moduleRef": "execution-browser-extension",
 	"packageName": "@tomflow/proflow-execution-browser-extension",
-	"moduleVersion": "0.1.49",
+	"moduleVersion": "0.1.50",
 	"kind": "browser-extension",
 	"templateVersion": "1.0.0",
 	"platformCompatibility": ">=1.0.0 <2.0.0",