@urateam/core 0.1.0

package/dist/__tests__/reproduce-bec113-pagination-warning.test.js

@@ -0,0 +1,226 @@
+/**
+ * Reproduction test for BEC-113:
+ * Fix stuck-issue recovery: pagination warning and scheduler integration
+ *
+ * Gaps confirmed by this file:
+ * 1. recoverStuckInProgressIssues does NOT log a warning when Linear returns
+ *    exactly 50 issues (the `first: 50` hard cap may silently truncate results).
+ * 2. The JSDoc on recoverStuckInProgressIssues does NOT document the 50-result
+ *    hard limit or the warning behavior.
+ *
+ * The scheduler integration (AC #1/#4) and the scheduler test (AC #4) were
+ * already implemented in BEC-91 and pass as-is — confirmed at the bottom of
+ * this file.
+ */
+import { describe, it, expect, vi, beforeEach } from "vitest";
+import { recoverStuckInProgressIssues } from "../pm/actions/recover-stuck.js";
+import { createPmScheduler } from "../pm/scheduler.js";
+// ---------------------------------------------------------------------------
+// Helpers
+// ---------------------------------------------------------------------------
+function makeLinearClient(inProgressIssues) {
+    return {
+        issues: vi.fn().mockResolvedValue({ nodes: inProgressIssues }),
+        updateIssue: vi.fn().mockResolvedValue({}),
+        createComment: vi.fn().mockResolvedValue({}),
+        workflowStates: vi.fn().mockResolvedValue({ nodes: [] }),
+    };
+}
+function makeDb(activeRows = [], runs = []) {
+    let callCount = 0;
+    return {
+        select: vi.fn().mockReturnValue({
+            from: vi.fn().mockReturnValue({
+                where: vi.fn().mockImplementation(() => {
+                    callCount++;
+                    if (callCount === 1)
+                        return Promise.resolve(activeRows);
+                    return Promise.resolve(runs);
+                }),
+            }),
+        }),
+    };
+}
+/** Build N fake Linear issues (all stuck, no active runs) */
+function makeFakeIssues(count) {
+    return Array.from({ length: count }, (_, i) => ({
+        id: `issue-${i}`,
+        identifier: `BEC-${100 + i}`,
+        title: `Fake issue ${i}`,
+        team: Promise.resolve({ id: "team-1" }),
+        state: Promise.resolve({ name: "In Progress" }),
+    }));
+}
+// ---------------------------------------------------------------------------
+// GAP 1: No pagination warning when Linear returns exactly 50 results
+// ---------------------------------------------------------------------------
+describe("BEC-113 GAP: pagination truncation warning", () => {
+    beforeEach(() => {
+        vi.clearAllMocks();
+    });
+    /**
+     * When the Linear query returns exactly 50 issues the function SHOULD log:
+     *   "stuck-issue query may be truncated — consider pagination"
+     *
+     * Currently this warning is MISSING — `recoverStuckInProgressIssues` has no
+     * check for `inProgressIssues.length === 50`.
+     *
+     * This test will FAIL until the warning is implemented.
+     */
+    it("FAILS: logs truncation warning when Linear returns exactly 50 issues", async () => {
+        const issues = makeFakeIssues(50); // Exactly at the hard-cap limit
+        const linearClient = makeLinearClient(issues);
+        // All issues are stuck (no active runs)
+        const db = makeDb([], []);
+        const stateMap = new Map([["team-1:Backlog", "state-backlog-1"]]);
+        const warnSpy = vi.spyOn(process.stdout, "write");
+        await recoverStuckInProgressIssues({
+            linearClient,
+            db: db,
+            teamIds: ["team-1"],
+            targetState: "Backlog",
+            maxPerTick: 5,
+            stateMap,
+        });
+        // pino may write Buffers or strings — coerce both to string for the check
+        const allOutput = warnSpy.mock.calls
+            .map((args) => {
+            const arg = args[0];
+            if (typeof arg === "string")
+                return arg;
+            if (Buffer.isBuffer(arg))
+                return arg.toString("utf-8");
+            return "";
+        })
+            .join("");
+        expect(allOutput).toContain("stuck-issue query may be truncated — consider pagination");
+    });
+    /**
+     * When the Linear query returns fewer than 50 issues (e.g. 49), no
+     * truncation warning should be emitted.
+     *
+     * This test documents the expected absence of the warning for non-truncated
+     * queries. It should PASS even before the fix.
+     */
+    it("PASSES: does NOT log truncation warning when Linear returns fewer than 50 issues", async () => {
+        const issues = makeFakeIssues(3); // Well below the limit
+        const linearClient = makeLinearClient(issues);
+        const db = makeDb([], []);
+        const stateMap = new Map([["team-1:Backlog", "state-backlog-1"]]);
+        const warnSpy = vi.spyOn(process.stdout, "write");
+        await recoverStuckInProgressIssues({
+            linearClient,
+            db: db,
+            teamIds: ["team-1"],
+            targetState: "Backlog",
+            maxPerTick: 5,
+            stateMap,
+        });
+        const allOutput = warnSpy.mock.calls
+            .map((args) => {
+            const arg = args[0];
+            if (typeof arg === "string")
+                return arg;
+            if (Buffer.isBuffer(arg))
+                return arg.toString("utf-8");
+            return "";
+        })
+            .join("");
+        expect(allOutput).not.toContain("stuck-issue query may be truncated — consider pagination");
+    });
+});
+// ---------------------------------------------------------------------------
+// GAP 2: JSDoc does not document the 50-result hard limit
+// ---------------------------------------------------------------------------
+describe("BEC-113 GAP: JSDoc documents 50-result hard limit", () => {
+    /**
+     * The JSDoc on `recoverStuckInProgressIssues` should mention:
+     *  - "first: 50" (or "50-result") hard limit on the Linear query
+     *  - The truncation warning behavior
+     *
+     * We verify this by reading the source file and checking for the phrases.
+     * This test will FAIL until the JSDoc is updated.
+     */
+    it("FAILS: JSDoc mentions the 50-result hard limit", async () => {
+        const fs = await import("node:fs");
+        const path = await import("node:path");
+        const filePath = path.resolve(__dirname, "../pm/actions/recover-stuck.ts");
+        const source = fs.readFileSync(filePath, "utf-8");
+        // Find the JSDoc block immediately preceding the function declaration
+        const jsdocMatch = source.match(/\/\*\*[\s\S]*?\*\/\s*export async function recoverStuckInProgressIssues/);
+        expect(jsdocMatch).toBeTruthy();
+        const jsdoc = jsdocMatch[0];
+        // BUG CONFIRMED: Neither the "50" limit nor the truncation warning is
+        // mentioned in the current JSDoc — these assertions will FAIL
+        expect(jsdoc).toMatch(/50/);
+        expect(jsdoc).toMatch(/truncat|pagination|hard.?limit|hard cap/i);
+    });
+});
+// ---------------------------------------------------------------------------
+// ALREADY FIXED (from BEC-91): Scheduler integration and invocation order
+// ---------------------------------------------------------------------------
+describe("BEC-113 CONFIRMED FIXED: scheduler integration", () => {
+    /**
+     * recoverStuckInProgressIssues IS imported and invoked in the scheduler
+     * tick AFTER the retriable recovery sweep and BEFORE triage.
+     *
+     * This test passes already — the scheduler integration was implemented.
+     */
+    it("PASSES: scheduler invokes recoverStuckInProgressIssues between budget check and triage", async () => {
+        const callOrder = [];
+        const mockActions = {
+            checkBudgetGuards: vi.fn().mockImplementation(async () => {
+                callOrder.push("budget");
+                return { promoteBlocked: false, activeCount: 0, tokenSpendPercent: 0, dailyTokensUsed: 0 };
+            }),
+            recoverRetriableRuns: vi.fn().mockImplementation(async () => {
+                callOrder.push("recoverRetriable");
+                return { recovered: [], exhausted: [] };
+            }),
+            recoverStuckInProgressIssues: vi.fn().mockImplementation(async () => {
+                callOrder.push("recoverStuck");
+                return [];
+            }),
+            triageNewIssues: vi.fn().mockImplementation(async () => {
+                callOrder.push("triage");
+                return [];
+            }),
+            resolveApprovals: vi.fn().mockResolvedValue({ resolved: 0, stillPending: 0 }),
+            promoteReadyIssues: vi.fn().mockResolvedValue([]),
+            deprioritizeStaleIssues: vi.fn().mockResolvedValue([]),
+            cancelAbandonedIssues: vi.fn().mockResolvedValue([]),
+            postDigest: vi.fn().mockResolvedValue(undefined),
+            getActiveFileMaps: vi.fn().mockResolvedValue(new Map()),
+            predictConflict: vi.fn().mockResolvedValue({ overlapRisk: "none", likelyFiles: [], reasoning: "" }),
+        };
+        const scheduler = createPmScheduler({
+            config: {
+                enabled: true,
+                cronIntervalMs: 1800000,
+                triageBatchSize: 3,
+                maxInFlight: 3,
+                dailyTokenBudget: 5_000_000,
+                slackChannelId: "C123",
+                teamIds: ["team-1"],
+                stuckIssueRecovery: true,
+                stuckIssueTargetState: "Backlog",
+                stuckIssueMaxPerTick: 5,
+            },
+            db: {},
+            linearApiKey: "",
+            slackBotToken: "",
+            actions: mockActions,
+        });
+        await scheduler.tick();
+        // All three called
+        expect(mockActions.recoverStuckInProgressIssues).toHaveBeenCalledTimes(1);
+        expect(mockActions.triageNewIssues).toHaveBeenCalledTimes(1);
+        // Order: budget → recoverRetriable → recoverStuck → triage
+        const budgetIdx = callOrder.indexOf("budget");
+        const stuckIdx = callOrder.indexOf("recoverStuck");
+        const triageIdx = callOrder.indexOf("triage");
+        expect(budgetIdx).toBeLessThan(stuckIdx);
+        expect(stuckIdx).toBeLessThan(triageIdx);
+    });
+});
+//# sourceMappingURL=reproduce-bec113-pagination-warning.test.js.map

