@broberg/lens-daemon 0.1.28 → 0.1.29

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@broberg/lens-daemon",
-  "version": "0.1.28",
+  "version": "0.1.29",
   "description": "The Lens module of the cardmem daemon (flows, critic, coverage, capture sessions, the /lens/* routes and the cardmem-lens MCP) as a package. The daemon mounts createLensRouter(deps) + createLensMcpRouter(deps). Bun runtime.",
   "type": "module",
   "license": "MIT",

package/src/flow.ts

@@ -1383,6 +1383,20 @@ function subst(str: string, vars: Record<string, unknown>): string {
   return str.replace(/\{\{(\w+)\}\}/g, (_, k) => (k in vars ? String(vars[k]) : `{{${k}}}`));
 }
 
+/** F009.134 — a loop row reaches a selector step's selector, frame and `at` too, not
+ *  only its value. cardmem's flow 62ca6136 sent `[data-go="{{s}}"]` to Playwright
+ *  literally on all 9 iterations. Other step kinds are returned unchanged. */
+function bindSelectorVars(step: FlowStep, vars: Record<string, unknown>): FlowStep {
+  if (step.type !== 'fillSelector' && step.type !== 'clickSelector') return step;
+  const frame = step.frame == null ? step.frame : Array.isArray(step.frame) ? step.frame.map((f) => subst(f, vars)) : subst(step.frame, vars);
+  return {
+    ...step,
+    selector: subst(step.selector, vars),
+    ...(frame != null ? { frame } : {}),
+    ...(step.type === 'clickSelector' && step.at ? { at: subst(step.at, vars) } : {}),
+  } as FlowStep;
+}
+
 const sel = (testid: string) => `[data-testid="${testid}"]`;
 const stepTimeout = (s: { timeout?: number }) => s.timeout ?? 8000;
 
@@ -2340,11 +2354,13 @@ async function execStep(page: Page, step: FlowStep, base: string, vars: Record<s
       }
       return shot;
     }
-    case 'fillSelector':
-      await resolveLocator(page, step.selector, step.frame)
+    case 'fillSelector': {
+      const b = bindSelectorVars(step, vars) as typeof step;
+      await resolveLocator(page, b.selector, b.frame)
         .first()
         .fill(subst(step.value, vars), { timeout: stepTimeout(step) });
       return null;
+    }
     case 'waitForBuild': {
       // F255.7 — polled from HERE, not through the page: a service worker will happily
       // serve a cached build and a page-side fetch would report the new marker on a page
@@ -2363,10 +2379,11 @@ async function execStep(page: Page, step: FlowStep, base: string, vars: Record<s
       };
     }
     case 'clickSelector': {
-      const loc = resolveLocator(page, step.selector, step.frame).first();
+      const b = bindSelectorVars(step, vars) as typeof step;
+      const loc = resolveLocator(page, b.selector, b.frame).first();
       await loc.click({
         timeout: stepTimeout(step),
-        ...(step.at ? { position: await positionFor(loc, step.at) } : {}),
+        ...(b.at ? { position: await positionFor(loc, b.at) } : {}),
       });
       return null;
     }
@@ -2839,7 +2856,7 @@ export async function runFlow(flow: Flow, env: AuthEnv, opts: RunFlowOptions = {
       // ── leaf step ──
       const i = idx;
       idx += 1;
-      const label = step.label ?? describe(step);
+      const label = step.label ?? describe(bindSelectorVars(step, vars));
       // F009.122 — which browser this step runs in. `sp` is the page every line
       // below acts on; the loop's own `page` is the default browser.
       const browserName = (step as { browser?: string }).browser ?? 'main';