agentirc-cli 5.0.1__tar.gz → 5.0.2__tar.gz

{agentirc_cli-5.0.1 → agentirc_cli-5.0.2}/CHANGELOG.md

@@ -4,6 +4,14 @@ All notable changes to this project will be documented in this file.
 
 Format follows [Keep a Changelog](https://keepachangelog.com/).
 
+## [5.0.2] - 2026-04-09
+
+
+### Fixed
+
+- Handle missing credential tool (secret-tool/security/powershell) gracefully instead of crashing the server
+- Report restart failures in mesh update instead of claiming success
+
 ## [5.0.1] - 2026-04-09
 
 

{agentirc_cli-5.0.1 → agentirc_cli-5.0.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentirc-cli
-Version: 5.0.1
+Version: 5.0.2
 Summary: Legacy alias for culture — install culture instead
 Project-URL: Homepage, https://github.com/OriNachum/culture
 Author: Ori Nachum

{agentirc_cli-5.0.1 → agentirc_cli-5.0.2}/culture/cli/mesh.py

@@ -469,16 +469,18 @@ def _upgrade_culture_package(args: argparse.Namespace) -> bool:
         os.execvp(culture_bin, reexec_args)
 
 
-def _wait_for_server_port(port: int, retries: int = 50, interval: float = 0.1) -> None:
-    """Poll until *port* accepts a TCP connection."""
+def _wait_for_server_port(host: str, port: int, retries: int = 50, interval: float = 0.1) -> bool:
+    """Poll until *host*:*port* accepts a TCP connection. Returns True on success."""
     import socket as _socket
 
+    probe = "localhost" if host in ("0.0.0.0", "::", "") else host
     for _ in range(retries):
         try:
-            with _socket.create_connection(("localhost", port), timeout=1):
-                return
+            with _socket.create_connection((probe, port), timeout=1):
+                return True
         except OSError:
             time.sleep(interval)
+    return False
 
 
 def _dry_run_restart(mesh, server_name: str) -> None:
@@ -509,13 +511,16 @@ def _restart_single_service(svc_name: str, fallback_cmd: list[str], restart_serv
 
 def _restart_mesh_services(
     mesh, server_name: str, culture_bin: str, config_path: str, dry_run: bool
-) -> None:
-    """Stop agents and server, regenerate service entries, then restart everything."""
+) -> bool:
+    """Stop agents and server, regenerate service entries, then restart everything.
+
+    Returns True if the server came up successfully, False otherwise.
+    """
     print(f"Restarting mesh node '{server_name}'...")
 
     if dry_run:
         _dry_run_restart(mesh, server_name)
-        return
+        return True
 
     for agent in mesh.agents:
         full_nick = f"{server_name}-{agent.nick}"
@@ -561,7 +566,17 @@ def _restart_mesh_services(
     ]
     _restart_single_service(server_svc, server_fallback, restart_service)
 
-    _wait_for_server_port(mesh.server.port)
+    if not _wait_for_server_port(mesh.server.host, mesh.server.port):
+        if sys.platform == "linux":
+            hint = f"journalctl --user -u culture-server-{server_name}"
+        else:
+            hint = f"~/.culture/logs/server-{server_name}.log"
+        print(
+            f"  Error: server {server_name} did not start on port {mesh.server.port}. "
+            f"Check logs: {hint}",
+            file=sys.stderr,
+        )
+        return False
 
     for agent in mesh.agents:
         full_nick = f"{server_name}-{agent.nick}"
@@ -579,6 +594,7 @@ def _restart_mesh_services(
         _restart_single_service(agent_svc, agent_fallback, restart_service)
 
     print()
+    return True
 
 
 def _resolve_mesh_for_server(server_name: str, config_path: str):
@@ -630,6 +646,7 @@ def _cmd_update(args: argparse.Namespace) -> None:
     culture_bin = shutil.which("culture") or "culture"
 
     running = list_servers()
+    failed = []
 
     if running:
         for srv in running:
@@ -640,7 +657,10 @@ def _cmd_update(args: argparse.Namespace) -> None:
                     file=sys.stderr,
                 )
                 continue
-            _restart_mesh_services(mesh, srv["name"], culture_bin, args.config, args.dry_run)
+            if not _restart_mesh_services(
+                mesh, srv["name"], culture_bin, args.config, args.dry_run
+            ):
+                failed.append(srv["name"])
     else:
         try:
             mesh = load_mesh_config(args.config)
@@ -648,6 +668,19 @@ def _cmd_update(args: argparse.Namespace) -> None:
             mesh = generate_mesh_from_agents(args.config)
             if mesh is None:
                 sys.exit(1)
-        _restart_mesh_services(mesh, mesh.server.name, culture_bin, args.config, args.dry_run)
+        if not _restart_mesh_services(
+            mesh, mesh.server.name, culture_bin, args.config, args.dry_run
+        ):
+            failed.append(mesh.server.name)
+
+    if failed:
+        print(
+            f"Update finished with errors. Failed to restart: {', '.join(failed)}",
+            file=sys.stderr,
+        )
+        sys.exit(1)
 
-    print("Update complete. All services restarted.")
+    if args.dry_run:
+        print("Dry run complete. No services were restarted.")
+    else:
+        print("Update complete. All services restarted.")

{agentirc_cli-5.0.1 → agentirc_cli-5.0.2}/culture/credentials.py

@@ -21,12 +21,17 @@ SERVICE_NAME = "culture"
 
 def _run(args: list[str], input: str | None = None) -> tuple[int, str]:
     """Run a command and return (returncode, stdout)."""
-    result = subprocess.run(
-        args,
-        input=input,
-        capture_output=True,
-        text=True,
-    )
+    try:
+        result = subprocess.run(
+            args,
+            input=input,
+            capture_output=True,
+            text=True,
+        )
+    except FileNotFoundError:
+        tool = {"darwin": "security", "win32": "powershell"}.get(sys.platform, "secret-tool")
+        logger.warning("Credential tool '%s' not found — is it installed?", tool)
+        return 127, ""
     return result.returncode, result.stdout.strip()
 
 

{agentirc_cli-5.0.1 → agentirc_cli-5.0.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "agentirc-cli"
-version = "5.0.1"
+version = "5.0.2"
 description = "Legacy alias for culture — install culture instead"
 readme = "README.md"
 license = "MIT"

agentirc_cli-5.0.2/tests/test_credentials.py

@@ -0,0 +1,24 @@
+"""Tests for culture.credentials — OS credential store helpers."""
+
+from unittest.mock import patch
+
+from culture.credentials import _run, lookup_credential, store_credential
+
+
+def test_run_missing_binary():
+    """_run() returns (127, '') when the binary is not found."""
+    rc, out = _run(["nonexistent-binary-xyz-12345"])
+    assert rc == 127
+    assert out == ""
+
+
+def test_lookup_credential_missing_tool():
+    """lookup_credential() returns None when the credential tool is missing."""
+    with patch("culture.credentials._run", return_value=(127, "")):
+        assert lookup_credential("some-peer") is None
+
+
+def test_store_credential_missing_tool():
+    """store_credential() returns False when the credential tool is missing."""
+    with patch("culture.credentials._run", return_value=(127, "")):
+        assert store_credential("some-peer", "password") is False

{agentirc_cli-5.0.1 → agentirc_cli-5.0.2}/uv.lock

@@ -561,7 +561,7 @@ wheels = [
 
 [[package]]
 name = "culture"
-version = "5.0.1"
+version = "5.0.2"
 source = { editable = "." }
 dependencies = [
     { name = "aiohttp" },