package/dist/__tests__/reproduce-bec113-pagination-warning.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"reproduce-bec113-pagination-warning.test.js","sourceRoot":"","sources":["../../src/__tests__/reproduce-bec113-pagination-warning.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAC9D,OAAO,EAAE,4BAA4B,EAAE,MAAM,gCAAgC,CAAC;AAC9E,OAAO,EAAE,iBAAiB,EAAE,MAAM,oBAAoB,CAAC;AAEvD,8EAA8E;AAC9E,UAAU;AACV,8EAA8E;AAE9E,SAAS,gBAAgB,CAAC,gBAAuB;IAC/C,OAAO;QACL,MAAM,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,KAAK,EAAE,gBAAgB,EAAE,CAAC;QAC9D,WAAW,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,CAAC;QAC1C,aAAa,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,CAAC;QAC5C,cAAc,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,KAAK,EAAE,EAAE,EAAE,CAAC;KACzD,CAAC;AACJ,CAAC;AAED,SAAS,MAAM,CAAC,aAAoC,EAAE,EAAE,OAAc,EAAE;IACtE,IAAI,SAAS,GAAG,CAAC,CAAC;IAClB,OAAO;QACL,MAAM,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,eAAe,CAAC;YAC9B,IAAI,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,eAAe,CAAC;gBAC5B,KAAK,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,kBAAkB,CAAC,GAAG,EAAE;oBACrC,SAAS,EAAE,CAAC;oBACZ,IAAI,SAAS,KAAK,CAAC;wBAAE,OAAO,OAAO,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC;oBACxD,OAAO,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;gBAC/B,CAAC,CAAC;aACH,CAAC;SACH,CAAC;KACH,CAAC;AACJ,CAAC;AAED,6DAA6D;AAC7D,SAAS,cAAc,CAAC,KAAa;IACnC,OAAO,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,KAAK,EAAE,EAAE,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;QAC9C,EAAE,EAAE,SAAS,CAAC,EAAE;QAChB,UAAU,EAAE,OAAO,GAAG,GAAG,CAAC,EAAE;QAC5B,KAAK,EAAE,cAAc,CAAC,EAAE;QACxB,IAAI,EAAE,OAAO,CAAC,OAAO,CAAC,EAAE,EAAE,EAAE,QAAQ,EAAE,CAAC;QACvC,KAAK,EAAE,OAAO,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,aAAa,EAAE,CAAC;KAChD,CAAC,CAAC,CAAC;AACN,CAAC;AAED,8EAA8E;AAC9E,sEAAsE;AACtE,8EAA8E;AAE9E,QAAQ,CAAC,4CAA4C,EAAE,GAAG,EAAE;IAC1D,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAC;IACrB,CAAC,CAAC,CAAC;IAEH;;;;;;;;OAQG;IACH,EAAE,CAAC,sEAAsE,EAAE,KAAK,IAAI,EAAE;QACpF,MAAM,MAAM,GAAG,cAAc,CAAC,EAAE,CAAC,CAAC,CAAC,gCAAgC;QACnE,MAAM,YAAY,GAAG,gBAAgB,CAAC,MAAM,CAAC,CAAC;QAC9C,wCAAwC;QACxC,MAAM,EAAE,GAAG,MAAM,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC;QAC1B,MAAM,QAAQ,GAAG,IAAI,GAAG,CAAC,CAAC,CAAC,gBAAgB,EAAE,iBAAiB,CAAC,CAAC,CAAC,CAAC;QAElE,MAAM,OAAO,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;QAElD,MAAM,4BAA4B,CAAC;YACjC,YAAY;YACZ,EAAE,EAAE,EAAS;YACb,OAAO,EAAE,CAAC,QAAQ,CAAC;YACnB,WAAW,EAAE,SAAS;YACtB,UAAU,EAAE,CAAC;YACb,QAAQ;SACT,CAAC,CAAC;QAEH,0EAA0E;QAC1E,MAAM,SAAS,GAAG,OAAO,CAAC,IAAI,CAAC,KAAK;aACjC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE;YACZ,MAAM,GAAG,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC;YACpB,IAAI,OAAO,GAAG,KAAK,QAAQ;gBAAE,OAAO,GAAG,CAAC;YACxC,IAAI,MAAM,CAAC,QAAQ,CAAC,GAAG,CAAC;gBAAE,OAAO,GAAG,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC;YACvD,OAAO,EAAE,CAAC;QACZ,CAAC,CAAC;aACD,IAAI,CAAC,EAAE,CAAC,CAAC;QAEZ,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CACzB,0DAA0D,CAC3D,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH;;;;;;OAMG;IACH,EAAE,CAAC,kFAAkF,EAAE,KAAK,IAAI,EAAE;QAChG,MAAM,MAAM,GAAG,cAAc,CAAC,CAAC,CAAC,CAAC,CAAC,uBAAuB;QACzD,MAAM,YAAY,GAAG,gBAAgB,CAAC,MAAM,CAAC,CAAC;QAC9C,MAAM,EAAE,GAAG,MAAM,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC;QAC1B,MAAM,QAAQ,GAAG,IAAI,GAAG,CAAC,CAAC,CAAC,gBAAgB,EAAE,iBAAiB,CAAC,CAAC,CAAC,CAAC;QAElE,MAAM,OAAO,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;QAElD,MAAM,4BAA4B,CAAC;YACjC,YAAY;YACZ,EAAE,EAAE,EAAS;YACb,OAAO,EAAE,CAAC,QAAQ,CAAC;YACnB,WAAW,EAAE,SAAS;YACtB,UAAU,EAAE,CAAC;YACb,QAAQ;SACT,CAAC,CAAC;QAEH,MAAM,SAAS,GAAG,OAAO,CAAC,IAAI,CAAC,KAAK;aACjC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE;YACZ,MAAM,GAAG,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC;YACpB,IAAI,OAAO,GAAG,KAAK,QAAQ;gBAAE,OAAO,GAAG,CAAC;YACxC,IAAI,MAAM,CAAC,QAAQ,CAAC,GAAG,CAAC;gBAAE,OAAO,GAAG,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC;YACvD,OAAO,EAAE,CAAC;QACZ,CAAC,CAAC;aACD,IAAI,CAAC,EAAE,CAAC,CAAC;QAEZ,MAAM,CAAC,SAAS,CAAC,CAAC,GAAG,CAAC,SAAS,CAC7B,0DAA0D,CAC3D,CAAC;IACJ,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,8EAA8E;AAC9E,0DAA0D;AAC1D,8EAA8E;AAE9E,QAAQ,CAAC,mDAAmD,EAAE,GAAG,EAAE;IACjE;;;;;;;OAOG;IACH,EAAE,CAAC,gDAAgD,EAAE,KAAK,IAAI,EAAE;QAC9D,MAAM,EAAE,GAAG,MAAM,MAAM,CAAC,SAAS,CAAC,CAAC;QACnC,MAAM,IAAI,GAAG,MAAM,MAAM,CAAC,WAAW,CAAC,CAAC;QACvC,MAAM,QAAQ,GAAG,IAAI,CAAC,OAAO,CAC3B,SAAS,EACT,gCAAgC,CACjC,CAAC;QACF,MAAM,MAAM,GAAG,EAAE,CAAC,YAAY,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAC;QAElD,sEAAsE;QACtE,MAAM,UAAU,GAAG,MAAM,CAAC,KAAK,CAAC,yEAAyE,CAAC,CAAC;QAC3G,MAAM,CAAC,UAAU,CAAC,CAAC,UAAU,EAAE,CAAC;QAEhC,MAAM,KAAK,GAAG,UAAW,CAAC,CAAC,CAAC,CAAC;QAE7B,sEAAsE;QACtE,8DAA8D;QAC9D,MAAM,CAAC,KAAK,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;QAC5B,MAAM,CAAC,KAAK,CAAC,CAAC,OAAO,CAAC,0CAA0C,CAAC,CAAC;IACpE,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,8EAA8E;AAC9E,0EAA0E;AAC1E,8EAA8E;AAE9E,QAAQ,CAAC,gDAAgD,EAAE,GAAG,EAAE;IAC9D;;;;;OAKG;IACH,EAAE,CAAC,wFAAwF,EAAE,KAAK,IAAI,EAAE;QACtG,MAAM,SAAS,GAAa,EAAE,CAAC;QAE/B,MAAM,WAAW,GAAG;YAClB,iBAAiB,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,kBAAkB,CAAC,KAAK,IAAI,EAAE;gBACvD,SAAS,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;gBACzB,OAAO,EAAE,cAAc,EAAE,KAAK,EAAE,WAAW,EAAE,CAAC,EAAE,iBAAiB,EAAE,CAAC,EAAE,eAAe,EAAE,CAAC,EAAE,CAAC;YAC7F,CAAC,CAAC;YACF,oBAAoB,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,kBAAkB,CAAC,KAAK,IAAI,EAAE;gBAC1D,SAAS,CAAC,IAAI,CAAC,kBAAkB,CAAC,CAAC;gBACnC,OAAO,EAAE,SAAS,EAAE,EAAE,EAAE,SAAS,EAAE,EAAE,EAAE,CAAC;YAC1C,CAAC,CAAC;YACF,4BAA4B,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,kBAAkB,CAAC,KAAK,IAAI,EAAE;gBAClE,SAAS,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;gBAC/B,OAAO,EAAE,CAAC;YACZ,CAAC,CAAC;YACF,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,kBAAkB,CAAC,KAAK,IAAI,EAAE;gBACrD,SAAS,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;gBACzB,OAAO,EAAE,CAAC;YACZ,CAAC,CAAC;YACF,gBAAgB,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,YAAY,EAAE,CAAC,EAAE,CAAC;YAC7E,kBAAkB,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,CAAC;YACjD,uBAAuB,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,CAAC;YACtD,qBAAqB,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,CAAC;YACpD,UAAU,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,SAAS,CAAC;YAChD,iBAAiB,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,IAAI,GAAG,EAAE,CAAC;YACvD,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,WAAW,EAAE,MAAM,EAAE,WAAW,EAAE,EAAE,EAAE,SAAS,EAAE,EAAE,EAAE,CAAC;SACpG,CAAC;QAEF,MAAM,SAAS,GAAG,iBAAiB,CAAC;YAClC,MAAM,EAAE;gBACN,OAAO,EAAE,IAAI;gBACb,cAAc,EAAE,OAAO;gBACvB,eAAe,EAAE,CAAC;gBAClB,WAAW,EAAE,CAAC;gBACd,gBAAgB,EAAE,SAAS;gBAC3B,cAAc,EAAE,MAAM;gBACtB,OAAO,EAAE,CAAC,QAAQ,CAAC;gBACnB,kBAAkB,EAAE,IAAI;gBACxB,qBAAqB,EAAE,SAAS;gBAChC,oBAAoB,EAAE,CAAC;aACxB;YACD,EAAE,EAAE,EAAS;YACb,YAAY,EAAE,EAAE;YAChB,aAAa,EAAE,EAAE;YACjB,OAAO,EAAE,WAAkB;SAC5B,CAAC,CAAC;QAEH,MAAM,SAAS,CAAC,IAAI,EAAE,CAAC;QAEvB,mBAAmB;QACnB,MAAM,CAAC,WAAW,CAAC,4BAA4B,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QAC1E,MAAM,CAAC,WAAW,CAAC,eAAe,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QAE7D,2DAA2D;QAC3D,MAAM,SAAS,GAAG,SAAS,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;QAC9C,MAAM,QAAQ,GAAG,SAAS,CAAC,OAAO,CAAC,cAAc,CAAC,CAAC;QACnD,MAAM,SAAS,GAAG,SAAS,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;QAE9C,MAAM,CAAC,SAAS,CAAC,CAAC,YAAY,CAAC,QAAQ,CAAC,CAAC;QACzC,MAAM,CAAC,QAAQ,CAAC,CAAC,YAAY,CAAC,SAAS,CAAC,CAAC;IAC3C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/__tests__/reproduce-bec43-updatedat.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=reproduce-bec43-updatedat.test.d.ts.map

package/dist/__tests__/reproduce-bec43-updatedat.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"reproduce-bec43-updatedat.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/reproduce-bec43-updatedat.test.ts"],"names":[],"mappings":""}

