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

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

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

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

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

{remo_cli-1.0.2 → 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.2 → 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.2 → remo_cli-1.0.3}/uv.lock

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