prizmkit 1.1.114 → 1.1.116

package/bundled/dev-pipeline/prizmkit_runtime/runner_models.py

@@ -82,7 +82,6 @@ class RunnerInvocation:
     max_retries: int | None = None
     max_infra_retries: int = 3
     mode: str | None = None
-    critic: bool | None = None
     dry_run: bool = False
     clean: bool = False
     no_reset: bool = False
@@ -100,7 +99,6 @@ class PromptGenerationResult:
     model: str = ""
     pipeline_mode: str = ""
     agent_count: int = 0
-    critic_enabled: bool = False
     checkpoint_path: Path | None = None
     artifact_path: Path | None = None
     raw: Mapping[str, object] = field(default_factory=dict)
@@ -234,7 +232,6 @@ def parse_invocation(
     max_retries: int | None = None
     max_infra_retries = 3
     mode: str | None = None
-    critic: bool | None = None
     dry_run = False
     clean = False
     no_reset = False
@@ -254,10 +251,6 @@ def parse_invocation(
             clean = True
         elif arg == "--no-reset":
             no_reset = True
-        elif arg == "--critic":
-            critic = True
-        elif arg == "--no-critic":
-            critic = False
         elif arg == "--features" and index + 1 < len(args):
             index += 1
             item_filter = args[index]
@@ -305,7 +298,6 @@ def parse_invocation(
         max_retries=max_retries,
         max_infra_retries=max_infra_retries,
         mode=mode,
-        critic=critic,
         dry_run=dry_run,
         clean=clean,
         no_reset=no_reset,

package/bundled/dev-pipeline/prizmkit_runtime/runner_prompts.py

@@ -49,8 +49,6 @@ def generate_prompt(
     ]
     if invocation.mode:
         command.extend(["--mode", invocation.mode])
-    if invocation.critic is not None:
-        command.extend(["--critic", "true" if invocation.critic else "false"])
     if continuation:
         command.extend(["--continuation-mode", "context_overflow"])
         for flag, key in (
@@ -90,7 +88,6 @@ def generate_prompt(
         model=str(data.get("model") or ""),
         pipeline_mode=str(data.get("pipeline_mode") or ""),
         agent_count=int(data.get("agent_count") or 0),
-        critic_enabled=_truthy(data.get("critic_enabled")),
         checkpoint_path=Path(str(checkpoint)) if checkpoint else None,
         artifact_path=_artifact_path(family, visible_root, item_id, data),
         raw=data,

package/bundled/dev-pipeline/prizmkit_runtime/runners.py

@@ -48,6 +48,14 @@ def run_pipeline_command(kind: str, action: str, legacy_args: tuple[str, ...], p
     invocation = _resolve_invocation_paths(invocation, paths.project_root)
     if invocation.help_requested:
         return CommandResult(0, f"Python {kind} runner", _help_text(family))
+    if invocation.unknown_args:
+        return CommandResult(
+            2,
+            f"Python {kind} runner",
+            "Unsupported option(s): {}".format(
+                ", ".join(invocation.unknown_args)
+            ),
+        )
     if action == "status":
         status = status_text(family, invocation, paths.project_root)
         return CommandResult(status.return_code, f"Python {kind} status", status.stdout.strip() or status.stderr.strip())
@@ -332,16 +340,15 @@ def _emit_prompt_summary(prompt: PromptGenerationResult, session_id: str) -> Non
     if prompt.pipeline_mode:
         _emit_info(f"Pipeline mode: {_pipeline_mode_label(prompt.pipeline_mode)}")
     if prompt.agent_count:
-        critic = "enabled" if prompt.critic_enabled else "disabled"
-        _emit_info(f"Agents: {prompt.agent_count} (critic: {critic})")
+        _emit_info(f"Agents: {prompt.agent_count}")
     _emit_info(f"Spawning AI CLI session: {session_id}")
 
 
 def _pipeline_mode_label(mode: str) -> str:
     labels = {
         "lite": "lite (Tier 1 — Single Agent)",
-        "standard": "standard (Tier 2 — Orchestrator + Critic/Reviewer)",
-        "full": "full (Tier 3 — Orchestrator + Critic/Reviewer)",
+        "standard": "standard (Tier 2 — Orchestrator + Reviewer)",
+        "full": "full (Tier 3 — Orchestrator + Reviewer)",
     }
     return labels.get(mode, mode)
 
@@ -527,6 +534,7 @@ def _process_item(
                     cwd=execution_root,
                     log_path=session_paths.session_log,
                     progress_path=session_paths.progress_json,
+                    checkpoint_path=prompt.checkpoint_path,
                     heartbeat_path=session_paths.heartbeat_json,
                     effort=config.effort,
                     verbose=env.verbose,
@@ -864,7 +872,6 @@ def _invocation_for_execution_family(invocation: RunnerInvocation, family: Runne
         max_retries=invocation.max_retries,
         max_infra_retries=invocation.max_infra_retries,
         mode=invocation.mode,
-        critic=invocation.critic,
         dry_run=invocation.dry_run,
         clean=invocation.clean,
         no_reset=invocation.no_reset,
@@ -985,7 +992,6 @@ def _resolve_invocation_paths(invocation: RunnerInvocation, project_root: Path)
         max_retries=invocation.max_retries,
         max_infra_retries=invocation.max_infra_retries,
         mode=invocation.mode,
-        critic=invocation.critic,
         dry_run=invocation.dry_run,
         clean=invocation.clean,
         no_reset=invocation.no_reset,
@@ -1000,5 +1006,5 @@ def _help_text(family: RunnerFamily) -> str:
     return (
         f"Python foreground runner for {family.kind}.\n"
         "Supported legacy options include plan list path, item id, --features for feature, "
-        "--max-retries, --max-infra-retries, --mode, --critic/--no-critic, --resume-phase, and --dry-run."
+        "--max-retries, --max-infra-retries, --mode, --resume-phase, and --dry-run."
     )

package/bundled/dev-pipeline/prizmkit_runtime/sessions.py

@@ -17,6 +17,7 @@ from collections.abc import Callable, Mapping, Sequence
 from dataclasses import dataclass, field
 from pathlib import Path
 
+from .checkpoint_state import load_checkpoint_state
 from .heartbeat import HeartbeatConfig, HeartbeatMonitor, HeartbeatState, observe_heartbeat_state, write_stale_marker
 from .launch_profiles import HeadlessLaunchProfile
 from .processes import ProcessHandle, ProcessSpec, launch_process, terminate_process_tree
@@ -45,6 +46,7 @@ class AISessionConfig:
     log_path: Path | None = None
     backup_log_path: Path | None = None
     progress_path: Path | None = None
+    checkpoint_path: Path | None = None
     heartbeat_path: Path | None = None
     effort: str | None = None
     verbose: bool = False
@@ -513,12 +515,18 @@ def _display_heartbeat_state(
     )
 
 
+def _checkpoint_phase_label(checkpoint_path: Path | None) -> str:
+    state = load_checkpoint_state(checkpoint_path)
+    return state.current_step_name
+
+
 def _compact_progress_line(
     started_epoch: float,
     session_log: Path,
     progress_path: Path | None,
     heartbeat_state: HeartbeatState | None = None,
     *,
+    checkpoint_path: Path | None = None,
     stale_threshold_seconds: float = 0,
 ) -> str:
     data = _read_json_object(progress_path)
@@ -547,7 +555,7 @@ def _compact_progress_line(
         if subagents:
             child = f"{child}/{subagents}"
         parts.append(f"child: {child}")
-    phase = str(data.get("current_phase") or "").strip()
+    phase = _checkpoint_phase_label(checkpoint_path)
     if phase:
         parts.append(f"phase: {phase}")
     tool = str(data.get("current_tool") or "").strip()
@@ -571,6 +579,7 @@ def _emit_progress_tick(
     progress_path: Path | None,
     last_line: str,
     *,
+    checkpoint_path: Path | None = None,
     monitor_state: HeartbeatState | None = None,
     fallback_state: HeartbeatState | None = None,
     interval_seconds: float = 30,
@@ -589,6 +598,7 @@ def _emit_progress_tick(
         session_log,
         progress_path,
         heartbeat_state,
+        checkpoint_path=checkpoint_path,
         stale_threshold_seconds=stale_threshold_seconds,
     )
     if line != last_line:
@@ -828,6 +838,7 @@ class AISessionLauncher:
                                 session_log,
                                 self.config.progress_path,
                                 last_progress_line,
+                                checkpoint_path=self.config.checkpoint_path,
                                 monitor_state=heartbeat.state if heartbeat is not None else None,
                                 fallback_state=display_heartbeat_state,
                                 interval_seconds=live_progress_interval,
@@ -853,6 +864,7 @@ class AISessionLauncher:
                     session_log,
                     self.config.progress_path,
                     final_heartbeat_state,
+                    checkpoint_path=self.config.checkpoint_path,
                     stale_threshold_seconds=stale_threshold_seconds,
                 )
                 if final_line != last_progress_line:

package/bundled/dev-pipeline/scripts/continuation.py

@@ -1,6 +1,13 @@
 """Continuation prompt and durable handoff helpers for pipeline prompt generators."""
 
 import os
+import sys
+
+_PIPELINE_ROOT = os.path.abspath(os.path.join(os.path.dirname(__file__), ".."))
+if _PIPELINE_ROOT not in sys.path:
+    sys.path.insert(0, _PIPELINE_ROOT)
+
+from prizmkit_runtime.checkpoint_state import checkpoint_cursor as shared_checkpoint_cursor  # noqa: E402
 
 SUPPORTED_CONTINUATION_MODES = {"context_overflow"}
 TASK_TYPES = {"feature", "bugfix", "refactor"}
@@ -166,46 +173,7 @@ def runtime_paths(args, project_root, task_type, task_id, previous_session_id):
 
 def checkpoint_cursor(checkpoint):
     """Summarize current checkpoint state and cursor."""
-    steps = checkpoint.get("steps", []) if isinstance(checkpoint, dict) else []
-    completed = []
-    skipped = []
-    in_progress = []
-    pending = []
-    failed = []
-    first_cursor = None
-
-    for step in steps:
-        skill = step.get("skill", "unknown")
-        label = "{} ({})".format(step.get("id", "?"), skill)
-        status = step.get("status", "pending")
-        if status == "completed":
-            completed.append(label)
-        elif status == "skipped":
-            skipped.append(label)
-        elif status == "in_progress":
-            in_progress.append(label)
-            if first_cursor is None:
-                first_cursor = label
-        elif status == "failed":
-            failed.append(label)
-            if first_cursor is None:
-                first_cursor = label
-        else:
-            pending.append(label)
-            if first_cursor is None:
-                first_cursor = label
-
-    if first_cursor is None:
-        first_cursor = "(all checkpoint steps completed or skipped)"
-
-    return {
-        "completed": completed,
-        "skipped": skipped,
-        "in_progress": in_progress,
-        "pending": pending,
-        "failed": failed,
-        "first_cursor": first_cursor,
-    }
+    return shared_checkpoint_cursor(checkpoint)
 
 
 def _list_or_none(values):