@urateam/core 0.1.25 → 0.1.27

package/dist/__tests__/agent-stream-cache-tokens.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=agent-stream-cache-tokens.test.d.ts.map

package/dist/__tests__/agent-stream-cache-tokens.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"agent-stream-cache-tokens.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/agent-stream-cache-tokens.test.ts"],"names":[],"mappings":""}

package/dist/__tests__/agent-stream-cache-tokens.test.js

@@ -0,0 +1,25 @@
+import { describe, it, expect } from "vitest";
+import { consumeAgentStream } from "../executor/agent-stream.js";
+describe("consumeAgentStream — cache tokens (BEC: cache telemetry)", () => {
+    it("accumulates cache_creation_input_tokens and cache_read_input_tokens from message.usage", async () => {
+        async function* fakeStream() {
+            yield { type: "assistant", usage: { input_tokens: 100, cache_creation_input_tokens: 5000, cache_read_input_tokens: 0, output_tokens: 200 }, content: [{ type: "text", text: "" }] };
+            yield { type: "assistant", usage: { input_tokens: 50, cache_creation_input_tokens: 0, cache_read_input_tokens: 5000, output_tokens: 300 }, content: [{ type: "text", text: "" }] };
+            yield { type: "assistant", usage: { input_tokens: 50, cache_creation_input_tokens: 0, cache_read_input_tokens: 5000, output_tokens: 250 }, content: [{ type: "text", text: "done" }] };
+        }
+        const result = await consumeAgentStream(fakeStream());
+        expect(result.inputTokens).toBe(200);
+        expect(result.outputTokens).toBe(750);
+        expect(result.cacheCreationInputTokens).toBe(5000);
+        expect(result.cacheReadInputTokens).toBe(10000);
+    });
+    it("treats missing cache fields as 0 (backward compat with non-cache responses)", async () => {
+        async function* fakeStream() {
+            yield { type: "assistant", usage: { input_tokens: 100, output_tokens: 200 }, content: [{ type: "text", text: "" }] };
+        }
+        const result = await consumeAgentStream(fakeStream());
+        expect(result.cacheCreationInputTokens).toBe(0);
+        expect(result.cacheReadInputTokens).toBe(0);
+    });
+});
+//# sourceMappingURL=agent-stream-cache-tokens.test.js.map

package/dist/__tests__/agent-stream-cache-tokens.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"agent-stream-cache-tokens.test.js","sourceRoot":"","sources":["../../src/__tests__/agent-stream-cache-tokens.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,kBAAkB,EAAE,MAAM,6BAA6B,CAAC;AAEjE,QAAQ,CAAC,0DAA0D,EAAE,GAAG,EAAE;IACxE,EAAE,CAAC,wFAAwF,EAAE,KAAK,IAAI,EAAE;QACtG,KAAK,SAAS,CAAC,CAAC,UAAU;YACxB,MAAM,EAAE,IAAI,EAAE,WAAW,EAAE,KAAK,EAAE,EAAE,YAAY,EAAE,GAAG,EAAE,2BAA2B,EAAE,IAAI,EAAE,uBAAuB,EAAE,CAAC,EAAE,aAAa,EAAE,GAAG,EAAE,EAAE,OAAO,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC;YACpL,MAAM,EAAE,IAAI,EAAE,WAAW,EAAE,KAAK,EAAE,EAAE,YAAY,EAAE,EAAE,EAAE,2BAA2B,EAAE,CAAC,EAAE,uBAAuB,EAAE,IAAI,EAAE,aAAa,EAAE,GAAG,EAAE,EAAE,OAAO,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC;YACnL,MAAM,EAAE,IAAI,EAAE,WAAW,EAAE,KAAK,EAAE,EAAE,YAAY,EAAE,EAAE,EAAE,2BAA2B,EAAE,CAAC,EAAE,uBAAuB,EAAE,IAAI,EAAE,aAAa,EAAE,GAAG,EAAE,EAAE,OAAO,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC,EAAE,CAAC;QACzL,CAAC;QACD,MAAM,MAAM,GAAG,MAAM,kBAAkB,CAAC,UAAU,EAAE,CAAC,CAAC;QACtD,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACrC,MAAM,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACtC,MAAM,CAAC,MAAM,CAAC,wBAAwB,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACnD,MAAM,CAAC,MAAM,CAAC,oBAAoB,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAClD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6EAA6E,EAAE,KAAK,IAAI,EAAE;QAC3F,KAAK,SAAS,CAAC,CAAC,UAAU;YACxB,MAAM,EAAE,IAAI,EAAE,WAAW,EAAE,KAAK,EAAE,EAAE,YAAY,EAAE,GAAG,EAAE,aAAa,EAAE,GAAG,EAAE,EAAE,OAAO,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC;QACvH,CAAC;QACD,MAAM,MAAM,GAAG,MAAM,kBAAkB,CAAC,UAAU,EAAE,CAAC,CAAC;QACtD,MAAM,CAAC,MAAM,CAAC,wBAAwB,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAChD,MAAM,CAAC,MAAM,CAAC,oBAAoB,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAC9C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/__tests__/audit/pm-skipped-circuit-breaker-event.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=pm-skipped-circuit-breaker-event.test.d.ts.map

package/dist/__tests__/audit/pm-skipped-circuit-breaker-event.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"pm-skipped-circuit-breaker-event.test.d.ts","sourceRoot":"","sources":["../../../src/__tests__/audit/pm-skipped-circuit-breaker-event.test.ts"],"names":[],"mappings":""}

package/dist/__tests__/audit/pm-skipped-circuit-breaker-event.test.js

@@ -0,0 +1,32 @@
+import { describe, it, expect } from "vitest";
+import { pmSkippedCircuitBreakerEvent } from "../../audit/events.js";
+describe("pmSkippedCircuitBreakerEvent (BEC-181)", () => {
+    it("returns an audit event with the documented shape", () => {
+        const event = pmSkippedCircuitBreakerEvent({
+            issueId: "BEC-157",
+            failureCount: 4,
+            threshold: 3,
+            source: "promote",
+        });
+        expect(event.eventType).toBe("pm.skipped_circuit_breaker");
+        expect(event.actor).toBe("pm-agent");
+        expect(event.actorType).toBe("pm-agent");
+        expect(event.issueId).toBe("BEC-157");
+        expect(event.payload).toEqual({
+            failureCount: 4,
+            threshold: 3,
+            source: "promote",
+        });
+        expect(event.id).toMatch(/^evt_/);
+    });
+    it("captures source as 'start-todo' when fired from start-todo path", () => {
+        const event = pmSkippedCircuitBreakerEvent({
+            issueId: "BEC-999",
+            failureCount: 5,
+            threshold: 3,
+            source: "start-todo",
+        });
+        expect(event.payload.source).toBe("start-todo");
+    });
+});
+//# sourceMappingURL=pm-skipped-circuit-breaker-event.test.js.map

