agentweaver 0.1.14 → 0.1.15

package/dist/pipeline/flow-specs/plan-revise.json

@@ -0,0 +1,261 @@
+{
+  "kind": "plan-revise-flow",
+  "version": 1,
+  "description": "Consumes a design-review verdict artifact and the latest planning artifacts, then produces revised design, implementation plan, and QA artifacts at the next iteration number without modifying originals.",
+  "phases": [
+    {
+      "id": "phase_1_load_contract",
+      "steps": [
+        {
+          "id": "validate_review_markdown",
+          "node": "file-check",
+          "params": {
+            "path": { "ref": "params.reviewFile" },
+            "panelTitle": { "const": "Design Review Verdict Markdown" },
+            "foundMessage": { "const": "Design review verdict markdown found." },
+            "tone": { "const": "cyan" }
+          },
+          "expect": [
+            {
+              "kind": "require-file",
+              "path": { "ref": "params.reviewFile" },
+              "message": "Plan-revise contract is missing the required design-review verdict markdown."
+            }
+          ]
+        },
+        {
+          "id": "validate_review_json",
+          "node": "file-check",
+          "params": {
+            "path": { "ref": "params.reviewJsonFile" },
+            "panelTitle": { "const": "Design Review Verdict JSON" },
+            "foundMessage": { "const": "Design review verdict JSON found." },
+            "tone": { "const": "cyan" }
+          },
+          "expect": [
+            {
+              "kind": "require-file",
+              "path": { "ref": "params.reviewJsonFile" },
+              "message": "Plan-revise contract is missing the required design-review verdict JSON."
+            }
+          ]
+        },
+        {
+          "id": "validate_source_design_markdown",
+          "node": "file-check",
+          "params": {
+            "path": { "ref": "params.designFile" },
+            "panelTitle": { "const": "Source Design Markdown" },
+            "foundMessage": { "const": "Source design markdown artifact found." },
+            "tone": { "const": "cyan" }
+          },
+          "expect": [
+            {
+              "kind": "require-file",
+              "path": { "ref": "params.designFile" },
+              "message": "Plan-revise contract is missing the required source design markdown."
+            }
+          ]
+        },
+        {
+          "id": "validate_source_design_json",
+          "node": "file-check",
+          "params": {
+            "path": { "ref": "params.designJsonFile" },
+            "panelTitle": { "const": "Source Design JSON" },
+            "foundMessage": { "const": "Source design structured artifact found." },
+            "tone": { "const": "cyan" }
+          },
+          "expect": [
+            {
+              "kind": "require-file",
+              "path": { "ref": "params.designJsonFile" },
+              "message": "Plan-revise contract is missing the required source design JSON."
+            }
+          ]
+        },
+        {
+          "id": "validate_source_plan_markdown",
+          "node": "file-check",
+          "params": {
+            "path": { "ref": "params.planFile" },
+            "panelTitle": { "const": "Source Plan Markdown" },
+            "foundMessage": { "const": "Source implementation plan markdown found." },
+            "tone": { "const": "cyan" }
+          },
+          "expect": [
+            {
+              "kind": "require-file",
+              "path": { "ref": "params.planFile" },
+              "message": "Plan-revise contract is missing the required source plan markdown."
+            }
+          ]
+        },
+        {
+          "id": "validate_source_plan_json",
+          "node": "file-check",
+          "params": {
+            "path": { "ref": "params.planJsonFile" },
+            "panelTitle": { "const": "Source Plan JSON" },
+            "foundMessage": { "const": "Source implementation plan structured artifact found." },
+            "tone": { "const": "cyan" }
+          },
+          "expect": [
+            {
+              "kind": "require-file",
+              "path": { "ref": "params.planJsonFile" },
+              "message": "Plan-revise contract is missing the required source plan JSON."
+            }
+          ]
+        },
+        {
+          "id": "validate_source_qa_markdown",
+          "when": { "ref": "params.hasQaArtifacts" },
+          "node": "file-check",
+          "params": {
+            "path": { "ref": "params.qaFilePath" },
+            "panelTitle": { "const": "Source QA Markdown" },
+            "foundMessage": { "const": "Source QA markdown artifact found." },
+            "tone": { "const": "cyan" }
+          },
+          "expect": [
+            {
+              "kind": "require-file",
+              "path": { "ref": "params.qaFilePath" },
+              "message": "Plan-revise contract expected a source QA markdown artifact, but the file is missing."
+            }
+          ]
+        },
+        {
+          "id": "validate_source_qa_json",
+          "when": { "ref": "params.hasQaArtifacts" },
+          "node": "file-check",
+          "params": {
+            "path": { "ref": "params.qaJsonFilePath" },
+            "panelTitle": { "const": "Source QA JSON" },
+            "foundMessage": { "const": "Source QA structured artifact found." },
+            "tone": { "const": "cyan" }
+          },
+          "expect": [
+            {
+              "kind": "require-file",
+              "path": { "ref": "params.qaJsonFilePath" },
+              "message": "Plan-revise contract expected a source QA JSON artifact, but the file is missing."
+            }
+          ]
+        }
+      ]
+    },
+    {
+      "id": "phase_2_revise",
+      "steps": [
+        {
+          "id": "run_plan_revise",
+          "node": "llm-prompt",
+          "prompt": {
+            "templateRef": "plan-revise",
+            "vars": {
+              "review_file": { "ref": "params.reviewFile" },
+              "review_json_file": { "ref": "params.reviewJsonFile" },
+              "design_file": { "ref": "params.designFile" },
+              "design_json_file": { "ref": "params.designJsonFile" },
+              "plan_file": { "ref": "params.planFile" },
+              "plan_json_file": { "ref": "params.planJsonFile" },
+              "qa_file": { "ref": "params.qaFile" },
+              "qa_json_file": { "ref": "params.qaJsonFile" },
+              "jira_task_file": { "ref": "params.jiraTaskFile" },
+              "jira_attachments_manifest_file": { "ref": "params.jiraAttachmentsManifestFile" },
+              "jira_attachments_context_file": { "ref": "params.jiraAttachmentsContextFile" },
+              "planning_answers_json_file": { "ref": "params.planningAnswersJsonFile" },
+              "revised_design_file": { "ref": "params.revisedDesignFile" },
+              "revised_design_json_file": { "ref": "params.revisedDesignJsonFile" },
+              "revised_plan_file": { "ref": "params.revisedPlanFile" },
+              "revised_plan_json_file": { "ref": "params.revisedPlanJsonFile" },
+              "revised_qa_file": { "ref": "params.revisedQaFile" },
+              "revised_qa_json_file": { "ref": "params.revisedQaJsonFile" }
+            },
+            "extraPrompt": { "ref": "params.extraPrompt" },
+            "format": "task-prompt"
+          },
+          "params": {
+            "labelText": {
+              "template": "Revising planning artifacts (iteration {outputIteration})",
+              "vars": {
+                "outputIteration": { "ref": "params.outputIteration" }
+              }
+            },
+            "model": { "ref": "params.llmModel" },
+            "executor": { "ref": "params.llmExecutor" },
+            "requiredArtifacts": {
+              "list": [
+                { "ref": "params.revisedDesignFile" },
+                { "ref": "params.revisedPlanFile" },
+                { "ref": "params.revisedQaFile" }
+              ]
+            }
+          },
+          "expect": [
+            {
+              "kind": "require-artifacts",
+              "when": { "not": { "ref": "context.dryRun" } },
+              "paths": {
+                "list": [
+                  { "ref": "params.revisedDesignFile" },
+                  { "ref": "params.revisedPlanFile" },
+                  { "ref": "params.revisedQaFile" }
+                ]
+              },
+              "message": "Plan-revise did not produce the required revised markdown artifacts."
+            },
+            {
+              "kind": "require-structured-artifacts",
+              "when": { "not": { "ref": "context.dryRun" } },
+              "items": [
+                {
+                  "path": { "ref": "params.revisedDesignJsonFile" },
+                  "schemaId": "implementation-design/v1"
+                },
+                {
+                  "path": { "ref": "params.revisedPlanJsonFile" },
+                  "schemaId": "implementation-plan/v1"
+                },
+                {
+                  "path": { "ref": "params.revisedQaJsonFile" },
+                  "schemaId": "qa-plan/v1"
+                }
+              ],
+              "message": "Plan-revise produced invalid structured artifacts."
+            }
+          ]
+        }
+      ]
+    },
+    {
+      "id": "phase_3_finalize",
+      "steps": [
+        {
+          "id": "check_ready_to_merge",
+          "node": "file-check",
+          "params": {
+            "path": {
+              "artifact": {
+                "kind": "ready-to-merge-file",
+                "taskKey": { "ref": "params.taskKey" }
+              }
+            },
+            "panelTitle": { "const": "Ready To Merge" },
+            "foundMessage": { "const": "Plan-revise completed and all blocking findings addressed. File ready-to-merge.md has been created." },
+            "tone": { "const": "green" }
+          }
+        },
+        {
+          "id": "notify_revise_complete",
+          "node": "telegram-notify",
+          "params": {
+            "message": { "const": "Plan-revise phase complete" }
+          }
+        }
+      ]
+    }
+  ]
+}

