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

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

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

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

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

{remo_cli-1.0.0 → remo_cli-1.0.2}/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.0 → remo_cli-1.0.2}/src/remo_cli/core/validation.py

@@ -26,6 +26,38 @@ def validate_name(value: str, label: str = "name") -> None:
         )
 
 
+def parse_volume_size(value: str) -> str:
+    """Normalize a user-provided volume size to a pure integer string.
+
+    Accepts a plain integer (``"100"``) or an integer with a common
+    size suffix (``"100G"``, ``"100GB"``, ``"100GiB"`` — case-insensitive)
+    and returns the integer portion as a string. Empty input passes
+    through unchanged so callers can treat ``""`` as "not provided".
+
+    Raises :class:`click.BadParameter` for anything else.
+    """
+    if not value:
+        return value
+    cleaned = value.strip()
+    lowered = cleaned.lower()
+    for suffix in ("gib", "gb", "g"):
+        if lowered.endswith(suffix):
+            cleaned = cleaned[: -len(suffix)].strip()
+            break
+    try:
+        as_int = int(cleaned)
+    except ValueError as exc:
+        raise click.BadParameter(
+            f"Invalid volume size: '{value}'. Expected an integer,"
+            " optionally with a G/GB/GiB suffix."
+        ) from exc
+    if as_int <= 0:
+        raise click.BadParameter(
+            f"Volume size must be a positive integer, got '{value}'."
+        )
+    return str(as_int)
+
+
 def validate_port(value: int) -> None:
     if not isinstance(value, int) or value < 1 or value > 65535:
         raise click.BadParameter(

{remo_cli-1.0.0 → remo_cli-1.0.2}/src/remo_cli/providers/aws.py

@@ -27,7 +27,7 @@ from remo_cli.core.known_hosts import (
 )
 from remo_cli.core.output import confirm, print_error, print_info, print_success, print_warning
 from remo_cli.core.ssh import detect_timezone, require_session_manager_plugin
-from remo_cli.core.validation import build_tool_args, validate_name
+from remo_cli.core.validation import build_tool_args, parse_volume_size, validate_name
 from remo_cli.core.version import get_current_version
 from remo_cli.models.host import KnownHost
 
@@ -388,6 +388,7 @@ def create(
     """
     if name:
         validate_name(name, "instance name")
+    volume_size = parse_volume_size(volume_size)
 
     print_info("Creating AWS EC2 instance...")
 
@@ -558,6 +559,7 @@ def update(
     """
     if name:
         validate_name(name, "instance name")
+    volume_size = parse_volume_size(volume_size)
 
     resource_name = name or os.environ.get("USER", "remo")
     region = get_aws_region(resource_name)

{remo_cli-1.0.0 → remo_cli-1.0.2}/src/remo_cli/providers/hetzner.py

@@ -22,7 +22,7 @@ from remo_cli.core.known_hosts import (
 )
 from remo_cli.core.output import confirm, print_error, print_info, print_success, print_warning
 from remo_cli.core.ssh import detect_timezone
-from remo_cli.core.validation import build_tool_args, validate_name
+from remo_cli.core.validation import build_tool_args, parse_volume_size, validate_name
 from remo_cli.core.version import get_current_version
 from remo_cli.models.host import KnownHost
 
@@ -94,6 +94,7 @@ def create(
     """
     if name:
         validate_name(name, "server name")
+    volume_size = parse_volume_size(volume_size)
 
     print_info("Creating Hetzner VM...")
 
@@ -215,6 +216,7 @@ def update(
     """
     if name:
         validate_name(name, "server name")
+    volume_size = parse_volume_size(volume_size)
 
     server_name = name or "remo"
 

{remo_cli-1.0.0 → remo_cli-1.0.2}/src/remo_cli/providers/incus.py

@@ -20,7 +20,7 @@ from remo_cli.core.known_hosts import (
 )
 from remo_cli.core.output import confirm, print_error, print_info, print_warning
 from remo_cli.core.ssh import detect_timezone
-from remo_cli.core.validation import build_tool_args, validate_name
+from remo_cli.core.validation import build_tool_args, parse_volume_size, validate_name
 from remo_cli.core.version import get_current_version
 from remo_cli.models.host import KnownHost
 
@@ -172,6 +172,7 @@ def create(
     Returns the ansible-playbook exit code (0 on success).
     """
     validate_name(name, "container name")
+    volume_size = parse_volume_size(volume_size)
 
     print_info(f"Creating Incus container '{name}'...")
 
@@ -304,6 +305,7 @@ def update(
     Returns the ansible-playbook exit code (0 on success).
     """
     validate_name(name, "container name")
+    volume_size = parse_volume_size(volume_size)
 
     # If --host not specified, look up container in known_hosts.
     if not host:

{remo_cli-1.0.0 → remo_cli-1.0.2}/src/remo_cli/providers/proxmox.py

@@ -26,7 +26,7 @@ from remo_cli.core.known_hosts import (
 )
 from remo_cli.core.output import confirm, print_error, print_info, print_warning
 from remo_cli.core.ssh import detect_timezone
-from remo_cli.core.validation import build_tool_args, validate_name
+from remo_cli.core.validation import build_tool_args, parse_volume_size, validate_name
 from remo_cli.core.version import get_current_version
 from remo_cli.models.host import KnownHost
 
@@ -187,6 +187,7 @@ def create(
     Returns the ansible-playbook exit code (0 on success).
     """
     validate_name(name, "container name")
+    volume_size = parse_volume_size(volume_size)
 
     if not host:
         print_error("Proxmox host is required (use --host).")
@@ -350,6 +351,7 @@ def update(
     Returns the ansible-playbook exit code (0 on success).
     """
     validate_name(name, "container name")
+    volume_size = parse_volume_size(volume_size)
 
     vmid = ""
     if not host:

{remo_cli-1.0.0 → remo_cli-1.0.2}/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.0 → remo_cli-1.0.2}/tests/unit/core/test_validation.py

@@ -8,6 +8,7 @@ import click
 from remo_cli.core.validation import (
     ALL_TOOLS,
     build_tool_args,
+    parse_volume_size,
     validate_name,
     validate_port,
     validate_region,
@@ -259,3 +260,32 @@ class TestBuildToolArgs:
         args = build_tool_args(only=ALL_TOOLS, skip=())
         for tool in ALL_TOOLS:
             assert f"configure_{tool}=true" in args
+
+
+class TestParseVolumeSize:
+    """Tests for parse_volume_size()."""
+
+    def test_plain_integer(self):
+        assert parse_volume_size("100") == "100"
+
+    def test_empty_passes_through(self):
+        assert parse_volume_size("") == ""
+
+    @pytest.mark.parametrize("value", ["100G", "100GB", "100GiB", "100gb", "100gib"])
+    def test_strips_size_suffix(self, value):
+        assert parse_volume_size(value) == "100"
+
+    def test_strips_whitespace(self):
+        assert parse_volume_size("  100G  ") == "100"
+
+    def test_rejects_non_numeric(self):
+        with pytest.raises(click.BadParameter):
+            parse_volume_size("big")
+
+    def test_rejects_zero(self):
+        with pytest.raises(click.BadParameter):
+            parse_volume_size("0")
+
+    def test_rejects_negative(self):
+        with pytest.raises(click.BadParameter):
+            parse_volume_size("-5")

{remo_cli-1.0.0 → remo_cli-1.0.2}/uv.lock

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