@intrect/openswarm 0.9.3 → 0.10.1

package/dist/agents/multiLensReview.js

@@ -0,0 +1,148 @@
+// ============================================
+// OpenSwarm - Multi-Lens Reviewer Fan-out (INT-2230)
+// ============================================
+//
+// PoC: verify one worker result through several independent review "lenses" in
+// parallel, then merge into a single ReviewResult. Each lens is just a focused
+// prompt injected into `taskDescription` — no extra adapter/infra needed — so the
+// existing reviewer agent runs N times, each told to concentrate on one concern
+// (correctness / security / regression-risk). The merge takes the WORST decision
+// and unions the issues/suggestions/recommendedActions so nothing a lens caught
+// is dropped. Gated + opt-in (config.review.multiLens.enabled, default false) to
+// avoid multiplying usage on every task.
+import { runReviewer } from './reviewer.js';
+import { runPool } from '../support/concurrencyPool.js';
+import { RateLimitError } from '../adapters/rateLimitError.js';
+/** The three default lenses a fan-out review runs in parallel. */
+export const REVIEW_LENSES = [
+    {
+        key: 'correctness',
+        focus: 'logic errors, unhandled edge cases, off-by-one, wrong assumptions, error handling',
+    },
+    {
+        key: 'security',
+        focus: 'injection, unsafe input, leaked secrets/keys, auth gaps, unsafe deserialization',
+    },
+    {
+        key: 'regression-risk',
+        focus: 'breaks existing behavior, changes a shared contract, missing/!updated tests, side effects on callers',
+    },
+];
+/**
+ * Append a lens directive to the base task description so the reviewer
+ * concentrates on one concern. Other concerns are explicitly delegated to the
+ * sibling lenses so a single reviewer doesn't try to cover everything.
+ */
+export function buildLensTaskDescription(base, lens) {
+    return `${base}\n\n## Review lens: ${lens.key}\nFocus your review specifically on: ${lens.focus}. Other concerns are secondary — another reviewer covers them.`;
+}
+// Merge
+/** Severity ordering: reject is worst, approve is best. */
+const DECISION_RANK = {
+    approve: 0,
+    revise: 1,
+    reject: 2,
+};
+/** First non-empty line of a feedback blob, trimmed (one-line lens summary). */
+function firstLine(text) {
+    if (!text)
+        return '';
+    for (const line of text.split('\n')) {
+        const trimmed = line.trim();
+        if (trimmed)
+            return trimmed;
+    }
+    return '';
+}
+/** Stable dedup of strings by trimmed/lowercased value, keeping first original. */
+function dedupStrings(values) {
+    const seen = new Set();
+    const out = [];
+    for (const v of values) {
+        const key = v.trim().toLowerCase();
+        if (!key || seen.has(key))
+            continue;
+        seen.add(key);
+        out.push(v);
+    }
+    return out;
+}
+/**
+ * Combine several lens reviews into one verdict:
+ * - decision = the WORST across lenses (reject > revise > approve)
+ * - issues / suggestions / recommendedActions = deduped union
+ * - feedback = one line per lens result, joined
+ * Empty input yields a clean approve (nothing flagged).
+ */
+export function mergeReviewResults(results) {
+    if (results.length === 0) {
+        return { decision: 'approve', feedback: '', issues: [], suggestions: [], recommendedActions: [] };
+    }
+    let decision = 'approve';
+    for (const r of results) {
+        if (DECISION_RANK[r.decision] > DECISION_RANK[decision])
+            decision = r.decision;
+    }
+    const issues = dedupStrings(results.flatMap((r) => r.issues ?? []));
+    const suggestions = dedupStrings(results.flatMap((r) => r.suggestions ?? []));
+    const actionSeen = new Set();
+    const recommendedActions = [];
+    for (const action of results.flatMap((r) => r.recommendedActions ?? [])) {
+        const key = `${action.type}|${action.location ?? ''}|${action.title}`;
+        if (actionSeen.has(key))
+            continue;
+        actionSeen.add(key);
+        recommendedActions.push(action);
+    }
+    const feedback = results
+        .map((r) => firstLine(r.feedback))
+        .filter(Boolean)
+        .join('\n');
+    return { decision, feedback, issues, suggestions, recommendedActions };
+}
+// Gating
+/**
+ * Decide whether a task is worth the extra cost of multi-lens fan-out. Triggers
+ * on a wide change surface, high priority (1=Urgent, 2=High), or an explicit
+ * `deep-review` label. Threshold defaults to 3 changed files.
+ */
+export function shouldFanoutReview(task, opts) {
+    const fileThreshold = opts?.fileThreshold ?? 3;
+    const fileCount = task.filesChanged?.length ?? 0;
+    if (fileCount >= fileThreshold)
+        return true;
+    if (task.priority != null && task.priority <= 2)
+        return true;
+    return !!task.labels?.includes('deep-review');
+}
+// Fan-out
+/**
+ * Run every lens in parallel and merge. Each lens calls the reviewer with a
+ * lens-scoped task description. A lens that fails for a non-rate-limit reason is
+ * skipped (its slot carries an error from the pool). A RateLimitError from ANY
+ * lens propagates so the pipeline's existing usage-limit handling / claude
+ * fallback can take over (INT-2192) instead of silently dropping a lens.
+ * `deps` is injectable for tests.
+ */
+export async function runMultiLensReview(options, deps) {
+    const review = deps?.review ?? runReviewer;
+    const lenses = deps?.lenses ?? REVIEW_LENSES;
+    const concurrency = deps?.concurrency ?? lenses.length;
+    const settled = await runPool(lenses, concurrency, async (lens) => {
+        const result = await review({
+            ...options,
+            taskDescription: buildLensTaskDescription(options.taskDescription, lens),
+        });
+        // Tag the feedback with the lens so the merged summary reads "[correctness] …".
+        return { ...result, feedback: `[${lens.key}] ${firstLine(result.feedback)}` };
+    });
+    // A rate limit on any lens means the quota is exhausted — propagate so the
+    // pipeline pauses / falls back rather than merging a partial review.
+    for (const s of settled) {
+        if (s.error instanceof RateLimitError)
+            throw s.error;
+    }
+    const ok = settled.filter((s) => s.value !== undefined).map((s) => s.value);
+    return mergeReviewResults(ok);
+}
+//# sourceMappingURL=multiLensReview.js.map

