lee-spec-kit 0.6.35 → 0.6.37

package/dist/index.js

@@ -597,6 +597,8 @@ var koMessages = {
   projectCommitUpdate: 'cd "{projectGitCwd}" && (git diff --cached --quiet && echo "\uC2A4\uD14C\uC774\uC9D5\uB41C \uD30C\uC77C\uC774 \uC5C6\uC2B5\uB2C8\uB2E4. \uC774\uBC88 \uD0DC\uC2A4\uD06C\uC5D0\uC11C \uC218\uC815\uD55C \uD30C\uC77C\uB9CC \uC120\uD0DD\uD574 git add [files] \uD6C4 \uB2E4\uC2DC \uC2E4\uD589\uD558\uC138\uC694." && exit 1 || git commit -m "feat({folderName}): {commitTopic}")',
   reviewFixCommitIssueGuidance: "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(#{issueNumber}): <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)",
   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)",
+  prePrFixCommitIssueGuidance: "pre-PR \uB9AC\uBDF0 \uC9C0\uC801\uC0AC\uD56D \uC218\uC815 \uCEE4\uBC0B\uC744 \uC9C4\uD589\uD558\uC138\uC694. pre-PR \uC218\uC815 \uD30C\uC77C\uB9CC \uC2A4\uD14C\uC774\uC9D5\uD55C \uB4A4 `fix(#{issueNumber}): <pre-pr-fix-summary>` \uD615\uC2DD\uC73C\uB85C \uCEE4\uBC0B\uD558\uC138\uC694. `<pre-pr-fix-summary>`\uC5D0\uB294 pre-PR\uC5D0\uC11C \uD574\uACB0\uD55C \uC9C0\uC801\uC0AC\uD56D\uC744 \uC791\uC131\uD558\uC138\uC694.",
+  prePrFixCommitGuidance: "pre-PR \uB9AC\uBDF0 \uC9C0\uC801\uC0AC\uD56D \uC218\uC815 \uCEE4\uBC0B\uC744 \uC9C4\uD589\uD558\uC138\uC694. pre-PR \uC218\uC815 \uD30C\uC77C\uB9CC \uC2A4\uD14C\uC774\uC9D5\uD55C \uB4A4 `fix(pre-pr): <pre-pr-fix-summary>` \uD615\uC2DD\uC73C\uB85C \uCEE4\uBC0B\uD558\uC138\uC694. `<pre-pr-fix-summary>`\uC5D0\uB294 pre-PR\uC5D0\uC11C \uD574\uACB0\uD55C \uC9C0\uC801\uC0AC\uD56D\uC744 \uC791\uC131\uD558\uC138\uC694.",
   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}") && WT="{projectGitCwd}/.worktrees/feat-{issueNumber}-{slug}" && for f in .env .env.local .env.development .env.development.local .env.test .env.test.local .env.production .env.production.local; do [ -f "{projectGitCwd}/$f" ] && [ ! -e "$WT/$f" ] && cp "{projectGitCwd}/$f" "$WT/$f" || true; done && echo "worktree: {projectGitCwd}/.worktrees/feat-{issueNumber}-{slug}"',
   moveToExistingWorktree: '\uD574\uB2F9 feature\uC6A9 worktree\uAC00 \uC774\uBBF8 \uC788\uC2B5\uB2C8\uB2E4. \uBA3C\uC800 `cd "{worktreePath}"`\uB85C \uC774\uB3D9\uD55C \uB4A4 context\uB97C \uB2E4\uC2DC \uC2E4\uD589\uD558\uC138\uC694.',
