sequant 1.20.3 → 2.0.0

package/dist/src/lib/workflow/metrics-schema.d.ts

@@ -21,19 +21,19 @@ import { z } from "zod";
  */
 export declare const RunOutcomeSchema: z.ZodEnum<{
     success: "success";
-    failed: "failed";
     partial: "partial";
+    failed: "failed";
 }>;
 export type RunOutcome = z.infer<typeof RunOutcomeSchema>;
 /**
  * Available phases (aligned with run-log-schema.ts)
  */
 export declare const MetricPhaseSchema: z.ZodEnum<{
+    qa: "qa";
     loop: "loop";
     spec: "spec";
-    exec: "exec";
-    qa: "qa";
     "security-review": "security-review";
+    exec: "exec";
     testgen: "testgen";
     test: "test";
 }>;
@@ -67,18 +67,18 @@ export declare const MetricRunSchema: z.ZodObject<{
     date: z.ZodString;
     issues: z.ZodArray<z.ZodNumber>;
     phases: z.ZodArray<z.ZodEnum<{
+        qa: "qa";
         loop: "loop";
         spec: "spec";
-        exec: "exec";
-        qa: "qa";
         "security-review": "security-review";
+        exec: "exec";
         testgen: "testgen";
         test: "test";
     }>>;
     outcome: z.ZodEnum<{
         success: "success";
-        failed: "failed";
         partial: "partial";
+        failed: "failed";
     }>;
     duration: z.ZodNumber;
     model: z.ZodString;
@@ -107,18 +107,18 @@ export declare const MetricsSchema: z.ZodObject<{
         date: z.ZodString;
         issues: z.ZodArray<z.ZodNumber>;
         phases: z.ZodArray<z.ZodEnum<{
+            qa: "qa";
             loop: "loop";
             spec: "spec";
-            exec: "exec";
-            qa: "qa";
             "security-review": "security-review";
+            exec: "exec";
             testgen: "testgen";
             test: "test";
         }>>;
         outcome: z.ZodEnum<{
             success: "success";
-            failed: "failed";
             partial: "partial";
+            failed: "failed";
         }>;
         duration: z.ZodNumber;
         model: z.ZodString;

package/dist/src/lib/workflow/phase-detection.d.ts

@@ -86,6 +86,29 @@ export declare function getResumablePhases(requestedPhases: readonly string[], c
 export declare function isPhaseCompletedOrPast(targetPhase: Phase, comments: {
     body: string;
 }[]): boolean;
+/**
+ * Result of detecting prior QA findings from issue comments.
+ */
+export interface PriorQAFindings {
+    /** The commit SHA from the last QA phase marker (if present) */
+    commitSHA: string | null;
+    /** ISO 8601 timestamp of the last QA completion */
+    timestamp: string;
+    /** Status of the last QA run */
+    status: string;
+}
+/**
+ * Detect the most recent QA phase marker from issue comments.
+ *
+ * Extracts the commit SHA and metadata from the latest QA phase marker,
+ * enabling incremental re-runs by identifying what changed since last QA.
+ *
+ * @param comments - Array of objects with a `body` string field
+ * @returns Prior QA findings, or null if no QA phase markers found
+ */
+export declare function detectPriorQAFindings(comments: {
+    body: string;
+}[]): PriorQAFindings | null;
 /**
  * Get the current phase status for an issue from GitHub comments.
  *

package/dist/src/lib/workflow/phase-detection.js

@@ -10,7 +10,7 @@
  * <!-- SEQUANT_PHASE: {"phase":"exec","status":"completed","timestamp":"..."} -->
  * ```
  */
-import { execSync } from "child_process";
+import { GitHubProvider } from "./platforms/github.js";
 import { PhaseMarkerSchema, WORKFLOW_PHASES, } from "./state-schema.js";
 /** Regex to extract phase marker JSON from HTML comments */
 const PHASE_MARKER_REGEX = /<!-- SEQUANT_PHASE: (\{[^}]+\}) -->/g;
@@ -178,6 +178,37 @@ export function isPhaseCompletedOrPast(targetPhase, comments) {
     }
     return false;
 }
+/**
+ * Detect the most recent QA phase marker from issue comments.
+ *
+ * Extracts the commit SHA and metadata from the latest QA phase marker,
+ * enabling incremental re-runs by identifying what changed since last QA.
+ *
+ * @param comments - Array of objects with a `body` string field
+ * @returns Prior QA findings, or null if no QA phase markers found
+ */
+export function detectPriorQAFindings(comments) {
+    const allMarkers = [];
+    for (const comment of comments) {
+        const markers = parsePhaseMarkers(comment.body);
+        for (const marker of markers) {
+            if (marker.phase === "qa") {
+                allMarkers.push(marker);
+            }
+        }
+    }
+    if (allMarkers.length === 0) {
+        return null;
+    }
+    // Sort by timestamp descending, return latest
+    allMarkers.sort((a, b) => new Date(b.timestamp).getTime() - new Date(a.timestamp).getTime());
+    const latest = allMarkers[0];
+    return {
+        commitSHA: latest.commitSHA ?? null,
+        timestamp: latest.timestamp,
+        status: latest.status,
+    };
+}
 /**
  * Get the current phase status for an issue from GitHub comments.
  *
@@ -187,16 +218,12 @@ export function isPhaseCompletedOrPast(targetPhase, comments) {
  * @returns Latest phase marker, or null if no markers found or on error
  */
 export function getIssuePhase(issueNumber) {
-    try {
-        const output = execSync(`gh issue view ${issueNumber} --json comments --jq '[.comments[].body]'`, { encoding: "utf-8", stdio: ["pipe", "pipe", "pipe"] });
-        const bodies = JSON.parse(output);
-        const comments = bodies.map((body) => ({ body }));
-        return detectPhaseFromComments(comments);
-    }
-    catch {
-        // GitHub CLI failure — fall through to normal execution
+    const github = new GitHubProvider();
+    const bodies = github.fetchIssueCommentBodiesSync(String(issueNumber));
+    if (bodies.length === 0)
         return null;
-    }
+    const comments = bodies.map((body) => ({ body }));
+    return detectPhaseFromComments(comments);
 }
 /**
  * Get completed phases for an issue from GitHub comments.
@@ -207,15 +234,10 @@ export function getIssuePhase(issueNumber) {
  * @returns Array of completed phase names, or empty array on error
  */
 export function getCompletedPhases(issueNumber) {
-    try {
-        const output = execSync(`gh issue view ${issueNumber} --json comments --jq '[.comments[].body]'`, { encoding: "utf-8", stdio: ["pipe", "pipe", "pipe"] });
-        const bodies = JSON.parse(output);
-        const comments = bodies.map((body) => ({ body }));
-        return getCompletedPhasesFromComments(comments);
-    }
-    catch {
-        return [];
-    }
+    const github = new GitHubProvider();
+    const bodies = github.fetchIssueCommentBodiesSync(String(issueNumber));
+    const comments = bodies.map((body) => ({ body }));
+    return getCompletedPhasesFromComments(comments);
 }
 /**
  * Get resumable phases for an issue from GitHub comments.
@@ -228,14 +250,8 @@ export function getCompletedPhases(issueNumber) {
  * @returns Filtered phases that still need to run
  */
 export function getResumablePhasesForIssue(issueNumber, requestedPhases) {
-    try {
-        const output = execSync(`gh issue view ${issueNumber} --json comments --jq '[.comments[].body]'`, { encoding: "utf-8", stdio: ["pipe", "pipe", "pipe"] });
-        const bodies = JSON.parse(output);
-        const comments = bodies.map((body) => ({ body }));
-        return getResumablePhases(requestedPhases, comments);
-    }
-    catch {
-        // On error, return all phases (no filtering)
-        return [...requestedPhases];
-    }
+    const github = new GitHubProvider();
+    const bodies = github.fetchIssueCommentBodiesSync(String(issueNumber));
+    const comments = bodies.map((body) => ({ body }));
+    return getResumablePhases(requestedPhases, comments);
 }

package/dist/src/lib/workflow/phase-executor.d.ts

@@ -1,19 +1,56 @@
 /**
  * Phase execution engine for workflow orchestration.
  *
- * Handles executing individual phases via the Claude Agent SDK,
+ * Handles executing individual phases via an AgentDriver interface,
  * including cold-start retry logic and MCP fallback strategies.
+ *
+ * The SDK import has been moved to ClaudeCodeDriver — this module
+ * is agent-agnostic.
  */
 import { ShutdownManager } from "../shutdown.js";
 import { PhaseSpinner } from "../phase-spinner.js";
 import { Phase, ExecutionConfig, PhaseResult, QaVerdict } from "./types.js";
+import type { QaSummary } from "./run-log-schema.js";
+/**
+ * Spec-specific retry configuration.
+ * Spec failures have a higher failure rate (~8.6%) than other phases due to
+ * transient GitHub API issues and rate limits. One extra retry with backoff
+ * recovers most of these without user intervention.
+ */
+/** @internal Exported for testing only */
+export declare const SPEC_RETRY_BACKOFF_MS = 5000;
+/** @internal Exported for testing only */
+export declare const SPEC_EXTRA_RETRIES = 1;
 export declare function parseQaVerdict(output: string): QaVerdict | null;
+/**
+ * Parse condensed QA summary from QA phase output (#434).
+ *
+ * Handles multiple AC table formats produced by the QA skill:
+ * - 5-column: | AC-N | source | desc | STATUS | notes |
+ * - 4-column: | AC-N | desc | STATUS | notes |
+ * - 3-column: | AC-N | desc | STATUS |
+ *
+ * Status cells may contain emoji prefixes (✅ MET), shorthand
+ * (PARTIAL), or trailing text (MET — explanation).
+ *
+ * @internal Exported for testing only
+ */
+export declare function parseQaSummary(output: string): QaSummary | null;
 /**
  * Format duration in human-readable format
  */
 export declare function formatDuration(seconds: number): string;
 /**
- * Execute a single phase for an issue using Claude Agent SDK
+ * Get the prompt for a phase with the issue number substituted.
+ * Selects self-contained prompts for non-Claude agents.
+ * Includes AGENTS.md content as context so non-Claude agents
+ * receive project conventions and workflow instructions.
+ *
+ * @internal Exported for testing only
+ */
+export declare function getPhasePrompt(phase: Phase, issueNumber: number, agent?: string): Promise<string>;
+/**
+ * Execute a single phase for an issue using the configured AgentDriver.
  */
 declare function executePhase(issueNumber: number, phase: Phase, config: ExecutionConfig, sessionId?: string, worktreePath?: string, shutdownManager?: ShutdownManager, spinner?: PhaseSpinner): Promise<PhaseResult & {
     sessionId?: string;
@@ -34,7 +71,9 @@ declare function executePhase(issueNumber: number, phase: Phase, config: Executi
  */
 export declare function executePhaseWithRetry(issueNumber: number, phase: Phase, config: ExecutionConfig, sessionId?: string, worktreePath?: string, shutdownManager?: ShutdownManager, spinner?: PhaseSpinner, 
 /** @internal Injected for testing — defaults to module-level executePhase */
-executePhaseFn?: typeof executePhase): Promise<PhaseResult & {
+executePhaseFn?: typeof executePhase, 
+/** @internal Injected for testing — defaults to setTimeout-based delay */
+delayFn?: (ms: number) => Promise<void>): Promise<PhaseResult & {
     sessionId?: string;
 }>;
 export {};