@jsenv/navi 0.29.309 → 0.29.310

package/dist/dev/jsenv_navi.js

@@ -2866,7 +2866,17 @@ const updateActions = ({
    * - willRunSet: actions that will be run
    * - willPromoteSet: prerun actions that become run-requested
    * - stays*Set: actions that remain in their current state
+   *
+   * A proxy in any of the sets stands for the action it currently targets,
+   * exactly as proxy.run()/rerun()/reset() do: what enters activationWeakSet
+   * is always the target. A proxy kept in there would retarget on its own —
+   * to a never-run child once its params go falsy — and getActivationInfo
+   * would then meet an IDLE member on every later update of the document.
    */
+  prerunSet = resolveActionProxies(prerunSet);
+  runSet = resolveActionProxies(runSet);
+  rerunSet = resolveActionProxies(rerunSet);
+  resetSet = resolveActionProxies(resetSet);
 
   const { runningSet, settledSet } = getActivationInfo();
 
@@ -3088,13 +3098,6 @@ ${lines.join("\n")}`);
   // Step 5: Execute preruns and runs
   {
     const onActionToRunOrPrerun = (actionToPrerunOrRun, isPrerun) => {
-      if (actionToPrerunOrRun.isProxy) {
-        // maybe remove this check once the API is stable because
-        // nothing in the API should allow this to happen
-        throw new Error(
-          `Proxy should not be reach this point, use the underlying action instead`,
-        );
-      }
       const actionSpecificSignal = abortSignalMap.get(actionToPrerunOrRun);
       const effectiveSignal = actionSpecificSignal || abortSignal;
 
@@ -3222,6 +3225,14 @@ ${lines.join("\n")}`);
   };
 };
 
+const resolveActionProxies = (actionSet) => {
+  const resolvedSet = new Set();
+  for (const action of actionSet) {
+    resolvedSet.add(action.isProxy ? action.getCurrentAction() : action);
+  }
+  return resolvedSet;
+};
+
 const NO_PARAMS = { __no_params__: true };
 const mergeActionParams = (currentParams, newParams) => {
   // The order of these two checks is load-bearing. Checking `undefined` first