issue-flow 0.11.1 → 0.13.0

package/prompts/pr.md

@@ -14,13 +14,20 @@ __ISSUE_BODY__
 </issue-body>
 
 Steps:
-1. Read the task plan from __TASKS_PATH__ if it exists
-2. Review the git log for this branch: git log __BASE_BRANCH__..HEAD --oneline
-3. Review the diff: git diff __BASE_BRANCH__...HEAD --stat
-4. Create a well-structured PR using gh pr create
+1. Check whether this branch already has an open Pull Request:
+   `gh pr list --head __BRANCH_NAME__ --state open --json number,url`.
+   If it does, do NOT create a second one — report the existing URL and stop.
+   The pipeline checks this before invoking you and normally skips this phase
+   entirely, so reaching this point with a Pull Request already open means the
+   check could not run: it is the last guard against a duplicate, not a
+   formality.
+2. Read the task plan from __TASKS_PATH__ if it exists
+3. Review the git log for this branch: git log __BASE_BRANCH__..HEAD --oneline
+4. Review the diff: git diff __BASE_BRANCH__...HEAD --stat
+5. Create a well-structured PR using gh pr create
 
 The PR should:
-- Have a clear, concise title (under 70 characters)
+- Use this title verbatim: `__PR_TITLE_CONVENTION__`
 - Include a summary of changes
 - Include a test plan
 - Cite where the demand is described: the reference above (`__ISSUE_URL__`)
@@ -29,7 +36,7 @@ Issue reference line(s):
 - Include the lines below verbatim in the PR body, each entry on its own line,
   with no surrounding code fence or quoting — GitHub only auto-closes an issue
   when the `Closes #N` line is plain body text:
-__ISSUE_CLOSES__
+__ISSUE_REFERENCE__
 - When nothing follows the line above, the issue has no GitHub counterpart: do
   NOT invent a "Closes #" reference, and cite `__ISSUE_URL__` in the body instead
 

package/web/AGENTS.md

@@ -10,7 +10,7 @@ vitest. Toda verificação de mudança aqui é manual, no navegador.
 
 O painel consome `GET api/sessions` (lista enriquecida para o dashboard),
 `GET api/status` (o `SessionSnapshot` serializado, opcionalmente com
-`?session=<id>`) e `GET api/health`. Ele precisa renderizar **session.json de
+`?session=<id>`), `GET api/events?session=<id>` (journal) e `GET api/health`. Ele precisa renderizar **session.json de
 execuções antigas**, então todo campo pode chegar como `undefined` (não
 existia na versão que gravou o arquivo) além do `null` (existe, não
 informado). Os dois significam "não informado" e nunca podem virar `0`,
@@ -36,7 +36,7 @@ do snapshot.
 
 ## Abas, Kanban e drawer
 
-O painel tem duas abas ("Execução" e "Kanban") e um drawer de detalhes por
+O painel tem três abas ("Execução", "Kanban" e "Histórico") e um drawer de detalhes por
 story. Três regras seguram esse conjunto:
 
 - **Acesso a story sempre por `getStoryById()` / `getStories()`.** Elas são a
@@ -51,7 +51,7 @@ story. Três regras seguram esse conjunto:
   então uma referência guardada na abertura apontaria para um nó fora do
   documento. Pelo mesmo motivo o foco volta ao card via
   `[data-story-id="…"]` no momento de fechar.
-- **`renderKanban()` e `renderDrawer()` são chamadas incondicionalmente** dentro
+- **`renderKanban()`, `renderHistory()` e `renderDrawer()` são chamadas incondicionalmente** dentro
   de `render()`, não ao trocar de aba. Uma aba inativa não pode ficar defasada,
   e é o `renderDrawer()` de cada poll que mantém o drawer aberto em dia (e o
   fecha quando a story some do plano).

package/web/public/app.css

@@ -47,6 +47,10 @@
   box-sizing: border-box;
 }
 
