opensteer 0.8.14 → 0.8.15

package/dist/cli/bin.cjs

@@ -7212,11 +7212,11 @@ var targetByElementSchema2 = objectSchema(
 var targetByPersistSchema2 = objectSchema(
   {
     kind: enumSchema(["persist"]),
-    name: stringSchema()
+    persist: stringSchema()
   },
   {
     title: "OpensteerTargetByPersist",
-    required: ["kind", "name"]
+    required: ["kind", "persist"]
   }
 );
 var targetBySelectorSchema2 = objectSchema(
@@ -7263,8 +7263,7 @@ var opensteerResolvedTargetSchema = objectSchema(
 var opensteerActionResultSchema = objectSchema(
   {
     target: opensteerResolvedTargetSchema,
-    point: pointSchema,
-    persisted: stringSchema()
+    point: pointSchema
   },
   {
     title: "OpensteerActionResult",
@@ -11678,8 +11677,8 @@ var OpensteerCloudClient = class {
         entries: entries.map((entry) => ({
           workspace: entry.workspace,
           method: entry.method,
-          descriptionHash: entry.nameHash,
-          ...entry.name === void 0 ? {} : { description: entry.name },
+          persistHash: entry.persistHash,
+          ...entry.persist === void 0 ? {} : { persist: entry.persist },
           path: entry.path,
           ...entry.schemaHash === void 0 ? {} : { schemaHash: entry.schemaHash },
           createdAt: entry.createdAt,
@@ -13504,8 +13503,8 @@ function createDomDescriptorStore(options) {
   }
   return new MemoryDomDescriptorStore(namespace);
 }
-function hashDomDescriptorName(name) {
-  return sha256Hex2(name.trim());
+function hashDomDescriptorPersist(persist) {
+  return sha256Hex2(persist.trim());
 }
 var DOM_DESCRIPTOR_METHOD_ALIASES = Object.freeze({
   click: "dom.click",
@@ -13519,14 +13518,14 @@ function normalizeDomDescriptorMethod(method) {
 }
 function buildDomDescriptorKeys(options) {
   const namespace = normalizeDomDescriptorNamespace(options.namespace);
-  const nameHash = hashDomDescriptorName(options.name);
+  const persistHash = hashDomDescriptorPersist(options.persist);
   const rawMethod = options.method.trim();
   const canonicalMethod = normalizeDomDescriptorMethod(rawMethod);
   const methods = /* @__PURE__ */ new Set([canonicalMethod]);
   if (rawMethod.length > 0) {
     methods.add(rawMethod);
   }
-  return [...methods].map((method) => `dom:${namespace}:${method}:${nameHash}`);
+  return [...methods].map((method) => `dom:${namespace}:${method}:${persistHash}`);
 }
 function buildDomDescriptorKey(options) {
   return buildDomDescriptorKeys(options)[0];
@@ -13542,7 +13541,7 @@ function buildDomDescriptorPayload(input) {
   return {
     kind: "dom-target",
     method: normalizeDomDescriptorMethod(input.method),
-    name: input.name,
+    persist: input.persist,
     path: sanitizeReplayElementPath(input.path),
     ...input.sourceUrl === void 0 ? {} : { sourceUrl: input.sourceUrl }
   };
@@ -13559,8 +13558,7 @@ function parseDomDescriptorRecord(record) {
   if (raw.kind !== "dom-target") {
     return void 0;
   }
-  const name = typeof raw.name === "string" ? raw.name : typeof raw.description === "string" ? raw.description : void 0;
-  if (typeof raw.method !== "string" || name === void 0) {
+  if (typeof raw.method !== "string" || typeof raw.persist !== "string") {
     return void 0;
   }
   if (!raw.path || typeof raw.path !== "object" || Array.isArray(raw.path)) {
@@ -13572,7 +13570,7 @@ function parseDomDescriptorRecord(record) {
   const normalizedPayload = {
     kind: "dom-target",
     method: normalizeDomDescriptorMethod(raw.method),
-    name,
+    persist: raw.persist,
     path: sanitizeReplayElementPath(raw.path),
     ...typeof raw.sourceUrl === "string" ? { sourceUrl: raw.sourceUrl } : {}
   };
@@ -13594,7 +13592,7 @@ var FilesystemDomDescriptorStore = class {
     for (const key of buildDomDescriptorKeys({
       namespace: this.namespace,
       method: input.method,
-      name: input.name
+      persist: input.persist
     })) {
       const record = await this.registry.resolve({ key });
       if (!record) {
@@ -13609,7 +13607,7 @@ var FilesystemDomDescriptorStore = class {
     const key = buildDomDescriptorKey({
       namespace: this.namespace,
       method: input.method,
-      name: input.name
+      persist: input.persist
     });
     const version = buildDomDescriptorVersion(payload);
     const existing = await this.registry.resolve({ key, version });
@@ -13652,7 +13650,7 @@ var MemoryDomDescriptorStore = class {
     for (const key of buildDomDescriptorKeys({
       namespace: this.namespace,
       method: input.method,
-      name: input.name
+      persist: input.persist
     })) {
       const record = this.latestByKey.get(key);
       if (record) {
@@ -13666,7 +13664,7 @@ var MemoryDomDescriptorStore = class {
     const key = buildDomDescriptorKey({
       namespace: this.namespace,
       method: input.method,
-      name: input.name
+      persist: input.persist
     });
     const version = buildDomDescriptorVersion(payload);
     const existing = this.recordsByKey.get(key)?.get(version);
@@ -14598,15 +14596,15 @@ var DefaultDomRuntime = class {
   async resolveDescriptorTarget(session, pageRef, method, target) {
     const descriptor = await this.descriptors.read({
       method,
-      name: target.name
+      persist: target.persist
     });
     if (!descriptor) {
       throw new OpensteerProtocolError(
         "not-found",
-        `no stored DOM descriptor found for "${target.name}"`,
+        `no stored DOM descriptor found for "${target.persist}"`,
         {
           details: {
-            name: target.name,
+            persist: target.persist,
             kind: "dom-descriptor"
           }
         }
@@ -14620,7 +14618,7 @@ var DefaultDomRuntime = class {
       pageRef,
       descriptor.payload.path,
       "descriptor",
-      descriptor.payload.name,
+      descriptor.payload.persist,
       descriptor
     );
   }
@@ -14650,7 +14648,7 @@ var DefaultDomRuntime = class {
     const replayPath = await this.tryBuildPathFromNode(snapshot, node);
     const descriptor = target.persist === void 0 ? void 0 : await writeDescriptor({
       method,
-      name: target.persist,
+      persist: target.persist,
       path: replayPath ?? await this.buildPathForNode(snapshot, node),
       sourceUrl: snapshot.url
     });
@@ -17666,8 +17664,7 @@ function parseExtractionDescriptorRecord(record) {
     return void 0;
   }
   const raw = payload;
-  const persist = typeof raw.persist === "string" ? raw.persist : raw.description;
-  if (raw.kind !== "dom-extraction" || typeof persist !== "string") {
+  if (raw.kind !== "dom-extraction" || typeof raw.persist !== "string") {
     return void 0;
   }
   const root = normalizePersistedExtractionNode(raw.root, "root");
@@ -17682,7 +17679,7 @@ function parseExtractionDescriptorRecord(record) {
     updatedAt: record.updatedAt,
     payload: {
       kind: "dom-extraction",
-      persist,
+      persist: raw.persist,
       root,
       ...typeof raw.schemaHash === "string" ? { schemaHash: raw.schemaHash } : {},
       ...typeof raw.sourceUrl === "string" ? { sourceUrl: raw.sourceUrl } : {}
@@ -18937,7 +18934,7 @@ function ensureSparseCountersForAllRecords(counterRecords) {
     )
   );
 }
-async function clearLiveCounters(engine, pageRef) {
+async function clearOpensteerLiveCounters(engine, pageRef) {
   const frames = await engine.listFrames({ pageRef });
   const failures = [];
   for (const frame of frames) {
@@ -19035,7 +19032,7 @@ async function compileOpensteerSnapshot(options) {
         engine: options.engine,
         pageRef: options.pageRef
       });
-      await clearLiveCounters(options.engine, options.pageRef);
+      await clearOpensteerLiveCounters(options.engine, options.pageRef);
       await assignSparseCountersToLiveDom(options.engine, options.pageRef);
       const pageInfo = await options.engine.getPageInfo({ pageRef: options.pageRef });
       const mainSnapshot = await getMainDocumentSnapshot(options.engine, options.pageRef);
@@ -19072,7 +19069,7 @@ async function compileOpensteerSnapshot(options) {
         counters: [...compiledHtml.counterRecords.values()].map(toPublicCounterRecord)
       };
     } catch (error) {
-      await clearLiveCounters(options.engine, options.pageRef).catch(() => void 0);
+      await clearOpensteerLiveCounters(options.engine, options.pageRef).catch(() => void 0);
       if (attempt < MAX_LIVE_COUNTER_SYNC_ATTEMPTS && isLiveCounterSyncError(error)) {
         lastCounterSyncError = error;
         continue;
@@ -22436,6 +22433,7 @@ var OpensteerSessionRuntime = class {
             });
             boundaryDiagnostics = takeActionBoundaryDiagnostics(timeout.signal);
             timeout.throwIfAborted();
+            await this.invalidateLiveSnapshotCounters([pageRef, output2.pageRef], timeout);
             this.pageRef = output2.pageRef;
             const artifacts2 = await this.persistComputerArtifacts(output2, timeout);
             return {
@@ -22625,7 +22623,7 @@ var OpensteerSessionRuntime = class {
           mutationCaptureDiagnostics = diagnostics;
         }
       );
-      const output = toOpensteerActionResult(executed.result, preparedTarget.persisted);
+      const output = toOpensteerActionResult(executed.result);
       const actionEvents = "events" in executed.result ? executed.result.events : void 0;
       await this.appendTrace({
         operation,
@@ -22636,7 +22634,6 @@ var OpensteerSessionRuntime = class {
         data: {
           target: output.target,
           ...output.point === void 0 ? {} : { point: output.point },
-          ...output.persisted === void 0 ? {} : { persisted: output.persisted },
           ...boundaryDiagnostics === void 0 ? {} : { settle: boundaryDiagnostics },
           ...buildMutationCaptureTraceData(mutationCaptureDiagnostics)
         },
@@ -22700,7 +22697,7 @@ var OpensteerSessionRuntime = class {
       await timeout.runStep(
         () => this.requireDom().writeDescriptor({
           method,
-          name: persist,
+          persist,
           path: stablePath2,
           sourceUrl: resolved2.snapshot.url
         })
@@ -22708,9 +22705,8 @@ var OpensteerSessionRuntime = class {
       return {
         target: {
           kind: "descriptor",
-          name: persist
-        },
-        persisted: persist
+          persist
+        }
       };
     }
     const resolved = await timeout.runStep(
@@ -22733,7 +22729,7 @@ var OpensteerSessionRuntime = class {
     await timeout.runStep(
       () => this.requireDom().writeDescriptor({
         method,
-        name: persist,
+        persist,
         path: stablePath,
         sourceUrl: resolved.snapshot.url
       })
@@ -22741,9 +22737,8 @@ var OpensteerSessionRuntime = class {
     return {
       target: {
         kind: "descriptor",
-        name: persist
-      },
-      persisted: persist
+        persist
+      }
     };
   }
   async queryLiveNetwork(input, timeout, options = {}) {
@@ -22799,6 +22794,18 @@ var OpensteerSessionRuntime = class {
     const byRequestId = new Map(withBodies.map((record) => [record.record.requestId, record]));
     return limited.map((record) => byRequestId.get(record.record.requestId) ?? record);
   }
+  async invalidateLiveSnapshotCounters(pageRefs, timeout) {
+    const engine = this.requireEngine();
+    for (const pageRef of new Set(pageRefs)) {
+      try {
+        await timeout.runStep(() => clearOpensteerLiveCounters(engine, pageRef));
+      } catch (error) {
+        if (!isIgnorableRuntimeBindingError(error)) {
+          throw error;
+        }
+      }
+    }
+  }
   async captureScriptsInternal(pageRef, input, timeout) {
     const root = await this.ensureRoot();
     const evaluated = await timeout.runStep(
@@ -23924,7 +23931,7 @@ var OpensteerSessionRuntime = class {
     if (target.kind === "persist") {
       return {
         kind: "descriptor",
-        name: target.name
+        persist: target.persist
       };
     }
     if (target.kind === "selector") {
@@ -25641,7 +25648,7 @@ function normalizeNamespace2(value) {
   const normalized = String(value ?? "default").trim();
   return normalized.length === 0 ? "default" : normalized;
 }
-function toOpensteerActionResult(result, persisted) {
+function toOpensteerActionResult(result) {
   return {
     target: toOpensteerResolvedTarget2(result.resolved),
     ...result.point === void 0 ? {} : {
@@ -25649,8 +25656,7 @@ function toOpensteerActionResult(result, persisted) {
         x: result.point.x,
         y: result.point.y
       }
-    },
-    ...persisted === void 0 ? {} : { persisted }
+    }
   };
 }
 function toOpensteerResolvedTarget2(target) {
@@ -28875,11 +28881,11 @@ Navigation:
 
 DOM:
   snapshot [action|extraction]
-  click <element> [--button left|middle|right] [--persist <name>] [--capture-network <label>]
-  hover <element> [--persist <name>] [--capture-network <label>]
-  input <element> <text> [--press-enter] [--persist <name>] [--capture-network <label>]
-  scroll <direction> <amount> [--element <n>] [--persist <name>] [--capture-network <label>]
-  extract <schema> [--persist <name>]
+  click <element> [--button left|middle|right] [--persist <key>] [--capture-network <label>]
+  hover <element> [--persist <key>] [--capture-network <label>]
+  input <element> <text> [--press-enter] [--persist <key>] [--capture-network <label>]
+  scroll <direction> <amount> [--element <n>] [--persist <key>] [--capture-network <label>]
+  extract <schema> [--persist <key>]
   evaluate <script>
   init-script <script>
 
@@ -29399,7 +29405,7 @@ async function buildOperationInput(operation, parsed, runtime) {
       const direction = readSingleDirection(parsed.rest[0]);
       const amount = readRequiredPositiveInteger(parsed.rest[1], "scroll requires an amount.");
       const element = readOptionalNumber(parsed.rawOptions, "element");
-      const persist = readPersistName(parsed, "scroll");
+      const persist = readPersistKey(parsed, "scroll");
       const captureNetwork = readSingle(parsed.rawOptions, "capture-network");
       return {
         target: element === void 0 ? {
@@ -29419,7 +29425,7 @@ async function buildOperationInput(operation, parsed, runtime) {
       if (parsed.rest[0] === void 0) {
         throw new Error("extract requires a schema.");
       }
-      const persist = readExtractPersistName(parsed);
+      const persist = readExtractPersistKey(parsed);
       return {
         schema: parseRequiredJsonObjectArgument(joinRest(parsed.rest, 0), "extract schema"),
         ...persist === void 0 ? {} : { persist }
@@ -29638,7 +29644,7 @@ function buildElementTargetInput(parsed, verb) {
     parsed.rest[0],
     `${verb} requires an element number.`
   );
-  const persist = readPersistName(parsed, verb);
+  const persist = readPersistKey(parsed, verb);
   const captureNetwork = readSingle(parsed.rawOptions, "capture-network");
   return {
     target: {
@@ -29847,26 +29853,26 @@ function readKeyModifiers(value) {
   }
   return [...new Set(modifiers)];
 }
-function readPersistName(parsed, verb) {
+function readPersistKey(parsed, verb) {
   const value = readSingle(parsed.rawOptions, "persist");
   if (value === void 0) {
     return void 0;
   }
   if (value === "true" || value === "false") {
-    throw new Error(`${verb} requires "--persist <name>" when using --persist.`);
+    throw new Error(`${verb} requires "--persist <key>" when using --persist.`);
   }
   if (verb === "scroll" && readOptionalNumber(parsed.rawOptions, "element") === void 0) {
-    throw new Error('scroll requires "--element <n>" when using "--persist <name>".');
+    throw new Error('scroll requires "--element <n>" when using "--persist <key>".');
   }
   return value;
 }
-function readExtractPersistName(parsed) {
+function readExtractPersistKey(parsed) {
   const value = readSingle(parsed.rawOptions, "persist");
   if (value === void 0) {
     return void 0;
   }
   if (value === "true" || value === "false") {
-    throw new Error('extract requires "--persist <name>" when using --persist.');
+    throw new Error('extract requires "--persist <key>" when using --persist.');
   }
   return value;
 }
@@ -29963,9 +29969,9 @@ function formatActionOutput(result, input) {
       ...readNumberField(point, "y") === void 0 ? {} : { y: readNumberField(point, "y") }
     };
   }
-  const persisted = readStringField(result, "persisted");
-  if (persisted !== void 0) {
-    output.persisted = persisted;
+  const persist = readStringField(target, "persist");
+  if (persist !== void 0) {
+    output.persist = persist;
   }
   const text = readStringField(input, "text");
   if (text !== void 0) {