package/dist/agents/multiLensReview.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"multiLensReview.js","sourceRoot":"","sources":["../../src/agents/multiLensReview.ts"],"names":[],"mappings":"AAAA,+CAA+C;AAC/C,qDAAqD;AACrD,+CAA+C;AAC/C,EAAE;AACF,+EAA+E;AAC/E,+EAA+E;AAC/E,kFAAkF;AAClF,gFAAgF;AAChF,iFAAiF;AACjF,gFAAgF;AAChF,iFAAiF;AACjF,yCAAyC;AAIzC,OAAO,EAAE,WAAW,EAAE,MAAM,eAAe,CAAC;AAC5C,OAAO,EAAE,OAAO,EAAE,MAAM,+BAA+B,CAAC;AACxD,OAAO,EAAE,cAAc,EAAE,MAAM,+BAA+B,CAAC;AAU/D,kEAAkE;AAClE,MAAM,CAAC,MAAM,aAAa,GAAiB;IACzC;QACE,GAAG,EAAE,aAAa;QAClB,KAAK,EAAE,mFAAmF;KAC3F;IACD;QACE,GAAG,EAAE,UAAU;QACf,KAAK,EAAE,iFAAiF;KACzF;IACD;QACE,GAAG,EAAE,iBAAiB;QACtB,KAAK,EAAE,sGAAsG;KAC9G;CACF,CAAC;AAEF;;;;GAIG;AACH,MAAM,UAAU,wBAAwB,CAAC,IAAY,EAAE,IAAgB;IACrE,OAAO,GAAG,IAAI,uBAAuB,IAAI,CAAC,GAAG,wCAAwC,IAAI,CAAC,KAAK,gEAAgE,CAAC;AAClK,CAAC;AAED,QAAQ;AAER,2DAA2D;AAC3D,MAAM,aAAa,GAAmC;IACpD,OAAO,EAAE,CAAC;IACV,MAAM,EAAE,CAAC;IACT,MAAM,EAAE,CAAC;CACV,CAAC;AAEF,gFAAgF;AAChF,SAAS,SAAS,CAAC,IAAwB;IACzC,IAAI,CAAC,IAAI;QAAE,OAAO,EAAE,CAAC;IACrB,KAAK,MAAM,IAAI,IAAI,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC;QACpC,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,EAAE,CAAC;QAC5B,IAAI,OAAO;YAAE,OAAO,OAAO,CAAC;IAC9B,CAAC;IACD,OAAO,EAAE,CAAC;AACZ,CAAC;AAED,mFAAmF;AACnF,SAAS,YAAY,CAAC,MAAgB;IACpC,MAAM,IAAI,GAAG,IAAI,GAAG,EAAU,CAAC;IAC/B,MAAM,GAAG,GAAa,EAAE,CAAC;IACzB,KAAK,MAAM,CAAC,IAAI,MAAM,EAAE,CAAC;QACvB,MAAM,GAAG,GAAG,CAAC,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;QACnC,IAAI,CAAC,GAAG,IAAI,IAAI,CAAC,GAAG,CAAC,GAAG,CAAC;YAAE,SAAS;QACpC,IAAI,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;QACd,GAAG,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IACd,CAAC;IACD,OAAO,GAAG,CAAC;AACb,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,kBAAkB,CAAC,OAAuB;IACxD,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACzB,OAAO,EAAE,QAAQ,EAAE,SAAS,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,WAAW,EAAE,EAAE,EAAE,kBAAkB,EAAE,EAAE,EAAE,CAAC;IACpG,CAAC;IAED,IAAI,QAAQ,GAAmB,SAAS,CAAC;IACzC,KAAK,MAAM,CAAC,IAAI,OAAO,EAAE,CAAC;QACxB,IAAI,aAAa,CAAC,CAAC,CAAC,QAAQ,CAAC,GAAG,aAAa,CAAC,QAAQ,CAAC;YAAE,QAAQ,GAAG,CAAC,CAAC,QAAQ,CAAC;IACjF,CAAC;IAED,MAAM,MAAM,GAAG,YAAY,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,IAAI,EAAE,CAAC,CAAC,CAAC;IACpE,MAAM,WAAW,GAAG,YAAY,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,WAAW,IAAI,EAAE,CAAC,CAAC,CAAC;IAE9E,MAAM,UAAU,GAAG,IAAI,GAAG,EAAU,CAAC;IACrC,MAAM,kBAAkB,GAAoD,EAAE,CAAC;IAC/E,KAAK,MAAM,MAAM,IAAI,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,kBAAkB,IAAI,EAAE,CAAC,EAAE,CAAC;QACxE,MAAM,GAAG,GAAG,GAAG,MAAM,CAAC,IAAI,IAAI,MAAM,CAAC,QAAQ,IAAI,EAAE,IAAI,MAAM,CAAC,KAAK,EAAE,CAAC;QACtE,IAAI,UAAU,CAAC,GAAG,CAAC,GAAG,CAAC;YAAE,SAAS;QAClC,UAAU,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;QACpB,kBAAkB,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IAClC,CAAC;IAED,MAAM,QAAQ,GAAG,OAAO;SACrB,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,SAAS,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC;SACjC,MAAM,CAAC,OAAO,CAAC;SACf,IAAI,CAAC,IAAI,CAAC,CAAC;IAEd,OAAO,EAAE,QAAQ,EAAE,QAAQ,EAAE,MAAM,EAAE,WAAW,EAAE,kBAAkB,EAAE,CAAC;AACzE,CAAC;AAED,SAAS;AAET;;;;GAIG;AACH,MAAM,UAAU,kBAAkB,CAChC,IAAuE,EACvE,IAAiC;IAEjC,MAAM,aAAa,GAAG,IAAI,EAAE,aAAa,IAAI,CAAC,CAAC;IAC/C,MAAM,SAAS,GAAG,IAAI,CAAC,YAAY,EAAE,MAAM,IAAI,CAAC,CAAC;IACjD,IAAI,SAAS,IAAI,aAAa;QAAE,OAAO,IAAI,CAAC;IAC5C,IAAI,IAAI,CAAC,QAAQ,IAAI,IAAI,IAAI,IAAI,CAAC,QAAQ,IAAI,CAAC;QAAE,OAAO,IAAI,CAAC;IAC7D,OAAO,CAAC,CAAC,IAAI,CAAC,MAAM,EAAE,QAAQ,CAAC,aAAa,CAAC,CAAC;AAChD,CAAC;AAED,UAAU;AAEV;;;;;;;GAOG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB,CACtC,OAAwB,EACxB,IAIC;IAED,MAAM,MAAM,GAAG,IAAI,EAAE,MAAM,IAAI,WAAW,CAAC;IAC3C,MAAM,MAAM,GAAG,IAAI,EAAE,MAAM,IAAI,aAAa,CAAC;IAC7C,MAAM,WAAW,GAAG,IAAI,EAAE,WAAW,IAAI,MAAM,CAAC,MAAM,CAAC;IAEvD,MAAM,OAAO,GAAG,MAAM,OAAO,CAAC,MAAM,EAAE,WAAW,EAAE,KAAK,EAAE,IAAI,EAAE,EAAE;QAChE,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC;YAC1B,GAAG,OAAO;YACV,eAAe,EAAE,wBAAwB,CAAC,OAAO,CAAC,eAAe,EAAE,IAAI,CAAC;SACzE,CAAC,CAAC;QACH,gFAAgF;QAChF,OAAO,EAAE,GAAG,MAAM,EAAE,QAAQ,EAAE,IAAI,IAAI,CAAC,GAAG,KAAK,SAAS,CAAC,MAAM,CAAC,QAAQ,CAAC,EAAE,EAAE,CAAC;IAChF,CAAC,CAAC,CAAC;IAEH,2EAA2E;IAC3E,qEAAqE;IACrE,KAAK,MAAM,CAAC,IAAI,OAAO,EAAE,CAAC;QACxB,IAAI,CAAC,CAAC,KAAK,YAAY,cAAc;YAAE,MAAM,CAAC,CAAC,KAAK,CAAC;IACvD,CAAC;IAED,MAAM,EAAE,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,EAA+C,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,SAAS,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC;IACzH,OAAO,kBAAkB,CAAC,EAAE,CAAC,CAAC;AAChC,CAAC"}

