Add checkpoint discovery hints to agent prompts

.egg-state/agent-outputs/integrator-output.json

@@ -1,92 +1,78 @@
 {
   "status": "pass",
   "phase": "implement",
-  "pipeline_id": "issue-850",
-  "issue": 850,
-  "summary": "Two-tier health check framework integration verified across both implementation phases. All tests pass, all cross-phase interfaces are correct, no regressions detected.",
+  "pipeline_id": "issue-887",
+  "issue": 887,
+  "summary": "Checkpoint discoverability changes verified across orchestrator prompts, agent mode commands, rules, and documentation. All 6683 passing tests pass (2 pre-existing failures unrelated to this branch). One lint/format fix applied to the new test file.",
   "test_results": {
-    "health_check_tests": {
-      "passed": 419,
-      "failed": 0,
-      "skipped": 0,
-      "description": "All health check framework tests (types, tier1, tier2, context, runner, integration, lifecycle, tester coverage)"
-    },
-    "orchestrator_tests": {
-      "passed": 1169,
-      "failed": 3,
-      "skipped": 0,
-      "description": "Full orchestrator test suite. 3 failures are pre-existing in test_docker_client.py (unchanged files, same failures on origin/main)"
+    "full_suite": {
+      "passed": 6683,
+      "failed": 2,
+      "skipped": 85,
+      "description": "Full test suite (tests/, gateway/tests/, orchestrator/tests/). 2 failures are pre-existing in test_docker_client.py (unchanged files, not modified on this branch)."
     },
-    "gateway_tests": {
-      "passed": 1523,
-      "failed": 0,
-      "skipped": 3,
-      "description": "Full gateway test suite - no regressions"
-    },
-    "shared_tests": {
-      "passed": 84,
+    "checkpoint_discovery_tests": {
+      "passed": 29,
       "failed": 0,
       "skipped": 0,
-      "description": "Full shared module test suite - no regressions"
+      "description": "New test_checkpoint_discovery.py covering _build_role_context, _build_agent_prompt, and _build_phase_scoped_prompt checkpoint hints for all roles."
     },
-    "root_tests": {
-      "passed": 3744,
+    "entrypoint_tests": {
+      "passed": 89,
       "failed": 0,
-      "skipped": 80,
-      "description": "Full root test suite including all components - no regressions"
+      "skipped": 0,
+      "description": "Sandbox entrypoint tests (unchanged by this branch, pre-existing pass)."
     }
   },
   "integration_checks": {
-    "types_interface": {
-      "status": "pass",
-      "detail": "Tier 2 agent inspector correctly imports and uses HealthCheck, HealthResult, HealthStatus, HealthAction, HealthTier, HealthTrigger from types.py"
-    },
-    "context_usage": {
-      "status": "pass",
-      "detail": "Agent inspector correctly uses PipelineHealthContext and all lazy-loaded methods (git_log, git_diff_stat, agent_outputs, contract, live_container_ids)"
-    },
-    "runner_escalation": {
-      "status": "pass",
-      "detail": "Runner correctly handles Tier 1 then Tier 2 with escalation logic: WAVE_COMPLETE escalates on DEGRADED, PHASE_COMPLETE/ON_DEMAND always escalate, STARTUP/RUNTIME_TICK never escalate"
-    },
-    "routes_integration": {
+    "cross_reference_consistency": {
       "status": "pass",
-      "detail": "routes/health.py uses runner for ON_DEMAND checks. routes/phases.py gates phase advance on PHASE_COMPLETE checks with FAIL_PIPELINE blocking"
+      "detail": "All egg-checkpoint CLI commands are consistently referenced across orchestrator prompts (pipelines.py), agent mode commands (tester/coder/integrator/documenter-mode.md), rules (checkpoint.md, mission.md), and documentation (checkpoint-access.md, sdlc-pipeline.md). Environment variables ($EGG_PIPELINE_ID, $EGG_ISSUE_NUMBER) are correctly scoped per role."
     },
-    "init_exports": {
+    "orchestrator_prompt_hints": {
       "status": "pass",
-      "detail": "All __init__.py files in health_checks/, tier1/, tier2/ export needed symbols correctly"
+      "detail": "_build_role_context adds general checkpoint pointer for all execution roles. _build_agent_prompt adds role-specific hints: tester gets coder checkpoint list, documenter gets context --files, integrator gets context + cost. _build_phase_scoped_prompt adds failed session hint in revision checklists. Analysis roles (architect, task_planner, risk_analyst) correctly excluded."
     },
-    "multi_agent_integration": {
+    "agent_mode_commands": {
       "status": "pass",
-      "detail": "multi_agent.py correctly calls health checks on wave completion with WAVE_COMPLETE trigger, breaks on FAIL_PIPELINE action"
+      "detail": "All 4 mode commands (tester, coder, integrator, documenter) have new sections with checkpoint CLI examples matching the orchestrator prompt hints. Content is additive only, no existing sections modified."
     },
-    "container_monitor_integration": {
+    "rules_updates": {
       "status": "pass",
-      "detail": "container_monitor.py integrates via set_health_check_runner, fires RUNTIME_TICK checks on container state changes"
+      "detail": "checkpoint.md has new 'When to Use' section with role-specific guidance. mission.md adds Checkpoints row to context sources table and checkpoint hint to workflow step. Both are consistent with orchestrator prompts."
     },
-    "events_integration": {
+    "documentation_updates": {
       "status": "pass",
-      "detail": "events.py defines all health check event types (STARTED, COMPLETED, DEGRADED, FAILED) that runner emits"
+      "detail": "checkpoint-access.md adds documenter and coder-revision workflow examples, updates tester example to use --pipeline flag, adds cost command to integrator example. sdlc-pipeline.md adds checkpoint as 4th context persistence mechanism and documents checkpoint discovery hints in prompts."
     },
-    "cli_initialization": {
+    "scope_verification": {
       "status": "pass",
-      "detail": "cli.py registers all checks (4 Tier 1 + 1 Tier 2), stores runner on app.config, wires into container monitor, runs STARTUP checks"
+      "detail": "All 11 changed files (vs merge base) are within issue #887 scope. No unrelated changes mixed in. entrypoint.py/test_entrypoint.py changes visible in full diff are from PR #893 already merged to main before this branch diverged."
     }
   },
   "lint_results": {
     "ruff_check": "pass",
     "ruff_format": "pass",
-    "description": "All source and test files pass ruff check and format verification"
+    "description": "All files pass ruff check and format after integrator fix to test_checkpoint_discovery.py (import sorting and formatting)."
   },
+  "integrator_fixes": [
+    {
+      "file": "orchestrator/tests/test_checkpoint_discovery.py",
+      "commit": "161a24e2b",
+      "description": "Fixed ruff I001 (import block sorting) and ruff format (extra blank line, trailing whitespace). Auto-fixed with ruff check --fix and ruff format."
+    }
+  ],
   "pre_existing_issues": [
     {
       "file": "orchestrator/tests/test_docker_client.py",
-      "description": "3 test failures in test_docker_client.py (TestDockerClientConnection::test_is_connected_false, TestContainerCreation::test_create_container_image_not_found, TestContainerOperations::test_start_container_not_found). These are pre-existing on origin/main and unrelated to issue #850 changes."
+      "tests": ["TestDockerClientConnection::test_is_connected_false", "TestContainerOperations::test_start_container_not_found"],
+      "description": "2 pre-existing test failures in test_docker_client.py. Files not modified on this branch. Failures exist on origin/main."
     }
   ],
-  "files_changed": 36,
-  "lines_added_approx": 10802,
-  "integration_fixes_needed": 0,
-  "verdict": "All cross-phase integration points verified clean. No fixes required. The two-tier health check framework (phase-1: core + Tier 1, phase-2: Tier 2 agent inspector) is correctly integrated with the orchestrator lifecycle."
+  "files_changed": 11,
+  "lines_added_approx": 576,
+  "lines_removed_approx": 5,
+  "integration_fixes_needed": 1,
+  "integration_fixes_applied": 1,
+  "verdict": "All checkpoint discoverability changes are clean and well-integrated. Prompt hints, mode commands, rules, and documentation are internally consistent. One minor lint/format fix applied to the new test file. No regressions introduced."
 }