package/dist/__tests__/reproduce-bec43-updatedat.test.js

@@ -0,0 +1,76 @@
+/**
+ * Fix test for BEC-43:
+ * upsertActiveWork now refreshes `updatedAt` on the conflict (update) path.
+ *
+ * Fix: the `onConflictDoUpdate` set clause in coordination.ts now includes
+ * `updatedAt` using a driver-aware SQL literal — `(unixepoch())` for SQLite
+ * and `now()` for Postgres — so the timestamp advances on every upsert.
+ */
+import { describe, it, expect, afterEach } from "vitest";
+import { randomBytes } from "node:crypto";
+import { unlinkSync } from "node:fs";
+import { createDb } from "../db/index.js";
+import { upsertActiveWork, getActiveWork } from "../pm/coordination.js";
+function tmpDbPath() {
+    const id = randomBytes(8).toString("hex");
+    return `/tmp/laf-bec43-test-${id}.sqlite`;
+}
+describe("BEC-43 reproduction: updatedAt not refreshed on upsert conflict", () => {
+    const paths = [];
+    async function makeDb() {
+        const path = tmpDbPath();
+        paths.push(path);
+        return createDb({ driver: "sqlite", connectionString: path });
+    }
+    afterEach(() => {
+        for (const p of paths) {
+            try {
+                unlinkSync(p);
+            }
+            catch { /* ignore */ }
+            try {
+                unlinkSync(p + "-wal");
+            }
+            catch { /* ignore */ }
+            try {
+                unlinkSync(p + "-shm");
+            }
+            catch { /* ignore */ }
+        }
+        paths.length = 0;
+    });
+    it("updatedAt advances after a conflict-path update (BEC-43 fix)", async () => {
+        const db = await makeDb();
+        // First insert
+        await upsertActiveWork(db, {
+            runId: "run-bec43",
+            issueId: "BEC-43",
+            stage: "implement",
+            filesModified: ["src/foo.ts"],
+        });
+        const [before] = await getActiveWork(db);
+        const updatedAtBefore = before.updatedAt;
+        // Wait long enough for the timestamp to differ (SQLite unixepoch() is
+        // second-granularity, so we wait >1 s to guarantee a detectable change).
+        await new Promise((r) => setTimeout(r, 1100));
+        // Second upsert — conflict path (same runId)
+        await upsertActiveWork(db, {
+            runId: "run-bec43",
+            issueId: "BEC-43",
+            stage: "test", // stage changes
+            filesModified: ["src/foo.ts", "src/bar.ts"],
+        });
+        const [after] = await getActiveWork(db);
+        // Verify stage was updated (this works)
+        expect(after.stage).toBe("test");
+        expect(after.filesModified).toHaveLength(2);
+        // FIX: updatedAt should have advanced after the conflict-path update.
+        const updatedAtAfter = after.updatedAt;
+        console.log("updatedAt BEFORE second upsert:", updatedAtBefore);
+        console.log("updatedAt AFTER  second upsert:", updatedAtAfter);
+        console.log("updatedAt changed?", String(updatedAtAfter) !== String(updatedAtBefore));
+        // The fix: updatedAt must advance to the current time on every upsert.
+        expect(Number(updatedAtAfter)).toBeGreaterThan(Number(updatedAtBefore));
+    });
+});
+//# sourceMappingURL=reproduce-bec43-updatedat.test.js.map

