remo-cli 1.0.1__tar.gz → 1.0.3__tar.gz

{remo_cli-1.0.1 → remo_cli-1.0.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: remo-cli
-Version: 1.0.1
+Version: 1.0.3
 Summary: Remote development environment CLI
 License-Expression: MIT
 License-File: LICENSE

{remo_cli-1.0.1 → remo_cli-1.0.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "remo-cli"
-version = "1.0.1"
+version = "1.0.3"
 description = "Remote development environment CLI"
 readme = "README.md"
 license = "MIT"

{remo_cli-1.0.1 → remo_cli-1.0.3}/src/remo_cli/cli/main.py

@@ -53,7 +53,24 @@ def completion(shell: str) -> None:
 
     ctx = click.get_current_context()
     instance = completer_cls(ctx.find_root().command, {}, "remo", "_REMO_COMPLETE")
-    click.echo(instance.source())
+    source = instance.source()
+
+    if shell == "fish":
+        # Click's completion handler emits a bare newline when there are no
+        # matches (e.g., `remo upd<TAB>` — no such command). Some fish versions
+        # then iterate the for-loop once with an empty $completion, and
+        # `string split "," ""` errors out. Guard the loop body so empty
+        # candidates are skipped.
+        source = source.replace(
+            "for completion in $response;\n        set -l metadata",
+            "for completion in $response;\n"
+            "        if test -z \"$completion\";\n"
+            "            continue;\n"
+            "        end;\n"
+            "        set -l metadata",
+        )
+
+    click.echo(source)
 
 
 def _register_commands() -> None:

{remo_cli-1.0.1 → remo_cli-1.0.3}/src/remo_cli/cli/shell.py

@@ -93,3 +93,16 @@ def _run_provider_update(host) -> None:  # noqa: ANN001
         # Incus name in known_hosts is "host/container" — extract just the container name
         container_name = host.name.split("/", maxsplit=1)[-1] if "/" in host.name else host.name
         incus_update(name=container_name)
+    elif host.type == "proxmox":
+        from remo_cli.providers.proxmox import update as proxmox_update  # noqa: PLC0415
+        # Proxmox name in known_hosts is "node/container".
+        # The proxmox SSH user is stored in the region slot (see providers.proxmox.create).
+        proxmox_host, _, container_name = host.name.partition("/")
+        if not container_name:
+            container_name = host.name
+            proxmox_host = ""
+        proxmox_update(
+            name=container_name,
+            host=proxmox_host,
+            user=host.region or "",
+        )

{remo_cli-1.0.1 → remo_cli-1.0.3}/tests/unit/cli/test_main.py

@@ -170,3 +170,42 @@ class TestIncusGroup:
         expected = ["create", "destroy", "update", "list", "sync", "bootstrap"]
         for name in expected:
             assert name in result.output, f"Incus subcommand '{name}' missing from help output"
+
+
+# ---------------------------------------------------------------------------
+# remo completion <shell>
+# ---------------------------------------------------------------------------
+
+
+class TestCompletionCommand:
+    """Verify ``remo completion <shell>`` emits a usable activation script."""
+
+    def test_fish_completion_emits_script(self):
+        result = _invoke("completion", "fish")
+        assert result.exit_code == 0
+        assert "function _remo_completion" in result.output
+
+    def test_fish_completion_skips_empty_candidates(self):
+        """Click emits a bare newline when no completions match (e.g.,
+        `remo upd<TAB>`). Some fish versions iterate the for-loop once with
+        an empty $completion, which then breaks `string split "," ""`. The
+        emitted script must guard against this.
+        """
+        result = _invoke("completion", "fish")
+        assert result.exit_code == 0
+        assert 'if test -z "$completion"' in result.output
+        # Sanity: the guard sits *inside* the for-loop, before string split
+        loop_idx = result.output.index("for completion in $response;")
+        guard_idx = result.output.index('if test -z "$completion"')
+        split_idx = result.output.index('set -l metadata (string split')
+        assert loop_idx < guard_idx < split_idx
+
+    def test_bash_completion_emits_script(self):
+        result = _invoke("completion", "bash")
+        assert result.exit_code == 0
+        assert "_remo_completion" in result.output
+
+    def test_zsh_completion_emits_script(self):
+        result = _invoke("completion", "zsh")
+        assert result.exit_code == 0
+        assert "_remo_completion" in result.output

{remo_cli-1.0.1 → remo_cli-1.0.3}/tests/unit/cli/test_shell.py

@@ -158,3 +158,21 @@ class TestRunProviderUpdate:
         _run_provider_update(host)
 
         mock_update.assert_called_once_with(name="devcontainer")
+
+    def test_proxmox_update_extracts_node_and_container(self, mocker):
+        from remo_cli.cli.shell import _run_provider_update
+
+        host = KnownHost(
+            type="proxmox",
+            name="lab1/dev1",
+            host="192.168.1.46",
+            user="remo",
+            instance_id="100",
+            access_mode="direct",
+            region="root",
+        )
+        mock_update = mocker.patch("remo_cli.providers.proxmox.update", return_value=0)
+
+        _run_provider_update(host)
+
+        mock_update.assert_called_once_with(name="dev1", host="lab1", user="root")

{remo_cli-1.0.1 → remo_cli-1.0.3}/uv.lock

@@ -771,7 +771,7 @@ wheels = [
 
 [[package]]
 name = "remo-cli"
-version = "1.0.1"
+version = "1.0.3"
 source = { editable = "." }
 dependencies = [
     { name = "ansible-core" },