package/dist/pipeline/flow-specs/task-describe.json

@@ -235,10 +235,37 @@
                 }
               },
               "message": "Task describe mode did not produce the Jira description artifact."
+            }
+          ]
+        },
+        {
+          "id": "ensure_task_describe_json_from_jira_markdown",
+          "node": "ensure-summary-json",
+          "when": {
+            "all": [
+              { "ref": "steps.task_describe.collect_task_source.value.values.jira_ref" },
+              { "not": { "ref": "context.dryRun" } }
+            ]
+          },
+          "params": {
+            "markdownFile": {
+              "artifact": {
+                "kind": "jira-description-file",
+                "taskKey": { "ref": "params.taskKey" },
+                "iteration": { "ref": "params.iteration" }
+              }
             },
+            "outputFile": {
+              "artifact": {
+                "kind": "jira-description-json-file",
+                "taskKey": { "ref": "params.taskKey" },
+                "iteration": { "ref": "params.iteration" }
+              }
+            }
+          },
+          "expect": [
             {
               "kind": "require-structured-artifacts",
-              "when": { "not": { "ref": "context.dryRun" } },
               "items": [
                 {
                   "path": {
@@ -330,10 +357,41 @@
                 }
               },
               "message": "Task describe mode did not produce the Jira description artifact."
+            }
+          ]
+        },
+        {
+          "id": "ensure_task_describe_json_from_input_markdown",
+          "node": "ensure-summary-json",
+          "when": {
+            "all": [
+              {
+                "not": {
+                  "ref": "steps.task_describe.collect_task_source.value.values.jira_ref"
+                }
+              },
+              { "not": { "ref": "context.dryRun" } }
+            ]
+          },
+          "params": {
+            "markdownFile": {
+              "artifact": {
+                "kind": "jira-description-file",
+                "taskKey": { "ref": "params.taskKey" },
+                "iteration": { "ref": "params.iteration" }
+              }
             },
+            "outputFile": {
+              "artifact": {
+                "kind": "jira-description-json-file",
+                "taskKey": { "ref": "params.taskKey" },
+                "iteration": { "ref": "params.iteration" }
+              }
+            }
+          },
+          "expect": [
             {
               "kind": "require-structured-artifacts",
-              "when": { "not": { "ref": "context.dryRun" } },
               "items": [
                 {
                   "path": {

package/dist/pipeline/node-registry.js

@@ -3,6 +3,7 @@ import { buildReviewFixPromptNode } from "./nodes/build-review-fix-prompt-node.j
 import { codexPromptNode } from "./nodes/codex-prompt-node.js";
 import { commandCheckNode } from "./nodes/command-check-node.js";
 import { commitMessageFormNode } from "./nodes/commit-message-form-node.js";
+import { ensureSummaryJsonNode } from "./nodes/ensure-summary-json-node.js";
 import { fetchGitLabDiffNode } from "./nodes/fetch-gitlab-diff-node.js";
 import { fetchGitLabReviewNode } from "./nodes/fetch-gitlab-review-node.js";
 import { fileCheckNode } from "./nodes/file-check-node.js";
@@ -32,6 +33,7 @@ const builtInNodes = {
     "codex-prompt": codexPromptNode,
     "command-check": commandCheckNode,
     "commit-message-form": commitMessageFormNode,
+    "ensure-summary-json": ensureSummaryJsonNode,
     "fetch-gitlab-diff": fetchGitLabDiffNode,
     "fetch-gitlab-review": fetchGitLabReviewNode,
     "file-check": fileCheckNode,
@@ -90,6 +92,12 @@ const builtInNodeMetadata = {
         prompt: "forbidden",
         requiredParams: ["commitMessageFile", "formId", "title", "outputFile"],
     },
+    "ensure-summary-json": {
+        kind: "ensure-summary-json",
+        version: 1,
+        prompt: "forbidden",
+        requiredParams: ["markdownFile", "outputFile"],
+    },
     "fetch-gitlab-diff": {
         kind: "fetch-gitlab-diff",
         version: 1,

package/dist/pipeline/nodes/ensure-summary-json-node.js

@@ -0,0 +1,59 @@
+import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
+import path from "node:path";
+import { TaskRunnerError } from "../../errors.js";
+function toSummaryText(markdown) {
+    return markdown
+        .replace(/\r\n/g, "\n")
+        .replace(/^#{1,6}\s+/gm, "")
+        .replace(/^\s*[-*+]\s+/gm, "- ")
+        .replace(/`([^`]*)`/g, "$1")
+        .replace(/\*\*(.*?)\*\*/g, "$1")
+        .replace(/__(.*?)__/g, "$1")
+        .replace(/\n{3,}/g, "\n\n")
+        .trim();
+}
+function hasValidSummaryArtifact(outputFile) {
+    if (!existsSync(outputFile)) {
+        return false;
+    }
+    try {
+        const parsed = JSON.parse(readFileSync(outputFile, "utf8"));
+        return typeof parsed.summary === "string" && parsed.summary.trim().length > 0;
+    }
+    catch {
+        return false;
+    }
+}
+export const ensureSummaryJsonNode = {
+    kind: "ensure-summary-json",
+    version: 1,
+    async run(_context, params) {
+        if (hasValidSummaryArtifact(params.outputFile)) {
+            return {
+                value: {
+                    outputFile: params.outputFile,
+                    created: false,
+                    repaired: false,
+                },
+            };
+        }
+        if (!existsSync(params.markdownFile)) {
+            throw new TaskRunnerError(`Cannot create summary JSON ${params.outputFile}: markdown source ${params.markdownFile} was not found.`);
+        }
+        const summary = toSummaryText(readFileSync(params.markdownFile, "utf8"));
+        if (!summary) {
+            throw new TaskRunnerError(`Cannot create summary JSON ${params.outputFile}: markdown source ${params.markdownFile} is empty.`);
+        }
+        mkdirSync(path.dirname(params.outputFile), { recursive: true });
+        const repaired = existsSync(params.outputFile);
+        writeFileSync(params.outputFile, `${JSON.stringify({ summary }, null, 2)}\n`, "utf8");
+        return {
+            value: {
+                outputFile: params.outputFile,
+                created: !repaired,
+                repaired,
+            },
+            outputs: [{ kind: "artifact", path: params.outputFile, required: true }],
+        };
+    },
+};

package/dist/pipeline/prompt-registry.js

@@ -1,8 +1,9 @@
-import { BUG_ANALYZE_PROMPT_TEMPLATE, COMMIT_MESSAGE_PROMPT_TEMPLATE, GITLAB_DIFF_REVIEW_PROMPT_TEMPLATE, GITLAB_REVIEW_PROMPT_TEMPLATE, BUG_FIX_PROMPT_TEMPLATE, IMPLEMENT_PROMPT_TEMPLATE, JIRA_DESCRIPTION_PROMPT_TEMPLATE, MR_DESCRIPTION_PROMPT_TEMPLATE, PLAN_QUESTIONS_PROMPT_TEMPLATE, PLAN_PROMPT_TEMPLATE, REVIEW_FIX_PROMPT_TEMPLATE, REVIEW_PROJECT_PROMPT_TEMPLATE, REVIEW_PROMPT_TEMPLATE, REVIEW_SUMMARY_PROMPT_TEMPLATE, RUN_GO_LINTER_LOOP_FIX_PROMPT_TEMPLATE, RUN_GO_TESTS_LOOP_FIX_PROMPT_TEMPLATE, TASK_SUMMARY_PROMPT_TEMPLATE, } from "../prompts.js";
+import { BUG_ANALYZE_PROMPT_TEMPLATE, COMMIT_MESSAGE_PROMPT_TEMPLATE, DESIGN_REVIEW_PROMPT_TEMPLATE, GITLAB_DIFF_REVIEW_PROMPT_TEMPLATE, GITLAB_REVIEW_PROMPT_TEMPLATE, BUG_FIX_PROMPT_TEMPLATE, IMPLEMENT_PROMPT_TEMPLATE, JIRA_DESCRIPTION_PROMPT_TEMPLATE, MR_DESCRIPTION_PROMPT_TEMPLATE, PLAN_PROMPT_TEMPLATE, PLAN_QUESTIONS_PROMPT_TEMPLATE, PLAN_REVISE_PROMPT_TEMPLATE, REVIEW_FIX_PROMPT_TEMPLATE, REVIEW_PROJECT_PROMPT_TEMPLATE, REVIEW_PROMPT_TEMPLATE, REVIEW_SUMMARY_PROMPT_TEMPLATE, RUN_GO_LINTER_LOOP_FIX_PROMPT_TEMPLATE, RUN_GO_TESTS_LOOP_FIX_PROMPT_TEMPLATE, TASK_SUMMARY_PROMPT_TEMPLATE, } from "../prompts.js";
 const promptTemplates = {
     "bug-analyze": BUG_ANALYZE_PROMPT_TEMPLATE,
     "bug-fix": BUG_FIX_PROMPT_TEMPLATE,
     "commit-message": COMMIT_MESSAGE_PROMPT_TEMPLATE,
+    "design-review": DESIGN_REVIEW_PROMPT_TEMPLATE,
     "gitlab-diff-review": GITLAB_DIFF_REVIEW_PROMPT_TEMPLATE,
     "gitlab-review": GITLAB_REVIEW_PROMPT_TEMPLATE,
     implement: IMPLEMENT_PROMPT_TEMPLATE,
@@ -10,6 +11,7 @@ const promptTemplates = {
     "mr-description": MR_DESCRIPTION_PROMPT_TEMPLATE,
     "plan-questions": PLAN_QUESTIONS_PROMPT_TEMPLATE,
     plan: PLAN_PROMPT_TEMPLATE,
+    "plan-revise": PLAN_REVISE_PROMPT_TEMPLATE,
     review: REVIEW_PROMPT_TEMPLATE,
     "review-project": REVIEW_PROJECT_PROMPT_TEMPLATE,
     "review-fix": REVIEW_FIX_PROMPT_TEMPLATE,

package/dist/pipeline/spec-types.js

@@ -7,6 +7,8 @@ export const ARTIFACT_REF_KINDS = [
     "bug-fix-plan-json-file",
     "design-file",
     "design-json-file",
+    "design-review-file",
+    "design-review-json-file",
     "gitlab-diff-file",
     "gitlab-diff-json-file",
     "gitlab-diff-review-input-json-file",

package/dist/pipeline/value-resolver.js

@@ -1,5 +1,5 @@
 import { existsSync } from "node:fs";
-import { artifactFile, bugAnalyzeArtifacts, bugAnalyzeFile, bugAnalyzeJsonFile, bugFixDesignFile, bugFixDesignJsonFile, bugFixPlanFile, bugFixPlanJsonFile, designFile, designJsonFile, gitlabDiffFile, gitlabDiffJsonFile, gitlabDiffReviewInputJsonFile, gitlabReviewFile, gitlabReviewInputJsonFile, gitlabReviewJsonFile, jiraAttachmentsContextFile, jiraAttachmentsManifestFile, jiraDescriptionFile, jiraDescriptionJsonFile, jiraTaskFile, mrDescriptionFile, mrDescriptionJsonFile, planningAnswersJsonFile, planningQuestionsJsonFile, planArtifacts, planFile, planJsonFile, qaFile, qaJsonFile, readyToMergeFile, reviewAssessmentFile, reviewAssessmentJsonFile, reviewFile, reviewFixFile, reviewFixJsonFile, reviewJsonFile, runGoLinterResultJsonFile, runGoTestsResultJsonFile, taskSummaryFile, taskDescribeInputJsonFile, taskSummaryJsonFile, gitStatusJsonFile, gitCommitMessageJsonFile, gitCommitInputJsonFile, selectFilesOutputJsonFile, commitMessageOutputJsonFile, gitDiffFile as gitDiffFileHelper, } from "../artifacts.js";
+import { artifactFile, bugAnalyzeArtifacts, bugAnalyzeFile, bugAnalyzeJsonFile, bugFixDesignFile, bugFixDesignJsonFile, bugFixPlanFile, bugFixPlanJsonFile, designFile, designJsonFile, designReviewFile, designReviewJsonFile, gitlabDiffFile, gitlabDiffJsonFile, gitlabDiffReviewInputJsonFile, gitlabReviewFile, gitlabReviewInputJsonFile, gitlabReviewJsonFile, jiraAttachmentsContextFile, jiraAttachmentsManifestFile, jiraDescriptionFile, jiraDescriptionJsonFile, jiraTaskFile, mrDescriptionFile, mrDescriptionJsonFile, planningAnswersJsonFile, planningQuestionsJsonFile, planArtifacts, planFile, planJsonFile, qaFile, qaJsonFile, readyToMergeFile, reviewAssessmentFile, reviewAssessmentJsonFile, reviewFile, reviewFixFile, reviewFixJsonFile, reviewJsonFile, runGoLinterResultJsonFile, runGoTestsResultJsonFile, taskSummaryFile, taskDescribeInputJsonFile, taskSummaryJsonFile, gitStatusJsonFile, gitCommitMessageJsonFile, gitCommitInputJsonFile, selectFilesOutputJsonFile, commitMessageOutputJsonFile, gitDiffFile as gitDiffFileHelper, } from "../artifacts.js";
 import { TaskRunnerError } from "../errors.js";
 import { formatTemplate } from "../prompts.js";
 function readStepRef(segments, context, originalPath) {
@@ -88,6 +88,16 @@ function resolveArtifact(spec, context) {
             return designFile(taskKey, iteration);
         case "design-json-file":
             return designJsonFile(taskKey, iteration);
+        case "design-review-file":
+            if (iteration === undefined) {
+                throw new TaskRunnerError("design-review-file requires iteration");
+            }
+            return designReviewFile(taskKey, iteration);
+        case "design-review-json-file":
+            if (iteration === undefined) {
+                throw new TaskRunnerError("design-review-json-file requires iteration");
+            }
+            return designReviewJsonFile(taskKey, iteration);
         case "gitlab-diff-file":
             return gitlabDiffFile(taskKey, iteration);
         case "gitlab-diff-json-file":

package/dist/prompts.js

@@ -52,13 +52,32 @@ export const MR_DESCRIPTION_PROMPT_TEMPLATE = "Review the task in {jira_task_fil
     "Prepare a very brief intent description for the merge request without implementation details, file lists, or technical details. " +
     `First write the source-of-truth JSON to {mr_description_json_file}. ${strictSchemaInstruction("{mr_description_json_file}", "mr-description/v1")}Then write the derivative markdown version to {mr_description_file}. `;
 export const IMPLEMENT_PROMPT_TEMPLATE = "Use only structured artifacts as source of truth. " +
-    "Analyze the system design {design_json_file}, implementation plan {plan_json_file}, and proceed with implementation according to the plan. " +
-    "Markdown artifacts are intended only for human reading and should not define the implementation. ";
+    "Analyze the system design {design_json_file}, implementation plan {plan_json_file}, and QA plan {qa_json_file}, then proceed with implementation according to those artifacts. " +
+    "Treat the QA plan as the source of truth for the minimum required test scenarios, edge cases, regression checks, and validation behavior that the implementation must satisfy. " +
+    "When the repository contains automated tests, add or update tests for the key scenarios from the QA plan whenever it is practical in the current codebase. " +
+    "If some QA scenarios cannot be automated in the current change, still implement the code so those scenarios are satisfied and keep them explicit in your reasoning while editing. " +
+    "Markdown artifacts such as {design_file}, {plan_file}, and {qa_file} are intended only for human reading and should not define the implementation. ";
 export const REVIEW_PROMPT_TEMPLATE = "Conduct a code review of the current changes. " +
     "Use only structured artifacts as source of truth: the task in {jira_task_file}, design in {design_json_file}, and plan in {plan_json_file}. " +
     `First write the structured result to {review_json_file}. ${strictSchemaInstruction("{review_json_file}", "review-findings/v1")}` +
     "Then write the derivative markdown version to {review_file}. " +
     "If ready_to_merge=true and there are no blockers preventing merge - create the ready-to-merge.md file.";
+export const DESIGN_REVIEW_PROMPT_TEMPLATE = "Conduct a structured planning critique as a specification critic, not as an implementer. " +
+    "Use structured JSON artifacts as the source of truth for semantics. " +
+    "Required planning inputs: design markdown {design_file}, design JSON {design_json_file}, implementation plan markdown {plan_file}, implementation plan JSON {plan_json_file}. " +
+    "Review the markdown files as derivative human-readable renderings of the same planning run, but do not let markdown override the structured JSON. " +
+    "Optional supplemental context is provided through these variables and may contain the literal value 'not provided' when absent: QA markdown {qa_file}, QA JSON {qa_json_file}, Jira task JSON {jira_task_file}, Jira attachments manifest {jira_attachments_manifest_file}, Jira attachments context {jira_attachments_context_file}, planning answers JSON {planning_answers_json_file}. " +
+    "When an optional variable is 'not provided', treat that source as unavailable and do not invent details from it. " +
+    "Evaluate completeness, consistency, implementation readiness, risk coverage, QA coverage, and scope discipline across the available planning artifacts and optional context. " +
+    "Identify blocking findings, major non-blocking findings, warnings, missing information, consistency check results, QA coverage gaps, and concise recommended actions. " +
+    "Use exactly one status value: approved, approved_with_warnings, or needs_revision. " +
+    "Set status to needs_revision when any blocking finding exists or when required information is missing in a way that blocks safe implementation start. " +
+    "Set status to approved_with_warnings when there are no blocking findings, but there are major findings, warnings, non-blocking missing information items, QA coverage gaps, or non-blocking consistency issues. " +
+    "Set status to approved only when there are no unresolved blocking findings, major findings, warnings, missing information items, or QA coverage gaps. " +
+    `First write the structured design review to {review_json_file}. ${strictSchemaInstruction("{review_json_file}", "design-review/v1")}` +
+    "Then write the derivative markdown version to {review_file}. " +
+    "Create ready-to-merge.md only when status is approved or approved_with_warnings. " +
+    "Do not create ready-to-merge.md when status is needs_revision.";
 export const REVIEW_PROJECT_PROMPT_TEMPLATE = "Conduct a code review of current changes in the project without Jira context. " +
     "Evaluate the quality of changes based on current code, tests, regression risks, and overall engineering quality. " +
     `First write the structured result to {review_json_file}. ${strictSchemaInstruction("{review_json_file}", "review-findings/v1")}` +
@@ -116,6 +135,34 @@ export const COMMIT_MESSAGE_PROMPT_TEMPLATE = "Generate a commit message for the
     "3) Include task key from Jira task. " +
     "4) Commit message language: English. " +
     "5) Write JSON to {commit_message_json_file}: {\"subject\": \"...\"}.";
+export const PLAN_REVISE_PROMPT_TEMPLATE = "Revise the planning artifacts based on the design-review verdict. " +
+    "Use structured JSON artifacts as the source of truth for semantics. " +
+    "First revise the structured JSON artifacts; only after the JSON is complete and schema-valid should you write the derivative markdown files. " +
+    "Markdown must not influence JSON structure or types. " +
+    "The design-review verdict JSON {review_json_file} is the primary source of revision instructions — treat its blocking findings, major findings, and recommended actions as mandatory revision targets. " +
+    "The design-review markdown {review_file} is a derivative rendering only and must not override the structured verdict. " +
+    "Required source planning inputs: design JSON {design_json_file}, design markdown {design_file}, plan JSON {plan_json_file}, plan markdown {plan_file}. " +
+    "Optional source QA inputs (may be 'not provided'): QA JSON {qa_json_file}, QA markdown {qa_file}. " +
+    "When QA inputs are 'not provided', synthesize a new QA plan from the revised design and plan. " +
+    "Optional supplemental context (may be 'not provided'): Jira task JSON {jira_task_file}, Jira attachments manifest {jira_attachments_manifest_file}, Jira attachments context {jira_attachments_context_file}, planning answers JSON {planning_answers_json_file}. " +
+    "When an optional variable is 'not provided', treat that source as unavailable and do not invent details from it. " +
+    "For every blocking finding and major finding in the verdict, address it directly in the revised artifacts. " +
+    "Preserve all content from the original artifacts that is not directly addressed by findings in the verdict — do not drop details, sections, or decisions that remain valid. " +
+    "Preserve semantics, not source formatting conventions from the verdict: do not copy verdict-style nested objects into fields whose schemas require plain strings. " +
+    "For implementation-design/v1 specifically, goals, non_goals, components, current_state, target_state, business_rules, migration_strategy, database_changes, api_changes, risks, acceptance_criteria, and open_questions must remain arrays of non-empty strings. " +
+    "Only affected_code and decisions may contain nested objects in implementation-design/v1. " +
+    "If you need to preserve extra detail such as mitigation, resolution, or answer text for a string-array field, fold that detail into a single English sentence string instead of creating an object. " +
+    "Produce the following revised outputs: " +
+    `revised design JSON to {revised_design_json_file}. ${strictSchemaInstruction("{revised_design_json_file}", "implementation-design/v1")}` +
+    "Revised design markdown to {revised_design_file}. " +
+    `revised plan JSON to {revised_plan_json_file}. ${strictSchemaInstruction("{revised_plan_json_file}", "implementation-plan/v1")}` +
+    "Revised plan markdown to {revised_plan_file}. " +
+    `revised QA JSON to {revised_qa_json_file}. ${strictSchemaInstruction("{revised_qa_json_file}", "qa-plan/v1")}` +
+    "Revised QA markdown to {revised_qa_file}. " +
+    "Create ready-to-merge.md only when all blocking findings from the verdict have been addressed in the revised artifacts. " +
+    "Do not create ready-to-merge.md if any blocking finding remains unresolved. " +
+    "JSON files must be valid and contain only JSON without markdown wrapping. " +
+    "Markdown files must be comprehensive derivative representations of the corresponding JSON artifacts.";
 export const AUTO_REVIEW_FIX_EXTRA_PROMPT = "Fix only blockers, criticals, and important issues";
 export function formatTemplate(template, values) {
     let result = template;