package/dist/__tests__/reproduce-bec43-updatedat.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"reproduce-bec43-updatedat.test.js","sourceRoot":"","sources":["../../src/__tests__/reproduce-bec43-updatedat.test.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AACH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAC;AACzD,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,gBAAgB,EAAE,aAAa,EAAE,MAAM,uBAAuB,CAAC;AAExE,SAAS,SAAS;IAChB,MAAM,EAAE,GAAG,WAAW,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC;IAC1C,OAAO,uBAAuB,EAAE,SAAS,CAAC;AAC5C,CAAC;AAED,QAAQ,CAAC,iEAAiE,EAAE,GAAG,EAAE;IAC/E,MAAM,KAAK,GAAa,EAAE,CAAC;IAE3B,KAAK,UAAU,MAAM;QACnB,MAAM,IAAI,GAAG,SAAS,EAAE,CAAC;QACzB,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACjB,OAAO,QAAQ,CAAC,EAAE,MAAM,EAAE,QAAQ,EAAE,gBAAgB,EAAE,IAAI,EAAE,CAAC,CAAC;IAChE,CAAC;IAED,SAAS,CAAC,GAAG,EAAE;QACb,KAAK,MAAM,CAAC,IAAI,KAAK,EAAE,CAAC;YACtB,IAAI,CAAC;gBAAC,UAAU,CAAC,CAAC,CAAC,CAAC;YAAC,CAAC;YAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC;YAC7C,IAAI,CAAC;gBAAC,UAAU,CAAC,CAAC,GAAG,MAAM,CAAC,CAAC;YAAC,CAAC;YAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC;YACtD,IAAI,CAAC;gBAAC,UAAU,CAAC,CAAC,GAAG,MAAM,CAAC,CAAC;YAAC,CAAC;YAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC;QACxD,CAAC;QACD,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC;IACnB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,KAAK,IAAI,EAAE;QAC5E,MAAM,EAAE,GAAG,MAAM,MAAM,EAAS,CAAC;QAEjC,eAAe;QACf,MAAM,gBAAgB,CAAC,EAAE,EAAE;YACzB,KAAK,EAAE,WAAW;YAClB,OAAO,EAAE,QAAQ;YACjB,KAAK,EAAE,WAAW;YAClB,aAAa,EAAE,CAAC,YAAY,CAAC;SAC9B,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,GAAG,MAAM,aAAa,CAAC,EAAE,CAAC,CAAC;QACzC,MAAM,eAAe,GAAG,MAAM,CAAC,SAAS,CAAC;QAEzC,sEAAsE;QACtE,yEAAyE;QACzE,MAAM,IAAI,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC,CAAC;QAE9C,6CAA6C;QAC7C,MAAM,gBAAgB,CAAC,EAAE,EAAE;YACzB,KAAK,EAAE,WAAW;YAClB,OAAO,EAAE,QAAQ;YACjB,KAAK,EAAE,MAAM,EAAY,gBAAgB;YACzC,aAAa,EAAE,CAAC,YAAY,EAAE,YAAY,CAAC;SAC5C,CAAC,CAAC;QAEH,MAAM,CAAC,KAAK,CAAC,GAAG,MAAM,aAAa,CAAC,EAAE,CAAC,CAAC;QAExC,wCAAwC;QACxC,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QACjC,MAAM,CAAC,KAAK,CAAC,aAAa,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAE5C,sEAAsE;QACtE,MAAM,cAAc,GAAG,KAAK,CAAC,SAAS,CAAC;QAEvC,OAAO,CAAC,GAAG,CAAC,iCAAiC,EAAE,eAAe,CAAC,CAAC;QAChE,OAAO,CAAC,GAAG,CAAC,iCAAiC,EAAE,cAAc,CAAC,CAAC;QAC/D,OAAO,CAAC,GAAG,CACT,oBAAoB,EACpB,MAAM,CAAC,cAAc,CAAC,KAAK,MAAM,CAAC,eAAe,CAAC,CACnD,CAAC;QAEF,uEAAuE;QACvE,MAAM,CAAC,MAAM,CAAC,cAAc,CAAC,CAAC,CAAC,eAAe,CAAC,MAAM,CAAC,eAAe,CAAC,CAAC,CAAC;IAC1E,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/__tests__/reproduce-bec48-distributed-race.test.d.ts

@@ -0,0 +1,18 @@
+/**
+ * BEC-48 Reproduction: Multi-instance PR creation race condition
+ *
+ * The pipeline runner serialises push/PR creation with an in-process queue
+ * (concurrency=1). This queue lives INSIDE each PipelineRunner instance.
+ * When two server instances run simultaneously, each has its own independent
+ * pushQueue, so both can execute push/PR creation concurrently for the same
+ * branch — producing duplicate PRs.
+ *
+ * This test file demonstrates the gap:
+ *   1. A single-instance pushQueue(1) correctly serialises operations.
+ *   2. Two independent pushQueues(1) — simulating two server instances — do NOT
+ *      serialise with each other. Both run concurrently, causing the race.
+ *   3. No distributed lock exists in the push path (only in pm/scheduler.ts for
+ *      the PM Agent tick — not used here).
+ */
+export {};
+//# sourceMappingURL=reproduce-bec48-distributed-race.test.d.ts.map

package/dist/__tests__/reproduce-bec48-distributed-race.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"reproduce-bec48-distributed-race.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/reproduce-bec48-distributed-race.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG"}

