@runcofactor/cli 0.15.1 → 0.15.2

package/dist/index.js

@@ -827,7 +827,7 @@ async function executeCommand(command, rest) {
     return;
   }
   if (command === "workflow") {
-    const { runWorkflow } = await import("./workflow-4J23FV7K.js");
+    const { runWorkflow } = await import("./workflow-ZGJU4QCV.js");
     await runWorkflow(rest);
     return;
   }

package/dist/skills/optimize-cache/SKILL.md

@@ -175,14 +175,20 @@ Every cached browser action's locator must pass offline verification against
 the trace-reconstructed DOM of the action's before-state:
 
 1. Materialize the substrate for the action (its before-snapshot).
-2. Derive the locator from the substrate (the deterministic builder owns the
+2. Pick the target `ref` from THAT substrate, not from the trace action's
+   `params.selector`. Trace `aria-ref=e...` values are only hints: ref numbers
+   can change when the DOM is reconstructed, so copying the trace selector can
+   point at a different element. Read the substrate around the intended target
+   and choose the ref in the reconstructed tree.
+3. Derive the locator from the substrate (the deterministic builder owns the
    predicate ladder — do not hand-craft selectors when the builder succeeds).
-3. Verify: the strategy must resolve to EXACTLY the recorded target element
+4. Verify: the strategy must resolve to EXACTLY the recorded target element
    (handle identity, count of one) and must not lean on volatile identifier
    tokens.
 
 On failure, in order:
-- Re-check the target ref — did you pick the right element in the substrate?
+- Re-check the target ref — did you pick the right element in the substrate,
+  or accidentally reuse a trace `aria-ref` that now names something else?
 - Try the builder against a corrected ref or with the run's variable values
   (a row may only be distinguishable via a bound variable).
 - If the element is one of several identical widgets (responsive duplicates,

package/dist/{src-2FK3JHGW.js → src-7LAXXTVG.js}

@@ -5625,6 +5625,16 @@ function buildCacheRecipe(opts) {
     agentMacroResolutions: opts.agentMacroResolutions,
     variableValues: opts.variableValues
   });
+  const selectorlessElementActions = actionsJson.map((action, index) => ({ action, stepOrder: ordered[index]?.[0] })).filter(({ action }) => action.requiresAgent);
+  if (selectorlessElementActions.length > 0) {
+    throw new RecipeValidationError(
+      "Recipe contains cached actions without verified selectors",
+      selectorlessElementActions.map(({ action, stepOrder }) => {
+        const orderLabel = stepOrder === void 0 ? "unknown step" : `step ${stepOrder}`;
+        return `${orderLabel}: ${action.actionType} requires a verified selector; call build_locator or mark the step as a gap`;
+      })
+    );
+  }
   const actionIndexByOrder = new Map(ordered.map(([order], i) => [order, i]));
   if (opts.requiredTokens?.length) {
     const present = collectTokens({ playbook: opts.playbookSteps, actions: actionsJson });
@@ -5956,4 +5966,4 @@ export {
   verifySelectorAgainstRef,
   waitForTraceDomHydration
 };
-//# sourceMappingURL=src-2FK3JHGW.js.map
+//# sourceMappingURL=src-7LAXXTVG.js.map