@team-agent/installer 0.2.11 → 0.3.1

package/src/team_agent/leader_binding.py

@@ -1,183 +0,0 @@
-"""Family A — positive-source owner binding (0.2.6).
-
-Constitution MUST-11 / MUST-NOT-12: the caller identity for owner binding
-is sourced from the caller-supplied positive facts only:
-
-  1. ``$TMUX_PANE``  (the tmux pane the user invoked the CLI from).
-  2. ``tmux display-message -p -t $TMUX_PANE '#{pane_current_command}'``
-     (a single targeted lookup to confirm the caller pane is hosting a
-     leader CLI).
-
-Reverse enumeration of panes / windows / clients is forbidden. Heuristic
-ranking ("active pane", "current client", "first leader-shaped pane") is
-forbidden. ``$TMUX_PANE`` missing → refuse and emit ``owner.bind_refused``.
-The pane's current command is diagnostic metadata only. Successful binds emit
-``owner.bound_from_caller_pane`` and force-write every owner identity
-field; old fields are not merged or migrated.
-"""
-
-from __future__ import annotations
-
-import hashlib
-import os
-import subprocess
-from datetime import datetime, timezone
-from pathlib import Path
-from typing import Any
-
-from team_agent.events import EventLog
-
-
-def run_cmd(args: list[str], timeout: int = 5) -> subprocess.CompletedProcess[str]:
-    """Tmux command bridge — kept here so contract tests can patch
-    ``team_agent.leader_binding.run_cmd`` to observe the exact tmux call
-    pattern this module makes."""
-    return subprocess.run(
-        args, text=True, capture_output=True, timeout=timeout, check=False
-    )
-
-
-_HINT_RUN_FROM_LEADER_PANE = (
-    "run team-agent from inside your leader pane "
-    "(the tmux pane you want to own this team)."
-)
-
-
-def bind_owner_from_caller_pane(
-    workspace: Path,
-    team_id: str,
-    override_uuid: str | None = None,
-) -> dict[str, Any]:
-    """Derive the owner identity from the caller pane.
-
-    Returns
-    -------
-    On success::
-
-        {
-            "ok": True,
-            "owner": {
-                "pane_id": ..., "leader_session_uuid": ...,
-                "machine_fingerprint": ..., "provider": ...,
-                "os_user": ..., "claimed_at": <iso8601>,
-            },
-            "caller_pane_id": ..., "caller_current_command": ...,
-            "team_id": team_id,
-        }
-
-    On refusal::
-
-        {
-            "ok": False,
-            "reason": "caller_pane_missing",
-            "caller_pane_id": ..., "caller_current_command": ...,
-            "hint": ...,
-        }
-    """
-    event_log = EventLog(workspace)
-    caller_pane = os.environ.get("TMUX_PANE") or ""
-    if not caller_pane:
-        hint = _HINT_RUN_FROM_LEADER_PANE
-        event_log.write(
-            "owner.bind_refused",
-            reason="caller_pane_missing",
-            caller_pane_id="",
-            caller_current_command="",
-            team_id=team_id,
-            hint=hint,
-        )
-        return {
-            "ok": False,
-            "reason": "caller_pane_missing",
-            "caller_pane_id": "",
-            "caller_current_command": "",
-            "hint": hint,
-        }
-    proc = run_cmd(
-        [
-            "tmux",
-            "display-message",
-            "-p",
-            "-t",
-            caller_pane,
-            "#{pane_current_command}",
-        ],
-        timeout=5,
-    )
-    if getattr(proc, "returncode", 1) != 0:
-        caller_command = ""
-    else:
-        caller_command = (getattr(proc, "stdout", "") or "").strip()
-    machine_fingerprint = os.environ.get("TEAM_AGENT_MACHINE_FINGERPRINT") or ""
-    os_user = os.environ.get("USER") or os.environ.get("USERNAME") or ""
-    provider = (
-        os.environ.get("TEAM_AGENT_LEADER_PROVIDER")
-        or _provider_from_command(caller_command)
-    )
-    if override_uuid:
-        leader_session_uuid = override_uuid
-    else:
-        env_override = (
-            os.environ.get("TEAM_AGENT_LEADER_SESSION_UUID_OVERRIDE") or ""
-        )
-        if env_override:
-            leader_session_uuid = env_override
-        else:
-            leader_session_uuid = derive_leader_session_uuid(
-                machine_fingerprint, workspace, os_user, team_id
-            )
-    owner = {
-        "pane_id": caller_pane,
-        "leader_session_uuid": leader_session_uuid,
-        "machine_fingerprint": machine_fingerprint,
-        "provider": provider,
-        "os_user": os_user,
-        "claimed_at": datetime.now(timezone.utc).isoformat(),
-    }
-    return {
-        "ok": True,
-        "owner": owner,
-        "caller_pane_id": caller_pane,
-        "caller_current_command": caller_command,
-        "team_id": team_id,
-    }
-
-
-def derive_leader_session_uuid(
-    machine_fingerprint: str, workspace: Path, os_user: str, team_id: str
-) -> str:
-    workspace_abspath = str(Path(workspace).resolve())
-    payload = "\0".join((machine_fingerprint, workspace_abspath, os_user, team_id))
-    return hashlib.sha256(payload.encode("utf-8")).hexdigest()[:32]
-
-
-def _provider_from_command(cmd: str) -> str:
-    if cmd in {"claude", "claude.exe"}:
-        return "claude"
-    if cmd == "codex":
-        return "codex"
-    return ""
-
-
-def emit_owner_bound_event(
-    workspace: Path,
-    *,
-    caller_pane_id: str,
-    caller_current_command: str,
-    derived_leader_session_uuid: str,
-    team_id: str,
-    old_leader_session_uuid: str | None,
-) -> None:
-    """Audit hook for successful owner bind. Use after a caller has
-    accepted ``bind_owner_from_caller_pane`` and persisted the new owner.
-
-    Only short uuid prefixes go to the event log so we do not leak the
-    full session uuid into operator-readable transcripts."""
-    EventLog(workspace).write(
-        "owner.bound_from_caller_pane",
-        caller_pane_id=caller_pane_id,
-        caller_current_command=caller_current_command,
-        derived_uuid_prefix=(derived_leader_session_uuid or "")[:12],
-        team_id=team_id,
-        old_uuid_prefix=(old_leader_session_uuid or "")[:12],
-    )

