browser-workspace 0.4.0 → 0.4.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "browser-workspace",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "description": "Agent browser workspace with persistent sessions, tab leasing, snapshots, and platform workflows.",
   "bin": {
     "browser-workspace": "bin/browser-workspace"

package/platforms/xhs/actions/_manage.py

@@ -17,7 +17,7 @@ def open_or_reuse_xhs(url):
         target=tabs[0] if tabs else None
     if target:
         switch_tab(target)
-        if page_info()["url"] != url:
+        if (current_tab().get("url") or "") != url:
             goto_url(url)
     else:
         new_tab(url)
@@ -86,7 +86,7 @@ def open_update_editor(row):
         return None, clicked
     for _ in range(30):
         time.sleep(.5)
-        url=page_info()["url"]
+        url=current_tab().get("url") or ""
         if "/publish/update" in url:
             tab=current_tab()
             return {"update_url":url,"target_id":tab.get("targetId") or tab.get("target_id")}, None
@@ -212,7 +212,7 @@ def open_post_detail(note_id):
     ready=js('document.querySelector(".note-container") ? "yes" : "no"')
     if ready!="yes":
         return None, "post_detail_not_opened"
-    return {"profile_url":profile,"detail_url":page_info()["url"]}, None
+    return {"profile_url":profile,"detail_url":current_tab().get("url") or ""}, None
 
 
 def expand_replies():

package/platforms/xhs/actions/_post.py

@@ -17,7 +17,7 @@ def open_or_reuse_xhs(url):
         target=tabs[0] if tabs else None
     if target:
         switch_tab(target)
-        if page_info()["url"] != url:
+        if (current_tab().get("url") or "") != url:
             goto_url(url)
     else:
         new_tab(url)
@@ -47,7 +47,7 @@ open_or_reuse_xhs("https://creator.xiaohongshu.com/publish/publish")
 wait_for_load()
 time.sleep(3)
 
-url = page_info()["url"]
+url = current_tab().get("url") or ""
 if "login" in url:
     print("ERROR: Not logged in. Log in at creator.xiaohongshu.com first.")
     capture_screenshot()

package/platforms/xhs/tests/test_contract.py

@@ -0,0 +1,17 @@
+from pathlib import Path
+
+
+HERE = Path(__file__).resolve().parents[1]
+
+
+def test_actions_do_not_use_page_info_as_tab_url_authority():
+    for name in ("_post.py", "_manage.py"):
+        source = (HERE / "actions" / name).read_text()
+        assert 'page_info()["url"]' not in source
+
+
+def test_actions_use_current_tab_url_for_navigation_identity():
+    post = (HERE / "actions" / "_post.py").read_text()
+    manage = (HERE / "actions" / "_manage.py").read_text()
+    assert '(current_tab().get("url") or "") != url' in post
+    assert '(current_tab().get("url") or "") != url' in manage