+[hidden] {
+  display: none !important;
+}
+
 body {
   margin: 0;
   background: var(--bg);
@@ -454,6 +458,34 @@ progress {
   border-bottom: 1px solid var(--border);
 }
 
+.plain-list .history-entry {
+  align-items: flex-start;
+}
+
+.history-type {
+  flex: 0 0 9rem;
+  color: var(--muted);
+}
+
+.history-message {
+  min-width: 0;
+  overflow-wrap: anywhere;
+}
+
+@media (max-width: 560px) {
+  .plain-list .history-entry {
+    flex-wrap: wrap;
+  }
+
+  .history-type {
+    flex-basis: auto;
+  }
+
+  .history-message {
+    flex-basis: 100%;
+  }
+}
+
 .plain-list li:last-child {
   border-bottom: none;
 }
@@ -904,6 +936,12 @@ progress {
   font: inherit;
   cursor: pointer;
   display: grid;
+  /* O grid externo estica os cards de uma linha para a mesma altura. Sem isto,
+     `align-content: stretch` distribui a sobra entre as faixas — inclusive a do
+     resumo — e a faixa fica mais alta que as 3 linhas do clamp, deixando vazar
+     uma 4ª linha cortada por baixo do reticente. Empacotar no topo manda a
+     sobra para o fim do card, que é onde ela não engana ninguém. */
+  align-content: start;
   gap: 10px;
   transition: border-color 0.15s ease, box-shadow 0.15s ease;
 }

package/web/public/app.js

@@ -1,6 +1,6 @@
 // Interface do monitoramento web do issue-flow.
 // JS puro, sem framework e sem recursos externos — funciona offline.
-// Consome GET api/sessions, GET api/status (ETag/304) e GET api/health;
+// Consome GET api/sessions, GET api/status (ETag/304), GET api/events e GET api/health;
 // todo texto dinâmico entra via textContent (nunca innerHTML com dados
 // do snapshot).
 (() => {
@@ -79,6 +79,7 @@
     progressPercent: document.getElementById('progress-percent'),
     progressCounters: document.getElementById('progress-counters'),
     now: document.getElementById('now'),
+    resilience: document.getElementById('resilience'),
     phases: document.getElementById('phases'),
     nextSteps: document.getElementById('next-steps'),
     stories: document.getElementById('stories'),
@@ -86,6 +87,8 @@
     pullRequests: document.getElementById('pull-requests'),
     logFilter: document.getElementById('log-filter'),
     logs: document.getElementById('logs'),
+    historyFilter: document.getElementById('history-filter'),
+    history: document.getElementById('history'),
     sessionMeta: document.getElementById('session-meta'),
     tabs: Array.prototype.slice.call(document.querySelectorAll('[role="tab"]')),
     kanban: document.getElementById('kanban'),
@@ -105,6 +108,8 @@
     polling: false,
     logFilter: 'all',
     activeTab: 'tab-execution',
+    historyFilter: 'all',
+    events: [],
     // Só o id: o card que abriu o drawer é destruído no próximo render, então
     // guardar o nó levaria a uma referência morta.
     selectedStoryId: null,
@@ -116,6 +121,7 @@
     detailSessionId: null,
     viewMode: 'detail', // 'detail' | 'dashboard'
     statusUrl: 'api/status',
+    eventsUrl: null,
     // Se selectSession/clearSessionSelection chega durante um poll, reexecuta.
     pollAgain: false,
   };
@@ -372,9 +378,6 @@
       if (!stillThere) {
         state.selectedSessionId = null;
         state.etag = null;
-      } else if (sessions.length === 1) {
-        // Seleção explícita deixa de fazer sentido com uma única sessão.
-        state.selectedSessionId = null;
       }
     }
 
@@ -387,10 +390,6 @@
       return { mode: 'detail', session: null };
     }
 
-    if (sessions.length === 1) {
-      return { mode: 'detail', session: sessions[0] };
-    }
-
     return { mode: 'dashboard', session: null };
   }
 
@@ -398,9 +397,9 @@
     state.viewMode = mode;
     els.viewDashboard.hidden = mode !== 'dashboard';
     els.viewDetail.hidden = mode !== 'detail';
-    // Back só quando o usuário escolheu um card e ainda há dashboard para voltar.
+    // Back quando o usuário escolheu um card e existe uma lista para voltar.
     const showBack =
