agentic-runner 3.28.0__tar.gz → 3.29.0__tar.gz

{agentic_runner-3.28.0 → agentic_runner-3.29.0}/PKG-INFO

@@ -1,11 +1,11 @@
 Metadata-Version: 2.5
 Name: agentic-runner
-Version: 3.28.0
+Version: 3.29.0
 Summary: The Agentic OS Runner: an activity-only Temporal executor for workspaces, Agent Runtimes and verb seams
 License-Expression: AGPL-3.0-only
 License-File: LICENSE
 Requires-Python: >=3.12
-Requires-Dist: agentic-runner-contracts==3.28.0
+Requires-Dist: agentic-runner-contracts==3.29.0
 Requires-Dist: cryptography<51,>=50
 Requires-Dist: httpx<1.0,>=0.27
 Requires-Dist: pydantic-settings<3.0,>=2.0

{agentic_runner-3.28.0 → agentic_runner-3.29.0}/pyproject.toml

@@ -10,7 +10,7 @@ dependencies = [
   # Exactly this release's contracts: the two ship as one version, and an unpinned
   # requirement let `uv tool upgrade` keep an older contracts wheel the Runner cannot
   # import from (QTS-1473). `scripts/set-version.py` moves it with the version.
-  "agentic-runner-contracts==3.28.0",
+  "agentic-runner-contracts==3.29.0",
   # Imported directly by `registration.py` (the Ed25519 identity that signs every
   # heartbeat); already present transitively through pyjwt[crypto], declared because
   # we import it.

{agentic_runner-3.28.0 → agentic_runner-3.29.0}/src/agentic_runner/__init__.py

@@ -7,6 +7,6 @@ stay on the platform; this package imports none of them, and a CI partition test
 (``tests/test_package_partition.py``) fails the build if that changes.
 """
 
-__version__ = "3.28.0"
+__version__ = "3.29.0"
 
 __all__ = ["__version__"]

agentic_runner-3.29.0/src/agentic_runner/organisation_name.py

@@ -0,0 +1,30 @@
+"""The Organisation this Runner serves, by the name the last heartbeat ack gave (QTS-1909).
+
+``agentic-runner list`` is a separate process and reads it from this file, which the
+running Runner rewrites whenever an ack changes it. Absent while the control plane names
+no Organisation -- one below contracts 3.29 -- so ``list`` shows the id instead.
+"""
+
+from __future__ import annotations
+
+from pathlib import Path
+from typing import Final
+
+from agentic_runner.private_state import private_read, private_write
+
+__all__ = ["ORGANISATION_NAME_FILENAME", "load_organisation_name", "save_organisation_name"]
+
+ORGANISATION_NAME_FILENAME: Final[str] = "organisation-name"
+
+
+def save_organisation_name(state_dir: Path, name: str | None) -> None:
+    path = state_dir / ORGANISATION_NAME_FILENAME
+    if name is None:
+        path.unlink(missing_ok=True)
+    else:
+        private_write(path, name.encode("utf-8"))
+
+
+def load_organisation_name(state_dir: Path) -> str | None:
+    raw = private_read(state_dir / ORGANISATION_NAME_FILENAME)
+    return None if raw is None else raw.decode("utf-8", errors="replace")

{agentic_runner-3.28.0 → agentic_runner-3.29.0}/src/agentic_runner/service.py

@@ -33,7 +33,7 @@ from collections.abc import AsyncIterator, Awaitable, Callable, Mapping, Sequenc
 from dataclasses import dataclass, field
 from datetime import UTC, datetime, timedelta
 from pathlib import Path
-from typing import Any, Final, Protocol
+from typing import Any, Final, Literal, Protocol
 from uuid import UUID
 
 import httpx
@@ -85,6 +85,7 @@ from agentic_runner.llm_proxy import (
 from agentic_runner.mcp import ToolServerHealthLog
 from agentic_runner.message_store import MESSAGES_DIR, MessageStore, TemporalWorkflowSignaller
 from agentic_runner.oauth_connectors import OAuthConnectors
+from agentic_runner.organisation_name import save_organisation_name
 from agentic_runner.pool_key import (
     POOL_KEY_ENV,
     POOL_KEY_FILENAME,
@@ -852,6 +853,9 @@ class ControlPlaneStream:
     # QTS-1773: the control plane's contracts version, from its ack; None until an ack from
     # one on 3.27 or later, the first to send it and to parse `RunnerInstall.name`.
     _control_plane_contracts: str | None = None
+    # QTS-1909: the name in the copy `list` reads, None while there is none. "unwritten"
+    # until this process first writes it, so the first ack replaces what a previous one left.
+    _organisation_name: str | None | Literal["unwritten"] = "unwritten"
     _last_seen: tuple[float, datetime] | None = None
 
     def _observe_sleep(self, now: datetime) -> None:
@@ -1081,6 +1085,7 @@ class ControlPlaneStream:
         self._take_pool(ack.pool)
         if ack.contracts_version is not None:
             self._control_plane_contracts = ack.contracts_version
+        self._take_organisation_name(ack.organisation_name)
         await self._sync_llm_slots()
         if ack.runner_token is not None:
             self.runner_token = ack.runner_token
@@ -1139,6 +1144,15 @@ class ControlPlaneStream:
         if self.state_dir is not None:
             save_llm_contracts(self.state_dir, terms)
 
+    def _take_organisation_name(self, name: str | None) -> None:
+        """Keep the copy ``list`` reads when the ack's name changes. Unlike the Contract
+        terms, an ack without one clears it: only the name, never the id, can go stale."""
+
+        if self.state_dir is None or name == self._organisation_name:
+            return
+        save_organisation_name(self.state_dir, name)
+        self._organisation_name = name
+
     def _take_pool(self, pool: PoolTerms | None) -> None:
         """Keep the ack's pool. An ack without one (a personal Runner, or a control plane
         below 3.26) leaves what is held."""

{agentic_runner-3.28.0 → agentic_runner-3.29.0}/src/agentic_runner/workstation.py

@@ -55,7 +55,9 @@ from agentic_runner.config import CONFIG_FILE_ENV
 from agentic_runner.host_store import open_workstation_store
 from agentic_runner.lifecycle import LifecycleOutbox
 from agentic_runner.llm_contracts import load_llm_contracts
+from agentic_runner.organisation_name import load_organisation_name
 from agentic_runner.pool_key import parse_pool_key, save_pool_key
+from agentic_runner.private_state import UnsafeStateError
 from agentic_runner.registration import load_state
 from agentic_runner.service import HEARTBEAT_STAMP_NAME, PIDFILE_NAME
 from agentic_runner.workers.acp_runtime import ACP_BRIDGES
@@ -63,6 +65,7 @@ from agentic_runner_contracts.public_metadata import NAMESPACE_PREFIX
 from agentic_runner_contracts.runner_registration import (
     RUNNER_NAME_PATTERN,
     CliVersion,
+    ContractLlmTerms,
     EgressPosture,
     HostAttestation,
     InstallChannel,
@@ -669,15 +672,18 @@ _LIST_HEADER: Final = (
 def list_lines(root: Path) -> list[str]:
     """``list``: one row per Runner installed here (QTS-1774).
 