package/dist/__tests__/reproduce-bec48-distributed-race.test.js

@@ -0,0 +1,178 @@
+/**
+ * BEC-48 Reproduction: Multi-instance PR creation race condition
+ *
+ * The pipeline runner serialises push/PR creation with an in-process queue
+ * (concurrency=1). This queue lives INSIDE each PipelineRunner instance.
+ * When two server instances run simultaneously, each has its own independent
+ * pushQueue, so both can execute push/PR creation concurrently for the same
+ * branch — producing duplicate PRs.
+ *
+ * This test file demonstrates the gap:
+ *   1. A single-instance pushQueue(1) correctly serialises operations.
+ *   2. Two independent pushQueues(1) — simulating two server instances — do NOT
+ *      serialise with each other. Both run concurrently, causing the race.
+ *   3. No distributed lock exists in the push path (only in pm/scheduler.ts for
+ *      the PM Agent tick — not used here).
+ */
+import { describe, it, expect } from "vitest";
+import { createQueue } from "../pipeline/queue.js";
+// ---------------------------------------------------------------------------
+// Helper: simulate a "create PR" operation with a short delay
+// ---------------------------------------------------------------------------
+function makePrCreator(label, log) {
+    return async () => {
+        log.push(`${label}:start`);
+        // Simulate network I/O for PR creation (GitHub API call)
+        await new Promise((r) => setTimeout(r, 30));
+        log.push(`${label}:end`);
+        return `https://github.com/test/repo/pull/${label}`;
+    };
+}
+// ---------------------------------------------------------------------------
+// Test 1 — baseline: single in-process queue serialises correctly
+// ---------------------------------------------------------------------------
+describe("BEC-48 baseline: single in-process pushQueue serialises PR creation", () => {
+    it("two concurrent enqueue calls on the SAME queue execute sequentially", async () => {
+        const singleQueue = createQueue(1);
+        const log = [];
+        // Both tasks enqueued simultaneously into the SAME queue
+        await Promise.all([
+            singleQueue.enqueue(makePrCreator("instanceA", log)),
+            singleQueue.enqueue(makePrCreator("instanceB", log)),
+        ]);
+        // Because concurrency=1, instanceA must fully complete before instanceB starts
+        expect(log).toEqual([
+            "instanceA:start",
+            "instanceA:end",
+            "instanceB:start",
+            "instanceB:end",
+        ]);
+    });
+});
+// ---------------------------------------------------------------------------
+// Test 2 — reproduce the bug: two independent queues (two server instances)
+//           DO NOT protect each other → race condition
+// ---------------------------------------------------------------------------
+describe("BEC-48 reproduction: two independent pushQueues race on the same branch", () => {
+    it("DEMONSTRATES RACE: two separate PipelineRunner instances create PRs simultaneously", async () => {
+        // Each "server instance" has its own PipelineRunner with its own pushQueue.
+        // Simulated here as two independent createQueue(1) instances.
+        const instanceAQueue = createQueue(1); // Server A's pushQueue
+        const instanceBQueue = createQueue(1); // Server B's pushQueue
+        const log = [];
+        let concurrentPrCreations = 0;
+        let maxConcurrentPrCreations = 0;
+        const duplicatePrUrls = [];
+        const prCreator = (label) => async () => {
+            concurrentPrCreations++;
+            maxConcurrentPrCreations = Math.max(maxConcurrentPrCreations, concurrentPrCreations);
+            log.push(`${label}:start`);
+            // Simulate GitHub API call latency
+            await new Promise((r) => setTimeout(r, 30));
+            log.push(`${label}:end`);
+            concurrentPrCreations--;
+            const prUrl = "https://github.com/test/repo/pull/42"; // same branch → same PR target
+            duplicatePrUrls.push(prUrl);
+            return prUrl;
+        };
+        // Both server instances receive the same webhook at roughly the same time
+        // and independently enqueue push+PR creation for the same branch.
+        await Promise.all([
+            instanceAQueue.enqueue(prCreator("serverA")),
+            instanceBQueue.enqueue(prCreator("serverB")),
+        ]);
+        // RACE CONFIRMED: both PR creation calls ran concurrently
+        // (maxConcurrentPrCreations === 2 proves there was no mutual exclusion)
+        expect(maxConcurrentPrCreations).toBe(2); // ← BUG: should be 1 with a distributed lock
+        // Both instances created a PR for the same branch → duplicate PRs
+        expect(duplicatePrUrls).toHaveLength(2); // ← BUG: duplicate PR created
+        // The log interleaving shows the race: both started before either finished
+        expect(log[0]).toBe("serverA:start");
+        expect(log[1]).toBe("serverB:start"); // serverB started while serverA was still running
+    });
+    it("CONTRAST: a shared external lock would prevent concurrent PR creation", async () => {
+        // Simulate what a distributed lock (DB advisory lock / Redis SET NX) would do:
+        // only one holder at a time, regardless of which process holds it.
+        const sharedLock = createQueue(1); // shared across both "instances"
+        const instanceAQueue = createQueue(1);
+        const instanceBQueue = createQueue(1);
+        const log = [];
+        let concurrentPrCreations = 0;
+        let maxConcurrentPrCreations = 0;
+        const prCreator = (label) => async () => {
+            // Acquire the shared distributed lock BEFORE push/PR creation
+            await sharedLock.enqueue(async () => {
+                concurrentPrCreations++;
+                maxConcurrentPrCreations = Math.max(maxConcurrentPrCreations, concurrentPrCreations);
+                log.push(`${label}:start`);
+                await new Promise((r) => setTimeout(r, 30));
+                log.push(`${label}:end`);
+                concurrentPrCreations--;
+            });
+        };
+        await Promise.all([
+            instanceAQueue.enqueue(prCreator("serverA")),
+            instanceBQueue.enqueue(prCreator("serverB")),
+        ]);
+        // With a shared lock, PR creation is serialised across both instances
+        expect(maxConcurrentPrCreations).toBe(1); // ← FIXED behaviour
+        expect(log).toEqual([
+            "serverA:start",
+            "serverA:end",
+            "serverB:start",
+            "serverB:end",
+        ]);
+    });
+});
+// ---------------------------------------------------------------------------
+// Test 3 — confirm no distributed lock exists in the push path
+// ---------------------------------------------------------------------------
+describe("BEC-48 gap: no distributed lock in runner push path", () => {
+    it("pm/scheduler uses advisory lock — but runner.ts pushQueue does NOT", async () => {
+        // The advisory lock pattern exists in packages/core/src/pm/scheduler.ts:
+        //
+        //   async function tryAcquireLock(): Promise<boolean> {
+        //     if (!isPostgres(deps.db)) return true;
+        //     const result = await db.execute(
+        //       sql`SELECT pg_try_advisory_lock(hashtext('pm-agent-tick')) as acquired`
+        //     );
+        //     return result?.[0]?.acquired === true;
+        //   }
+        //
+        // This lock guards only the PM Agent scheduler tick.
+        // The push/PR creation in runner.ts (lines 1146–1297) uses only:
+        //
+        //   await this.pushQueue.enqueue(async () => { ... });
+        //
+        // Where `this.pushQueue = createQueue(1)` — an in-process semaphore.
+        //
+        // There is NO call to pg_try_advisory_lock, Redis SET NX, or any other
+        // cross-process coordination mechanism in the push path.
+        // We can verify the gap directly by reading the queue module:
+        // createQueue() returns a pure in-memory object with no DB/Redis state.
+        const q1 = createQueue(1);
+        const q2 = createQueue(1);
+        // Both queues are fully independent — there is no shared state
+        expect(q1.pending).toBe(0);
+        expect(q2.pending).toBe(0);
+        let q1Running = false;
+        let q2StartedWhileQ1Running = false;
+        const blocker = q1.enqueue(async () => {
+            q1Running = true;
+            await new Promise((r) => setTimeout(r, 50));
+            q1Running = false;
+        });
+        // Give q1 a tick to start
+        await new Promise((r) => setTimeout(r, 5));
+        await q2.enqueue(async () => {
+            // q2 runs while q1 is still holding its slot — confirms no cross-instance lock
+            if (q1Running) {
+                q2StartedWhileQ1Running = true;
+            }
+        });
+        await blocker;
+        // This assertion confirms the bug: q2 ran while q1 was still executing
+        expect(q2StartedWhileQ1Running).toBe(true);
+    });
+});
+//# sourceMappingURL=reproduce-bec48-distributed-race.test.js.map