-      mode === 'detail' && state.selectedSessionId !== null && state.sessions.length >= 2;
+      mode === 'detail' && state.selectedSessionId !== null && state.sessions.length >= 1;
     els.backToDashboard.hidden = !showBack;
     if (mode === 'dashboard' && state.selectedStoryId !== null) {
       closeDrawer();
@@ -414,10 +413,17 @@
     return session.statusUrl.replace(/^\//, '');
   }
 
+  function eventsUrlFor(session) {
+    if (!session || !session.eventsUrl) return null;
+    return session.eventsUrl.replace(/^\//, '');
+  }
+
   function clearDetailState() {
     state.etag = null;
     state.snapshot = null;
     state.detailSessionId = null;
+    state.events = [];
+    state.eventsUrl = null;
   }
 
   function requestPoll() {
@@ -491,6 +497,7 @@
         state.statusUrl = nextUrl;
         state.etag = null;
       }
+      state.eventsUrl = eventsUrlFor(resolved.session);
 
       // Evita flash da sessão anterior enquanto o status novo carrega.
       if (sessionChanged || !state.snapshot) {
@@ -516,6 +523,13 @@
         state.snapshot = await res.json();
         render();
       }
+      if (state.eventsUrl) {
+        const eventsRes = await fetch(state.eventsUrl, { cache: 'no-store' });
+        if (!eventsRes.ok) throw new Error('HTTP ' + eventsRes.status);
+        const entries = await eventsRes.json();
+        state.events = Array.isArray(entries) ? entries : [];
+        renderHistory();
+      }
       setViewMode('detail');
       state.failures = 0;
       els.banner.hidden = true;
@@ -614,6 +628,24 @@
             ? formatAgo(session.startedAt)
             : '—';
       meta.appendChild(el('span', null, elapsed));
+      // Resilience: how hard this run has had to work, and when it last moved.
+      // A card that only shows a percentage cannot tell a run that is
+      // progressing from one that has been retrying for twenty minutes.
+      if (typeof session.retries === 'number' && session.retries > 0) {
+        meta.appendChild(el('span', null, session.retries + ' retry(s)'));
+      }
+      if (typeof session.correctionCycle === 'number' && session.correctionCycle > 0) {
+        meta.appendChild(el('span', null, 'correção ' + session.correctionCycle));
+      }
+      if (session.updatedAt) {
+        meta.appendChild(
+          el('span', null, 'atividade ' + formatAgo(session.lastActivityAt || session.updatedAt)),
+        );
+      }
+      if (session.provider) meta.appendChild(el('span', null, 'provider ' + session.provider));
+      if (typeof session.attempt === 'number' && session.attempt > 0) {
+        meta.appendChild(el('span', null, 'tentativa ' + session.attempt));
+      }
       card.appendChild(meta);
 
       const progress = el('span', 'dashboard-progress');
@@ -656,6 +688,7 @@
     renderRepository(snapshot);
     renderProgress(snapshot);
     renderNow(snapshot);
+    renderResilience(snapshot);
     renderPhases(snapshot);
     renderNextSteps(snapshot);
     renderStories(snapshot);
@@ -664,6 +697,7 @@
     renderDrawer(snapshot);
     renderGit(snapshot);
     renderLogs(snapshot);
+    renderHistory();
     renderMeta(snapshot);
     renderTimers();
     renderTitle(snapshot);
@@ -845,6 +879,26 @@
     els.now.appendChild(grid);
   }
 
+  function renderResilience(snapshot) {
+    clear(els.resilience);
+    const resilience = snapshot.resilience || {};
+    const grid = el('dl', 'now-grid');
+    nowRow(grid, 'Tentativa', resilience.attempt > 0 ? String(resilience.attempt) : '—');
+    nowRow(grid, 'Provider', resilience.provider || '—');
+    nowRow(grid, 'Modelo', resilience.model || '—');
+    nowRow(grid, 'Última falha', resilience.lastFailureKind || '—');
+    nowRow(
+      grid,
+      'Cooldown',
+      resilience.cooldownUntil ? formatClock(resilience.cooldownUntil) : '—',
+    );
+    const activity = resilience.lastActivityAt
+      ? formatClock(resilience.lastActivityAt) + ' (' + formatAgo(resilience.lastActivityAt) + ')'
+      : '—';
+    nowRow(grid, 'Última atividade', activity);
+    els.resilience.appendChild(grid);
+  }
+
   function renderPhases(snapshot) {
     clear(els.phases);
     if (snapshot.phases.length === 0) {
@@ -1135,6 +1189,70 @@
     }
   }
 
+  const RESILIENCE_EVENTS = new Set([
+    'retry',
+    'agent:attempt',
+    'agent:activity',
+    'agent:result',
+    'failover',
+  ]);
+
+  function historyMessage(event) {
+    switch (event.type) {
+      case 'session:start':
+        return 'Sessão iniciada';
+      case 'session:end':
+        return 'Sessão encerrada: ' + event.status;
+      case 'phase:start':
+        return 'Fase iniciada: ' + event.phase;
+      case 'phase:end':
+        return 'Fase encerrada: ' + event.phase + (event.success ? ' (ok)' : ' (falhou)');
+      case 'iteration:start':
+        return 'Iteração ' + event.iteration + ' iniciada';
+      case 'iteration:end':
+        return 'Iteração ' + event.iteration + ' encerrada';
+      case 'retry':
+        return 'Retry ' + event.attempt + (event.kind ? ': ' + event.kind : '');
+      case 'agent:attempt':
+        return 'Tentativa ' + event.attempt + ' com ' + event.provider;
+      case 'agent:activity':
+        return 'Atividade recebida de ' + event.provider;
+      case 'agent:result':
+        return event.provider + (event.success ? ' concluiu a tentativa' : ' falhou: ' + (event.failureKind || 'desconhecida'));
+      case 'failover':
+        return 'Failover de ' + event.from + ' para ' + event.to + (event.reason ? ': ' + event.reason : '');
+      case 'correction:cycle':
+        return 'Ciclo de correção ' + event.cycle + '/' + event.maxCycles;
+      case 'log':
+        return event.message;
+      default:
+        return event.type;
+    }
+  }
+
+  function renderHistory() {
+    clear(els.history);
+    const entries = state.events.filter((entry) => {
+      const type = entry && entry.event && entry.event.type;
+      if (!type) return false;
+      if (state.historyFilter === 'resilience') return RESILIENCE_EVENTS.has(type);
+      if (state.historyFilter === 'pipeline') return !RESILIENCE_EVENTS.has(type);
+      return true;
+    });
+    if (entries.length === 0) {
+      els.history.appendChild(el('p', 'empty', 'Nenhum evento para exibir.'));
+      return;
+    }
+    for (const entry of entries.slice().reverse()) {
+      const event = entry.event;
+      const item = el('li', 'history-entry');
+      item.appendChild(el('span', 'log-time', formatClock(event.at)));
+      item.appendChild(el('span', 'history-type mono', event.type));
+      item.appendChild(el('span', 'history-message', historyMessage(event)));
+      els.history.appendChild(item);
+    }
+  }
+
   function renderMeta(snapshot) {
     const parts = [];
     if (snapshot.sessionId) parts.push('sessão ' + snapshot.sessionId);
@@ -1201,6 +1319,10 @@
       state.logFilter = els.logFilter.value;
       if (state.snapshot) renderLogs(state.snapshot);
     });
