@fairfox/polly 0.35.0 → 0.36.0

package/dist/tools/verify/specs/tla/MessageRouter.tla

@@ -40,6 +40,14 @@ ContextType == {"background", "content", "popup", "devtools", "options", "offscr
 PortState == {"connected", "disconnected"}
 MessageStatus == {"pending", "routing", "delivered", "failed", "timeout"}
 
+\* Sentinel value for messages that have not been routed to a concrete
+\* target yet. Once RouteMessage selects a target from msg.targets, that
+\* target is recorded in deliveredTo so step-temporal postcondition
+\* properties can identify which single target was actually mutated.
+\* Without this field a property that quantifies over msg.targets
+\* false-positives whenever routing picks one target out of many.
+NoTarget == "NoTarget"
+
 Message == [
     id: Nat,
     source: ContextType,
@@ -47,7 +55,8 @@ Message == [
     tabId: Nat,
     msgType: STRING,  \* Message type for handler dispatch
     status: MessageStatus,
-    timestamp: Nat
+    timestamp: Nat,
+    deliveredTo: ContextType \cup {NoTarget}
 ]
 
 -----------------------------------------------------------------------------
@@ -97,7 +106,8 @@ SendMessage(source, targetSet, tabId, messageType) ==
                tabId |-> tabId,
                msgType |-> messageType,
                status |-> "pending",
-               timestamp |-> time
+               timestamp |-> time,
+               deliveredTo |-> NoTarget
            ]
        IN /\ messages' = Append(messages, newMsg)
           /\ pendingRequests' = pendingRequests @@
@@ -118,7 +128,8 @@ RouteMessage(msgIndex) ==
              \E target \in msg.targets :
                 /\ IF target \in Contexts /\ ports[target] = "connected"
                    THEN \* Successful delivery to this target
-                        /\ messages' = [messages EXCEPT ![msgIndex].status = "delivered"]
+                        /\ messages' = [messages EXCEPT ![msgIndex].status = "delivered",
+                                                       ![msgIndex].deliveredTo = target]
                         /\ delivered' = delivered \union {msg.id}
                         /\ pendingRequests' = [id \in DOMAIN pendingRequests \ {msg.id} |->
                                                pendingRequests[id]]

package/dist/tools/verify/src/cli.js

@@ -1422,7 +1422,7 @@ var init_tla = __esm(() => {
       const predicateClauses = postconditions.map((pc) => this.tsExpressionToTLA(pc.expression, true)).filter((p) => p && p.length > 0).map((p) => p.replace(/\[ctx\]/g, "[target]"));
       if (predicateClauses.length === 0)
         return;
-      const conjunction = predicateClauses.length === 1 ? predicateClauses[0] : predicateClauses.map((c) => `        /\\ ${c}`).join(`
+      const conjunction = predicateClauses.length === 1 ? predicateClauses[0] : predicateClauses.map((c) => `             /\\ ${c}`).join(`
 `);
       const propertyName = `EnsuresAfter_${actionName}`;
       const messages = postconditions.map((pc) => pc.message).filter((m) => Boolean(m)).join("; ");
@@ -1431,7 +1431,7 @@ var init_tla = __esm(() => {
 ` + `    (messages[m].status = "pending"
 ` + `     /\\ messages'[m].status = "delivered"
 ` + `     /\\ messages[m].msgType = "${messageType}")
-` + `    => \\A target \\in messages[m].targets :
+` + `    => LET target == messages'[m].deliveredTo IN
 ` + `         (target \\in Contexts /\\ ports[target] = "connected")
 ` + `         =>
 ` + (predicateClauses.length === 1 ? `         ${predicateClauses[0]}` : conjunction);
@@ -2018,7 +2018,8 @@ var init_tla = __esm(() => {
       this.line("      /\\ \\E target \\in msg.targets :");
       this.line('            /\\ IF target \\in Contexts /\\ ports[target] = "connected"');
       this.line("               THEN \\* Successful delivery - route AND invoke handler");
-      this.line(`                    /\\ messages' = [messages EXCEPT ![msgIndex].status = "delivered"]`);
+      this.line(`                    /\\ messages' = [messages EXCEPT ![msgIndex].status = "delivered",`);
+      this.line("                                                   ![msgIndex].deliveredTo = target]");
       this.line("                    /\\ delivered' = delivered \\union {msg.id}");
       this.line("                    /\\ pendingRequests' = [id \\in DOMAIN pendingRequests \\ {msg.id} |->");
       this.line("                                           pendingRequests[id]]");
@@ -7716,4 +7717,4 @@ main().catch((error) => {
   process.exit(1);
 });
 
-//# debugId=616D5E99FD251E9364756E2164756E21
+//# debugId=F1333D1589EFA94964756E2164756E21