package/dist/__tests__/audit/pm-skipped-circuit-breaker-event.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"pm-skipped-circuit-breaker-event.test.js","sourceRoot":"","sources":["../../../src/__tests__/audit/pm-skipped-circuit-breaker-event.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,4BAA4B,EAAE,MAAM,uBAAuB,CAAC;AAErE,QAAQ,CAAC,wCAAwC,EAAE,GAAG,EAAE;IACtD,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC1D,MAAM,KAAK,GAAG,4BAA4B,CAAC;YACzC,OAAO,EAAE,SAAS;YAClB,YAAY,EAAE,CAAC;YACf,SAAS,EAAE,CAAC;YACZ,MAAM,EAAE,SAAS;SAClB,CAAC,CAAC;QACH,MAAM,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,4BAA4B,CAAC,CAAC;QAC3D,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QACrC,MAAM,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QACzC,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QACtC,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC;YAC5B,YAAY,EAAE,CAAC;YACf,SAAS,EAAE,CAAC;YACZ,MAAM,EAAE,SAAS;SAClB,CAAC,CAAC;QACH,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC;IACpC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iEAAiE,EAAE,GAAG,EAAE;QACzE,MAAM,KAAK,GAAG,4BAA4B,CAAC;YACzC,OAAO,EAAE,SAAS;YAClB,YAAY,EAAE,CAAC;YACf,SAAS,EAAE,CAAC;YACZ,MAAM,EAAE,YAAY;SACrB,CAAC,CAAC;QACH,MAAM,CAAE,KAAK,CAAC,OAA8B,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;IAC1E,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/__tests__/audit/review-model-low-output-ratio-event.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=review-model-low-output-ratio-event.test.d.ts.map

package/dist/__tests__/audit/review-model-low-output-ratio-event.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-model-low-output-ratio-event.test.d.ts","sourceRoot":"","sources":["../../../src/__tests__/audit/review-model-low-output-ratio-event.test.ts"],"names":[],"mappings":""}

package/dist/__tests__/audit/review-model-low-output-ratio-event.test.js

@@ -0,0 +1,23 @@
+import { describe, it, expect } from "vitest";
+import { reviewModelLowOutputRatioEvent } from "../../audit/events.js";
+describe("reviewModelLowOutputRatioEvent", () => {
+    it("returns an audit event with the documented shape", () => {
+        const event = reviewModelLowOutputRatioEvent({
+            modelId: "gpt-oss-120b:free",
+            outputRatio: 0.011,
+            runs: 10,
+            threshold: 0.05,
+        });
+        expect(event.eventType).toBe("review.model_low_output_ratio");
+        expect(event.actor).toBe("system");
+        expect(event.actorType).toBe("system");
+        expect(event.payload).toEqual({
+            modelId: "gpt-oss-120b:free",
+            outputRatio: 0.011,
+            runs: 10,
+            threshold: 0.05,
+        });
+        expect(event.id).toMatch(/^evt_/);
+    });
+});
+//# sourceMappingURL=review-model-low-output-ratio-event.test.js.map

package/dist/__tests__/audit/review-model-low-output-ratio-event.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-model-low-output-ratio-event.test.js","sourceRoot":"","sources":["../../../src/__tests__/audit/review-model-low-output-ratio-event.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,8BAA8B,EAAE,MAAM,uBAAuB,CAAC;AAEvE,QAAQ,CAAC,gCAAgC,EAAE,GAAG,EAAE;IAC9C,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC1D,MAAM,KAAK,GAAG,8BAA8B,CAAC;YAC3C,OAAO,EAAE,mBAAmB;YAC5B,WAAW,EAAE,KAAK;YAClB,IAAI,EAAE,EAAE;YACR,SAAS,EAAE,IAAI;SAChB,CAAC,CAAC;QACH,MAAM,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,+BAA+B,CAAC,CAAC;QAC9D,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACnC,MAAM,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACvC,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC;YAC5B,OAAO,EAAE,mBAAmB;YAC5B,WAAW,EAAE,KAAK;YAClB,IAAI,EAAE,EAAE;YACR,SAAS,EAAE,IAAI;SAChB,CAAC,CAAC;QACH,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC;IACpC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/__tests__/audit-immutability.test.js

@@ -52,7 +52,9 @@ describe("audit_events immutability", () => {
             "packages/core/src/pm/scheduler.ts",
             "packages/core/src/pm/actions/triage.ts",
             "packages/core/src/pm/actions/promote.ts",
+            "packages/core/src/pm/actions/start-todo.ts",
             "packages/core/src/pm/actions/resolve-approvals.ts",
+            "packages/core/src/pipeline/review-providers-runner.ts",
             "packages/core/src/release-manager/scheduler.ts",
             "packages/core/src/release-manager/slack-handler.ts",
             "packages/core/src/repo/agent-branch-sweep-runner.ts",

package/dist/__tests__/audit-immutability.test.js.map

@@ -1 +1 @@
-{"version":3,"file":"audit-immutability.test.js","sourceRoot":"","sources":["../../src/__tests__/audit-immutability.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AACH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAClD,OAAO,IAAI,MAAM,WAAW,CAAC;AAE7B,QAAQ,CAAC,2BAA2B,EAAE,GAAG,EAAE;IACzC,EAAE,CAAC,gEAAgE,EAAE,GAAG,EAAE;QACxE,MAAM,QAAQ,GAAG,IAAI,CAAC,OAAO,CAAC,SAAS,EAAE,aAAa,CAAC,CAAC;QACxD,MAAM,OAAO,GAAG;YACd,sCAAsC;YACtC,qDAAqD;YACrD,wDAAwD;SACzD,CAAC;QAEF,MAAM,QAAQ,GAAG;YACf,iCAAiC;YACjC,iCAAiC;SAClC,CAAC;QAEF,MAAM,OAAO,GAAa,EAAE,CAAC;QAC7B,KAAK,MAAM,GAAG,IAAI,QAAQ,EAAE,CAAC;YAC3B,IAAI,CAAC;gBACH,MAAM,GAAG,GAAG,YAAY,CACtB,KAAK,EACL,CAAC,MAAM,EAAE,KAAK,EAAE,GAAG,EAAE,IAAI,EAAE,kBAAkB,CAAC,EAC9C,EAAE,GAAG,EAAE,QAAQ,EAAE,QAAQ,EAAE,MAAM,EAAE,CACpC,CAAC;gBACF,OAAO,CAAC,IAAI,CAAC,GAAG,GAAG,CAAC,IAAI,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC;YAC1D,CAAC;YAAC,MAAM,CAAC;gBACP,0DAA0D;YAC5D,CAAC;QACH,CAAC;QAED,MAAM,SAAS,GAAG,OAAO;aACtB,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAE,CAAC;aAClC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,IAAI,KAAK,CAAC,CAAC,CAAC,CAAC;QAE1E,MAAM,CACJ,SAAS,EACT,2CAA2C,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAClE,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAChB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mFAAmF,EAAE,GAAG,EAAE;QAC3F,MAAM,QAAQ,GAAG,IAAI,CAAC,OAAO,CAAC,SAAS,EAAE,aAAa,CAAC,CAAC;QACxD,wCAAwC;QACxC,sCAAsC;QACtC,oDAAoD;QACpD,iFAAiF;QACjF,4EAA4E;QAC5E,yEAAyE;QACzE,sBAAsB;QACtB,MAAM,OAAO,GAAG;YACd,mCAAmC;YACnC,8BAA8B;YAC9B,mCAAmC;YACnC,wCAAwC;YACxC,yCAAyC;YACzC,mDAAmD;YACnD,gDAAgD;YAChD,oDAAoD;YACpD,qDAAqD;YACrD,gCAAgC;YAChC,6BAA6B;YAC7B,wDAAwD;SACzD,CAAC;QAEF,IAAI,OAAO,GAAa,EAAE,CAAC;QAC3B,IAAI,CAAC;YACH,MAAM,GAAG,GAAG,YAAY,CACtB,KAAK,EACL,CAAC,MAAM,EAAE,KAAK,EAAE,wBAAwB,EAAE,IAAI,EAAE,kBAAkB,CAAC,EACnE,EAAE,GAAG,EAAE,QAAQ,EAAE,QAAQ,EAAE,MAAM,EAAE,CACpC,CAAC;YACF,OAAO,GAAG,GAAG,CAAC,IAAI,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;QACnD,CAAC;QAAC,MAAM,CAAC;YACP,aAAa;QACf,CAAC;QAED,MAAM,SAAS,GAAG,OAAO;aACtB,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAE,CAAC;aAClC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,IAAI,KAAK,CAAC,CAAC,CAAC,CAAC;QAE1E,MAAM,CACJ,SAAS,EACT,kDAAkD,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CACzE,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAChB,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
+{"version":3,"file":"audit-immutability.test.js","sourceRoot":"","sources":["../../src/__tests__/audit-immutability.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AACH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAClD,OAAO,IAAI,MAAM,WAAW,CAAC;AAE7B,QAAQ,CAAC,2BAA2B,EAAE,GAAG,EAAE;IACzC,EAAE,CAAC,gEAAgE,EAAE,GAAG,EAAE;QACxE,MAAM,QAAQ,GAAG,IAAI,CAAC,OAAO,CAAC,SAAS,EAAE,aAAa,CAAC,CAAC;QACxD,MAAM,OAAO,GAAG;YACd,sCAAsC;YACtC,qDAAqD;YACrD,wDAAwD;SACzD,CAAC;QAEF,MAAM,QAAQ,GAAG;YACf,iCAAiC;YACjC,iCAAiC;SAClC,CAAC;QAEF,MAAM,OAAO,GAAa,EAAE,CAAC;QAC7B,KAAK,MAAM,GAAG,IAAI,QAAQ,EAAE,CAAC;YAC3B,IAAI,CAAC;gBACH,MAAM,GAAG,GAAG,YAAY,CACtB,KAAK,EACL,CAAC,MAAM,EAAE,KAAK,EAAE,GAAG,EAAE,IAAI,EAAE,kBAAkB,CAAC,EAC9C,EAAE,GAAG,EAAE,QAAQ,EAAE,QAAQ,EAAE,MAAM,EAAE,CACpC,CAAC;gBACF,OAAO,CAAC,IAAI,CAAC,GAAG,GAAG,CAAC,IAAI,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC;YAC1D,CAAC;YAAC,MAAM,CAAC;gBACP,0DAA0D;YAC5D,CAAC;QACH,CAAC;QAED,MAAM,SAAS,GAAG,OAAO;aACtB,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAE,CAAC;aAClC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,IAAI,KAAK,CAAC,CAAC,CAAC,CAAC;QAE1E,MAAM,CACJ,SAAS,EACT,2CAA2C,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAClE,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAChB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mFAAmF,EAAE,GAAG,EAAE;QAC3F,MAAM,QAAQ,GAAG,IAAI,CAAC,OAAO,CAAC,SAAS,EAAE,aAAa,CAAC,CAAC;QACxD,wCAAwC;QACxC,sCAAsC;QACtC,oDAAoD;QACpD,iFAAiF;QACjF,4EAA4E;QAC5E,yEAAyE;QACzE,sBAAsB;QACtB,MAAM,OAAO,GAAG;YACd,mCAAmC;YACnC,8BAA8B;YAC9B,mCAAmC;YACnC,wCAAwC;YACxC,yCAAyC;YACzC,4CAA4C;YAC5C,mDAAmD;YACnD,uDAAuD;YACvD,gDAAgD;YAChD,oDAAoD;YACpD,qDAAqD;YACrD,gCAAgC;YAChC,6BAA6B;YAC7B,wDAAwD;SACzD,CAAC;QAEF,IAAI,OAAO,GAAa,EAAE,CAAC;QAC3B,IAAI,CAAC;YACH,MAAM,GAAG,GAAG,YAAY,CACtB,KAAK,EACL,CAAC,MAAM,EAAE,KAAK,EAAE,wBAAwB,EAAE,IAAI,EAAE,kBAAkB,CAAC,EACnE,EAAE,GAAG,EAAE,QAAQ,EAAE,QAAQ,EAAE,MAAM,EAAE,CACpC,CAAC;YACF,OAAO,GAAG,GAAG,CAAC,IAAI,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;QACnD,CAAC;QAAC,MAAM,CAAC;YACP,aAAa;QACf,CAAC;QAED,MAAM,SAAS,GAAG,OAAO;aACtB,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAE,CAAC;aAClC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,IAAI,KAAK,CAAC,CAAC,CAAC,CAAC;QAE1E,MAAM,CACJ,SAAS,EACT,kDAAkD,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CACzE,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAChB,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/__tests__/bec-181-circuit-breaker-audit-gap.test.d.ts

@@ -0,0 +1,24 @@
+/**
+ * BEC-181 — Fix: circuit breaker now emits pm.skipped_circuit_breaker audit events.
+ *
+ * This file verifies:
+ *
+ * 1. WORKING: The circuit breaker correctly skips issues in both promoteReadyIssues
+ *    and startTodoIssues when an issue has ≥ maxConsecutiveFailures failed runs.
+ *    countConsecutiveFailures returns 4 for 4 consecutive failed runs with no
+ *    intervening completion.
+ *
+ * 2. FIXED: When the breaker fires in promote or start-todo, a `pm.skipped_circuit_breaker`
+ *    audit event is now written to the database. Operators can distinguish
+ *    "breaker prevented a re-promotion" from "issue was never a candidate"
+ *    by querying audit_events.
+ *
+ * Acceptance criteria from BEC-181 that this test covers:
+ *   - countConsecutiveFailures is verified to return 4 for 4 consecutive failures
+ *   - promote and start-todo skip issues at/above the threshold (end-to-end)
+ *   - audit_events table has 1 row with eventType "pm.skipped_circuit_breaker" after a skip
+ *   - AuditEventTypeSchema contains "pm.skipped_circuit_breaker"
+ *   - audit/events.ts exports pmSkippedCircuitBreakerEvent builder
+ */
+export {};
+//# sourceMappingURL=bec-181-circuit-breaker-audit-gap.test.d.ts.map

package/dist/__tests__/bec-181-circuit-breaker-audit-gap.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"bec-181-circuit-breaker-audit-gap.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/bec-181-circuit-breaker-audit-gap.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;GAqBG"}

package/dist/__tests__/bec-181-circuit-breaker-audit-gap.test.js

@@ -0,0 +1,253 @@
+/**
+ * BEC-181 — Fix: circuit breaker now emits pm.skipped_circuit_breaker audit events.
+ *
+ * This file verifies:
+ *
+ * 1. WORKING: The circuit breaker correctly skips issues in both promoteReadyIssues
+ *    and startTodoIssues when an issue has ≥ maxConsecutiveFailures failed runs.
+ *    countConsecutiveFailures returns 4 for 4 consecutive failed runs with no
+ *    intervening completion.
+ *
+ * 2. FIXED: When the breaker fires in promote or start-todo, a `pm.skipped_circuit_breaker`
+ *    audit event is now written to the database. Operators can distinguish
+ *    "breaker prevented a re-promotion" from "issue was never a candidate"
+ *    by querying audit_events.
+ *
+ * Acceptance criteria from BEC-181 that this test covers:
+ *   - countConsecutiveFailures is verified to return 4 for 4 consecutive failures
+ *   - promote and start-todo skip issues at/above the threshold (end-to-end)
+ *   - audit_events table has 1 row with eventType "pm.skipped_circuit_breaker" after a skip
+ *   - AuditEventTypeSchema contains "pm.skipped_circuit_breaker"
+ *   - audit/events.ts exports pmSkippedCircuitBreakerEvent builder
+ */
+import { describe, it, expect, vi, afterEach } from "vitest";
+import { randomBytes } from "node:crypto";
+import { unlinkSync } from "node:fs";
+import { createDb } from "../db/index.js";
+import { pipelineRuns, auditEvents } from "../db/schema.js";
+import { countConsecutiveFailures } from "../pm/actions/db-queries.js";
+import { promoteReadyIssues } from "../pm/actions/promote.js";
+import { startTodoIssues } from "../pm/actions/start-todo.js";
+// ---------------------------------------------------------------------------
+// Helpers
+// ---------------------------------------------------------------------------
+function tmpDbPath() {
+    return `/tmp/bec-181-repro-${randomBytes(8).toString("hex")}.sqlite`;
+}
+const paths = [];
+afterEach(() => {
+    for (const p of paths) {
+        for (const suffix of ["", "-wal", "-shm"]) {
+            try {
+                unlinkSync(p + suffix);
+            }
+            catch { /* ignore */ }
+        }
+    }
+    paths.length = 0;
+});
+async function makeDb() {
+    const path = tmpDbPath();
+    paths.push(path);
+    return createDb({ driver: "sqlite", connectionString: path });
+}
+/**
+ * Seed N consecutive failed runs for issueId, spaced 5 seconds apart.
+ * All are terminal (status="failed") with no completion between them.
+ */
+async function seedConsecutiveFailures(db, issueId, count) {
+    const t0 = new Date(Date.now() - 60 * 60_000); // 1 hour ago so not "recent"
+    const rows = Array.from({ length: count }, (_, i) => ({
+        id: `run-${issueId}-${i}`,
+        issueId,
+        issueTitle: `Issue ${issueId}`,
+        pipelineKey: "auto-implement",
+        repoUrl: "https://github.com/org/repo",
+        status: "failed",
+        startedAt: new Date(t0.getTime() + i * 5_000),
+        completedAt: new Date(t0.getTime() + i * 5_000 + 1_000),
+    }));
+    await db.insert(pipelineRuns).values(rows);
+}
+// ---------------------------------------------------------------------------
+// PART 1: Verify countConsecutiveFailures returns 4 for 4 consecutive failures
+// ---------------------------------------------------------------------------
+describe("BEC-181 Part 1: countConsecutiveFailures (working correctly)", () => {
+    it("returns 4 when issue has 4 consecutive failed runs and no completed run", async () => {
+        const db = await makeDb();
+        await seedConsecutiveFailures(db, "BEC-147", 4);
+        const count = await countConsecutiveFailures(db, "BEC-147");
+        // EXPECTED: 4 — the circuit breaker should engage at threshold 3
+        expect(count).toBe(4);
+    });
+    it("returns 3 when issue has exactly 3 consecutive failed runs (at default threshold)", async () => {
+        const db = await makeDb();
+        await seedConsecutiveFailures(db, "BEC-157", 3);
+        const count = await countConsecutiveFailures(db, "BEC-157");
+        // EXPECTED: 3 — at the default threshold of 3, breaker should engage
+        expect(count).toBe(3);
+    });
+});
+// ---------------------------------------------------------------------------
+// PART 2: Verify promoteReadyIssues skips the issue (circuit breaker fires)
+// ---------------------------------------------------------------------------
+/** Allow fire-and-forget audit DB writes to complete before asserting. */
+async function flushFireAndForget() {
+    await new Promise((r) => setImmediate(r));
+    await new Promise((r) => setImmediate(r));
+}
+/**
+ * Shared helper: runs promoteReadyIssues with the circuit breaker enabled
+ * for a single pre-seeded issue and asserts both the skip and the audit event.
+ */
+async function assertPromoteBreakerSkip(opts) {
+    const db = await makeDb();
+    await seedConsecutiveFailures(db, opts.issueId, opts.failureCount);
+    const issues = [
+        {
+            id: opts.linearUuid,
+            identifier: opts.issueId,
+            title: opts.issueTitle,
+            description: "Some description",
+            priority: 2,
+            labels: vi.fn().mockResolvedValue({ nodes: [{ name: "auto-implement" }] }),
+            team: Promise.resolve({ id: "team-1" }),
+            url: `https://linear.app/beckerspace/issue/${opts.issueId}`,
+        },
+    ];
+    const client = {
+        issues: vi.fn().mockResolvedValue({ nodes: issues }),
+        updateIssue: vi.fn(),
+        createComment: vi.fn(),
+    };
+    const stateMap = new Map([["team-1:Todo", "state-todo"]]);
+    const conflictChecker = vi.fn();
+    // Omit getFailureCount to exercise the batchCountConsecutiveFailures path.
+    // The production scheduler also omits getFailureCount after BEC-181 so
+    // both promote and start-todo benefit from the batch optimization.
+    const results = await promoteReadyIssues({
+        linearClient: client,
+        teamIds: ["team-1"],
+        slotsAvailable: 1,
+        checkConflict: conflictChecker,
+        stateMap,
+        db,
+        maxConsecutiveFailures: opts.threshold,
+    });
+    // Circuit breaker fires and prevents promotion
+    expect(results).toHaveLength(1);
+    expect(results[0].promoted).toBe(false);
+    expect(results[0].reason).toMatch(/circuit.breaker/i);
+    expect(results[0].reason).toContain(`${opts.failureCount} consecutive failed runs`);
+    expect(client.updateIssue).not.toHaveBeenCalled();
+    // Breaker fires before conflict detection (saves tokens)
+    expect(conflictChecker).not.toHaveBeenCalled();
+    await flushFireAndForget();
+    // Audit event is now written when the breaker fires
+    const auditRows = await db.select().from(auditEvents);
+    expect(auditRows).toHaveLength(1);
+    expect(auditRows[0].eventType).toBe("pm.skipped_circuit_breaker");
+    expect(auditRows[0].issueId).toBe(opts.issueId);
+    expect(JSON.parse(auditRows[0].payload).failureCount).toBe(opts.failureCount);
+    expect(JSON.parse(auditRows[0].payload).threshold).toBe(opts.threshold);
+    expect(JSON.parse(auditRows[0].payload).source).toBe("promote");
+}
+describe("BEC-181 Part 2: promoteReadyIssues circuit-breaker skip (working correctly)", () => {
+    it("skips promotion for issue with 4 consecutive failures and writes a pm.skipped_circuit_breaker audit event (fix verified)", () => assertPromoteBreakerSkip({
+        issueId: "BEC-147",
+        issueTitle: "Tech debt: changelog UNRELEASED section stale",
+        linearUuid: "uuid-bec147",
+        failureCount: 4,
+        threshold: 3,
+    }));
+    it("skips promotion for BEC-157 with exactly 3 consecutive failures (at threshold)", () => assertPromoteBreakerSkip({
+        issueId: "BEC-157",
+        issueTitle: "Pipeline: filter agent scratchpad files",
+        linearUuid: "uuid-bec157",
+        failureCount: 3,
+        threshold: 3,
+    }));
+});
+// ---------------------------------------------------------------------------
+// PART 3: Verify startTodoIssues skips the issue (circuit breaker fires)
+// ---------------------------------------------------------------------------
+describe("BEC-181 Part 3: startTodoIssues circuit-breaker skip (working correctly)", () => {
+    const pipelineConfigs = {
+        "auto-implement": {
+            stages: ["triage", "implement", "review"],
+        },
+    };
+    const repoConfigs = {
+        "team-1": {
+            url: "https://github.com/org/repo",
+            defaultBranch: "main",
+        },
+    };
+    it("skips start for issue with 4 consecutive failures WITHOUT touching Linear SDK, and writes an audit event (fix verified)", async () => {
+        const db = await makeDb();
+        await seedConsecutiveFailures(db, "BEC-147", 4);
+        // Spy on Linear SDK lazy relations to prove they are NOT called
+        const labelsSpy = vi.fn().mockResolvedValue({ nodes: [{ name: "auto-implement" }] });
+        const issue = {
+            id: "uuid-bec147",
+            identifier: "BEC-147",
+            title: "Tech debt: changelog UNRELEASED section stale",
+            description: "",
+            priority: 2,
+            team: Promise.resolve({ id: "team-1" }),
+            project: Promise.resolve({ id: "proj-1" }),
+            labels: labelsSpy,
+        };
+        // Use the real DB — seeded rows are 1 hour old, outside the 30-minute recent window,
+        // and all are "failed" (not "queued"/"running"), so getActiveAndRecentIssueIds
+        // correctly returns empty sets, making BEC-147 appear as an orphaned Todo issue.
+        const runner = { start: vi.fn() };
+        // Omit getFailureCount to exercise the batchCountConsecutiveFailures path
+        // (single DB round-trip). The production scheduler also omits getFailureCount
+        // after BEC-181 so it benefits from the batch optimization.
+        const input = {
+            linearClient: { issues: vi.fn().mockResolvedValue({ nodes: [issue] }) },
+            db,
+            teamIds: ["team-1"],
+            runner: runner,
+            pipelineConfigs,
+            repoConfigs,
+            maxPerTick: 5,
+            maxConsecutiveFailures: 3,
+        };
+        const results = await startTodoIssues(input);
+        // VERIFIED WORKING: breaker skips the issue
+        expect(results).toHaveLength(1);
+        expect(results[0].started).toBe(false);
+        expect(results[0].reason).toMatch(/circuit.breaker/i);
+        expect(results[0].reason).toContain("4 consecutive failed runs");
+        expect(runner.start).not.toHaveBeenCalled();
+        // Breaker fires BEFORE Linear SDK calls (saves 3 round-trips per tick per doom-looping issue)
+        expect(labelsSpy).not.toHaveBeenCalled();
+        // Allow fire-and-forget audit DB writes to complete before asserting.
+        await flushFireAndForget();
+        // VERIFIED FIX: audit event is now written when the circuit breaker fires in start-todo
+        const auditRows = await db.select().from(auditEvents);
+        expect(auditRows).toHaveLength(1);
+        expect(auditRows[0].eventType).toBe("pm.skipped_circuit_breaker");
+        expect(auditRows[0].issueId).toBe("BEC-147");
+        expect(JSON.parse(auditRows[0].payload).failureCount).toBe(4);
+        expect(JSON.parse(auditRows[0].payload).threshold).toBe(3);
+        expect(JSON.parse(auditRows[0].payload).source).toBe("start-todo");
+    });
+});
+// ---------------------------------------------------------------------------
+// PART 4: Confirm pm.skipped_circuit_breaker is present in AuditEventTypeSchema (fix verified)
+// ---------------------------------------------------------------------------
+describe("BEC-181 Part 4: pm.skipped_circuit_breaker event type is now defined (gap closed)", () => {
+    it("AuditEventTypeSchema contains pm.skipped_circuit_breaker", async () => {
+        const { AuditEventTypeSchema } = await import("../types.js");
+        const types = AuditEventTypeSchema.options;
+        expect(types).toContain("pm.skipped_circuit_breaker");
+    });
+    it("audit/events.ts exports pmSkippedCircuitBreakerEvent builder", async () => {
+        const auditModule = await import("../audit/events.js");
+        expect(typeof auditModule.pmSkippedCircuitBreakerEvent).toBe("function");
+    });
+});
+//# sourceMappingURL=bec-181-circuit-breaker-audit-gap.test.js.map

package/dist/__tests__/bec-181-circuit-breaker-audit-gap.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"bec-181-circuit-breaker-audit-gap.test.js","sourceRoot":"","sources":["../../src/__tests__/bec-181-circuit-breaker-audit-gap.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;GAqBG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAC;AAC7D,OAAO,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAC1C,OAAO,EAAE,UAAU,EAAE,MAAM,SAAS,CAAC;AACrC,OAAO,EAAE,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AAC1C,OAAO,EAAE,YAAY,EAAE,WAAW,EAAE,MAAM,iBAAiB,CAAC;AAC5D,OAAO,EAAE,wBAAwB,EAAE,MAAM,6BAA6B,CAAC;AACvE,OAAO,EAAE,kBAAkB,EAAE,MAAM,0BAA0B,CAAC;AAC9D,OAAO,EAAE,eAAe,EAAuB,MAAM,6BAA6B,CAAC;AAEnF,8EAA8E;AAC9E,UAAU;AACV,8EAA8E;AAE9E,SAAS,SAAS;IAChB,OAAO,sBAAsB,WAAW,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,KAAK,CAAC,SAAS,CAAC;AACvE,CAAC;AAED,MAAM,KAAK,GAAa,EAAE,CAAC;AAC3B,SAAS,CAAC,GAAG,EAAE;IACb,KAAK,MAAM,CAAC,IAAI,KAAK,EAAE,CAAC;QACtB,KAAK,MAAM,MAAM,IAAI,CAAC,EAAE,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,CAAC;YAC1C,IAAI,CAAC;gBAAC,UAAU,CAAC,CAAC,GAAG,MAAM,CAAC,CAAC;YAAC,CAAC;YAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC;QACxD,CAAC;IACH,CAAC;IACD,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC;AACnB,CAAC,CAAC,CAAC;AAEH,KAAK,UAAU,MAAM;IACnB,MAAM,IAAI,GAAG,SAAS,EAAE,CAAC;IACzB,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACjB,OAAO,QAAQ,CAAC,EAAE,MAAM,EAAE,QAAQ,EAAE,gBAAgB,EAAE,IAAI,EAAE,CAAC,CAAC;AAChE,CAAC;AAED;;;GAGG;AACH,KAAK,UAAU,uBAAuB,CAAC,EAAO,EAAE,OAAe,EAAE,KAAa;IAC5E,MAAM,EAAE,GAAG,IAAI,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,EAAE,GAAG,MAAM,CAAC,CAAC,CAAC,6BAA6B;IAC5E,MAAM,IAAI,GAAG,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,KAAK,EAAE,EAAE,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;QACpD,EAAE,EAAE,OAAO,OAAO,IAAI,CAAC,EAAE;QACzB,OAAO;QACP,UAAU,EAAE,SAAS,OAAO,EAAE;QAC9B,WAAW,EAAE,gBAAgB;QAC7B,OAAO,EAAE,6BAA6B;QACtC,MAAM,EAAE,QAAQ;QAChB,SAAS,EAAE,IAAI,IAAI,CAAC,EAAE,CAAC,OAAO,EAAE,GAAG,CAAC,GAAG,KAAK,CAAC;QAC7C,WAAW,EAAE,IAAI,IAAI,CAAC,EAAE,CAAC,OAAO,EAAE,GAAG,CAAC,GAAG,KAAK,GAAG,KAAK,CAAC;KACxD,CAAC,CAAC,CAAC;IACJ,MAAM,EAAE,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;AAC7C,CAAC;AAED,8EAA8E;AAC9E,+EAA+E;AAC/E,8EAA8E;AAE9E,QAAQ,CAAC,8DAA8D,EAAE,GAAG,EAAE;IAC5E,EAAE,CAAC,yEAAyE,EAAE,KAAK,IAAI,EAAE;QACvF,MAAM,EAAE,GAAG,MAAM,MAAM,EAAS,CAAC;QACjC,MAAM,uBAAuB,CAAC,EAAE,EAAE,SAAS,EAAE,CAAC,CAAC,CAAC;QAChD,MAAM,KAAK,GAAG,MAAM,wBAAwB,CAAC,EAAE,EAAE,SAAS,CAAC,CAAC;QAC5D,iEAAiE;QACjE,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IACxB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mFAAmF,EAAE,KAAK,IAAI,EAAE;QACjG,MAAM,EAAE,GAAG,MAAM,MAAM,EAAS,CAAC;QACjC,MAAM,uBAAuB,CAAC,EAAE,EAAE,SAAS,EAAE,CAAC,CAAC,CAAC;QAChD,MAAM,KAAK,GAAG,MAAM,wBAAwB,CAAC,EAAE,EAAE,SAAS,CAAC,CAAC;QAC5D,qEAAqE;QACrE,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IACxB,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,8EAA8E;AAC9E,4EAA4E;AAC5E,8EAA8E;AAE9E,0EAA0E;AAC1E,KAAK,UAAU,kBAAkB;IAC/B,MAAM,IAAI,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,CAAC;IAC1C,MAAM,IAAI,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,CAAC;AAC5C,CAAC;AAED;;;GAGG;AACH,KAAK,UAAU,wBAAwB,CAAC,IAMvC;IACC,MAAM,EAAE,GAAG,MAAM,MAAM,EAAS,CAAC;IACjC,MAAM,uBAAuB,CAAC,EAAE,EAAE,IAAI,CAAC,OAAO,EAAE,IAAI,CAAC,YAAY,CAAC,CAAC;IAEnE,MAAM,MAAM,GAAG;QACb;YACE,EAAE,EAAE,IAAI,CAAC,UAAU;YACnB,UAAU,EAAE,IAAI,CAAC,OAAO;YACxB,KAAK,EAAE,IAAI,CAAC,UAAU;YACtB,WAAW,EAAE,kBAAkB;YAC/B,QAAQ,EAAE,CAAC;YACX,MAAM,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,IAAI,EAAE,gBAAgB,EAAE,CAAC,EAAE,CAAC;YAC1E,IAAI,EAAE,OAAO,CAAC,OAAO,CAAC,EAAE,EAAE,EAAE,QAAQ,EAAE,CAAC;YACvC,GAAG,EAAE,wCAAwC,IAAI,CAAC,OAAO,EAAE;SAC5D;KACF,CAAC;IACF,MAAM,MAAM,GAAG;QACb,MAAM,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC;QACpD,WAAW,EAAE,EAAE,CAAC,EAAE,EAAE;QACpB,aAAa,EAAE,EAAE,CAAC,EAAE,EAAE;KACvB,CAAC;IACF,MAAM,QAAQ,GAAG,IAAI,GAAG,CAAC,CAAC,CAAC,aAAa,EAAE,YAAY,CAAC,CAAC,CAAC,CAAC;IAC1D,MAAM,eAAe,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;IAEhC,2EAA2E;IAC3E,uEAAuE;IACvE,mEAAmE;IACnE,MAAM,OAAO,GAAG,MAAM,kBAAkB,CAAC;QACvC,YAAY,EAAE,MAAa;QAC3B,OAAO,EAAE,CAAC,QAAQ,CAAC;QACnB,cAAc,EAAE,CAAC;QACjB,aAAa,EAAE,eAAe;QAC9B,QAAQ;QACR,EAAE;QACF,sBAAsB,EAAE,IAAI,CAAC,SAAS;KACvC,CAAC,CAAC;IAEH,+CAA+C;IAC/C,MAAM,CAAC,OAAO,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IAChC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACxC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,kBAAkB,CAAC,CAAC;IACtD,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,GAAG,IAAI,CAAC,YAAY,0BAA0B,CAAC,CAAC;IACpF,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IAClD,yDAAyD;IACzD,MAAM,CAAC,eAAe,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IAE/C,MAAM,kBAAkB,EAAE,CAAC;IAE3B,oDAAoD;IACpD,MAAM,SAAS,GAAG,MAAO,EAAU,CAAC,MAAM,EAAE,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;IAC/D,MAAM,CAAC,SAAS,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IAClC,MAAM,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,4BAA4B,CAAC,CAAC;IAClE,MAAM,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;IAChD,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;IAC9E,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IACxE,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;AAClE,CAAC;AAED,QAAQ,CAAC,6EAA6E,EAAE,GAAG,EAAE;IAC3F,EAAE,CAAC,0HAA0H,EAAE,GAAG,EAAE,CAClI,wBAAwB,CAAC;QACvB,OAAO,EAAE,SAAS;QAClB,UAAU,EAAE,+CAA+C;QAC3D,UAAU,EAAE,aAAa;QACzB,YAAY,EAAE,CAAC;QACf,SAAS,EAAE,CAAC;KACb,CAAC,CAAC,CAAC;IAEN,EAAE,CAAC,gFAAgF,EAAE,GAAG,EAAE,CACxF,wBAAwB,CAAC;QACvB,OAAO,EAAE,SAAS;QAClB,UAAU,EAAE,yCAAyC;QACrD,UAAU,EAAE,aAAa;QACzB,YAAY,EAAE,CAAC;QACf,SAAS,EAAE,CAAC;KACb,CAAC,CAAC,CAAC;AACR,CAAC,CAAC,CAAC;AAEH,8EAA8E;AAC9E,yEAAyE;AACzE,8EAA8E;AAE9E,QAAQ,CAAC,0EAA0E,EAAE,GAAG,EAAE;IACxF,MAAM,eAAe,GAAwB;QAC3C,gBAAgB,EAAE;YAChB,MAAM,EAAE,CAAC,QAAQ,EAAE,WAAW,EAAE,QAAQ,CAAC;SAC1C;KACF,CAAC;IACF,MAAM,WAAW,GAAwB;QACvC,QAAQ,EAAE;YACR,GAAG,EAAE,6BAA6B;YAClC,aAAa,EAAE,MAAM;SACtB;KACF,CAAC;IAEF,EAAE,CAAC,yHAAyH,EAAE,KAAK,IAAI,EAAE;QACvI,MAAM,EAAE,GAAG,MAAM,MAAM,EAAS,CAAC;QACjC,MAAM,uBAAuB,CAAC,EAAE,EAAE,SAAS,EAAE,CAAC,CAAC,CAAC;QAEhD,gEAAgE;QAChE,MAAM,SAAS,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,IAAI,EAAE,gBAAgB,EAAE,CAAC,EAAE,CAAC,CAAC;QACrF,MAAM,KAAK,GAAG;YACZ,EAAE,EAAE,aAAa;YACjB,UAAU,EAAE,SAAS;YACrB,KAAK,EAAE,+CAA+C;YACtD,WAAW,EAAE,EAAE;YACf,QAAQ,EAAE,CAAC;YACX,IAAI,EAAE,OAAO,CAAC,OAAO,CAAC,EAAE,EAAE,EAAE,QAAQ,EAAE,CAAC;YACvC,OAAO,EAAE,OAAO,CAAC,OAAO,CAAC,EAAE,EAAE,EAAE,QAAQ,EAAE,CAAC;YAC1C,MAAM,EAAE,SAAS;SAClB,CAAC;QAEF,qFAAqF;QACrF,+EAA+E;QAC/E,iFAAiF;QACjF,MAAM,MAAM,GAAG,EAAE,KAAK,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC;QAClC,0EAA0E;QAC1E,8EAA8E;QAC9E,4DAA4D;QAC5D,MAAM,KAAK,GAAmB;YAC5B,YAAY,EAAE,EAAE,MAAM,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,KAAK,EAAE,CAAC,KAAK,CAAC,EAAE,CAAC,EAAE;YACvE,EAAE;YACF,OAAO,EAAE,CAAC,QAAQ,CAAC;YACnB,MAAM,EAAE,MAAa;YACrB,eAAe;YACf,WAAW;YACX,UAAU,EAAE,CAAC;YACb,sBAAsB,EAAE,CAAC;SAC1B,CAAC;QAEF,MAAM,OAAO,GAAG,MAAM,eAAe,CAAC,KAAK,CAAC,CAAC;QAE7C,4CAA4C;QAC5C,MAAM,CAAC,OAAO,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAChC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QACvC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,kBAAkB,CAAC,CAAC;QACtD,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,2BAA2B,CAAC,CAAC;QACjE,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QAC5C,8FAA8F;QAC9F,MAAM,CAAC,SAAS,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QAEzC,sEAAsE;QACtE,MAAM,kBAAkB,EAAE,CAAC;QAE3B,wFAAwF;QACxF,MAAM,SAAS,GAAG,MAAO,EAAU,CAAC,MAAM,EAAE,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;QAC/D,MAAM,CAAC,SAAS,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAClC,MAAM,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,4BAA4B,CAAC,CAAC;QAClE,MAAM,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QAC7C,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAC9D,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAC3D,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;IACrE,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,8EAA8E;AAC9E,+FAA+F;AAC/F,8EAA8E;AAE9E,QAAQ,CAAC,mFAAmF,EAAE,GAAG,EAAE;IACjG,EAAE,CAAC,0DAA0D,EAAE,KAAK,IAAI,EAAE;QACxE,MAAM,EAAE,oBAAoB,EAAE,GAAG,MAAM,MAAM,CAAC,aAAa,CAAC,CAAC;QAC7D,MAAM,KAAK,GAAa,oBAAoB,CAAC,OAAO,CAAC;QACrD,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,4BAA4B,CAAC,CAAC;IACxD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,KAAK,IAAI,EAAE;QAC5E,MAAM,WAAW,GAAG,MAAM,MAAM,CAAC,oBAAoB,CAAC,CAAC;QACvD,MAAM,CAAC,OAAQ,WAAmB,CAAC,4BAA4B,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;IACpF,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/__tests__/executor/review-feedback-profile-override.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=review-feedback-profile-override.test.d.ts.map

package/dist/__tests__/executor/review-feedback-profile-override.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-feedback-profile-override.test.d.ts","sourceRoot":"","sources":["../../../src/__tests__/executor/review-feedback-profile-override.test.ts"],"names":[],"mappings":""}

package/dist/__tests__/executor/review-feedback-profile-override.test.js

@@ -0,0 +1,32 @@
+import { describe, it, expect } from "vitest";
+import { applyReviewFeedbackProfileOverride } from "../../executor/executor.js";
+const baseProfile = {
+    tools: ["Read", "Write", "Edit"],
+    maxInputTokens: 100_000,
+    maxTurns: 100,
+    model: "claude-sonnet-4-6",
+};
+describe("applyReviewFeedbackProfileOverride (BEC-182)", () => {
+    it("caps maxTurns + maxInputTokens for implement stage when review-feedback context is present", () => {
+        const out = applyReviewFeedbackProfileOverride(baseProfile, "implement", true);
+        expect(out.maxTurns).toBe(30);
+        expect(out.maxInputTokens).toBe(60_000);
+        // Other fields preserved
+        expect(out.tools).toEqual(baseProfile.tools);
+        expect(out.model).toBe(baseProfile.model);
+    });
+    it("returns the profile unchanged on non-implement stages even with review-feedback context", () => {
+        const out = applyReviewFeedbackProfileOverride(baseProfile, "test", true);
+        expect(out).toEqual(baseProfile);
+    });
+    it("returns the profile unchanged when review-feedback context is absent", () => {
+        const out = applyReviewFeedbackProfileOverride(baseProfile, "implement", false);
+        expect(out).toEqual(baseProfile);
+    });
+    it("does not mutate the input profile", () => {
+        const before = JSON.parse(JSON.stringify(baseProfile));
+        applyReviewFeedbackProfileOverride(baseProfile, "implement", true);
+        expect(baseProfile).toEqual(before);
+    });
+});
+//# sourceMappingURL=review-feedback-profile-override.test.js.map

package/dist/__tests__/executor/review-feedback-profile-override.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-feedback-profile-override.test.js","sourceRoot":"","sources":["../../../src/__tests__/executor/review-feedback-profile-override.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,kCAAkC,EAAE,MAAM,4BAA4B,CAAC;AAEhF,MAAM,WAAW,GAAG;IAClB,KAAK,EAAE,CAAC,MAAM,EAAE,OAAO,EAAE,MAAM,CAAC;IAChC,cAAc,EAAE,OAAO;IACvB,QAAQ,EAAE,GAAG;IACb,KAAK,EAAE,mBAAmB;CAC3B,CAAC;AAEF,QAAQ,CAAC,8CAA8C,EAAE,GAAG,EAAE;IAC5D,EAAE,CAAC,4FAA4F,EAAE,GAAG,EAAE;QACpG,MAAM,GAAG,GAAG,kCAAkC,CAAC,WAAW,EAAE,WAAW,EAAE,IAAI,CAAC,CAAC;QAC/E,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAC9B,MAAM,CAAC,GAAG,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QACxC,yBAAyB;QACzB,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC,OAAO,CAAC,WAAW,CAAC,KAAK,CAAC,CAAC;QAC7C,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,KAAK,CAAC,CAAC;IAC5C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yFAAyF,EAAE,GAAG,EAAE;QACjG,MAAM,GAAG,GAAG,kCAAkC,CAAC,WAAW,EAAE,MAAM,EAAE,IAAI,CAAC,CAAC;QAC1E,MAAM,CAAC,GAAG,CAAC,CAAC,OAAO,CAAC,WAAW,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sEAAsE,EAAE,GAAG,EAAE;QAC9E,MAAM,GAAG,GAAG,kCAAkC,CAAC,WAAW,EAAE,WAAW,EAAE,KAAK,CAAC,CAAC;QAChF,MAAM,CAAC,GAAG,CAAC,CAAC,OAAO,CAAC,WAAW,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;QAC3C,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,WAAW,CAAC,CAAC,CAAC;QACvD,kCAAkC,CAAC,WAAW,EAAE,WAAW,EAAE,IAAI,CAAC,CAAC;QACnE,MAAM,CAAC,WAAW,CAAC,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/__tests__/executor/review-feedback-prompt.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=review-feedback-prompt.test.d.ts.map

package/dist/__tests__/executor/review-feedback-prompt.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-feedback-prompt.test.d.ts","sourceRoot":"","sources":["../../../src/__tests__/executor/review-feedback-prompt.test.ts"],"names":[],"mappings":""}

package/dist/__tests__/executor/review-feedback-prompt.test.js

@@ -0,0 +1,29 @@
+import { describe, it, expect } from "vitest";
+import { implementTemplate } from "../../executor/prompt/templates.js";
+const baseIssue = { id: "BEC-999", identifier: "BEC-999", title: "T", description: "D", priority: 3, labels: [], slug: "bec-999-t" };
+const baseRepo = { url: "https://github.com/x/y", defaultBranch: "main", testCommand: "pnpm test", buildCommand: "pnpm build" };
+describe("implementTemplate — review-feedback prompt (BEC-182)", () => {
+    const feedback = {
+        prBranch: "agent/BEC-999-foo",
+        prUrl: "https://github.com/x/y/pull/42",
+        comments: [{ author: "reviewer", body: "rename foo to bar", file: "src/x.ts", line: 12, createdAt: "2026-05-08T00:00:00Z" }],
+    };
+    it("includes the bounded-scope language", () => {
+        const out = implementTemplate(baseIssue, baseRepo, undefined, feedback);
+        expect(out).toMatch(/Address ONLY the listed comments/);
+        expect(out).toMatch(/Do NOT refactor adjacent code/);
+    });
+    it("instructs the agent to read the diff first", () => {
+        const out = implementTemplate(baseIssue, baseRepo, undefined, feedback);
+        expect(out).toMatch(/git diff origin\/main\.\.\.HEAD/);
+    });
+    it("makes build/test conditional on text-only changes", () => {
+        const out = implementTemplate(baseIssue, baseRepo, undefined, feedback);
+        expect(out).toMatch(/Skip build\/test ONLY if every change is text-only/);
+    });
+    it("instructs the agent to stop and report rather than spelunk on failure", () => {
+        const out = implementTemplate(baseIssue, baseRepo, undefined, feedback);
+        expect(out).toMatch(/stop and report what blocks the resolution/);
+    });
+});
+//# sourceMappingURL=review-feedback-prompt.test.js.map

package/dist/__tests__/executor/review-feedback-prompt.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-feedback-prompt.test.js","sourceRoot":"","sources":["../../../src/__tests__/executor/review-feedback-prompt.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,oCAAoC,CAAC;AAEvE,MAAM,SAAS,GAAG,EAAE,EAAE,EAAE,SAAS,EAAE,UAAU,EAAE,SAAS,EAAE,KAAK,EAAE,GAAG,EAAE,WAAW,EAAE,GAAG,EAAE,QAAQ,EAAE,CAAC,EAAE,MAAM,EAAE,EAAE,EAAE,IAAI,EAAE,WAAW,EAAS,CAAC;AAC5I,MAAM,QAAQ,GAAG,EAAE,GAAG,EAAE,wBAAwB,EAAE,aAAa,EAAE,MAAM,EAAE,WAAW,EAAE,WAAW,EAAE,YAAY,EAAE,YAAY,EAAS,CAAC;AAEvI,QAAQ,CAAC,sDAAsD,EAAE,GAAG,EAAE;IACpE,MAAM,QAAQ,GAAG;QACf,QAAQ,EAAE,mBAAmB;QAC7B,KAAK,EAAE,gCAAgC;QACvC,QAAQ,EAAE,CAAC,EAAE,MAAM,EAAE,UAAU,EAAE,IAAI,EAAE,mBAAmB,EAAE,IAAI,EAAE,UAAU,EAAE,IAAI,EAAE,EAAE,EAAE,SAAS,EAAE,sBAAsB,EAAE,CAAC;KACtH,CAAC;IAET,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;QAC7C,MAAM,GAAG,GAAG,iBAAiB,CAAC,SAAS,EAAE,QAAQ,EAAE,SAAS,EAAE,QAAQ,CAAC,CAAC;QACxE,MAAM,CAAC,GAAG,CAAC,CAAC,OAAO,CAAC,kCAAkC,CAAC,CAAC;QACxD,MAAM,CAAC,GAAG,CAAC,CAAC,OAAO,CAAC,+BAA+B,CAAC,CAAC;IACvD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;QACpD,MAAM,GAAG,GAAG,iBAAiB,CAAC,SAAS,EAAE,QAAQ,EAAE,SAAS,EAAE,QAAQ,CAAC,CAAC;QACxE,MAAM,CAAC,GAAG,CAAC,CAAC,OAAO,CAAC,iCAAiC,CAAC,CAAC;IACzD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,GAAG,EAAE;QAC3D,MAAM,GAAG,GAAG,iBAAiB,CAAC,SAAS,EAAE,QAAQ,EAAE,SAAS,EAAE,QAAQ,CAAC,CAAC;QACxE,MAAM,CAAC,GAAG,CAAC,CAAC,OAAO,CAAC,oDAAoD,CAAC,CAAC;IAC5E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uEAAuE,EAAE,GAAG,EAAE;QAC/E,MAAM,GAAG,GAAG,iBAAiB,CAAC,SAAS,EAAE,QAAQ,EAAE,SAAS,EAAE,QAAQ,CAAC,CAAC;QACxE,MAAM,CAAC,GAAG,CAAC,CAAC,OAAO,CAAC,4CAA4C,CAAC,CAAC;IACpE,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/__tests__/extract-handoff.test.js

@@ -441,6 +441,80 @@ describe("extractHandoff", () => {
             await rm(dir, { recursive: true, force: true });
         }
     });
+    // ---------------------------------------------------------------------------
+    // BEC-167: Review stage HandoffArtifact envelope — no soup-gate placeholder
+    // ---------------------------------------------------------------------------
+    it("review stage HandoffArtifact with reviewFindings parses as structured (BEC-167)", async () => {
+        // Reproduces the fix for BEC-167: the review agent now emits a full
+        // HandoffArtifact envelope. This must parse via the fast path
+        // (structured: true) and produce a real prose summary — not the
+        // "not parseable prose" soup-gate placeholder.
+        const reviewArtifact = {
+            stage: "review",
+            summary: "The implementation looks correct. No blocking issues found.",
+            filesChanged: ["packages/core/src/security/review-checklist.ts"],
+            approach: "Updated the output format to emit a HandoffArtifact envelope.",
+            context: {
+                issueIntent: "Fix review stage placeholder output",
+                constraints: [],
+                assumptions: [],
+                reviewFindings: [],
+            },
+            tokenBudget: {
+                contextTokensUsed: 500,
+                recommendedMaxTurns: 10,
+            },
+        };
+        const output = wrapInJsonBlock(reviewArtifact);
+        const result = await extractHandoff(output, "run-1", "ISS-1", "review", "/tmp");
+        // Must take the fast path — structured agent output, not git fallback
+        expect(result.structured).toBe(true);
+        // Summary must be real prose, NOT the soup-gate placeholder
+        expect(result.artifact.summary).toBe("The implementation looks correct. No blocking issues found.");
+        expect(result.artifact.summary).not.toContain("not parseable prose");
+        expect(result.artifact.summary).not.toContain("Stage review completed");
+        // reviewFindings accessible via context
+        expect(result.artifact.context.reviewFindings).toEqual([]);
+    });
+    it("review stage HandoffArtifact with blocking findings parses correctly (BEC-167)", async () => {
+        // Verifies that reviewFindings inside context round-trips through
+        // parseHandoffArtifact — downstream stages (review-fix loop, fanout)
+        // read handoff.context.reviewFindings to detect blocking issues.
+        const reviewArtifact = {
+            stage: "review",
+            summary: "Found one blocking security issue: SQL injection in user query.",
+            filesChanged: ["src/db/user-query.ts"],
+            approach: "Reviewed user query builder for injection vulnerabilities.",
+            context: {
+                issueIntent: "Add user search endpoint",
+                constraints: [],
+                assumptions: [],
+                reviewFindings: [
+                    {
+                        severity: "blocking",
+                        file: "src/db/user-query.ts",
+                        line: 42,
+                        category: "SQL Injection",
+                        description: "Unparameterized query allows SQL injection.",
+                        fix: "Use parameterized statements via the ORM.",
+                    },
+                ],
+            },
+            tokenBudget: {
+                contextTokensUsed: 800,
+                recommendedMaxTurns: 10,
+            },
+        };
+        const output = wrapInJsonBlock(reviewArtifact);
+        const result = await extractHandoff(output, "run-1", "ISS-1", "review", "/tmp");
+        expect(result.structured).toBe(true);
+        expect(result.artifact.summary).toContain("SQL injection");
+        expect(result.artifact.summary).not.toContain("not parseable prose");
+        // reviewFindings must be accessible for the review-fix loop in runner.ts
+        expect(result.artifact.context.reviewFindings).toHaveLength(1);
+        expect(result.artifact.context.reviewFindings[0].severity).toBe("blocking");
+        expect(result.artifact.context.reviewFindings[0].category).toBe("SQL Injection");
+    });
     it("returns structured: false when git diff fails (not a git repo)", async () => {
         const dir = await mkdtemp(join(tmpdir(), "extract-test-"));
         // Not a git repo — no .git directory