planpong 0.3.0 → 0.5.0

package/dist/src/prompts/planner.js

@@ -17,7 +17,10 @@ Output ONLY the markdown plan. No preamble, no commentary.
 
 ${requirements}`;
 }
-export function buildRevisionPrompt(currentPlan, feedback, keyDecisions, priorContext, phase = "detail") {
+export function buildRevisionPrompt(currentPlan, feedback, keyDecisions, priorContext, phase = "detail", structuredOutput = false, revisionMode = "full") {
+    // Direction phase always uses full-plan output (sweeping rewrites are
+    // expected). Risk + detail phases honor `revisionMode`.
+    const useEdits = revisionMode === "edits" && phase !== "direction";
     const contextBlock = priorContext
         ? `\n## Prior Research & Constraints\n\n${priorContext}\n`
         : "";
@@ -104,7 +107,65 @@ For each response, cite specific evidence: reference the plan section, the resea
 - Do not reorganize or rephrase parts of the plan unrelated to risk feedback.`
             : `- Only modify sections of the plan that are directly addressed by accepted feedback. Do not reorganize, rephrase, or "improve" parts of the plan that aren't related to any issue.
 - Preserve the plan's existing structure, headings, and formatting. Your job is surgical revision, not rewriting.`;
-    return `${roleInstructions}
+    const fullSchemaBlock = `\`\`\`
+{
+  "responses": [
+    {
+      "issue_id": "F1",
+      "action": "accepted" | "rejected" | "deferred",
+      "severity_dispute": {                    // optional
+        "original": "P1",
+        "revised": "P2",
+        "justification": "..."
+      },
+      "rationale": "Detailed explanation of why this action was taken"
+    }
+  ],
+  "updated_plan": "The full updated plan in markdown (incorporate accepted changes)"
+}
+\`\`\``;
+    // Edits-mode schema for risk + detail phases: planner emits a structured
+    // edit list instead of the full plan. Each edit is section-scoped — the
+    // applier locates the section heading, then finds `before` within that
+    // section's content. `before` MUST be unique within the section.
+    const editsSchemaBlock = `\`\`\`
+{
+  "responses": [
+    {
+      "issue_id": "F1",
+      "action": "accepted" | "rejected" | "deferred",
+      "severity_dispute": {                    // optional
+        "original": "P1",
+        "revised": "P2",
+        "justification": "..."
+      },
+      "rationale": "Detailed explanation of why this action was taken"
+    }
+  ],
+  "edits": [
+    {
+      "section": "Steps",
+      "before": "verbatim text from the plan to replace (must appear EXACTLY ONCE within the named section)",
+      "after": "replacement text (may be empty for deletion)"
+    }
+  ]
+}
+\`\`\``;
+    const schemaBlock = useEdits ? editsSchemaBlock : fullSchemaBlock;
+    // Edits-mode constraints. The applier is strict: section-scoped lookup,
+    // unique-match-within-section enforcement, no plan-wide fallback. The
+    // prompt has to make those constraints visible so the planner produces
+    // edits that actually apply.
+    const editsConstraints = `- Output edits, NOT the full plan. The plan is updated server-side by replaying your edits.
+- Each edit's \`section\` is the nearest markdown heading label (e.g., "Steps", "Limitations & Future Work"). The applier searches ONLY within that section's content.
+- Each edit's \`before\` must appear EXACTLY ONCE within the named section, character-for-character (whitespace tolerant). If the same text appears multiple times in the section, expand \`before\` with surrounding context until it is unique within the section.
+- Use the SHORTEST \`before\` that is unambiguous. Do not quote large unchanged blocks.
+- \`after\` is the replacement. Empty string deletes; non-empty replaces.
+- For an addition, set \`before\` to a short stable anchor (e.g., the line before the insertion point) and \`after\` to that anchor plus the new content.
+- Edits run sequentially: later edits see earlier edits' results. If you have multiple edits in the same section, order them so each one's \`before\` is unique against the running plan state.
+- Keep the total number of edits small — only what is needed to address accepted issues. One issue typically maps to one edit.`;
+    const fullModeConstraint = `- The \`updated_plan\` must be the complete plan markdown, not a diff.`;
+    const commonBody = `${roleInstructions}
 ${contextBlock}${decisionsBlock}
 ## Current Plan
 
@@ -116,35 +177,229 @@ ${currentPlan}
 
 ${issuesList}
 
-## Your Task
+## Your Task`;
+    const outputConstraint = useEdits ? editsConstraints : fullModeConstraint;
+    if (structuredOutput) {
+        // Structured-output mode. Some providers constrain output at the token
+        // level; others only validate post-hoc. Emphatic JSON-only instructions
+        // help advisory providers comply; constrained providers ignore them.
+        return `${commonBody}
+
+Output ONLY a single JSON object conforming to the schema below. The first character of your response must be \`{\` and the last must be \`}\`. No prose. No markdown. No code fences. No preamble or explanation. No trailing text.
+
+Schema:
+
+${schemaBlock}
+
+Constraints embedded in your JSON response:
+- Every issue MUST have an entry in \`responses\`. Do not skip any.
+${outputConstraint}
+${surgicalConstraint}
+- Do NOT modify the \`**planpong:**\` status line — it is managed automatically.`;
+    }
+    return `${commonBody}
 
 Respond with a JSON object wrapped in <planpong-revision> tags. The JSON must match this schema:
 
-\`\`\`
+${schemaBlock}
+
+IMPORTANT:
+- Every issue MUST have a response. Do not skip any.
+${outputConstraint}
+${surgicalConstraint}
+- Do NOT modify the \`**planpong:**\` status line — it is managed automatically.
+- Wrap your JSON response in <planpong-revision>...</planpong-revision> tags.
+
+<planpong-revision>
+YOUR_JSON_HERE
+</planpong-revision>`;
+}
+/**
+ * Build a minimal revision prompt for resumed planner sessions.
+ *
+ * The planner is already in a persistent CLI conversation that has the
+ * plan, the prior reviewer feedback, and the planner's own prior rationales
+ * in context. We do NOT re-send "Current Plan", "Prior Decisions", or
+ * "Key Decisions" — the model has all of that. Only the new feedback +
+ * minimal phase reminder + output schema instructions.
+ *
+ * The output schema and surgical constraints stay because they're per-call
+ * directives, not stable context. (We don't trust that the model won't
+ * drift in format across many turns of a long session.)
+ */
+export function buildIncrementalRevisionPrompt(feedback, phase, structuredOutput, revisionMode = "full") {
+    const useEdits = revisionMode === "edits" && phase !== "direction";
+    const issuesList = feedback.issues
+        .map((issue) => `### ${issue.id} (${issue.severity}): ${issue.title}\n**Section:** ${issue.section}\n**Description:** ${issue.description}\n**Suggestion:** ${issue.suggestion}`)
+        .join("\n\n");
+    const phaseLabel = phase === "direction"
+        ? "DIRECTION"
+        : phase === "risk"
+            ? "RISK / PRE-MORTEM"
+            : "DETAIL";
+    const surgicalConstraint = phase === "direction"
+        ? `- You may make structural changes (reorder sections, change approach, adjust scope) if accepted feedback warrants it.
+- Preserve sections that aren't affected by the feedback.`
+        : phase === "risk"
+            ? `- Add mitigations, verification steps, or fallback procedures for accepted risks.
+- You may add new sections (e.g., "Risks & Mitigations") if needed.
+- Do not reorganize or rephrase parts of the plan unrelated to risk feedback.`
+            : `- Only modify sections of the plan that are directly addressed by accepted feedback.
+- Preserve the plan's existing structure, headings, and formatting. Surgical revision, not rewriting.`;
+    const fullSchemaBlock = `\`\`\`
 {
   "responses": [
-    {
-      "issue_id": "F1",
-      "action": "accepted" | "rejected" | "deferred",
-      "severity_dispute": {                    // optional
-        "original": "P1",
-        "revised": "P2",
-        "justification": "..."
-      },
-      "rationale": "Detailed explanation of why this action was taken"
-    }
+    { "issue_id": "F1", "action": "accepted" | "rejected" | "deferred",
+      "severity_dispute": { "original": "P1", "revised": "P2", "justification": "..." },  // optional
+      "rationale": "Why this action" }
   ],
-  "updated_plan": "The full updated plan in markdown (incorporate accepted changes)"
+  "updated_plan": "The full updated plan in markdown"
 }
-\`\`\`
+\`\`\``;
+    const editsSchemaBlock = `\`\`\`
+{
+  "responses": [
+    { "issue_id": "F1", "action": "accepted" | "rejected" | "deferred",
+      "severity_dispute": { "original": "P1", "revised": "P2", "justification": "..." },  // optional
+      "rationale": "Why this action" }
+  ],
+  "edits": [
+    { "section": "Steps", "before": "verbatim text from the plan (must appear EXACTLY ONCE within the named section)", "after": "replacement" }
+  ]
+}
+\`\`\``;
+    const schemaBlock = useEdits ? editsSchemaBlock : fullSchemaBlock;
+    const editsConstraints = `- Output edits, NOT the full plan. The plan is updated server-side by replaying your edits.
+- Each edit's \`section\` is the nearest markdown heading. \`before\` must appear EXACTLY ONCE within that section.
+- Use the SHORTEST \`before\` that is unambiguous within the section.
+- For an addition: \`before\` = a stable anchor; \`after\` = anchor + new content.
+- Edits run sequentially: later edits see earlier edits' results.`;
+    const fullModeConstraint = `- The \`updated_plan\` must be the complete plan markdown, not a diff.`;
+    const outputConstraint = useEdits ? editsConstraints : fullModeConstraint;
+    const body = `## ${phaseLabel} ROUND — Round ${"current round in your session memory"}: New Reviewer Feedback
+
+You are continuing the same revision conversation. The plan, your prior rationales, and the prior rounds of reviewer feedback are already in your context — do not re-emit them.
+
+The reviewer has produced new feedback for this round. Process it below.
+
+**Summary:** ${feedback.summary}
+
+${issuesList}
+
+## Your Task`;
+    if (structuredOutput) {
+        return `${body}
+
+Output ONLY a single JSON object conforming to the schema below. The first character of your response must be \`{\` and the last must be \`}\`. No prose. No markdown. No code fences. No preamble or explanation. No trailing text.
+
+Schema:
+
+${schemaBlock}
+
+Constraints:
+- Every issue MUST have an entry in \`responses\`. Do not skip any.
+${outputConstraint}
+${surgicalConstraint}
+- Do NOT modify the \`**planpong:**\` status line — it is managed automatically.`;
+    }
+    return `${body}
+
+Respond with a JSON object wrapped in <planpong-revision> tags conforming to:
+
+${schemaBlock}
 
 IMPORTANT:
 - Every issue MUST have a response. Do not skip any.
-- The \`updated_plan\` must be the complete plan markdown, not a diff.
+${outputConstraint}
 ${surgicalConstraint}
 - Do NOT modify the \`**planpong:**\` status line — it is managed automatically.
 - Wrap your JSON response in <planpong-revision>...</planpong-revision> tags.
 
+<planpong-revision>
+YOUR_JSON_HERE
+</planpong-revision>`;
+}
+/**
+ * Build a targeted retry prompt for failed edits in edits-mode revisions.
+ * Given the partially-edited plan and the list of edits that failed first
+ * pass, asks the planner to re-express each failed edit with corrected
+ * `section` and `before` values.
+ *
+ * The retry prompt is small — it does not re-include the full feedback or
+ * key decisions, only the failed edits and the current state of the plan.
+ */
+export function buildEditsRetryPrompt(currentPlan, failures, structuredOutput) {
+    const failureBlock = failures
+        .map((f, i) => {
+        const reasonHelp = f.reason === "no-match"
+            ? "Your `before` did not match any text in that section. Re-quote a verbatim string from the plan."
+            : f.reason === "multi-match"
+                ? "Your `before` matched multiple times in that section. Add surrounding context until it is unique within the section."
+                : f.reason === "section-not-found"
+                    ? "The section heading does not exist in the plan. Pick a heading that does exist."
+                    : f.reason === "status-line"
+                        ? "Your edit modified the **planpong:** status line, which is reserved. Move the change elsewhere."
+                        : "Edit failed. Re-express it.";
+        const diagnosticLine = f.diagnostic ? `\nDiagnostic: ${f.diagnostic}` : "";
+        return `### Failed edit ${i + 1}
+**Section:** ${f.edit.section}
+**Reason:** ${f.reason}${diagnosticLine}
+**Help:** ${reasonHelp}
+**Original \`before\`:**
+\`\`\`
+${f.edit.before}
+\`\`\`
+**Original \`after\`:**
+\`\`\`
+${f.edit.after}
+\`\`\``;
+    })
+        .join("\n\n");
+    const schemaBlock = `\`\`\`
+{
+  "edits": [
+    { "section": "Steps", "before": "verbatim", "after": "replacement" }
+  ]
+}
+\`\`\``;
+    const body = `You are correcting failed edits. The previous edit list partially applied; the edits below failed at the apply step.
+
+For EACH failed edit, produce a corrected edit with the same intent but a working \`section\` and \`before\`. You may also revise \`after\` if the previous version assumed text that is no longer present.
+
+## Current Plan (with previously-successful edits already applied)
+
+${currentPlan}
+
+## Failed Edits
+
+${failureBlock}
+
+## Your Task`;
+    if (structuredOutput) {
+        return `${body}
+
+Output ONLY a single JSON object with an \`edits\` array. The first character of your response must be \`{\` and the last must be \`}\`. Do NOT include \`responses\` — those are already finalized.
+
+Schema:
+
+${schemaBlock}
+
+- Output one corrected edit per failure (same count, same order).
+- Each \`before\` must appear EXACTLY ONCE within the named section in the current plan above.
+- Do NOT modify the \`**planpong:**\` status line.`;
+    }
+    return `${body}
+
+Respond with a JSON object wrapped in <planpong-revision> tags. The JSON must match this schema:
+
+${schemaBlock}
+
+IMPORTANT:
+- Output one corrected edit per failure (same count, same order). Do NOT include \`responses\`.
+- Each \`before\` must appear EXACTLY ONCE within the named section in the current plan above.
+- Do NOT modify the \`**planpong:**\` status line.
+- Wrap your JSON response in <planpong-revision>...</planpong-revision> tags.
+
 <planpong-revision>
 YOUR_JSON_HERE
 </planpong-revision>`;

package/dist/src/prompts/planner.js.map

@@ -1 +1 @@
-{"version":3,"file":"planner.js","sourceRoot":"","sources":["../../../src/prompts/planner.ts"],"names":[],"mappings":"AAGA,MAAM,UAAU,sBAAsB,CACpC,YAAoB,EACpB,QAAgB;IAEhB,OAAO;;;;;;;;;;;;;;;;EAgBP,YAAY,EAAE,CAAC;AACjB,CAAC;AAED,MAAM,UAAU,mBAAmB,CACjC,WAAmB,EACnB,QAAwB,EACxB,YAA2B,EAC3B,YAA2B,EAC3B,QAAqB,QAAQ;IAE7B,MAAM,YAAY,GAAG,YAAY;QAC/B,CAAC,CAAC,wCAAwC,YAAY,IAAI;QAC1D,CAAC,CAAC,EAAE,CAAC;IAEP,MAAM,cAAc,GAAG,YAAY;QACjC,CAAC,CAAC,mCAAmC,YAAY,IAAI;QACrD,CAAC,CAAC,EAAE,CAAC;IAEP,MAAM,UAAU,GAAG,QAAQ,CAAC,MAAM;SAC/B,GAAG,CACF,CAAC,KAAK,EAAE,EAAE,CACR,OAAO,KAAK,CAAC,EAAE,KAAK,KAAK,CAAC,QAAQ,MAAM,KAAK,CAAC,KAAK,kBAAkB,KAAK,CAAC,OAAO,sBAAsB,KAAK,CAAC,WAAW,qBAAqB,KAAK,CAAC,UAAU,EAAE,CACnK;SACA,IAAI,CAAC,MAAM,CAAC,CAAC;IAEhB,MAAM,qBAAqB,GAAG;;;;;;;;;;;;;;;;;;oLAkBoJ,CAAC;IAEnL,MAAM,gBAAgB,GAAG;;;;;;;;;;;;;;;;;;;4KAmBiJ,CAAC;IAE3K,MAAM,kBAAkB,GAAG;;;;;;;;;;;;;;;;;;;;;;;mTAuBsR,CAAC;IAElT,MAAM,gBAAgB,GACpB,KAAK,KAAK,WAAW;QACnB,CAAC,CAAC,qBAAqB;QACvB,CAAC,CAAC,KAAK,KAAK,MAAM;YAChB,CAAC,CAAC,gBAAgB;YAClB,CAAC,CAAC,kBAAkB,CAAC;IAE3B,MAAM,kBAAkB,GACtB,KAAK,KAAK,WAAW;QACnB,CAAC,CAAC;0DACkD;QACpD,CAAC,CAAC,KAAK,KAAK,MAAM;YAChB,CAAC,CAAC;;8EAEoE;YACtE,CAAC,CAAC;kHACwG,CAAC;IAEjH,OAAO,GAAG,gBAAgB;EAC1B,YAAY,GAAG,cAAc;;;EAG7B,WAAW;;;;eAIE,QAAQ,CAAC,OAAO;;EAE7B,UAAU;;;;;;;;;;;;;;;;;;;;;;;;;;;EA2BV,kBAAkB;;;;;;qBAMC,CAAC;AACtB,CAAC"}
+{"version":3,"file":"planner.js","sourceRoot":"","sources":["../../../src/prompts/planner.ts"],"names":[],"mappings":"AAGA,MAAM,UAAU,sBAAsB,CACpC,YAAoB,EACpB,QAAgB;IAEhB,OAAO;;;;;;;;;;;;;;;;EAgBP,YAAY,EAAE,CAAC;AACjB,CAAC;AAED,MAAM,UAAU,mBAAmB,CACjC,WAAmB,EACnB,QAAwB,EACxB,YAA2B,EAC3B,YAA2B,EAC3B,QAAqB,QAAQ,EAC7B,mBAA4B,KAAK,EACjC,eAAiC,MAAM;IAEvC,sEAAsE;IACtE,wDAAwD;IACxD,MAAM,QAAQ,GAAG,YAAY,KAAK,OAAO,IAAI,KAAK,KAAK,WAAW,CAAC;IACnE,MAAM,YAAY,GAAG,YAAY;QAC/B,CAAC,CAAC,wCAAwC,YAAY,IAAI;QAC1D,CAAC,CAAC,EAAE,CAAC;IAEP,MAAM,cAAc,GAAG,YAAY;QACjC,CAAC,CAAC,mCAAmC,YAAY,IAAI;QACrD,CAAC,CAAC,EAAE,CAAC;IAEP,MAAM,UAAU,GAAG,QAAQ,CAAC,MAAM;SAC/B,GAAG,CACF,CAAC,KAAK,EAAE,EAAE,CACR,OAAO,KAAK,CAAC,EAAE,KAAK,KAAK,CAAC,QAAQ,MAAM,KAAK,CAAC,KAAK,kBAAkB,KAAK,CAAC,OAAO,sBAAsB,KAAK,CAAC,WAAW,qBAAqB,KAAK,CAAC,UAAU,EAAE,CACnK;SACA,IAAI,CAAC,MAAM,CAAC,CAAC;IAEhB,MAAM,qBAAqB,GAAG;;;;;;;;;;;;;;;;;;oLAkBoJ,CAAC;IAEnL,MAAM,gBAAgB,GAAG;;;;;;;;;;;;;;;;;;;4KAmBiJ,CAAC;IAE3K,MAAM,kBAAkB,GAAG;;;;;;;;;;;;;;;;;;;;;;;mTAuBsR,CAAC;IAElT,MAAM,gBAAgB,GACpB,KAAK,KAAK,WAAW;QACnB,CAAC,CAAC,qBAAqB;QACvB,CAAC,CAAC,KAAK,KAAK,MAAM;YAChB,CAAC,CAAC,gBAAgB;YAClB,CAAC,CAAC,kBAAkB,CAAC;IAE3B,MAAM,kBAAkB,GACtB,KAAK,KAAK,WAAW;QACnB,CAAC,CAAC;0DACkD;QACpD,CAAC,CAAC,KAAK,KAAK,MAAM;YAChB,CAAC,CAAC;;8EAEoE;YACtE,CAAC,CAAC;kHACwG,CAAC;IAEjH,MAAM,eAAe,GAAG;;;;;;;;;;;;;;;;OAgBnB,CAAC;IAEN,yEAAyE;IACzE,wEAAwE;IACxE,uEAAuE;IACvE,iEAAiE;IACjE,MAAM,gBAAgB,GAAG;;;;;;;;;;;;;;;;;;;;;;OAsBpB,CAAC;IAEN,MAAM,WAAW,GAAG,QAAQ,CAAC,CAAC,CAAC,gBAAgB,CAAC,CAAC,CAAC,eAAe,CAAC;IAElE,wEAAwE;IACxE,sEAAsE;IACtE,uEAAuE;IACvE,6BAA6B;IAC7B,MAAM,gBAAgB,GAAG;;;;;;;+HAOoG,CAAC;IAC9H,MAAM,kBAAkB,GAAG,wEAAwE,CAAC;IAEpG,MAAM,UAAU,GAAG,GAAG,gBAAgB;EACtC,YAAY,GAAG,cAAc;;;EAG7B,WAAW;;;;eAIE,QAAQ,CAAC,OAAO;;EAE7B,UAAU;;aAEC,CAAC;IAEZ,MAAM,gBAAgB,GAAG,QAAQ,CAAC,CAAC,CAAC,gBAAgB,CAAC,CAAC,CAAC,kBAAkB,CAAC;IAE1E,IAAI,gBAAgB,EAAE,CAAC;QACrB,uEAAuE;QACvE,wEAAwE;QACxE,qEAAqE;QACrE,OAAO,GAAG,UAAU;;;;;;EAMtB,WAAW;;;;EAIX,gBAAgB;EAChB,kBAAkB;iFAC6D,CAAC;IAChF,CAAC;IAED,OAAO,GAAG,UAAU;;;;EAIpB,WAAW;;;;EAIX,gBAAgB;EAChB,kBAAkB;;;;;;qBAMC,CAAC;AACtB,CAAC;AAED;;;;;;;;;;;;GAYG;AACH,MAAM,UAAU,8BAA8B,CAC5C,QAAwB,EACxB,KAAkB,EAClB,gBAAyB,EACzB,eAAiC,MAAM;IAEvC,MAAM,QAAQ,GAAG,YAAY,KAAK,OAAO,IAAI,KAAK,KAAK,WAAW,CAAC;IAEnE,MAAM,UAAU,GAAG,QAAQ,CAAC,MAAM;SAC/B,GAAG,CACF,CAAC,KAAK,EAAE,EAAE,CACR,OAAO,KAAK,CAAC,EAAE,KAAK,KAAK,CAAC,QAAQ,MAAM,KAAK,CAAC,KAAK,kBAAkB,KAAK,CAAC,OAAO,sBAAsB,KAAK,CAAC,WAAW,qBAAqB,KAAK,CAAC,UAAU,EAAE,CACnK;SACA,IAAI,CAAC,MAAM,CAAC,CAAC;IAEhB,MAAM,UAAU,GACd,KAAK,KAAK,WAAW;QACnB,CAAC,CAAC,WAAW;QACb,CAAC,CAAC,KAAK,KAAK,MAAM;YAChB,CAAC,CAAC,mBAAmB;YACrB,CAAC,CAAC,QAAQ,CAAC;IAEjB,MAAM,kBAAkB,GACtB,KAAK,KAAK,WAAW;QACnB,CAAC,CAAC;0DACkD;QACpD,CAAC,CAAC,KAAK,KAAK,MAAM;YAChB,CAAC,CAAC;;8EAEoE;YACtE,CAAC,CAAC;sGAC4F,CAAC;IAErG,MAAM,eAAe,GAAG;;;;;;;;;OASnB,CAAC;IAEN,MAAM,gBAAgB,GAAG;;;;;;;;;;;OAWpB,CAAC;IAEN,MAAM,WAAW,GAAG,QAAQ,CAAC,CAAC,CAAC,gBAAgB,CAAC,CAAC,CAAC,eAAe,CAAC;IAElE,MAAM,gBAAgB,GAAG;;;;kEAIuC,CAAC;IACjE,MAAM,kBAAkB,GAAG,wEAAwE,CAAC;IACpG,MAAM,gBAAgB,GAAG,QAAQ,CAAC,CAAC,CAAC,gBAAgB,CAAC,CAAC,CAAC,kBAAkB,CAAC;IAE1E,MAAM,IAAI,GAAG,MAAM,UAAU,kBAAkB,sCAAsC;;;;;;eAMxE,QAAQ,CAAC,OAAO;;EAE7B,UAAU;;aAEC,CAAC;IAEZ,IAAI,gBAAgB,EAAE,CAAC;QACrB,OAAO,GAAG,IAAI;;;;;;EAMhB,WAAW;;;;EAIX,gBAAgB;EAChB,kBAAkB;iFAC6D,CAAC;IAChF,CAAC;IAED,OAAO,GAAG,IAAI;;;;EAId,WAAW;;;;EAIX,gBAAgB;EAChB,kBAAkB;;;;;;qBAMC,CAAC;AACtB,CAAC;AAED;;;;;;;;GAQG;AACH,MAAM,UAAU,qBAAqB,CACnC,WAAmB,EACnB,QAKE,EACF,gBAAyB;IAEzB,MAAM,YAAY,GAAG,QAAQ;SAC1B,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE;QACZ,MAAM,UAAU,GACd,CAAC,CAAC,MAAM,KAAK,UAAU;YACrB,CAAC,CAAC,iGAAiG;YACnG,CAAC,CAAC,CAAC,CAAC,MAAM,KAAK,aAAa;gBAC1B,CAAC,CAAC,sHAAsH;gBACxH,CAAC,CAAC,CAAC,CAAC,MAAM,KAAK,mBAAmB;oBAChC,CAAC,CAAC,iFAAiF;oBACnF,CAAC,CAAC,CAAC,CAAC,MAAM,KAAK,aAAa;wBAC1B,CAAC,CAAC,iGAAiG;wBACnG,CAAC,CAAC,6BAA6B,CAAC;QAC1C,MAAM,cAAc,GAAG,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,iBAAiB,CAAC,CAAC,UAAU,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAC3E,OAAO,mBAAmB,CAAC,GAAG,CAAC;eACtB,CAAC,CAAC,IAAI,CAAC,OAAO;cACf,CAAC,CAAC,MAAM,GAAG,cAAc;YAC3B,UAAU;;;EAGpB,CAAC,CAAC,IAAI,CAAC,MAAM;;;;EAIb,CAAC,CAAC,IAAI,CAAC,KAAK;OACP,CAAC;IACJ,CAAC,CAAC;SACD,IAAI,CAAC,MAAM,CAAC,CAAC;IAEhB,MAAM,WAAW,GAAG;;;;;;OAMf,CAAC;IAEN,MAAM,IAAI,GAAG;;;;;;EAMb,WAAW;;;;EAIX,YAAY;;aAED,CAAC;IAEZ,IAAI,gBAAgB,EAAE,CAAC;QACrB,OAAO,GAAG,IAAI;;;;;;EAMhB,WAAW;;;;mDAIsC,CAAC;IAClD,CAAC;IAED,OAAO,GAAG,IAAI;;;;EAId,WAAW;;;;;;;;;;qBAUQ,CAAC;AACtB,CAAC"}

package/dist/src/prompts/reviewer.d.ts

@@ -1,7 +1,20 @@
 import type { IssueResponse } from "../schemas/revision.js";
 export type ReviewPhase = "direction" | "risk" | "detail";
 export declare function getReviewPhase(round: number): ReviewPhase;
-export declare function buildReviewPrompt(planContent: string, priorDecisions: string | null, phase?: ReviewPhase): string;
+/**
+ * Incremental review prompt for resumed reviewer sessions.
+ *
+ * The reviewer has already seen the full plan (round 1) and produced its
+ * own prior critique. Instead of re-sending the full plan markdown, we
+ * send only what's changed since the model last saw it (a markdown diff)
+ * plus the new phase instructions.
+ *
+ * Falls back to full-plan content if `planDiffOrContent` is the entire
+ * plan rather than a diff (caller's choice — see operations.ts logic that
+ * skips diffing on certain cases).
+ */
+export declare function buildIncrementalReviewPrompt(planDiffOrContent: string, priorDecisions: string | null, phase?: ReviewPhase, structuredOutput?: boolean): string;
+export declare function buildReviewPrompt(planContent: string, priorDecisions: string | null, phase?: ReviewPhase, structuredOutput?: boolean): string;
 export declare function formatPriorDecisions(rounds: Array<{
     round: number;
     responses: IssueResponse[];

package/dist/src/prompts/reviewer.js

@@ -74,6 +74,7 @@ function buildDetailReviewInstructions(priorDecisions) {
 
 The plan's overall direction has already been validated. Focus on implementation completeness and correctness.
 
+- When the plan references files, function names, exports, imports, configs, or APIs, VERIFY them against the codebase using your tools (Read, Grep, Glob, Bash). A plan that references \`src/foo/bar.ts\` should have a corresponding file; a plan that calls \`someLib.thing()\` should match the library's actual API. Flag verifiable mismatches as P1 (would break implementation) — these are the issues only an independent reviewer catches.
 - Only flag issues you have concrete evidence for. Cite the plan section.
 - Assign severity honestly:
   - P1 = blocks implementation or causes failure
@@ -180,7 +181,19 @@ If the plan is approved with no issues, use:
 { "verdict": "approved", "summary": "...", "issues": [] }
 \`\`\``;
 }
-export function buildReviewPrompt(planContent, priorDecisions, phase = "detail") {
+/**
+ * Incremental review prompt for resumed reviewer sessions.
+ *
+ * The reviewer has already seen the full plan (round 1) and produced its
+ * own prior critique. Instead of re-sending the full plan markdown, we
+ * send only what's changed since the model last saw it (a markdown diff)
+ * plus the new phase instructions.
+ *
+ * Falls back to full-plan content if `planDiffOrContent` is the entire
+ * plan rather than a diff (caller's choice — see operations.ts logic that
+ * skips diffing on certain cases).
+ */
+export function buildIncrementalReviewPrompt(planDiffOrContent, priorDecisions, phase = "detail", structuredOutput = false) {
     const instructions = phase === "direction"
         ? buildDirectionReviewInstructions()
         : phase === "risk"
@@ -191,6 +204,76 @@ export function buildReviewPrompt(planContent, priorDecisions, phase = "detail")
         : phase === "risk"
             ? buildRiskJsonSchema()
             : buildDetailJsonSchema();
+    const isDiff = planDiffOrContent.startsWith("```diff");
+    const planSection = isDiff
+        ? `## Plan Changes Since Last Round
+
+The plan has been revised in response to your prior feedback. Below is what changed (the rest of the plan is unchanged from your context).
+
+${planDiffOrContent}`
+        : `## Plan to Review
+
+${planDiffOrContent}`;
+    if (structuredOutput) {
+        return `${instructions}
+${planSection}
+
+## Your Task
+
+You are continuing the same review conversation. Your prior round's feedback and the plan you reviewed are in your context — use them.
+
+Output ONLY a single JSON object conforming to the schema below. The first character of your response must be \`{\` and the last must be \`}\`. No prose. No markdown. No code fences. No preamble or explanation. No trailing text.
+
+Schema:
+
+${jsonSchema}`;
+    }
+    return `${instructions}
+${planSection}
+
+## Your Task
+
+You are continuing the same review conversation. Your prior round's feedback and the plan you reviewed are in your context — use them.
+
+Respond with a JSON object wrapped in <planpong-feedback> tags conforming to:
+
+${jsonSchema}
+
+IMPORTANT: Wrap your JSON response in <planpong-feedback>...</planpong-feedback> tags.
+
+<planpong-feedback>
+YOUR_JSON_HERE
+</planpong-feedback>`;
+}
+export function buildReviewPrompt(planContent, priorDecisions, phase = "detail", structuredOutput = false) {
+    const instructions = phase === "direction"
+        ? buildDirectionReviewInstructions()
+        : phase === "risk"
+            ? buildRiskReviewInstructions(priorDecisions)
+            : buildDetailReviewInstructions(priorDecisions);
+    const jsonSchema = phase === "direction"
+        ? buildDirectionJsonSchema()
+        : phase === "risk"
+            ? buildRiskJsonSchema()
+            : buildDetailJsonSchema();
+    if (structuredOutput) {
+        // Structured-output mode. Some providers (OpenAI/Codex) constrain output
+        // at the token level; others (Claude) only validate post-hoc. Emphatic
+        // JSON-only instructions help the advisory case comply; the constrained
+        // case ignores them harmlessly.
+        return `${instructions}
+## Plan to Review
+
+${planContent}
+
+## Your Task
+
+Output ONLY a single JSON object conforming to the schema below. The first character of your response must be \`{\` and the last must be \`}\`. No prose. No markdown. No code fences. No preamble or explanation. No trailing text.
+
+Schema:
+
+${jsonSchema}`;
+    }
     return `${instructions}
 ## Plan to Review
 

package/dist/src/prompts/reviewer.js.map

@@ -1 +1 @@
-{"version":3,"file":"reviewer.js","sourceRoot":"","sources":["../../../src/prompts/reviewer.ts"],"names":[],"mappings":"AAIA,MAAM,UAAU,cAAc,CAAC,KAAa;IAC1C,IAAI,KAAK,IAAI,CAAC;QAAE,OAAO,WAAW,CAAC;IACnC,IAAI,KAAK,KAAK,CAAC;QAAE,OAAO,MAAM,CAAC;IAC/B,OAAO,QAAQ,CAAC;AAClB,CAAC;AAED,SAAS,gCAAgC;IACvC,OAAO;;;;;;;;;;;;;;;;;;;;;;;;;oGAyB2F,CAAC;AACrG,CAAC;AAED,SAAS,2BAA2B,CAAC,cAA6B;IAChE,MAAM,UAAU,GAAG,cAAc;QAC/B,CAAC,CAAC,iCAAiC,cAAc,+IAA+I;QAChM,CAAC,CAAC,EAAE,CAAC;IAEP,OAAO;;;;;;;;;;;;;;;;;;;;;;;;;;;EA2BP,UAAU,EAAE,CAAC;AACf,CAAC;AAED,SAAS,6BAA6B,CAAC,cAA6B;IAClE,MAAM,UAAU,GAAG,cAAc;QAC/B,CAAC,CAAC,iCAAiC,cAAc,yLAAyL;QAC1O,CAAC,CAAC,EAAE,CAAC;IAEP,OAAO;;;;;;;;;;;;;;EAcP,UAAU,EAAE,CAAC;AACf,CAAC;AAED,SAAS,wBAAwB;IAC/B,OAAO;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;OA+BF,CAAC;AACR,CAAC;AAED,SAAS,mBAAmB;IAC1B,OAAO;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;OAkCF,CAAC;AACR,CAAC;AAED,SAAS,qBAAqB;IAC5B,OAAO;;;;;;;;;;;;;;;;;;;;OAoBF,CAAC;AACR,CAAC;AAED,MAAM,UAAU,iBAAiB,CAC/B,WAAmB,EACnB,cAA6B,EAC7B,QAAqB,QAAQ;IAE7B,MAAM,YAAY,GAChB,KAAK,KAAK,WAAW;QACnB,CAAC,CAAC,gCAAgC,EAAE;QACpC,CAAC,CAAC,KAAK,KAAK,MAAM;YAChB,CAAC,CAAC,2BAA2B,CAAC,cAAc,CAAC;YAC7C,CAAC,CAAC,6BAA6B,CAAC,cAAc,CAAC,CAAC;IAEtD,MAAM,UAAU,GACd,KAAK,KAAK,WAAW;QACnB,CAAC,CAAC,wBAAwB,EAAE;QAC5B,CAAC,CAAC,KAAK,KAAK,MAAM;YAChB,CAAC,CAAC,mBAAmB,EAAE;YACvB,CAAC,CAAC,qBAAqB,EAAE,CAAC;IAEhC,OAAO,GAAG,YAAY;;;EAGtB,WAAW;;;;;;EAMX,UAAU;;;;;;qBAMS,CAAC;AACtB,CAAC;AAED,MAAM,UAAU,oBAAoB,CAClC,MAIE;IAEF,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;QAC3B,KAAK,MAAM,QAAQ,IAAI,KAAK,CAAC,SAAS,EAAE,CAAC;YACvC,MAAM,KAAK,GAAG,KAAK,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,QAAQ,CAAC,QAAQ,CAAC,CAAC;YACnE,MAAM,QAAQ,GAAG,KAAK,EAAE,QAAQ,IAAI,IAAI,CAAC;YACzC,MAAM,KAAK,GAAG,KAAK,EAAE,KAAK,IAAI,QAAQ,CAAC,QAAQ,CAAC;YAChD,MAAM,MAAM,GAAG,QAAQ,CAAC,MAAM,CAAC,WAAW,EAAE,CAAC;YAC7C,MAAM,SAAS,GACb,QAAQ,CAAC,SAAS,CAAC,MAAM,GAAG,EAAE;gBAC5B,CAAC,CAAC,QAAQ,CAAC,SAAS,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,KAAK;gBACzC,CAAC,CAAC,QAAQ,CAAC,SAAS,CAAC;YACzB,KAAK,CAAC,IAAI,CACR,MAAM,KAAK,CAAC,KAAK,IAAI,QAAQ,CAAC,QAAQ,KAAK,QAAQ,MAAM,KAAK,MAAM,MAAM,KAAK,SAAS,GAAG,CAC5F,CAAC;QACJ,CAAC;IACH,CAAC;IACD,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC"}
+{"version":3,"file":"reviewer.js","sourceRoot":"","sources":["../../../src/prompts/reviewer.ts"],"names":[],"mappings":"AAIA,MAAM,UAAU,cAAc,CAAC,KAAa;IAC1C,IAAI,KAAK,IAAI,CAAC;QAAE,OAAO,WAAW,CAAC;IACnC,IAAI,KAAK,KAAK,CAAC;QAAE,OAAO,MAAM,CAAC;IAC/B,OAAO,QAAQ,CAAC;AAClB,CAAC;AAED,SAAS,gCAAgC;IACvC,OAAO;;;;;;;;;;;;;;;;;;;;;;;;;oGAyB2F,CAAC;AACrG,CAAC;AAED,SAAS,2BAA2B,CAAC,cAA6B;IAChE,MAAM,UAAU,GAAG,cAAc;QAC/B,CAAC,CAAC,iCAAiC,cAAc,+IAA+I;QAChM,CAAC,CAAC,EAAE,CAAC;IAEP,OAAO;;;;;;;;;;;;;;;;;;;;;;;;;;;EA2BP,UAAU,EAAE,CAAC;AACf,CAAC;AAED,SAAS,6BAA6B,CAAC,cAA6B;IAClE,MAAM,UAAU,GAAG,cAAc;QAC/B,CAAC,CAAC,iCAAiC,cAAc,yLAAyL;QAC1O,CAAC,CAAC,EAAE,CAAC;IAEP,OAAO;;;;;;;;;;;;;;;EAeP,UAAU,EAAE,CAAC;AACf,CAAC;AAED,SAAS,wBAAwB;IAC/B,OAAO;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;OA+BF,CAAC;AACR,CAAC;AAED,SAAS,mBAAmB;IAC1B,OAAO;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;OAkCF,CAAC;AACR,CAAC;AAED,SAAS,qBAAqB;IAC5B,OAAO;;;;;;;;;;;;;;;;;;;;OAoBF,CAAC;AACR,CAAC;AAED;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,4BAA4B,CAC1C,iBAAyB,EACzB,cAA6B,EAC7B,QAAqB,QAAQ,EAC7B,mBAA4B,KAAK;IAEjC,MAAM,YAAY,GAChB,KAAK,KAAK,WAAW;QACnB,CAAC,CAAC,gCAAgC,EAAE;QACpC,CAAC,CAAC,KAAK,KAAK,MAAM;YAChB,CAAC,CAAC,2BAA2B,CAAC,cAAc,CAAC;YAC7C,CAAC,CAAC,6BAA6B,CAAC,cAAc,CAAC,CAAC;IAEtD,MAAM,UAAU,GACd,KAAK,KAAK,WAAW;QACnB,CAAC,CAAC,wBAAwB,EAAE;QAC5B,CAAC,CAAC,KAAK,KAAK,MAAM;YAChB,CAAC,CAAC,mBAAmB,EAAE;YACvB,CAAC,CAAC,qBAAqB,EAAE,CAAC;IAEhC,MAAM,MAAM,GAAG,iBAAiB,CAAC,UAAU,CAAC,SAAS,CAAC,CAAC;IACvD,MAAM,WAAW,GAAG,MAAM;QACxB,CAAC,CAAC;;;;EAIJ,iBAAiB,EAAE;QACjB,CAAC,CAAC;;EAEJ,iBAAiB,EAAE,CAAC;IAEpB,IAAI,gBAAgB,EAAE,CAAC;QACrB,OAAO,GAAG,YAAY;EACxB,WAAW;;;;;;;;;;EAUX,UAAU,EAAE,CAAC;IACb,CAAC;IAED,OAAO,GAAG,YAAY;EACtB,WAAW;;;;;;;;EAQX,UAAU;;;;;;qBAMS,CAAC;AACtB,CAAC;AAED,MAAM,UAAU,iBAAiB,CAC/B,WAAmB,EACnB,cAA6B,EAC7B,QAAqB,QAAQ,EAC7B,mBAA4B,KAAK;IAEjC,MAAM,YAAY,GAChB,KAAK,KAAK,WAAW;QACnB,CAAC,CAAC,gCAAgC,EAAE;QACpC,CAAC,CAAC,KAAK,KAAK,MAAM;YAChB,CAAC,CAAC,2BAA2B,CAAC,cAAc,CAAC;YAC7C,CAAC,CAAC,6BAA6B,CAAC,cAAc,CAAC,CAAC;IAEtD,MAAM,UAAU,GACd,KAAK,KAAK,WAAW;QACnB,CAAC,CAAC,wBAAwB,EAAE;QAC5B,CAAC,CAAC,KAAK,KAAK,MAAM;YAChB,CAAC,CAAC,mBAAmB,EAAE;YACvB,CAAC,CAAC,qBAAqB,EAAE,CAAC;IAEhC,IAAI,gBAAgB,EAAE,CAAC;QACrB,yEAAyE;QACzE,uEAAuE;QACvE,wEAAwE;QACxE,gCAAgC;QAChC,OAAO,GAAG,YAAY;;;EAGxB,WAAW;;;;;;;;EAQX,UAAU,EAAE,CAAC;IACb,CAAC;IAED,OAAO,GAAG,YAAY;;;EAGtB,WAAW;;;;;;EAMX,UAAU;;;;;;qBAMS,CAAC;AACtB,CAAC;AAED,MAAM,UAAU,oBAAoB,CAClC,MAIE;IAEF,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;QAC3B,KAAK,MAAM,QAAQ,IAAI,KAAK,CAAC,SAAS,EAAE,CAAC;YACvC,MAAM,KAAK,GAAG,KAAK,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,QAAQ,CAAC,QAAQ,CAAC,CAAC;YACnE,MAAM,QAAQ,GAAG,KAAK,EAAE,QAAQ,IAAI,IAAI,CAAC;YACzC,MAAM,KAAK,GAAG,KAAK,EAAE,KAAK,IAAI,QAAQ,CAAC,QAAQ,CAAC;YAChD,MAAM,MAAM,GAAG,QAAQ,CAAC,MAAM,CAAC,WAAW,EAAE,CAAC;YAC7C,MAAM,SAAS,GACb,QAAQ,CAAC,SAAS,CAAC,MAAM,GAAG,EAAE;gBAC5B,CAAC,CAAC,QAAQ,CAAC,SAAS,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,KAAK;gBACzC,CAAC,CAAC,QAAQ,CAAC,SAAS,CAAC;YACzB,KAAK,CAAC,IAAI,CACR,MAAM,KAAK,CAAC,KAAK,IAAI,QAAQ,CAAC,QAAQ,KAAK,QAAQ,MAAM,KAAK,MAAM,MAAM,KAAK,SAAS,GAAG,CAC5F,CAAC;QACJ,CAAC;IACH,CAAC;IACD,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC"}

package/dist/src/providers/claude.d.ts

@@ -1,8 +1,11 @@
 import type { Provider, InvokeOptions, ProviderResponse } from "./types.js";
 export declare class ClaudeProvider implements Provider {
     name: string;
+    private capabilityCache;
     invoke(prompt: string, options: InvokeOptions): Promise<ProviderResponse>;
     isAvailable(): Promise<boolean>;
+    checkStructuredOutputSupport(): Promise<boolean>;
+    markNonCapable(): void;
     getModels(): string[];
     getEffortLevels(): string[];
 }