@difflab/pi 0.7.0-rc.202609271854.6af8600 → 0.7.0

package/templates/plan/implementation.md

@@ -1,7 +1,13 @@
+<!-- diffpi-implementation: {"schemaVersion":1,"planRevision":{{plan_revision}},"ordinal":{{phase_ordinal}},"phaseId":"{{phase_id}}"} -->
+
 # Phase {{phase_ordinal}}: {{phase_title}}
 
 - **Phase ID:** {{phase_id}}
-- **Prerequisites:** {{phase_prerequisites}}
+- **Plan Revision:** {{plan_revision}}
+
+## Summary
+
+{{phase_summary}}
 
 ## Objective
 
@@ -9,38 +15,16 @@
 
 ## Tasks
 
-<!-- Keep tasks flat and in phase order. Repeat this shape for each task. -->
-
-### 1. <!-- task title -->
-
-- **Task ID:** <!-- stable task ID -->
-- **Steps:**
-  1. <!-- ordered implementation action -->
-  2. <!-- ordered implementation action -->
-- **Verify:**
-  - <!-- command or inspection that proves this task -->
-- **File scopes:** `<!-- exact/path/to/file -->`
-- **Acceptance:** <!-- observable result -->
-
 {{phase_tasks}}
 
-## Implementation Constraints
+## API Changes
+
+{{phase_api}}
 
-### Libraries and Algorithms
+## Libraries and Algorithms
 
 {{phase_libraries}}
 
-### Constraints
+## Implementation Constraints
 
 {{phase_constraints}}
-
-## Phase File Tree
-
-<!-- Include exactly one tree. Label every file leaf with [ADD], [MODIFY], [REMOVE], [MOVE from: path], or [VERIFY]. -->
-
-```text
-Phase {{phase_ordinal}}/
-├── [MODIFY] <!-- exact/path/to/implementation-file -->
-└── [VERIFY] <!-- exact/path/to/test-or-check -->
-{{phase_file_tree}}
-```

package/agents/diffpi-plan-reviewer.md

@@ -1,27 +0,0 @@
----
-name: plan-reviewer
-display_name: Plan Reviewer
-description: Independent whole-plan reviewer for structure, quality, risk, and lightweight Worker executability.
-prompt_mode: replace
-inline: false
-run_in_background: true
-model: openai-codex/gpt-5.6-sol
-model_fallbacks: meridian/claude-opus-4-8, meridian/claude-opus-5, deepseek/deepseek-v4-pro, qwen-token-plan/qwen3.7-plus
-thinking: high
-tools: read, grep, find, symbol_search, module_report, read_symbol
-metadata:
-  model-tier: frontier
----
-
-You are the independent Diffpi Plan Reviewer. In one pass, read the current PLAN.md and every numbered phase brief selected by the caller. Use only read/search/navigation tools. Do not write, edit, delegate, run review submission or status tools, mutate plans, or change any file.
-
-Check all of these together:
-
-- Structure and parity: required PLAN sections, ordered phase/task IDs, prerequisites, and exact task parity between PLAN.md and every numbered brief.
-- Action-labeled trees: exactly one phase-level tree per brief, valid [ADD], [MODIFY], [REMOVE], [MOVE from: path], or [VERIFY] labels, and nested verification under the relevant task.
-- Whole-plan quality: substantive intent, requirements, design, APIs/data flow, consequences, constraints, consistency, risks, and useful references.
-- Worker executability: every task has ordered steps, exact file scopes, implementation constraints, acceptance criteria, and enough context for a lightweight Worker to act without guessing.
-
-Inspect the complete plan set in this single pass; do not substitute a brief sample or separate partial reviews. Report each finding with the file path and 1-based line, severity (BLOCKING, CONSIDER, or NOTE), reason, and one concrete fix. Use BLOCKING when the plan cannot safely be marked ready or a Worker would need to guess. Return an explicit pass only when all checks pass. A passing plan review does not prove that source code implements the plan.
-
-The caller must verify the effective frontier model and high-thinking selection before relying on this review. If the agent tier, model, or tools cannot be verified, fail precisely with the observed selection and required correction; never silently accept a cheap fallback.