package/src/team_agent/lifecycle/__init__.py

@@ -1,5 +0,0 @@
-"""Lifecycle/state/spec lane package.
-
-Future Team and Agent lifecycle orchestration moves here behind runtime.py
-facade wiring.
-"""

package/src/team_agent/lifecycle/agents.py

@@ -1,278 +0,0 @@
-from __future__ import annotations
-
-import copy
-from pathlib import Path
-from typing import Any
-
-from team_agent.errors import RuntimeError
-from team_agent.events import EventLog
-from team_agent.message_store import MessageStore
-from team_agent.spec import load_spec, validate_spec
-from team_agent.state import (
-    check_team_owner,
-    load_runtime_state,
-    resolve_team_scoped_state,
-    save_runtime_state,
-    save_team_scoped_state,
-    write_spec,
-    write_team_state,
-)
-
-
-def remove_agent(
-    workspace: Path,
-    agent_id: str,
-    *,
-    from_spec: bool = False,
-    confirm: bool = False,
-    force: bool = False,
-    team: str | None = None,
-) -> dict[str, Any]:
-    import team_agent.runtime as runtime
-
-    workspace = workspace.resolve()
-    state, refusal = resolve_team_scoped_state(workspace, team)
-    if refusal:
-        return refusal
-    gate = check_team_owner(state)
-    if gate:
-        return gate
-    spec_path = Path(state.get("spec_path", workspace / "team.spec.yaml"))
-    spec = load_spec(spec_path)
-    agent = _find_worker(spec, agent_id)
-    if not agent:
-        raise RuntimeError(f"unknown worker agent id: {agent_id}")
-
-    event_log = EventLog(workspace)
-    store = MessageStore(workspace)
-    agent_state = state.get("agents", {}).get(agent_id) or {}
-    dynamic_role_file = _dynamic_role_file(workspace, agent_id, agent_state)
-    dynamic_agent = bool(agent_state.get("dynamic_role_file") or agent.get("forked_from"))
-    running = _is_running(runtime, state, agent_id, agent_state)
-
-    if not dynamic_agent and not (from_spec and confirm):
-        return {"ok": False, "agent_id": agent_id, "status": "refused", "reason": "from_spec_confirm_required"}
-    if running and not force:
-        return {"ok": False, "agent_id": agent_id, "status": "refused", "reason": "force_required"}
-
-    rollback = _RemoveRollback(workspace, spec_path, spec, state, dynamic_role_file, store, agent_id, False)
-    stopped: dict[str, Any] | None = None
-    cleared_locations: list[str] = []
-    current_step = "init"
-    current_resource: str | None = None
-
-    def _step_done(name: str, resource: str | None = None, **extra: Any) -> None:
-        cleared_locations.append(name)
-        event_log.write(
-            "lifecycle.remove_step_completed",
-            agent_id=agent_id,
-            step=name,
-            resource=resource,
-            **extra,
-        )
-
-    try:
-        if running and force:
-            current_step, current_resource = "stop_agent", agent_id
-            stopped = runtime.stop_agent(workspace, agent_id, team=team)
-            rollback.restore_running = True
-            state, _refusal_after = resolve_team_scoped_state(workspace, team)
-            _step_done("stop_agent", resource=agent_id, stopped=stopped)
-
-        current_step, current_resource = "workspace_state", "state.json:agents"
-        removed_state = copy.deepcopy(state)
-        removed_state.get("agents", {}).pop(agent_id, None)
-        save_team_scoped_state(workspace, removed_state)
-        _step_done("workspace_state", resource=current_resource)
-
-        current_step, current_resource = "spec_yaml", str(spec_path)
-        removed_spec = copy.deepcopy(spec)
-        removed_spec["agents"] = [item for item in removed_spec.get("agents", []) if item.get("id") != agent_id]
-        startup_order = removed_spec.get("runtime", {}).get("startup_order")
-        if isinstance(startup_order, list):
-            removed_spec["runtime"]["startup_order"] = [item for item in startup_order if item != agent_id]
-        validate_spec(removed_spec, base_dir=spec_path.parent)
-        current_step, current_resource = "team_state_md", "team_state.md"
-        team_state_path = write_team_state(workspace, removed_spec, removed_state)
-        _step_done("team_state_md", resource=str(team_state_path))
-        current_step, current_resource = "spec_yaml", str(spec_path)
-        write_spec(spec_path, removed_spec)
-        _step_done("spec_yaml", resource=str(spec_path))
-
-        current_step, current_resource = "role_file", str(dynamic_role_file)
-        role_file_removed = _remove_dynamic_role_file(dynamic_role_file, bool(agent_state.get("dynamic_role_file")))
-        if role_file_removed:
-            _step_done("role_file", resource=str(dynamic_role_file))
-
-        current_step, current_resource = "agent_health", agent_id
-        _delete_agent_health(store, agent_id)
-        _step_done("agent_health", resource=agent_id)
-    except Exception as exc:
-        rollback_result = rollback.restore(runtime, event_log)
-        event_log.write(
-            "remove_agent.rollback",
-            agent_id=agent_id,
-            ok=rollback_result["ok"],
-            error=str(exc),
-            failed_step=current_step,
-            resource=current_resource,
-            cleared_before_failure=cleared_locations,
-            rollback=rollback_result,
-        )
-        event_log.write(
-            "lifecycle.remove_rolled_back",
-            agent_id=agent_id,
-            ok=rollback_result["ok"],
-            failed_step=current_step,
-            resource=current_resource,
-            rollback_errors=rollback_result.get("errors", []),
-        )
-        raise RuntimeError(
-            f"remove-agent failed for {agent_id} at step={current_step} "
-            f"resource={current_resource}: {exc}; rollback_ok={rollback_result['ok']}"
-        ) from exc
-
-    runtime._save_team_runtime_snapshot(workspace, removed_state)
-    warning = None
-    try:
-        # Storage commit is authoritative; final success event logging is best-effort.
-        event_log.write(
-            "remove_agent.complete",
-            agent_id=agent_id,
-            from_spec=from_spec,
-            force=force,
-            stopped=stopped,
-            role_file_removed=role_file_removed,
-            cleared_locations=cleared_locations,
-        )
-    except Exception as exc:
-        warning = f"remove-agent completed but success event logging failed: {exc}"
-    return {
-        "ok": True,
-        "agent_id": agent_id,
-        "status": "removed",
-        "from_spec": from_spec,
-        "force": force,
-        "stopped": stopped,
-        "state_file": str(team_state_path),
-        "role_file_removed": role_file_removed,
-        "cleared_locations": cleared_locations,
-        **({"warning": warning} if warning else {}),
-    }
-
-
-class _RemoveRollback:
-    def __init__(
-        self,
-        workspace: Path,
-        spec_path: Path,
-        spec: dict[str, Any],
-        state: dict[str, Any],
-        dynamic_role_file: Path,
-        store: MessageStore,
-        agent_id: str,
-        restore_running: bool,
-    ) -> None:
-        self.workspace = workspace
-        self.spec_path = spec_path
-        self.spec_text = spec_path.read_text(encoding="utf-8")
-        self.spec = copy.deepcopy(spec)
-        self.state = copy.deepcopy(state)
-        self.team_state_path = workspace / spec.get("context", {}).get("state_file", "team_state.md")
-        self.team_state_text = self.team_state_path.read_text(encoding="utf-8") if self.team_state_path.exists() else None
-        self.dynamic_role_file = dynamic_role_file
-        self.dynamic_role_bytes = dynamic_role_file.read_bytes() if dynamic_role_file.exists() else None
-        self.health = copy.deepcopy(store.agent_health().get(agent_id))
-        self.agent_id = agent_id
-        self.restore_running = restore_running
-
-    def restore(self, runtime: Any, event_log: EventLog) -> dict[str, Any]:
-        errors: list[str] = []
-        try:
-            self.spec_path.write_text(self.spec_text, encoding="utf-8")
-        except Exception as exc:
-            errors.append(f"spec:{exc}")
-        try:
-            save_team_scoped_state(self.workspace, self.state)
-        except Exception as exc:
-            errors.append(f"workspace_state:{exc}")
-        try:
-            if self.team_state_text is None:
-                self.team_state_path.unlink(missing_ok=True)
-            else:
-                self.team_state_path.parent.mkdir(parents=True, exist_ok=True)
-                self.team_state_path.write_text(self.team_state_text, encoding="utf-8")
-        except Exception as exc:
-            errors.append(f"team_state:{exc}")
-        try:
-            if self.dynamic_role_bytes is None:
-                self.dynamic_role_file.unlink(missing_ok=True)
-            else:
-                self.dynamic_role_file.parent.mkdir(parents=True, exist_ok=True)
-                self.dynamic_role_file.write_bytes(self.dynamic_role_bytes)
-        except Exception as exc:
-            errors.append(f"role_file:{exc}")
-        try:
-            _restore_agent_health(MessageStore(self.workspace), self.agent_id, self.health)
-        except Exception as exc:
-            errors.append(f"agent_health:{exc}")
-        if self.restore_running and not errors:
-            try:
-                runtime.start_agent(self.workspace, self.agent_id, force=True, allow_fresh=True)
-            except Exception as exc:
-                errors.append(f"worker_restore:{exc}")
-        result = {"ok": not errors, "errors": errors}
-        if errors:
-            event_log.write("remove_agent.rollback_failed", agent_id=self.agent_id, errors=errors)
-        return result
-
-
-def _find_worker(spec: dict[str, Any], agent_id: str) -> dict[str, Any] | None:
-    if spec.get("leader", {}).get("id") == agent_id:
-        return None
-    for agent in spec.get("agents", []):
-        if agent.get("id") == agent_id:
-            return agent
-    return None
-
-
-def _dynamic_role_file(workspace: Path, agent_id: str, agent_state: dict[str, Any]) -> Path:
-    raw = agent_state.get("dynamic_role_file")
-    if raw:
-        path = Path(str(raw))
-        return path if path.is_absolute() else workspace / path
-    return workspace / ".team" / "dynamic-role-files" / f"{agent_id}.md"
-
-
-def _is_running(runtime: Any, state: dict[str, Any], agent_id: str, agent_state: dict[str, Any]) -> bool:
-    if str(agent_state.get("status") or "").lower() in {"running", "busy"}:
-        return True
-    session_name = state.get("session_name")
-    window = agent_state.get("window") or agent_id
-    return bool(session_name and runtime._tmux_window_exists(str(session_name), str(window)))
-
-
-def _remove_dynamic_role_file(path: Path, required: bool) -> bool:
-    if path.exists():
-        path.unlink()
-        return True
-    if required:
-        raise RuntimeError(f"dynamic role file missing: {path}")
-    return False
-
-
-def _delete_agent_health(store: MessageStore, agent_id: str) -> None:
-    store.delete_agent_health(agent_id)
-
-
-def _restore_agent_health(store: MessageStore, agent_id: str, row: dict[str, Any] | None) -> None:
-    if not row:
-        _delete_agent_health(store, agent_id)
-        return
-    store.upsert_agent_health(
-        agent_id,
-        row.get("status") or "IDLE",
-        last_output_at=row.get("last_output_at"),
-        context_usage_pct=row.get("context_usage_pct"),
-        current_task_id=row.get("current_task_id"),
-    )