lee-spec-kit 0.6.22 → 0.6.24

package/dist/index.js

@@ -206,8 +206,10 @@ var ko = {
     "context.list.recordPrLink": "PR \uB9C1\uD06C \uAE30\uB85D",
     "context.list.addPrePrReviewField": "Pre-PR Review \uD544\uB4DC \uCD94\uAC00",
     "context.list.completePrePrReview": "Pre-PR \uB9AC\uBDF0 \uC644\uB8CC \uCC98\uB9AC",
+    "context.list.addPrePrFindings": "Pre-PR Findings \uAE30\uB85D",
     "context.list.addPrePrEvidence": "Pre-PR Evidence \uADFC\uAC70 \uCD94\uAC00",
     "context.list.addPrePrDecision": "Pre-PR Decision \uAE30\uB85D",
+    "context.list.resolvePrePrDecision": "Pre-PR Decision\uC744 approve\uB85C \uC815\uB9AC",
     "context.list.addPrReviewEvidence": "PR \uB9AC\uBDF0 Evidence \uC694\uC57D \uCD94\uAC00",
     "context.list.addPrReviewDecision": "PR \uB9AC\uBDF0 Decision \uAE30\uB85D",
     "context.list.setPrStatus": "PR \uC0C1\uD0DC \uC124\uC815",
@@ -487,7 +489,7 @@ var ko = {
     reviewFixCommitGuidance: "PR \uB9AC\uBDF0 \uC218\uC815 \uCEE4\uBC0B\uC744 \uC9C4\uD589\uD558\uC138\uC694. \uB9AC\uBDF0 \uBC18\uC601 \uD30C\uC77C\uB9CC \uC2A4\uD14C\uC774\uC9D5\uD55C \uB4A4 `fix(review): <review-fix-summary>` \uD615\uC2DD\uC73C\uB85C \uCEE4\uBC0B\uD558\uC138\uC694. `<review-fix-summary>`\uC5D0\uB294 \uC774\uBC88 \uCEE4\uBC0B\uC5D0\uC11C \uC2E4\uC81C\uB85C \uD574\uACB0\uD55C \uB9AC\uBDF0 \uD56D\uBAA9 \uC694\uC57D\uC744 \uC791\uC131\uD558\uC138\uC694. (\uD0DC\uC2A4\uD06C \uC81C\uBAA9 \uC7AC\uC0AC\uC6A9 \uAE08\uC9C0)",
     standaloneNeedsProjectRoot: "standalone \uBAA8\uB4DC\uC5D0\uC11C\uB294 projectRoot \uC124\uC815\uC774 \uD544\uC694\uD569\uB2C8\uB2E4. (npx lee-spec-kit config --project-root ...)",
     createBranch: 'cd "{projectGitCwd}" && mkdir -p .worktrees && (git worktree add ".worktrees/feat-{issueNumber}-{slug}" "feat/{issueNumber}-{slug}" || git worktree add -b "feat/{issueNumber}-{slug}" ".worktrees/feat-{issueNumber}-{slug}") && echo "worktree: {projectGitCwd}/.worktrees/feat-{issueNumber}-{slug}"',
-    worktreeCleanupCommand: 'cd "{projectGitCwd}" && git worktree remove "{worktreePath}" && git worktree prune',
+    worktreeCleanupCommand: 'cd "{projectGitCwd}" && git worktree remove "{worktreePath}" && git worktree prune && CURRENT_BRANCH=$(git branch --show-current) && DEFAULT_BRANCH=$(git symbolic-ref --quiet --short refs/remotes/origin/HEAD 2>/dev/null | cut -d/ -f2-) && TARGET_BRANCH="${DEFAULT_BRANCH:-$CURRENT_BRANCH}" && if [ -n "$TARGET_BRANCH" ]; then git checkout "$TARGET_BRANCH" >/dev/null 2>&1 || true; fi && if git rev-parse --abbrev-ref --symbolic-full-name "@{u}" >/dev/null 2>&1 && [ -z "$(git status --porcelain)" ]; then git pull --ff-only || true; fi',
     tasksAllDoneButNoChecklist: '\uC644\uB8CC \uC870\uAC74 \uCCB4\uD06C\uB9AC\uC2A4\uD2B8\uB97C \uC791\uC131\uD558\uC138\uC694. tasks.md\uC758 "\uC644\uB8CC \uC870\uAC74" \uC139\uC158\uC5D0 \uAC80\uC99D \uD56D\uBAA9\uC744 \uCD94\uAC00\uD558\uACE0, \uC0AC\uC6A9\uC790\uC640 \uD655\uC778 \uD6C4 \uCDA9\uC871 \uD56D\uBAA9\uC744 [x]\uB85C \uCCB4\uD06C\uD558\uC138\uC694. \uCD5C\uC885 \uC2B9\uC778(OK)\uB3C4 \uBC18\uC601\uD558\uC138\uC694.',
     tasksAllDoneButChecklist: "\uC644\uB8CC \uC870\uAC74 \uCCB4\uD06C\uB9AC\uC2A4\uD2B8\uC758 \uB0A8\uC740 \uD56D\uBAA9\uC744 \uC9C4\uD589\uD558\uC138\uC694. \uD604\uC7AC \uC9C4\uD589: ({checked}/{total}) \uC0AC\uC6A9\uC790\uC640 \uD655\uC778 \uD6C4 \uCDA9\uC871 \uD56D\uBAA9\uC744 [x]\uB85C \uCCB4\uD06C\uD558\uACE0 \uCD5C\uC885 \uC2B9\uC778(OK)\uC744 \uBC18\uC601\uD558\uC138\uC694.",
     finishDoingTask: '\uD604\uC7AC DOING/REVIEW \uD0DC\uC2A4\uD06C\uB97C \uC218\uD589\uD558\uC138\uC694: "{title}" ({done}/{total}) \uC644\uB8CC \uC2DC \uACB0\uACFC/\uAC80\uC99D\uC744 \uACF5\uC720\uD558\uACE0 \uC2B9\uC778(`A` \uB610\uB294 `A OK` \uD615\uC2DD)\uC744 \uBC1B\uC740 \uB4A4 DONE \uCC98\uB9AC',
@@ -542,6 +544,7 @@ var ko = {
     legacyTasksDocStatusField: "\uAD6C\uBC84\uC804 tasks.md \uD3EC\uB9F7\uC785\uB2C8\uB2E4. `\uBB38\uC11C \uC0C1\uD0DC` \uD544\uB4DC(Draft/Review/Approved)\uB97C \uCD94\uAC00\uD574 \uD0DC\uC2A4\uD06C \uC2B9\uC778 \uB2E8\uACC4\uB97C \uD65C\uC131\uD654\uD558\uC138\uC694.",
     legacyTasksPrFields: "\uAD6C\uBC84\uC804 tasks.md \uD3EC\uB9F7\uC785\uB2C8\uB2E4. PR \uB2E8\uACC4 \uC804\uC5D0 `PR` \uBC0F `PR \uC0C1\uD0DC` \uD544\uB4DC\uB97C \uCD94\uAC00\uD558\uC138\uC694.",
     legacyTasksPrePrReviewField: "\uAD6C\uBC84\uC804 tasks.md \uD3EC\uB9F7\uC785\uB2C8\uB2E4. PR \uB2E8\uACC4 \uC804\uC5D0 `PR \uC804 \uB9AC\uBDF0` \uD544\uB4DC\uB97C \uCD94\uAC00\uD558\uC138\uC694. (`- **PR \uC804 \uB9AC\uBDF0**: Pending | Done`)",
+    legacyTasksPrePrFindingsField: "\uAD6C\uBC84\uC804 tasks.md \uD3EC\uB9F7\uC785\uB2C8\uB2E4. PR \uB2E8\uACC4 \uC804\uC5D0 `PR \uC804 \uB9AC\uBDF0 Findings` \uD544\uB4DC\uB97C \uCD94\uAC00\uD558\uC138\uC694. (`- **PR \uC804 \uB9AC\uBDF0 Findings**: major=0, minor=0`)",
     legacyTasksPrePrEvidenceField: "\uAD6C\uBC84\uC804 tasks.md \uD3EC\uB9F7\uC785\uB2C8\uB2E4. PR \uB2E8\uACC4 \uC804\uC5D0 `PR \uC804 \uB9AC\uBDF0 Evidence` \uD544\uB4DC\uB97C \uCD94\uAC00\uD558\uC138\uC694.",
     legacyTasksPrePrDecisionField: "\uAD6C\uBC84\uC804 tasks.md \uD3EC\uB9F7\uC785\uB2C8\uB2E4. PR \uB2E8\uACC4 \uC804\uC5D0 `PR \uC804 \uB9AC\uBDF0 Decision` \uD544\uB4DC\uB97C \uCD94\uAC00\uD558\uC138\uC694. (`- **PR \uC804 \uB9AC\uBDF0 Decision**: \uACB0\uC815: ...`)",
     legacyTasksPrReviewEvidenceField: "\uAD6C\uBC84\uC804 tasks.md \uD3EC\uB9F7\uC785\uB2C8\uB2E4. \uB9AC\uBDF0 \uB2E8\uACC4 \uC804\uC5D0 `PR \uB9AC\uBDF0 Evidence` \uD544\uB4DC\uB97C \uCD94\uAC00\uD558\uC138\uC694.",
@@ -560,8 +563,10 @@ var ko = {
     workflowPrRemoteChecksPending: "\uC6D0\uACA9 PR \uCCB4\uD06C \uB300\uAE30\uAC00 {count}\uAC74 \uAC10\uC9C0\uB418\uC5C8\uC2B5\uB2C8\uB2E4. \uCCB4\uD06C \uC644\uB8CC \uD6C4 \uB2E4\uC2DC \uD655\uC778\uD558\uC138\uC694.",
     workflowPrePrReviewMissing: "\uC644\uB8CC \uC0C1\uD0DC\uC774\uC9C0\uB9CC `PR \uC804 \uB9AC\uBDF0` \uD544\uB4DC\uAC00 \uC5C6\uC2B5\uB2C8\uB2E4. (tasks.md\uC5D0 `- **PR \uC804 \uB9AC\uBDF0**: Pending | Done`\uC744 \uCD94\uAC00\uD558\uC138\uC694.)",
     workflowPrePrReviewNotDone: "\uC644\uB8CC \uC0C1\uD0DC\uC774\uC9C0\uB9CC `PR \uC804 \uB9AC\uBDF0`\uAC00 Done\uC774 \uC544\uB2D9\uB2C8\uB2E4. (\uC0AC\uC804 \uCF54\uB4DC\uB9AC\uBDF0 \uD6C4 Done\uC73C\uB85C \uC5C5\uB370\uC774\uD2B8\uD558\uC138\uC694.)",
-    workflowPrePrEvidenceMissing: "\uC644\uB8CC \uC0C1\uD0DC\uC774\uC9C0\uB9CC `PR \uC804 \uB9AC\uBDF0 Evidence`\uAC00 \uBE44\uC5B4\uC788\uC2B5\uB2C8\uB2E4. (\uB9AC\uBDF0 \uADFC\uAC70\uB97C \uAE30\uB85D\uD558\uC138\uC694.)",
-    workflowPrePrDecisionMissing: "\uC644\uB8CC \uC0C1\uD0DC\uC774\uC9C0\uB9CC `PR \uC804 \uB9AC\uBDF0 Decision`\uC774 \uBE44\uC5B4\uC788\uAC70\uB098 \uD615\uC2DD\uC774 \uC62C\uBC14\uB974\uC9C0 \uC54A\uC2B5\uB2C8\uB2E4. (`\uACB0\uC815: ...` \uB610\uB294 `decision: ...` \uD615\uC2DD)"
+    workflowPrePrFindingsMissing: "\uC644\uB8CC \uC0C1\uD0DC\uC774\uC9C0\uB9CC `PR \uC804 \uB9AC\uBDF0 Findings`\uAC00 \uC5C6\uAC70\uB098 \uD615\uC2DD\uC774 \uC62C\uBC14\uB974\uC9C0 \uC54A\uC2B5\uB2C8\uB2E4. (`major=<n>, minor=<n>` \uD615\uC2DD\uC73C\uB85C \uAE30\uB85D)",
+    workflowPrePrEvidenceMissing: "\uC644\uB8CC \uC0C1\uD0DC\uC774\uC9C0\uB9CC `PR \uC804 \uB9AC\uBDF0 Evidence`\uAC00 \uBE44\uC5B4\uC788\uAC70\uB098 \uC720\uD6A8\uD558\uC9C0 \uC54A\uC2B5\uB2C8\uB2E4. (path_required \uC815\uCC45\uC774\uBA74 \uC2E4\uC81C \uC874\uC7AC\uD558\uB294 \uACBD\uB85C\uB97C \uAE30\uB85D\uD558\uC138\uC694.)",
+    workflowPrePrDecisionMissing: "\uC644\uB8CC \uC0C1\uD0DC\uC774\uC9C0\uB9CC `PR \uC804 \uB9AC\uBDF0 Decision`\uC774 \uBE44\uC5B4\uC788\uAC70\uB098 \uD615\uC2DD\uC774 \uC62C\uBC14\uB974\uC9C0 \uC54A\uC2B5\uB2C8\uB2E4. (`decision: approve|changes_requested|blocked ...` \uD615\uC2DD)",
+    workflowPrePrDecisionNotApproved: "\uC644\uB8CC \uC0C1\uD0DC\uC774\uC9C0\uB9CC `PR \uC804 \uB9AC\uBDF0 Decision`\uC774 `{outcome}`\uC785\uB2C8\uB2E4. Findings\uB97C \uD574\uACB0\uD558\uACE0 pre-pr-review\uB97C \uC7AC\uC2E4\uD589\uD574 `approve`\uB85C \uB9DE\uCD94\uC138\uC694."
   }
 };
 var ko_default = ko;
@@ -688,8 +693,10 @@ var en = {
     "context.list.recordPrLink": "Record PR link",
     "context.list.addPrePrReviewField": "Add Pre-PR Review field",
     "context.list.completePrePrReview": "Complete Pre-PR review",
+    "context.list.addPrePrFindings": "Record Pre-PR Findings",
     "context.list.addPrePrEvidence": "Add Pre-PR Evidence",
     "context.list.addPrePrDecision": "Add Pre-PR Decision",
+    "context.list.resolvePrePrDecision": "Resolve Pre-PR decision to approve",
     "context.list.addPrReviewEvidence": "Add PR Review Evidence summary",
     "context.list.addPrReviewDecision": "Add PR Review Decision",
     "context.list.setPrStatus": "Set PR Status",
@@ -969,7 +976,7 @@ var en = {
     reviewFixCommitGuidance: "Commit PR review fixes. Stage only review-fix files, then commit with `fix(review): <review-fix-summary>`. `<review-fix-summary>` must describe review comments resolved in this commit (do not reuse task titles).",
     standaloneNeedsProjectRoot: "Standalone mode requires projectRoot. (npx lee-spec-kit config --project-root ...)",
     createBranch: 'cd "{projectGitCwd}" && mkdir -p .worktrees && (git worktree add ".worktrees/feat-{issueNumber}-{slug}" "feat/{issueNumber}-{slug}" || git worktree add -b "feat/{issueNumber}-{slug}" ".worktrees/feat-{issueNumber}-{slug}") && echo "worktree: {projectGitCwd}/.worktrees/feat-{issueNumber}-{slug}"',
-    worktreeCleanupCommand: 'cd "{projectGitCwd}" && git worktree remove "{worktreePath}" && git worktree prune',
+    worktreeCleanupCommand: 'cd "{projectGitCwd}" && git worktree remove "{worktreePath}" && git worktree prune && CURRENT_BRANCH=$(git branch --show-current) && DEFAULT_BRANCH=$(git symbolic-ref --quiet --short refs/remotes/origin/HEAD 2>/dev/null | cut -d/ -f2-) && TARGET_BRANCH="${DEFAULT_BRANCH:-$CURRENT_BRANCH}" && if [ -n "$TARGET_BRANCH" ]; then git checkout "$TARGET_BRANCH" >/dev/null 2>&1 || true; fi && if git rev-parse --abbrev-ref --symbolic-full-name "@{u}" >/dev/null 2>&1 && [ -z "$(git status --porcelain)" ]; then git pull --ff-only || true; fi',
     tasksAllDoneButNoChecklist: 'Create the completion checklist. Add verification items to the tasks.md "Completion Criteria" section, then mark satisfied items as [x] after user confirmation. Record final approval (OK) as well.',
     tasksAllDoneButChecklist: "Proceed with remaining completion checklist items. Current progress: ({checked}/{total}) Mark items as [x] only after user confirmation and real verification. Record final approval (OK) as well.",
     finishDoingTask: 'Continue working on the current DOING/REVIEW task: "{title}" ({done}/{total}) After it is complete, share outcome/verification and get approval before marking DONE',
@@ -1024,6 +1031,7 @@ var en = {
     legacyTasksDocStatusField: "Legacy tasks.md format detected. Add a `Doc Status` field (Draft/Review/Approved) to enable tasks approval.",
     legacyTasksPrFields: "Legacy tasks.md format detected. Add `PR` and `PR Status` fields before PR steps.",
     legacyTasksPrePrReviewField: "Legacy tasks.md format detected. Add `Pre-PR Review` before PR steps. (`- **Pre-PR Review**: Pending | Done`)",
+    legacyTasksPrePrFindingsField: "Legacy tasks.md format detected. Add `Pre-PR Findings` before PR steps. (`- **Pre-PR Findings**: major=0, minor=0`)",
     legacyTasksPrePrEvidenceField: "Legacy tasks.md format detected. Add `Pre-PR Evidence` before PR steps.",
     legacyTasksPrePrDecisionField: "Legacy tasks.md format detected. Add `Pre-PR Decision` before PR steps. (`- **Pre-PR Decision**: decision: ...`)",
     legacyTasksPrReviewEvidenceField: "Legacy tasks.md format detected. Add `PR Review Evidence` before review iteration.",
@@ -1042,8 +1050,10 @@ var en = {
     workflowPrRemoteChecksPending: "Remote PR has {count} pending check(s). Wait for checks to complete, then re-check.",
     workflowPrePrReviewMissing: "Implementation is done but `Pre-PR Review` is missing. (Add `- **Pre-PR Review**: Pending | Done` in tasks.md.)",
     workflowPrePrReviewNotDone: "Implementation is done but `Pre-PR Review` is not Done. (Run pre-PR review, then update it to Done.)",
-    workflowPrePrEvidenceMissing: "Implementation is done but `Pre-PR Evidence` is empty. (Record review evidence.)",
-    workflowPrePrDecisionMissing: "Implementation is done but `Pre-PR Decision` is empty/invalid. (Use `decision: ...` or `\uACB0\uC815: ...`.)"
+    workflowPrePrFindingsMissing: "Implementation is done but `Pre-PR Findings` is missing/invalid. (Use `major=<n>, minor=<n>`.)",
+    workflowPrePrEvidenceMissing: "Implementation is done but `Pre-PR Evidence` is empty/invalid. (Record a real existing path when path_required policy is enabled.)",
+    workflowPrePrDecisionMissing: "Implementation is done but `Pre-PR Decision` is empty/invalid. (Use `decision: approve|changes_requested|blocked ...`.)",
+    workflowPrePrDecisionNotApproved: "Implementation is done but `Pre-PR Decision` is `{outcome}`. Resolve findings and re-run pre-PR review until decision becomes `approve`."
   }
 };
 var en_default = en;
@@ -1066,7 +1076,10 @@ var I18N = {
 };
 function tr(lang, category, key, vars = {}) {
   const safeLang = normalizeLang(lang);
-  const template = I18N[safeLang]?.[category]?.[key] ?? I18N[DEFAULT_LANG]?.[category]?.[key] ?? I18N.ko?.[category]?.[key] ?? `${category}.${key}`;
+  const safeCategory = I18N[safeLang]?.[category];
+  const defaultCategory = I18N[DEFAULT_LANG]?.[category];
+  const koCategory = I18N.ko?.[category];
+  const template = safeCategory?.[key] ?? defaultCategory?.[key] ?? koCategory?.[key] ?? `${category}.${key}`;
   return formatTemplate(template, vars);
 }
 
@@ -2259,7 +2272,10 @@ async function runInit(options) {
           },
           prePrReview: {
             skills: ["code-review-excellence"],
-            minorPolicy: "warn"
+            fallback: "builtin-checklist",
+            evidenceMode: "path_required",
+            findings: "required",
+            decisionEnum: ["approve", "changes_requested", "blocked"]
           }
         },
         pr: {
@@ -2878,6 +2894,11 @@ var ACTION_CATEGORIES = [
 
 // src/utils/workflow.ts
 var DEFAULT_PRE_PR_REVIEW_SKILLS = ["code-review-excellence"];
+var DEFAULT_PRE_PR_DECISION_ENUM = [
+  "approve",
+  "changes_requested",
+  "blocked"
+];
 function resolveWorkflowPolicy(workflow) {
   const mode = workflow?.mode === "local" ? "local" : "github";
   const policy = mode === "local" ? {
@@ -2939,17 +2960,42 @@ function normalizeSkillList(input) {
   }
   return [...deduped];
 }
+function normalizeDecisionEnumList(input) {
+  if (!Array.isArray(input)) return [];
+  const deduped = /* @__PURE__ */ new Set();
+  for (const raw of input) {
+    const value = String(raw || "").trim().toLowerCase();
+    if (!value) continue;
+    if (value === "approve") {
+      deduped.add("approve");
+      continue;
+    }
+    if (value === "changes_requested") {
+      deduped.add("changes_requested");
+      continue;
+    }
+    if (value === "blocked") {
+      deduped.add("blocked");
+      continue;
+    }
+  }
+  return [...deduped];
+}
 function resolvePrePrReviewPolicy(workflow) {
   const workflowPolicy = resolveWorkflowPolicy(workflow);
   const configured = workflow?.prePrReview;
   const configuredSkills = normalizeSkillList(configured?.skills);
+  const configuredDecisionEnum = normalizeDecisionEnumList(
+    configured?.decisionEnum
+  );
   const configuredEnabled = typeof configured?.enabled === "boolean" ? configured.enabled : workflowPolicy.requirePr;
   return {
     enabled: workflowPolicy.requirePr ? configuredEnabled : false,
     skills: configuredSkills.length > 0 ? configuredSkills : DEFAULT_PRE_PR_REVIEW_SKILLS,
     fallback: configured?.fallback === "builtin-checklist" ? configured.fallback : "builtin-checklist",
-    blockOnFindings: typeof configured?.blockOnFindings === "boolean" ? configured.blockOnFindings : true,
-    minorPolicy: configured?.minorPolicy === "block" ? "block" : "warn"
+    evidenceMode: configured?.evidenceMode === "any" ? "any" : "path_required",
+    findings: configured?.findings === "optional" ? "optional" : "required",
+    decisionEnum: configuredDecisionEnum.length > 0 ? configuredDecisionEnum : DEFAULT_PRE_PR_DECISION_ENUM
   };
 }
 
@@ -2977,17 +3023,20 @@ function isPrePrReviewSatisfied(feature, prePrReviewPolicy) {
   if (!feature.docs.prePrEvidenceFieldExists || !feature.prePrReview.evidenceProvided) {
     return false;
   }
+  if (prePrReviewPolicy.findings === "required" && (!feature.docs.prePrFindingsFieldExists || !feature.prePrReview.findingsProvided)) {
+    return false;
+  }
   if (!feature.docs.prePrDecisionFieldExists || !feature.prePrReview.decisionProvided) {
     return false;
   }
+  if (feature.prePrReview.decisionOutcome !== "approve") {
+    return false;
+  }
   return true;
 }
 function isFeatureDone(feature, workflowPolicy, prePrReviewPolicy) {
   return feature.specStatus === "Approved" && feature.planStatus === "Approved" && !feature.git.docsHasUncommittedChanges && !feature.git.projectHasUncommittedChanges && feature.docs.tasksExists && feature.tasks.total > 0 && feature.tasks.total === feature.tasks.done && isCompletionChecklistDone(feature) && isTasksDocApproved(feature) && (!workflowPolicy.requireIssue || !!feature.issueNumber) && (!workflowPolicy.requirePr || isPrMetadataConfigured(feature) && !!feature.pr.link) && (!workflowPolicy.requireReview || feature.pr.status === "Approved") && isPrePrReviewSatisfied(feature, prePrReviewPolicy);
 }
-function formatSkillList(skills) {
-  return skills.join(", ");
-}
 function getPrReviewRemoteBlockReasons(feature, lang) {
   const remote = feature.pr.remote;
   if (!remote || !remote.available) return [];
@@ -3024,6 +3073,14 @@ function getPrReviewRemoteBlockReasons(feature, lang) {
 function normalizeCommitTopicText(value) {
   return value.replace(/\s+/g, " ").trim();
 }
+function toShellArg(value) {
+  return `"${value.replace(/\\/g, "\\\\").replace(/"/g, '\\"').replace(/\$/g, "\\$").replace(/`/g, "\\`")}"`;
+}
+function buildSelfCliCommand(args) {
+  const entry = process.argv[1] || "dist/index.js";
+  const base = [process.execPath, entry, "--no-banner", ...args];
+  return base.map((arg) => toShellArg(arg)).join(" ");
+}
 function toShellSafeCommitTopic(value) {
   return value.replace(/\\/g, "\\\\").replace(/"/g, '\\"').replace(/\$/g, "\\$").replace(/`/g, "\\`");
 }
@@ -3792,74 +3849,19 @@ ${tr(lang, "messages", "taskCommitGateWarnProceed", {
               }
             ];
           }
-          if (f.prePrReview.status !== "Done") {
-            if (!prePrReviewPolicy.skills.length) {
-              return [
-                {
-                  type: "instruction",
-                  category: "pre_pr_review",
-                  requiresUserCheck: true,
-                  message: tr(lang, "messages", "prePrReviewRun", {
-                    skills: "code-review-excellence",
-                    fallback: prePrReviewPolicy.fallback
-                  })
-                }
-              ];
-            }
-            return [
-              {
-                type: "instruction",
-                category: "pre_pr_review",
-                requiresUserCheck: true,
-                message: tr(lang, "messages", "prePrReviewRun", {
-                  skills: formatSkillList(prePrReviewPolicy.skills),
-                  fallback: prePrReviewPolicy.fallback
-                })
-              }
-            ];
-          }
-          if (!f.docs.prePrEvidenceFieldExists || !f.prePrReview.evidenceProvided) {
-            return [
-              {
-                type: "instruction",
-                category: "pre_pr_review",
-                requiresUserCheck: true,
-                message: tr(lang, "messages", "prePrReviewEvidenceMissing")
-              }
-            ];
-          }
-          if (!f.docs.prePrDecisionFieldExists || !f.prePrReview.decisionProvided) {
-            return [
-              {
-                type: "instruction",
-                category: "pre_pr_review",
-                requiresUserCheck: true,
-                message: tr(lang, "messages", "prePrReviewDecisionMissing")
-              }
-            ];
-          }
-          if (!prePrReviewPolicy.skills.length) {
-            return [
-              {
-                type: "instruction",
-                category: "pre_pr_review",
-                requiresUserCheck: true,
-                message: tr(lang, "messages", "prePrReviewRun", {
-                  skills: "code-review-excellence",
-                  fallback: prePrReviewPolicy.fallback
-                })
-              }
-            ];
+          const commandArgs = ["pre-pr-review", f.folderName];
+          if (f.type && f.type !== "single") {
+            commandArgs.push("--component", f.type);
           }
           return [
             {
-              type: "instruction",
+              type: "command",
               category: "pre_pr_review",
+              operationType: "local",
               requiresUserCheck: true,
-              message: tr(lang, "messages", "prePrReviewRun", {
-                skills: formatSkillList(prePrReviewPolicy.skills),
-                fallback: prePrReviewPolicy.fallback
-              })
+              scope: "docs",
+              cwd: f.git.docsGitCwd,
+              cmd: buildSelfCliCommand(commandArgs)
             }
           ];
         }
@@ -4556,6 +4558,52 @@ function hasStructuredReviewDecision(value) {
   if (!trimmed) return false;
   return /^(?:decision|결정)\s*[:：]\s*\S.+$/i.test(trimmed);
 }
+function parsePrePrDecisionOutcome(value) {
+  if (!value) return void 0;
+  const trimmed = value.trim();
+  if (!trimmed) return void 0;
+  const structured = trimmed.match(/^(?:decision|결정)\s*[:：]\s*(.+)$/i);
+  if (!structured) return void 0;
+  const payload = structured[1].trim().toLowerCase();
+  const normalized = payload.split(/[,\s-]+/)[0]?.replace(/[^a-z_]/g, "");
+  if (!normalized) return void 0;
+  if (normalized === "approve" || normalized === "approved") {
+    return "approve";
+  }
+  if (normalized === "changes_requested" || normalized === "changes" || normalized === "change") {
+    return "changes_requested";
+  }
+  if (normalized === "blocked" || normalized === "block") {
+    return "blocked";
+  }
+  return void 0;
+}
+function resolveEvidencePathValue(value) {
+  const trimmed = value.trim();
+  const mdLink = trimmed.match(/\(([^)]+)\)/);
+  if (mdLink && mdLink[1]) return mdLink[1].trim();
+  return trimmed.split(/\s+/)[0] || "";
+}
+async function isPrePrEvidenceProvided(rawValue, policy, context) {
+  if (isPlaceholderReviewEvidence(rawValue)) return false;
+  if (policy.evidenceMode !== "path_required") return true;
+  if (!rawValue) return false;
+  const evidencePath = resolveEvidencePathValue(rawValue);
+  if (!evidencePath) return false;
+  if (/^https?:\/\//i.test(evidencePath)) return false;
+  const candidates = /* @__PURE__ */ new Set();
+  if (path22.isAbsolute(evidencePath)) {
+    candidates.add(path22.resolve(evidencePath));
+  } else {
+    candidates.add(path22.resolve(context.featurePath, evidencePath));
+    candidates.add(path22.resolve(context.docsDir, evidencePath));
+    candidates.add(path22.resolve(path22.dirname(context.docsDir), evidencePath));
+  }
+  for (const candidate of candidates) {
+    if (await fs16.pathExists(candidate)) return true;
+  }
+  return false;
+}
 function parseIssueNumber(value) {
   if (!value) return void 0;
   const match = value.match(/#?(\d+)/);
@@ -4848,9 +4896,15 @@ function isPrePrReviewSatisfied2(feature, policy) {
   if (!feature.docs.prePrEvidenceFieldExists || !feature.prePrReview.evidenceProvided) {
     return false;
   }
+  if (policy.findings === "required" && (!feature.docs.prePrFindingsFieldExists || !feature.prePrReview.findingsProvided)) {
+    return false;
+  }
   if (!feature.docs.prePrDecisionFieldExists || !feature.prePrReview.decisionProvided) {
     return false;
   }
+  if (feature.prePrReview.decisionOutcome !== "approve") {
+    return false;
+  }
   return true;
 }
 async function parseFeature(featurePath, type, context, options) {
@@ -4917,9 +4971,11 @@ async function parseFeature(featurePath, type, context, options) {
   let completionChecklist;
   let prePrReviewStatus;
   let prePrFindings;
+  let prePrFindingsProvided = false;
   let prePrEvidence;
   let prePrEvidenceProvided = false;
   let prePrDecision;
+  let prePrDecisionOutcome;
   let prePrDecisionProvided = false;
   let prReviewFindings;
   let prReviewEvidence;
@@ -4994,6 +5050,7 @@ async function parseFeature(featurePath, type, context, options) {
       "Pre-PR Findings"
     ]);
     prePrFindings = parseReviewFindings(prePrFindingsValue);
+    prePrFindingsProvided = !!prePrFindings;
     const prePrEvidenceValue = extractFirstSpecValue(content, [
       "PR \uC804 \uB9AC\uBDF0 Evidence",
       "Pre-PR Evidence"
@@ -5003,7 +5060,11 @@ async function parseFeature(featurePath, type, context, options) {
       "Pre-PR Evidence"
     ]);
     prePrEvidence = prePrEvidenceValue?.trim();
-    prePrEvidenceProvided = !isPlaceholderReviewEvidence(prePrEvidenceValue);
+    prePrEvidenceProvided = await isPrePrEvidenceProvided(
+      prePrEvidenceValue,
+      prePrReviewPolicy,
+      { featurePath, docsDir: context.docsDir }
+    );
     const prePrDecisionValue = extractFirstSpecValue(content, [
       "PR \uC804 \uB9AC\uBDF0 Decision",
       "Pre-PR Decision"
@@ -5013,7 +5074,8 @@ async function parseFeature(featurePath, type, context, options) {
       "Pre-PR Decision"
     ]);
     prePrDecision = prePrDecisionValue?.trim();
-    prePrDecisionProvided = !isPlaceholderReviewEvidence(prePrDecisionValue) && hasStructuredReviewDecision(prePrDecisionValue);
+    prePrDecisionOutcome = parsePrePrDecisionOutcome(prePrDecisionValue);
+    prePrDecisionProvided = !isPlaceholderReviewEvidence(prePrDecisionValue) && hasStructuredReviewDecision(prePrDecisionValue) && !!prePrDecisionOutcome && prePrReviewPolicy.decisionEnum.includes(prePrDecisionOutcome);
     const prReviewFindingsValue = extractFirstSpecValue(content, [
       "PR \uB9AC\uBDF0 Findings",
       "PR Review Findings"
@@ -5188,6 +5250,9 @@ async function parseFeature(featurePath, type, context, options) {
   if (tasksExists && prePrReviewPolicy.enabled && !prePrReviewFieldExists) {
     warnings.push(tr(lang, "warnings", "legacyTasksPrePrReviewField"));
   }
+  if (tasksExists && prePrReviewPolicy.enabled && prePrReviewPolicy.findings === "required" && !prePrFindingsFieldExists) {
+    warnings.push(tr(lang, "warnings", "legacyTasksPrePrFindingsField"));
+  }
   if (tasksExists && prePrReviewPolicy.enabled && !prePrEvidenceFieldExists) {
     warnings.push(tr(lang, "warnings", "legacyTasksPrePrEvidenceField"));
   }
@@ -5232,12 +5297,15 @@ async function parseFeature(featurePath, type, context, options) {
     {
       docs: {
         prePrReviewFieldExists,
+        prePrFindingsFieldExists,
         prePrEvidenceFieldExists,
         prePrDecisionFieldExists
       },
       prePrReview: {
         status: prePrReviewStatus,
+        findingsProvided: prePrFindingsProvided,
         evidenceProvided: prePrEvidenceProvided,
+        decisionOutcome: prePrDecisionOutcome,
         decisionProvided: prePrDecisionProvided
       }
     },
@@ -5278,10 +5346,18 @@ async function parseFeature(featurePath, type, context, options) {
         warnings.push(tr(lang, "warnings", "workflowPrePrReviewMissing"));
       } else if (prePrReviewStatus !== "Done") {
         warnings.push(tr(lang, "warnings", "workflowPrePrReviewNotDone"));
+      } else if (prePrReviewPolicy.findings === "required" && (!prePrFindingsFieldExists || !prePrFindingsProvided)) {
+        warnings.push(tr(lang, "warnings", "workflowPrePrFindingsMissing"));
       } else if (!prePrEvidenceFieldExists || !prePrEvidenceProvided) {
         warnings.push(tr(lang, "warnings", "workflowPrePrEvidenceMissing"));
       } else if (!prePrDecisionFieldExists || !prePrDecisionProvided) {
         warnings.push(tr(lang, "warnings", "workflowPrePrDecisionMissing"));
+      } else if (prePrDecisionOutcome !== "approve") {
+        warnings.push(
+          tr(lang, "warnings", "workflowPrePrDecisionNotApproved", {
+            outcome: prePrDecisionOutcome || "-"
+          })
+        );
       }
     }
   }
@@ -5307,9 +5383,11 @@ async function parseFeature(featurePath, type, context, options) {
     prePrReview: {
       status: prePrReviewStatus,
       findings: prePrFindings,
+      findingsProvided: prePrFindingsProvided,
       evidence: prePrEvidence,
       evidenceProvided: prePrEvidenceProvided,
       decision: prePrDecision,
+      decisionOutcome: prePrDecisionOutcome,
       decisionProvided: prePrDecisionProvided
     },
     prReview: {
@@ -5867,6 +5945,18 @@ function normalizeSkillList2(raw) {
   }
   return [...deduped];
 }
+function normalizeDecisionEnumList2(raw) {
+  if (!Array.isArray(raw)) return [];
+  const deduped = /* @__PURE__ */ new Set();
+  for (const item of raw) {
+    const value = String(item || "").trim().toLowerCase();
+    if (!value) continue;
+    if (value === "approve" || value === "changes_requested" || value === "blocked") {
+      deduped.add(value);
+    }
+  }
+  return [...deduped];
+}
 async function backfillMissingConfigDefaults(docsDir) {
   const configPath = path22.join(docsDir, ".lee-spec-kit.json");
   if (!await fs16.pathExists(configPath)) {
@@ -5915,8 +6005,39 @@ async function backfillMissingConfigDefaults(docsDir) {
     }
   }
   setIfMissing(prePrReview, "fallback", "builtin-checklist", "workflow.prePrReview.fallback");
-  setIfMissing(prePrReview, "blockOnFindings", true, "workflow.prePrReview.blockOnFindings");
-  setIfMissing(prePrReview, "minorPolicy", "warn", "workflow.prePrReview.minorPolicy");
+  setIfMissing(
+    prePrReview,
+    "evidenceMode",
+    "path_required",
+    "workflow.prePrReview.evidenceMode"
+  );
+  if (prePrReview.evidenceMode !== void 0 && prePrReview.evidenceMode !== "path_required" && prePrReview.evidenceMode !== "any") {
+    prePrReview.evidenceMode = "path_required";
+    changedPaths.push("workflow.prePrReview.evidenceMode");
+  }
+  setIfMissing(
+    prePrReview,
+    "findings",
+    "required",
+    "workflow.prePrReview.findings"
+  );
+  if (prePrReview.findings !== void 0 && prePrReview.findings !== "required" && prePrReview.findings !== "optional") {
+    prePrReview.findings = "required";
+    changedPaths.push("workflow.prePrReview.findings");
+  }
+  if (prePrReview.decisionEnum === void 0) {
+    prePrReview.decisionEnum = ["approve", "changes_requested", "blocked"];
+    changedPaths.push("workflow.prePrReview.decisionEnum");
+  } else {
+    const normalizedDecisionEnum = normalizeDecisionEnumList2(prePrReview.decisionEnum);
+    if (normalizedDecisionEnum.length === 0) {
+      prePrReview.decisionEnum = ["approve", "changes_requested", "blocked"];
+      changedPaths.push("workflow.prePrReview.decisionEnum");
+    } else if (JSON.stringify(normalizedDecisionEnum) !== JSON.stringify(prePrReview.decisionEnum)) {
+      prePrReview.decisionEnum = normalizedDecisionEnum;
+      changedPaths.push("workflow.prePrReview.decisionEnum");
+    }
+  }
   if (!isPlainObject(raw.pr)) {
     raw.pr = {};
     changedPaths.push("pr");
@@ -6312,21 +6433,27 @@ function buildActionDetail(action, lang) {
     const branchMatch = command.match(/\bfeat\/([A-Za-z0-9._-]+)/);
     const worktree = worktreeMatch ? `.worktrees/${worktreeMatch[1]}` : null;
     const branch = branchMatch ? `feat/${branchMatch[1]}` : null;
+    if (action.type !== "command") {
+      return tr(lang, "cli", "context.commandDetail.branchCreateGeneric", {
+        scope: "project"
+      });
+    }
+    const scope = action.scope;
     if (worktree && branch) {
       return tr(lang, "cli", "context.commandDetail.branchCreateWithWorktree", {
-        scope: action.scope,
+        scope,
         worktree,
         branch
       });
     }
     if (branch) {
       return tr(lang, "cli", "context.commandDetail.branchCreateWithBranch", {
-        scope: action.scope,
+        scope,
         branch
       });
     }
     return tr(lang, "cli", "context.commandDetail.branchCreateGeneric", {
-      scope: action.scope
+      scope
     });
   };
   const extractCommitMessage = (command) => {
@@ -7182,12 +7309,18 @@ function getListLabel(f, stepsMap, lang, workflowPolicy, prePrReviewPolicy) {
     if (prePrReviewPolicy.enabled && f.prePrReview.status !== "Done") {
       return tr(lang, "cli", "context.list.completePrePrReview");
     }
+    if (prePrReviewPolicy.enabled && prePrReviewPolicy.findings === "required" && (!f.docs.prePrFindingsFieldExists || !f.prePrReview.findingsProvided)) {
+      return tr(lang, "cli", "context.list.addPrePrFindings");
+    }
     if (prePrReviewPolicy.enabled && (!f.docs.prePrEvidenceFieldExists || !f.prePrReview.evidenceProvided)) {
       return tr(lang, "cli", "context.list.addPrePrEvidence");
     }
     if (prePrReviewPolicy.enabled && (!f.docs.prePrDecisionFieldExists || !f.prePrReview.decisionProvided)) {
       return tr(lang, "cli", "context.list.addPrePrDecision");
     }
+    if (prePrReviewPolicy.enabled && f.prePrReview.decisionOutcome !== "approve") {
+      return tr(lang, "cli", "context.list.resolvePrePrDecision");
+    }
     if (workflowPolicy.requirePr && !f.pr.link) {
       return tr(lang, "cli", "context.list.recordPrLink");
     }
@@ -7244,7 +7377,9 @@ function toCompactFeature(feature) {
     prePrReview: {
       status: feature.prePrReview.status,
       findings: feature.prePrReview.findings,
+      findingsProvided: feature.prePrReview.findingsProvided,
       evidenceProvided: feature.prePrReview.evidenceProvided,
+      decisionOutcome: feature.prePrReview.decisionOutcome,
       decisionProvided: feature.prePrReview.decisionProvided
     },
     prReview: {
@@ -7399,7 +7534,18 @@ async function runContext(featureName, options) {
   const jsonMode = !!options.json || !!options.jsonCompact;
   if (jsonMode) {
     const primaryAction = state.actionOptions[0] ?? null;
-    const finalApprovalPrompt = buildFinalApprovalPrompt(lang, state.actionOptions);
+    const checkRequiredLabels = state.actionOptions.filter((option) => !!option.action.requiresUserCheck).map((option) => option.label);
+    const checkRequiredCategories = [
+      ...new Set(
+        state.actionOptions.filter((option) => !!option.action.requiresUserCheck).map((option) => option.action.category || "uncategorized")
+      )
+    ];
+    const approvalRequired = checkRequiredLabels.length > 0;
+    const finalApprovalPrompt = approvalRequired ? buildFinalApprovalPrompt(lang, state.actionOptions) : "";
+    const approvalUserFacingLines = approvalRequired ? [
+      ...state.actionOptions.map((o) => o.approvalPrompt),
+      finalApprovalPrompt
+    ].filter((line) => line.length > 0) : [];
     const approveCommand = buildApprovalCommand(
       state,
       featureName,
@@ -7452,19 +7598,20 @@ async function runContext(featureName, options) {
           activeCategories,
           knownCategories: ACTION_CATEGORIES,
           uncategorizedLabels,
+          checkRequiredLabels,
+          checkRequiredCategories,
+          approvalRequired,
           categoryPolicyGuidance: 'For approval.mode="category", match against `actionOptions[].category`.',
-          oneApprovalPerAction: true,
+          oneApprovalPerAction: approvalRequired,
           requireFreshContext: true,
           contextVersion: state.contextVersion,
           config: config.approval ?? { mode: "builtin" }
         },
         approvalRequest: {
+          required: approvalRequired,
           finalPrompt: finalApprovalPrompt,
-          userFacingLines: [
-            ...state.actionOptions.map((o) => o.approvalPrompt),
-            finalApprovalPrompt
-          ].filter((line) => line.length > 0),
-          labels: state.actionOptions.map((o) => o.label),
+          userFacingLines: approvalUserFacingLines,
+          labels: approvalRequired ? state.actionOptions.map((o) => o.label) : [],
           approveCommand,
           executeCommand,
           executeRequiresTicket: !!state.actionOptions[0]?.action?.requiresUserCheck
@@ -7523,27 +7670,28 @@ async function runContext(featureName, options) {
         activeCategories,
         knownCategories: ACTION_CATEGORIES,
         uncategorizedLabels,
+        checkRequiredLabels,
+        checkRequiredCategories,
+        approvalRequired,
         categoryPolicyGuidance: 'For approval.mode="category", match against `actionOptions[].category`.',
-        requireExplanationBeforeApproval: true,
-        requiredExplanationFields: [
+        requireExplanationBeforeApproval: approvalRequired,
+        requiredExplanationFields: approvalRequired ? [
           "actionOptions[].label",
           "actionOptions[].detail",
           "actionOptions[].approvalPrompt"
-        ],
+        ] : [],
         recommendation: "Before asking for approval, show only `actionOptions[].approvalPrompt` lines and `approvalRequest.finalPrompt` to the user. Keep `requiredDocs`, `checkPolicy`, and raw execution commands as internal guidance. For commit actions, include scope (`docs`/`project`) and commit message in the visible prompt. User replies should include the label token (e.g. `A`, `A OK`, `A proceed`, `A \uC9C4\uD589\uD574`). For command execution, prefer one-shot `npx lee-spec-kit flow <featureRef> --approve <LABEL> --execute` to avoid session mismatch after context compression/reset. Use ticket-based `context --execute --ticket` only when explicitly needed.",
-        oneApprovalPerAction: true,
+        oneApprovalPerAction: approvalRequired,
         requireFreshContext: true,
         contextVersion: state.contextVersion,
         config: config.approval ?? { mode: "builtin" }
       },
       approvalRequest: {
         guidance: "User-facing output must include only approval prompts (`A: ...`) and `finalPrompt`. Do not expose `requiredDocs`, `checkPolicy`, or raw `cmd` unless explicitly requested. For approved command actions, prefer one-shot `flow --approve <LABEL> --execute`.",
+        required: approvalRequired,
         finalPrompt: finalApprovalPrompt,
-        userFacingLines: [
-          ...state.actionOptions.map((o) => o.approvalPrompt),
-          finalApprovalPrompt
-        ].filter((line) => line.length > 0),
-        labels: state.actionOptions.map((o) => o.label),
+        userFacingLines: approvalUserFacingLines,
+        labels: approvalRequired ? state.actionOptions.map((o) => o.label) : [],
         approveCommand,
         executeCommand,
         executeRequiresTicket: !!state.actionOptions[0]?.action?.requiresUserCheck,
@@ -7823,7 +7971,7 @@ async function runContext(featureName, options) {
       );
     }
   }
-  if (actionOptions.length > 0) {
+  if (actionOptions.length > 0 && hasCheckAction) {
     const finalApprovalPrompt = buildFinalApprovalPrompt(lang, actionOptions);
     const approveCommand = buildApprovalCommand(
       state,
@@ -7950,7 +8098,7 @@ async function runApprovedOption(state, config, lang, featureName, selectionOpti
             contextVersion: freshState.contextVersion,
             executable: selectedAction.type === "command",
             executeRequiresTicket,
-            oneApprovalPerAction: true,
+            oneApprovalPerAction: executeRequiresTicket,
             approvalTicket: ticket ? {
               token: ticket.token,
               sessionId: ticket.sessionId,
@@ -9014,6 +9162,9 @@ function toAutoReasonCode(status) {
       return "AUTO_EXECUTION_FAILED";
   }
 }
+function isAutoRunFailureStatus(status) {
+  return status === "manual_required" || status === "selection_required" || status === "no_progress" || status === "request_label_missing" || status === "request_failed" || status === "execution_failed";
+}
 async function runAutoUntilCategory(config, featureName, selectionOptions, untilCategories, requestText, metadata) {
   const contextArgs = ["context", ...buildSelectionArgs(featureName, selectionOptions)];
   const gateSet = new Set(untilCategories);
@@ -9422,9 +9573,10 @@ async function runFlow(featureName, options) {
     }
   }
   if (options.json) {
+    const autoRunFailed = !!(autoRun && isAutoRunFailureStatus(autoRun.status));
     const payload = {
-      status: "ok",
-      reasonCode: "FLOW_SUMMARY",
+      status: autoRunFailed ? "error" : "ok",
+      reasonCode: autoRunFailed ? autoRun?.reasonCode || "AUTO_EXECUTION_FAILED" : "FLOW_SUMMARY",
       context: {
         before: {
           status: before.status,
@@ -9453,6 +9605,9 @@ async function runFlow(featureName, options) {
       suggestion: after.matchedFeature ? `npx lee-spec-kit context ${after.matchedFeature.folderName}${componentHint}` : `npx lee-spec-kit context${componentHint}`
     };
     console.log(JSON.stringify(payload, null, 2));
+    if (autoRunFailed) {
+      process.exitCode = 1;
+    }
     return;
   }
   console.log();
@@ -9498,6 +9653,9 @@ async function runFlow(featureName, options) {
     console.log(chalk6.gray("Auto gate reached. Reply with one of the labels shown above (example: A OK)."));
     console.log();
   }
+  if (autoRun && isAutoRunFailureStatus(autoRun.status)) {
+    process.exitCode = 1;
+  }
   if (after.matchedFeature) {
     console.log(
       chalk6.blue(
@@ -11986,6 +12144,263 @@ async function runOnboard(options) {
     process.exitCode = 1;
   }
 }
+function escapeRegExp4(value) {
+  return value.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+}
+function normalizeDecision(raw) {
+  if (!raw) return null;
+  const value = raw.trim().toLowerCase().replace(/[\s-]+/g, "_");
+  if (value === "approve" || value === "approved") return "approve";
+  if (value === "changes_requested" || value === "change_requested" || value === "changes") {
+    return "changes_requested";
+  }
+  if (value === "blocked" || value === "block") return "blocked";
+  return null;
+}
+function parseNonNegativeInt(raw, label) {
+  if (raw === void 0) return null;
+  const value = Number(raw);
+  if (!Number.isInteger(value) || value < 0) {
+    throw createCliError(
+      "INVALID_ARGUMENT",
+      `\`${label}\` must be a non-negative integer.`
+    );
+  }
+  return value;
+}
+function findSpecLineIndex(lines, keys) {
+  const escaped = keys.map((key) => escapeRegExp4(key));
+  const re = new RegExp(`^\\s*-\\s*\\*\\*(?:${escaped.join("|")})\\*\\*\\s*:\\s*`);
+  return lines.findIndex((line) => re.test(line));
+}
+function replaceSpecLine(line, keys, preferredKey, value) {
+  const escaped = keys.map((key) => escapeRegExp4(key));
+  const re = new RegExp(
+    `^(\\s*-\\s*\\*\\*)(?:${escaped.join("|")})(\\*\\*\\s*:\\s*)(.*)$`
+  );
+  if (!re.test(line)) return line;
+  return line.replace(re, `$1${preferredKey}$2${value}`);
+}
+function computeInsertIndex(lines, anchorKeys) {
+  const anchorIndex = findSpecLineIndex(lines, anchorKeys);
+  if (anchorIndex !== -1) {
+    let cursor = anchorIndex + 1;
+    while (cursor < lines.length && /^\s{2,}-\s+/.test(lines[cursor])) {
+      cursor += 1;
+    }
+    return cursor;
+  }
+  const sectionIndex = lines.findIndex(
+    (line) => /^\s*##\s+(Task List|태스크 목록)\s*$/.test(line)
+  );
+  if (sectionIndex !== -1) return sectionIndex;
+  return lines.length;
+}
+function upsertSpecLine(content, keys, preferredKey, value, anchorKeys) {
+  const lines = content.split("\n");
+  const index = findSpecLineIndex(lines, keys);
+  if (index !== -1) {
+    lines[index] = replaceSpecLine(lines[index], keys, preferredKey, value);
+    return lines.join("\n");
+  }
+  const insertAt = computeInsertIndex(lines, anchorKeys);
+  lines.splice(insertAt, 0, `- **${preferredKey}**: ${value}`);
+  return lines.join("\n");
+}
+function normalizePathForDoc(value) {
+  return value.replace(/\\/g, "/");
+}
+function getPreferredKeys(lang) {
+  if (lang === "ko") {
+    return {
+      review: "PR \uC804 \uB9AC\uBDF0",
+      findings: "PR \uC804 \uB9AC\uBDF0 Findings",
+      evidence: "PR \uC804 \uB9AC\uBDF0 Evidence",
+      decision: "PR \uC804 \uB9AC\uBDF0 Decision",
+      prStatus: "PR \uC0C1\uD0DC"
+    };
+  }
+  return {
+    review: "Pre-PR Review",
+    findings: "Pre-PR Findings",
+    evidence: "Pre-PR Evidence",
+    decision: "Pre-PR Decision",
+    prStatus: "PR Status"
+  };
+}
+function buildReportContent(input) {
+  const skills = input.skills.length > 0 ? input.skills.join(", ") : "code-review-excellence";
+  return `# Pre-PR Review Report: ${input.folderName}
+
+- Date: ${input.date}
+- Baseline: ${input.fallback}
+- Skills: ${skills}
+- Findings: major=${input.major}, minor=${input.minor}
+- Decision: ${input.decision}
+- Note: ${input.note}
+
+## Baseline Checklist
+
+- [x] Checked alignment with spec/plan/tasks.
+- [x] Reviewed risk areas (regression, security, side effects, release readiness).
+- [x] Reviewed maintainability (structure, reuse, obsolete code cleanup).
+- [x] Reviewed test/verification coverage (or recorded reason if not run).
+`;
+}
+function prePrReviewCommand(program2) {
+  program2.command("pre-pr-review [feature-name]").description("Run and record pre-PR review evidence for a feature").option("--component <component>", "Component name for multi projects").option(
+    "--decision <outcome>",
+    "Decision outcome: approve | changes_requested | blocked"
+  ).option("--major <count>", "Pre-PR major findings count").option("--minor <count>", "Pre-PR minor findings count").option("--note <text>", "Decision note text").option("--json", "Output JSON").action(async (featureName, options) => {
+    try {
+      await runPrePrReview(featureName, options);
+    } catch (error) {
+      const config = await getConfig(process.cwd());
+      const lang = config?.lang ?? DEFAULT_LANG;
+      const cliError = toCliError(error);
+      const suggestions = getCliErrorSuggestions(cliError.code, lang);
+      if (options.json) {
+        console.log(
+          JSON.stringify({
+            status: "error",
+            reasonCode: cliError.code,
+            error: cliError.message,
+            suggestions
+          })
+        );
+      } else {
+        console.error(chalk6.red(`[${cliError.code}] ${cliError.message}`));
+        printCliErrorSuggestions(suggestions, lang);
+      }
+      process.exitCode = 1;
+    }
+  });
+}
+async function runPrePrReview(featureName, options) {
+  const config = await getConfig(process.cwd());
+  if (!config) {
+    throw createCliError(
+      "CONFIG_NOT_FOUND",
+      "No lee-spec-kit config found in this workspace."
+    );
+  }
+  const selectionOptions = {
+    component: resolveComponentOption(options.component)
+  };
+  const state = await resolveContextSelection(config, featureName, selectionOptions);
+  if (state.status !== "single_matched" || !state.matchedFeature) {
+    throw createCliError(
+      "CONTEXT_SELECTION_REQUIRED",
+      "pre-pr-review requires a single matched feature. Pass <feature-name> explicitly."
+    );
+  }
+  const feature = state.matchedFeature;
+  if (!feature.docs.tasksExists) {
+    throw createCliError(
+      "PRECONDITION_FAILED",
+      `tasks.md not found for feature: ${feature.folderName}`
+    );
+  }
+  const tasksPath = path22.join(feature.path, "tasks.md");
+  const tasksContent = await fs16.readFile(tasksPath, "utf-8");
+  const policy = resolvePrePrReviewPolicy(config.workflow);
+  const preferred = getPreferredKeys(config.lang);
+  const date = getLocalDateString();
+  const major = parseNonNegativeInt(options.major, "--major") ?? feature.prePrReview.findings?.major ?? 0;
+  const minor = parseNonNegativeInt(options.minor, "--minor") ?? feature.prePrReview.findings?.minor ?? 0;
+  const explicitDecision = normalizeDecision(options.decision);
+  if (options.decision && !explicitDecision) {
+    throw createCliError(
+      "INVALID_ARGUMENT",
+      "`--decision` must be one of: approve, changes_requested, blocked."
+    );
+  }
+  const inferredDecision = major > 0 ? "changes_requested" : "approve";
+  const decision = explicitDecision || inferredDecision;
+  if (!policy.decisionEnum.includes(decision)) {
+    throw createCliError(
+      "INVALID_ARGUMENT",
+      `Decision "${decision}" is not allowed by workflow.prePrReview.decisionEnum.`
+    );
+  }
+  const note = options.note?.trim() || (decision === "approve" ? "baseline review completed without blocking findings" : decision === "changes_requested" ? "resolve findings before PR creation" : "blocked until prerequisite risk is resolved");
+  const reportPath = path22.join(feature.path, "pre-pr-review.md");
+  const reportContent = buildReportContent({
+    folderName: feature.folderName,
+    date,
+    decision,
+    major,
+    minor,
+    note,
+    fallback: policy.fallback,
+    skills: policy.skills
+  });
+  await fs16.writeFile(reportPath, reportContent, "utf-8");
+  const reportPathFromDocs = normalizePathForDoc(
+    path22.join(feature.docs.featurePathFromDocs, "pre-pr-review.md")
+  );
+  const evidencePath = path22.basename(config.docsDir) === "docs" ? normalizePathForDoc(path22.join("docs", reportPathFromDocs)) : reportPathFromDocs;
+  let nextTasks = tasksContent;
+  nextTasks = upsertSpecLine(
+    nextTasks,
+    ["PR \uC804 \uB9AC\uBDF0", "Pre-PR Review"],
+    preferred.review,
+    "Done",
+    ["PR \uC0C1\uD0DC", "PR Status"]
+  );
+  nextTasks = upsertSpecLine(
+    nextTasks,
+    ["PR \uC804 \uB9AC\uBDF0 Findings", "Pre-PR Findings"],
+    preferred.findings,
+    `major=${major}, minor=${minor}`,
+    ["PR \uC804 \uB9AC\uBDF0", "Pre-PR Review"]
+  );
+  nextTasks = upsertSpecLine(
+    nextTasks,
+    ["PR \uC804 \uB9AC\uBDF0 Evidence", "Pre-PR Evidence"],
+    preferred.evidence,
+    evidencePath,
+    ["PR \uC804 \uB9AC\uBDF0 Findings", "Pre-PR Findings", "PR \uC804 \uB9AC\uBDF0", "Pre-PR Review"]
+  );
+  nextTasks = upsertSpecLine(
+    nextTasks,
+    ["PR \uC804 \uB9AC\uBDF0 Decision", "Pre-PR Decision"],
+    preferred.decision,
+    `decision: ${decision} - ${note}`,
+    ["PR \uC804 \uB9AC\uBDF0 Evidence", "Pre-PR Evidence"]
+  );
+  if (nextTasks !== tasksContent) {
+    await fs16.writeFile(tasksPath, nextTasks, "utf-8");
+  }
+  if (options.json) {
+    console.log(
+      JSON.stringify(
+        {
+          status: "ok",
+          reasonCode: "PRE_PR_REVIEW_RECORDED",
+          feature: feature.folderName,
+          reportPath: normalizePathForDoc(reportPath),
+          evidencePath,
+          decision,
+          findings: { major, minor },
+          tasksUpdated: nextTasks !== tasksContent
+        },
+        null,
+        2
+      )
+    );
+    return;
+  }
+  console.log();
+  console.log(chalk6.green(`\u2705 pre-pr-review completed: ${feature.folderName}`));
+  console.log(chalk6.gray(`- Decision: ${decision}`));
+  console.log(chalk6.gray(`- Findings: major=${major}, minor=${minor}`));
+  console.log(chalk6.gray(`- Report: ${reportPath}`));
+  if (nextTasks !== tasksContent) {
+    console.log(chalk6.gray(`- tasks.md updated: ${tasksPath}`));
+  }
+  console.log();
+}
 function isBannerDisabled() {
   const v = (process.env.LEE_SPEC_KIT_NO_BANNER || "").trim();
   return v === "1";
@@ -12166,6 +12581,7 @@ githubCommand(program);
 docsCommand(program);
 detectCommand(program);
 onboardCommand(program);
+prePrReviewCommand(program);
 await program.parseAsync();
 //# sourceMappingURL=index.js.map
 //# sourceMappingURL=index.js.map