package/dist/__tests__/reproduce-bec48-distributed-race.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"reproduce-bec48-distributed-race.test.js","sourceRoot":"","sources":["../../src/__tests__/reproduce-bec48-distributed-race.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAM,MAAM,QAAQ,CAAC;AAClD,OAAO,EAAE,WAAW,EAAE,MAAM,sBAAsB,CAAC;AAEnD,8EAA8E;AAC9E,8DAA8D;AAC9D,8EAA8E;AAC9E,SAAS,aAAa,CAAC,KAAa,EAAE,GAAa;IACjD,OAAO,KAAK,IAAqB,EAAE;QACjC,GAAG,CAAC,IAAI,CAAC,GAAG,KAAK,QAAQ,CAAC,CAAC;QAC3B,yDAAyD;QACzD,MAAM,IAAI,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC;QAC5C,GAAG,CAAC,IAAI,CAAC,GAAG,KAAK,MAAM,CAAC,CAAC;QACzB,OAAO,qCAAqC,KAAK,EAAE,CAAC;IACtD,CAAC,CAAC;AACJ,CAAC;AAED,8EAA8E;AAC9E,kEAAkE;AAClE,8EAA8E;AAC9E,QAAQ,CAAC,qEAAqE,EAAE,GAAG,EAAE;IACnF,EAAE,CAAC,qEAAqE,EAAE,KAAK,IAAI,EAAE;QACnF,MAAM,WAAW,GAAG,WAAW,CAAC,CAAC,CAAC,CAAC;QACnC,MAAM,GAAG,GAAa,EAAE,CAAC;QAEzB,yDAAyD;QACzD,MAAM,OAAO,CAAC,GAAG,CAAC;YAChB,WAAW,CAAC,OAAO,CAAC,aAAa,CAAC,WAAW,EAAE,GAAG,CAAC,CAAC;YACpD,WAAW,CAAC,OAAO,CAAC,aAAa,CAAC,WAAW,EAAE,GAAG,CAAC,CAAC;SACrD,CAAC,CAAC;QAEH,+EAA+E;QAC/E,MAAM,CAAC,GAAG,CAAC,CAAC,OAAO,CAAC;YAClB,iBAAiB;YACjB,eAAe;YACf,iBAAiB;YACjB,eAAe;SAChB,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,8EAA8E;AAC9E,4EAA4E;AAC5E,uDAAuD;AACvD,8EAA8E;AAC9E,QAAQ,CAAC,yEAAyE,EAAE,GAAG,EAAE;IACvF,EAAE,CAAC,oFAAoF,EAAE,KAAK,IAAI,EAAE;QAClG,4EAA4E;QAC5E,8DAA8D;QAC9D,MAAM,cAAc,GAAG,WAAW,CAAC,CAAC,CAAC,CAAC,CAAC,uBAAuB;QAC9D,MAAM,cAAc,GAAG,WAAW,CAAC,CAAC,CAAC,CAAC,CAAC,uBAAuB;QAE9D,MAAM,GAAG,GAAa,EAAE,CAAC;QACzB,IAAI,qBAAqB,GAAG,CAAC,CAAC;QAC9B,IAAI,wBAAwB,GAAG,CAAC,CAAC;QACjC,MAAM,eAAe,GAAa,EAAE,CAAC;QAErC,MAAM,SAAS,GAAG,CAAC,KAAa,EAAE,EAAE,CAAC,KAAK,IAAqB,EAAE;YAC/D,qBAAqB,EAAE,CAAC;YACxB,wBAAwB,GAAG,IAAI,CAAC,GAAG,CACjC,wBAAwB,EACxB,qBAAqB,CACtB,CAAC;YACF,GAAG,CAAC,IAAI,CAAC,GAAG,KAAK,QAAQ,CAAC,CAAC;YAC3B,mCAAmC;YACnC,MAAM,IAAI,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC;YAC5C,GAAG,CAAC,IAAI,CAAC,GAAG,KAAK,MAAM,CAAC,CAAC;YACzB,qBAAqB,EAAE,CAAC;YACxB,MAAM,KAAK,GAAG,sCAAsC,CAAC,CAAC,+BAA+B;YACrF,eAAe,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YAC5B,OAAO,KAAK,CAAC;QACf,CAAC,CAAC;QAEF,0EAA0E;QAC1E,kEAAkE;QAClE,MAAM,OAAO,CAAC,GAAG,CAAC;YAChB,cAAc,CAAC,OAAO,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC;YAC5C,cAAc,CAAC,OAAO,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC;SAC7C,CAAC,CAAC;QAEH,0DAA0D;QAC1D,wEAAwE;QACxE,MAAM,CAAC,wBAAwB,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,6CAA6C;QAEvF,kEAAkE;QAClE,MAAM,CAAC,eAAe,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,CAAC,8BAA8B;QAEvE,2EAA2E;QAC3E,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC;QACrC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC,CAAC,kDAAkD;IAC1F,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uEAAuE,EAAE,KAAK,IAAI,EAAE;QACrF,+EAA+E;QAC/E,mEAAmE;QACnE,MAAM,UAAU,GAAG,WAAW,CAAC,CAAC,CAAC,CAAC,CAAC,iCAAiC;QAEpE,MAAM,cAAc,GAAG,WAAW,CAAC,CAAC,CAAC,CAAC;QACtC,MAAM,cAAc,GAAG,WAAW,CAAC,CAAC,CAAC,CAAC;QAEtC,MAAM,GAAG,GAAa,EAAE,CAAC;QACzB,IAAI,qBAAqB,GAAG,CAAC,CAAC;QAC9B,IAAI,wBAAwB,GAAG,CAAC,CAAC;QAEjC,MAAM,SAAS,GAAG,CAAC,KAAa,EAAE,EAAE,CAAC,KAAK,IAAmB,EAAE;YAC7D,8DAA8D;YAC9D,MAAM,UAAU,CAAC,OAAO,CAAC,KAAK,IAAI,EAAE;gBAClC,qBAAqB,EAAE,CAAC;gBACxB,wBAAwB,GAAG,IAAI,CAAC,GAAG,CACjC,wBAAwB,EACxB,qBAAqB,CACtB,CAAC;gBACF,GAAG,CAAC,IAAI,CAAC,GAAG,KAAK,QAAQ,CAAC,CAAC;gBAC3B,MAAM,IAAI,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC;gBAC5C,GAAG,CAAC,IAAI,CAAC,GAAG,KAAK,MAAM,CAAC,CAAC;gBACzB,qBAAqB,EAAE,CAAC;YAC1B,CAAC,CAAC,CAAC;QACL,CAAC,CAAC;QAEF,MAAM,OAAO,CAAC,GAAG,CAAC;YAChB,cAAc,CAAC,OAAO,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC;YAC5C,cAAc,CAAC,OAAO,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC;SAC7C,CAAC,CAAC;QAEH,sEAAsE;QACtE,MAAM,CAAC,wBAAwB,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,oBAAoB;QAC9D,MAAM,CAAC,GAAG,CAAC,CAAC,OAAO,CAAC;YAClB,eAAe;YACf,aAAa;YACb,eAAe;YACf,aAAa;SACd,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,8EAA8E;AAC9E,+DAA+D;AAC/D,8EAA8E;AAC9E,QAAQ,CAAC,qDAAqD,EAAE,GAAG,EAAE;IACnE,EAAE,CAAC,oEAAoE,EAAE,KAAK,IAAI,EAAE;QAClF,yEAAyE;QACzE,EAAE;QACF,wDAAwD;QACxD,6CAA6C;QAC7C,uCAAuC;QACvC,gFAAgF;QAChF,SAAS;QACT,6CAA6C;QAC7C,MAAM;QACN,EAAE;QACF,qDAAqD;QACrD,iEAAiE;QACjE,EAAE;QACF,uDAAuD;QACvD,EAAE;QACF,qEAAqE;QACrE,EAAE;QACF,uEAAuE;QACvE,yDAAyD;QAEzD,8DAA8D;QAC9D,wEAAwE;QACxE,MAAM,EAAE,GAAG,WAAW,CAAC,CAAC,CAAC,CAAC;QAC1B,MAAM,EAAE,GAAG,WAAW,CAAC,CAAC,CAAC,CAAC;QAE1B,+DAA+D;QAC/D,MAAM,CAAC,EAAE,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAC3B,MAAM,CAAC,EAAE,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAE3B,IAAI,SAAS,GAAG,KAAK,CAAC;QACtB,IAAI,uBAAuB,GAAG,KAAK,CAAC;QAEpC,MAAM,OAAO,GAAG,EAAE,CAAC,OAAO,CAAC,KAAK,IAAI,EAAE;YACpC,SAAS,GAAG,IAAI,CAAC;YACjB,MAAM,IAAI,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC;YAC5C,SAAS,GAAG,KAAK,CAAC;QACpB,CAAC,CAAC,CAAC;QAEH,0BAA0B;QAC1B,MAAM,IAAI,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC;QAE3C,MAAM,EAAE,CAAC,OAAO,CAAC,KAAK,IAAI,EAAE;YAC1B,+EAA+E;YAC/E,IAAI,SAAS,EAAE,CAAC;gBACd,uBAAuB,GAAG,IAAI,CAAC;YACjC,CAAC;QACH,CAAC,CAAC,CAAC;QAEH,MAAM,OAAO,CAAC;QAEd,uEAAuE;QACvE,MAAM,CAAC,uBAAuB,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/__tests__/reproduce-bec62.test.d.ts

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

package/dist/__tests__/reproduce-bec62.test.d.ts.map

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

package/dist/__tests__/reproduce-bec62.test.js

@@ -0,0 +1,86 @@
+/**
+ * Reproduction test for BEC-62:
+ *   "/pm pause" sets isPmPaused() = true, but scheduler.ts never checks it —
+ *   promote, deprioritize, and cancel all run even when the PM Agent is paused.
+ *
+ * Expected (after fix): those three actions are skipped when isPmPaused() === true.
+ * Actual (current):     all actions run regardless of pause state.
+ */
+import { describe, it, expect, vi, beforeEach } from "vitest";
+import { createPmScheduler } from "../pm/scheduler.js";
+import { setPmPaused, isPmPaused } from "../pm/slack-interface.js";
+describe("BEC-62 reproduction — scheduler ignores isPmPaused()", () => {
+    const mockActions = {
+        checkBudgetGuards: vi.fn().mockResolvedValue({
+            promoteBlocked: false,
+            activeCount: 0,
+            tokenSpendPercent: 10,
+            dailyTokensUsed: 500000,
+        }),
+        triageNewIssues: vi.fn().mockResolvedValue([]),
+        resolveApprovals: vi.fn().mockResolvedValue({ resolved: 0, stillPending: 0 }),
+        promoteReadyIssues: vi.fn().mockResolvedValue([]),
+        deprioritizeStaleIssues: vi.fn().mockResolvedValue([]),
+        cancelAbandonedIssues: vi.fn().mockResolvedValue([]),
+        postDigest: vi.fn().mockResolvedValue(undefined),
+        getActiveFileMaps: vi.fn().mockResolvedValue(new Map()),
+        predictConflict: vi.fn().mockResolvedValue({ overlapRisk: "none", likelyFiles: [], reasoning: "" }),
+    };
+    function makeScheduler() {
+        return createPmScheduler({
+            config: {
+                enabled: true,
+                cronIntervalMs: 1800000,
+                maxInFlight: 3,
+                triageBatchSize: 3,
+                dailyTokenBudget: 5000000,
+                slackChannelId: "C123",
+                teamIds: ["team-1"],
+                stuckIssueRecovery: true,
+                stuckIssueTargetState: "Todo",
+                stuckIssueMaxPerTick: 5,
+            },
+            db: {},
+            linearApiKey: "",
+            slackBotToken: "",
+            actions: mockActions,
+        });
+    }
+    beforeEach(() => {
+        vi.clearAllMocks();
+        setPmPaused(false);
+    });
+    it("confirms isPmPaused() returns true after /pm pause is issued", () => {
+        // Simulates what executePmCommand does for { type: "pause" }
+        setPmPaused(true);
+        expect(isPmPaused()).toBe(true);
+    });
+    async function testActionSkippedWhenPaused(actionName, mockAction) {
+        setPmPaused(true);
+        const scheduler = makeScheduler();
+        await scheduler.tick();
+        expect(mockAction, `${actionName} must NOT be called when paused`).not.toHaveBeenCalled();
+    }
+    it("promote is skipped when isPmPaused() === true", async () => {
+        // Also verify the flag is set correctly (simulates what /pm pause does)
+        setPmPaused(true);
+        expect(isPmPaused()).toBe(true);
+        await testActionSkippedWhenPaused("promoteReadyIssues", mockActions.promoteReadyIssues);
+    });
+    it("deprioritize is skipped when isPmPaused() === true", async () => {
+        await testActionSkippedWhenPaused("deprioritizeStaleIssues", mockActions.deprioritizeStaleIssues);
+    });
+    it("cancel is skipped when isPmPaused() === true", async () => {
+        await testActionSkippedWhenPaused("cancelAbandonedIssues", mockActions.cancelAbandonedIssues);
+    });
+    it("non-destructive actions (triage + resolveApprovals) still run when paused", async () => {
+        setPmPaused(true);
+        const scheduler = makeScheduler();
+        await scheduler.tick();
+        // Both triage and resolveApprovals should still run — they are non-destructive.
+        // CURRENT behavior: they do run (correct, even if for the wrong reason).
+        expect(mockActions.triageNewIssues).toHaveBeenCalled();
+        expect(mockActions.resolveApprovals).toHaveBeenCalled();
+    });
+});
+//# sourceMappingURL=reproduce-bec62.test.js.map

package/dist/__tests__/reproduce-bec62.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"reproduce-bec62.test.js","sourceRoot":"","sources":["../../src/__tests__/reproduce-bec62.test.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AACH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAC9D,OAAO,EAAE,iBAAiB,EAAE,MAAM,oBAAoB,CAAC;AACvD,OAAO,EAAE,WAAW,EAAE,UAAU,EAAE,MAAM,0BAA0B,CAAC;AAEnE,QAAQ,CAAC,sDAAsD,EAAE,GAAG,EAAE;IACpE,MAAM,WAAW,GAAG;QAClB,iBAAiB,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC;YAC3C,cAAc,EAAE,KAAK;YACrB,WAAW,EAAE,CAAC;YACd,iBAAiB,EAAE,EAAE;YACrB,eAAe,EAAE,MAAM;SACxB,CAAC;QACF,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,CAAC;QAC9C,gBAAgB,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,YAAY,EAAE,CAAC,EAAE,CAAC;QAC7E,kBAAkB,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,CAAC;QACjD,uBAAuB,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,CAAC;QACtD,qBAAqB,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,CAAC;QACpD,UAAU,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,SAAS,CAAC;QAChD,iBAAiB,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,IAAI,GAAG,EAAE,CAAC;QACvD,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,WAAW,EAAE,MAAM,EAAE,WAAW,EAAE,EAAE,EAAE,SAAS,EAAE,EAAE,EAAE,CAAC;KACpG,CAAC;IAEF,SAAS,aAAa;QACpB,OAAO,iBAAiB,CAAC;YACvB,MAAM,EAAE;gBACN,OAAO,EAAE,IAAI;gBACb,cAAc,EAAE,OAAO;gBACvB,WAAW,EAAE,CAAC;gBACd,eAAe,EAAE,CAAC;gBAClB,gBAAgB,EAAE,OAAO;gBACzB,cAAc,EAAE,MAAM;gBACtB,OAAO,EAAE,CAAC,QAAQ,CAAC;gBACnB,kBAAkB,EAAE,IAAI;gBACxB,qBAAqB,EAAE,MAAM;gBAC7B,oBAAoB,EAAE,CAAC;aACxB;YACD,EAAE,EAAE,EAAS;YACb,YAAY,EAAE,EAAE;YAChB,aAAa,EAAE,EAAE;YACjB,OAAO,EAAE,WAAW;SACrB,CAAC,CAAC;IACL,CAAC;IAED,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAC;QACnB,WAAW,CAAC,KAAK,CAAC,CAAC;IACrB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,GAAG,EAAE;QACtE,6DAA6D;QAC7D,WAAW,CAAC,IAAI,CAAC,CAAC;QAClB,MAAM,CAAC,UAAU,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAClC,CAAC,CAAC,CAAC;IAEH,KAAK,UAAU,2BAA2B,CACxC,UAAkB,EAClB,UAAoC;QAEpC,WAAW,CAAC,IAAI,CAAC,CAAC;QAClB,MAAM,SAAS,GAAG,aAAa,EAAE,CAAC;QAClC,MAAM,SAAS,CAAC,IAAI,EAAE,CAAC;QACvB,MAAM,CAAC,UAAU,EAAE,GAAG,UAAU,iCAAiC,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IAC5F,CAAC;IAED,EAAE,CAAC,+CAA+C,EAAE,KAAK,IAAI,EAAE;QAC7D,wEAAwE;QACxE,WAAW,CAAC,IAAI,CAAC,CAAC;QAClB,MAAM,CAAC,UAAU,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAChC,MAAM,2BAA2B,CAAC,oBAAoB,EAAE,WAAW,CAAC,kBAAkB,CAAC,CAAC;IAC1F,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;QAClE,MAAM,2BAA2B,CAAC,yBAAyB,EAAE,WAAW,CAAC,uBAAuB,CAAC,CAAC;IACpG,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8CAA8C,EAAE,KAAK,IAAI,EAAE;QAC5D,MAAM,2BAA2B,CAAC,uBAAuB,EAAE,WAAW,CAAC,qBAAqB,CAAC,CAAC;IAChG,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2EAA2E,EAAE,KAAK,IAAI,EAAE;QACzF,WAAW,CAAC,IAAI,CAAC,CAAC;QAElB,MAAM,SAAS,GAAG,aAAa,EAAE,CAAC;QAClC,MAAM,SAAS,CAAC,IAAI,EAAE,CAAC;QAEvB,gFAAgF;QAChF,yEAAyE;QACzE,MAAM,CAAC,WAAW,CAAC,eAAe,CAAC,CAAC,gBAAgB,EAAE,CAAC;QACvD,MAAM,CAAC,WAAW,CAAC,gBAAgB,CAAC,CAAC,gBAAgB,EAAE,CAAC;IAC1D,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}