webtap-tool 0.5.0__tar.gz → 0.5.2__tar.gz

{webtap_tool-0.5.0 → webtap_tool-0.5.2}/CHANGELOG.md

@@ -15,6 +15,34 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Removed
 
+## [0.5.2] - 2025-10-09
+
+### Added
+
+### Changed
+
+### Fixed
+- **setup-extension**: Update to download new side panel files (content.js, sidepanel.html, sidepanel.js) instead of old popup files
+- **setup-extension --force**: Now properly cleans old extension directory before downloading new files
+- **selections() command**: Updated error message to reference "side panel" instead of "popup"
+
+### Removed
+
+## [0.5.1] - 2025-10-09
+
+### Added
+- **js() persist parameter**: New `persist` parameter to control variable scope across calls (default: False)
+
+### Changed
+- **js() default scope**: Fresh scope by default (IIFE wrapping) to prevent const/let redeclaration errors
+- **js() selection wrapping**: Selection parameter always uses fresh scope to avoid element redeclaration
+
+### Fixed
+- **js() redeclaration errors**: Fixed "Identifier 'element' has already been declared" when using selection parameter multiple times
+- **js() const/let errors**: Fixed redeclaration errors when running multiple js() calls with const/let
+
+### Removed
+
 ## [0.5.0] - 2025-10-09
 
 ### Added

{webtap_tool-0.5.0 → webtap_tool-0.5.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: webtap-tool
-Version: 0.5.0
+Version: 0.5.2
 Summary: Terminal-based web page inspector for AI debugging sessions
 Author-email: Fredrik Angelsen <fredrikangelsen@gmail.com>
 Classifier: Development Status :: 3 - Alpha

{webtap_tool-0.5.0 → webtap_tool-0.5.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "webtap-tool"
-version = "0.5.0"
+version = "0.5.2"
 description = "Terminal-based web page inspector for AI debugging sessions"
 readme = "README.md"
 authors = [

{webtap_tool-0.5.0 → webtap_tool-0.5.2}/src/webtap/commands/javascript.py

@@ -15,26 +15,39 @@ mcp_desc = get_mcp_description("js")
     },
     fastmcp={"type": "tool", "description": mcp_desc} if mcp_desc else {"type": "tool"},
 )
-def js(state, code: str, selection: int = None, wait_return: bool = True, await_promise: bool = False) -> dict:  # pyright: ignore[reportArgumentType]
+def js(
+    state,
+    code: str,
+    selection: int = None,  # pyright: ignore[reportArgumentType]
+    persist: bool = False,
+    wait_return: bool = True,
+    await_promise: bool = False,
+) -> dict:
     """Execute JavaScript in the browser with optional element selection.
 
     Args:
         code: JavaScript code to execute (use 'element' variable if selection provided)
         selection: Browser element selection number (e.g., 1 for #1) - makes element available
+        persist: Keep variables in global scope across calls (default: False, uses fresh scope)
         wait_return: Wait for and return result (default: True)
         await_promise: Await promises before returning (default: False)
 
     Examples:
+        # Default: fresh scope (no redeclaration errors)
+        js("const x = 1")                              # ✓ x isolated
+        js("const x = 2")                              # ✓ No error, fresh scope
         js("document.title")                           # Get page title
-        js("document.body.innerText.length")           # Get text length
-        js("console.log('test')", wait_return=False)   # Fire and forget
         js("[...document.links].map(a => a.href)")    # Get all links
 
-        # With browser element selection
-        js("element.offsetWidth", selection=1)         # Use element #1 from browser()
+        # With browser element selection (always fresh scope)
+        js("element.offsetWidth", selection=1)         # Use element #1
         js("element.classList", selection=2)           # Use element #2
         js("element.getBoundingClientRect()", selection=1)
 
+        # Persistent scope: keep variables across calls
+        js("var data = fetch('/api')", persist=True)   # data persists
+        js("data.json()", persist=True)                # Access data from previous call
+
         # Async operations
         js("fetch('/api').then(r => r.json())", await_promise=True)
 
@@ -74,8 +87,13 @@ def js(state, code: str, selection: int = None, wait_return: bool = True, await_
         if not js_path:
             return error_response(f"Selection #{selection} has no jsPath")
 
-        # Wrap code with element variable
-        code = f"const element = {js_path}; {code}"
+        # Wrap code with element variable in fresh scope (IIFE)
+        # Selection always uses fresh scope to avoid element redeclaration errors
+        code = f"(() => {{ const element = {js_path}; return ({code}); }})()"
+    elif not persist:
+        # Default: wrap in IIFE for fresh scope (avoids const/let redeclaration errors)
+        code = f"(() => {{ return ({code}); }})()"
+    # else: persist=True, use code as-is (global scope)
 
     result = state.cdp.execute(
         "Runtime.evaluate", {"expression": code, "returnByValue": wait_return, "awaitPromise": await_promise}

{webtap_tool-0.5.0 → webtap_tool-0.5.2}/src/webtap/commands/selections.py

@@ -38,8 +38,8 @@ def selections(state, expr: str = None) -> dict:  # pyright: ignore[reportArgume
             "No browser selections available",
             suggestions=[
                 "Use the Chrome extension to select elements",
-                "Click 'Start Selection Mode' in the extension popup",
-                "Select elements on the page and submit a prompt",
+                "Click 'Start Selection Mode' in the extension side panel",
+                "Select elements on the page",
             ],
         )
 

{webtap_tool-0.5.0 → webtap_tool-0.5.2}/src/webtap/services/setup/extension.py

@@ -12,7 +12,7 @@ logger = logging.getLogger(__name__)
 
 # GitHub URLs for extension files
 EXTENSION_BASE_URL = "https://raw.githubusercontent.com/angelsen/tap-tools/main/packages/webtap/extension"
-EXTENSION_FILES = ["manifest.json", "popup.html", "popup.js"]
+EXTENSION_FILES = ["manifest.json", "content.js", "sidepanel.html", "sidepanel.js"]
 
 
 class ExtensionSetupService:
@@ -46,6 +46,13 @@ class ExtensionSetupService:
         # Ensure base directories exist
         ensure_directories()
 
+        # If force, clean out old extension files first
+        if force and self.extension_dir.exists():
+            import shutil
+
+            shutil.rmtree(self.extension_dir)
+            logger.info(f"Cleaned old extension directory: {self.extension_dir}")
+
         # Create extension directory
         self.extension_dir.mkdir(parents=True, exist_ok=True)