haltija 1.3.3 → 1.3.4

package/apps/desktop/package.json

@@ -1,6 +1,6 @@
 {
   "name": "haltija-desktop",
-  "version": "1.3.3",
+  "version": "1.3.4",
   "private": true,
   "description": "Haltija Desktop - God Mode Browser for AI Agents",
   "homepage": "https://github.com/tonioloewald/haltija",

package/apps/desktop/resources/component.js

@@ -46,7 +46,7 @@
   });
 
   // src/version.ts
-  var VERSION = "1.3.3";
+  var VERSION = "1.3.4";
 
   // src/text-selector.ts
   var TEXT_PSEUDO_RE = /:(?:text-is|has-text|text)\(/;
@@ -598,13 +598,13 @@
   }
   function resolveRefOrSelector(ref, selector) {
     if (ref) {
-      const result2 = refRegistry.resolveWithStatus(ref);
-      if (result2.element) {
+      const result = refRegistry.resolveWithStatus(ref);
+      if (result.element) {
         stats.refsResolved++;
-        return { element: result2.element, targetDesc: `@${ref}` };
+        return { element: result.element, targetDesc: `@${ref}` };
       }
       stats.refsStale++;
-      const errorMsg = result2.status === "never_assigned" ? `Ref @${ref} was never assigned (highest ref is @${refRegistry.getStats().highWaterMark})` : result2.status === "removed_from_dom" ? `Ref @${ref} points to an element that was removed from the DOM (try refreshing /tree)` : `Ref @${ref} is stale - element was garbage collected (try refreshing /tree)`;
+      const errorMsg = result.status === "never_assigned" ? `Ref @${ref} was never assigned (highest ref is @${refRegistry.getStats().highWaterMark})` : result.status === "removed_from_dom" ? `Ref @${ref} points to an element that was removed from the DOM (try refreshing /tree)` : `Ref @${ref} is stale - element was garbage collected (try refreshing /tree)`;
       return { element: null, targetDesc: `@${ref}`, error: errorMsg };
     }
     if (selector) {
@@ -615,13 +615,13 @@
   }
   function resolveRefOrSelectorAll(ref, selector) {
     if (ref) {
-      const result2 = refRegistry.resolveWithStatus(ref);
-      if (result2.element) {
+      const result = refRegistry.resolveWithStatus(ref);
+      if (result.element) {
         stats.refsResolved++;
-        return { elements: [result2.element], targetDesc: `@${ref}` };
+        return { elements: [result.element], targetDesc: `@${ref}` };
       }
       stats.refsStale++;
-      const errorMsg = result2.status === "never_assigned" ? `Ref @${ref} was never assigned (highest ref is @${refRegistry.getStats().highWaterMark})` : result2.status === "removed_from_dom" ? `Ref @${ref} points to an element that was removed from the DOM (try refreshing /tree)` : `Ref @${ref} is stale - element was garbage collected (try refreshing /tree)`;
+      const errorMsg = result.status === "never_assigned" ? `Ref @${ref} was never assigned (highest ref is @${refRegistry.getStats().highWaterMark})` : result.status === "removed_from_dom" ? `Ref @${ref} points to an element that was removed from the DOM (try refreshing /tree)` : `Ref @${ref} is stale - element was garbage collected (try refreshing /tree)`;
       return { elements: [], targetDesc: `@${ref}`, error: errorMsg };
     }
     if (selector) {
@@ -2870,8 +2870,8 @@
           headers: { "Content-Type": "application/json" },
           body: JSON.stringify({ action: "start", window: this.windowId })
         });
-        const result2 = await response.json();
-        if (result2.success) {
+        const result = await response.json();
+        if (result.success) {
           this.isRecording = true;
           this.serverManagedRecording = true;
           this.recordingStartTime = Date.now();
@@ -2883,7 +2883,7 @@
           this.updateRecordingUI(true);
           console.log("[Haltija] Recording started (server-managed)");
         } else {
-          console.error("[Haltija] Failed to start recording:", result2.error);
+          console.error("[Haltija] Failed to start recording:", result.error);
         }
       } catch (err) {
         console.error("[Haltija] Failed to start recording:", err);
@@ -2898,10 +2898,10 @@
           headers: { "Content-Type": "application/json" },
           body: JSON.stringify({ action: "stop", window: this.windowId })
         });
-        const result2 = await response.json();
-        if (result2.success && result2.data) {
-          this.recordingEvents = result2.data.events || [];
-          this.recordingStartTime = result2.data.startTime || this.recordingStartTime;
+        const result = await response.json();
+        if (result.success && result.data) {
+          this.recordingEvents = result.data.events || [];
+          this.recordingStartTime = result.data.startTime || this.recordingStartTime;
           const recordingId = `rec_${this.recordingStartTime}_${Math.random().toString(36).slice(2, 8)}`;
           this.lastRecordingId = recordingId;
           console.log(`[Haltija] Recording stopped (${this.recordingEvents.length} events)`);
@@ -3214,15 +3214,15 @@
           headers: { "Content-Type": "application/json" },
           body: JSON.stringify({ description, agentId })
         });
-        const result2 = await response.json();
-        if (result2.error) {
+        const result = await response.json();
+        if (result.error) {
           if (successMsg) {
-            successMsg.textContent = `⚠️ ${result2.error}`;
+            successMsg.textContent = `⚠️ ${result.error}`;
             successMsg.style.color = "#f59e0b";
           }
         } else {
           if (successMsg) {
-            successMsg.textContent = `✓ Sent to ${agentName}${result2.interrupted ? " (interrupted)" : ""}`;
+            successMsg.textContent = `✓ Sent to ${agentName}${result.interrupted ? " (interrupted)" : ""}`;
             successMsg.style.color = "#22c55e";
           }
           setTimeout(() => this.closeTestModal(), 1500);
@@ -3753,13 +3753,13 @@
         const haltija = window.haltija;
         if (haltija?.openAgentTab) {
           newAgentText.textContent = "Creating...";
-          const result2 = await haltija.openAgentTab();
-          if (result2?.shellId) {
+          const result = await haltija.openAgentTab();
+          if (result?.shellId) {
             await new Promise((r) => setTimeout(r, 300));
             if (type === "selection") {
-              this.sendSelectionToAgent(result2.shellId, result2.name || "agent");
+              this.sendSelectionToAgent(result.shellId, result.name || "agent");
             } else if (type === "recording") {
-              this.sendRecordingToAgent(result2.shellId, result2.name || "agent");
+              this.sendRecordingToAgent(result.shellId, result.name || "agent");
             }
           } else {
             console.error(`${LOG_PREFIX} Failed to create agent tab`);
@@ -3828,8 +3828,8 @@ ${elementSummary}${moreText}`;
             context: `selection from ${window.location.host}`
           })
         });
-        const result2 = await response.json();
-        if (result2.sent) {
+        const result = await response.json();
+        if (result.sent) {
           this.clearSelection();
           console.log(`${LOG_PREFIX} Selection sent to ${agentName}`);
         }
@@ -5344,11 +5344,11 @@ ${elementSummary}${moreText}`;
           this.respond(msg2.id, false, undefined, "Video capture requires the Haltija Desktop app. Run: npx haltija@latest -f");
           return;
         }
-        haltija.startVideoCapture({ maxDuration: payload2.maxDuration }).then((result2) => {
-          if (result2.success) {
-            this.respond(msg2.id, true, { recordingId: result2.recordingId });
+        haltija.startVideoCapture({ maxDuration: payload2.maxDuration }).then((result) => {
+          if (result.success) {
+            this.respond(msg2.id, true, { recordingId: result.recordingId });
           } else {
-            this.respond(msg2.id, false, undefined, result2.error || "Failed to start video");
+            this.respond(msg2.id, false, undefined, result.error || "Failed to start video");
           }
         }).catch((err) => {
           this.respond(msg2.id, false, undefined, `Failed to start video: ${err.message}`);
@@ -5358,11 +5358,11 @@ ${elementSummary}${moreText}`;
           this.respond(msg2.id, false, undefined, "Video capture requires the Haltija Desktop app. Run: npx haltija@latest -f");
           return;
         }
-        haltija.stopVideoCapture().then((result2) => {
-          if (result2.success) {
-            this.respond(msg2.id, true, { path: result2.path, duration: result2.duration, size: result2.size, format: result2.format });
+        haltija.stopVideoCapture().then((result) => {
+          if (result.success) {
+            this.respond(msg2.id, true, { path: result.path, duration: result.duration, size: result.size, format: result.format });
           } else {
-            this.respond(msg2.id, false, undefined, result2.error || "Failed to stop video");
+            this.respond(msg2.id, false, undefined, result.error || "Failed to stop video");
           }
         }).catch((err) => {
           this.respond(msg2.id, false, undefined, `Failed to stop video: ${err.message}`);
@@ -5372,8 +5372,8 @@ ${elementSummary}${moreText}`;
           this.respond(msg2.id, true, { recording: false });
           return;
         }
-        haltija.videoStatus().then((result2) => {
-          this.respond(msg2.id, true, result2);
+        haltija.videoStatus().then((result) => {
+          this.respond(msg2.id, true, result);
         }).catch(() => {
           this.respond(msg2.id, true, { recording: false });
         });
@@ -5390,8 +5390,8 @@ ${elementSummary}${moreText}`;
           this.respond(msg2.id, false, undefined, notAvailable);
           return;
         }
-        haltija.networkWatch(payload2 || {}).then((result2) => {
-          this.respond(msg2.id, result2.success !== false, result2);
+        haltija.networkWatch(payload2 || {}).then((result) => {
+          this.respond(msg2.id, result.success !== false, result);
         }).catch((err) => {
           this.respond(msg2.id, false, undefined, err.message);
         });
@@ -5400,8 +5400,8 @@ ${elementSummary}${moreText}`;
           this.respond(msg2.id, false, undefined, notAvailable);
           return;
         }
-        haltija.networkUnwatch().then((result2) => {
-          this.respond(msg2.id, true, result2);
+        haltija.networkUnwatch().then((result) => {
+          this.respond(msg2.id, true, result);
         }).catch((err) => {
           this.respond(msg2.id, false, undefined, err.message);
         });
@@ -5410,8 +5410,8 @@ ${elementSummary}${moreText}`;
           this.respond(msg2.id, false, undefined, notAvailable);
           return;
         }
-        haltija.networkLog(payload2 || {}).then((result2) => {
-          this.respond(msg2.id, true, result2);
+        haltija.networkLog(payload2 || {}).then((result) => {
+          this.respond(msg2.id, true, result);
         }).catch((err) => {
           this.respond(msg2.id, false, undefined, err.message);
         });
@@ -5420,8 +5420,8 @@ ${elementSummary}${moreText}`;
           this.respond(msg2.id, false, undefined, notAvailable);
           return;
         }
-        haltija.networkStats().then((result2) => {
-          this.respond(msg2.id, true, result2);
+        haltija.networkStats().then((result) => {
+          this.respond(msg2.id, true, result);
         }).catch((err) => {
           this.respond(msg2.id, false, undefined, err.message);
         });
@@ -5430,8 +5430,8 @@ ${elementSummary}${moreText}`;
           this.respond(msg2.id, false, undefined, notAvailable);
           return;
         }
-        haltija.networkClear().then((result2) => {
-          this.respond(msg2.id, true, result2);
+        haltija.networkClear().then((result) => {
+          this.respond(msg2.id, true, result);
         }).catch((err) => {
           this.respond(msg2.id, false, undefined, err.message);
         });
@@ -5877,9 +5877,9 @@ ${elementSummary}${moreText}`;
             try {
               const capturePromise = targetSelector ? haltija.captureElement(targetSelector) : haltija.capturePage();
               const timeoutPromise = new Promise((resolve) => setTimeout(() => resolve({ success: false, error: "Screenshot capture timed out after 10s" }), 1e4));
-              const result2 = await Promise.race([capturePromise, timeoutPromise]);
-              if (result2?.success && result2.data) {
-                const converted = await convertFormat(result2.data);
+              const result = await Promise.race([capturePromise, timeoutPromise]);
+              if (result?.success && result.data) {
+                const converted = await convertFormat(result.data);
                 this.respond(msg2.id, true, {
                   image: converted.image,
                   viewport,
@@ -5889,8 +5889,8 @@ ${elementSummary}${moreText}`;
                   source: "electron"
                 });
                 return;
-              } else if (result2 && !result2.success) {
-                this.respond(msg2.id, false, null, result2.error || "Electron capture failed");
+              } else if (result && !result.success) {
+                this.respond(msg2.id, false, null, result.error || "Electron capture failed");
                 return;
               }
             } catch (err) {
@@ -5961,14 +5961,29 @@ ${elementSummary}${moreText}`;
         this.respond(msg2.id, true);
       }
     }
-    async handleEvalMessage(msg) {
+    async handleEvalMessage(msg2) {
       try {
-        const result = eval(msg.payload.code);
+        const result = this.evalCode(msg2.payload.code);
         const resolved = result instanceof Promise ? await result : result;
-        this.respond(msg.id, true, resolved);
+        this.respond(msg2.id, true, resolved);
       } catch (err) {
-        this.respond(msg.id, false, null, err.message);
+        this.respond(msg2.id, false, null, err.message);
+      }
+    }
+    evalCode(code) {
+      try {
+        return eval(code);
+      } catch (err) {
+        if (!(err instanceof SyntaxError) || !/\b(await|return)\b/.test(code))
+          throw err;
+      }
+      try {
+        return eval(`(async () => (${code}))()`);
+      } catch (err) {
+        if (!(err instanceof SyntaxError))
+          throw err;
       }
+      return eval(`(async () => { ${code} })()`);
     }
     async handleFetchMessage(msg2) {
       const { url } = msg2.payload;
@@ -6046,14 +6061,14 @@ ${elementSummary}${moreText}`;
       let el = null;
       let targetDesc = "";
       if (ref) {
-        const result2 = refRegistry.resolveWithStatus(ref);
-        el = result2.element;
+        const result = refRegistry.resolveWithStatus(ref);
+        el = result.element;
         targetDesc = `@${ref}`;
         if (el) {
           stats.refsResolved++;
         } else {
           stats.refsStale++;
-          const errorMsg = result2.status === "never_assigned" ? `Ref @${ref} was never assigned (highest ref is @${refRegistry.getStats().highWaterMark})` : result2.status === "removed_from_dom" ? `Ref @${ref} points to an element that was removed from the DOM (try refreshing /tree)` : `Ref @${ref} is stale - element was garbage collected (try refreshing /tree)`;
+          const errorMsg = result.status === "never_assigned" ? `Ref @${ref} was never assigned (highest ref is @${refRegistry.getStats().highWaterMark})` : result.status === "removed_from_dom" ? `Ref @${ref} points to an element that was removed from the DOM (try refreshing /tree)` : `Ref @${ref} is stale - element was garbage collected (try refreshing /tree)`;
           this.respond(responseId, false, null, errorMsg);
           return;
         }
@@ -6398,14 +6413,14 @@ ${elementSummary}${moreText}`;
       let el = null;
       let targetDesc = "";
       if (payload2.ref) {
-        const result2 = refRegistry.resolveWithStatus(payload2.ref);
-        el = result2.element;
+        const result = refRegistry.resolveWithStatus(payload2.ref);
+        el = result.element;
         targetDesc = `@${payload2.ref}`;
         if (el) {
           stats.refsResolved++;
         } else {
           stats.refsStale++;
-          const errorMsg = result2.status === "never_assigned" ? `Ref @${payload2.ref} was never assigned (highest ref is @${refRegistry.getStats().highWaterMark})` : result2.status === "removed_from_dom" ? `Ref @${payload2.ref} points to an element that was removed from the DOM (try refreshing /tree)` : `Ref @${payload2.ref} is stale - element was garbage collected (try refreshing /tree)`;
+          const errorMsg = result.status === "never_assigned" ? `Ref @${payload2.ref} was never assigned (highest ref is @${refRegistry.getStats().highWaterMark})` : result.status === "removed_from_dom" ? `Ref @${payload2.ref} points to an element that was removed from the DOM (try refreshing /tree)` : `Ref @${payload2.ref} is stale - element was garbage collected (try refreshing /tree)`;
           this.respond(responseId, false, null, errorMsg);
           return;
         }
@@ -6474,15 +6489,15 @@ ${elementSummary}${moreText}`;
         let target = null;
         let targetDesc = "";
         if (ref) {
-          const result2 = refRegistry.resolveWithStatus(ref);
-          target = result2.element;
+          const result = refRegistry.resolveWithStatus(ref);
+          target = result.element;
           targetDesc = `@${ref}`;
           if (target) {
             stats.refsResolved++;
             target.focus();
           } else {
             stats.refsStale++;
-            const errorMsg = result2.status === "never_assigned" ? `Ref @${ref} was never assigned (highest ref is @${refRegistry.getStats().highWaterMark})` : result2.status === "removed_from_dom" ? `Ref @${ref} points to an element that was removed from the DOM (try refreshing /tree)` : `Ref @${ref} is stale - element was garbage collected (try refreshing /tree)`;
+            const errorMsg = result.status === "never_assigned" ? `Ref @${ref} was never assigned (highest ref is @${refRegistry.getStats().highWaterMark})` : result.status === "removed_from_dom" ? `Ref @${ref} points to an element that was removed from the DOM (try refreshing /tree)` : `Ref @${ref} is stale - element was garbage collected (try refreshing /tree)`;
             this.respond(responseId, false, null, errorMsg);
             return;
           }
@@ -6874,9 +6889,9 @@ ${elementSummary}${moreText}`;
         });
       } else if (action2 === "result") {
         if (this.selectionResult) {
-          const result2 = this.selectionResult;
+          const result = this.selectionResult;
           this.clearSelection();
-          this.respond(msg2.id, true, result2);
+          this.respond(msg2.id, true, result);
         } else {
           this.respond(msg2.id, false, null, "No selection available");
         }

package/dist/component.d.ts

@@ -20,7 +20,7 @@
  * - Option+Tab toggles visibility (but active state always shows briefly)
  * - Localhost only by default
  */
-export declare const VERSION = "1.3.3";
+export declare const VERSION = "1.3.4";
 export declare class DevChannel extends HTMLElement {
     static get tagName(): string;
     static elementCreator(): () => DevChannel;
@@ -268,6 +268,17 @@ export declare class DevChannel extends HTMLElement {
     private handleEventsMessage;
     private handleConsoleMessage;
     private handleEvalMessage;
+    /**
+     * Direct `eval` never inherits an async context, so top-level `await` is a
+     * SyntaxError no matter what encloses this call — likewise `return`. Retry
+     * such code inside an async IIFE: expression position first (it preserves the
+     * completion value), statement body second (which needs an explicit `return`).
+     *
+     * The retry is gated on the code mentioning `await`/`return` because a runtime
+     * SyntaxError (`JSON.parse('{')`) is indistinguishable from a parse failure
+     * here, and re-running would repeat any side effects the code already had.
+     */
+    private evalCode;
     private handleFetchMessage;
     private handleInteractionMessage;
     /**