@@ -618,6 +620,7 @@ var koMessages = {
   prePrReviewRun: "\uCF54\uB4DC \uB9AC\uBDF0 \uC5D0\uC774\uC804\uD2B8\uB97C \uC2E4\uD589\uD574 \uBE44\uC5B4\uC788\uC9C0 \uC54A\uC740 `commandsExecuted`\uB97C \uD3EC\uD568\uD55C `review-trace.json`\uC744 \uC0DD\uC131\uD55C \uB4A4, `pre-pr-review --evidence review-trace.json`\uC73C\uB85C \uB9AC\uBDF0 \uACB0\uACFC\uB97C \uAE30\uB85D\uD558\uC138\uC694. (\uD655\uC778 \uD544\uC694)",
   prePrReviewEvidenceMissing: "tasks.md\uC758 `PR \uC804 \uB9AC\uBDF0 Evidence`\uAC00 \uBE44\uC5B4\uC788\uAC70\uB098 \uC720\uD6A8\uD558\uC9C0 \uC54A\uC2B5\uB2C8\uB2E4. \uC2E4\uC81C \uD30C\uC77C \uACBD\uB85C\uC640 `Pre-PR Review Log`(\uB610\uB294 `PR \uC804 \uB9AC\uBDF0 \uB85C\uADF8`)\uC5D0 placeholder\uAC00 \uC544\uB2CC `Summary`/`Decision`/`Findings`(\uB610\uB294 \uBA85\uC2DC\uC801 `0 findings`)/`Residual Risks`/`Tests Run`/\uC2E4\uD589 \uBA85\uB839(`commandsExecuted`)\uC744 \uAE30\uB85D\uD558\uC138\uC694. (\uD655\uC778 \uD544\uC694)",
   prePrReviewDecisionMissing: "tasks.md\uC758 `PR \uC804 \uB9AC\uBDF0 Decision`\uC774 \uBE44\uC5B4\uC788\uAC70\uB098 \uACB0\uC815 \uD615\uC2DD\uC774 \uC5C6\uC2B5\uB2C8\uB2E4. `\uACB0\uC815: ...`(\uB610\uB294 `decision: ...`) \uD615\uC2DD\uC73C\uB85C \uAE30\uB85D\uD558\uC138\uC694. (\uD655\uC778 \uD544\uC694)",
+  prePrReviewFixRequired: "\uD604\uC7AC `PR \uC804 \uB9AC\uBDF0 Decision`\uC774 `{decision}`\uC785\uB2C8\uB2E4. PR \uC0DD\uC131 \uB2E8\uACC4\uB85C \uC774\uB3D9\uD558\uAE30 \uC804\uC5D0 pre-PR \uC9C0\uC801\uC0AC\uD56D\uC744 \uCF54\uB4DC\uC5D0 \uBC18\uC601\uD558\uC138\uC694. (\uD655\uC778 \uD544\uC694)",
   prePrReviewDecisionReconfirm: "\uD604\uC7AC `PR \uC804 \uB9AC\uBDF0 Decision`\uC774 `{decision}`\uC785\uB2C8\uB2E4. \uC9C0\uC801\uC0AC\uD56D\uC744 \uBC18\uC601\uD55C \uB4A4 \uC774\uC804 \uC0C1\uD0DC \uC7AC\uC0AC\uC6A9\uC744 \uB9C9\uAE30 \uC704\uD574 \uBA85\uC2DC\uC801\uC73C\uB85C Decision\uC744 \uC9C0\uC815\uD574 \uC7AC\uC2E4\uD589\uD558\uC138\uC694: `{command}` (\uD655\uC778 \uD544\uC694)",
   prReviewEvidenceFieldMissing: "tasks.md\uC5D0 `PR \uB9AC\uBDF0 Evidence` \uD544\uB4DC\uAC00 \uC5C6\uC2B5\uB2C8\uB2E4. `- **PR \uB9AC\uBDF0 Evidence**: -` \uD56D\uBAA9\uC744 \uCD94\uAC00\uD558\uACE0 \uB2E4\uC2DC \uC9C4\uD589\uD558\uC138\uC694. (\uD655\uC778 \uD544\uC694)",
   prReviewEvidenceMissing: "tasks.md\uC758 `PR \uB9AC\uBDF0 Evidence`\uAC00 \uBE44\uC5B4\uC788\uAC70\uB098 \uC720\uD6A8\uD558\uC9C0 \uC54A\uC2B5\uB2C8\uB2E4. `\uC694\uC57D: ...`(\uB610\uB294 `summary: ...`) \uD615\uC2DD\uC73C\uB85C \uAE30\uB85D\uD558\uAC70\uB098 `PR Review Log`(\uB610\uB294 `PR \uB9AC\uBDF0 \uB85C\uADF8`)\uC758 `Summary`/`Decision`\uC774 \uC788\uB294 \uD30C\uC77C \uACBD\uB85C\uB97C \uC9C0\uC815\uD558\uC138\uC694. (\uD655\uC778 \uD544\uC694)",
@@ -1139,6 +1142,8 @@ var enMessages = {
   projectCommitUpdate: 'cd "{projectGitCwd}" && (git diff --cached --quiet && echo "No staged files. Stage only files changed in this task with git add [files], then run again." && exit 1 || git commit -m "feat({folderName}): {commitTopic}")',
   reviewFixCommitIssueGuidance: "Commit PR review fixes. Stage only review-fix files, then commit with `fix(#{issueNumber}): <review-fix-summary>`. `<review-fix-summary>` must describe review comments resolved in this commit (do not reuse task titles).",
   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).",
+  prePrFixCommitIssueGuidance: "Commit fixes requested by pre-PR review. Stage only pre-PR fix files, then commit with `fix(#{issueNumber}): <pre-pr-fix-summary>`. `<pre-pr-fix-summary>` must describe findings resolved from pre-PR review.",
+  prePrFixCommitGuidance: "Commit fixes requested by pre-PR review. Stage only pre-PR fix files, then commit with `fix(pre-pr): <pre-pr-fix-summary>`. `<pre-pr-fix-summary>` must describe findings resolved from pre-PR review.",
   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}") && WT="{projectGitCwd}/.worktrees/feat-{issueNumber}-{slug}" && for f in .env .env.local .env.development .env.development.local .env.test .env.test.local .env.production .env.production.local; do [ -f "{projectGitCwd}/$f" ] && [ ! -e "$WT/$f" ] && cp "{projectGitCwd}/$f" "$WT/$f" || true; done && echo "worktree: {projectGitCwd}/.worktrees/feat-{issueNumber}-{slug}"',
   moveToExistingWorktree: 'A matching feature worktree already exists. Move first: `cd "{worktreePath}"`, then re-run context.',
