opensteer 0.5.2 → 0.5.4

package/README.md

@@ -134,12 +134,6 @@ Install the Opensteer skill pack:
 opensteer skills install
 ```
 
-Fallback (direct upstream `skills` CLI):
-
-```bash
-npx skills add https://github.com/steerlabs/opensteer-skills --skill opensteer
-```
-
 Claude Code marketplace plugin:
 
 ```text

package/dist/{chunk-SXPIGCSD.js → chunk-XIH3WGPY.js}

@@ -2999,8 +2999,56 @@ function applyCleaner(mode, html) {
       return cleanForAction(html);
   }
 }
+function canonicalizeDuplicateNodeIds($) {
+  const occurrencesByNodeId = /* @__PURE__ */ new Map();
+  let order = 0;
+  $("*").each(function() {
+    const element = this;
+    const nodeId = $(element).attr(OS_NODE_ID_ATTR);
+    if (!nodeId) {
+      order += 1;
+      return;
+    }
+    const list = occurrencesByNodeId.get(nodeId) || [];
+    list.push({
+      element,
+      order
+    });
+    occurrencesByNodeId.set(nodeId, list);
+    order += 1;
+  });
+  for (const occurrences of occurrencesByNodeId.values()) {
+    if (occurrences.length <= 1) continue;
+    const canonical = pickCanonicalNodeIdOccurrence($, occurrences);
+    for (const occurrence of occurrences) {
+      if (occurrence.element === canonical.element) continue;
+      $(occurrence.element).removeAttr(OS_NODE_ID_ATTR);
+    }
+  }
+}
+function pickCanonicalNodeIdOccurrence($, occurrences) {
+  let best = occurrences[0];
+  let bestScore = scoreNodeIdOccurrence($, best.element);
+  for (let i = 1; i < occurrences.length; i += 1) {
+    const candidate = occurrences[i];
+    const candidateScore = scoreNodeIdOccurrence($, candidate.element);
+    if (candidateScore > bestScore || candidateScore === bestScore && candidate.order < best.order) {
+      best = candidate;
+      bestScore = candidateScore;
+    }
+  }
+  return best;
+}
+function scoreNodeIdOccurrence($, element) {
+  const el = $(element);
+  const descendantCount = el.find("*").length;
+  const normalizedTextLength = el.text().replace(/\s+/g, " ").trim().length;
+  const attributeCount = Object.keys(el.attr() || {}).length;
+  return descendantCount * 100 + normalizedTextLength * 10 + attributeCount;
+}
 async function assignCounters(page, html, nodePaths, nodeMeta) {
   const $ = cheerio3.load(html, { xmlMode: false });
+  canonicalizeDuplicateNodeIds($);
   const counterIndex = /* @__PURE__ */ new Map();
   let nextCounter = 1;
   const assignedByNodeId = /* @__PURE__ */ new Map();
@@ -3182,44 +3230,63 @@ function stripNodeIds(html) {
   $(`[${OS_NODE_ID_ATTR}]`).removeAttr(OS_NODE_ID_ATTR);
   return $.html();
 }
+function isLiveCounterSyncFailure(error) {
+  if (!(error instanceof Error)) return false;
+  return error.message.startsWith(
+    "Failed to synchronize snapshot counters with the live DOM:"
+  );
+}
 async function prepareSnapshot(page, options = {}) {
   const mode = options.mode ?? "action";
   const withCounters = options.withCounters ?? true;
   const shouldMarkInteractive = options.markInteractive ?? true;
-  if (shouldMarkInteractive) {
-    await markInteractiveElements(page);
-  }
-  const serialized = await serializePageHTML(page);
-  const rawHtml = serialized.html;
-  const processedHtml = rawHtml;
-  const reducedHtml = applyCleaner(mode, processedHtml);
-  let cleanedHtml = reducedHtml;
-  let counterIndex = null;
-  if (withCounters) {
-    const counted = await assignCounters(
-      page,
+  const maxAttempts = withCounters ? 4 : 1;
+  let lastCounterSyncError = null;
+  for (let attempt = 1; attempt <= maxAttempts; attempt += 1) {
+    if (shouldMarkInteractive) {
+      await markInteractiveElements(page);
+    }
+    const serialized = await serializePageHTML(page);
+    const rawHtml = serialized.html;
+    const processedHtml = rawHtml;
+    const reducedHtml = applyCleaner(mode, processedHtml);
+    let cleanedHtml = reducedHtml;
+    let counterIndex = null;
+    if (withCounters) {
+      try {
+        const counted = await assignCounters(
+          page,
+          reducedHtml,
+          serialized.nodePaths,
+          serialized.nodeMeta
+        );
+        cleanedHtml = counted.html;
+        counterIndex = counted.counterIndex;
+      } catch (error) {
+        if (attempt < maxAttempts && isLiveCounterSyncFailure(error)) {
+          lastCounterSyncError = error;
+          continue;
+        }
+        throw error;
+      }
+    } else {
+      cleanedHtml = stripNodeIds(cleanedHtml);
+    }
+    if (mode === "extraction") {
+      const $unwrap = cheerio3.load(cleanedHtml, { xmlMode: false });
+      cleanedHtml = $unwrap("body").html()?.trim() || cleanedHtml;
+    }
+    return {
+      mode,
+      url: page.url(),
+      rawHtml,
+      processedHtml,
       reducedHtml,
-      serialized.nodePaths,
-      serialized.nodeMeta
-    );
-    cleanedHtml = counted.html;
-    counterIndex = counted.counterIndex;
-  } else {
-    cleanedHtml = stripNodeIds(cleanedHtml);
-  }
-  if (mode === "extraction") {
-    const $unwrap = cheerio3.load(cleanedHtml, { xmlMode: false });
-    cleanedHtml = $unwrap("body").html()?.trim() || cleanedHtml;
+      cleanedHtml,
+      counterIndex
+    };
   }
-  return {
-    mode,
-    url: page.url(),
-    rawHtml,
-    processedHtml,
-    reducedHtml,
-    cleanedHtml,
-    counterIndex
-  };
+  throw lastCounterSyncError || new Error("Failed to prepare snapshot after retrying counter sync.");
 }
 
 // src/element-path/errors.ts

package/dist/cli/server.cjs

@@ -4219,8 +4219,56 @@ function applyCleaner(mode, html) {
       return cleanForAction(html);
   }
 }
+function canonicalizeDuplicateNodeIds($) {
+  const occurrencesByNodeId = /* @__PURE__ */ new Map();
+  let order = 0;
+  $("*").each(function() {
+    const element = this;
+    const nodeId = $(element).attr(OS_NODE_ID_ATTR);
+    if (!nodeId) {
+      order += 1;
+      return;
+    }
+    const list = occurrencesByNodeId.get(nodeId) || [];
+    list.push({
+      element,
+      order
+    });
+    occurrencesByNodeId.set(nodeId, list);
+    order += 1;
+  });
+  for (const occurrences of occurrencesByNodeId.values()) {
+    if (occurrences.length <= 1) continue;
+    const canonical = pickCanonicalNodeIdOccurrence($, occurrences);
+    for (const occurrence of occurrences) {
+      if (occurrence.element === canonical.element) continue;
+      $(occurrence.element).removeAttr(OS_NODE_ID_ATTR);
+    }
+  }
+}
+function pickCanonicalNodeIdOccurrence($, occurrences) {
+  let best = occurrences[0];
+  let bestScore = scoreNodeIdOccurrence($, best.element);
+  for (let i = 1; i < occurrences.length; i += 1) {
+    const candidate = occurrences[i];
+    const candidateScore = scoreNodeIdOccurrence($, candidate.element);
+    if (candidateScore > bestScore || candidateScore === bestScore && candidate.order < best.order) {
+      best = candidate;
+      bestScore = candidateScore;
+    }
+  }
+  return best;
+}
+function scoreNodeIdOccurrence($, element) {
+  const el = $(element);
+  const descendantCount = el.find("*").length;
+  const normalizedTextLength = el.text().replace(/\s+/g, " ").trim().length;
+  const attributeCount = Object.keys(el.attr() || {}).length;
+  return descendantCount * 100 + normalizedTextLength * 10 + attributeCount;
+}
 async function assignCounters(page, html, nodePaths, nodeMeta) {
   const $ = cheerio3.load(html, { xmlMode: false });
+  canonicalizeDuplicateNodeIds($);
   const counterIndex = /* @__PURE__ */ new Map();
   let nextCounter = 1;
   const assignedByNodeId = /* @__PURE__ */ new Map();
@@ -4402,44 +4450,63 @@ function stripNodeIds(html) {
   $(`[${OS_NODE_ID_ATTR}]`).removeAttr(OS_NODE_ID_ATTR);
   return $.html();
 }
+function isLiveCounterSyncFailure(error) {
+  if (!(error instanceof Error)) return false;
+  return error.message.startsWith(
+    "Failed to synchronize snapshot counters with the live DOM:"
+  );
+}
 async function prepareSnapshot(page, options = {}) {
   const mode = options.mode ?? "action";
   const withCounters = options.withCounters ?? true;
   const shouldMarkInteractive = options.markInteractive ?? true;
-  if (shouldMarkInteractive) {
-    await markInteractiveElements(page);
-  }
-  const serialized = await serializePageHTML(page);
-  const rawHtml = serialized.html;
-  const processedHtml = rawHtml;
-  const reducedHtml = applyCleaner(mode, processedHtml);
-  let cleanedHtml = reducedHtml;
-  let counterIndex = null;
-  if (withCounters) {
-    const counted = await assignCounters(
-      page,
+  const maxAttempts = withCounters ? 4 : 1;
+  let lastCounterSyncError = null;
+  for (let attempt = 1; attempt <= maxAttempts; attempt += 1) {
+    if (shouldMarkInteractive) {
+      await markInteractiveElements(page);
+    }
+    const serialized = await serializePageHTML(page);
+    const rawHtml = serialized.html;
+    const processedHtml = rawHtml;
+    const reducedHtml = applyCleaner(mode, processedHtml);
+    let cleanedHtml = reducedHtml;
+    let counterIndex = null;
+    if (withCounters) {
+      try {
+        const counted = await assignCounters(
+          page,
+          reducedHtml,
+          serialized.nodePaths,
+          serialized.nodeMeta
+        );
+        cleanedHtml = counted.html;
+        counterIndex = counted.counterIndex;
+      } catch (error) {
+        if (attempt < maxAttempts && isLiveCounterSyncFailure(error)) {
+          lastCounterSyncError = error;
+          continue;
+        }
+        throw error;
+      }
+    } else {
+      cleanedHtml = stripNodeIds(cleanedHtml);
+    }
+    if (mode === "extraction") {
+      const $unwrap = cheerio3.load(cleanedHtml, { xmlMode: false });
+      cleanedHtml = $unwrap("body").html()?.trim() || cleanedHtml;
+    }
+    return {
+      mode,
+      url: page.url(),
+      rawHtml,
+      processedHtml,
       reducedHtml,
-      serialized.nodePaths,
-      serialized.nodeMeta
-    );
-    cleanedHtml = counted.html;
-    counterIndex = counted.counterIndex;
-  } else {
-    cleanedHtml = stripNodeIds(cleanedHtml);
-  }
-  if (mode === "extraction") {
-    const $unwrap = cheerio3.load(cleanedHtml, { xmlMode: false });
-    cleanedHtml = $unwrap("body").html()?.trim() || cleanedHtml;
+      cleanedHtml,
+      counterIndex
+    };
   }
-  return {
-    mode,
-    url: page.url(),
-    rawHtml,
-    processedHtml,
-    reducedHtml,
-    cleanedHtml,
-    counterIndex
-  };
+  throw lastCounterSyncError || new Error("Failed to prepare snapshot after retrying counter sync.");
 }
 
 // src/element-path/errors.ts
@@ -12943,7 +13010,7 @@ async function handleRequest(request, socket) {
         await launchPromise;
       }
       if (url) {
-        await instance.page.goto(url);
+        await instance.goto(url);
       }
       sendResponse(socket, {
         id,

package/dist/cli/server.js

@@ -1,7 +1,7 @@
 import {
   Opensteer,
   normalizeError
-} from "../chunk-SXPIGCSD.js";
+} from "../chunk-XIH3WGPY.js";
 import "../chunk-3H5RRIMZ.js";
 
 // src/cli/server.ts
@@ -425,7 +425,7 @@ async function handleRequest(request, socket) {
         await launchPromise;
       }
       if (url) {
-        await instance.page.goto(url);
+        await instance.goto(url);
       }
       sendResponse(socket, {
         id,

package/dist/index.cjs

@@ -4303,8 +4303,56 @@ function applyCleaner(mode, html) {
       return cleanForAction(html);
   }
 }
+function canonicalizeDuplicateNodeIds($) {
+  const occurrencesByNodeId = /* @__PURE__ */ new Map();
+  let order = 0;
+  $("*").each(function() {
+    const element = this;
+    const nodeId = $(element).attr(OS_NODE_ID_ATTR);
+    if (!nodeId) {
+      order += 1;
+      return;
+    }
+    const list = occurrencesByNodeId.get(nodeId) || [];
+    list.push({
+      element,
+      order
+    });
+    occurrencesByNodeId.set(nodeId, list);
+    order += 1;
+  });
+  for (const occurrences of occurrencesByNodeId.values()) {
+    if (occurrences.length <= 1) continue;
+    const canonical = pickCanonicalNodeIdOccurrence($, occurrences);
+    for (const occurrence of occurrences) {
+      if (occurrence.element === canonical.element) continue;
+      $(occurrence.element).removeAttr(OS_NODE_ID_ATTR);
+    }
+  }
+}
+function pickCanonicalNodeIdOccurrence($, occurrences) {
+  let best = occurrences[0];
+  let bestScore = scoreNodeIdOccurrence($, best.element);
+  for (let i = 1; i < occurrences.length; i += 1) {
+    const candidate = occurrences[i];
+    const candidateScore = scoreNodeIdOccurrence($, candidate.element);
+    if (candidateScore > bestScore || candidateScore === bestScore && candidate.order < best.order) {
+      best = candidate;
+      bestScore = candidateScore;
+    }
+  }
+  return best;
+}
+function scoreNodeIdOccurrence($, element) {
+  const el = $(element);
+  const descendantCount = el.find("*").length;
+  const normalizedTextLength = el.text().replace(/\s+/g, " ").trim().length;
+  const attributeCount = Object.keys(el.attr() || {}).length;
+  return descendantCount * 100 + normalizedTextLength * 10 + attributeCount;
+}
 async function assignCounters(page, html, nodePaths, nodeMeta) {
   const $ = cheerio3.load(html, { xmlMode: false });
+  canonicalizeDuplicateNodeIds($);
   const counterIndex = /* @__PURE__ */ new Map();
   let nextCounter = 1;
   const assignedByNodeId = /* @__PURE__ */ new Map();
@@ -4486,44 +4534,63 @@ function stripNodeIds(html) {
   $(`[${OS_NODE_ID_ATTR}]`).removeAttr(OS_NODE_ID_ATTR);
   return $.html();
 }
+function isLiveCounterSyncFailure(error) {
+  if (!(error instanceof Error)) return false;
+  return error.message.startsWith(
+    "Failed to synchronize snapshot counters with the live DOM:"
+  );
+}
 async function prepareSnapshot(page, options = {}) {
   const mode = options.mode ?? "action";
   const withCounters = options.withCounters ?? true;
   const shouldMarkInteractive = options.markInteractive ?? true;
-  if (shouldMarkInteractive) {
-    await markInteractiveElements(page);
-  }
-  const serialized = await serializePageHTML(page);
-  const rawHtml = serialized.html;
-  const processedHtml = rawHtml;
-  const reducedHtml = applyCleaner(mode, processedHtml);
-  let cleanedHtml = reducedHtml;
-  let counterIndex = null;
-  if (withCounters) {
-    const counted = await assignCounters(
-      page,
+  const maxAttempts = withCounters ? 4 : 1;
+  let lastCounterSyncError = null;
+  for (let attempt = 1; attempt <= maxAttempts; attempt += 1) {
+    if (shouldMarkInteractive) {
+      await markInteractiveElements(page);
+    }
+    const serialized = await serializePageHTML(page);
+    const rawHtml = serialized.html;
+    const processedHtml = rawHtml;
+    const reducedHtml = applyCleaner(mode, processedHtml);
+    let cleanedHtml = reducedHtml;
+    let counterIndex = null;
+    if (withCounters) {
+      try {
+        const counted = await assignCounters(
+          page,
+          reducedHtml,
+          serialized.nodePaths,
+          serialized.nodeMeta
+        );
+        cleanedHtml = counted.html;
+        counterIndex = counted.counterIndex;
+      } catch (error) {
+        if (attempt < maxAttempts && isLiveCounterSyncFailure(error)) {
+          lastCounterSyncError = error;
+          continue;
+        }
+        throw error;
+      }
+    } else {
+      cleanedHtml = stripNodeIds(cleanedHtml);
+    }
+    if (mode === "extraction") {
+      const $unwrap = cheerio3.load(cleanedHtml, { xmlMode: false });
+      cleanedHtml = $unwrap("body").html()?.trim() || cleanedHtml;
+    }
+    return {
+      mode,
+      url: page.url(),
+      rawHtml,
+      processedHtml,
       reducedHtml,
-      serialized.nodePaths,
-      serialized.nodeMeta
-    );
-    cleanedHtml = counted.html;
-    counterIndex = counted.counterIndex;
-  } else {
-    cleanedHtml = stripNodeIds(cleanedHtml);
-  }
-  if (mode === "extraction") {
-    const $unwrap = cheerio3.load(cleanedHtml, { xmlMode: false });
-    cleanedHtml = $unwrap("body").html()?.trim() || cleanedHtml;
+      cleanedHtml,
+      counterIndex
+    };
   }
-  return {
-    mode,
-    url: page.url(),
-    rawHtml,
-    processedHtml,
-    reducedHtml,
-    cleanedHtml,
-    counterIndex
-  };
+  throw lastCounterSyncError || new Error("Failed to prepare snapshot after retrying counter sync.");
 }
 
 // src/element-path/errors.ts

package/dist/index.js

@@ -77,7 +77,7 @@ import {
   switchTab,
   typeText,
   waitForVisualStability
-} from "./chunk-SXPIGCSD.js";
+} from "./chunk-XIH3WGPY.js";
 import {
   createResolveCallback
 } from "./chunk-SPHS6YWD.js";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opensteer",
-  "version": "0.5.2",
+  "version": "0.5.4",
   "description": "Open-source browser automation SDK with robust selectors and deterministic replay.",
   "license": "MIT",
   "type": "module",