+    els.historyFilter.addEventListener('change', () => {
+      state.historyFilter = els.historyFilter.value;
+      renderHistory();
+    });
 
     document.addEventListener('visibilitychange', () => {
       if (document.hidden) clearTimer();

package/web/public/index.html

@@ -69,6 +69,8 @@
               aria-controls="panel-execution" aria-selected="true">Execução</button>
       <button type="button" class="tab" role="tab" id="tab-kanban"
               aria-controls="panel-kanban" aria-selected="false">Kanban</button>
+      <button type="button" class="tab" role="tab" id="tab-history"
+              aria-controls="panel-history" aria-selected="false">Histórico</button>
     </div>
 
     <div class="tab-panel" id="panel-execution" role="tabpanel" aria-labelledby="tab-execution" tabindex="0">
@@ -97,6 +99,11 @@
       <div id="now"></div>
     </section>
 
+    <section class="card" aria-live="polite">
+      <h2>Resiliência</h2>
+      <div id="resilience"></div>
+    </section>
+
     <div class="columns">
       <section class="card">
         <h2>Fases</h2>
@@ -148,6 +155,23 @@
       </section>
     </div>
 
+    <div class="tab-panel" id="panel-history" role="tabpanel" aria-labelledby="tab-history" tabindex="0" hidden>
+      <section class="card">
+        <div class="section-head">
+          <h2>Histórico da execução</h2>
+          <label class="log-filter-control">
+            Evento
+            <select id="history-filter" aria-label="Filtro do histórico">
+              <option value="all">todos</option>
+              <option value="resilience">resiliência</option>
+              <option value="pipeline">pipeline</option>
+            </select>
+          </label>
+        </div>
+        <ol id="history" class="plain-list history"></ol>
+      </section>
+    </div>
+
     <footer class="muted footer">
       <span id="session-meta"></span>
     </footer>

package/dist/analyze-XV3K25K3.js.map

@@ -1 +0,0 @@
-{"version":3,"sources":["../src/commands/analyze.ts"],"sourcesContent":["import { mkdir, readFile, writeFile } from 'node:fs/promises';\nimport { DEFAULT_HEADLESS_TIMEOUT_MS, runHeadless } from '../core/headless.js';\nimport { applyPlaceholders, loadPrompt } from '../core/prompt-resolver.js';\nimport { publishPhaseMetrics } from '../core/session-metrics.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 { resolvePolicyPlaceholders } from '../policy/placeholders.js';\nimport { resolveIssuePaths } from '../storage/resolve.js';\nimport { printError, printInfo, printSuccess } from '../ui/logger.js';\n\nexport async function runAnalyze(issue: string, resolvedIssue?: ResolvedIssue): Promise<number> {\n  const issueNumber = issue.replace(/^#/, '');\n  const paths = await resolveIssuePaths(issueNumber);\n  const analysisPath = paths.analysisFile;\n\n  const resolution = await resolveCommandIssue(issueNumber, resolvedIssue);\n  if (!resolution.ok) {\n    return resolution.code;\n  }\n\n  await mkdir(paths.issueDir, { recursive: true });\n\n  const template = await loadPrompt('analyze');\n  const prompt = applyPlaceholders(template, {\n    // The repository's own conventions. Empty when it declares none, which is\n    // what keeps the rendered prompt identical to the pre-policy one.\n    ...(await resolvePolicyPlaceholders()),\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 startedAtMs = Date.now();\n  const result = await runHeadless({\n    prompt,\n    maxTurns: 30,\n    timeout: getGlobalTimeout() ?? DEFAULT_HEADLESS_TIMEOUT_MS,\n    // json (not text) so the CLI reports usage: the envelope's `result` field\n    // carries the same assistant text this phase already consumed.\n    outputFormat: 'json',\n    allowedTools: ['Bash', 'Read', 'Glob', 'Grep', 'Write'],\n    addDirs: [paths.issueDir],\n    statusMessage: `Analyzing issue #${issueNumber}...`,\n  });\n  // Before the success check: the tokens were spent either way.\n  publishPhaseMetrics('analyze', result.cost, startedAtMs);\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 = paths.tasksFile;\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;AAY3C,eAAsB,WAAW,OAAe,eAAgD;AAC9F,QAAM,cAAc,MAAM,QAAQ,MAAM,EAAE;AAC1C,QAAM,QAAQ,MAAM,kBAAkB,WAAW;AACjD,QAAM,eAAe,MAAM;AAE3B,QAAM,aAAa,MAAM,oBAAoB,aAAa,aAAa;AACvE,MAAI,CAAC,WAAW,IAAI;AAClB,WAAO,WAAW;AAAA,EACpB;AAEA,QAAM,MAAM,MAAM,UAAU,EAAE,WAAW,KAAK,CAAC;AAE/C,QAAM,WAAW,MAAM,WAAW,SAAS;AAC3C,QAAM,SAAS,kBAAkB,UAAU;AAAA;AAAA;AAAA,IAGzC,GAAI,MAAM,0BAA0B;AAAA,IACpC,kBAAkB;AAAA,IAClB,mBAAmB;AAAA;AAAA;AAAA,IAGnB,GAAG,kBAAkB,WAAW,QAAQ;AAAA,EAC1C,CAAC;AAED,QAAM,cAAc,KAAK,IAAI;AAC7B,QAAM,SAAS,MAAM,YAAY;AAAA,IAC/B;AAAA,IACA,UAAU;AAAA,IACV,SAAS,iBAAiB,KAAK;AAAA;AAAA;AAAA,IAG/B,cAAc;AAAA,IACd,cAAc,CAAC,QAAQ,QAAQ,QAAQ,QAAQ,OAAO;AAAA,IACtD,SAAS,CAAC,MAAM,QAAQ;AAAA,IACxB,eAAe,oBAAoB,WAAW;AAAA,EAChD,CAAC;AAED,sBAAoB,WAAW,OAAO,MAAM,WAAW;AAEvD,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,MAAM;AACxB,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":[]}

package/dist/chunk-34WNTK3K.js.map

@@ -1 +0,0 @@
-{"version":3,"sources":["../src/commands/review.ts"],"sourcesContent":["import { DEFAULT_HEADLESS_TIMEOUT_MS, runHeadless } from '../core/headless.js';\nimport { applyPlaceholders, loadPrompt } from '../core/prompt-resolver.js';\nimport { publishPhaseMetrics } from '../core/session-metrics.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 { resolvePolicyPlaceholders } from '../policy/placeholders.js';\nimport { resolveIssuePaths } from '../storage/resolve.js';\nimport { printError, printSuccess } from '../ui/logger.js';\n\nexport interface ReviewResult {\n  status: 'PASS' | 'FAIL';\n  findings?: string;\n}\n\n/**\n * Parse the <review-result> block from headless output.\n */\nfunction parseReviewResult(output: string): ReviewResult {\n  const match = output.match(/<review-result>([\\s\\S]*?)<\\/review-result>/);\n  if (!match) {\n    // Try to detect PASS/FAIL from the raw output\n    if (/STATUS:\\s*PASS/i.test(output)) {\n      return { status: 'PASS' };\n    }\n    if (/STATUS:\\s*FAIL/i.test(output)) {\n      const findingsMatch = output.match(/FINDINGS:([\\s\\S]*?)(?:$|<\\/)/);\n      return { status: 'FAIL', findings: findingsMatch?.[1]?.trim() ?? 'Unknown findings' };\n    }\n    // Default to PASS if no explicit status found and no errors\n    return { status: 'PASS' };\n  }\n\n  const block = match[1];\n  if (/STATUS:\\s*PASS/i.test(block)) {\n    return { status: 'PASS' };\n  }\n\n  const findingsMatch = block.match(/FINDINGS:([\\s\\S]*)/);\n  return {\n    status: 'FAIL',\n    findings: findingsMatch?.[1]?.trim() ?? 'Unknown findings',\n  };\n}\n\nexport async function runReview(issue: string, resolvedIssue?: ResolvedIssue): Promise<number> {\n  const issueNumber = issue.replace(/^#/, '');\n  const paths = await resolveIssuePaths(issueNumber);\n  const tasksPath = paths.tasksFile;\n\n  const resolution = await resolveCommandIssue(issueNumber, resolvedIssue);\n  if (!resolution.ok) {\n    return resolution.code;\n  }\n\n  const template = await loadPrompt('review');\n  const prompt = applyPlaceholders(template, {\n    // The repository's own conventions. Empty when it declares none, which is\n    // what keeps the rendered prompt identical to the pre-policy one.\n    ...(await resolvePolicyPlaceholders()),\n    __ISSUE_NUMBER__: issueNumber,\n    __TASKS_PATH__: tasksPath,\n    ...issuePlaceholders(resolution.resolved),\n  });\n\n  const startedAtMs = Date.now();\n  const result = await runHeadless({\n    prompt,\n    maxTurns: 25,\n    timeout: getGlobalTimeout() ?? DEFAULT_HEADLESS_TIMEOUT_MS,\n    // json (not text) so the CLI reports usage: the envelope's `result` field\n    // carries the same assistant text parseReviewResult() already consumed.\n    outputFormat: 'json',\n    allowedTools: ['Bash', 'Read', 'Glob', 'Grep'],\n    addDirs: [paths.issueDir],\n    statusMessage: `Reviewing issue #${issueNumber} resolution...`,\n  });\n  // Before the success check: the tokens were spent either way. A correction\n  // cycle calls runReview() again and the reducer sums both invocations.\n  publishPhaseMetrics('review', result.cost, startedAtMs);\n\n  if (!result.success) {\n    printError(`Review failed: ${result.error}`);\n    return 1;\n  }\n\n  const review = parseReviewResult(result.result);\n\n  if (review.status === 'PASS') {\n    // Update pipeline state\n    try {\n      const plan = await loadTaskPlan(tasksPath);\n      plan.pipeline.reviewCompleted = true;\n      plan.lastReviewFindings = null;\n      await saveTaskPlan(tasksPath, plan);\n    } catch {\n      // tasks.json may not exist\n    }\n\n    printSuccess('Review PASSED — implementation meets acceptance criteria');\n    return 0;\n  }\n\n  printError('Review FAILED — issues found:');\n  if (review.findings) {\n    console.log(review.findings);\n  }\n\n  // Persist the findings so a correction cycle can act on them: without this,\n  // a re-run of execute sees every userStories[].passes already true and\n  // exits immediately without attempting any fix (see engine.ts's\n  // \"already complete\" guards).\n  try {\n    const plan = await loadTaskPlan(tasksPath);\n    plan.pipeline.reviewCompleted = false;\n    plan.lastReviewFindings = review.findings ?? 'Unknown findings';\n    await saveTaskPlan(tasksPath, plan);\n  } catch {\n    // tasks.json may not exist\n  }\n\n  return 1;\n}\n"],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAmBA,SAAS,kBAAkB,QAA8B;AACvD,QAAM,QAAQ,OAAO,MAAM,4CAA4C;AACvE,MAAI,CAAC,OAAO;AAEV,QAAI,kBAAkB,KAAK,MAAM,GAAG;AAClC,aAAO,EAAE,QAAQ,OAAO;AAAA,IAC1B;AACA,QAAI,kBAAkB,KAAK,MAAM,GAAG;AAClC,YAAMA,iBAAgB,OAAO,MAAM,8BAA8B;AACjE,aAAO,EAAE,QAAQ,QAAQ,UAAUA,iBAAgB,CAAC,GAAG,KAAK,KAAK,mBAAmB;AAAA,IACtF;AAEA,WAAO,EAAE,QAAQ,OAAO;AAAA,EAC1B;AAEA,QAAM,QAAQ,MAAM,CAAC;AACrB,MAAI,kBAAkB,KAAK,KAAK,GAAG;AACjC,WAAO,EAAE,QAAQ,OAAO;AAAA,EAC1B;AAEA,QAAM,gBAAgB,MAAM,MAAM,oBAAoB;AACtD,SAAO;AAAA,IACL,QAAQ;AAAA,IACR,UAAU,gBAAgB,CAAC,GAAG,KAAK,KAAK;AAAA,EAC1C;AACF;AAEA,eAAsB,UAAU,OAAe,eAAgD;AAC7F,QAAM,cAAc,MAAM,QAAQ,MAAM,EAAE;AAC1C,QAAM,QAAQ,MAAM,kBAAkB,WAAW;AACjD,QAAM,YAAY,MAAM;AAExB,QAAM,aAAa,MAAM,oBAAoB,aAAa,aAAa;AACvE,MAAI,CAAC,WAAW,IAAI;AAClB,WAAO,WAAW;AAAA,EACpB;AAEA,QAAM,WAAW,MAAM,WAAW,QAAQ;AAC1C,QAAM,SAAS,kBAAkB,UAAU;AAAA;AAAA;AAAA,IAGzC,GAAI,MAAM,0BAA0B;AAAA,IACpC,kBAAkB;AAAA,IAClB,gBAAgB;AAAA,IAChB,GAAG,kBAAkB,WAAW,QAAQ;AAAA,EAC1C,CAAC;AAED,QAAM,cAAc,KAAK,IAAI;AAC7B,QAAM,SAAS,MAAM,YAAY;AAAA,IAC/B;AAAA,IACA,UAAU;AAAA,IACV,SAAS,iBAAiB,KAAK;AAAA;AAAA;AAAA,IAG/B,cAAc;AAAA,IACd,cAAc,CAAC,QAAQ,QAAQ,QAAQ,MAAM;AAAA,IAC7C,SAAS,CAAC,MAAM,QAAQ;AAAA,IACxB,eAAe,oBAAoB,WAAW;AAAA,EAChD,CAAC;AAGD,sBAAoB,UAAU,OAAO,MAAM,WAAW;AAEtD,MAAI,CAAC,OAAO,SAAS;AACnB,eAAW,kBAAkB,OAAO,KAAK,EAAE;AAC3C,WAAO;AAAA,EACT;AAEA,QAAM,SAAS,kBAAkB,OAAO,MAAM;AAE9C,MAAI,OAAO,WAAW,QAAQ;AAE5B,QAAI;AACF,YAAM,OAAO,MAAM,aAAa,SAAS;AACzC,WAAK,SAAS,kBAAkB;AAChC,WAAK,qBAAqB;AAC1B,YAAM,aAAa,WAAW,IAAI;AAAA,IACpC,QAAQ;AAAA,IAER;AAEA,iBAAa,+DAA0D;AACvE,WAAO;AAAA,EACT;AAEA,aAAW,oCAA+B;AAC1C,MAAI,OAAO,UAAU;AACnB,YAAQ,IAAI,OAAO,QAAQ;AAAA,EAC7B;AAMA,MAAI;AACF,UAAM,OAAO,MAAM,aAAa,SAAS;AACzC,SAAK,SAAS,kBAAkB;AAChC,SAAK,qBAAqB,OAAO,YAAY;AAC7C,UAAM,aAAa,WAAW,IAAI;AAAA,EACpC,QAAQ;AAAA,EAER;AAEA,SAAO;AACT;","names":["findingsMatch"]}

package/dist/chunk-3BOHTH4V.js

@@ -1,58 +0,0 @@
-#!/usr/bin/env node
-import {
-  retryDelaySeconds,
-  sleep
-} from "./chunk-HDV3YDTG.js";
-import {
-  getSessionPublisher,
-  isoNow,
-  printRetry
-} from "./chunk-HC2MT7BX.js";
-
-// src/core/phase-runner.ts
-import { readFile } from "fs/promises";
-var DEFAULT_PHASE_RETRY_LIMIT = 3;
-var DEFAULT_BACKOFF_BASE_SECONDS = 15;
-var DEFAULT_BACKOFF_MAX_SECONDS = 120;
-async function runPhaseWithRetry(options) {
-  const retryLimit = options.retryLimit ?? DEFAULT_PHASE_RETRY_LIMIT;
-  const backoffBaseSeconds = options.backoffBaseSeconds ?? DEFAULT_BACKOFF_BASE_SECONDS;
-  const backoffMaxSeconds = options.backoffMaxSeconds ?? DEFAULT_BACKOFF_MAX_SECONDS;
-  let lastResult = { ok: false, error: "Phase never attempted" };
-  let retryCount = 0;
-  for (let attemptNumber = 1; attemptNumber <= retryLimit; attemptNumber++) {
-    lastResult = await options.attempt(attemptNumber);
-    if (lastResult.ok) return lastResult;
-    if (!lastResult.transient || attemptNumber === retryLimit) return lastResult;
-    retryCount++;
-    const delaySeconds = retryDelaySeconds(retryCount, backoffBaseSeconds, backoffMaxSeconds);
-    getSessionPublisher().publish({
-      type: "retry",
-      at: isoNow(),
-      attempt: retryCount,
-      delaySeconds,
-      reason: lastResult.error
-    });
-    printRetry(
-      `Phase ${options.phase} hit a recoverable failure (attempt ${attemptNumber}/${retryLimit}): ${lastResult.error}. Retrying in ${delaySeconds}s.`
-    );
-    await sleep(delaySeconds);
-  }
-  return lastResult;
-}
-async function readFileWithGrace(path, delaysMs = [300, 800]) {
-  for (let i = 0; ; i++) {
-    try {
-      return await readFile(path, "utf-8");
-    } catch (err) {
-      if (i >= delaysMs.length) throw err;
-      await new Promise((resolve) => setTimeout(resolve, delaysMs[i]));
-    }
-  }
-}
-
-export {
-  runPhaseWithRetry,
-  readFileWithGrace
-};
-//# sourceMappingURL=chunk-3BOHTH4V.js.map

package/dist/chunk-3BOHTH4V.js.map

@@ -1 +0,0 @@
-{"version":3,"sources":["../src/core/phase-runner.ts"],"sourcesContent":["import { readFile } from 'node:fs/promises';\nimport { printRetry } from '../ui/logger.js';\nimport { retryDelaySeconds, sleep } from '../utils/retry.js';\nimport { getSessionPublisher } from './session-publisher.js';\nimport { isoNow } from './state-manager.js';\n\nexport interface PhaseAttemptResult {\n  ok: boolean;\n  /** Whether a failed attempt is worth retrying. Ignored when ok is true. */\n  transient?: boolean;\n  error?: string;\n}\n\nexport interface PhaseRetryOptions {\n  phase: string;\n  attempt: (attemptNumber: number) => Promise<PhaseAttemptResult>;\n  retryLimit?: number;\n  backoffBaseSeconds?: number;\n  backoffMaxSeconds?: number;\n}\n\nconst DEFAULT_PHASE_RETRY_LIMIT = 3;\nconst DEFAULT_BACKOFF_BASE_SECONDS = 15;\nconst DEFAULT_BACKOFF_MAX_SECONDS = 120;\n\n/**\n * Bounded retry wrapper for single-shot phases (prd, plan, pr) that invoke\n * the Claude CLI once and validate an artifact. Mirrors the transient-retry\n * pattern already used by the execute phase's iteration loop (core/engine.ts:\n * isTransientFailure + retryDelaySeconds), applied here to phases that\n * previously aborted the whole pipeline on the first failure with no chance\n * to recover. Non-transient failures (or exhausting the retry budget) still\n * end the phase — this bounds recovery attempts, it never retries forever.\n */\nexport async function runPhaseWithRetry(options: PhaseRetryOptions): Promise<PhaseAttemptResult> {\n  const retryLimit = options.retryLimit ?? DEFAULT_PHASE_RETRY_LIMIT;\n  const backoffBaseSeconds = options.backoffBaseSeconds ?? DEFAULT_BACKOFF_BASE_SECONDS;\n  const backoffMaxSeconds = options.backoffMaxSeconds ?? DEFAULT_BACKOFF_MAX_SECONDS;\n\n  let lastResult: PhaseAttemptResult = { ok: false, error: 'Phase never attempted' };\n  let retryCount = 0;\n\n  for (let attemptNumber = 1; attemptNumber <= retryLimit; attemptNumber++) {\n    lastResult = await options.attempt(attemptNumber);\n    if (lastResult.ok) return lastResult;\n    if (!lastResult.transient || attemptNumber === retryLimit) return lastResult;\n\n    retryCount++;\n    const delaySeconds = retryDelaySeconds(retryCount, backoffBaseSeconds, backoffMaxSeconds);\n    getSessionPublisher().publish({\n      type: 'retry',\n      at: isoNow(),\n      attempt: retryCount,\n      delaySeconds,\n      reason: lastResult.error,\n    });\n    printRetry(\n      `Phase ${options.phase} hit a recoverable failure (attempt ${attemptNumber}/${retryLimit}): ${lastResult.error}. Retrying in ${delaySeconds}s.`,\n    );\n    await sleep(delaySeconds);\n  }\n\n  return lastResult;\n}\n\n/**\n * Re-read a file a few times with short delays before giving up. Absorbs a\n * brief filesystem-visibility lag between the Claude CLI's Write tool\n * completing and this process observing the result, without spending a full\n * phase retry (and a new Claude invocation) on what may just be a timing gap.\n */\nexport async function readFileWithGrace(\n  path: string,\n  delaysMs: number[] = [300, 800],\n): Promise<string> {\n  for (let i = 0; ; i++) {\n    try {\n      return await readFile(path, 'utf-8');\n    } catch (err) {\n      if (i >= delaysMs.length) throw err;\n      await new Promise((resolve) => setTimeout(resolve, delaysMs[i]));\n    }\n  }\n}\n"],"mappings":";;;;;;;;;;;;AAAA,SAAS,gBAAgB;AAqBzB,IAAM,4BAA4B;AAClC,IAAM,+BAA+B;AACrC,IAAM,8BAA8B;AAWpC,eAAsB,kBAAkB,SAAyD;AAC/F,QAAM,aAAa,QAAQ,cAAc;AACzC,QAAM,qBAAqB,QAAQ,sBAAsB;AACzD,QAAM,oBAAoB,QAAQ,qBAAqB;AAEvD,MAAI,aAAiC,EAAE,IAAI,OAAO,OAAO,wBAAwB;AACjF,MAAI,aAAa;AAEjB,WAAS,gBAAgB,GAAG,iBAAiB,YAAY,iBAAiB;AACxE,iBAAa,MAAM,QAAQ,QAAQ,aAAa;AAChD,QAAI,WAAW,GAAI,QAAO;AAC1B,QAAI,CAAC,WAAW,aAAa,kBAAkB,WAAY,QAAO;AAElE;AACA,UAAM,eAAe,kBAAkB,YAAY,oBAAoB,iBAAiB;AACxF,wBAAoB,EAAE,QAAQ;AAAA,MAC5B,MAAM;AAAA,MACN,IAAI,OAAO;AAAA,MACX,SAAS;AAAA,MACT;AAAA,MACA,QAAQ,WAAW;AAAA,IACrB,CAAC;AACD;AAAA,MACE,SAAS,QAAQ,KAAK,uCAAuC,aAAa,IAAI,UAAU,MAAM,WAAW,KAAK,iBAAiB,YAAY;AAAA,IAC7I;AACA,UAAM,MAAM,YAAY;AAAA,EAC1B;AAEA,SAAO;AACT;AAQA,eAAsB,kBACpB,MACA,WAAqB,CAAC,KAAK,GAAG,GACb;AACjB,WAAS,IAAI,KAAK,KAAK;AACrB,QAAI;AACF,aAAO,MAAM,SAAS,MAAM,OAAO;AAAA,IACrC,SAAS,KAAK;AACZ,UAAI,KAAK,SAAS,OAAQ,OAAM;AAChC,YAAM,IAAI,QAAQ,CAAC,YAAY,WAAW,SAAS,SAAS,CAAC,CAAC,CAAC;AAAA,IACjE;AAAA,EACF;AACF;","names":[]}