package/dist/agents/pairPipeline.d.ts

@@ -51,6 +51,8 @@ export interface PipelineConfig {
         relevantFiles: string[];
         suggestedApproach: string;
         projectStats?: string;
+        completionCriteria?: string[];
+        sufficient?: boolean;
         impactAnalysis?: import('../knowledge/types.js').ImpactAnalysis;
         registrySnapshot?: Array<{
             filePath: string;
@@ -76,7 +78,9 @@ export interface PipelineResult {
     success: boolean;
     sessionId: string;
     stages: StageResult[];
-    finalStatus: 'approved' | 'rejected' | 'failed' | 'cancelled' | 'decomposed';
+    finalStatus: 'approved' | 'rejected' | 'failed' | 'cancelled' | 'decomposed' | 'rate_limited' | 'infra_error';
+    /** Unix timestamp (ms) when the rate-limit quota resets — set when finalStatus is 'rate_limited'. */
+    rateLimitResetsAt?: number;
     totalDuration: number;
     /** Total number of completed iterations */
     iterations: number;
@@ -146,6 +150,8 @@ export declare class PairPipeline extends EventEmitter {
     private matchesProfile;
     private getProfileForTask;
     private getModelForRole;
+    /** Reasoning effort from the matched jobProfile (heavy tasks reason harder). */
+    private getEffortForTask;
     /**
      * Run pipeline
      *

package/dist/agents/pairPipeline.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"pairPipeline.d.ts","sourceRoot":"","sources":["../../src/agents/pairPipeline.ts"],"names":[],"mappings":"AAKA,OAAO,EAAE,YAAY,EAAE,MAAM,aAAa,CAAC;AAC3C,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,oCAAoC,CAAC;AACnE,OAAO,KAAK,EAAE,YAAY,EAAE,YAAY,EAAE,WAAW,EAAE,MAAM,gBAAgB,CAAC;AAC9E,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,aAAa,CAAC;AAChD,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,iBAAiB,CAAC;AACxD,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,cAAc,CAAC;AAClD,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,sBAAsB,CAAC;AAClE,OAAO,KAAK,EAAE,aAAa,EAAE,UAAU,EAAE,oBAAoB,EAAE,UAAU,EAAE,MAAM,kBAAkB,CAAC;AACpG,OAAO,EAAE,KAAK,QAAQ,EAA8B,MAAM,2BAA2B,CAAC;AAKtF,OAAO,EAAa,KAAK,eAAe,EAAE,MAAM,qBAAqB,CAAC;AACtE,OAAO,EACL,KAAK,eAAe,EAOrB,MAAM,iBAAiB,CAAC;AAezB,MAAM,WAAW,cAAc;IAC7B,gDAAgD;IAChD,MAAM,EAAE,aAAa,EAAE,CAAC;IACxB,0CAA0C;IAC1C,kBAAkB,CAAC,EAAE,OAAO,CAAC;IAC7B,oCAAoC;IACpC,wBAAwB,CAAC,EAAE,OAAO,CAAC;IACnC,oEAAoE;IACpE,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB;;;;;OAKG;IACH,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,6BAA6B;IAC7B,KAAK,CAAC,EAAE;QACN,MAAM,CAAC,EAAE,UAAU,CAAC;QACpB,QAAQ,CAAC,EAAE,UAAU,CAAC;QACtB,MAAM,CAAC,EAAE,UAAU,CAAC;QACpB,UAAU,CAAC,EAAE,UAAU,CAAC;QACxB,OAAO,CAAC,EAAE,UAAU,CAAC;QACrB,kBAAkB,CAAC,EAAE,UAAU,CAAC;KACjC,CAAC;IACF,oCAAoC;IACpC,MAAM,CAAC,EAAE,OAAO,CAAC,oBAAoB,CAAC,CAAC;IACvC,gDAAgD;IAChD,WAAW,CAAC,EAAE,UAAU,EAAE,CAAC;IAC3B,8EAA8E;IAC9E,wBAAwB,CAAC,EAAE,OAAO,CAAC;IACnC,8DAA8D;IAC9D,yBAAyB,CAAC,EAAE,MAAM,CAAC;IACnC,4EAA4E;IAC5E,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,2DAA2D;IAC3D,aAAa,CAAC,EAAE;QACd,QAAQ,EAAE,MAAM,CAAC;QACjB,aAAa,EAAE,MAAM,CAAC;QACtB,aAAa,EAAE,MAAM,EAAE,CAAC;QACxB,iBAAiB,EAAE,MAAM,CAAC;QAC1B,YAAY,CAAC,EAAE,MAAM,CAAC;QACtB,cAAc,CAAC,EAAE,OAAO,uBAAuB,EAAE,cAAc,CAAC;QAChE,gBAAgB,CAAC,EAAE,KAAK,CAAC;YAAE,QAAQ,EAAE,MAAM,CAAC;YAAC,OAAO,EAAE,MAAM,CAAC;YAAC,UAAU,EAAE,MAAM,EAAE,CAAA;SAAE,CAAC,CAAC;KACvF,CAAC;CACH;AAED,MAAM,WAAW,WAAW;IAC1B,KAAK,EAAE,aAAa,CAAC;IACrB,OAAO,EAAE,OAAO,CAAC;IACjB,MAAM,EAAE,YAAY,GAAG,YAAY,GAAG,YAAY,GAAG,gBAAgB,GAAG,aAAa,GAAG,qBAAqB,GAAG;QAAE,OAAO,EAAE,KAAK,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,CAAC;IAClJ,QAAQ,EAAE,MAAM,CAAC;IACjB,kCAAkC;IAClC,SAAS,EAAE,MAAM,CAAC;IAClB,uCAAuC;IACvC,WAAW,EAAE,MAAM,CAAC;CACrB;AAED,MAAM,WAAW,cAAc;IAC7B,OAAO,EAAE,OAAO,CAAC;IACjB,SAAS,EAAE,MAAM,CAAC;IAClB,MAAM,EAAE,WAAW,EAAE,CAAC;IACtB,WAAW,EAAE,UAAU,GAAG,UAAU,GAAG,QAAQ,GAAG,WAAW,GAAG,YAAY,CAAC;IAC7E,aAAa,EAAE,MAAM,CAAC;IACtB,2CAA2C;IAC3C,UAAU,EAAE,MAAM,CAAC;IACnB,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,gBAAgB,CAAC,EAAE,gBAAgB,CAAC;IACpC,aAAa,CAAC,EAAE,aAAa,CAAC;IAC9B,qBAAqB,CAAC,EAAE,qBAAqB,CAAC;IAC9C,mCAAmC;IACnC,WAAW,CAAC,EAAE;QACZ,eAAe,CAAC,EAAE,MAAM,CAAC;QACzB,WAAW,CAAC,EAAE,MAAM,CAAC;QACrB,WAAW,CAAC,EAAE,MAAM,CAAC;QACrB,SAAS,CAAC,EAAE,MAAM,CAAC;KACpB,CAAC;IACF,gDAAgD;IAChD,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,mCAAmC;IACnC,SAAS,CAAC,EAAE,QAAQ,CAAC;CACtB;AAED,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,QAAQ,CAAC;IACf,WAAW,EAAE,MAAM,CAAC;IACpB,OAAO,EAAE,WAAW,CAAC;IACrB,MAAM,EAAE,cAAc,CAAC;IACvB,yCAAyC;IACzC,gBAAgB,EAAE,MAAM,CAAC;IACzB,oGAAoG;IACpG,UAAU,EAAE,MAAM,CAAC;IACnB,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,gBAAgB,CAAC,EAAE,gBAAgB,CAAC;IACpC,aAAa,CAAC,EAAE,aAAa,CAAC;IAC9B,qBAAqB,CAAC,EAAE,qBAAqB,CAAC;IAC9C,YAAY,CAAC,EAAE,eAAe,CAAC;IAC/B,6EAA6E;IAC7E,UAAU,EAAE,eAAe,CAAC;IAC5B;;;;;OAKG;IACH,cAAc,CAAC,EAAE,WAAW,GAAG,QAAQ,CAAC;CACzC;AAED;;;GAGG;AACH,wBAAgB,eAAe,CAAC,IAAI,EAAE,QAAQ,EAAE,WAAW,EAAE,MAAM,GAAG,MAAM,CAe3E;AAID,MAAM,MAAM,iBAAiB,GACzB,aAAa,GACb,gBAAgB,GAChB,YAAY,GACZ,iBAAiB,GACjB,oBAAoB,GACpB,gBAAgB,GAChB,mBAAmB,GACnB,eAAe,GACf,MAAM,CAAC;AAIX,qFAAqF;AACrF,qBAAa,sBAAuB,SAAQ,KAAK;;CAKhD;AAED,qBAAa,YAAa,SAAQ,YAAY;IAC5C,OAAO,CAAC,MAAM,CAAiB;IAC/B,OAAO,CAAC,aAAa,CAAgB;IACrC,OAAO,CAAC,WAAW,CAAe;IAClC,sFAAsF;IACtF,OAAO,CAAC,WAAW,CAAC,CAAc;IAElC,kFAAkF;IAClF,OAAO,CAAC,cAAc;gBAIV,MAAM,EAAE,cAAc;IAiBlC,OAAO,CAAC,cAAc;IAQtB,OAAO,CAAC,iBAAiB;IAKzB,OAAO,CAAC,eAAe;IASvB;;;;;OAKG;IACG,GAAG,CAAC,IAAI,EAAE,QAAQ,EAAE,WAAW,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE;QAAE,MAAM,CAAC,EAAE,WAAW,CAAA;KAAE,GAAG,OAAO,CAAC,cAAc,CAAC;IA6HxG;;;OAGG;YACW,oBAAoB;IAqGlC;;OAEG;IACH,OAAO,CAAC,QAAQ;IAIhB;;OAEG;YACW,QAAQ;IA2RtB;;OAEG;IACH,OAAO,CAAC,cAAc;IA4BtB;;;;;;OAMG;IACH,OAAO,CAAC,qBAAqB;IA4B7B;;;;;;OAMG;YACW,oBAAoB;IAwTlC;;OAEG;IACH,OAAO,CAAC,WAAW;CAyDpB;AAID;;GAEG;AACH,wBAAgB,qBAAqB,CAAC,aAAa,SAAI,GAAG,YAAY,CAKrE;AAED;;GAEG;AACH,wBAAgB,kBAAkB,CAChC,MAAM,CAAC,EAAE,OAAO,CAAC,cAAc,CAAC,GAC/B,YAAY,CAQd;AAED;;GAEG;AACH,wBAAgB,wBAAwB,CACtC,KAAK,EAAE,cAAc,CAAC,OAAO,CAAC,EAC9B,aAAa,SAAI,EACjB,MAAM,CAAC,EAAE,OAAO,CAAC,oBAAoB,CAAC,EACtC,WAAW,CAAC,EAAE,UAAU,EAAE,EAC1B,aAAa,CAAC,EAAE,cAAc,CAAC,eAAe,CAAC,EAC/C,cAAc,CAAC,EAAE,MAAM,GACtB,YAAY,CA+Bd;AAsGD,OAAO,EAAE,oBAAoB,EAAE,yBAAyB,EAAE,MAAM,qBAAqB,CAAC"}
+{"version":3,"file":"pairPipeline.d.ts","sourceRoot":"","sources":["../../src/agents/pairPipeline.ts"],"names":[],"mappings":"AAKA,OAAO,EAAE,YAAY,EAAE,MAAM,aAAa,CAAC;AAC3C,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,oCAAoC,CAAC;AACnE,OAAO,KAAK,EAAE,YAAY,EAAE,YAAY,EAAE,WAAW,EAAE,MAAM,gBAAgB,CAAC;AAC9E,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,aAAa,CAAC;AAChD,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,iBAAiB,CAAC;AACxD,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,cAAc,CAAC;AAClD,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,sBAAsB,CAAC;AAClE,OAAO,KAAK,EAAE,aAAa,EAAE,UAAU,EAAE,oBAAoB,EAAE,UAAU,EAAE,MAAM,kBAAkB,CAAC;AACpG,OAAO,EAAE,KAAK,QAAQ,EAA8B,MAAM,2BAA2B,CAAC;AAKtF,OAAO,EAAa,KAAK,eAAe,EAAE,MAAM,qBAAqB,CAAC;AACtE,OAAO,EACL,KAAK,eAAe,EAOrB,MAAM,iBAAiB,CAAC;AAiBzB,MAAM,WAAW,cAAc;IAC7B,gDAAgD;IAChD,MAAM,EAAE,aAAa,EAAE,CAAC;IACxB,0CAA0C;IAC1C,kBAAkB,CAAC,EAAE,OAAO,CAAC;IAC7B,oCAAoC;IACpC,wBAAwB,CAAC,EAAE,OAAO,CAAC;IACnC,oEAAoE;IACpE,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB;;;;;OAKG;IACH,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,6BAA6B;IAC7B,KAAK,CAAC,EAAE;QACN,MAAM,CAAC,EAAE,UAAU,CAAC;QACpB,QAAQ,CAAC,EAAE,UAAU,CAAC;QACtB,MAAM,CAAC,EAAE,UAAU,CAAC;QACpB,UAAU,CAAC,EAAE,UAAU,CAAC;QACxB,OAAO,CAAC,EAAE,UAAU,CAAC;QACrB,kBAAkB,CAAC,EAAE,UAAU,CAAC;KACjC,CAAC;IACF,oCAAoC;IACpC,MAAM,CAAC,EAAE,OAAO,CAAC,oBAAoB,CAAC,CAAC;IACvC,gDAAgD;IAChD,WAAW,CAAC,EAAE,UAAU,EAAE,CAAC;IAC3B,8EAA8E;IAC9E,wBAAwB,CAAC,EAAE,OAAO,CAAC;IACnC,8DAA8D;IAC9D,yBAAyB,CAAC,EAAE,MAAM,CAAC;IACnC,4EAA4E;IAC5E,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,2DAA2D;IAC3D,aAAa,CAAC,EAAE;QACd,QAAQ,EAAE,MAAM,CAAC;QACjB,aAAa,EAAE,MAAM,CAAC;QACtB,aAAa,EAAE,MAAM,EAAE,CAAC;QACxB,iBAAiB,EAAE,MAAM,CAAC;QAC1B,YAAY,CAAC,EAAE,MAAM,CAAC;QACtB,kBAAkB,CAAC,EAAE,MAAM,EAAE,CAAC;QAC9B,UAAU,CAAC,EAAE,OAAO,CAAC;QACrB,cAAc,CAAC,EAAE,OAAO,uBAAuB,EAAE,cAAc,CAAC;QAChE,gBAAgB,CAAC,EAAE,KAAK,CAAC;YAAE,QAAQ,EAAE,MAAM,CAAC;YAAC,OAAO,EAAE,MAAM,CAAC;YAAC,UAAU,EAAE,MAAM,EAAE,CAAA;SAAE,CAAC,CAAC;KACvF,CAAC;CACH;AAED,MAAM,WAAW,WAAW;IAC1B,KAAK,EAAE,aAAa,CAAC;IACrB,OAAO,EAAE,OAAO,CAAC;IACjB,MAAM,EAAE,YAAY,GAAG,YAAY,GAAG,YAAY,GAAG,gBAAgB,GAAG,aAAa,GAAG,qBAAqB,GAAG;QAAE,OAAO,EAAE,KAAK,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,CAAC;IAClJ,QAAQ,EAAE,MAAM,CAAC;IACjB,kCAAkC;IAClC,SAAS,EAAE,MAAM,CAAC;IAClB,uCAAuC;IACvC,WAAW,EAAE,MAAM,CAAC;CACrB;AAED,MAAM,WAAW,cAAc;IAC7B,OAAO,EAAE,OAAO,CAAC;IACjB,SAAS,EAAE,MAAM,CAAC;IAClB,MAAM,EAAE,WAAW,EAAE,CAAC;IACtB,WAAW,EAAE,UAAU,GAAG,UAAU,GAAG,QAAQ,GAAG,WAAW,GAAG,YAAY,GAAG,cAAc,GAAG,aAAa,CAAC;IAC9G,qGAAqG;IACrG,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,aAAa,EAAE,MAAM,CAAC;IACtB,2CAA2C;IAC3C,UAAU,EAAE,MAAM,CAAC;IACnB,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,gBAAgB,CAAC,EAAE,gBAAgB,CAAC;IACpC,aAAa,CAAC,EAAE,aAAa,CAAC;IAC9B,qBAAqB,CAAC,EAAE,qBAAqB,CAAC;IAC9C,mCAAmC;IACnC,WAAW,CAAC,EAAE;QACZ,eAAe,CAAC,EAAE,MAAM,CAAC;QACzB,WAAW,CAAC,EAAE,MAAM,CAAC;QACrB,WAAW,CAAC,EAAE,MAAM,CAAC;QACrB,SAAS,CAAC,EAAE,MAAM,CAAC;KACpB,CAAC;IACF,gDAAgD;IAChD,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,mCAAmC;IACnC,SAAS,CAAC,EAAE,QAAQ,CAAC;CACtB;AAED,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,QAAQ,CAAC;IACf,WAAW,EAAE,MAAM,CAAC;IACpB,OAAO,EAAE,WAAW,CAAC;IACrB,MAAM,EAAE,cAAc,CAAC;IACvB,yCAAyC;IACzC,gBAAgB,EAAE,MAAM,CAAC;IACzB,oGAAoG;IACpG,UAAU,EAAE,MAAM,CAAC;IACnB,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,gBAAgB,CAAC,EAAE,gBAAgB,CAAC;IACpC,aAAa,CAAC,EAAE,aAAa,CAAC;IAC9B,qBAAqB,CAAC,EAAE,qBAAqB,CAAC;IAC9C,YAAY,CAAC,EAAE,eAAe,CAAC;IAC/B,6EAA6E;IAC7E,UAAU,EAAE,eAAe,CAAC;IAC5B;;;;;OAKG;IACH,cAAc,CAAC,EAAE,WAAW,GAAG,QAAQ,CAAC;CACzC;AAED;;;GAGG;AACH,wBAAgB,eAAe,CAAC,IAAI,EAAE,QAAQ,EAAE,WAAW,EAAE,MAAM,GAAG,MAAM,CAe3E;AAID,MAAM,MAAM,iBAAiB,GACzB,aAAa,GACb,gBAAgB,GAChB,YAAY,GACZ,iBAAiB,GACjB,oBAAoB,GACpB,gBAAgB,GAChB,mBAAmB,GACnB,eAAe,GACf,MAAM,CAAC;AAIX,qFAAqF;AACrF,qBAAa,sBAAuB,SAAQ,KAAK;;CAKhD;AAED,qBAAa,YAAa,SAAQ,YAAY;IAC5C,OAAO,CAAC,MAAM,CAAiB;IAC/B,OAAO,CAAC,aAAa,CAAgB;IACrC,OAAO,CAAC,WAAW,CAAe;IAClC,sFAAsF;IACtF,OAAO,CAAC,WAAW,CAAC,CAAc;IAElC,kFAAkF;IAClF,OAAO,CAAC,cAAc;gBAIV,MAAM,EAAE,cAAc;IAiBlC,OAAO,CAAC,cAAc;IAQtB,OAAO,CAAC,iBAAiB;IAKzB,OAAO,CAAC,eAAe;IAKvB,gFAAgF;IAChF,OAAO,CAAC,gBAAgB;IAQxB;;;;;OAKG;IACG,GAAG,CAAC,IAAI,EAAE,QAAQ,EAAE,WAAW,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE;QAAE,MAAM,CAAC,EAAE,WAAW,CAAA;KAAE,GAAG,OAAO,CAAC,cAAc,CAAC;IA4IxG;;;OAGG;YACW,oBAAoB;IAuGlC;;OAEG;IACH,OAAO,CAAC,QAAQ;IAIhB;;OAEG;YACW,QAAQ;IA2StB;;OAEG;IACH,OAAO,CAAC,cAAc;IA4BtB;;;;;;OAMG;IACH,OAAO,CAAC,qBAAqB;IA4B7B;;;;;;OAMG;YACW,oBAAoB;IA2TlC;;OAEG;IACH,OAAO,CAAC,WAAW;CAyDpB;AAID;;GAEG;AACH,wBAAgB,qBAAqB,CAAC,aAAa,SAAI,GAAG,YAAY,CAKrE;AAED;;GAEG;AACH,wBAAgB,kBAAkB,CAChC,MAAM,CAAC,EAAE,OAAO,CAAC,cAAc,CAAC,GAC/B,YAAY,CAQd;AAED;;GAEG;AACH,wBAAgB,wBAAwB,CACtC,KAAK,EAAE,cAAc,CAAC,OAAO,CAAC,EAC9B,aAAa,SAAI,EACjB,MAAM,CAAC,EAAE,OAAO,CAAC,oBAAoB,CAAC,EACtC,WAAW,CAAC,EAAE,UAAU,EAAE,EAC1B,aAAa,CAAC,EAAE,cAAc,CAAC,eAAe,CAAC,EAC/C,cAAc,CAAC,EAAE,MAAM,GACtB,YAAY,CA+Bd;AAsGD,OAAO,EAAE,oBAAoB,EAAE,yBAAyB,EAAE,MAAM,qBAAqB,CAAC"}

package/dist/agents/pairPipeline.js

@@ -19,6 +19,8 @@ import * as documenterAgent from './documenter.js';
 import * as auditorAgent from './auditor.js';
 import * as skillDocumenterAgent from './skillDocumenter.js';
 import { createStuckDetector } from '../support/stuckDetector.js';
+import { RateLimitError } from '../adapters/rateLimitError.js';
+import { isInfraError } from '../adapters/errorClassification.js';
 /**
  * Build a consistent task prefix for logging across all pipeline stages.
  * Format: "ProjectName | INT-XXX | worktree/abc123" or "ProjectName | INT-XXX"
@@ -94,6 +96,10 @@ export class PairPipeline extends EventEmitter {
         const profile = this.getProfileForTask(task);
         return profile?.roles?.[stage] || this.config.roles?.[stage]?.model;
     }
+    /** Reasoning effort from the matched jobProfile (heavy tasks reason harder). */
+    getEffortForTask(task) {
+        return this.getProfileForTask(task)?.effort;
+    }
     // ============================================
     // Main Execution
     // ============================================
@@ -187,9 +193,23 @@ export class PairPipeline extends EventEmitter {
             // Cancellation (project disable / manual stop) is not a failure — surface it
             // as 'cancelled' so the scheduler doesn't count it failed or trigger a retry.
             const cancelled = error instanceof PipelineCancelledError || !!this.abortSignal?.aborted;
+            // A 429/usage-limit propagates up here from any stage (worker/reviewer/…).
+            // Surface it as its own finalStatus so the runner pauses until quota resets
+            // instead of counting a failure and spamming Linear comments. (INT-1906)
+            const rateLimited = !cancelled && error instanceof RateLimitError;
+            // An infra/CLI failure (worker/reviewer never ran: non-zero exit, auth,
+            // spawn, timeout) is not a task failure — surface it distinctly so the
+            // runner does a backoff retry instead of counting it toward STUCK. (INT-2010)
+            const infra = !cancelled && !rateLimited && isInfraError(error);
             if (cancelled) {
                 console.log(`[${context.taskPrefix}] Pipeline cancelled`);
             }
+            else if (rateLimited) {
+                console.warn(`[${context.taskPrefix}] Pipeline rate-limited: ${error.message}`);
+            }
+            else if (infra) {
+                console.warn(`[${context.taskPrefix}] Pipeline infra error (not counted toward STUCK): ${error instanceof Error ? error.message : String(error)}`);
+            }
             else {
                 console.error('[%s] Error:', context.taskPrefix, error);
             }
@@ -198,7 +218,10 @@ export class PairPipeline extends EventEmitter {
                 success: false,
                 sessionId: session.id,
                 stages,
-                finalStatus: cancelled ? 'cancelled' : 'failed',
+                finalStatus: cancelled ? 'cancelled' : rateLimited ? 'rate_limited' : infra ? 'infra_error' : 'failed',
+                rateLimitResetsAt: rateLimited && error.resetsAt
+                    ? error.resetsAt * 1000
+                    : undefined,
                 totalDuration: Date.now() - startTime,
                 iterations: context.currentIteration,
                 workerResult: context.workerResult,
@@ -233,6 +256,8 @@ export class PairPipeline extends EventEmitter {
                     relevantFiles: draft.relevantFiles,
                     suggestedApproach: draft.suggestedApproach,
                     projectStats: draft.projectStats,
+                    completionCriteria: draft.completionCriteria,
+                    sufficient: draft.sufficient,
                 };
                 if (draft.impactAnalysis) {
                     wc.impactAnalysis = draft.impactAnalysis;
@@ -352,11 +377,15 @@ export class PairPipeline extends EventEmitter {
                         this.emit('log', { line: `[verbose] Worker context: ${modCount} affected modules, ${briefCount} file briefs` });
                     }
                     // Self-repair feedback: objective lint/test errors (reflection trail)
-                    // are always carried forward — they are ground truth and survive a
-                    // fresh-context reset. Subjective reviewer feedback is dropped on fresh
-                    // context and only included when it was the latest revise reason.
+                    // are always carried forward — ground truth that survives a fresh-context
+                    // reset. The reviewer's revision prompt is ALSO preserved across fresh
+                    // context (INT-1705): it carries the task requirement (e.g. "wire it into
+                    // the heartbeat / add the call site"), not chat pollution — dropping it
+                    // made the worker repeat the same partial impl forever. Fresh context
+                    // still clears the worker's own chat history; only the reviewer's task
+                    // signal is kept.
                     const reflectionPart = buildReflectionFeedback(context.reflection);
-                    const includeReview = !useFreshContext && context.feedbackSource === 'review' && !!context.reviewResult;
+                    const includeReview = context.feedbackSource === 'review' && !!context.reviewResult;
                     const reviewPart = includeReview
                         ? reviewerAgent.buildRevisionPrompt(context.reviewResult)
                         : undefined;
@@ -367,9 +396,20 @@ export class PairPipeline extends EventEmitter {
                         projectPath: context.projectPath,
                         previousFeedback: combinedFeedback,
                         timeoutMs: this.config.roles?.worker?.timeoutMs ?? 0,
-                        model: overrides?.model ?? this.config.roles?.worker?.model,
+                        // getModelForRole gives the matched jobProfile's model precedence (config's
+                        // light/heavy → gpt-5.5/5.4), falling back to roles.worker.model. Reading
+                        // roles.worker.model directly here silently dropped the jobProfile model, so a
+                        // codex worker fell through to the CLI's config.toml default (Codex-Spark). (INT-1599)
+                        model: overrides?.model ?? this.getModelForRole('worker', context.task),
                         maxTurns: this.config.roles?.worker?.maxTurns,
                         adapterName: this.config.roles?.worker?.adapter,
+                        reasoningEffort: this.getEffortForTask(context.task),
+                        // No-edit guard (re-applied from stranded feat/v0.7.0 commit 2eea3bc):
+                        // reasoning workers frequently end with analysis only and never call
+                        // edit_file. Without this the guard defaults to 0 (disabled) — measured:
+                        // codex spark AND gpt-5.5 both read 30-37× and shipped 0 edits. Push the
+                        // worker to actually edit before concluding.
+                        nudgeMaxOnNoEdit: 3,
                         issueIdentifier: context.task.issueIdentifier || context.task.issueId,
                         projectName: context.task.linearProject?.name,
                         onLog,
@@ -413,24 +453,25 @@ export class PairPipeline extends EventEmitter {
                         throw new Error('Worker result required for reviewer');
                     }
                     // Pre-check disabled - Haiku format compliance issues causing false rejections
-                    // Proceed directly to full Sonnet review for reliability
-                    // Reduce review depth when worker confidence is very high
-                    let reviewerMaxTurns = this.config.roles?.reviewer?.maxTurns;
-                    if (context.workerResult?.confidencePercent && context.workerResult.confidencePercent > 90) {
-                        const cappedTurns = Math.min(reviewerMaxTurns ?? 10, 5);
-                        console.log(`[${prefix}] High worker confidence (${context.workerResult.confidencePercent}%), limiting reviewer to ${cappedTurns} turns`);
-                        reviewerMaxTurns = cappedTurns;
-                    }
-                    console.log(`[${prefix}] Running full review (Sonnet)...`);
+                    // Proceed directly to full review for reliability.
+                    // NOTE: the old "high worker confidence → fewer reviewer turns" shortcut was
+                    // removed (INT-1914): worker confidence is self-reported, so a confidently
+                    // scaffolded task was getting LESS review — exactly the wrong incentive. The
+                    // completion-criteria hard gate is the real check now.
+                    const reviewerMaxTurns = this.config.roles?.reviewer?.maxTurns;
+                    console.log(`[${prefix}] Running full review...`);
                     result = await reviewerAgent.runReviewer({
                         taskTitle: context.task.title,
                         taskDescription: context.task.description || '',
                         workerResult: context.workerResult,
                         projectPath: context.projectPath,
                         timeoutMs: this.config.roles?.reviewer?.timeoutMs ?? 0,
-                        model: this.config.roles?.reviewer?.model,
+                        // jobProfile model precedence (see worker stage above). (INT-1599)
+                        model: this.getModelForRole('reviewer', context.task),
                         maxTurns: reviewerMaxTurns,
                         adapterName: this.config.roles?.reviewer?.adapter,
+                        reasoningEffort: this.getEffortForTask(context.task),
+                        completionCriteria: this.config.draftAnalysis?.completionCriteria,
                         processContext: { taskId: context.task.id, stage: 'reviewer' },
                         signal: this.abortSignal,
                     });
@@ -777,6 +818,58 @@ export class PairPipeline extends EventEmitter {
                     continue;
                 }
             }
+            // ========== TESTER (before reviewer — INT-1703) ==========
+            // Run the tester first so the reviewer judges code + test outcomes
+            // together, and so a failing test drives INT-1679 self-repair WITHOUT
+            // spending a reviewer pass. Runs exactly once per iteration.
+            if (hasTester) {
+                // Skip tester if no code files changed (configurable, default true)
+                const skipIfNoCode = this.config.skipTesterIfNoCodeChange ?? true;
+                const codeExtensions = /\.(ts|tsx|js|jsx|py|rs|go|java|rb|c|cpp|h|hpp)$/;
+                const changedFiles = context.workerResult?.filesChanged || [];
+                const hasCodeChange = changedFiles.some(f => codeExtensions.test(f));
+                if (skipIfNoCode && !hasCodeChange) {
+                    console.log(`[${context.taskPrefix}] Skipping tester: no code files changed (${changedFiles.length} files: ${changedFiles.join(', ') || 'none'})`);
+                }
+                else {
+                    const testerResult = await this.runStage('tester', context);
+                    stages.push(testerResult);
+                    if (!testerResult.success && !this.config.continueOnTestFail) {
+                        // Test failure is objective ground truth → record into the reflection
+                        // trail and drive a bounded self-repair retry (INT-1679).
+                        console.log(`[${context.taskPrefix}] Tester failed, retrying...`);
+                        agentPair.trackFailure(context.session.id); // Track for fresh context decision
+                        const failedTests = context.testerResult?.failedTests ?? [];
+                        const testErrors = failedTests.length > 0
+                            ? failedTests
+                            : [context.testerResult?.error || `Tests failed (${context.testerResult?.testsFailed ?? 0} failing)`];
+                        const { progressed } = recordReflection(context.reflection, {
+                            iteration: context.currentIteration,
+                            source: 'test',
+                            errors: testErrors,
+                        });
+                        if (context.testerResult) {
+                            context.reviewResult = {
+                                decision: 'revise',
+                                feedback: testerAgent.buildTestFixPrompt(context.testerResult),
+                                issues: context.testerResult.failedTests,
+                                suggestions: context.testerResult.suggestions,
+                            };
+                        }
+                        context.feedbackSource = 'objective';
+                        this.emit('iteration:fail', {
+                            iteration: context.currentIteration,
+                            stage: 'tester',
+                            context,
+                        });
+                        agentPair.updateSessionStatus(context.session.id, 'revising');
+                        if (this.shouldAbortSelfRepair(context, progressed, 'test')) {
+                            return { success: false };
+                        }
+                        continue;
+                    }
+                } // end else (has code change)
+            }
             // ========== REVIEWER ==========
             if (hasReviewer) {
                 agentPair.updateSessionStatus(context.session.id, 'reviewing');
@@ -825,58 +918,9 @@ export class PairPipeline extends EventEmitter {
                     agentPair.updateSessionStatus(context.session.id, 'revising');
                     continue;
                 }
-                // approve → proceed to Tester
+                // approve → done (tester already ran before the reviewer — INT-1703)
                 agentPair.resetFailureStreak(context.session.id); // Reset on approval
             }
-            // ========== TESTER ==========
-            if (hasTester) {
-                // Skip tester if no code files changed (configurable, default true)
-                const skipIfNoCode = this.config.skipTesterIfNoCodeChange ?? true;
-                const codeExtensions = /\.(ts|tsx|js|jsx|py|rs|go|java|rb|c|cpp|h|hpp)$/;
-                const changedFiles = context.workerResult?.filesChanged || [];
-                const hasCodeChange = changedFiles.some(f => codeExtensions.test(f));
-                if (skipIfNoCode && !hasCodeChange) {
-                    console.log(`[${context.taskPrefix}] Skipping tester: no code files changed (${changedFiles.length} files: ${changedFiles.join(', ') || 'none'})`);
-                }
-                else {
-                    const testerResult = await this.runStage('tester', context);
-                    stages.push(testerResult);
-                    if (!testerResult.success && !this.config.continueOnTestFail) {
-                        // Test failure is objective ground truth → record into the reflection
-                        // trail and drive a bounded self-repair retry.
-                        console.log(`[${context.taskPrefix}] Tester failed, retrying...`);
-                        agentPair.trackFailure(context.session.id); // Track for fresh context decision
-                        const failedTests = context.testerResult?.failedTests ?? [];
-                        const testErrors = failedTests.length > 0
-                            ? failedTests
-                            : [context.testerResult?.error || `Tests failed (${context.testerResult?.testsFailed ?? 0} failing)`];
-                        const { progressed } = recordReflection(context.reflection, {
-                            iteration: context.currentIteration,
-                            source: 'test',
-                            errors: testErrors,
-                        });
-                        if (context.testerResult) {
-                            context.reviewResult = {
-                                decision: 'revise',
-                                feedback: testerAgent.buildTestFixPrompt(context.testerResult),
-                                issues: context.testerResult.failedTests,
-                                suggestions: context.testerResult.suggestions,
-                            };
-                        }
-                        context.feedbackSource = 'objective';
-                        this.emit('iteration:fail', {
-                            iteration: context.currentIteration,
-                            stage: 'tester',
-                            context,
-                        });
-                        agentPair.updateSessionStatus(context.session.id, 'revising');
-                        if (this.shouldAbortSelfRepair(context, progressed, 'test')) {
-                            return { success: false };
-                        }
-                        continue;
-                    }
-                } // end else (has code change)
-            }
             // ========== ALL PASSED ==========
             console.log(`[${context.taskPrefix}] Iteration ${context.currentIteration} completed successfully`);
             this.emit('iteration:complete', {