cli-to-tools 0.2.1__tar.gz → 0.2.2__tar.gz

{cli_to_tools-0.2.1 → cli_to_tools-0.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cli-to-tools
-Version: 0.2.1
+Version: 0.2.2
 Summary: Translate shell command lines into ordered, structured tool calls.
 License: MIT
 Project-URL: Homepage, https://agentltl.github.io/shell/
@@ -46,11 +46,18 @@ The library only translates. Executing the command line stays with the caller.
 Paste this into your coding agent (Claude Code, Codex, Copilot CLI, Mistral Vibe...):
 
 ```
-Install cli-to-tools from https://github.com/AgentLTL/cli-to-tools in a virtual environment (pip install "git+https://github.com/AgentLTL/cli-to-tools.git"), then use cli_to_tools.Translator to translate a shell command line of mine into tool calls and show me the result.
+Install cli-to-tools (pip install cli-to-tools, in a virtual environment), then use cli_to_tools.Translator to translate a shell command line of mine into tool calls and show me the result.
 ```
 
 Or install manually:
 
+```bash
+pip install cli-to-tools
+pip install "cli-to-tools[agentltl]"   # with AgentLTL, for cli_to_tools.agentltl
+```
+
+To work on the library itself, from a clone:
+
 ```bash
 pip install -e ".[dev]"
 pip install -e /path/to/AgentLTL      # only for cli_to_tools.agentltl

{cli_to_tools-0.2.1 → cli_to_tools-0.2.2}/README.md

@@ -27,11 +27,18 @@ The library only translates. Executing the command line stays with the caller.
 Paste this into your coding agent (Claude Code, Codex, Copilot CLI, Mistral Vibe...):
 
 ```
-Install cli-to-tools from https://github.com/AgentLTL/cli-to-tools in a virtual environment (pip install "git+https://github.com/AgentLTL/cli-to-tools.git"), then use cli_to_tools.Translator to translate a shell command line of mine into tool calls and show me the result.
+Install cli-to-tools (pip install cli-to-tools, in a virtual environment), then use cli_to_tools.Translator to translate a shell command line of mine into tool calls and show me the result.
 ```
 
 Or install manually:
 
+```bash
+pip install cli-to-tools
+pip install "cli-to-tools[agentltl]"   # with AgentLTL, for cli_to_tools.agentltl
+```
+
+To work on the library itself, from a clone:
+
 ```bash
 pip install -e ".[dev]"
 pip install -e /path/to/AgentLTL      # only for cli_to_tools.agentltl

{cli_to_tools-0.2.1 → cli_to_tools-0.2.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "cli-to-tools"
-version = "0.2.1"
+version = "0.2.2"
 description = "Translate shell command lines into ordered, structured tool calls."
 readme = "README.md"
 requires-python = ">=3.10"

{cli_to_tools-0.2.1 → cli_to_tools-0.2.2}/src/cli_to_tools/_effects.py

@@ -35,6 +35,16 @@ UNKNOWN_PATHS = "unknown_paths"
 _DEVICES = ("/dev/null", "/dev/stdout", "/dev/stderr", "/dev/tty")
 
 
+def posix(path: str) -> str:
+    """*path* with ``/`` separators on every platform (a no-op off Windows).
+
+    Rules write path globs with ``/`` (``**/.env``, ``src/*.py``), and two calls name the
+    same file the same way, so every path this package reports uses ``/``; Python opens
+    ``C:/proj/a.txt`` on Windows just as well as ``C:\\proj\\a.txt``.
+    """
+    return path.replace(os.sep, "/") if os.sep != "/" else path
+
+
 @dataclass(frozen=True)
 class Paths:
     """Where relative paths are resolved from (``cwd``), and the project root, if any."""
@@ -42,10 +52,28 @@ class Paths:
     cwd: str = ""
     root: str = ""
 
+    def __post_init__(self) -> None:
+        object.__setattr__(self, "cwd", posix(self.cwd))
+        object.__setattr__(self, "root", posix(self.root))
+
+
+WINDOWS = os.name == "nt"
+
+
+def is_absolute(path: str) -> bool:
+    """A rooted path: ``/x``, ``~``, or on Windows ``C:\\x``, ``C:/x`` and ``\\x``.
+
+    Off Windows ``\\x`` and ``C:\\x`` are ordinary relative file names."""
+    if path.startswith(("/", "~")):
+        return True
+    return WINDOWS and (path.startswith("\\") or (len(path) > 2 and path[1] == ":"
+                                                   and path[2] in "/\\" and path[0].isalpha()))
+
 
 def is_device(target: str) -> bool:
     """Writing to these discards or shows the output; it overwrites no file."""
-    return target in _DEVICES or target.startswith("/dev/fd/")
+    return (target in _DEVICES or target.startswith("/dev/fd/")
+            or (WINDOWS and target.upper() == "NUL"))
 
 
 def redirect_files(redirects: Sequence[Dict[str, str]]) -> Tuple[List[str], List[str], List[str]]:
@@ -69,7 +97,7 @@ def redirect_files(redirects: Sequence[Dict[str, str]]) -> Tuple[List[str], List
 
 
 def absolute(value: str, paths: Paths) -> str:
-    return os.path.normpath(os.path.join(paths.cwd or os.getcwd(), os.path.expanduser(value)))
+    return posix(os.path.normpath(os.path.join(paths.cwd or os.getcwd(), os.path.expanduser(value))))
 
 
 def canonical(value: str, paths: Paths) -> str:
@@ -114,7 +142,7 @@ def relative_path_keys(args: Dict[str, Any], keys: Iterable[str]) -> List[str]:
     """The path arguments among *keys* holding a relative path."""
     out = []
     for key in keys:
-        if any(v and not v.startswith(("/", "~", "$", "-")) and "://" not in v
+        if any(v and not is_absolute(v) and not v.startswith(("$", "-")) and "://" not in v
                for v in strings(args.get(key))):
             out.append(key)
     return out
@@ -138,8 +166,8 @@ def cd_target(name: str, args: Dict[str, Any], here: Optional[str]) -> Optional[
         return None
     target = os.path.expanduser(target)
     if os.path.isabs(target):
-        return os.path.normpath(target)
-    return None if here is None else os.path.normpath(os.path.join(here, target))
+        return posix(os.path.normpath(target))
+    return None if here is None else posix(os.path.normpath(os.path.join(here, target)))
 
 
 def patch_targets(name: str, args: Dict[str, Any], paths: Paths) -> None:

{cli_to_tools-0.2.1 → cli_to_tools-0.2.2}/src/cli_to_tools.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cli-to-tools
-Version: 0.2.1
+Version: 0.2.2
 Summary: Translate shell command lines into ordered, structured tool calls.
 License: MIT
 Project-URL: Homepage, https://agentltl.github.io/shell/
@@ -46,11 +46,18 @@ The library only translates. Executing the command line stays with the caller.
 Paste this into your coding agent (Claude Code, Codex, Copilot CLI, Mistral Vibe...):
 
 ```
-Install cli-to-tools from https://github.com/AgentLTL/cli-to-tools in a virtual environment (pip install "git+https://github.com/AgentLTL/cli-to-tools.git"), then use cli_to_tools.Translator to translate a shell command line of mine into tool calls and show me the result.
+Install cli-to-tools (pip install cli-to-tools, in a virtual environment), then use cli_to_tools.Translator to translate a shell command line of mine into tool calls and show me the result.
 ```
 
 Or install manually:
 
+```bash
+pip install cli-to-tools
+pip install "cli-to-tools[agentltl]"   # with AgentLTL, for cli_to_tools.agentltl
+```
+
+To work on the library itself, from a clone:
+
 ```bash
 pip install -e ".[dev]"
 pip install -e /path/to/AgentLTL      # only for cli_to_tools.agentltl

{cli_to_tools-0.2.1 → cli_to_tools-0.2.2}/tests/test_effects.py

@@ -1,5 +1,7 @@
 """Translator(shell_effects=True): redirections, paths after cd, run-time-only files."""
 
+import os
+
 import pytest
 
 from cli_to_tools import Paths, SpecRegistry, Translator, UNKNOWN_PATHS
@@ -34,6 +36,25 @@ def test_redirections(registry):
     assert a["redirect_to"] == ["/dev/null"] and "overwrite_to" not in a
 
 
+def test_windows_path_forms_are_rooted_and_nul_is_a_device_only_on_windows(monkeypatch):
+    from cli_to_tools import _effects
+    monkeypatch.setattr(_effects, "WINDOWS", True)
+    assert all(_effects.is_absolute(p) for p in ("C:\\foo", "c:/foo", "\\foo", "/foo", "~"))
+    assert not _effects.is_absolute("foo") and not _effects.is_absolute("C:foo")
+    assert _effects.is_device("NUL") and _effects.is_device("nul")
+
+    monkeypatch.setattr(_effects, "WINDOWS", False)   # POSIX: these are ordinary file names
+    assert not _effects.is_absolute("C:\\foo") and not _effects.is_absolute("\\foo")
+    assert _effects.is_absolute("/foo") and _effects.is_absolute("~")
+    assert not _effects.is_device("NUL") and _effects.is_device("/dev/null")
+
+
+@pytest.mark.skipif(os.name == "nt", reason="paths are resolved the Windows way there")
+def test_on_posix_a_backslash_name_is_relative_and_NUL_is_a_file(registry):
+    assert args("cd $D; rm '\\foo'", registry)[UNKNOWN_PATHS] == ["paths"]
+    assert args("echo x > NUL", registry)["overwrite_to"] == ["/proj/NUL"]
+
+
 def test_cd_moves_where_relative_paths_point(registry):
     assert args("cd secrets && rm key", registry)["paths"] == ["/proj/secrets/key"]
     assert args("cd secrets; cd ..; rm key", registry)["paths"] == ["/proj/key"]
@@ -62,8 +83,9 @@ def test_files_only_known_at_run_time(registry):
 
 def test_resolved_variables_and_home(registry, tmp_path, monkeypatch):
     monkeypatch.setenv("HOME", str(tmp_path))
+    monkeypatch.setenv("USERPROFILE", str(tmp_path))     # what `~` expands to on Windows
     assert args("D=secrets; rm $D/key", registry)["paths"] == ["/proj/secrets/key"]
-    assert args("rm ~/notes", registry)["paths"] == [str(tmp_path / "notes")]
+    assert args("rm ~/notes", registry)["paths"] == [(tmp_path / "notes").as_posix()]
 
 
 def test_patch_targets_come_from_the_diff(registry, tmp_path):
@@ -71,7 +93,7 @@ def test_patch_targets_come_from_the_diff(registry, tmp_path):
         "diff --git a/migrations/001.sql b/migrations/001.sql\n"
         "--- a/migrations/001.sql\n+++ b/migrations/001.sql\n@@ -1 +1 @@\n-x\n+y\n")
     assert args("git apply fix.diff", registry, str(tmp_path))["paths"] == [
-        str(tmp_path / "migrations/001.sql")]
+        (tmp_path / "migrations/001.sql").as_posix()]
     assert args("git apply missing.diff", registry, str(tmp_path))[UNKNOWN_PATHS] is True