-    Ids, not names: the Runner is told neither the Organisation's nor the Contract's name.
-    Eight characters, as the console shows a Runner's id beside its name (Spec: Runners
-    rev 7).
+    The Organisation's and the Contract's names once an ack from a control plane on 3.29
+    named them (QTS-1909); before that, the first eight characters of their ids, as the
+    console shows a Runner's id beside its name (Spec: Runners rev 7).
     """
 
     names = installed_names(root)
     if not names:
         return [f"no workstation Runner is installed under {root}"]
-    rows = [_LIST_HEADER, *(_list_row(RunnerPaths(root=root, name=name)) for name in names)]
+    rows = [
+        _LIST_HEADER,
+        *(_list_row_or_reason(RunnerPaths(root=root, name=name)) for name in names),
+    ]
     widths = [max(len(row[column]) for row in rows) for column in range(len(_LIST_HEADER) - 1)]
     return [
         "  ".join([*(cell.ljust(width) for cell, width in zip(row, widths, strict=False)), row[-1]])
@@ -685,20 +691,37 @@ def list_lines(root: Path) -> list[str]:
     ]
 
 
+def _list_row_or_reason(paths: RunnerPaths) -> tuple[str, str, str, str, str, str]:
+    """One install that cannot be read must not hide the others' rows (QTS-1910).
+
+    The first line only: a pydantic error runs to one line per field, and the row is one.
+    """
+
+    try:
+        return _list_row(paths)
+    except (OSError, ValueError, UnsafeStateError) as error:
+        reason = str(error).splitlines()[0] if str(error) else type(error).__name__
+        return (paths.name, "-", "-", "-", "-", f"unreadable: {reason}")
+
+
 def _list_row(paths: RunnerPaths) -> tuple[str, str, str, str, str, str]:
     state = load_state(paths.state_dir)
     contracts = load_llm_contracts(paths.state_dir)
+    organisation_name = _printable(load_organisation_name(paths.state_dir))
     try:
         version = paths.started_version.read_text(encoding="utf-8").strip() or "-"
     except OSError:
         version = "-"
     return (
         paths.name,
-        state.temporal_namespace.removeprefix(NAMESPACE_PREFIX)[:8] if state else "-",
+        (
+            organisation_name
+            or (state.temporal_namespace.removeprefix(NAMESPACE_PREFIX)[:8] if state else "-")
+        ),
         (
             "-"
             if not contracts
-            else str(contracts[0].contract_id)[:8]
+            else _contract_label(contracts[0])
             if len(contracts) == 1
             else f"{len(contracts)} Contracts"
         ),
@@ -708,6 +731,15 @@ def _list_row(paths: RunnerPaths) -> tuple[str, str, str, str, str, str]:
     )
 
 
+def _contract_label(terms: ContractLlmTerms) -> str:
+    return _printable(terms.contract_name) or str(terms.contract_id)[:8]
+
+
+def _printable(name: str | None) -> str:
+    # A name someone typed in the console: an escape sequence in it would reach the terminal.
+    return "".join(character for character in name or "" if character.isprintable())
+
+
 def next_step_lines(host_party: str) -> list[str]:
     """Where to go once the Runner is live (QTS-1314): the dogfood install ended at
     ``started`` and left the person with no idea what the console wanted next.