loom-code 0.1.1__tar.gz → 0.1.2__tar.gz

{loom_code-0.1.1 → loom_code-0.1.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: loom-code
-Version: 0.1.1
+Version: 0.1.2
 Summary: loom-code — a loomflow-native terminal coding agent
 Author: Anupam Nautiyal
 License: MIT License
@@ -106,11 +106,12 @@ load-bearing — the agent loop, tools, planning, memory — is loomflow.
 ## Install
 
 ```bash
-pipx install git+https://github.com/Anurich/loomflow-cli
+pipx install loom-code
 ```
 
-(`pip install` works too; `pipx` keeps CLI tools in their own venvs.
-No pipx? `brew install pipx` or `python -m pip install --user pipx`.)
+(`pip install loom-code` works too; `pipx` keeps CLI tools in their
+own venvs. No pipx? `brew install pipx` or
+`python -m pip install --user pipx`.)
 
 Requires Python 3.11+. To update: `pipx upgrade loom-code`.
 

{loom_code-0.1.1 → loom_code-0.1.2}/README.md

@@ -52,11 +52,12 @@ load-bearing — the agent loop, tools, planning, memory — is loomflow.
 ## Install
 
 ```bash
-pipx install git+https://github.com/Anurich/loomflow-cli
+pipx install loom-code
 ```
 
-(`pip install` works too; `pipx` keeps CLI tools in their own venvs.
-No pipx? `brew install pipx` or `python -m pip install --user pipx`.)
+(`pip install loom-code` works too; `pipx` keeps CLI tools in their
+own venvs. No pipx? `brew install pipx` or
+`python -m pip install --user pipx`.)
 
 Requires Python 3.11+. To update: `pipx upgrade loom-code`.
 

{loom_code-0.1.1 → loom_code-0.1.2}/loom_code/__init__.py

@@ -19,4 +19,4 @@ belongs in the framework, not here. loom-code is the dogfood test
 that keeps loomflow honest.
 """
 
-__version__ = "0.1.1"
+__version__ = "0.1.2"

{loom_code-0.1.1 → loom_code-0.1.2}/loom_code/approval.py

@@ -112,6 +112,30 @@ def _read_key_raw(fd: int) -> str:
     return ch.lower()
 
 
+def _read_key_msvcrt() -> str:
+    """Windows equivalent of :func:`_read_key_raw` — one LOGICAL key
+    via ``msvcrt.getwch()`` (already raw + no echo, so no mode
+    setup/teardown is needed).
+
+    Arrow keys arrive as a TWO-event sequence: a ``'\\xe0'`` (or
+    ``'\\x00'`` for some layouts) prefix, then ``'H'`` (up) /
+    ``'P'`` (down). Ctrl-C surfaces as ``'\\x03'`` and maps to the
+    SAFE 'esc', mirroring the POSIX reader."""
+    import msvcrt
+
+    ch = msvcrt.getwch()
+    if ch in ("\r", "\n"):
+        return "enter"
+    if ch == "\x03":  # Ctrl-C
+        return "esc"
+    if ch == "\x1b":
+        return "esc"
+    if ch in ("\xe0", "\x00"):  # extended-key prefix
+        final = msvcrt.getwch()
+        return {"H": "up", "P": "down"}.get(final, "esc")
+    return ch.lower()
+
+
 def _read_key() -> str:
     """Single-key read that manages its own raw-mode window. Prefer
     :func:`_read_key_raw` inside a selector that enters raw mode ONCE
@@ -187,23 +211,54 @@ def _select_option(options: list[tuple[str, str]], default: int = 0) -> str:
             out.write("\r\n")  # explicit CR+LF for raw mode
         out.flush()
 
-    # Enter raw mode ONCE for the whole selector session — no
+    # Pick the platform's key reader + raw-mode strategy.
+    #
+    # POSIX: enter raw mode ONCE for the whole selector session — no
     # per-keypress termios churn, and no cooked-mode gap between keys
     # where type-ahead would echo raw escape bytes onto the prompt.
-    import termios
-    import tty
+    #
+    # Windows: there is NO termios (the bare import crashed /set_model
+    # for pipx users with ModuleNotFoundError). msvcrt.getwch() is
+    # already raw + unbuffered, so no mode management is needed at
+    # all; ``os.system("")`` nudges legacy conhost into processing the
+    # ANSI redraw sequences (Windows Terminal has VT on by default).
+    def _restore() -> None:
+        return None
+
+    def _reader() -> str:
+        return _read_key()
 
-    fd = sys.stdin.fileno()
     try:
-        old = termios.tcgetattr(fd)
-    except Exception:
-        old = None
-    if old is not None:
-        tty.setraw(fd)
+        import termios
+        import tty
+
+        fd = sys.stdin.fileno()
+        try:
+            old = termios.tcgetattr(fd)
+        except Exception:
+            old = None
+        if old is not None:
+            tty.setraw(fd)
+
+            def _reader() -> str:
+                return _read_key_raw(fd)
+
+            def _restore() -> None:
+                termios.tcsetattr(fd, termios.TCSADRAIN, old)
+    except ImportError:
+        try:
+            import msvcrt  # noqa: F401 — probe: Windows console?
+            import os
+
+            os.system("")  # enable VT processing on legacy conhost
+            _reader = _read_key_msvcrt
+        except ImportError:
+            pass  # exotic platform → keep the _read_key fallback
+
     try:
         _draw(first=True)
         while True:
-            key = _read_key_raw(fd) if old is not None else _read_key()
+            key = _reader()
             if key == "up":
                 idx = (idx - 1) % n
             elif key == "down":
@@ -221,8 +276,7 @@ def _select_option(options: list[tuple[str, str]], default: int = 0) -> str:
                 continue  # unknown key: ignore, keep waiting
             _draw(first=False)
     finally:
-        if old is not None:
-            termios.tcsetattr(fd, termios.TCSADRAIN, old)
+        _restore()
 
 
 def _read_single_key() -> str:

{loom_code-0.1.1 → loom_code-0.1.2}/loom_code.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: loom-code
-Version: 0.1.1
+Version: 0.1.2
 Summary: loom-code — a loomflow-native terminal coding agent
 Author: Anupam Nautiyal
 License: MIT License
@@ -106,11 +106,12 @@ load-bearing — the agent loop, tools, planning, memory — is loomflow.
 ## Install
 
 ```bash
-pipx install git+https://github.com/Anurich/loomflow-cli
+pipx install loom-code
 ```
 
-(`pip install` works too; `pipx` keeps CLI tools in their own venvs.
-No pipx? `brew install pipx` or `python -m pip install --user pipx`.)
+(`pip install loom-code` works too; `pipx` keeps CLI tools in their
+own venvs. No pipx? `brew install pipx` or
+`python -m pip install --user pipx`.)
 
 Requires Python 3.11+. To update: `pipx upgrade loom-code`.
 

{loom_code-0.1.1 → loom_code-0.1.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "loom-code"
-version = "0.1.1"
+version = "0.1.2"
 description = "loom-code — a loomflow-native terminal coding agent"
 readme = "README.md"
 license = { file = "LICENSE" }
@@ -129,7 +129,7 @@ select = ["E", "F", "I", "B", "UP", "ASYNC"]
 # publishing — no tokens). Daily flow stays plain git; ``make release
 # BUMP=patch|minor|major`` is the explicit "this is a release" flag.
 [tool.bumpversion]
-current_version = "0.1.1"
+current_version = "0.1.2"
 search = "{current_version}"
 replace = "{new_version}"
 commit = true

{loom_code-0.1.1 → loom_code-0.1.2}/tests/test_approval.py

@@ -70,3 +70,78 @@ async def test_approval_gate_allow_all_works_for_every_tool() -> None:
     for tool in ("bash", "edit", "write"):
         call = ToolCall(tool=tool, args={})
         assert await gate.handler(call) is True
+
+
+# ---- Windows key reader (msvcrt) -------------------------------------
+# termios doesn't exist on Windows; the selector dispatches to
+# _read_key_msvcrt there. We can't run real Windows in CI, but the
+# reader's decode logic is pure — drive it with a fake msvcrt module.
+
+
+def _fake_msvcrt(keys: list[str]):
+    """A stand-in msvcrt whose getwch() pops from ``keys``."""
+    import types
+
+    mod = types.ModuleType("msvcrt")
+    seq = iter(keys)
+    mod.getwch = lambda: next(seq)
+    return mod
+
+
+def test_msvcrt_reader_decodes_logical_keys(monkeypatch) -> None:
+    import sys as _sys
+
+    from loom_code.approval import _read_key_msvcrt
+
+    cases = [
+        (["\r"], "enter"),
+        (["\n"], "enter"),
+        (["\x03"], "esc"),   # Ctrl-C → SAFE cancel
+        (["\x1b"], "esc"),
+        (["\xe0", "H"], "up"),
+        (["\xe0", "P"], "down"),
+        (["\x00", "H"], "up"),     # alternate extended prefix
+        (["\xe0", "K"], "esc"),    # unknown extended key → safe
+        (["3"], "3"),
+        (["Y"], "y"),
+    ]
+    for keys, expected in cases:
+        monkeypatch.setitem(
+            _sys.modules, "msvcrt", _fake_msvcrt(keys)
+        )
+        assert _read_key_msvcrt() == expected, (keys, expected)
+
+
+def test_select_option_survives_missing_termios(monkeypatch) -> None:
+    """Regression: on Windows there is no termios — the selector's
+    bare ``import termios`` crashed /set_model with
+    ModuleNotFoundError for every pipx-on-Windows user. With the
+    dispatch fix, a missing termios falls through to the msvcrt
+    reader (faked here): ↓ then Enter must select option 2 and
+    never raise."""
+    import builtins
+    import sys as _sys
+
+    from loom_code import approval
+
+    real_import = builtins.__import__
+
+    def _no_termios(name, *args, **kwargs):
+        if name in ("termios", "tty"):
+            raise ImportError(f"No module named '{name}'")
+        return real_import(name, *args, **kwargs)
+
+    monkeypatch.setattr(builtins, "__import__", _no_termios)
+    monkeypatch.setitem(
+        _sys.modules, "msvcrt", _fake_msvcrt(["\xe0", "P", "\r"])
+    )
+    monkeypatch.delitem(_sys.modules, "termios", raising=False)
+    monkeypatch.delitem(_sys.modules, "tty", raising=False)
+    # stdin must look like a TTY to reach the interactive path
+    monkeypatch.setattr(
+        approval.sys.stdin, "isatty", lambda: True, raising=False
+    )
+    result = approval._select_option(
+        [("a", "first"), ("b", "second"), ("c", "third")]
+    )
+    assert result == "b"  # ↓ moved 0→1, Enter picked it