msdev 0.13.0__tar.gz → 0.14.0__tar.gz

{msdev-0.13.0 → msdev-0.14.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: msdev
-Version: 0.13.0
+Version: 0.14.0
 Summary: Portable nodes and execution environments for msModelSlim development
 Author: msModelSlim Agent Contributors
 License-Expression: MulanPSL-2.0
@@ -104,6 +104,7 @@ key, and deploys `msdevd`:
 ```bash
 msdev node connect dev-122 --hostname 80.48.33.133 --user root --password PASSWORD
 msdev node connect dev-122
+msdev node bootstrap dev-122
 msdev node sync-ssh
 msdev node list --ssh
 msdev node list
@@ -116,8 +117,9 @@ config. `node sync-ssh` registers Host aliases that were added manually.
 
 Connect atomically creates a same-named host Env on first use and uploads a
 **static Go msdevd** binary matching the node architecture
-(`linux-amd64` / `linux-arm64`) when the agent is missing or stale. No Python
-is required on the node.
+(`linux-amd64` / `linux-arm64`) when the agent is missing or when the remote
+`ping` version does not match the local CLI. Use **`msdev node bootstrap NODE`**
+to force a redeploy after upgrading msdev. No Python is required on the node.
 
 Build the static daemons before release or local testing:
 

{msdev-0.13.0 → msdev-0.14.0}/README.md

@@ -79,6 +79,7 @@ key, and deploys `msdevd`:
 ```bash
 msdev node connect dev-122 --hostname 80.48.33.133 --user root --password PASSWORD
 msdev node connect dev-122
+msdev node bootstrap dev-122
 msdev node sync-ssh
 msdev node list --ssh
 msdev node list
@@ -91,8 +92,9 @@ config. `node sync-ssh` registers Host aliases that were added manually.
 
 Connect atomically creates a same-named host Env on first use and uploads a
 **static Go msdevd** binary matching the node architecture
-(`linux-amd64` / `linux-arm64`) when the agent is missing or stale. No Python
-is required on the node.
+(`linux-amd64` / `linux-arm64`) when the agent is missing or when the remote
+`ping` version does not match the local CLI. Use **`msdev node bootstrap NODE`**
+to force a redeploy after upgrading msdev. No Python is required on the node.
 
 Build the static daemons before release or local testing:
 

{msdev-0.13.0 → msdev-0.14.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "msdev"
-version = "0.13.0"
+version = "0.14.0"
 description = "Portable nodes and execution environments for msModelSlim development"
 readme = "README.md"
 requires-python = ">=3.10"

{msdev-0.13.0 → msdev-0.14.0}/src/msdev/__init__.py

@@ -1,3 +1,3 @@
 """Portable node and execution-environment tooling for msModelSlim."""
 
-__version__ = "0.13.0"
+__version__ = "0.14.0"

{msdev-0.13.0 → msdev-0.14.0}/src/msdev/cli.py

@@ -240,6 +240,10 @@ def command_node(
         print(f"node: {args.name}")
         _print_json(service.status(args.name))
         return 0
+    if action == "bootstrap":
+        result = service.bootstrap(args.name, package_path=args.package)
+        _print_json(result)
+        return 0
     if action in {"connect", "disconnect"}:
         provision = None
         if action == "connect" and args.name and not getattr(args, "all", False):
@@ -613,6 +617,20 @@ def build_parser() -> argparse.ArgumentParser:
     )
     node_status = node_sub.add_parser("status")
     node_status.add_argument("name")
+    node_bootstrap = node_sub.add_parser(
+        "bootstrap",
+        help=(
+            "force-redeploy msdevd on the node "
+            "(use after local upgrade when remote version mismatches)"
+        ),
+    )
+    node_bootstrap.add_argument("name")
+    node_bootstrap.add_argument(
+        "--package",
+        type=Path,
+        default=None,
+        help="optional source checkout for source-layout bootstrap",
+    )
     node_connect = node_sub.add_parser(
         "connect",
         help=(

{msdev-0.13.0 → msdev-0.14.0}/src/msdev/core/services/node.py

@@ -6,6 +6,7 @@ from dataclasses import asdict, dataclass
 from pathlib import Path
 from typing import Any
 
+from ... import __version__
 from ..resources import Node
 from ..ssh_config import (
     SshHostSpec,
@@ -18,6 +19,19 @@ from ..transport import DaemonUnavailableError, RpcError
 from .context import ServiceContext
 
 
+def remote_daemon_version_matches(
+    ping: Any,
+    *,
+    expected: str | None = None,
+) -> bool:
+    """Return True when ping reports the same msdevd version as the local CLI."""
+    target = expected if expected is not None else __version__
+    if not isinstance(ping, dict):
+        return False
+    remote = ping.get("version")
+    return isinstance(remote, str) and remote == target
+
+
 @dataclass(frozen=True)
 class NodeAddRequest:
     name: str
@@ -220,29 +234,52 @@ class NodeService:
         name: str,
         package_path: Path | None = None,
     ) -> dict[str, Any]:
+        """Force-redeploy msdevd on the node (ignores current ping/version)."""
         node = self.context.node_store.get(self._required(name, "node name"))
-        return self._ensure_agent_ready(node, package_path=package_path)
+        return self._ensure_agent_ready(
+            node,
+            package_path=package_path,
+            force=True,
+        )
 
     def _ensure_agent_ready(
         self,
         node: Node,
         *,
         package_path: Path | None = None,
+        force: bool = False,
     ) -> dict[str, Any]:
         transport = self.context.create_transport(node)
-        try:
-            ping = transport.call("ping")
-            if isinstance(ping, dict):
-                return {"ready": True, "bootstrapped": False, "ping": ping}
-            return {"ready": True, "bootstrapped": False}
-        except DaemonUnavailableError:
-            if not node.auto_bootstrap:
-                raise RpcError(
-                    f"msdevd is not ready on node {node.name!r}; "
-                    "enable auto-bootstrap on the node or fix the host agent"
-                ) from None
-            result = self.context.bootstrap(node, package_path)
-            return {"ready": True, "bootstrapped": True, **result}
+        reason = "forced" if force else None
+        if not force:
+            try:
+                ping = transport.call("ping")
+                if remote_daemon_version_matches(ping):
+                    return {
+                        "ready": True,
+                        "bootstrapped": False,
+                        "reason": "version_match",
+                        "ping": ping if isinstance(ping, dict) else None,
+                    }
+                if isinstance(ping, dict):
+                    reason = "version_mismatch"
+                else:
+                    reason = "unavailable"
+            except DaemonUnavailableError:
+                reason = "unavailable"
+                if not node.auto_bootstrap:
+                    raise RpcError(
+                        f"msdevd is not ready on node {node.name!r}; "
+                        "enable auto-bootstrap on the node, or run "
+                        f"`msdev node bootstrap {node.name}`"
+                    ) from None
+        result = self.context.bootstrap(node, package_path)
+        return {
+            "ready": True,
+            "bootstrapped": True,
+            "reason": reason or "unavailable",
+            **result,
+        }
 
     def _connection_nodes(
         self,

{msdev-0.13.0 → msdev-0.14.0}/src/msdev/core/sftp_transfer.py

@@ -3,10 +3,13 @@
 from __future__ import annotations
 
 import subprocess
-from pathlib import Path
+from pathlib import Path, PureWindowsPath
 from typing import Any
 
-from .compat import popen_kwargs
+from .compat import IS_WINDOWS, popen_kwargs
+
+# Characters that need quoting in OpenSSH sftp batch arguments.
+_SFTP_SPECIAL = frozenset(' \t"\'*?[]{}\\')
 
 
 def _run_kw(env: dict[str, str] | None = None) -> dict[str, Any]:
@@ -16,6 +19,35 @@ def _run_kw(env: dict[str, str] | None = None) -> dict[str, Any]:
     return kwargs
 
 
+def format_sftp_local_path(path: Path) -> str:
+    """Format a local path for an OpenSSH ``sftp -b`` argument.
+
+    Windows backslashes are treated as escapes by sftp batch mode, so convert
+    drive paths to forward slashes (``C:/Users/...``). Quote when spaces or
+    other special characters remain.
+    """
+    resolved = path.expanduser().resolve()
+    text = str(resolved)
+    if IS_WINDOWS or "\\" in text:
+        # PureWindowsPath.as_posix keeps the drive letter and uses '/'.
+        text = PureWindowsPath(text).as_posix()
+    return _quote_sftp_arg(text)
+
+
+def format_sftp_remote_path(path: str) -> str:
+    """Format a remote POSIX path for an OpenSSH ``sftp -b`` argument."""
+    return _quote_sftp_arg(path)
+
+
+def _quote_sftp_arg(text: str) -> str:
+    if not text:
+        return '""'
+    if any(ch in _SFTP_SPECIAL for ch in text):
+        escaped = text.replace("\\", "\\\\").replace('"', '\\"')
+        return f'"{escaped}"'
+    return text
+
+
 def run_sftp_batch(
     sftp_argv: list[str],
     ssh_host: str,
@@ -60,8 +92,9 @@ def sftp_put_file(
     timeout: float | None,
     env: dict[str, str] | None = None,
 ) -> None:
-    local = str(local_path.resolve())
-    batch = f"put {local} {remote_path}\n"
+    local = format_sftp_local_path(local_path)
+    remote = format_sftp_remote_path(remote_path)
+    batch = f"put {local} {remote}\n"
     run_sftp_batch(
         sftp_argv,
         ssh_host,
@@ -82,8 +115,9 @@ def sftp_get_file(
     timeout: float | None,
     env: dict[str, str] | None = None,
 ) -> None:
-    local = str(local_path.expanduser().resolve())
-    batch = f"get {remote_path} {local}\n"
+    local = format_sftp_local_path(local_path)
+    remote = format_sftp_remote_path(remote_path)
+    batch = f"get {remote} {local}\n"
     run_sftp_batch(
         sftp_argv,
         ssh_host,
@@ -105,10 +139,12 @@ def sftp_put_rename(
     timeout: float | None,
     env: dict[str, str] | None = None,
 ) -> None:
-    local = str(local_path.resolve())
+    local = format_sftp_local_path(local_path)
+    staging = format_sftp_remote_path(staging_path)
+    destination = format_sftp_remote_path(destination_path)
     batch = (
-        f"put {local} {staging_path}\n"
-        f"rename {staging_path} {destination_path}\n"
+        f"put {local} {staging}\n"
+        f"rename {staging} {destination}\n"
     )
     run_sftp_batch(
         sftp_argv,

msdev-0.14.0/src/msdev/helpers/msdevd-linux-amd64.sha256

@@ -0,0 +1 @@
+7b3d2dcd31cc752b44289c5b0936569be7df1aef162d07c1b3c00e886cfea884  msdevd-linux-amd64

msdev-0.14.0/src/msdev/helpers/msdevd-linux-arm64.sha256

@@ -0,0 +1 @@
+aeb5721e49746058e286ed14621208823ee092372c0a7d772cfdd9924d5b71e1  msdevd-linux-arm64

{msdev-0.13.0 → msdev-0.14.0}/src/msdev/skills/msdev-cli/SKILL.md

@@ -56,6 +56,7 @@ msdev node sync-ssh
 msdev node list --ssh
 msdev node connect NODE
 msdev node connect NODE --identity FILE
+msdev node bootstrap NODE
 msdev node status NODE
 msdev node disconnect NODE
 msdev node remove NODE
@@ -96,7 +97,9 @@ registers existing Host blocks.
 
 `msdev node connect` opens SSH persistence, saves `--password` into SSH config,
 installs the local public key when a password is available, and deploys/starts
-`msdevd` when needed. Users do not run a separate add or bootstrap step.
+`msdevd` when missing or when the remote `ping` version does not match the
+local CLI. Use `msdev node bootstrap NODE` to force a redeploy after upgrading
+msdev (version mismatch is not always visible as a hard failure).
 
 `msdev env open` runs the env `setup` script once and establishes the shared
 session. `msdev exec` opens the env automatically when needed.
@@ -178,5 +181,6 @@ captures one result. Local cancellation terminates the remote process group.
 
 - Ask before destructive or externally visible changes unless authorized.
 - Track temporary resources and remove only those created by the task.
-- If `msdevd` is missing or stale, run `msdev node connect NODE`, then retry.
+- If `msdevd` is missing or its version mismatches the local CLI, run
+  `msdev node bootstrap NODE` (or `msdev node connect NODE`), then retry.
 - If connect fails, report the error; do not bypass msdev with another transport.

{msdev-0.13.0 → msdev-0.14.0}/src/msdev.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: msdev
-Version: 0.13.0
+Version: 0.14.0
 Summary: Portable nodes and execution environments for msModelSlim development
 Author: msModelSlim Agent Contributors
 License-Expression: MulanPSL-2.0
@@ -104,6 +104,7 @@ key, and deploys `msdevd`:
 ```bash
 msdev node connect dev-122 --hostname 80.48.33.133 --user root --password PASSWORD
 msdev node connect dev-122
+msdev node bootstrap dev-122
 msdev node sync-ssh
 msdev node list --ssh
 msdev node list
@@ -116,8 +117,9 @@ config. `node sync-ssh` registers Host aliases that were added manually.
 
 Connect atomically creates a same-named host Env on first use and uploads a
 **static Go msdevd** binary matching the node architecture
-(`linux-amd64` / `linux-arm64`) when the agent is missing or stale. No Python
-is required on the node.
+(`linux-amd64` / `linux-arm64`) when the agent is missing or when the remote
+`ping` version does not match the local CLI. Use **`msdev node bootstrap NODE`**
+to force a redeploy after upgrading msdev. No Python is required on the node.
 
 Build the static daemons before release or local testing:
 

{msdev-0.13.0 → msdev-0.14.0}/src/msdev.egg-info/SOURCES.txt

@@ -81,6 +81,7 @@ tests/test_model_cli.py
 tests/test_model_services.py
 tests/test_msdev_cli_skill.py
 tests/test_multiplex.py
+tests/test_node_bootstrap_version_scenario.py
 tests/test_node_connect_provision_scenario.py
 tests/test_node_connect_save_password_scenario.py
 tests/test_node_env_config.py

{msdev-0.13.0 → msdev-0.14.0}/tests/test_cli.py

@@ -307,6 +307,27 @@ class CliParserTest(unittest.TestCase):
                 {"node_id": "node-1"},
                 'node: worker\n{\n  "node_id": "node-1"\n}\n',
             ),
+            (
+                ["node", "bootstrap", "worker"],
+                "bootstrap",
+                {
+                    "ready": True,
+                    "bootstrapped": True,
+                    "reason": "forced",
+                    "daemon": {"ok": True, "version": "0.14.0"},
+                },
+                (
+                    "{\n"
+                    '  "bootstrapped": true,\n'
+                    '  "daemon": {\n'
+                    '    "ok": true,\n'
+                    '    "version": "0.14.0"\n'
+                    "  },\n"
+                    '  "ready": true,\n'
+                    '  "reason": "forced"\n'
+                    "}\n"
+                ),
+            ),
             (
                 ["node", "connect", "worker"],
                 "connect",

{msdev-0.13.0 → msdev-0.14.0}/tests/test_cli_only_architecture.py

@@ -30,8 +30,13 @@ class CliOnlyArchitectureTests(unittest.TestCase):
                 parser.parse_args(["npu", "list", "--node", "worker"])
             with self.assertRaises(SystemExit):
                 parser.parse_args(["model", "list", "--node", "worker"])
-            with self.assertRaises(SystemExit):
-                parser.parse_args(["node", "bootstrap", "worker"])
+
+    def test_node_bootstrap_is_present(self):
+        args = build_parser().parse_args(["node", "bootstrap", "worker"])
+        self.assertEqual(args.command, "node")
+        self.assertEqual(args.node_command, "bootstrap")
+        self.assertEqual(args.name, "worker")
+        self.assertIsNone(args.package)
 
     def test_skill_command_is_absent(self):
         parser = build_parser()

{msdev-0.13.0 → msdev-0.14.0}/tests/test_config_bundle.py

@@ -56,7 +56,7 @@ class ConfigBundleTests(unittest.TestCase):
             summary = export_bundle(
                 export_dir,
                 config_path=resources,
-                msdev_version="0.13.0",
+                msdev_version="0.14.0",
             )
             self.assertFalse(summary["archive"])
             self.assertTrue((export_dir / BUNDLE_MANIFEST).is_file())
@@ -100,7 +100,7 @@ class ConfigBundleTests(unittest.TestCase):
             )
 
             archive = root / "msdev-config.tar.gz"
-            export_bundle(archive, config_path=resources, msdev_version="0.13.0")
+            export_bundle(archive, config_path=resources, msdev_version="0.14.0")
             self.assertTrue(archive.is_file())
 
             target_resources = root / "imported" / "resources.json"
@@ -126,7 +126,7 @@ class ConfigBundleTests(unittest.TestCase):
             )
 
             bundle_dir = root / "bundle"
-            export_bundle(bundle_dir, config_path=source_resources, msdev_version="0.13.0")
+            export_bundle(bundle_dir, config_path=source_resources, msdev_version="0.14.0")
             result = import_bundle(bundle_dir, config_path=target_resources)
             self.assertEqual(result["imported_nodes"], 0)
             self.assertIn("node:worker", result["skipped"])

{msdev-0.13.0 → msdev-0.14.0}/tests/test_daemon.py

@@ -683,7 +683,7 @@ class RpcDispatcherTest(unittest.TestCase):
                 started = time.monotonic()
                 first = json.loads(proxy.stdout.readline())
                 self.assertEqual(first["event"], "stdout")
-                self.assertLess(time.monotonic() - started, 0.3)
+                self.assertLess(time.monotonic() - started, 1.0)
                 events = [first]
                 while events[-1]["event"] != "result":
                     events.append(json.loads(proxy.stdout.readline()))

{msdev-0.13.0 → msdev-0.14.0}/tests/test_go_msdevd_scenario.py

@@ -104,7 +104,7 @@ class GoMsdevdScenarioTests(unittest.TestCase):
         self.assertNotIn("error", response)
         result = response["result"]
         self.assertTrue(result["ok"])
-        self.assertEqual(result["version"], "0.13.0")
+        self.assertEqual(result["version"], "0.14.0")
         self.assertIn("node_id", result)
 
     def test_exec(self):

{msdev-0.13.0 → msdev-0.14.0}/tests/test_msdev_cli_skill.py

@@ -45,9 +45,9 @@ class MsdevCliSkillTests(unittest.TestCase):
         self.assertIn("msdev env open", content)
         self.assertIn("msdev exec ENV", content)
         self.assertIn("msdev node connect", content)
+        self.assertIn("msdev node bootstrap", content)
         self.assertIn("msdev node guide NODE", content)
         self.assertIn("msdev env shell ENV", content)
-        self.assertNotIn("msdev node bootstrap", content)
         self.assertNotIn("msdev model list", content)
         self.assertNotIn("msdev npu list", content)
         self.assertIn("Do not use MCP", content)
@@ -61,6 +61,7 @@ class MsdevCliSkillTests(unittest.TestCase):
         required_commands = (
             "msdev node connect",
             "msdev node connect NODE --hostname",
+            "msdev node bootstrap NODE",
             "msdev node remove",
             "msdev env add",
             "msdev env open",

msdev-0.14.0/tests/test_node_bootstrap_version_scenario.py

@@ -0,0 +1,173 @@
+#!/usr/bin/env python3
+"""Scenario: CLI node bootstrap force-redeploys; connect matches ping version."""
+
+from __future__ import annotations
+
+import json
+import tempfile
+import unittest
+from io import StringIO
+from pathlib import Path
+from unittest.mock import Mock, patch
+
+from msdev import __version__
+from msdev.cli import build_parser, command_node
+from msdev.core.resources import EnvironmentStore, Node, NodeStore
+from msdev.core.services.context import ServiceContext
+from msdev.core.services.node import NodeService, remote_daemon_version_matches
+
+
+class NodeBootstrapVersionScenarioTests(unittest.TestCase):
+    def test_remote_daemon_version_matches_helper(self) -> None:
+        self.assertTrue(
+            remote_daemon_version_matches(
+                {"ok": True, "version": __version__},
+            )
+        )
+        self.assertFalse(
+            remote_daemon_version_matches(
+                {"ok": True, "version": "0.0.0"},
+            )
+        )
+        self.assertFalse(remote_daemon_version_matches({"ok": True}))
+        self.assertFalse(remote_daemon_version_matches("not-a-dict"))
+
+    def test_cli_bootstrap_force_redeploys_even_when_ping_succeeds(self) -> None:
+        with tempfile.TemporaryDirectory() as temp_dir:
+            root = Path(temp_dir)
+            resources = root / "resources.json"
+            node_store = NodeStore(resources)
+            node_store.add(
+                Node(name="worker", ssh_host="worker", auto_bootstrap=True)
+            )
+            environment_store = EnvironmentStore(resources)
+
+            bootstrap_result = {
+                "ready": True,
+                "bootstrapped": True,
+                "reason": "forced",
+                "daemon": {
+                    "ok": True,
+                    "version": __version__,
+                    "node_id": "n1",
+                },
+                "layout": "static",
+            }
+            args = build_parser().parse_args(
+                ["--config", str(resources), "node", "bootstrap", "worker"]
+            )
+            stdout = StringIO()
+            with (
+                patch("msdev.cli.NodeService") as service_cls,
+                patch("sys.stdout", stdout),
+            ):
+                service = service_cls.return_value
+                service.bootstrap.return_value = bootstrap_result
+                code = command_node(args, node_store, environment_store)
+
+            self.assertEqual(code, 0)
+            service.bootstrap.assert_called_once_with("worker", package_path=None)
+            payload = json.loads(stdout.getvalue())
+            self.assertTrue(payload["bootstrapped"])
+            self.assertEqual(payload["reason"], "forced")
+            self.assertEqual(payload["daemon"]["version"], __version__)
+
+    def test_connect_redeploys_when_remote_version_mismatches(self) -> None:
+        with tempfile.TemporaryDirectory() as temp_dir:
+            root = Path(temp_dir)
+            resources = root / "resources.json"
+            node = Node(name="worker", ssh_host="worker", auto_bootstrap=True)
+            node_store = NodeStore(resources)
+            node_store.add(node)
+            environment_store = EnvironmentStore(resources)
+
+            transport = Mock()
+            transport.connect.return_value = {
+                "connected": True,
+                "already_connected": False,
+                "persist": "yes",
+            }
+            transport.call.return_value = {
+                "ok": True,
+                "version": "0.12.0",
+                "node_id": "old",
+            }
+            bootstrap = Mock(
+                return_value={
+                    "daemon": {
+                        "ok": True,
+                        "version": __version__,
+                        "node_id": "new",
+                    },
+                    "layout": "static",
+                }
+            )
+            context = ServiceContext(
+                node_store=node_store,
+                environment_store=environment_store,
+                create_transport=Mock(return_value=transport),
+                bootstrap=bootstrap,
+            )
+
+            args = build_parser().parse_args(
+                ["--config", str(resources), "node", "connect", "worker"]
+            )
+            stdout = StringIO()
+            with (
+                patch("msdev.cli._service_context", return_value=context),
+                patch("msdev.cli.NodeService", NodeService),
+                patch("sys.stdout", stdout),
+            ):
+                code = command_node(args, node_store, environment_store)
+
+            self.assertEqual(code, 0, stdout.getvalue())
+            bootstrap.assert_called_once_with(node, None)
+            self.assertIn("agent=ready", stdout.getvalue())
+            transport.call.assert_called_with("ping")
+
+    def test_connect_skips_redeploy_when_version_matches(self) -> None:
+        with tempfile.TemporaryDirectory() as temp_dir:
+            root = Path(temp_dir)
+            resources = root / "resources.json"
+            node = Node(name="worker", ssh_host="worker", auto_bootstrap=True)
+            node_store = NodeStore(resources)
+            node_store.add(node)
+            environment_store = EnvironmentStore(resources)
+
+            transport = Mock()
+            transport.connect.return_value = {
+                "connected": True,
+                "already_connected": False,
+                "persist": "yes",
+            }
+            transport.call.return_value = {
+                "ok": True,
+                "version": __version__,
+                "node_id": "n1",
+            }
+            bootstrap = Mock()
+            context = ServiceContext(
+                node_store=node_store,
+                environment_store=environment_store,
+                create_transport=Mock(return_value=transport),
+                bootstrap=bootstrap,
+            )
+
+            args = build_parser().parse_args(
+                ["--config", str(resources), "node", "connect", "worker"]
+            )
+            stdout = StringIO()
+            with (
+                patch("msdev.cli._service_context", return_value=context),
+                patch("msdev.cli.NodeService", NodeService),
+                patch("sys.stdout", stdout),
+            ):
+                code = command_node(args, node_store, environment_store)
+
+            self.assertEqual(code, 0, stdout.getvalue())
+            bootstrap.assert_not_called()
+            self.assertIn("agent=ready", stdout.getvalue())
+
+
+if __name__ == "__main__":
+    unittest.main()