@weavetab/mcp 2.5.0 → 2.5.1

package/dist/tools/select.js

@@ -7,8 +7,9 @@ import { buildActionMap, safeSelector } from "../cdp/walker.js";
 async function getSelector(session, options, config) {
     if (options.selector)
         return options.selector;
-    if (options.ref || options.id) {
-        const ref = (options.ref || options.id);
+    const targetRef = options.ref || options.id || options.target;
+    if (targetRef) {
+        const ref = targetRef;
         if (ref.startsWith("w:")) {
             return `[data-wt-id="${ref.slice(2)}"]`;
         }
@@ -43,60 +44,60 @@ async function getSelector(session, options, config) {
             throw new Error("Could not resolve node");
         const { result } = await session.Runtime.callFunctionOn({
             objectId: object.objectId,
-            functionDeclaration: `function() {
-        if (this.id) return '#' + CSS.escape(this.id);
-        if (this.name) return 'select[name="' + CSS.escape(this.name) + '"]';
-        var idx = 0, el = this;
-        while (el.previousElementSibling) { idx++; el = el.previousElementSibling; }
-        var parent = this.parentElement;
-        if (!parent) return 'select:last-of-type';
-        var tag = this.tagName.toLowerCase();
-        return tag + ':nth-child(' + (idx + 1) + ')';
+            functionDeclaration: `function() {
+        if (this.id) return '#' + CSS.escape(this.id);
+        if (this.name) return 'select[name="' + CSS.escape(this.name) + '"]';
+        var idx = 0, el = this;
+        while (el.previousElementSibling) { idx++; el = el.previousElementSibling; }
+        var parent = this.parentElement;
+        if (!parent) return 'select:last-of-type';
+        var tag = this.tagName.toLowerCase();
+        return tag + ':nth-child(' + (idx + 1) + ')';
       }()`,
             returnByValue: true
         });
         return result.value;
     }
-    throw new Error("browser_select requires either selector, nodeId, ref, or id");
+    throw new Error("browser_select requires either selector, nodeId, ref, target, or id");
 }
 function escapeSelector(sel) {
     return JSON.stringify(sel);
 }
 export async function browserSelect(session, options, _config) {
-    if (!options.selector && !options.nodeId && !options.ref && !options.id) {
-        throw new Error("browser_select requires either selector, nodeId, ref, or id");
+    if (!options.selector && !options.nodeId && !options.ref && !options.id && !options.target) {
+        throw new Error("browser_select requires either selector, nodeId, ref, target, or id");
     }
     const selector = await getSelector(session, options, _config);
     const value = options.value;
     const getSelectState = async () => {
         const { result } = await session.Runtime.evaluate({
-            expression: `(function() {
-        try {
-          var el = document.querySelector(${escapeSelector(selector)});
-          if (!el) return JSON.stringify({ exists: "false" });
-          if (el.tagName === "SELECT") {
-            var idx = el.selectedIndex;
-            var opt = idx >= 0 ? el.options[idx] : null;
-            return JSON.stringify({
-              exists: "true",
-              isNative: "true",
-              selectedIndex: String(idx),
-              value: el.value,
-              selectedText: opt ? opt.text : ""
-            });
-          }
-          // Custom UI Dropdown / Combobox / Popover
-          var text = (el.innerText || el.textContent || "").trim();
-          var val = el.getAttribute("data-value") || el.getAttribute("aria-valuenow") || text;
-          return JSON.stringify({
-            exists: "true",
-            isNative: "false",
-            value: val,
-            selectedText: text
-          });
-        } catch(e) {
-          return JSON.stringify({ exists: "false", error: e.message });
-        }
+            expression: `(function() {
+        try {
+          var el = document.querySelector(${escapeSelector(selector)});
+          if (!el) return JSON.stringify({ exists: "false" });
+          if (el.tagName === "SELECT") {
+            var idx = el.selectedIndex;
+            var opt = idx >= 0 ? el.options[idx] : null;
+            return JSON.stringify({
+              exists: "true",
+              isNative: "true",
+              selectedIndex: String(idx),
+              value: el.value,
+              selectedText: opt ? opt.text : ""
+            });
+          }
+          // Custom UI Dropdown / Combobox / Popover
+          var text = (el.innerText || el.textContent || "").trim();
+          var val = el.getAttribute("data-value") || el.getAttribute("aria-valuenow") || text;
+          return JSON.stringify({
+            exists: "true",
+            isNative: "false",
+            value: val,
+            selectedText: text
+          });
+        } catch(e) {
+          return JSON.stringify({ exists: "false", error: e.message });
+        }
       })()`,
             returnByValue: true
         });
@@ -104,88 +105,88 @@ export async function browserSelect(session, options, _config) {
     };
     const action = async () => {
         const { result } = await session.Runtime.evaluate({
-            expression: `(function(selector, value) {
-        try {
-          const el = document.querySelector(selector);
-          if (!el) return 'Element not found';
-
-          // ── NATIVE <select> ELEMENT ──────────────────────────────────────
-          if (el.tagName === 'SELECT') {
-            let foundIndex = -1;
-            for (let i = 0; i < el.options.length; i++) {
-              const opt = el.options[i];
-              if (opt.value === value || opt.text === value || opt.text.trim() === value.trim()) {
-                foundIndex = i;
-                break;
-              }
-            }
-            if (foundIndex === -1) return 'Option not found in <select>: ' + value;
-
-            el.selectedIndex = foundIndex;
-
-            const nativeSetter = Object.getOwnPropertyDescriptor(window.HTMLSelectElement.prototype, 'value')?.set;
-            if (nativeSetter) nativeSetter.call(el, el.options[foundIndex].value);
-
-            el.dispatchEvent(new Event('input', { bubbles: true }));
-            el.dispatchEvent(new Event('change', { bubbles: true }));
-            el.options[foundIndex].dispatchEvent(new MouseEvent('click', { bubbles: true }));
-
-            return 'selected:' + el.selectedIndex;
-          }
-
-          // ── CUSTOM ARIA / REACT / RADIX / SHADCN DROPDOWN ────────────────
-          // 1. Open popover if closed
-          const isExpanded = el.getAttribute('aria-expanded') === 'true';
-          if (!isExpanded) {
-            el.click();
-            el.dispatchEvent(new MouseEvent('mousedown', { bubbles: true }));
-            el.dispatchEvent(new MouseEvent('mouseup', { bubbles: true }));
-          }
-
-          // 2. Scan options in open popover / listbox
-          const targetText = value.toLowerCase().trim();
-          const optionSelectors = [
-            '[role="option"]',
-            '[role="menuitem"]',
-            '[role="treeitem"]',
-            '[data-radix-collection-item]',
-            'li',
-            '.option',
-            '.select-option',
-            '[data-value]'
-          ];
-
-          let candidates = Array.from(document.querySelectorAll(optionSelectors.join(',')));
-          // Fallback to all visible clickable items inside active popovers/listboxes
-          if (candidates.length === 0) {
-            const containers = document.querySelectorAll('[role="listbox"], [role="menu"], [data-radix-popper-content-wrapper], .select-popover, .dropdown-menu');
-            containers.forEach(c => {
-              candidates.push(...Array.from(c.querySelectorAll('*')));
-            });
-          }
-
-          let matchedEl = null;
-          for (const cand of candidates) {
-            const text = (cand.innerText || cand.textContent || '').toLowerCase().trim();
-            const valAttr = (cand.getAttribute('data-value') || cand.getAttribute('value') || '').toLowerCase().trim();
-            if (text === targetText || valAttr === targetText || text.includes(targetText)) {
-              matchedEl = cand;
-              break;
-            }
-          }
-
-          if (!matchedEl) {
-            return 'Custom option not found: ' + value;
-          }
-
-          matchedEl.click();
-          matchedEl.dispatchEvent(new MouseEvent('mousedown', { bubbles: true }));
-          matchedEl.dispatchEvent(new MouseEvent('mouseup', { bubbles: true }));
-
-          return 'selected:custom';
-        } catch (e) {
-          return e.message || 'Unknown error';
-        }
+            expression: `(function(selector, value) {
+        try {
+          const el = document.querySelector(selector);
+          if (!el) return 'Element not found';
+
+          // ── NATIVE <select> ELEMENT ──────────────────────────────────────
+          if (el.tagName === 'SELECT') {
+            let foundIndex = -1;
+            for (let i = 0; i < el.options.length; i++) {
+              const opt = el.options[i];
+              if (opt.value === value || opt.text === value || opt.text.trim() === value.trim()) {
+                foundIndex = i;
+                break;
+              }
+            }
+            if (foundIndex === -1) return 'Option not found in <select>: ' + value;
+
+            el.selectedIndex = foundIndex;
+
+            const nativeSetter = Object.getOwnPropertyDescriptor(window.HTMLSelectElement.prototype, 'value')?.set;
+            if (nativeSetter) nativeSetter.call(el, el.options[foundIndex].value);
+
+            el.dispatchEvent(new Event('input', { bubbles: true }));
+            el.dispatchEvent(new Event('change', { bubbles: true }));
+            el.options[foundIndex].dispatchEvent(new MouseEvent('click', { bubbles: true }));
+
+            return 'selected:' + el.selectedIndex;
+          }
+
+          // ── CUSTOM ARIA / REACT / RADIX / SHADCN DROPDOWN ────────────────
+          // 1. Open popover if closed
+          const isExpanded = el.getAttribute('aria-expanded') === 'true';
+          if (!isExpanded) {
+            el.click();
+            el.dispatchEvent(new MouseEvent('mousedown', { bubbles: true }));
+            el.dispatchEvent(new MouseEvent('mouseup', { bubbles: true }));
+          }
+
+          // 2. Scan options in open popover / listbox
+          const targetText = value.toLowerCase().trim();
+          const optionSelectors = [
+            '[role="option"]',
+            '[role="menuitem"]',
+            '[role="treeitem"]',
+            '[data-radix-collection-item]',
+            'li',
+            '.option',
+            '.select-option',
+            '[data-value]'
+          ];
+
+          let candidates = Array.from(document.querySelectorAll(optionSelectors.join(',')));
+          // Fallback to all visible clickable items inside active popovers/listboxes
+          if (candidates.length === 0) {
+            const containers = document.querySelectorAll('[role="listbox"], [role="menu"], [data-radix-popper-content-wrapper], .select-popover, .dropdown-menu');
+            containers.forEach(c => {
+              candidates.push(...Array.from(c.querySelectorAll('*')));
+            });
+          }
+
+          let matchedEl = null;
+          for (const cand of candidates) {
+            const text = (cand.innerText || cand.textContent || '').toLowerCase().trim();
+            const valAttr = (cand.getAttribute('data-value') || cand.getAttribute('value') || '').toLowerCase().trim();
+            if (text === targetText || valAttr === targetText || text.includes(targetText)) {
+              matchedEl = cand;
+              break;
+            }
+          }
+
+          if (!matchedEl) {
+            return 'Custom option not found: ' + value;
+          }
+
+          matchedEl.click();
+          matchedEl.dispatchEvent(new MouseEvent('mousedown', { bubbles: true }));
+          matchedEl.dispatchEvent(new MouseEvent('mouseup', { bubbles: true }));
+
+          return 'selected:custom';
+        } catch (e) {
+          return e.message || 'Unknown error';
+        }
       })(${escapeSelector(selector)}, ${escapeSelector(value)})`,
             returnByValue: true
         });
@@ -208,7 +209,12 @@ export async function browserSelect(session, options, _config) {
             input: { ref: selector, value },
             result: { success: true, stuck: false },
         });
-        return { success: true, message: `Selected value "${value}" on ${beforeState.isNative === "true" ? "native <select>" : "custom UI dropdown"} (${confirm.attempts} attempts, settled ${confirm.settleMs}ms)` };
+        return {
+            success: true,
+            value,
+            selected: value,
+            message: `Selected value "${value}" on ${beforeState.isNative === "true" ? "native <select>" : "custom UI dropdown"} (${confirm.attempts} attempts, settled ${confirm.settleMs}ms)`
+        };
     }
     recordAction({
         timestamp: new Date().toISOString(),
@@ -218,6 +224,7 @@ export async function browserSelect(session, options, _config) {
     });
     return {
         success: false,
+        value,
         message: `Selection did not take effect. Before: value="${beforeState.value}". After: value="${confirm.after.value}".`
     };
 }

package/dist/tools/snapshot.js

@@ -14,12 +14,14 @@ export async function browserSnapshot(session, options, config) {
     if (!filename || !filename.trim()) {
         throw new Error("browser_snapshot requires a filename.");
     }
-    // Strip any absolute path provided by the caller — always save in the managed snapshot dir
-    filename = path.basename(filename);
-    if (!filename.endsWith(".mhtml"))
-        filename += ".mhtml";
-    const snapshotDir = getOutputDir("snapshots", config);
-    const filePath = path.join(snapshotDir, filename);
+    // If caller provided an absolute path, respect it; otherwise store in snapshots directory
+    let filePath = filename;
+    if (!path.isAbsolute(filePath)) {
+        const snapshotDir = getOutputDir("snapshots", config);
+        filePath = path.join(snapshotDir, path.basename(filename));
+    }
+    if (!filePath.endsWith(".mhtml"))
+        filePath += ".mhtml";
     if (action === "capture") {
         // Always hide overlay before capture (default true); set hideOverlay: false to skip.
         const shouldHide = options.hideOverlay !== false;

package/dist/tools/storage.d.ts

@@ -7,12 +7,16 @@ import CDP from "chrome-remote-interface";
 import type { Config } from "../config/loader.js";
 export interface StorageOptions {
     scope: "local" | "session";
-    action: "get" | "set" | "clear";
+    action: "get" | "set" | "remove" | "list" | "clear";
     key?: string;
     value?: string;
 }
 export declare function browserStorage(session: CDP.Client, options: StorageOptions, _config: Config): Promise<{
     success: boolean;
     result?: string | null;
+    data?: {
+        key: string;
+        value: string | null;
+    }[];
     message?: string;
 }>;

package/dist/tools/storage.js

@@ -19,6 +19,22 @@ export async function browserStorage(session, options, _config) {
         });
         return { success: true, message: `Set ${scope}Storage key ${key}` };
     }
+    else if (action === "remove") {
+        if (!key)
+            throw new Error("browser_storage remove requires a 'key'");
+        await session.Runtime.evaluate({
+            expression: `(function(k){ ${scope}Storage.removeItem(k); })(${JSON.stringify(key)})`
+        });
+        return { success: true, message: `Removed ${scope}Storage key ${key}` };
+    }
+    else if (action === "list") {
+        const { result } = await session.Runtime.evaluate({
+            expression: `(function(){ var out = []; var s = ${scope}Storage; for (var i = 0; i < s.length; i++) { var k = s.key(i); out.push({ key: k, value: s.getItem(k) }); } return out; })()`,
+            returnByValue: true
+        });
+        const data = Array.isArray(result.value) ? result.value : [];
+        return { success: true, data, message: `${scope}Storage has ${data.length} key(s)` };
+    }
     else if (action === "clear") {
         await session.Runtime.evaluate({
             expression: `${scope}Storage.clear()`

package/dist/tools/tabs.d.ts

@@ -18,11 +18,13 @@ export declare function WeavetabOpen(url: string, config: Config, reuse?: boolea
     reused?: boolean;
 }>;
 export declare function WeavetabSwitch(tabId: string, config: Config, appName?: string): Promise<{
-    success: true;
-    tab_id: string;
-    title: string;
-    url: string;
-    contextSwitchCostMs: number;
+    success: boolean;
+    tab_id?: string;
+    title?: string;
+    url?: string;
+    contextSwitchCostMs?: number;
+    error_code?: string;
+    message?: string;
 }>;
 export declare function WeavetabList(config: Config, appName?: string): Promise<{
     tabs: TabInfo[];

package/dist/tools/tabs.js

@@ -109,14 +109,31 @@ async function enforceMaxTabs(config) {
     lockedTabIds.delete(victim.id);
     logAction("browser_tabs", victim.id, `auto-closed idle tab: ${victim.title}`);
 }
+function normalizeUrl(u) {
+    try {
+        const parsed = new URL(u);
+        let path = parsed.pathname;
+        if (path.length > 1 && path.endsWith('/')) {
+            path = path.slice(0, -1);
+        }
+        return `${parsed.protocol}//${parsed.host.toLowerCase()}${path}${parsed.search}${parsed.hash}`.replace(/\/$/, "");
+    }
+    catch {
+        return u.trim().replace(/\/+$/, "").toLowerCase();
+    }
+}
 // ─── Tools ────────────────────────────────────────────────────────────────────
 export async function WeavetabOpen(url, config, reuse = true, appName) {
-    // Infer appName from active session if omitted
     let resolvedAppName = appName;
     if (!resolvedAppName) {
-        const lastApp = sessionRegistry.getLastActiveApp();
-        if (lastApp) {
-            resolvedAppName = lastApp;
+        // Only inherit the sticky Electron app when no web-browser session is active
+        // (mirrors inferTargetApp()). Otherwise a plain `open` must target the browser:
+        // Electron targets cannot create new tabs via CDP (Target.createTarget fails).
+        if (!sessionRegistry.isBrowserSessionActive()) {
+            const lastApp = sessionRegistry.getLastActiveApp();
+            if (lastApp) {
+                resolvedAppName = lastApp;
+            }
         }
     }
     const port = resolvePort(config, resolvedAppName);
@@ -124,7 +141,8 @@ export async function WeavetabOpen(url, config, reuse = true, appName) {
     if (reuse && url) {
         try {
             const targets = await getTabList(getConfigAgentId(config), resolvedAppName);
-            const existing = targets.find((t) => t.url === url);
+            const targetNorm = normalizeUrl(url);
+            const existing = targets.find((t) => normalizeUrl(t.url) === targetNorm);
             if (existing) {
                 const client = await connectToTab(existing.id, undefined, undefined, undefined, resolvedAppName);
                 await client.session.Target.activateTarget({ targetId: existing.id });
@@ -141,39 +159,74 @@ export async function WeavetabOpen(url, config, reuse = true, appName) {
     }
     const bctxId = resolvedAppName ? undefined : sessionRegistry.getBrowserContextIdForAgent(getConfigAgentId(config));
     const metaSession = await CDP({ host: "127.0.0.1", port });
-    const { targetId } = await metaSession.Target.createTarget({ url, browserContextId: bctxId });
+    let targetId;
+    try {
+        ({ targetId } = await metaSession.Target.createTarget({ url, browserContextId: bctxId }));
+    }
+    catch (err) {
+        await metaSession.close().catch(() => { });
+        if (resolvedAppName) {
+            throw new Error(`[!] Electron app "${resolvedAppName}" does not support opening new tabs or windows via CDP ` +
+                `(Target.createTarget: ${err?.message || err}). Use browser_navigate with ` +
+                `{ app: "${resolvedAppName}", url: "${url}" } to load this URL in the app's existing window instead.`);
+        }
+        throw err;
+    }
     await metaSession.close();
     recordTabActivity(targetId);
-    await connectToTab(targetId, port, getConfigAgentId(config), bctxId, appName);
-    await evaluateTabSemantics(targetId, appName);
-    logAction("browser_tabs", url, `opened ${targetId}`);
+    const client = await connectToTab(targetId, port, getConfigAgentId(config), bctxId, resolvedAppName);
+    try {
+        await client.session.Target.activateTarget({ targetId });
+    }
+    catch { /* non-fatal */ }
+    sessionRegistry.setActive(targetId, getConfigAgentId(config), resolvedAppName);
+    await evaluateTabSemantics(targetId, resolvedAppName);
+    logAction("browser_tabs", url, `opened and activated ${targetId}`);
     return { success: true, tabId: targetId, url };
 }
 export async function WeavetabSwitch(tabId, config, appName) {
-    const client = await connectToTab(tabId, undefined, undefined, undefined, appName);
-    const startSwitch = Date.now();
-    await client.session.Target.activateTarget({ targetId: tabId });
-    sessionRegistry.setActive(tabId, getConfigAgentId(config), appName);
-    recordTabActivity(tabId);
-    const { frameTree } = await client.session.Page.getFrameTree();
-    let title = tabId;
     try {
-        const titleRes = await client.session.Runtime.evaluate({ expression: 'document.title', returnByValue: true });
-        if (titleRes?.result?.value) {
-            title = titleRes.result.value;
+        const client = await connectToTab(tabId, undefined, undefined, undefined, appName);
+        const startSwitch = Date.now();
+        await client.session.Target.activateTarget({ targetId: tabId });
+        sessionRegistry.setActive(tabId, getConfigAgentId(config), appName);
+        recordTabActivity(tabId);
+        const { frameTree } = await client.session.Page.getFrameTree();
+        let title = client.title || "";
+        try {
+            const titleRes = await client.session.Runtime.evaluate({ expression: 'document.title', returnByValue: true });
+            if (titleRes?.result?.value) {
+                title = titleRes.result.value;
+            }
+        }
+        catch { }
+        const url = frameTree?.frame?.url || client.url || "";
+        if (!title) {
+            title = frameTree?.frame?.name || url || tabId;
         }
+        const cost = Date.now() - startSwitch;
+        logAction("browser_tabs", tabId, `switched to ${title} (cost: ${cost}ms)`);
+        resetLoopCounter("tab_switch");
+        return { success: true, tab_id: tabId, title, url, contextSwitchCostMs: cost };
+    }
+    catch (err) {
+        return {
+            success: false,
+            error_code: "TAB_NOT_FOUND",
+            message: `Failed to switch to tab ${tabId}: ${err?.message || err}`,
+        };
     }
-    catch { }
-    const url = frameTree.frame.url;
-    const cost = Date.now() - startSwitch;
-    logAction("browser_tabs", tabId, `switched to ${title} (cost: ${cost}ms)`);
-    resetLoopCounter("tab_switch");
-    return { success: true, tab_id: tabId, title, url, contextSwitchCostMs: cost };
 }
 export async function WeavetabList(config, appName) {
     const targets = await getTabList(getConfigAgentId(config), appName);
     const activeTargetId = sessionRegistry.getActive(getConfigAgentId(config), appName)?.targetId;
     const tabs = targets
+        .filter((t) => {
+        if (t.url && (t.url.startsWith("chrome-extension://") || t.url.includes("/hud/index.html"))) {
+            return false;
+        }
+        return true;
+    })
         .map((t) => ({
         id: t.id,
         title: t.title,
@@ -186,7 +239,7 @@ export async function WeavetabList(config, appName) {
                 ? "locked"
                 : isIdleTab(t.id, activeTargetId)
                     ? "idle"
-                    : "loading",
+                    : "alive",
     }));
     return { tabs };
 }

package/dist/tools/thought.js

@@ -1,6 +1,6 @@
 // src/tools/thought.ts
 // browser_thought — agent-to-user communication via green thought bubbles
-// Merged: browser_feedback (one-way) + browser_ask (two-way with input)
+// Merged: browser_feedback (one-way) + legacy browser_ask (two-way with input) into browser_thoughts
 import { logAction } from "../audit/logger.js";
 import { sendToExtension } from "../cdp/helpers.js";
 function sleep(ms) {
@@ -38,10 +38,10 @@ export async function browserThought(session, args) {
             await sleep(500);
             try {
                 const { result } = await session.Runtime.evaluate({
-                    expression: `(() => { try {
-            var el = document.getElementById('__wt_ask_bridge');
-            if (!el || el.dataset.status === 'pending') return null;
-            return JSON.stringify({ status: el.dataset.status, result: el.dataset.result });
+                    expression: `(() => { try {
+            var el = document.getElementById('__wt_ask_bridge');
+            if (!el || el.dataset.status === 'pending') return null;
+            return JSON.stringify({ status: el.dataset.status, result: el.dataset.result });
           } catch(e) { return null; } })()`,
                     returnByValue: true,
                 });
@@ -50,9 +50,9 @@ export async function browserThought(session, args) {
                     if (data.status === 'submitted') {
                         logAction("browser_thought", "answered", data.result);
                         await session.Runtime.evaluate({
-                            expression: `(function() {
-                  var el = document.getElementById('__wt_ask_bridge');
-                  if (el) el.remove();
+                            expression: `(function() {
+                  var el = document.getElementById('__wt_ask_bridge');
+                  if (el) el.remove();
                 })()`
                         }).catch(() => { });
                         return { success: true, response: data.result, status: "answered" };