witbitz-code 1.6.2__tar.gz → 1.6.4__tar.gz

{witbitz_code-1.6.2/src/witbitz_code.egg-info → witbitz_code-1.6.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: witbitz-code
-Version: 1.6.2
+Version: 1.6.4
 Summary: Reach OpenCode on this computer from the Witbitz Spaces Code section, through an end-to-end sealed relay.
 Author: Witbitz
 License-Expression: MIT

{witbitz_code-1.6.2 → witbitz_code-1.6.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "witbitz-code"
-version = "1.6.2"
+version = "1.6.4"
 description = "Reach OpenCode on this computer from the Witbitz Spaces Code section, through an end-to-end sealed relay."
 readme = "README.md"
 requires-python = ">=3.10"

{witbitz_code-1.6.2 → witbitz_code-1.6.4}/src/witbitz_code/__init__.py

@@ -4,4 +4,4 @@ The Python implementation of tools/witbitz-code.mjs (docs/opencode-relay.md phas
 vectors and cross-implementation tests.
 """
 
-__version__ = "1.6.2"
+__version__ = "1.6.4"

{witbitz_code-1.6.2 → witbitz_code-1.6.4}/src/witbitz_code/auto.py

@@ -206,6 +206,25 @@ _KEY_FILE = re.compile(r"\.(pem|key|p12|pfx|jks|keystore)\Z", re.ASCII | re.IGNO
 _SECRETISH = re.compile(r"secret|credential", re.ASCII | re.IGNORECASE)
 
 
+_WIN_ABS = re.compile(r"\A(?:\\\\\?\\)?([A-Za-z]):[\\/]")
+
+
+def windows_to_posix(path: Any) -> Any:
+    """A Windows path in the form every rule here reads: C:\\Users\\x, C:/Users/x → /c/Users/x (Git Bash's own spelling).
+    Every rule tests "absolute" as a leading / — so on Windows (the owner, 2026-10-09: "The auto mode is not working
+    (windows)") the project folder was never absolute and Auto would not switch on, and C:/… in a command read as a path
+    INSIDE the project. Anything else is returned as it came."""
+    if not isinstance(path, str):
+        return path
+    m = _WIN_ABS.match(path)
+    return "/" + m.group(1).lower() + "/" + path[m.end():].replace("\\", "/") if m else path
+
+
+def _windows_project(ctx: dict) -> bool:
+    """A project on Windows: there a backslash separates folders too (..\\..\\x climbs out exactly as ../../x does)."""
+    return bool(_WIN_ABS.match(str(ctx.get("directory") or "")))
+
+
 def _sensitive_path(abs_path: str) -> bool:
     parts = abs_path.split("/")
     base = parts[-1] or ""
@@ -219,7 +238,7 @@ def _sensitive_path(abs_path: str) -> bool:
 
 
 def _project_dir(directory: Any) -> str:
-    d = _js.js_string(directory) if _js.truthy(directory) else ""
+    d = windows_to_posix(_js.js_string(directory)) if _js.truthy(directory) else ""
     return re.sub(r"/+\Z", "", posix_normalize(d))
 
 
@@ -246,6 +265,7 @@ def _word_stays_in_project(w: str, ctx: dict) -> bool:
         v = v[eq + 1:]
         if not v:
             return True
+    v = windows_to_posix(v.replace("\\", "/") if _windows_project(ctx) and not v.startswith("~") else v)
     if "$" in v or re.search(r"(^|/)\.\.(/|\Z)", v):
         return False
     if v.startswith("~"):
@@ -333,6 +353,8 @@ def _edit_inside_project(patterns: Any, ctx: dict) -> bool:
     for p in patterns:
         if not isinstance(p, str) or not p or re.search(r"[*?\[\]{}]", p):
             return False  # a glob is not a precise target
+        if _windows_project(ctx):
+            p = windows_to_posix(p.replace("\\", "/") if not _WIN_ABS.match(p) else p)
         a = posix_normalize(p if p.startswith("/") else d + "/" + p)
         r = next((x for x in roots if a.startswith(x + "/")), "")
         if not (r and not _sensitive_path(a[len(r):])):

{witbitz_code-1.6.2 → witbitz_code-1.6.4}/src/witbitz_code/auto_runner.py

@@ -33,7 +33,7 @@ import httpx
 
 from . import _js
 from .folders import allowed_folders
-from .auto import SCRATCH_DIR, action_for, classify_deterministic, log_record, parse_verdict, reviewer_prompt, scratch_paths
+from .auto import SCRATCH_DIR, windows_to_posix, action_for, classify_deterministic, log_record, parse_verdict, reviewer_prompt, scratch_paths
 from .policy import SUBAGENT_GATED
 
 REVIEW_TITLE = "witbitz-auto-review"  # the page never lists a session with this title
@@ -146,7 +146,7 @@ class AutoRunner:
             doc = _js.parse(self._state_path.read_text(encoding="utf-8"))
             sessions = doc.get("sessions") if isinstance(doc, dict) else None
             for sid, v in (sessions.items() if isinstance(sessions, dict) else []):
-                if sid.startswith("ses") and isinstance(v, dict) and isinstance(v.get("dir"), str) and v["dir"].startswith("/"):
+                if sid.startswith("ses") and isinstance(v, dict) and isinstance(v.get("dir"), str) and windows_to_posix(v["dir"]).startswith("/"):
                     at = _js.to_number(v.get("at", _js.UNDEFINED))
                     self._auto[sid] = {"dir": v["dir"], "at": _js.clean_number(at) if _js.truthy(at) else 0}
         except Exception as e:
@@ -470,7 +470,8 @@ class AutoRunner:
         if not isinstance(session_id, str) or not _SESSION_ID.fullmatch(session_id):
             return False
         if on:
-            if not isinstance(directory, str) or not directory.startswith("/") or _js.utf16_len(directory) > 1024:
+            # Windows: C:\\Users\\… is absolute too (auto.windows_to_posix); the folder is kept as given for OpenCode's API.
+            if not isinstance(directory, str) or not windows_to_posix(directory).startswith("/") or _js.utf16_len(directory) > 1024:
                 return False
             self._auto[session_id] = {"dir": directory, "at": self._now()}  # an existing key keeps its place, as Map.set does
         else:

{witbitz_code-1.6.2 → witbitz_code-1.6.4}/src/witbitz_code/ui.py

@@ -63,11 +63,15 @@ def computer_label() -> dict:
 def open_browser(url: str) -> bool:
     """WSL has no browser of its own: ask Windows (wslview, then cmd's start, then explorer)."""
     if ports.on_wsl():
+        # WAIT for the hand-off (~0.4 s measured): a second copy started from the Start menu finds the first, opens its page
+        # and exits — and an opener still running then was cut off with it (the owner, 2026-10-09: "the wsl does not open
+        # anything"). From a Windows folder, or cmd says "UNC paths are not supported".
+        cwd = "/mnt/c" if os.path.isdir("/mnt/c") else None
         for argv in (["wslview", url], ["cmd.exe", "/c", "start", "", url], ["explorer.exe", url]):
             try:
-                subprocess.Popen(argv, stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)  # noqa: S603 — fixed argv, our own URL
+                subprocess.run(argv, stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL, cwd=cwd, timeout=15, check=False)  # noqa: S603 — fixed argv, our own URL
                 return True
-            except OSError:
+            except (OSError, subprocess.SubprocessError):
                 continue
         return False
     try:

{witbitz_code-1.6.2 → witbitz_code-1.6.4/src/witbitz_code.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: witbitz-code
-Version: 1.6.2
+Version: 1.6.4
 Summary: Reach OpenCode on this computer from the Witbitz Spaces Code section, through an end-to-end sealed relay.
 Author: Witbitz
 License-Expression: MIT

{witbitz_code-1.6.2 → witbitz_code-1.6.4}/tests/test_auto.py

@@ -629,3 +629,52 @@ def test_wiring_the_page_switches_auto_and_the_verdict_comes_back(make, tmp_path
                 await connector.stop()
 
     run(go())
+
+
+# ── WINDOWS (the owner, 2026-10-09: "The auto mode is not working (windows)") ────────────────────────────────────────────
+# Every rule read "absolute" as a leading /: Auto would not switch on for C:\…, and C:/… in a command read as INSIDE the
+# project. A Windows request now gets exactly the decision its Linux twin gets.
+_WIN_TWINS = [
+    ("ls src", "ls src"),
+    ("cat /etc/hosts", "cat C:/Windows/win.ini"),
+    ("cat /home/u/repo/src/a.py", "cat C:/Users/u/repo/src/a.py"),
+    ("cat /home/u/repo/.git/config", "cat /c/Users/u/repo/.git/config"),
+]
+_WIN_EDITS = [("src/a.py", "src\\a.py"), ("../../x", "..\\..\\x"), ("/home/u/repo/src/a.py", "C:\\Users\\u\\repo\\src\\a.py"),
+              ("/home/u/other/a.py", "C:\\Users\\u\\other\\a.py")]
+
+
+def _stage(req, directory, home):
+    r = classify_deterministic({"id": "per_x", "sessionID": "ses_x", "always": [], **req}, directory=directory, home=home)
+    return r["stage"] if r else "review"
+
+
+@pytest.mark.parametrize("lin,win", _WIN_TWINS)
+def test_a_windows_command_is_judged_like_its_linux_twin(lin, win):
+    b = lambda c: {"permission": "bash", "patterns": [c], "metadata": {"command": c}}  # noqa: E731
+    assert _stage(b(win), "C:\\Users\\u\\repo", "C:\\Users\\u") == _stage(b(lin), "/home/u/repo", "/home/u")
+
+
+@pytest.mark.parametrize("lin,win", _WIN_EDITS)
+def test_a_windows_edit_is_judged_like_its_linux_twin(lin, win):
+    e = lambda p: {"permission": "edit", "patterns": [p], "metadata": {}}  # noqa: E731
+    assert _stage(e(win), "C:\\Users\\u\\repo", "C:\\Users\\u") == _stage(e(lin), "/home/u/repo", "/home/u")
+
+
+def test_outside_the_project_on_windows_is_never_allowed_by_itself():
+    assert _stage({"permission": "bash", "patterns": ["cat C:/Windows/win.ini"], "metadata": {"command": "cat C:/Windows/win.ini"}},
+                  "C:\\Users\\u\\repo", "C:\\Users\\u") != "fast-allow"
+    assert _stage({"permission": "edit", "patterns": ["..\\..\\x"], "metadata": {}}, "C:\\Users\\u\\repo", "C:\\Users\\u") != "fast-allow"
+
+
+def test_auto_switches_on_for_a_windows_folder_and_talks_to_opencode_with_it_as_given(tmp_path):
+    win = "C:\\Users\\u\\repo"
+
+    async def body(oc, runner, verdicts, make, client):
+        assert runner.set_auto("ses_main", win, True), "C:\\… is a project folder (it was refused: Auto never ran on Windows)"
+        assert runner.set_auto("ses_main", "relative\\dir", True) is False
+        assert runner.set_auto("ses_main", win, True)
+        oc.ask(permission="bash", patterns=["rm -rf ~"], metadata={"command": "rm -rf ~"})
+        assert await until(lambda: oc.replies(), 4)
+        assert oc.replies()[0]["dir"] == win, "OpenCode is asked with the folder exactly as it named it"
+    with_runner(tmp_path, body)

{witbitz_code-1.6.2 → witbitz_code-1.6.4}/tests/test_ui.py

@@ -263,3 +263,16 @@ def test_quit_stops_serving_and_ends_the_manager():
             await m.shutdown()
             await srv.close()
     run(go())
+
+
+def test_in_wsl_the_page_opener_is_waited_for(monkeypatch):
+    """A second copy (the Start-menu entry) opens the first one's page and exits: an opener still running was cut off with
+    it (the owner, 2026-10-09: "the wsl does not open anything")."""
+    import subprocess as sp
+    calls = []
+    monkeypatch.setattr(ui.ports, "on_wsl", lambda: True)
+    monkeypatch.setattr(ui.subprocess, "run", lambda argv, **kw: calls.append((argv, kw)) or sp.CompletedProcess(argv, 0))
+    monkeypatch.setattr(ui.subprocess, "Popen", lambda *a, **k: (_ for _ in ()).throw(AssertionError("never fire-and-forget")))
+    assert ui.open_browser("http://127.0.0.1:4291/?t=x") is True
+    argv, kw = calls[0]
+    assert argv == ["wslview", "http://127.0.0.1:4291/?t=x"] and kw["timeout"] == 15