prizmkit 1.1.99 → 1.1.101

package/bundled/dev-pipeline/tests/test_generate_bootstrap_prompt.py

@@ -1,8 +1,10 @@
 """Tests for generate-bootstrap-prompt.py core functions."""
 
+import json
 import os
 import re
 import time
+from argparse import Namespace
 from pathlib import Path
 
 from generate_bootstrap_prompt import (
@@ -16,9 +18,13 @@ from generate_bootstrap_prompt import (
     process_conditional_blocks,
     process_mode_blocks,
     assemble_sections,
+    build_replacements,
     generate_checkpoint_definition,
     merge_checkpoint_state,
+    load_log_size_section,
+    main as generate_bootstrap_main,
 )
+from continuation import append_continuation_handoff, checkpoint_cursor
 
 
 def scoped_report_text(verdict="PASS", boundary_missing=0,
@@ -172,6 +178,107 @@ class TestFormatGlobalContext:
         assert "**lang**: TypeScript" in result
 
 
+class TestExplicitProjectRoot:
+    def test_build_replacements_uses_cli_project_root_for_rendered_paths(self, tmp_path, monkeypatch):
+        monkeypatch.setenv("PRIZMKIT_PLATFORM", "claude")
+        execution_root = tmp_path / "worktree"
+        main_checkout = tmp_path / "main"
+        execution_root.mkdir()
+        main_checkout.mkdir()
+        (execution_root / ".prizmkit" / "plans").mkdir(parents=True)
+        (execution_root / ".prizmkit" / "plans" / "project-brief.md").write_text("worktree brief", encoding="utf-8")
+        args = Namespace(
+            feature_list=str(main_checkout / ".prizmkit" / "plans" / "feature-list.json"),
+            feature_id="F-001",
+            session_id="session-1",
+            run_id="run-1",
+            retry_count="0",
+            resume_phase="null",
+            state_dir=str(main_checkout / ".prizmkit" / "state" / "features"),
+            output=str(main_checkout / ".prizmkit" / "state" / "features" / "F-001" / "sessions" / "session-1" / "bootstrap-prompt.md"),
+            template=None,
+            mode=None,
+            critic=None,
+            extract_baselines=False,
+            project_root=str(execution_root),
+        )
+        feature = {"id": "F-001", "title": "Worktree Prompt", "estimated_complexity": "medium"}
+
+        replacements, _resume, _browser_enabled, _browser_tool = build_replacements(
+            args,
+            feature,
+            [feature],
+            {},
+            str(main_checkout / "dev-pipeline" / "scripts"),
+        )
+
+        assert replacements["{{PROJECT_ROOT}}"] == str(execution_root.resolve())
+        assert replacements["{{SESSION_STATUS_PATH}}"] == str(
+            main_checkout / ".prizmkit" / "state" / "features" / "F-001" /
+            "sessions" / "session-1" / "session-status.json"
+        )
+        assert not replacements["{{SESSION_STATUS_PATH}}"].startswith(str(execution_root.resolve()))
+        assert replacements["{{PROJECT_BRIEF}}"] == "worktree brief"
+        assert replacements["{{DEV_SUBAGENT_PATH}}"].startswith(str(execution_root.resolve()))
+
+
+    def test_main_writes_checkpoint_under_cli_project_root_and_status_under_state_dir(self, tmp_path, monkeypatch):
+        monkeypatch.setenv("PRIZMKIT_PLATFORM", "claude")
+        execution_root = tmp_path / "worktree"
+        main_checkout = tmp_path / "main"
+        pipeline_root = Path(__file__).resolve().parents[1]
+        feature_list = main_checkout / ".prizmkit" / "plans" / "feature-list.json"
+        state_dir = main_checkout / ".prizmkit" / "state" / "features"
+        output_path = state_dir / "F-001" / "sessions" / "session-1" / "bootstrap-prompt.md"
+        feature_list.parent.mkdir(parents=True)
+        state_dir.mkdir(parents=True)
+        execution_root.mkdir(parents=True)
+        (execution_root / ".claude" / "agents").mkdir(parents=True)
+        (execution_root / ".claude" / "agents" / "prizm-dev-team-dev.md").write_text("dev", encoding="utf-8")
+        (execution_root / ".claude" / "agents" / "prizm-dev-team-reviewer.md").write_text("reviewer", encoding="utf-8")
+        feature_list.write_text(
+            json.dumps({"features": [{"id": "F-001", "title": "Worktree Prompt", "estimated_complexity": "medium", "status": "pending"}]}),
+            encoding="utf-8",
+        )
+
+        monkeypatch.setattr(
+            "sys.argv",
+            [
+                "generate-bootstrap-prompt.py",
+                "--feature-list",
+                str(feature_list),
+                "--feature-id",
+                "F-001",
+                "--session-id",
+                "session-1",
+                "--run-id",
+                "run-1",
+                "--retry-count",
+                "0",
+                "--resume-phase",
+                "null",
+                "--state-dir",
+                str(state_dir),
+                "--project-root",
+                str(execution_root),
+                "--output",
+                str(output_path),
+                "--template",
+                str(pipeline_root / "templates" / "bootstrap-tier1.md"),
+            ],
+        )
+
+        try:
+            generate_bootstrap_main()
+        except SystemExit as exc:
+            assert exc.code == 0
+
+        main_checkpoint = main_checkout / ".prizmkit" / "specs" / "001-worktree-prompt" / "workflow-checkpoint.json"
+        rendered = output_path.read_text(encoding="utf-8")
+        assert not main_checkpoint.exists()
+        assert str(execution_root.resolve()) in rendered
+
+
 # ---------------------------------------------------------------------------
 # get_completed_dependencies
 # ---------------------------------------------------------------------------
@@ -656,3 +763,103 @@ class TestScopedFeatureTestGate:
         merged = merge_checkpoint_state(existing, fresh, str(tmp_path))
 
         assert merged["steps"][0]["status"] == "pending"
+
+
+# ---------------------------------------------------------------------------
+# Active compaction harness rollback
+# ---------------------------------------------------------------------------
+
+class TestHeadlessRecoveryReframing:
+    def test_log_size_awareness_is_passive_checkpoint_guidance(self):
+        content = load_log_size_section(str(Path("dev-pipeline/scripts").resolve()))
+
+        assert "Session Log Budget Awareness" in content
+        assert "passive checkpoint guidance" in content
+        assert "durable checkpoints and artifacts" in content
+        assert "fresh continuation session" in content
+        assert "optional interactive convenience only" in content
+        assert "COMPACT_NEEDED" not in content
+        assert "monitor-log" not in content
+        assert "background monitoring subagent" not in content
+        assert "Execute `/compact`" not in content
+        assert "run `/compact`" not in content.lower()
+
+    def test_assembled_prompts_do_not_spawn_log_monitor_or_require_compact(self):
+        sections_dir = Path("dev-pipeline/templates/sections").resolve()
+
+        for mode in ("lite", "standard", "full"):
+            rendered = "\n".join(
+                section for _name, section in assemble_sections(
+                    mode, str(sections_dir), init_done=True, is_resume=False,
+                    critic_enabled=False, browser_enabled=False,
+                )
+            )
+            assert "Headless recovery must rely on durable checkpoints" in rendered
+            assert "fresh continuation session" in rendered
+            assert "monitor-log" not in rendered
+            assert "COMPACT_NEEDED" not in rendered
+            assert "log-size monitor" not in rendered
+            assert "The log-size monitor is the only allowed" not in rendered
+            assert "Run this command and keep it running" not in rendered
+            assert "Execute `/compact`" not in rendered
+            assert "run `/compact` after" not in rendered
+
+
+# ---------------------------------------------------------------------------
+# continuation handoff
+# ---------------------------------------------------------------------------
+
+class TestContinuationHandoff:
+    def test_addendum_and_summary_include_required_context(self, tmp_path):
+        args = Namespace(
+            output=str(tmp_path / ".prizmkit/state/features/F-123/sessions/F-123-new/bootstrap-prompt.md"),
+            state_dir=str(tmp_path / ".prizmkit/state/features"),
+            continuation_mode="context_overflow",
+            previous_session_id="F-123-old",
+            continuation_count="2",
+            context_overflow_count="2",
+            active_dev_branch="dev/F-123",
+            base_branch="main",
+            continuation_summary_path="",
+        )
+        checkpoint = {"steps": [
+            {"id": "S01", "skill": "test-baseline", "status": "completed"},
+            {"id": "S02", "skill": "context-snapshot", "status": "skipped"},
+            {"id": "S03", "skill": "prizmkit-implement", "status": "pending"},
+        ]}
+
+        rendered = append_continuation_handoff(
+            "NORMAL BOOTSTRAP", args, str(tmp_path), "feature", "F-123",
+            "123-continuation", checkpoint,
+        )
+
+        assert rendered.startswith("NORMAL BOOTSTRAP")
+        assert "## Continuation Addendum" in rendered
+        assert "This is an automatic continuation after context overflow." in rendered
+        assert "This is a fresh headless AI CLI session for the same task." in rendered
+        assert "Continue on the current branch/worktree." in rendered
+        assert "Do not reset, clean, switch branches, or discard work." in rendered
+        assert "Start with `git status` and `git diff`." in rendered
+        assert "Treat completed/skipped checkpoint steps as already done." in rendered
+        assert "Do not read the previous full `session.log` unless explicitly necessary." in rendered
+        assert "prefer synthesized artifacts and summaries over logs" in rendered
+        assert "Do not attempt `/compact` as the recovery mechanism." in rendered
+        assert ".prizmkit/specs/123-continuation/workflow-checkpoint.json" in rendered
+        assert ".prizmkit/state/features/F-123/sessions/F-123-new/logs/session.log" in rendered
+        assert ".prizmkit/state/features/F-123/sessions/F-123-new/logs/progress.json" in rendered
+        assert ".prizmkit/state/features/F-123/sessions/F-123-new/logs/ai.pid" in rendered
+
+        summary = tmp_path / ".prizmkit/specs/123-continuation/continuation-summary.md"
+        assert summary.exists()
+        summary_text = summary.read_text(encoding="utf-8")
+        assert "Previous session id: F-123-old" in summary_text
+        assert "First pending/in-progress checkpoint step: S03 (prizmkit-implement)" in summary_text
+        assert "Workflow checkpoint: .prizmkit/specs/123-continuation/workflow-checkpoint.json" in summary_text
+
+    def test_checkpoint_cursor_prefers_in_progress_before_pending(self):
+        checkpoint = {"steps": [
+            {"id": "S01", "skill": "done", "status": "completed"},
+            {"id": "S02", "skill": "active", "status": "in_progress"},
+            {"id": "S03", "skill": "later", "status": "pending"},
+        ]}
+        assert checkpoint_cursor(checkpoint)["first_cursor"] == "S02 (active)"

package/bundled/dev-pipeline/tests/test_generate_bugfix_prompt.py

@@ -1,6 +1,10 @@
 """Tests for generate-bugfix-prompt.py core functions."""
 
+from argparse import Namespace
+from pathlib import Path
+
 from generate_bugfix_prompt import (
+    build_replacements,
     find_bug,
     format_acceptance_criteria,
     format_global_context,
@@ -8,7 +12,9 @@ from generate_bugfix_prompt import (
     format_environment,
     process_conditional_blocks,
     render_template,
+    load_log_size_section,
 )
+from continuation import append_continuation_handoff
 
 
 # ---------------------------------------------------------------------------
@@ -150,6 +156,45 @@ class TestRenderTemplate:
         assert "id=B-001" in result
 
 
+class TestExplicitProjectRoot:
+    def test_build_replacements_uses_cli_project_root_for_rendered_paths(self, tmp_path, monkeypatch):
+        monkeypatch.setenv("PRIZMKIT_PLATFORM", "claude")
+        execution_root = tmp_path / "worktree"
+        main_checkout = tmp_path / "main"
+        execution_root.mkdir()
+        main_checkout.mkdir()
+        args = Namespace(
+            bug_list=str(main_checkout / ".prizmkit" / "plans" / "bug-fix-list.json"),
+            bug_id="B-001",
+            session_id="session-1",
+            run_id="run-1",
+            retry_count="0",
+            resume_phase="null",
+            state_dir=str(main_checkout / ".prizmkit" / "state" / "bugfix"),
+            output=str(main_checkout / ".prizmkit" / "state" / "bugfix" / "B-001" / "sessions" / "session-1" / "bootstrap-prompt.md"),
+            template=None,
+            mode=None,
+            critic=None,
+            project_root=str(execution_root),
+        )
+        bug = {"id": "B-001", "title": "Worktree Bug", "description": "d"}
+
+        replacements = build_replacements(
+            args,
+            bug,
+            {},
+            str(main_checkout / "dev-pipeline" / "scripts"),
+        )
+
+        assert replacements["{{PROJECT_ROOT}}"] == str(execution_root.resolve())
+        assert replacements["{{SESSION_STATUS_PATH}}"] == str(
+            main_checkout / ".prizmkit" / "state" / "bugfix" / "B-001" /
+            "sessions" / "session-1" / "session-status.json"
+        )
+        assert not replacements["{{SESSION_STATUS_PATH}}"].startswith(str(execution_root.resolve()))
+        assert replacements["{{DEV_SUBAGENT_PATH}}"].startswith(str(execution_root.resolve()))
+
+
 # ---------------------------------------------------------------------------
 # Reuse shared helpers (same as bootstrap prompt — verify they are present)
 # ---------------------------------------------------------------------------
@@ -166,3 +211,58 @@ class TestSharedHelpers:
     def test_format_global_context_dict(self):
         result = format_global_context({"lang": "Python"})
         assert "**lang**: Python" in result
+
+
+# ---------------------------------------------------------------------------
+# Active compaction harness rollback
+# ---------------------------------------------------------------------------
+
+class TestHeadlessRecoveryReframing:
+    def test_bugfix_log_size_section_is_passive_checkpoint_guidance(self):
+        content = load_log_size_section(str(Path("dev-pipeline/scripts").resolve()))
+
+        assert "passive checkpoint guidance" in content
+        assert "Headless recovery must rely on durable checkpoints" in content
+        assert "fresh continuation session" in content
+        assert "optional interactive convenience only" in content
+        assert "COMPACT_NEEDED" not in content
+        assert "monitor-log" not in content
+        assert "background monitoring subagent" not in content
+        assert "Execute `/compact`" not in content
+        assert "run `/compact`" not in content.lower()
+
+    def test_bugfix_template_does_not_spawn_monitor_or_require_compact(self):
+        content = Path("dev-pipeline/templates/bugfix-bootstrap-prompt.md").read_text(encoding="utf-8")
+
+        assert "Do not spawn persistent background subagents for headless recovery" in content
+        assert "log-size monitor" not in content
+        assert "monitor-log" not in content
+        assert "COMPACT_NEEDED" not in content
+        assert "Run this command and keep it running" not in content
+        assert "Execute `/compact`" not in content
+        assert "run `/compact` after" not in content
+
+
+class TestBugfixContinuationHandoff:
+    def test_writes_specs_and_task_local_summaries(self, tmp_path):
+        args = Namespace(
+            output=str(tmp_path / ".prizmkit/state/bugfix/B-007/sessions/B-007-new/bootstrap-prompt.md"),
+            state_dir=str(tmp_path / ".prizmkit/state/bugfix"),
+            continuation_mode="context_overflow",
+            previous_session_id="B-007-old",
+            continuation_count="1",
+            context_overflow_count="1",
+            active_dev_branch="bugfix/B-007",
+            base_branch="main",
+            continuation_summary_path=".prizmkit/specs/B-007/continuation-summary.md",
+        )
+        prompt = append_continuation_handoff(
+            "BUGFIX PROMPT", args, str(tmp_path), "bugfix", "B-007", "B-007",
+            {"steps": [{"id": "S01", "skill": "diagnose", "status": "pending"}]},
+        )
+
+        assert prompt.startswith("BUGFIX PROMPT")
+        assert "This is an automatic continuation after context overflow." in prompt
+        assert "Do not attempt `/compact` as the recovery mechanism." in prompt
+        assert (tmp_path / ".prizmkit/specs/B-007/continuation-summary.md").exists()
+        assert (tmp_path / ".prizmkit/bugfix/B-007/continuation-summary.md").exists()

package/bundled/dev-pipeline/tests/test_generate_refactor_prompt.py

@@ -0,0 +1,43 @@
+"""Tests for generate-refactor-prompt.py continuation behavior."""
+
+from argparse import Namespace
+
+from continuation import append_continuation_handoff
+from generate_refactor_prompt import generate_refactor_checkpoint, merge_refactor_checkpoint_state
+
+
+class TestRefactorContinuationHandoff:
+    def test_writes_specs_and_task_local_summaries(self, tmp_path):
+        args = Namespace(
+            output=str(tmp_path / ".prizmkit/state/refactor/R-007/sessions/R-007-new/bootstrap-prompt.md"),
+            state_dir=str(tmp_path / ".prizmkit/state/refactor"),
+            continuation_mode="context_overflow",
+            previous_session_id="R-007-old",
+            continuation_count="1",
+            context_overflow_count="1",
+            active_dev_branch="refactor/R-007",
+            base_branch="main",
+            continuation_summary_path=".prizmkit/specs/R-007/continuation-summary.md",
+        )
+        prompt = append_continuation_handoff(
+            "REFACTOR PROMPT", args, str(tmp_path), "refactor", "R-007", "R-007",
+            {"steps": [{"id": "S01", "skill": "plan", "status": "pending"}]},
+        )
+
+        assert prompt.startswith("REFACTOR PROMPT")
+        assert "This is a fresh headless AI CLI session for the same task." in prompt
+        assert "Do not attempt `/compact` as the recovery mechanism." in prompt
+        assert (tmp_path / ".prizmkit/specs/R-007/continuation-summary.md").exists()
+        assert (tmp_path / ".prizmkit/refactor/R-007/continuation-summary.md").exists()
+
+
+class TestRefactorCheckpointMerge:
+    def test_preserves_skipped_without_artifacts(self, tmp_path):
+        fresh = generate_refactor_checkpoint("R-008", "R-008-new")
+        existing = generate_refactor_checkpoint("R-008", "R-008-old")
+        existing["steps"][0]["status"] = "skipped"
+
+        merged = merge_refactor_checkpoint_state(existing, fresh, str(tmp_path))
+
+        assert merged["steps"][0]["status"] == "skipped"
+        assert merged["steps"][1]["status"] == "pending"