issue-flow 0.6.0 → 0.7.1

package/README.md

@@ -87,7 +87,7 @@ npx issue-flow run 42 --mode manual
 npx issue-flow run 42 --web
 ```
 
-Executes all phases in order: **init** → **analyze** → **prd** → **plan** → **execute** → **review** → **pr**. Automatically resumes from the last incomplete phase if pipeline state exists. On review failure, runs correction cycles (re-execute + re-review) up to `maxCorrectionCycles`.
+Executes all phases in order: **init** → **analyze** → **prd** → **plan** → **execute** → **review** → **pr**. Automatically resumes from the last incomplete phase if pipeline state exists. On review failure, its findings are saved verbatim to `lastReviewFindings` in `tasks.json`, and a correction cycle (re-execute + re-review) runs up to `maxCorrectionCycles`. The re-execute step reads `lastReviewFindings` and treats the issue as unresolved even if every user story already has `passes: true`, until the findings are addressed and the field is cleared back to `null`.
 
 ### `init` -- Check prerequisites
 
@@ -216,6 +216,8 @@ The `pipeline` field tracks which phases have completed, enabling resume from an
 }
 ```
 
+The top-level `lastReviewFindings` field (`string | null`) holds the verbatim findings of the most recent failed `review` phase. Non-null overrides the "issue already complete" check even when every user story has `passes: true`, so a correction cycle's re-execute step is guaranteed to run instead of exiting immediately — see `core/engine.ts`.
+
 ## Architecture
 
 ```

package/dist/{analyze-VVG7UABS.js → analyze-R25HDGBJ.js}

@@ -1,14 +1,18 @@
 #!/usr/bin/env node
 import {
   issuePlaceholders,
-  resolveCommandIssue,
+  resolveCommandIssue
+} from "./chunk-RN2FAMUU.js";
+import {
   runHeadless
-} from "./chunk-SVOWWZQJ.js";
+} from "./chunk-TCOS2KFB.js";
 import {
   applyPlaceholders,
   loadPrompt
 } from "./chunk-SLHZPB3C.js";
-import "./chunk-4HT4EVYV.js";
+import {
+  getIssueDir
+} from "./chunk-2AVZZAC6.js";
 import {
   getGlobalTimeout,
   loadTaskPlan,
@@ -16,14 +20,14 @@ import {
   printInfo,
   printSuccess,
   saveTaskPlan
-} from "./chunk-WZDCTAWZ.js";
+} from "./chunk-JNEHVQH5.js";
 
 // src/commands/analyze.ts
 import { mkdir, readFile, writeFile } from "fs/promises";
 import { join } from "path";
 async function runAnalyze(issue, resolvedIssue) {
   const issueNumber = issue.replace(/^#/, "");
-  const issueDir = join("issues", issueNumber);
+  const issueDir = await getIssueDir(issueNumber);
   const analysisPath = join(issueDir, "analysis.md");
   const resolution = await resolveCommandIssue(issueNumber, resolvedIssue);
   if (!resolution.ok) {
@@ -73,4 +77,4 @@ async function runAnalyze(issue, resolvedIssue) {
 export {
   runAnalyze
 };
-//# sourceMappingURL=analyze-VVG7UABS.js.map
+//# sourceMappingURL=analyze-R25HDGBJ.js.map

package/dist/analyze-R25HDGBJ.js.map

@@ -0,0 +1 @@
+{"version":3,"sources":["../src/commands/analyze.ts"],"sourcesContent":["import { mkdir, readFile, writeFile } from 'node:fs/promises';\nimport { join } from 'node:path';\nimport { runHeadless } from '../core/headless.js';\nimport { applyPlaceholders, loadPrompt } from '../core/prompt-resolver.js';\nimport { loadTaskPlan, saveTaskPlan } from '../core/state-manager.js';\nimport { getGlobalTimeout } from '../core/verbose.js';\nimport { issuePlaceholders, resolveCommandIssue } from '../issues/context.js';\nimport type { ResolvedIssue } from '../issues/types.js';\nimport { printError, printInfo, printSuccess } from '../ui/logger.js';\nimport { getIssueDir } from '../utils/git.js';\n\nexport async function runAnalyze(issue: string, resolvedIssue?: ResolvedIssue): Promise<number> {\n  const issueNumber = issue.replace(/^#/, '');\n  const issueDir = await getIssueDir(issueNumber);\n  const analysisPath = join(issueDir, 'analysis.md');\n\n  const resolution = await resolveCommandIssue(issueNumber, resolvedIssue);\n  if (!resolution.ok) {\n    return resolution.code;\n  }\n\n  await mkdir(issueDir, { recursive: true });\n\n  const template = await loadPrompt('analyze');\n  const prompt = applyPlaceholders(template, {\n    __ISSUE_NUMBER__: issueNumber,\n    __ANALYSIS_PATH__: analysisPath,\n    // Last: the Issue content is substituted in but never scanned again, so a\n    // body that happens to contain a placeholder is left untouched.\n    ...issuePlaceholders(resolution.resolved),\n  });\n\n  const result = await runHeadless({\n    prompt,\n    maxTurns: 30,\n    timeout: getGlobalTimeout() ?? 300_000,\n    outputFormat: 'text',\n    allowedTools: ['Bash', 'Read', 'Glob', 'Grep', 'Write'],\n    statusMessage: `Analyzing issue #${issueNumber}...`,\n  });\n\n  if (!result.success) {\n    printError(`Analysis failed: ${result.error}`);\n    return 1;\n  }\n\n  // Verify the file was created\n  try {\n    const content = await readFile(analysisPath, 'utf-8');\n    if (content.length < 10) {\n      printError('Analysis file was created but appears empty');\n      return 1;\n    }\n  } catch {\n    // File wasn't created by headless — save the result as analysis\n    printInfo('Headless did not create analysis file; saving output directly');\n    await writeFile(analysisPath, result.result, 'utf-8');\n  }\n\n  // Update pipeline state\n  const tasksPath = join(issueDir, 'tasks.json');\n  try {\n    const plan = await loadTaskPlan(tasksPath);\n    plan.pipeline.analyzeCompleted = true;\n    await saveTaskPlan(tasksPath, plan);\n  } catch {\n    // tasks.json may not exist yet — that's OK for standalone analyze\n  }\n\n  printSuccess(`Analysis saved to ${analysisPath}`);\n  return 0;\n}\n"],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;AAAA,SAAS,OAAO,UAAU,iBAAiB;AAC3C,SAAS,YAAY;AAUrB,eAAsB,WAAW,OAAe,eAAgD;AAC9F,QAAM,cAAc,MAAM,QAAQ,MAAM,EAAE;AAC1C,QAAM,WAAW,MAAM,YAAY,WAAW;AAC9C,QAAM,eAAe,KAAK,UAAU,aAAa;AAEjD,QAAM,aAAa,MAAM,oBAAoB,aAAa,aAAa;AACvE,MAAI,CAAC,WAAW,IAAI;AAClB,WAAO,WAAW;AAAA,EACpB;AAEA,QAAM,MAAM,UAAU,EAAE,WAAW,KAAK,CAAC;AAEzC,QAAM,WAAW,MAAM,WAAW,SAAS;AAC3C,QAAM,SAAS,kBAAkB,UAAU;AAAA,IACzC,kBAAkB;AAAA,IAClB,mBAAmB;AAAA;AAAA;AAAA,IAGnB,GAAG,kBAAkB,WAAW,QAAQ;AAAA,EAC1C,CAAC;AAED,QAAM,SAAS,MAAM,YAAY;AAAA,IAC/B;AAAA,IACA,UAAU;AAAA,IACV,SAAS,iBAAiB,KAAK;AAAA,IAC/B,cAAc;AAAA,IACd,cAAc,CAAC,QAAQ,QAAQ,QAAQ,QAAQ,OAAO;AAAA,IACtD,eAAe,oBAAoB,WAAW;AAAA,EAChD,CAAC;AAED,MAAI,CAAC,OAAO,SAAS;AACnB,eAAW,oBAAoB,OAAO,KAAK,EAAE;AAC7C,WAAO;AAAA,EACT;AAGA,MAAI;AACF,UAAM,UAAU,MAAM,SAAS,cAAc,OAAO;AACpD,QAAI,QAAQ,SAAS,IAAI;AACvB,iBAAW,6CAA6C;AACxD,aAAO;AAAA,IACT;AAAA,EACF,QAAQ;AAEN,cAAU,+DAA+D;AACzE,UAAM,UAAU,cAAc,OAAO,QAAQ,OAAO;AAAA,EACtD;AAGA,QAAM,YAAY,KAAK,UAAU,YAAY;AAC7C,MAAI;AACF,UAAM,OAAO,MAAM,aAAa,SAAS;AACzC,SAAK,SAAS,mBAAmB;AACjC,UAAM,aAAa,WAAW,IAAI;AAAA,EACpC,QAAQ;AAAA,EAER;AAEA,eAAa,qBAAqB,YAAY,EAAE;AAChD,SAAO;AACT;","names":[]}