lee-spec-kit 0.6.40 → 0.6.42

package/dist/index.js

@@ -478,7 +478,7 @@ var koContext = {
   "context.commandDetail.codeReviewPushFix": "({scope}) \uB9AC\uBDF0 \uC218\uC815 \uCEE4\uBC0B\uC744 push\uD558\uC138\uC694",
   "context.commandDetail.prePrReviewRun": "({scope}) \uBCF4\uC870 \uC5D0\uC774\uC804\uD2B8(sub-agent)\uB85C PR \uC804 \uB9AC\uBDF0\uB97C \uC2E4\uD589\uD574 `review-trace.json`\uC744 \uC900\uBE44\uD558\uC138\uC694",
   "context.commandDetail.prePrReviewRecord": "({scope}) Pre-PR \uB9AC\uBDF0 evidence\uB97C decisions.md\uC640 tasks.md\uC5D0 \uAE30\uB85D\uD558\uC138\uC694",
-  "context.commandDetail.codeReviewRun": "({scope}) \uBCF4\uC870 \uC5D0\uC774\uC804\uD2B8(sub-agent)\uB85C PR \uB9AC\uBDF0\uB97C \uC2E4\uD589\uD574 evidence/\uC218\uC815 \uC694\uC57D\uC744 \uC900\uBE44\uD558\uC138\uC694",
+  "context.commandDetail.codeReviewRun": "({scope}) \uB9AC\uBDF0 \uCF54\uBA58\uD2B8\uB97C \uD655\uC778\uD558\uACE0, \uBCF4\uC870 \uC5D0\uC774\uC804\uD2B8(sub-agent)\uB85C \uC218\uC815 \uC791\uC5C5/evidence \uC815\uB9AC\uB97C \uC9C4\uD589\uD558\uC138\uC694",
   "context.actionSummary.runDocsCommand": "\uBB38\uC11C \uC791\uC5C5 \uBA85\uB839\uC744 \uC2E4\uD589\uD558\uC138\uC694",
   "context.actionSummary.runProjectCommand": "\uD504\uB85C\uC81D\uD2B8 \uC791\uC5C5 \uBA85\uB839\uC744 \uC2E4\uD589\uD558\uC138\uC694",
   "context.actionDetail.featureFolder": "Feature \uD3F4\uB354\uC640 \uAE30\uBCF8 \uBB38\uC11C \uACE8\uACA9\uC744 \uC900\uBE44\uD558\uC138\uC694",
@@ -496,12 +496,12 @@ var koContext = {
   "context.actionDetail.issueCreatePrepareFromDoc": "issue.md \uCD08\uC548\uC744 \uBCF4\uC644\uD558\uACE0 \uC0C1\uD0DC\uB97C Ready\uB85C \uC124\uC815\uD558\uC138\uC694",
   "context.actionDetail.issueCreateFromDoc": "Ready \uC0C1\uD0DC issue.md\uB85C \uC774\uC288\uB97C \uC0DD\uC131\uD558\uACE0 \uBC88\uD638\uB97C \uB3D9\uAE30\uD654\uD558\uC138\uC694",
   "context.actionDetail.taskExecute": "\uD604\uC7AC \uD0DC\uC2A4\uD06C\uB97C \uC9C4\uD589\uD558\uC138\uC694",
-  "context.actionDetail.taskExecuteRun": "\uBCF4\uC870 \uC5D0\uC774\uC804\uD2B8(sub-agent) \uC791\uC5C5 handoff\uB97C \uC900\uBE44\uD558\uACE0 \uD0DC\uC2A4\uD06C\uB97C \uC2DC\uC791\uD558\uC138\uC694. (TODO\uBA74 DOING\uC73C\uB85C \uBCC0\uACBD)",
-  "context.actionDetail.taskExecuteContinue": "\uBCF4\uC870 \uC5D0\uC774\uC804\uD2B8(sub-agent) \uC791\uC5C5 handoff\uB97C \uC900\uBE44\uD574 \uC9C4\uD589 \uC911\uC778 \uD0DC\uC2A4\uD06C\uB97C \uC774\uC5B4\uAC00\uC138\uC694",
+  "context.actionDetail.taskExecuteRun": "\uBCF4\uC870 \uC5D0\uC774\uC804\uD2B8(sub-agent) \uC791\uC5C5 handoff\uB97C \uC900\uBE44\uD558\uACE0 \uD0DC\uC2A4\uD06C\uB97C \uC2DC\uC791\uD558\uC138\uC694: {task}. (TODO\uBA74 DOING\uC73C\uB85C \uBCC0\uACBD)",
+  "context.actionDetail.taskExecuteContinue": "\uBCF4\uC870 \uC5D0\uC774\uC804\uD2B8(sub-agent) \uC791\uC5C5 handoff\uB97C \uC900\uBE44\uD574 \uC9C4\uD589 \uC911\uC778 \uD0DC\uC2A4\uD06C\uB97C \uC774\uC5B4\uAC00\uC138\uC694: {task}",
   "context.actionDetail.reviewFixCommit": "\uD574\uACB0\uD55C \uB9AC\uBDF0 \uD56D\uBAA9 \uC694\uC57D\uC73C\uB85C \uB9AC\uBDF0 \uC218\uC815 \uCEE4\uBC0B\uC744 \uB9CC\uB4DC\uC138\uC694",
   "context.actionDetail.prePrReviewRun": "\uBCF4\uC870 \uC5D0\uC774\uC804\uD2B8(sub-agent)\uB85C PR \uC804 \uB9AC\uBDF0\uB97C \uC2E4\uD589\uD574 `review-trace.json`\uC744 \uC900\uBE44\uD558\uC138\uC694",
   "context.actionDetail.prePrReviewRecord": "PR \uC804 \uB9AC\uBDF0 evidence\uB97C decisions.md\uC640 tasks.md\uC5D0 \uAE30\uB85D\uD558\uC138\uC694",
-  "context.actionDetail.codeReviewRun": "\uBCF4\uC870 \uC5D0\uC774\uC804\uD2B8(sub-agent)\uB85C PR \uB9AC\uBDF0\uB97C \uC2E4\uD589\uD574 evidence/\uC218\uC815 \uC694\uC57D\uC744 \uC900\uBE44\uD558\uC138\uC694",
+  "context.actionDetail.codeReviewRun": "\uB9AC\uBDF0 \uCF54\uBA58\uD2B8\uB97C \uD655\uC778\uD558\uACE0, \uBCF4\uC870 \uC5D0\uC774\uC804\uD2B8(sub-agent)\uB85C \uC218\uC815 \uC791\uC5C5/evidence \uC815\uB9AC\uB97C \uC9C4\uD589\uD558\uC138\uC694",
   "context.actionDetail.prCreate": "PR\uC744 \uC0DD\uC131\uD558\uACE0 tasks \uAE30.md\uC758 PR \uC815\uBCF4\uB97C \uB9DE\uCD94\uC138\uC694",
   "context.actionDetail.prCreateRequiredSequence": "PR 2\uB2E8\uACC4(\uCD08\uC548/\uC2B9\uC778 \uD6C4 \uC0DD\uC131/\uB3D9\uAE30\uD654)\uB97C \uC21C\uC11C\uB300\uB85C \uC644\uB8CC\uD558\uC138\uC694",
   "context.actionDetail.prCreatePrepareFromDoc": "pr.md \uCD08\uC548\uC744 \uBCF4\uC644\uD558\uACE0 \uC0C1\uD0DC\uB97C Ready\uB85C \uC124\uC815\uD558\uC138\uC694",
@@ -1030,7 +1030,7 @@ var enContext = {
   "context.commandDetail.codeReviewPushFix": "({scope}) push review-fix commits",
   "context.commandDetail.prePrReviewRun": "({scope}) run the pre-PR review via a helper agent/sub-agent and prepare `review-trace.json`",
   "context.commandDetail.prePrReviewRecord": "({scope}) record pre-PR review evidence into decisions.md + tasks.md",
-  "context.commandDetail.codeReviewRun": "({scope}) run the PR review via a helper agent/sub-agent and prepare evidence/fix summary",
+  "context.commandDetail.codeReviewRun": "({scope}) check PR review comments, then use a helper agent/sub-agent for the follow-up fixes and evidence summary",
   "context.actionSummary.runDocsCommand": "Run docs command",
   "context.actionSummary.runProjectCommand": "Run project command",
   "context.actionDetail.featureFolder": "Prepare feature folder and baseline docs",
@@ -1048,12 +1048,12 @@ var enContext = {
   "context.actionDetail.issueCreatePrepareFromDoc": "Refine issue.md draft and set Status to Ready",
   "context.actionDetail.issueCreateFromDoc": "Create GitHub Issue from ready issue.md and sync Issue",
   "context.actionDetail.taskExecute": "Proceed with the current task",
-  "context.actionDetail.taskExecuteRun": "Prepare helper agent/sub-agent task handoff and start the task. (TODO becomes DOING)",
-  "context.actionDetail.taskExecuteContinue": "Prepare helper agent/sub-agent handoff and continue the in-progress task",
+  "context.actionDetail.taskExecuteRun": "Prepare helper agent/sub-agent task handoff and start the task: {task}. (TODO becomes DOING)",
+  "context.actionDetail.taskExecuteContinue": "Prepare helper agent/sub-agent handoff and continue the in-progress task: {task}",
   "context.actionDetail.reviewFixCommit": "Create a review-fix commit with resolved feedback summary",
   "context.actionDetail.prePrReviewRun": "Run the pre-PR review via a helper agent/sub-agent and prepare `review-trace.json`",
   "context.actionDetail.prePrReviewRecord": "Record pre-PR review evidence into decisions.md and tasks.md",
-  "context.actionDetail.codeReviewRun": "Run the PR review via a helper agent/sub-agent and prepare evidence/fix summary",
+  "context.actionDetail.codeReviewRun": "Check PR review comments, then use a helper agent/sub-agent for the follow-up fixes and evidence summary",
   "context.actionDetail.prCreate": "Create PR and sync PR fields in tasks.md",
   "context.actionDetail.prCreateRequiredSequence": "Complete PR 2-step flow: prepare draft + OK, then create and sync",
   "context.actionDetail.prCreatePrepareFromDoc": "Refine pr.md draft and set Status to Ready",
@@ -3769,6 +3769,14 @@ function resolveProjectCommitTopic(feature) {
   const topic = withoutTaskId || normalizeCommitTopicText(feature.folderName);
   return toShellSafeCommitTopic(topic);
 }
+function resolveTaskUiLabel(task) {
+  if (!task) return "T-unknown task";
+  const id = task.id?.trim();
+  const title = task.title.trim();
+  const normalizedTitle = id ? title.replace(new RegExp(`^${id.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}\\s+`), "") : title;
+  if (id) return `${id} - ${normalizedTitle || title}`;
+  return title || "T-unknown task";
+}
 function getReviewFixCommitGuidance(feature, lang, options) {
   const prePr = !!options?.prePr;
   if (prePr) {
@@ -4066,6 +4074,9 @@ function getStepDefinitions(ctx) {
       requiresUserCheck: true,
       taskExecutePhase: "complete",
       uiDetailKey: "context.actionDetail.taskExecuteContinue",
+      uiDetailParams: {
+        task: resolveTaskUiLabel(f.activeTask)
+      },
       scope: "docs",
       cwd: f.git.docsGitCwd,
       cmd: buildSelfCliCommand(
@@ -4188,6 +4199,9 @@ ${tr(lang, "messages", "taskCommitGateWarnProceed", {
         requiresUserCheck: true,
         taskExecutePhase: "start",
         uiDetailKey: "context.actionDetail.taskExecuteRun",
+        uiDetailParams: {
+          task: resolveTaskUiLabel(f.nextTodoTask)
+        },
         scope: "docs",
         cwd: f.git.docsGitCwd,
         cmd: buildSelfCliCommand(
@@ -4348,7 +4362,7 @@ ${tr(lang, "messages", "prePrReviewDecisionReconfirm", {
   const isCodeReviewNeedEvidence = (f) => isCodeReviewCurrent(f) && f.pr.status === "Review" && workflowPolicy.requireReview && !isCodeReviewNeedEvidenceField(f) && !f.prReview.evidenceProvided;
   const isCodeReviewNeedDecisionField = (f) => isCodeReviewCurrent(f) && f.pr.status === "Review" && workflowPolicy.requireReview && !isCodeReviewNeedEvidenceField(f) && f.prReview.evidenceProvided && !f.docs.prReviewDecisionFieldExists;
   const isCodeReviewNeedDecision = (f) => isCodeReviewCurrent(f) && f.pr.status === "Review" && workflowPolicy.requireReview && !isCodeReviewNeedEvidenceField(f) && f.prReview.evidenceProvided && !isCodeReviewNeedDecisionField(f) && !f.prReview.decisionProvided;
-  const isCodeReviewRun = (f) => isCodeReviewCurrent(f) && f.pr.status === "Review" && workflowPolicy.requireReview && !isCodeReviewNeedEvidenceField(f) && !f.prReview.evidenceProvided && !f.prReview.decisionProvided;
+  const isCodeReviewRun = (f) => isCodeReviewCurrent(f) && f.pr.status === "Review" && workflowPolicy.requireReview && (f.git.projectBranchAhead || 0) === 0 && !isCodeReviewNeedEvidenceField(f) && !f.prReview.evidenceProvided && !f.prReview.decisionProvided;
   const isCodeReviewFinalize = (f) => isCodeReviewCurrent(f) && !isCodeReviewSyncApproved(f) && (!workflowPolicy.requireReview || f.pr.status === "Review" && !isCodeReviewNeedEvidenceField(f) && !isCodeReviewNeedEvidence(f) && !isCodeReviewNeedDecisionField(f) && !isCodeReviewNeedDecision(f) && f.prReview.evidenceProvided && f.prReview.decisionProvided);
   const isCodeReviewRequestReview = (f) => isCodeReviewCurrent(f) && !!f.pr.status && f.pr.status !== "Review";
   const getCodeReviewRunActions = (f) => [
@@ -5687,13 +5701,21 @@ function hasStructuredReviewSummary(value) {
   if (!value) return false;
   const trimmed = value.trim();
   if (!trimmed) return false;
-  return /^(?:summary|요약)\s*[:：]\s*\S.+$/i.test(trimmed);
+  const match = trimmed.match(/^(?:summary|요약)\s*[:：]\s*(.+)$/i);
+  if (!match) return false;
+  const payload = (match[1] || "").trim();
+  if (!payload) return false;
+  return !isReviewDraftPlaceholder(payload) && !isPlaceholderReviewEvidence(payload);
 }
 function hasStructuredReviewDecision(value) {
   if (!value) return false;
   const trimmed = value.trim();
   if (!trimmed) return false;
-  return /^(?:decision|결정)\s*[:：]\s*\S.+$/i.test(trimmed);
+  const match = trimmed.match(/^(?:decision|결정)\s*[:：]\s*(.+)$/i);
+  if (!match) return false;
+  const payload = (match[1] || "").trim();
+  if (!payload) return false;
+  return !isReviewDraftPlaceholder(payload) && !isPlaceholderReviewEvidence(payload);
 }
 function parsePrePrDecisionOutcome(value) {
   if (!value) return void 0;
@@ -8122,11 +8144,11 @@ function annotateActions(actions) {
 }
 function getActionSummary(action, lang) {
   if (action.uiSummaryKey) {
-    const localized = tr(lang, "cli", action.uiSummaryKey);
+    const localized = tr(lang, "cli", action.uiSummaryKey, action.uiDetailParams);
     if (localized !== `cli.${action.uiSummaryKey}`) return localized;
   }
   if (action.uiDetailKey) {
-    const localized = tr(lang, "cli", action.uiDetailKey);
+    const localized = tr(lang, "cli", action.uiDetailKey, action.uiDetailParams);
     if (localized !== `cli.${action.uiDetailKey}`) return localized;
   }
   const detailKey = action.category ? ACTION_DETAIL_KEY_BY_CATEGORY[action.category] : void 0;
@@ -8151,7 +8173,7 @@ function toOneLine(text) {
 }
 function buildActionDetail(action, lang) {
   if (action.uiDetailKey) {
-    const localized = tr(lang, "cli", action.uiDetailKey);
+    const localized = tr(lang, "cli", action.uiDetailKey, action.uiDetailParams);
     if (localized !== `cli.${action.uiDetailKey}`) return localized;
   }
   const formatBranchCreateDetail = (command) => {
@@ -8274,7 +8296,8 @@ function buildActionSnapshot(actionOptions) {
         operationType: action.operationType,
         requiresUserCheck: !!action.requiresUserCheck,
         uiSummaryKey: action.uiSummaryKey,
-        uiDetailKey: action.uiDetailKey
+        uiDetailKey: action.uiDetailKey,
+        uiDetailParams: action.uiDetailParams ? JSON.stringify(action.uiDetailParams) : void 0
       };
     }
     return {
@@ -8285,7 +8308,8 @@ function buildActionSnapshot(actionOptions) {
       operationType: action.operationType,
       requiresUserCheck: !!action.requiresUserCheck,
       uiSummaryKey: action.uiSummaryKey,
-      uiDetailKey: action.uiDetailKey
+      uiDetailKey: action.uiDetailKey,
+      uiDetailParams: action.uiDetailParams ? JSON.stringify(action.uiDetailParams) : void 0
     };
   });
 }
@@ -8907,6 +8931,9 @@ function toCompactActionOption(option) {
   if (option.action.taskExecutePhase) {
     base.taskExecutePhase = option.action.taskExecutePhase;
   }
+  if (option.action.uiDetailParams) {
+    base.uiDetailParams = option.action.uiDetailParams;
+  }
   if (option.action.type === "command") {
     base.scope = option.action.scope;
     base.cwd = option.action.cwd;
@@ -9197,7 +9224,11 @@ async function runApprovedOption(state, config, lang, featureName, selectionOpti
     approval,
     state.actionOptions.map((o) => o.label)
   );
-  parsedLabel = parsedApproval?.label ?? null;
+  const replanOptions = state.actionOptions.filter(
+    (option) => option.action.category === "user_request_replan"
+  );
+  const implicitReplanRequest = approval.trim();
+  parsedLabel = parsedApproval?.label ?? (replanOptions.length > 0 && implicitReplanRequest ? replanOptions[0].label : null);
   if (!parsedLabel) {
     throw createCliError(
       "INVALID_APPROVAL",
@@ -9241,7 +9272,7 @@ async function runApprovedOption(state, config, lang, featureName, selectionOpti
   }
   const selectedAction = freshSelected.action;
   if (selectedAction.category === "user_request_replan") {
-    const requestText = parsedApproval?.requestText?.trim();
+    const requestText = (parsedApproval?.requestText?.trim() || (parsedApproval ? "" : implicitReplanRequest)).trim();
     if (!requestText) {
       throw createCliError(
         "INVALID_APPROVAL",
@@ -12401,6 +12432,12 @@ function sanitizeDraftMetadataValue(raw) {
   if (/^\(.+\)$/.test(value)) return void 0;
   return value;
 }
+function sanitizeDraftTitleValue(raw) {
+  const value = sanitizeDraftMetadataValue(raw);
+  if (!value) return void 0;
+  const normalized = value.replace(/`/g, "").replace(/\*\*(.*?)\*\*/g, "$1").replace(/\[(.*?)\]\((.*?)\)/g, "$1").replace(/\s+/g, " ").trim();
+  return normalized || void 0;
+}
 function parseWorkflowDraftStatus(raw) {
   const value = (raw || "").trim();
   if (!value) return void 0;
@@ -12412,7 +12449,7 @@ function parseWorkflowDraftMetadata(content) {
   const status = parseWorkflowDraftStatus(
     extractDraftMetadataValue(content, ["Status", "\uC0C1\uD0DC"])
   );
-  const title = sanitizeDraftMetadataValue(
+  const title = sanitizeDraftTitleValue(
     extractDraftMetadataValue(content, ["Title", "\uC81C\uBAA9", "PR Title", "PR \uC81C\uBAA9"])
   );
   const labels = sanitizeDraftMetadataValue(
@@ -13436,7 +13473,9 @@ function insertFieldInGithubIssueSection(content, key, value) {
 }
 function insertFieldInMetadataSection(content, key, value) {
   const lines = content.split("\n");
-  const headingIndex = lines.findIndex((line) => /^\s*##\s+Metadata\s*$/.test(line));
+  const headingIndex = lines.findIndex(
+    (line) => /^\s*##\s+(?:Metadata|메타데이터)\s*$/.test(line)
+  );
   if (headingIndex < 0) return { content, changed: false };
   let end = lines.length;
   for (let i = headingIndex + 1; i < lines.length; i++) {
@@ -15199,14 +15238,6 @@ function asNonEmptyString(value, fallback) {
   const trimmed = value.trim();
   return trimmed || fallback;
 }
-function asRequiredNonEmptyString(value, field) {
-  const normalized = asNonEmptyString(value, "");
-  if (normalized) return normalized;
-  throw createCliError(
-    "VALIDATION_FAILED",
-    `Evidence JSON ${field} is required.`
-  );
-}
 function asRequiredBoolean(value, field) {
   if (typeof value === "boolean") return value;
   throw createCliError(
@@ -15223,9 +15254,67 @@ function asRequiredNonNegativeInteger(value, field) {
     `Evidence JSON ${field} must be a non-negative integer.`
   );
 }
+function asRequiredTextLike(value, field) {
+  if (typeof value === "string") {
+    const trimmed = value.trim();
+    if (trimmed) return trimmed;
+  }
+  if (typeof value === "number" && Number.isFinite(value) && !Number.isNaN(value)) {
+    return String(value);
+  }
+  throw createCliError(
+    "VALIDATION_FAILED",
+    `Evidence JSON ${field} is required.`
+  );
+}
+function isPlaceholderReviewEvidence2(value) {
+  return /^(?:-|#)?\s*(?:tbd|todo|n\/a|na|none|pending|미정|없음|-)\s*$/i.test(
+    value.trim()
+  );
+}
+function isReviewDraftPlaceholder2(value) {
+  return /^(?:-|#)?\s*(?:tbd|todo|pending|fill(?:\s+in)?|template|example|미정|작성|기입|n\/a|na)\b/i.test(
+    value.trim()
+  );
+}
+function isExplicitNoResidualRiskEntry2(value) {
+  const trimmed = value.trim().toLowerCase();
+  return trimmed === "none" || trimmed === "no residual risk" || trimmed === "no residual risks" || trimmed === "no residual risks found" || trimmed === "no residual risks found in reviewed scope" || trimmed === "\uC794\uC5EC \uB9AC\uC2A4\uD06C \uC5C6\uC74C" || trimmed === "\uC794\uC5EC \uC704\uD5D8 \uC5C6\uC74C";
+}
+function asRequiredReviewField(value, field, options) {
+  const normalized = asRequiredTextLike(value, field);
+  const allowExplicitNone = !!options?.allowExplicitNone;
+  if (isReviewDraftPlaceholder2(normalized)) {
+    throw createCliError(
+      "VALIDATION_FAILED",
+      `Evidence JSON ${field} contains draft placeholder text.`
+    );
+  }
+  if (isPlaceholderReviewEvidence2(normalized) && !(allowExplicitNone && normalized.trim().toLowerCase() === "none")) {
+    throw createCliError(
+      "VALIDATION_FAILED",
+      `Evidence JSON ${field} contains placeholder evidence text.`
+    );
+  }
+  return normalized;
+}
 function normalizeCommandsExecuted(value) {
-  if (!Array.isArray(value)) return [];
-  return value.map((entry) => typeof entry === "string" ? entry.trim() : "").filter(Boolean);
+  if (value == null) return [];
+  if (!Array.isArray(value)) {
+    throw createCliError(
+      "VALIDATION_FAILED",
+      'Evidence JSON "commandsExecuted" must be an array when provided.'
+    );
+  }
+  return value.map((entry, index) => {
+    if (typeof entry !== "string" || !entry.trim()) {
+      throw createCliError(
+        "VALIDATION_FAILED",
+        `Evidence JSON commandsExecuted[${index}] must be a non-empty string.`
+      );
+    }
+    return entry.trim();
+  });
 }
 function normalizeGitPath3(value) {
   return value.trim().replace(/\\/g, "/").replace(/^\.\/+/, "").replace(/\/+$/, "");
@@ -15243,6 +15332,16 @@ function uniquePaths(values) {
   }
   return out;
 }
+function normalizeFileLine(value, field) {
+  const normalized = asRequiredTextLike(value, field);
+  if (!/^\d+(?:[-:]\d+)?$/.test(normalized)) {
+    throw createCliError(
+      "VALIDATION_FAILED",
+      `Evidence JSON ${field} must start with a numeric line reference (for example "88" or "88-96").`
+    );
+  }
+  return normalized;
+}
 function normalizeEvidenceFiles(value) {
   if (!Array.isArray(value)) {
     throw createCliError(
@@ -15265,22 +15364,61 @@ function normalizeEvidenceFiles(value) {
         `Evidence JSON files[${index}].path is required.`
       );
     }
-    const review = file.review || {};
+    const review = file.review && typeof file.review === "object" ? file.review : file;
     return {
       path: filePath,
       review: {
-        risk: asNonEmptyString(review.risk, "not specified"),
-        security: asNonEmptyString(review.security, "not specified"),
-        perf: asNonEmptyString(review.perf, "not specified"),
-        maintainability: asNonEmptyString(
+        risk: asRequiredTextLike(review.risk, `"files[${index}].risk"`),
+        security: asRequiredTextLike(
+          review.security,
+          `"files[${index}].security"`
+        ),
+        perf: asRequiredTextLike(
+          review.perf ?? review.performance,
+          `"files[${index}].perf"`
+        ),
+        maintainability: asRequiredTextLike(
           review.maintainability,
-          "not specified"
+          `"files[${index}].maintainability"`
         ),
-        fileLine: asNonEmptyString(review.fileLine, "-")
+        fileLine: normalizeFileLine(
+          review.fileLine,
+          `"files[${index}].fileLine"`
+        )
       }
     };
   });
 }
+function normalizeResidualRisks(value) {
+  if (typeof value === "string" && value.trim()) {
+    const normalized = asRequiredReviewField(
+      value,
+      '"residualRisks"',
+      { allowExplicitNone: true }
+    );
+    if (!isExplicitNoResidualRiskEntry2(normalized) && isPlaceholderReviewEvidence2(normalized)) {
+      throw createCliError(
+        "VALIDATION_FAILED",
+        'Evidence JSON "residualRisks" contains placeholder evidence text.'
+      );
+    }
+    return [normalized];
+  }
+  if (Array.isArray(value)) {
+    const entries = value.map(
+      (entry, index) => asRequiredReviewField(entry, `"residualRisks[${index}]"`, {
+        allowExplicitNone: true
+      })
+    ).filter(
+      (entry) => isExplicitNoResidualRiskEntry2(entry) || !isReviewDraftPlaceholder2(entry) && !isPlaceholderReviewEvidence2(entry)
+    );
+    if (entries.length > 0) return entries;
+  }
+  throw createCliError(
+    "VALIDATION_FAILED",
+    'Evidence JSON "residualRisks" must be a non-empty string or string array.'
+  );
+}
 var PrePrReviewValidator = class {
   constructor(ctx) {
     this.ctx = ctx;
@@ -15307,18 +15445,19 @@ var PrePrReviewValidator = class {
       );
     }
     const normalizedEvidence = {
-      summary: asRequiredNonEmptyString(evidence.summary, '"summary"'),
-      featureIntentSummary: asRequiredNonEmptyString(
+      summary: asRequiredReviewField(evidence.summary, '"summary"'),
+      featureIntentSummary: asRequiredReviewField(
         evidence.featureIntentSummary,
         '"featureIntentSummary"'
       ),
-      implementationFit: asRequiredNonEmptyString(
+      implementationFit: asRequiredReviewField(
         evidence.implementationFit,
         '"implementationFit"'
       ),
-      missingCases: asRequiredNonEmptyString(
+      missingCases: asRequiredReviewField(
         evidence.missingCases,
-        '"missingCases"'
+        '"missingCases"',
+        { allowExplicitNone: true }
       ),
       specAlignmentChecked: asRequiredBoolean(
         evidence.specAlignmentChecked,
@@ -15333,7 +15472,7 @@ var PrePrReviewValidator = class {
         '"blockingFindings"'
       ),
       files: normalizeEvidenceFiles(evidence.files),
-      residualRisks: asNonEmptyString(evidence.residualRisks, "Not specified"),
+      residualRisks: normalizeResidualRisks(evidence.residualRisks),
       commandsExecuted: normalizeCommandsExecuted(evidence.commandsExecuted)
     };
     if (normalizedEvidence.blockingFindings > normalizedEvidence.findingCount) {
@@ -15515,7 +15654,7 @@ var DEFAULT_EVIDENCE_FOR_ANY_MODE = {
   findingCount: 0,
   blockingFindings: 0,
   files: [],
-  residualRisks: "Not specified",
+  residualRisks: ["none"],
   commandsExecuted: []
 };
 function escapeRegExp4(value) {
@@ -15602,7 +15741,7 @@ ${normalizedCommands.map((c) => `  - \`${c}\``).join("\n")}
 
 ` : "";
   let filesSection = "";
-  if (input.evidence.files.length === 0) {
+  if (input.evidence.findingCount === 0 || input.evidence.files.length === 0) {
     filesSection = "  - 0 findings";
   } else {
     filesSection = input.evidence.files.map((f) => {
@@ -15613,6 +15752,7 @@ ${normalizedCommands.map((c) => `  - \`${c}\``).join("\n")}
     - Maintainability: ${f.review.maintainability}`;
     }).join("\n");
   }
+  const residualRisksSection = input.evidence.residualRisks.length > 0 ? input.evidence.residualRisks.map((entry) => `  - ${entry}`).join("\n") : "  - none";
   const mainScopeFiles = input.scope.mainChangedFiles.length > 0 ? input.scope.mainChangedFiles.map((entry) => `    - ${entry}`).join("\n") : "    - (none)";
   const worktreeScopeFiles = input.scope.worktreeChangedFiles.length > 0 ? input.scope.worktreeChangedFiles.map((entry) => `    - ${entry}`).join("\n") : "    - (none)";
   return `## Pre-PR Review Log (${input.date})
@@ -15631,7 +15771,7 @@ ${normalizedCommands.map((c) => `  - \`${c}\``).join("\n")}
 ${commandsRun}
 
 - **Residual Risks**:
-  - ${input.evidence.residualRisks}
+${residualRisksSection}
 
 - **Review Scope**:
   - **Main Base Ref**: ${input.scope.baseRef}
@@ -15992,18 +16132,20 @@ async function runPrePrReview(featureName, options) {
 function buildCodeReviewRunPrompt(input) {
   if (input.lang === "ko") {
     return [
-      "PR \uB9AC\uBDF0 \uC2E4\uD589\uC744 \uC2DC\uC791\uD558\uC138\uC694.",
+      "PR \uB9AC\uBDF0 \uCF54\uBA58\uD2B8\uB97C \uD655\uC778\uD558\uACE0 \uBCF4\uC870 \uC5D0\uC774\uC804\uD2B8\uB85C \uC218\uC815 \uC791\uC5C5\uC744 \uC9C4\uD589\uD558\uC138\uC694.",
       `- Feature: ${input.featureRef}`,
       `- ${input.basePrompt}`,
-      "- \uB9AC\uBDF0 \uCF54\uBA58\uD2B8 \uBC18\uC601 \uD6C4 `tasks.md`\uC758 `PR \uC804 \uB9AC\uBDF0 Evidence/Decision`\uC774 \uC544\uB2C8\uB77C `PR Review Evidence/Decision`\uC744 \uCD5C\uC2E0\uC73C\uB85C \uAE30\uB85D\uD558\uC138\uC694.",
+      "- \uC0AC\uB78C/CodeRabbit\uC774 \uB0A8\uAE34 \uB9AC\uBDF0 \uCF54\uBA58\uD2B8\uB97C \uAC80\uD1A0\uD558\uACE0 \uD544\uC694\uD55C \uCF54\uB4DC/\uBB38\uC11C \uC218\uC815\uC744 \uC9C4\uD589\uD558\uC138\uC694.",
+      "- \uC218\uC815 \uB0B4\uC6A9\uACFC \uAC80\uD1A0 \uACB0\uACFC\uB97C \uBC18\uC601\uD55C \uB4A4 `tasks.md`\uC758 `PR Review Evidence/Decision`\uC744 \uCD5C\uC2E0\uC73C\uB85C \uAE30\uB85D\uD558\uC138\uC694.",
       "- \uAD00\uB828 \uC218\uC815\uC774 \uC0DD\uAE30\uBA74 \uCF54\uB4DC/\uBB38\uC11C \uBCC0\uACBD\uC744 \uC815\uB9AC\uD558\uACE0, push/merge\uB294 \uBA54\uC778 \uC5D0\uC774\uC804\uD2B8 \uCD5C\uC885 \uC0C1\uD0DC\uC5D0\uC11C\uB9CC \uC9C4\uD589\uD558\uC138\uC694."
     ].join("\n");
   }
   return [
-    "Start the PR review execution.",
+    "Review PR comments and use a helper agent/sub-agent for the follow-up fixes.",
     `- Feature: ${input.featureRef}`,
     `- ${input.basePrompt}`,
-    "- Update `PR Review Evidence` and `PR Review Decision` in `tasks.md` after reviewing comments and applying fixes.",
+    "- Check human/CodeRabbit review comments and make the required code/docs changes.",
+    "- Update `PR Review Evidence` and `PR Review Decision` in `tasks.md` after applying the fixes and summarizing the outcome.",
     "- If review fixes are needed, keep code/docs changes ready for the main-agent finalize state. Push/merge stays in the main agent."
   ].join("\n");
 }
@@ -16053,7 +16195,7 @@ async function runCodeReviewRun(featureName, options) {
   console.log();
   console.log(
     chalk8.yellow(
-      config.lang === "ko" ? "\uC774 \uBA85\uB839\uC740 PR \uB9AC\uBDF0 handoff\uB9CC \uC900\uBE44\uD569\uB2C8\uB2E4. \uB9AC\uBDF0 evidence/decision\uC744 \uC9C1\uC811 \uAE30\uB85D\uD558\uAC70\uB098 \uC6CC\uD06C\uD50C\uB85C\uC6B0 \uC0C1\uD0DC\uB97C \uBC14\uB85C \uB118\uAE30\uC9C0 \uC54A\uC2B5\uB2C8\uB2E4." : "This command only prepares the PR review handoff. It does not record review evidence/decision or advance workflow state by itself."
+      config.lang === "ko" ? "\uC774 \uBA85\uB839\uC740 PR \uB9AC\uBDF0 \uCF54\uBA58\uD2B8 \uB300\uC751\uC6A9 handoff\uB9CC \uC900\uBE44\uD569\uB2C8\uB2E4. \uCF54\uBA58\uD2B8\uB97C \uC9C1\uC811 \uC77D\uC5B4\uC624\uAC70\uB098 evidence/decision\uC744 \uC790\uB3D9 \uAE30\uB85D\uD558\uC9C0 \uC54A\uC73C\uBA70, \uC0C1\uD0DC\uB3C4 \uBC14\uB85C \uB118\uAE30\uC9C0 \uC54A\uC2B5\uB2C8\uB2E4." : "This command only prepares a handoff for addressing PR review comments. It does not fetch comments automatically, record review evidence/decision, or advance workflow state by itself."
     )
   );
   console.log(chalk8.gray(`- substate: ${payload.substateId}`));