cdx-manager 0.8.0 → 0.9.0

package/changelogs/CHANGELOGS_0_9_0.md

@@ -0,0 +1,14 @@
+# CDX Manager 0.9.0
+
+## Highlights
+
+- Adds an observable assistant run registry for CDX sessions.
+- Generates the Logics corpus for assistant-run observability so follow-up work can be tracked from requests through closeout.
+- Closes the observable run registry workflow chain after validation.
+
+## Validation
+
+- `npm run prepublishOnly`
+- `npm pack --dry-run`
+- `logics-manager lint --require-status`
+- `logics-manager audit --legacy-cutoff-version 1.1.0 --group-by-doc`

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cdx-manager",
-  "version": "0.8.0",
+  "version": "0.9.0",
   "description": "Terminal session manager for Codex and Claude accounts.",
   "license": "MIT",
   "author": "Alexandre Agostini",

package/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "cdx-manager"
-version = "0.8.0"
+version = "0.9.0"
 description = "Terminal session manager for Codex and Claude accounts."
 readme = "README.md"
 requires-python = ">=3.9"

package/src/cli.py

@@ -31,6 +31,9 @@ from .cli_commands import (
     handle_repair,
     handle_rename,
     handle_run,
+    handle_run_report,
+    handle_run_status,
+    handle_runs,
     handle_select,
     handle_stats,
     handle_status,
@@ -61,7 +64,7 @@ from .status_view import (
 )
 from .update_check import check_for_update, check_logics_manager_for_update
 
-VERSION = "0.8.0"
+VERSION = "0.9.0"
 
 
 # ---------------------------------------------------------------------------
@@ -81,6 +84,9 @@ def _print_help(use_color=False):
         f"  {_style('cdx next [--json] [--refresh]', '36', use_color)}",
         f"  {_style('cdx select --provider PROVIDER [--min-reasoning-effort low|medium|high] [--min-power low|medium|high] [--require-ready] [--refresh] --json', '36', use_color)}",
         f"  {_style('cdx run [session] --cwd PATH (--prompt-file PATH|--prompt TEXT) [--provider PROVIDER] [--model MODEL] [--reasoning-effort low|medium|high] [--power low|medium|high] [--permission review|default|auto|full|workspace-write|read-only|danger-full-access] [--timeout-seconds N] --json', '36', use_color)}",
+        f"  {_style('cdx runs [--limit N] --json', '36', use_color)}",
+        f"  {_style('cdx run-status <run_id> --json', '36', use_color)}",
+        f"  {_style('cdx run-report <run_id> --json', '36', use_color)}",
         f"  {_style('cdx context show|path|init|edit|clear|set [text...] [--json]', '36', use_color)}",
         f"  {_style('cdx config <name> [--json]', '36', use_color)}",
         f"  {_style('cdx configs [--json]', '36', use_color)}",
@@ -371,6 +377,15 @@ def main(argv, options=None):
     if command == "run":
         return handle_run(rest, ctx)
 
+    if command == "runs":
+        return handle_runs(rest, ctx)
+
+    if command == "run-status":
+        return handle_run_status(rest, ctx)
+
+    if command == "run-report":
+        return handle_run_report(rest, ctx)
+
     if command == "login":
         return handle_login(rest, ctx)
 

package/src/cli_commands.py

@@ -33,6 +33,7 @@ from .notify import (
 )
 from .provider_runtime import (
     _ensure_session_authentication,
+    _headless_artifact_paths,
     _list_launch_transcript_paths,
     _normalize_reasoning_effort,
     _probe_provider_auth,
@@ -41,6 +42,7 @@ from .provider_runtime import (
 )
 from .repair import format_repair_report, repair_health
 from .backup_bundle import read_bundle_meta
+from .run_registry import RunRegistry, build_code_review_report
 from .run_usage import extract_run_usage
 from .run_command import read_run_prompt, run_cdx_error_code, run_result_payload
 from .status_view import _format_status_detail, _format_status_rows, format_priority_instruction, recommend_priority_rows
@@ -66,7 +68,10 @@ STATS_USAGE = "Usage: cdx stats [name] [--since 7d|today|DATE] [--from DATE] [--
 LAST_USAGE = "Usage: cdx last [--json]"
 SELECT_USAGE = "Usage: cdx select --provider PROVIDER [--min-reasoning-effort low|medium|high] [--min-power low|medium|high] [--require-ready] [--refresh] --json"
 NEXT_USAGE = "Usage: cdx next [--json] [--refresh]"
-RUN_USAGE = "Usage: cdx run [session] --cwd PATH (--prompt-file PATH|--prompt TEXT) [--provider PROVIDER] [--model MODEL] [--reasoning-effort low|medium|high] [--power low|medium|high] [--permission review|default|auto|full|workspace-write|read-only|danger-full-access] [--timeout-seconds N] --json"
+RUN_USAGE = "Usage: cdx run [session] --cwd PATH (--prompt-file PATH|--prompt TEXT) [--provider PROVIDER] [--model MODEL] [--kind assistant|code-review] [--reasoning-effort low|medium|high] [--power low|medium|high] [--permission review|default|auto|full|workspace-write|read-only|danger-full-access] [--timeout-seconds N] --json"
+RUNS_USAGE = "Usage: cdx runs [--limit N] --json"
+RUN_STATUS_USAGE = "Usage: cdx run-status <run_id> --json"
+RUN_REPORT_USAGE = "Usage: cdx run-report <run_id> --json"
 API_SCHEMA_VERSION = 1
 HANDOFF_TRANSCRIPT_CHARS = 120000
 HANDOFF_NATIVE_TRANSCRIPT_CANDIDATES = 64
@@ -695,6 +700,7 @@ def _parse_run_args(args):
         "--prompt": {"key": "prompt", "type": "str", "default": None},
         "--provider": {"key": "provider", "type": "str", "default": None, "transform": lambda value: _parse_provider_filter(value, RUN_USAGE)},
         "--model": {"key": "model", "type": "str", "default": None},
+        "--kind": {"key": "kind", "type": "str", "default": "assistant"},
         "--reasoning-effort": {"key": "reasoning_effort", "type": "str", "default": None},
         "--power": {"key": "power", "type": "str", "default": None},
         "--permission": {"key": "permission", "type": "str", "default": None, "transform": _normalize_run_permission},
@@ -712,6 +718,8 @@ def _parse_run_args(args):
         raise CdxError(RUN_USAGE)
     if bool(parsed["prompt_file"]) == bool(parsed["prompt"]):
         raise CdxError(RUN_USAGE)
+    if parsed["kind"] not in ("assistant", "code-review"):
+        raise CdxError(RUN_USAGE)
     effort = _normalize_reasoning_effort(
         reasoning_effort=parsed["reasoning_effort"],
         power=parsed["power"],
@@ -723,6 +731,7 @@ def _parse_run_args(args):
         "cwd": parsed["cwd"],
         "prompt_file": parsed["prompt_file"],
         "prompt": parsed["prompt"],
+        "kind": parsed["kind"],
         "model": parsed["model"],
         "permission": parsed["permission"],
         "timeout_seconds": parsed["timeout_seconds"],
@@ -1338,7 +1347,65 @@ def handle_next(rest, ctx):
     return 0
 
 
+def _parse_runs_args(rest):
+    return _parse_flag_args(rest, {
+        "--limit": {"key": "limit", "type": "str", "default": 20, "transform": _parse_positive_int},
+        "--json": {"key": "json", "type": "bool", "default": False},
+    }, RUNS_USAGE, max_positionals=0)
+
+
+def _parse_run_id_json_args(rest, usage):
+    parsed = _parse_flag_args(rest, {
+        "--json": {"key": "json", "type": "bool", "default": False},
+    }, usage, positionals_key="ids", max_positionals=1)
+    if not parsed["json"] or len(parsed["ids"]) != 1:
+        raise CdxError(usage)
+    return {"run_id": parsed["ids"][0], "json": True}
+
+
+def _run_registry(ctx):
+    return RunRegistry(ctx["service"]["base_dir"])
+
+
+def handle_runs(rest, ctx):
+    parsed = _parse_runs_args(rest)
+    if not parsed["json"]:
+        raise CdxError(RUNS_USAGE)
+    runs = _run_registry(ctx).list(limit=parsed["limit"])
+    _write_json(ctx, _json_success("runs", "Runs loaded.", runs=runs))
+    return 0
+
+
+def handle_run_status(rest, ctx):
+    parsed = _parse_run_id_json_args(rest, RUN_STATUS_USAGE)
+    run = _run_registry(ctx).get(parsed["run_id"])
+    if not run:
+        _write_json(ctx, _json_failure("run-status", "run_not_found", f"Unknown run: {parsed['run_id']}"))
+        return 1
+    _write_json(ctx, _json_success("run-status", "Run loaded.", run=run))
+    return 0
+
+
+def handle_run_report(rest, ctx):
+    parsed = _parse_run_id_json_args(rest, RUN_REPORT_USAGE)
+    run = _run_registry(ctx).get(parsed["run_id"])
+    if not run:
+        _write_json(ctx, _json_failure("run-report", "run_not_found", f"Unknown run: {parsed['run_id']}"))
+        return 1
+    _write_json(ctx, _json_success("run-report", "Run report loaded.", report={
+        "run": run,
+        "final_payload": run.get("final_payload"),
+        "task_report": run.get("task_report"),
+        "artifacts": run.get("artifacts") or {},
+        "usage": run.get("usage"),
+        "error": run.get("error"),
+    }))
+    return 0
+
+
 def handle_run(rest, ctx):
+    registry = None
+    run_id = None
     try:
         parsed = _parse_run_args(rest)
         session = None
@@ -1397,6 +1464,22 @@ def handle_run(rest, ctx):
             signal_emitter=ctx.get("signal_emitter"),
             trust_local_credentials=False,
         )
+        registry = _run_registry(ctx)
+        artifacts = _headless_artifact_paths(run_session)
+        run_id = artifacts["run_id"]
+        registry.start(
+            run_id,
+            kind=parsed.get("kind"),
+            session=run_session.get("name"),
+            provider=run_session.get("provider"),
+            model=parsed.get("model") or ((run_session.get("launch") or {}).get("model")),
+            cwd=cwd,
+            artifacts={
+                "transcript_path": artifacts.get("transcript_path"),
+                "stdout_path": artifacts.get("stdout_path"),
+                "stderr_path": artifacts.get("stderr_path"),
+            },
+        )
         run_info = _run_headless_provider_command(
             run_session,
             cwd=cwd,
@@ -1404,29 +1487,32 @@ def handle_run(rest, ctx):
             initial_prompt=prompt,
             timeout_seconds=parsed.get("timeout_seconds"),
             spawn=ctx.get("spawn_headless") or ctx.get("spawn"),
+            run_id=run_id,
         )
         usage = extract_run_usage(run_session.get("provider"), run_info.get("stdout_path"))
         run_info = {**run_info, "usage": usage}
         ok = run_info.get("returncode") == 0
         if ok:
+            final_payload = run_result_payload(API_SCHEMA_VERSION, True, parsed, run_session, run_info=run_info)
+            task_report = build_code_review_report(run_id, final_payload) if parsed.get("kind") == "code-review" else None
+            registry.finish(
+                run_id,
+                status="succeeded",
+                final_payload=final_payload,
+                run_info=run_info,
+                task_report=task_report,
+            )
             ctx["service"]["record_launch_history"](session["name"], {
                 "status": "success",
                 "cwd": cwd,
                 "exit_code": 0,
                 **run_info,
             })
-            _write_json(ctx, run_result_payload(API_SCHEMA_VERSION, True, parsed, run_session, run_info=run_info))
+            _write_json(ctx, final_payload)
             return 0
         message = "Provider process timed out." if run_info.get("timed_out") else "Provider process exited with a non-zero status."
         error = CdxError(message, run_info.get("returncode") or 1)
-        ctx["service"]["record_launch_history"](session["name"], {
-            "status": "failed",
-            "cwd": cwd,
-            "error": str(error),
-            "exit_code": error.exit_code,
-            **run_info,
-        })
-        _write_json(ctx, run_result_payload(
+        final_payload = run_result_payload(
             API_SCHEMA_VERSION,
             False,
             parsed,
@@ -1435,11 +1521,27 @@ def handle_run(rest, ctx):
             error=error,
             error_source="provider",
             error_code="provider_timeout" if run_info.get("timed_out") else "provider_failed",
-        ))
+        )
+        registry.finish(
+            run_id,
+            status="timed_out" if run_info.get("timed_out") else "failed",
+            final_payload=final_payload,
+            run_info=run_info,
+            error=final_payload.get("error"),
+            task_report=build_code_review_report(run_id, final_payload) if parsed.get("kind") == "code-review" else None,
+        )
+        ctx["service"]["record_launch_history"](session["name"], {
+            "status": "failed",
+            "cwd": cwd,
+            "error": str(error),
+            "exit_code": error.exit_code,
+            **run_info,
+        })
+        _write_json(ctx, final_payload)
         return error.exit_code or 1
     except CdxError as error:
         run_info = getattr(error, "run_info", None)
-        _write_json(ctx, run_result_payload(
+        final_payload = run_result_payload(
             API_SCHEMA_VERSION,
             False,
             locals().get("parsed", {}) or {},
@@ -1448,7 +1550,16 @@ def handle_run(rest, ctx):
             error=error,
             error_source="cdx",
             error_code=run_cdx_error_code(error),
-        ))
+        )
+        if registry and run_id:
+            registry.finish(
+                run_id,
+                status="failed",
+                final_payload=final_payload,
+                run_info=run_info,
+                error=final_payload.get("error"),
+            )
+        _write_json(ctx, final_payload)
         return error.exit_code
 
 

package/src/cli_view.py

@@ -95,6 +95,9 @@ def handle_view(rest, ctx):
         result = run_logics_viewer(executable, cwd, env=env, runner=ctx.get("spawn_sync"))
     except FileNotFoundError as error:
         raise CdxError(f"logics-manager is required for cdx view. {LOGICS_MANAGER_INSTALL_HINT}") from error
+    except KeyboardInterrupt:
+        ctx["out"]("\n")
+        return 130
     returncode = getattr(result, "returncode", 0)
     if returncode not in (0, None):
         raise CdxError("logics-manager view failed.", exit_code=returncode)

package/src/run_registry.py

@@ -0,0 +1,195 @@
+import json
+import os
+import tempfile
+from datetime import datetime, timezone
+
+
+def utc_now_iso():
+    return datetime.now(timezone.utc).isoformat().replace("+00:00", "Z")
+
+
+def registry_path(base_dir):
+    return os.path.join(base_dir, "runs.json")
+
+
+def _read_registry(path):
+    try:
+        with open(path, "r", encoding="utf-8") as handle:
+            data = json.load(handle)
+    except FileNotFoundError:
+        return {"schema_version": 1, "runs": []}
+    if not isinstance(data, dict):
+        return {"schema_version": 1, "runs": []}
+    runs = data.get("runs")
+    if not isinstance(runs, list):
+        runs = []
+    return {"schema_version": int(data.get("schema_version") or 1), "runs": [run for run in runs if isinstance(run, dict)]}
+
+
+def _write_registry(path, data):
+    os.makedirs(os.path.dirname(path), exist_ok=True)
+    fd, temp_path = tempfile.mkstemp(prefix=".runs-", suffix=".json", dir=os.path.dirname(path))
+    try:
+        with os.fdopen(fd, "w", encoding="utf-8") as handle:
+            json.dump(data, handle, indent=2, sort_keys=True)
+            handle.write("\n")
+        os.replace(temp_path, path)
+    finally:
+        try:
+            os.unlink(temp_path)
+        except FileNotFoundError:
+            pass
+
+
+def _is_pid_alive(pid):
+    if not pid:
+        return False
+    try:
+        os.kill(int(pid), 0)
+        return True
+    except (OSError, ValueError):
+        return False
+
+
+def _refresh_stale_runs(data):
+    now = utc_now_iso()
+    changed = False
+    for run in data["runs"]:
+        if run.get("status") != "running":
+            continue
+        pid = run.get("pid")
+        if pid and _is_pid_alive(pid):
+            continue
+        run["status"] = "stale"
+        run["ended_at"] = now
+        run["error"] = {"code": "stale_process", "message": "Run was marked running but no live provider process was found."}
+        changed = True
+    return changed
+
+
+def _base_record(run_id, *, kind, session, provider, model, cwd, artifacts=None):
+    return {
+        "run_id": run_id,
+        "kind": kind or "assistant",
+        "status": "running",
+        "session": session,
+        "provider": provider,
+        "model": model,
+        "cwd": os.path.abspath(cwd),
+        "pid": None,
+        "started_at": utc_now_iso(),
+        "ended_at": None,
+        "duration_seconds": None,
+        "exit_code": None,
+        "usage": None,
+        "artifacts": dict(artifacts or {}),
+        "error": None,
+        "task_report": None,
+        "final_payload": None,
+    }
+
+
+class RunRegistry:
+    def __init__(self, base_dir):
+        self.path = registry_path(base_dir)
+
+    def start(self, run_id, *, kind, session, provider, model, cwd, artifacts=None):
+        data = _read_registry(self.path)
+        data["runs"] = [run for run in data["runs"] if run.get("run_id") != run_id]
+        record = _base_record(run_id, kind=kind, session=session, provider=provider, model=model, cwd=cwd, artifacts=artifacts)
+        data["runs"].insert(0, record)
+        _write_registry(self.path, data)
+        return record
+
+    def finish(self, run_id, *, status, final_payload=None, run_info=None, error=None, task_report=None):
+        data = _read_registry(self.path)
+        now = utc_now_iso()
+        for run in data["runs"]:
+            if run.get("run_id") != run_id:
+                continue
+            run["status"] = status
+            run["ended_at"] = now
+            if run.get("started_at"):
+                try:
+                    start = datetime.fromisoformat(str(run["started_at"]).replace("Z", "+00:00"))
+                    end = datetime.fromisoformat(now.replace("Z", "+00:00"))
+                    run["duration_seconds"] = (end - start).total_seconds()
+                except ValueError:
+                    run["duration_seconds"] = None
+            if run_info:
+                run["pid"] = run_info.get("pid")
+                run["exit_code"] = run_info.get("returncode")
+                run["artifacts"] = {
+                    **(run.get("artifacts") or {}),
+                    "transcript_path": run_info.get("transcript_path"),
+                    "stdout_path": run_info.get("stdout_path"),
+                    "stderr_path": run_info.get("stderr_path"),
+                }
+            if final_payload:
+                run["usage"] = final_payload.get("usage")
+                run["final_payload"] = final_payload
+            if error:
+                run["error"] = error
+            if task_report:
+                run["task_report"] = task_report
+            _write_registry(self.path, data)
+            return run
+        return None
+
+    def list(self, limit=20):
+        data = _read_registry(self.path)
+        changed = _refresh_stale_runs(data)
+        if changed:
+            _write_registry(self.path, data)
+        return data["runs"][: max(0, int(limit or 20))]
+
+    def get(self, run_id):
+        data = _read_registry(self.path)
+        changed = _refresh_stale_runs(data)
+        if changed:
+            _write_registry(self.path, data)
+        for run in data["runs"]:
+            if run.get("run_id") == run_id:
+                return run
+        return None
+
+
+def read_text_file(path, limit=120000):
+    if not path:
+        return ""
+    try:
+        with open(path, "r", encoding="utf-8", errors="replace") as handle:
+            return handle.read(limit)
+    except OSError:
+        return ""
+
+
+def build_code_review_report(run_id, final_payload):
+    stdout = read_text_file(final_payload.get("stdout_path") if final_payload else None)
+    parsed = None
+    try:
+        candidate = json.loads(stdout or "{}")
+        if isinstance(candidate, dict):
+            parsed = candidate
+    except json.JSONDecodeError:
+        parsed = None
+    findings = parsed.get("findings") if parsed else []
+    if not isinstance(findings, list):
+        findings = []
+    summary = parsed.get("summary") if parsed else None
+    next_steps = parsed.get("next_steps") if parsed else None
+    if not isinstance(next_steps, list):
+        next_steps = []
+    return {
+        "schema_version": 1,
+        "kind": "code-review",
+        "run_id": run_id,
+        "summary": summary or "Code review completed. Structured findings were not emitted by the provider.",
+        "findings": [finding for finding in findings if isinstance(finding, dict)],
+        "next_steps": [str(step) for step in next_steps],
+        "artifacts": {
+            "transcript_path": final_payload.get("transcript_path") if final_payload else None,
+            "stdout_path": final_payload.get("stdout_path") if final_payload else None,
+            "stderr_path": final_payload.get("stderr_path") if final_payload else None,
+        },
+    }