@@ -1160,6 +1165,7 @@ var enMessages = {
   prePrReviewRun: "Run the code review agent and generate `review-trace.json` with non-empty `commandsExecuted`, then execute `pre-pr-review --evidence review-trace.json` to record findings. (CHECK required)",
   prePrReviewEvidenceMissing: "tasks.md `Pre-PR Evidence` is empty/invalid. Point to a real file and include a `Pre-PR Review Log` section with non-placeholder `Summary`, `Decision`, `Findings` (or explicit `0 findings`), `Residual Risks`, `Tests Run`, and real executed commands (`commandsExecuted`). (CHECK required)",
   prePrReviewDecisionMissing: "tasks.md `Pre-PR Decision` is empty/placeholder or missing decision format. Record it as `decision: ...` (or `\uACB0\uC815: ...`). (CHECK required)",
+  prePrReviewFixRequired: "Current `Pre-PR Decision` is `{decision}`. Apply the requested fixes from pre-PR findings before moving to PR creation. (CHECK required)",
   prePrReviewDecisionReconfirm: "Current `Pre-PR Decision` is `{decision}`. After fixing findings, rerun pre-PR review with an explicit decision to avoid replaying the prior state: `{command}` (CHECK required)",
   prReviewEvidenceFieldMissing: "tasks.md is missing the `PR Review Evidence` field. Add `- **PR Review Evidence**: -` and continue. (CHECK required)",
   prReviewEvidenceMissing: "tasks.md `PR Review Evidence` is empty/invalid. Use `summary: ...` (or `\uC694\uC57D: ...`), or point to a file containing `PR Review Log` with non-placeholder `Summary` and `Decision`. (CHECK required)",
@@ -3349,7 +3355,8 @@ function getPrePrReviewPrompt(lang, skills, fallbackText) {
 3. \uD655\uC778\uB41C \uAC01 \uD30C\uC77C\uC5D0 \uB300\uD574 risk, security, perf, maintainability \uD3C9\uAC00\uC640 \uAD6C\uCCB4\uC801\uC778 fileLine \uC704\uCE58\uAC00 \uD3EC\uD568\uB41C 'review-trace.json' \uC99D\uAC70 \uD30C\uC77C\uC744 \uC0DD\uC131\uD558\uC138\uC694. \uC794\uC5EC \uC704\uD5D8(residualRisks)\uACFC \uC2E4\uD589\uD55C \uBA85\uB839\uC5B4(commandsExecuted)\uB3C4 \uD3EC\uD568\uD558\uC138\uC694.
 4. \uAE30\uBCF8 \uBCA0\uC774\uC2A4\uB77C\uC778\uC740 '${fallbackText}'\uC774\uBA70, 'create-pr' \uBB38\uC11C\uC758 'Pre-PR \uAE30\uBCF8 \uCCB4\uD06C\uB9AC\uC2A4\uD2B8' \uC139\uC158\uC744 \uC218\uD589\uD558\uC138\uC694. 
 5. \uC6B0\uC120\uC21C\uC704 \uC2A4\uD0AC: ${skills.length > 0 ? skills.join(", ") : "\uC5C6\uC74C"} \uB85C \uC2EC\uD654 \uAC80\uD1A0\uB97C \uC9C4\uD589\uD558\uC138\uC694.
-\uC644\uB8CC \uD6C4 'pnpm lee pre-pr-review <feature> --evidence review-trace.json --decision approve' \uB97C \uC2E4\uD589\uD558\uC138\uC694.`;
+6. \uC9C0\uC801\uC0AC\uD56D\uC774 \uB0A8\uC544 \uC788\uC73C\uBA74 \uBA3C\uC800 'npx lee-spec-kit pre-pr-review <feature> --evidence review-trace.json --decision changes_requested' \uB85C \uAE30\uB85D\uD558\uACE0 \uCF54\uB4DC\uB97C \uC218\uC815\uD558\uC138\uC694.
+7. \uC218\uC815/\uC7AC\uAC80\uC99D \uD6C4 \uCD5C\uC885 \uC2B9\uC778 \uC2DC\uC810\uC5D0 'npx lee-spec-kit pre-pr-review <feature> --evidence review-trace.json --decision approve' \uB97C \uC2E4\uD589\uD558\uC138\uC694.`;
   }
   return `Conduct a pre-PR code review.
 1. Run automated analyzers (e.g., vitest, biome check, pnpm audit).
@@ -3359,7 +3366,8 @@ function getPrePrReviewPrompt(lang, skills, fallbackText) {
 3. Generate a 'review-trace.json' file for all changed files, including evaluations for risk, security, perf, maintainability, and specific fileLine locators. Also include residualRisks and commandsExecuted array.
 4. The baseline is '${fallbackText}'. Always perform the 'Pre-PR Core Checklist' section of the 'create-pr' document. 
 5. Priority skills: ${skills.length > 0 ? skills.join(", ") : "None"} for deeper technical review.
-After completion, execute 'pnpm lee pre-pr-review <feature> --evidence review-trace.json --decision approve'.`;
+6. If unresolved findings remain, first record them with 'npx lee-spec-kit pre-pr-review <feature> --evidence review-trace.json --decision changes_requested' and apply code fixes.
+7. After fixes and re-validation, run 'npx lee-spec-kit pre-pr-review <feature> --evidence review-trace.json --decision approve' for final pre-PR approval.`;
 }
 function getCodeReviewPrompt(lang) {
   if (lang === "ko") {
@@ -3384,6 +3392,9 @@ function isPrMetadataConfigured(feature) {
 function isReviewIterationPhase(feature, workflowPolicy) {
   return workflowPolicy.requirePr && workflowPolicy.requireReview && isPrMetadataConfigured(feature) && !!feature.pr.link && feature.pr.status === "Review";
 }
+function isPrePrFixIterationPhase(feature, workflowPolicy, prePrReviewPolicy) {
+  return prePrReviewPolicy.enabled && workflowPolicy.requirePr && feature.prePrReview.status === "Done" && !!feature.prePrReview.decisionOutcome && feature.prePrReview.decisionOutcome !== "approve" && (!isPrMetadataConfigured(feature) || !feature.pr.link);
+}
 function isPrePrReviewSatisfied(feature, prePrReviewPolicy) {
   if (!prePrReviewPolicy.enabled) return true;
   if (!feature.docs.prePrReviewFieldExists || feature.prePrReview.status !== "Done") {
@@ -3460,6 +3471,7 @@ function buildPrePrReviewCommandArgs(feature, evidencePath, decision) {
 }
 function resolvePrePrReviewEvidencePath(feature) {
   const docsRoot = feature.git.docsGitCwd;
+  const docsParent = path12.dirname(docsRoot);
   const candidates = [];
   const explicit = (feature.prePrReview.evidence || "").trim();
   if (explicit && explicit !== "-") {
@@ -3467,8 +3479,16 @@ function resolvePrePrReviewEvidencePath(feature) {
       candidates.push(explicit);
     } else {
       candidates.push(path12.resolve(docsRoot, explicit));
+      candidates.push(path12.resolve(docsParent, explicit));
       candidates.push(path12.resolve(process.cwd(), explicit));
       candidates.push(path12.resolve(feature.path, explicit));
+      const normalizedExplicit = explicit.replace(/\\/g, "/");
+      if (normalizedExplicit.startsWith("docs/")) {
+        const withoutDocsPrefix = normalizedExplicit.slice("docs/".length);
+        if (withoutDocsPrefix) {
+          candidates.push(path12.resolve(docsRoot, withoutDocsPrefix));
+        }
+      }
     }
   }
   candidates.push(path12.resolve(process.cwd(), "review-trace.json"));
@@ -3501,6 +3521,17 @@ function resolveProjectCommitTopic(feature) {
   const topic = withoutTaskId || normalizeCommitTopicText(feature.folderName);
   return toShellSafeCommitTopic(topic);
 }
+function getReviewFixCommitGuidance(feature, lang, options) {
+  const prePr = !!options?.prePr;
+  if (prePr) {
+    return feature.issueNumber ? tr(lang, "messages", "prePrFixCommitIssueGuidance", {
+      issueNumber: feature.issueNumber
+    }) : tr(lang, "messages", "prePrFixCommitGuidance");
+  }
+  return feature.issueNumber ? tr(lang, "messages", "reviewFixCommitIssueGuidance", {
+    issueNumber: feature.issueNumber
+  }) : tr(lang, "messages", "reviewFixCommitGuidance");
+}
 function resolveManagedWorktreeCleanupPaths(projectGitCwd) {
   if (!projectGitCwd) return null;
   const normalized = path12.resolve(projectGitCwd);
@@ -4013,7 +4044,16 @@ function getStepDefinitions(ctx) {
               ];
             }
             if (f.git.projectHasUncommittedChanges) {
-              if (isReviewIterationPhase(f, workflowPolicy)) {
+              const reviewIterationPhase = isReviewIterationPhase(
+                f,
+                workflowPolicy
+              );
+              const prePrFixIterationPhase = isPrePrFixIterationPhase(
+                f,
+                workflowPolicy,
+                prePrReviewPolicy
+              );
+              if (reviewIterationPhase || prePrFixIterationPhase) {
                 if (!f.git.projectGitCwd) {
                   return [
                     {
@@ -4032,11 +4072,8 @@ function getStepDefinitions(ctx) {
                     type: "instruction",
                     category: "review_fix_commit",
                     requiresUserCheck: true,
-                    message: f.issueNumber ? tr(lang, "messages", "reviewFixCommitIssueGuidance", {
-                      projectGitCwd: f.git.projectGitCwd,
-                      issueNumber: f.issueNumber
-                    }) : tr(lang, "messages", "reviewFixCommitGuidance", {
-                      projectGitCwd: f.git.projectGitCwd
+                    message: getReviewFixCommitGuidance(f, lang, {
+                      prePr: prePrFixIterationPhase
                     })
                   }
                 ];
@@ -4262,7 +4299,16 @@ ${tr(lang, "messages", "taskCommitGateWarnProceed", {
               }
             ];
           }
-          if (isReviewIterationPhase(f, workflowPolicy)) {
+          const reviewIterationPhase = isReviewIterationPhase(
+            f,
+            workflowPolicy
+          );
+          const prePrFixIterationPhase = isPrePrFixIterationPhase(
+            f,
+            workflowPolicy,
+            prePrReviewPolicy
+          );
+          if (reviewIterationPhase || prePrFixIterationPhase) {
             if (!f.git.projectGitCwd) {
               return [
                 {
@@ -4277,11 +4323,8 @@ ${tr(lang, "messages", "taskCommitGateWarnProceed", {
                 type: "instruction",
                 category: "review_fix_commit",
                 requiresUserCheck: true,
-                message: f.issueNumber ? tr(lang, "messages", "reviewFixCommitIssueGuidance", {
-                  projectGitCwd: f.git.projectGitCwd,
-                  issueNumber: f.issueNumber
-                }) : tr(lang, "messages", "reviewFixCommitGuidance", {
-                  projectGitCwd: f.git.projectGitCwd
+                message: getReviewFixCommitGuidance(f, lang, {
+                  prePr: prePrFixIterationPhase
                 })
               }
             ];
@@ -4338,33 +4381,41 @@ ${tr(lang, "messages", "taskCommitGateWarnProceed", {
               }
             ];
           }
-          const evidencePath = resolvePrePrReviewEvidencePath(f);
-          if (!evidencePath) {
+          if (f.prePrReview.decisionOutcome && f.prePrReview.decisionOutcome !== "approve") {
+            const rerunEvidencePath = resolvePrePrReviewEvidencePath(f) || "review-trace.json";
+            const rerunCommand = buildSelfCliCommand(
+              buildPrePrReviewCommandArgs(f, rerunEvidencePath, "approve")
+            );
             return [
               {
                 type: "instruction",
-                category: "pre_pr_review",
+                category: "review_fix_commit",
                 requiresUserCheck: true,
-                message: getPrePrReviewPrompt(
-                  lang,
-                  prePrReviewPolicy.skills,
-                  prePrReviewPolicy.fallback
-                )
+                message: `${tr(lang, "messages", "prePrReviewFixRequired", {
+                  decision: f.prePrReview.decisionOutcome
+                })}
+${getReviewFixCommitGuidance(f, lang, {
+                  prePr: true
+                })}
+${tr(lang, "messages", "prePrReviewDecisionReconfirm", {
+                  decision: f.prePrReview.decisionOutcome,
+                  command: rerunCommand
+                })}`
               }
             ];
           }
-          if (f.prePrReview.decisionOutcome && f.prePrReview.decisionOutcome !== "approve") {
+          const evidencePath = resolvePrePrReviewEvidencePath(f);
+          if (!evidencePath) {
             return [
               {
                 type: "instruction",
                 category: "pre_pr_review",
                 requiresUserCheck: true,
-                message: tr(lang, "messages", "prePrReviewDecisionReconfirm", {
-                  decision: f.prePrReview.decisionOutcome,
-                  command: buildSelfCliCommand(
-                    buildPrePrReviewCommandArgs(f, evidencePath, "approve")
-                  )
-                })
+                message: getPrePrReviewPrompt(
+                  lang,
+                  prePrReviewPolicy.skills,
+                  prePrReviewPolicy.fallback
+                )
               }
             ];
           }
@@ -5317,11 +5368,14 @@ function hasPrePrReviewLogQuality(content, options) {
     if (!hasValidReviewLogEntries(decisionEntries)) continue;
     const findingsEntries = collectStructuredReviewEntries(section, [
       "Findings",
+      "Findings (Changed Files)",
       "\uC9C0\uC801\uC0AC\uD56D",
-      "\uC9C0\uC801 \uC0AC\uD56D"
+      "\uC9C0\uC801 \uC0AC\uD56D",
+      "\uC9C0\uC801\uC0AC\uD56D (\uBCC0\uACBD \uD30C\uC77C)",
+      "\uC9C0\uC801 \uC0AC\uD56D (\uBCC0\uACBD \uD30C\uC77C)"
     ]);
     const hasActionableFindings = findingsEntries.map((entry) => entry.trim()).some(
-      (entry) => entry.length > 0 && !isReviewDraftPlaceholder(entry) && !isPlaceholderReviewEvidence(entry) && /\S+:\d+/.test(entry)
+      (entry) => entry.length > 0 && !isReviewDraftPlaceholder(entry) && !isPlaceholderReviewEvidence(entry) && (/\S+:\d+/.test(entry) || /\b[Ll]ines?\s+\d+/.test(entry))
     );
     const hasExplicitZeroFindings = findingsEntries.some(
       (entry) => isExplicitZeroFindingsEntry(entry)
@@ -5340,14 +5394,15 @@ function hasPrePrReviewLogQuality(content, options) {
       "\uC2E4\uD589 \uD14C\uC2A4\uD2B8",
       "\uD14C\uC2A4\uD2B8 \uC2E4\uD589"
     ]);
-    if (!hasValidReviewLogEntries(testsRunEntries)) continue;
+    const commandsEntries = collectStructuredReviewEntries(section, [
+      "Commands Executed",
+      "Executed Commands",
+      "\uC2E4\uD589 \uBA85\uB839\uC5B4",
+      "\uC2E4\uD589 \uBA85\uB839"
+    ]);
+    const hasTestsRunEntries = hasValidReviewLogEntries(testsRunEntries) || hasValidReviewLogEntries(commandsEntries);
+    if (!hasTestsRunEntries) continue;
     if (requireCommandsExecuted) {
-      const commandsEntries = collectStructuredReviewEntries(section, [
-        "Commands Executed",
-        "Executed Commands",
-        "\uC2E4\uD589 \uBA85\uB839\uC5B4",
-        "\uC2E4\uD589 \uBA85\uB839"
-      ]);
       const hasCommandsExecuted = commandsEntries.map((entry) => entry.trim()).some(
         (entry) => entry.length > 0 && !isReviewDraftPlaceholder(entry) && !isPlaceholderReviewEvidence(entry) && !isNoCommandsPlaceholder(entry)
       );
@@ -5392,6 +5447,13 @@ function resolveLocalEvidencePathCandidates(rawValue, context) {
     candidates.add(path12.resolve(context.featurePath, evidencePath));
     candidates.add(path12.resolve(context.docsDir, evidencePath));
     candidates.add(path12.resolve(path12.dirname(context.docsDir), evidencePath));
+    const normalizedEvidencePath = evidencePath.replace(/\\/g, "/");
+    if (normalizedEvidencePath.startsWith("docs/")) {
+      const withoutDocsPrefix = normalizedEvidencePath.slice("docs/".length);
+      if (withoutDocsPrefix) {
+        candidates.add(path12.resolve(context.docsDir, withoutDocsPrefix));
+      }
+    }
   }
   return [...candidates];
 }
@@ -11976,9 +12038,49 @@ function extractMarkdownByHeadings(content, headings, levels) {
   }
   return lines.slice(start, end).join("\n");
 }
+function removeMarkdownByHeadings(content, headings, levels) {
+  const targets = new Set(headings.map((heading) => normalizeHeading(heading)));
+  const lines = content.split("\n");
+  const levelSet = new Set(levels);
+  let start = -1;
+  let startLevel = 0;
+  for (let i = 0; i < lines.length; i++) {
+    const match = lines[i].match(/^\s*(#{2,6})\s+(.+?)\s*$/);
+    if (!match) continue;
+    const level = match[1].length;
+    if (!levelSet.has(level)) continue;
+    if (!targets.has(normalizeHeading(match[2]))) continue;
+    start = i;
+    startLevel = level;
+    break;
+  }
+  if (start < 0) return content;
+  let end = lines.length;
+  for (let i = start + 1; i < lines.length; i++) {
+    const heading = lines[i].match(/^\s*(#{2,6})\s+(.+?)\s*$/);
+    if (!heading) continue;
+    const level = heading[1].length;
+    if (level <= startLevel) {
+      end = i;
+      break;
+    }
+  }
+  const next = [...lines.slice(0, start), ...lines.slice(end)].join("\n");
+  const hasTrailingNewline = /\n$/.test(content);
+  const normalized = next.replace(/\n{3,}/g, "\n\n").trimEnd();
+  if (!normalized) return "";
+  return hasTrailingNewline ? `${normalized}
+` : normalized;
+}
 function extractMarkdownSection(content, headings) {
   return extractMarkdownByHeadings(content, headings, [2]);
 }
+function stripIssueDraftMetadataSection(content) {
+  return stripWorkflowDraftMetadataSection(content);
+}
+function stripWorkflowDraftMetadataSection(content) {
+  return removeMarkdownByHeadings(content, ["Metadata", "\uBA54\uD0C0\uB370\uC774\uD130"], [2]);
+}
 function isTemplateLine(line) {
   const trimmed = line.trim();
   if (!trimmed) return true;
@@ -12985,8 +13087,20 @@ function githubCommand(program2) {
           kindLabel: tg(config.lang, "kindIssue"),
           lang: config.lang
         });
-        const body = preparedBody.body;
-        const bodyFile = preparedBody.bodyFile;
+        const body = stripIssueDraftMetadataSection(preparedBody.body);
+        let bodyFile = preparedBody.bodyFile;
+        if (options.create && body !== preparedBody.body) {
+          const sanitizedBodyFile = toBodyFilePath(
+            void 0,
+            "issue",
+            config.docsDir,
+            `${feature.type}-issue-sanitized`,
+            config.lang
+          );
+          await fs.ensureDir(path12.dirname(sanitizedBodyFile));
+          await fs.writeFile(sanitizedBodyFile, body, "utf-8");
+          bodyFile = sanitizedBodyFile;
+        }
         const title = options.title?.trim() || (preparedBody.source === "workflow-ready" ? preparedBody.draftMetadata?.title : void 0) || defaultTitle;
         const labels = parseLabels(
           optionLabels || (preparedBody.source === "workflow-ready" ? preparedBody.draftMetadata?.labels : void 0),
@@ -13193,8 +13307,22 @@ function githubCommand(program2) {
           kindLabel: tg(config.lang, "kindPr"),
           lang: config.lang
         });
-        let body = preparedBody.body;
+        let body = stripWorkflowDraftMetadataSection(
+          preparedBody.body
+        );
         let bodyFile = preparedBody.bodyFile;
+        if (options.create && body !== preparedBody.body) {
+          const sanitizedBodyFile = toBodyFilePath(
+            void 0,
+            "pr",
+            config.docsDir,
+            `${feature.type}-pr-sanitized`,
+            config.lang
+          );
+          await fs.ensureDir(path12.dirname(sanitizedBodyFile));
+          await fs.writeFile(sanitizedBodyFile, body, "utf-8");
+          bodyFile = sanitizedBodyFile;
+        }
         const title = options.title?.trim() || (preparedBody.source === "workflow-ready" ? preparedBody.draftMetadata?.title : void 0) || defaultTitle;
         const labels = parseLabels(
           optionLabels || (preparedBody.source === "workflow-ready" ? preparedBody.draftMetadata?.labels : void 0),
@@ -14606,7 +14734,24 @@ function getPreferredKeys(lang) {
 }
 function buildReportContent(input) {
   const skills = input.skills.length > 0 ? input.skills.join(", ") : "code-review-excellence";
-  const commandsRun = input.evidence.commandsExecuted.length > 0 ? input.evidence.commandsExecuted.map((c) => `  - \`${c}\``).join("\n") : "  - None specified";
+  const normalizedCommands = input.evidence.commandsExecuted.map((entry) => normalizeShellLikeCommand(entry)).filter(Boolean);
+  const commandsRun = normalizedCommands.length > 0 ? normalizedCommands.map((c) => `  - \`${c}\``).join("\n") : "  - None specified";
+  const testCommandTokens = [
+    "test",
+    "vitest",
+    "jest",
+    "playwright",
+    "cypress",
+    "tsc",
+    "biome check",
+    "eslint",
+    "audit"
+  ];
+  const testsRunCommands = normalizedCommands.filter((entry) => {
+    const lowered = entry.toLowerCase();
+    return testCommandTokens.some((token) => lowered.includes(token));
+  });
+  const testsRun = (testsRunCommands.length > 0 ? testsRunCommands : normalizedCommands).map((entry) => `  - \`${entry}\` -> executed`).join("\n");
   let filesSection = "";
   if (input.evidence.files.length === 0) {
     filesSection = "  - 0 findings";
@@ -14631,6 +14776,9 @@ function buildReportContent(input) {
 - **Commands Executed**:
 ${commandsRun}
 
+- **Tests Run**:
+${testsRun || "  - Not recorded"}
+
 - **Residual Risks**:
   - ${input.evidence.residualRisks}
 
@@ -14643,7 +14791,7 @@ ${mainScopeFiles}
   - **Worktree Changed Files**:
 ${worktreeScopeFiles}
 
-- **Findings (Changed Files)**:
+- **Findings**:
 ${filesSection}
 
 - **Trace**: pre-pr-review command executed and synced with tasks.md