factory-factory 0.4.1 → 0.4.2

package/dist/src/backend/services/github/service/{pr-fetch-registry.js → pr-fetch-coordinator.js}

@@ -1,99 +1,110 @@
 import { SERVICE_CACHE_TTL_MS, SERVICE_LIMITS } from '../../../../backend/services/constants.js';
 const DEFAULT_COOLDOWN_MS = 90_000;
-export class PRFetchRegistry {
+export class PRFetchCoordinator {
     lastFetchedAt = new Map();
     inFlightClaims = new Map();
     nextClaimToken = 0;
-    pruneExpiredInFlight(now) {
-        for (const [workspaceId, claim] of this.inFlightClaims) {
-            if (now - claim.startedAt >= SERVICE_CACHE_TTL_MS.workspacePrFetchInFlight) {
-                this.inFlightClaims.delete(workspaceId);
-            }
+    async coordinate(workspaceId, fetch, options) {
+        const now = Date.now();
+        this.pruneExpiredInFlight(now);
+        if (this.inFlightClaims.has(workspaceId)) {
+            return { status: 'skipped', reason: 'recently_fetched' };
         }
-    }
-    ensureCapacityFor(workspaceId) {
-        if (this.lastFetchedAt.has(workspaceId) || this.inFlightClaims.has(workspaceId)) {
-            return;
+        if (!options?.ignoreCooldown && this.isWithinCooldown(workspaceId, now, options?.cooldownMs)) {
+            return { status: 'skipped', reason: 'recently_fetched' };
         }
-        const workspaceIds = new Set([...this.lastFetchedAt.keys(), ...this.inFlightClaims.keys()]);
-        if (workspaceIds.size < SERVICE_LIMITS.workspaceScopedCacheMaxEntries) {
-            return;
-        }
-        let oldestWorkspaceId;
-        let oldestTimestamp = Number.POSITIVE_INFINITY;
-        for (const [candidateWorkspaceId, timestamp] of this.lastFetchedAt) {
-            if (timestamp < oldestTimestamp) {
-                oldestWorkspaceId = candidateWorkspaceId;
-                oldestTimestamp = timestamp;
+        const claimToken = this.claim(workspaceId, now);
+        try {
+            const value = await fetch();
+            if (options?.countsAsFetched?.(value) ?? true) {
+                this.recordCompleted(workspaceId, claimToken);
             }
-        }
-        for (const [candidateWorkspaceId, claim] of this.inFlightClaims) {
-            if (claim.startedAt < oldestTimestamp) {
-                oldestWorkspaceId = candidateWorkspaceId;
-                oldestTimestamp = claim.startedAt;
+            else {
+                this.release(workspaceId, claimToken);
             }
+            return { status: 'fetched', value };
         }
-        if (oldestWorkspaceId !== undefined) {
-            this.lastFetchedAt.delete(oldestWorkspaceId);
-            this.inFlightClaims.delete(oldestWorkspaceId);
+        catch (error) {
+            this.release(workspaceId, claimToken);
+            throw error;
         }
     }
-    startFetch(workspaceId) {
-        const now = Date.now();
-        this.pruneExpiredInFlight(now);
+    removeWorkspace(workspaceId) {
+        this.pruneExpiredInFlight(Date.now());
+        this.lastFetchedAt.delete(workspaceId);
+        this.inFlightClaims.delete(workspaceId);
+    }
+    size() {
+        this.pruneExpiredInFlight(Date.now());
+        return {
+            completed: this.lastFetchedAt.size,
+            inFlight: this.inFlightClaims.size,
+        };
+    }
+    clear() {
+        this.lastFetchedAt.clear();
+        this.inFlightClaims.clear();
+    }
+    isWithinCooldown(workspaceId, now, cooldownMs) {
+        const lastFetch = this.lastFetchedAt.get(workspaceId);
+        if (lastFetch === undefined) {
+            return false;
+        }
+        return now - lastFetch < (cooldownMs ?? DEFAULT_COOLDOWN_MS);
+    }
+    claim(workspaceId, now) {
         this.ensureCapacityFor(workspaceId);
         this.nextClaimToken += 1;
         const claimToken = this.nextClaimToken;
         this.inFlightClaims.set(workspaceId, { startedAt: now, claimToken });
         return claimToken;
     }
-    register(workspaceId, claimToken) {
-        const now = Date.now();
-        this.pruneExpiredInFlight(now);
+    release(workspaceId, claimToken) {
+        if (this.inFlightClaims.get(workspaceId)?.claimToken === claimToken) {
+            this.inFlightClaims.delete(workspaceId);
+        }
+    }
+    recordCompleted(workspaceId, claimToken) {
         if (this.inFlightClaims.get(workspaceId)?.claimToken !== claimToken) {
             return;
         }
         this.inFlightClaims.delete(workspaceId);
-        this.lastFetchedAt.set(workspaceId, now);
+        this.lastFetchedAt.set(workspaceId, Date.now());
     }
-    cancelFetch(workspaceId, claimToken) {
-        this.pruneExpiredInFlight(Date.now());
-        if (this.inFlightClaims.get(workspaceId)?.claimToken === claimToken) {
-            this.inFlightClaims.delete(workspaceId);
+    pruneExpiredInFlight(now) {
+        for (const [workspaceId, claim] of this.inFlightClaims) {
+            if (now - claim.startedAt >= SERVICE_CACHE_TTL_MS.workspacePrFetchInFlight) {
+                this.inFlightClaims.delete(workspaceId);
+            }
         }
     }
-    isRecentlyFetched(workspaceId, cooldownMs = DEFAULT_COOLDOWN_MS) {
-        const now = Date.now();
-        this.pruneExpiredInFlight(now);
-        if (this.inFlightClaims.has(workspaceId)) {
-            return true;
+    ensureCapacityFor(workspaceId) {
+        if (this.lastFetchedAt.has(workspaceId) || this.inFlightClaims.has(workspaceId)) {
+            return;
         }
-        const lastFetch = this.lastFetchedAt.get(workspaceId);
-        if (lastFetch === undefined) {
-            return false;
+        const workspaceIds = new Set([...this.lastFetchedAt.keys(), ...this.inFlightClaims.keys()]);
+        if (workspaceIds.size < SERVICE_LIMITS.workspaceScopedCacheMaxEntries) {
+            return;
+        }
+        const oldestWorkspaceId = this.oldestBy(this.lastFetchedAt, (timestamp) => timestamp) ??
+            this.oldestBy(this.inFlightClaims, (claim) => claim.startedAt);
+        if (oldestWorkspaceId !== undefined) {
+            this.lastFetchedAt.delete(oldestWorkspaceId);
+            this.inFlightClaims.delete(oldestWorkspaceId);
         }
-        return now - lastFetch < cooldownMs;
-    }
-    isFetchInFlight(workspaceId) {
-        this.pruneExpiredInFlight(Date.now());
-        return this.inFlightClaims.has(workspaceId);
-    }
-    removeWorkspace(workspaceId) {
-        this.pruneExpiredInFlight(Date.now());
-        this.lastFetchedAt.delete(workspaceId);
-        this.inFlightClaims.delete(workspaceId);
-    }
-    size() {
-        this.pruneExpiredInFlight(Date.now());
-        return {
-            completed: this.lastFetchedAt.size,
-            inFlight: this.inFlightClaims.size,
-        };
     }
-    clear() {
-        this.lastFetchedAt.clear();
-        this.inFlightClaims.clear();
+    oldestBy(entries, startedAt) {
+        let oldestWorkspaceId;
+        let oldestTimestamp = Number.POSITIVE_INFINITY;
+        for (const [workspaceId, value] of entries) {
+            const timestamp = startedAt(value);
+            if (timestamp < oldestTimestamp) {
+                oldestWorkspaceId = workspaceId;
+                oldestTimestamp = timestamp;
+            }
+        }
+        return oldestWorkspaceId;
     }
 }
-export const prFetchRegistry = new PRFetchRegistry();
-//# sourceMappingURL=pr-fetch-registry.js.map
+export const prFetchCoordinator = new PRFetchCoordinator();
+//# sourceMappingURL=pr-fetch-coordinator.js.map

package/dist/src/backend/services/github/service/pr-fetch-coordinator.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"pr-fetch-coordinator.js","sourceRoot":"","sources":["../../../../../../src/backend/services/github/service/pr-fetch-coordinator.ts"],"names":[],"mappings":"AAiCA,OAAO,EAAE,oBAAoB,EAAE,cAAc,EAAE,MAAM,8BAA8B,CAAC;AAEpF,MAAM,mBAAmB,GAAG,MAAM,CAAC;AAkCnC,MAAM,OAAO,kBAAkB;IAGZ,aAAa,GAAG,IAAI,GAAG,EAAkB,CAAC;IAC1C,cAAc,GAAG,IAAI,GAAG,EAA8B,CAAC;IAChE,cAAc,GAAG,CAAC,CAAC;IAM3B,KAAK,CAAC,UAAU,CACd,WAAmB,EACnB,KAAuB,EACvB,OAA8B;QAE9B,MAAM,GAAG,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;QACvB,IAAI,CAAC,oBAAoB,CAAC,GAAG,CAAC,CAAC;QAE/B,IAAI,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,WAAW,CAAC,EAAE,CAAC;YACzC,OAAO,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,EAAE,kBAAkB,EAAE,CAAC;QAC3D,CAAC;QACD,IAAI,CAAC,OAAO,EAAE,cAAc,IAAI,IAAI,CAAC,gBAAgB,CAAC,WAAW,EAAE,GAAG,EAAE,OAAO,EAAE,UAAU,CAAC,EAAE,CAAC;YAC7F,OAAO,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,EAAE,kBAAkB,EAAE,CAAC;QAC3D,CAAC;QAED,MAAM,UAAU,GAAG,IAAI,CAAC,KAAK,CAAC,WAAW,EAAE,GAAG,CAAC,CAAC;QAChD,IAAI,CAAC;YACH,MAAM,KAAK,GAAG,MAAM,KAAK,EAAE,CAAC;YAC5B,IAAI,OAAO,EAAE,eAAe,EAAE,CAAC,KAAK,CAAC,IAAI,IAAI,EAAE,CAAC;gBAC9C,IAAI,CAAC,eAAe,CAAC,WAAW,EAAE,UAAU,CAAC,CAAC;YAChD,CAAC;iBAAM,CAAC;gBACN,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,UAAU,CAAC,CAAC;YACxC,CAAC;YACD,OAAO,EAAE,MAAM,EAAE,SAAS,EAAE,KAAK,EAAE,CAAC;QACtC,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YAGf,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,UAAU,CAAC,CAAC;YACtC,MAAM,KAAK,CAAC;QACd,CAAC;IACH,CAAC;IAKD,eAAe,CAAC,WAAmB;QACjC,IAAI,CAAC,oBAAoB,CAAC,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC;QACtC,IAAI,CAAC,aAAa,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QACvC,IAAI,CAAC,cAAc,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;IAC1C,CAAC;IAKD,IAAI;QACF,IAAI,CAAC,oBAAoB,CAAC,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC;QACtC,OAAO;YACL,SAAS,EAAE,IAAI,CAAC,aAAa,CAAC,IAAI;YAClC,QAAQ,EAAE,IAAI,CAAC,cAAc,CAAC,IAAI;SACnC,CAAC;IACJ,CAAC;IAKD,KAAK;QACH,IAAI,CAAC,aAAa,CAAC,KAAK,EAAE,CAAC;QAC3B,IAAI,CAAC,cAAc,CAAC,KAAK,EAAE,CAAC;IAC9B,CAAC;IAEO,gBAAgB,CAAC,WAAmB,EAAE,GAAW,EAAE,UAAmB;QAC5E,MAAM,SAAS,GAAG,IAAI,CAAC,aAAa,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;QACtD,IAAI,SAAS,KAAK,SAAS,EAAE,CAAC;YAC5B,OAAO,KAAK,CAAC;QACf,CAAC;QACD,OAAO,GAAG,GAAG,SAAS,GAAG,CAAC,UAAU,IAAI,mBAAmB,CAAC,CAAC;IAC/D,CAAC;IAEO,KAAK,CAAC,WAAmB,EAAE,GAAW;QAC5C,IAAI,CAAC,iBAAiB,CAAC,WAAW,CAAC,CAAC;QACpC,IAAI,CAAC,cAAc,IAAI,CAAC,CAAC;QACzB,MAAM,UAAU,GAAG,IAAI,CAAC,cAAc,CAAC;QACvC,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,WAAW,EAAE,EAAE,SAAS,EAAE,GAAG,EAAE,UAAU,EAAE,CAAC,CAAC;QACrE,OAAO,UAAU,CAAC;IACpB,CAAC;IAQO,OAAO,CAAC,WAAmB,EAAE,UAAkB;QACrD,IAAI,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,WAAW,CAAC,EAAE,UAAU,KAAK,UAAU,EAAE,CAAC;YACpE,IAAI,CAAC,cAAc,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAC1C,CAAC;IACH,CAAC;IAEO,eAAe,CAAC,WAAmB,EAAE,UAAkB;QAC7D,IAAI,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,WAAW,CAAC,EAAE,UAAU,KAAK,UAAU,EAAE,CAAC;YACpE,OAAO;QACT,CAAC;QACD,IAAI,CAAC,cAAc,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QACxC,IAAI,CAAC,aAAa,CAAC,GAAG,CAAC,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC;IAClD,CAAC;IAEO,oBAAoB,CAAC,GAAW;QACtC,KAAK,MAAM,CAAC,WAAW,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC,cAAc,EAAE,CAAC;YACvD,IAAI,GAAG,GAAG,KAAK,CAAC,SAAS,IAAI,oBAAoB,CAAC,wBAAwB,EAAE,CAAC;gBAC3E,IAAI,CAAC,cAAc,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;YAC1C,CAAC;QACH,CAAC;IACH,CAAC;IAEO,iBAAiB,CAAC,WAAmB;QAC3C,IAAI,IAAI,CAAC,aAAa,CAAC,GAAG,CAAC,WAAW,CAAC,IAAI,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,WAAW,CAAC,EAAE,CAAC;YAChF,OAAO;QACT,CAAC;QAED,MAAM,YAAY,GAAG,IAAI,GAAG,CAAC,CAAC,GAAG,IAAI,CAAC,aAAa,CAAC,IAAI,EAAE,EAAE,GAAG,IAAI,CAAC,cAAc,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC;QAC5F,IAAI,YAAY,CAAC,IAAI,GAAG,cAAc,CAAC,8BAA8B,EAAE,CAAC;YACtE,OAAO;QACT,CAAC;QAOD,MAAM,iBAAiB,GACrB,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,aAAa,EAAE,CAAC,SAAS,EAAE,EAAE,CAAC,SAAS,CAAC;YAE3D,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,cAAc,EAAE,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC;QAEjE,IAAI,iBAAiB,KAAK,SAAS,EAAE,CAAC;YACpC,IAAI,CAAC,aAAa,CAAC,MAAM,CAAC,iBAAiB,CAAC,CAAC;YAC7C,IAAI,CAAC,cAAc,CAAC,MAAM,CAAC,iBAAiB,CAAC,CAAC;QAChD,CAAC;IACH,CAAC;IAEO,QAAQ,CACd,OAAuB,EACvB,SAA+B;QAE/B,IAAI,iBAAqC,CAAC;QAC1C,IAAI,eAAe,GAAG,MAAM,CAAC,iBAAiB,CAAC;QAC/C,KAAK,MAAM,CAAC,WAAW,EAAE,KAAK,CAAC,IAAI,OAAO,EAAE,CAAC;YAC3C,MAAM,SAAS,GAAG,SAAS,CAAC,KAAK,CAAC,CAAC;YACnC,IAAI,SAAS,GAAG,eAAe,EAAE,CAAC;gBAChC,iBAAiB,GAAG,WAAW,CAAC;gBAChC,eAAe,GAAG,SAAS,CAAC;YAC9B,CAAC;QACH,CAAC;QACD,OAAO,iBAAiB,CAAC;IAC3B,CAAC;CACF;AAED,MAAM,CAAC,MAAM,kBAAkB,GAAG,IAAI,kBAAkB,EAAE,CAAC"}

package/dist/src/backend/services/github/service/pr-fetch-coordinator.test.d.ts

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

package/dist/src/backend/services/github/service/pr-fetch-coordinator.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"pr-fetch-coordinator.test.d.ts","sourceRoot":"","sources":["../../../../../../src/backend/services/github/service/pr-fetch-coordinator.test.ts"],"names":[],"mappings":""}

package/dist/src/backend/services/github/service/pr-fetch-coordinator.test.js

@@ -0,0 +1,222 @@
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { PRFetchCoordinator } from './pr-fetch-coordinator.js';
+const COOLDOWN_MS = 90_000;
+const IN_FLIGHT_TTL_MS = 10 * 60_000;
+const MAX_ENTRIES = 1024;
+function deferredFetch() {
+    let settle;
+    let fail;
+    const promise = new Promise((resolveFn, rejectFn) => {
+        settle = resolveFn;
+        fail = rejectFn;
+    });
+    return {
+        fetch: () => promise,
+        resolve: () => {
+            settle('fetched');
+        },
+        reject: () => {
+            fail(new Error('boom'));
+        },
+    };
+}
+describe('PRFetchCoordinator', () => {
+    let coordinator;
+    beforeEach(() => {
+        vi.useFakeTimers();
+        vi.setSystemTime(1000);
+        coordinator = new PRFetchCoordinator();
+    });
+    afterEach(() => {
+        vi.useRealTimers();
+    });
+    describe('running the fetch', () => {
+        it('runs the fetch and returns its value', async () => {
+            const fetch = vi.fn(() => Promise.resolve('value'));
+            await expect(coordinator.coordinate('ws-1', fetch)).resolves.toEqual({
+                status: 'fetched',
+                value: 'value',
+            });
+            expect(fetch).toHaveBeenCalledTimes(1);
+        });
+        it('propagates a rejection instead of swallowing it', async () => {
+            await expect(coordinator.coordinate('ws-1', () => Promise.reject(new Error('boom')))).rejects.toThrow('boom');
+        });
+        it('claims the workspace before the fetch is awaited', () => {
+            const deferred = deferredFetch();
+            const second = vi.fn(() => Promise.resolve('second'));
+            void coordinator.coordinate('ws-1', deferred.fetch);
+            void coordinator.coordinate('ws-1', second);
+            expect(second).not.toHaveBeenCalled();
+            deferred.resolve();
+        });
+    });
+    describe('skipping', () => {
+        it('skips a workspace whose fetch is still in flight', async () => {
+            const deferred = deferredFetch();
+            const first = coordinator.coordinate('ws-1', deferred.fetch);
+            const second = vi.fn(() => Promise.resolve('second'));
+            await expect(coordinator.coordinate('ws-1', second)).resolves.toEqual({
+                status: 'skipped',
+                reason: 'recently_fetched',
+            });
+            expect(second).not.toHaveBeenCalled();
+            deferred.resolve();
+            await first;
+        });
+        it('does not skip a different workspace', async () => {
+            const deferred = deferredFetch();
+            const first = coordinator.coordinate('ws-1', deferred.fetch);
+            const other = vi.fn(() => Promise.resolve('other'));
+            await expect(coordinator.coordinate('ws-2', other)).resolves.toEqual({
+                status: 'fetched',
+                value: 'other',
+            });
+            deferred.resolve();
+            await first;
+        });
+        it('skips within the default cooldown and runs once it has elapsed', async () => {
+            await coordinator.coordinate('ws-1', () => Promise.resolve('first'));
+            vi.advanceTimersByTime(COOLDOWN_MS - 1);
+            const suppressed = vi.fn(() => Promise.resolve('suppressed'));
+            await expect(coordinator.coordinate('ws-1', suppressed)).resolves.toMatchObject({
+                status: 'skipped',
+            });
+            expect(suppressed).not.toHaveBeenCalled();
+            vi.advanceTimersByTime(1);
+            await expect(coordinator.coordinate('ws-1', () => Promise.resolve('second'))).resolves.toEqual({ status: 'fetched', value: 'second' });
+        });
+        it('honors a longer custom cooldown after the default one expires', async () => {
+            await coordinator.coordinate('ws-1', () => Promise.resolve('first'));
+            vi.advanceTimersByTime(COOLDOWN_MS);
+            await expect(coordinator.coordinate('ws-1', () => Promise.resolve('default'), {})).resolves.toMatchObject({ status: 'fetched' });
+            vi.advanceTimersByTime(COOLDOWN_MS);
+            await expect(coordinator.coordinate('ws-1', () => Promise.resolve('custom'), { cooldownMs: 120_000 })).resolves.toMatchObject({ status: 'skipped' });
+        });
+    });
+    describe('ignoreCooldown', () => {
+        it('runs despite a recent completed fetch', async () => {
+            await coordinator.coordinate('ws-1', () => Promise.resolve('first'));
+            await expect(coordinator.coordinate('ws-1', () => Promise.resolve('second'), { ignoreCooldown: true })).resolves.toEqual({ status: 'fetched', value: 'second' });
+        });
+        it('still defers to a fetch that is actively in flight', async () => {
+            const deferred = deferredFetch();
+            const first = coordinator.coordinate('ws-1', deferred.fetch);
+            const second = vi.fn(() => Promise.resolve('second'));
+            await expect(coordinator.coordinate('ws-1', second, { ignoreCooldown: true })).resolves.toMatchObject({ status: 'skipped' });
+            expect(second).not.toHaveBeenCalled();
+            deferred.resolve();
+            await first;
+        });
+    });
+    describe('countsAsFetched', () => {
+        it('does not start a cooldown when the value reports failure', async () => {
+            await coordinator.coordinate('ws-1', () => Promise.resolve({ success: false }), {
+                countsAsFetched: (value) => value.success,
+            });
+            expect(coordinator.size()).toEqual({ completed: 0, inFlight: 0 });
+            await expect(coordinator.coordinate('ws-1', () => Promise.resolve({ success: true }))).resolves.toMatchObject({ status: 'fetched' });
+        });
+        it('starts a cooldown when the value reports success', async () => {
+            await coordinator.coordinate('ws-1', () => Promise.resolve({ success: true }), {
+                countsAsFetched: (value) => value.success,
+            });
+            expect(coordinator.size()).toEqual({ completed: 1, inFlight: 0 });
+        });
+        it('leaves no cooldown behind when the fetch throws', async () => {
+            await expect(coordinator.coordinate('ws-1', () => Promise.reject(new Error('boom')))).rejects.toThrow('boom');
+            expect(coordinator.size()).toEqual({ completed: 0, inFlight: 0 });
+            await expect(coordinator.coordinate('ws-1', () => Promise.resolve('retry'))).resolves.toEqual({ status: 'fetched', value: 'retry' });
+        });
+        it('releases the claim when the fetch throws', async () => {
+            const deferred = deferredFetch();
+            const failing = coordinator.coordinate('ws-1', deferred.fetch);
+            deferred.reject();
+            await expect(failing).rejects.toThrow('boom');
+            expect(coordinator.size()).toEqual({ completed: 0, inFlight: 0 });
+        });
+    });
+    describe('retention', () => {
+        it('removes completed and in-flight entries for one workspace', async () => {
+            await coordinator.coordinate('ws-1', () => Promise.resolve('done'));
+            const deferred = deferredFetch();
+            const inFlight = coordinator.coordinate('ws-2', deferred.fetch);
+            coordinator.removeWorkspace('ws-1');
+            coordinator.removeWorkspace('ws-2');
+            expect(coordinator.size()).toEqual({ completed: 0, inFlight: 0 });
+            deferred.resolve();
+            await inFlight;
+        });
+        it('allows missing and repeated workspace removal', () => {
+            expect(() => coordinator.removeWorkspace('missing')).not.toThrow();
+            expect(() => coordinator.removeWorkspace('missing')).not.toThrow();
+            expect(coordinator.size()).toEqual({ completed: 0, inFlight: 0 });
+        });
+        it('does not record a completion after workspace cleanup', async () => {
+            const deferred = deferredFetch();
+            const pending = coordinator.coordinate('ws-1', deferred.fetch);
+            coordinator.removeWorkspace('ws-1');
+            deferred.resolve();
+            await pending;
+            expect(coordinator.size()).toEqual({ completed: 0, inFlight: 0 });
+        });
+        it('expires an abandoned in-flight claim', async () => {
+            const deferred = deferredFetch();
+            const abandoned = coordinator.coordinate('ws-1', deferred.fetch);
+            vi.advanceTimersByTime(IN_FLIGHT_TTL_MS);
+            const replacement = vi.fn(() => Promise.resolve('replacement'));
+            await expect(coordinator.coordinate('ws-1', replacement)).resolves.toEqual({
+                status: 'fetched',
+                value: 'replacement',
+            });
+            deferred.resolve();
+            await abandoned;
+        });
+        it('keeps the newer claim when an expired one settles late', async () => {
+            const abandonedFetch = deferredFetch();
+            const abandoned = coordinator.coordinate('ws-1', abandonedFetch.fetch);
+            vi.advanceTimersByTime(IN_FLIGHT_TTL_MS);
+            const replacementFetch = deferredFetch();
+            const replacement = coordinator.coordinate('ws-1', replacementFetch.fetch);
+            abandonedFetch.resolve();
+            await abandoned;
+            expect(coordinator.size()).toEqual({ completed: 0, inFlight: 1 });
+            replacementFetch.resolve();
+            await replacement;
+            expect(coordinator.size()).toEqual({ completed: 1, inFlight: 0 });
+        });
+        it('evicts a completed timestamp before an older in-flight claim', async () => {
+            const deferred = deferredFetch();
+            const inFlight = coordinator.coordinate('claimed', deferred.fetch);
+            for (let index = 0; index < MAX_ENTRIES; index += 1) {
+                vi.advanceTimersByTime(1);
+                await coordinator.coordinate(`completed-${index}`, () => Promise.resolve('done'));
+            }
+            const duplicate = vi.fn(() => Promise.resolve('duplicate'));
+            await expect(coordinator.coordinate('claimed', duplicate)).resolves.toMatchObject({
+                status: 'skipped',
+            });
+            expect(duplicate).not.toHaveBeenCalled();
+            deferred.resolve();
+            await inFlight;
+        });
+        it('evicts the oldest workspace when capacity is reached', async () => {
+            await coordinator.coordinate('oldest-completed', () => Promise.resolve('done'));
+            const held = [];
+            for (let index = 0; index < MAX_ENTRIES; index += 1) {
+                vi.advanceTimersByTime(1);
+                const deferred = deferredFetch();
+                held.push({
+                    deferred,
+                    pending: coordinator.coordinate(`in-flight-${index}`, deferred.fetch),
+                });
+            }
+            expect(coordinator.size()).toEqual({ completed: 0, inFlight: MAX_ENTRIES });
+            for (const { deferred, pending } of held) {
+                deferred.resolve();
+                await pending;
+            }
+        });
+    });
+});
+//# sourceMappingURL=pr-fetch-coordinator.test.js.map

package/dist/src/backend/services/github/service/pr-fetch-coordinator.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"pr-fetch-coordinator.test.js","sourceRoot":"","sources":["../../../../../../src/backend/services/github/service/pr-fetch-coordinator.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AACzE,OAAO,EAAE,kBAAkB,EAAE,MAAM,wBAAwB,CAAC;AAE5D,MAAM,WAAW,GAAG,MAAM,CAAC;AAC3B,MAAM,gBAAgB,GAAG,EAAE,GAAG,MAAM,CAAC;AACrC,MAAM,WAAW,GAAG,IAAI,CAAC;AAGzB,SAAS,aAAa;IAKpB,IAAI,MAAgC,CAAC;IACrC,IAAI,IAA6B,CAAC;IAClC,MAAM,OAAO,GAAG,IAAI,OAAO,CAAS,CAAC,SAAS,EAAE,QAAQ,EAAE,EAAE;QAC1D,MAAM,GAAG,SAAS,CAAC;QACnB,IAAI,GAAG,QAAQ,CAAC;IAClB,CAAC,CAAC,CAAC;IACH,OAAO;QACL,KAAK,EAAE,GAAG,EAAE,CAAC,OAAO;QACpB,OAAO,EAAE,GAAG,EAAE;YACZ,MAAM,CAAC,SAAS,CAAC,CAAC;QACpB,CAAC;QACD,MAAM,EAAE,GAAG,EAAE;YACX,IAAI,CAAC,IAAI,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC;QAC1B,CAAC;KACF,CAAC;AACJ,CAAC;AAED,QAAQ,CAAC,oBAAoB,EAAE,GAAG,EAAE;IAClC,IAAI,WAA+B,CAAC;IAEpC,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAC;QACnB,EAAE,CAAC,aAAa,CAAC,IAAI,CAAC,CAAC;QACvB,WAAW,GAAG,IAAI,kBAAkB,EAAE,CAAC;IACzC,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,EAAE,CAAC,aAAa,EAAE,CAAC;IACrB,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,mBAAmB,EAAE,GAAG,EAAE;QACjC,EAAE,CAAC,sCAAsC,EAAE,KAAK,IAAI,EAAE;YACpD,MAAM,KAAK,GAAG,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC;YAEpD,MAAM,MAAM,CAAC,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,KAAK,CAAC,CAAC,CAAC,QAAQ,CAAC,OAAO,CAAC;gBACnE,MAAM,EAAE,SAAS;gBACjB,KAAK,EAAE,OAAO;aACf,CAAC,CAAC;YACH,MAAM,CAAC,KAAK,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QACzC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;YAC/D,MAAM,MAAM,CACV,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,MAAM,CAAC,IAAI,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,CACxE,CAAC,OAAO,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC;QAC5B,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;YAC1D,MAAM,QAAQ,GAAG,aAAa,EAAE,CAAC;YACjC,MAAM,MAAM,GAAG,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC,CAAC;YAItD,KAAK,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,QAAQ,CAAC,KAAK,CAAC,CAAC;YACpD,KAAK,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;YAE5C,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;YACtC,QAAQ,CAAC,OAAO,EAAE,CAAC;QACrB,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,UAAU,EAAE,GAAG,EAAE;QACxB,EAAE,CAAC,kDAAkD,EAAE,KAAK,IAAI,EAAE;YAChE,MAAM,QAAQ,GAAG,aAAa,EAAE,CAAC;YACjC,MAAM,KAAK,GAAG,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,QAAQ,CAAC,KAAK,CAAC,CAAC;YAE7D,MAAM,MAAM,GAAG,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC,CAAC;YACtD,MAAM,MAAM,CAAC,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC,CAAC,QAAQ,CAAC,OAAO,CAAC;gBACpE,MAAM,EAAE,SAAS;gBACjB,MAAM,EAAE,kBAAkB;aAC3B,CAAC,CAAC;YACH,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;YAEtC,QAAQ,CAAC,OAAO,EAAE,CAAC;YACnB,MAAM,KAAK,CAAC;QACd,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,qCAAqC,EAAE,KAAK,IAAI,EAAE;YACnD,MAAM,QAAQ,GAAG,aAAa,EAAE,CAAC;YACjC,MAAM,KAAK,GAAG,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,QAAQ,CAAC,KAAK,CAAC,CAAC;YAE7D,MAAM,KAAK,GAAG,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC;YACpD,MAAM,MAAM,CAAC,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,KAAK,CAAC,CAAC,CAAC,QAAQ,CAAC,OAAO,CAAC;gBACnE,MAAM,EAAE,SAAS;gBACjB,KAAK,EAAE,OAAO;aACf,CAAC,CAAC;YAEH,QAAQ,CAAC,OAAO,EAAE,CAAC;YACnB,MAAM,KAAK,CAAC;QACd,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,gEAAgE,EAAE,KAAK,IAAI,EAAE;YAC9E,MAAM,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC;YAErE,EAAE,CAAC,mBAAmB,CAAC,WAAW,GAAG,CAAC,CAAC,CAAC;YACxC,MAAM,UAAU,GAAG,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,CAAC;YAC9D,MAAM,MAAM,CAAC,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC,CAAC,QAAQ,CAAC,aAAa,CAAC;gBAC9E,MAAM,EAAE,SAAS;aAClB,CAAC,CAAC;YACH,MAAM,CAAC,UAAU,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;YAE1C,EAAE,CAAC,mBAAmB,CAAC,CAAC,CAAC,CAAC;YAC1B,MAAM,MAAM,CACV,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC,CAChE,CAAC,QAAQ,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,SAAS,EAAE,KAAK,EAAE,QAAQ,EAAE,CAAC,CAAC;QAC7D,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;YAC7E,MAAM,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC;YAErE,EAAE,CAAC,mBAAmB,CAAC,WAAW,CAAC,CAAC;YAEpC,MAAM,MAAM,CACV,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,SAAS,CAAC,EAAE,EAAE,CAAC,CACrE,CAAC,QAAQ,CAAC,aAAa,CAAC,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC,CAAC;YAEhD,EAAE,CAAC,mBAAmB,CAAC,WAAW,CAAC,CAAC;YACpC,MAAM,MAAM,CACV,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,QAAQ,CAAC,EAAE,EAAE,UAAU,EAAE,OAAO,EAAE,CAAC,CACzF,CAAC,QAAQ,CAAC,aAAa,CAAC,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC,CAAC;QAClD,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,gBAAgB,EAAE,GAAG,EAAE;QAC9B,EAAE,CAAC,uCAAuC,EAAE,KAAK,IAAI,EAAE;YACrD,MAAM,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC;YAErE,MAAM,MAAM,CACV,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,QAAQ,CAAC,EAAE,EAAE,cAAc,EAAE,IAAI,EAAE,CAAC,CAC1F,CAAC,QAAQ,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,SAAS,EAAE,KAAK,EAAE,QAAQ,EAAE,CAAC,CAAC;QAC7D,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;YAClE,MAAM,QAAQ,GAAG,aAAa,EAAE,CAAC;YACjC,MAAM,KAAK,GAAG,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,QAAQ,CAAC,KAAK,CAAC,CAAC;YAE7D,MAAM,MAAM,GAAG,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC,CAAC;YACtD,MAAM,MAAM,CACV,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,MAAM,EAAE,EAAE,cAAc,EAAE,IAAI,EAAE,CAAC,CACjE,CAAC,QAAQ,CAAC,aAAa,CAAC,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC,CAAC;YAChD,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;YAEtC,QAAQ,CAAC,OAAO,EAAE,CAAC;YACnB,MAAM,KAAK,CAAC;QACd,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;QAC/B,EAAE,CAAC,0DAA0D,EAAE,KAAK,IAAI,EAAE;YACxE,MAAM,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,KAAK,EAAE,CAAC,EAAE;gBAC9E,eAAe,EAAE,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,OAAO;aAC1C,CAAC,CAAC;YAEH,MAAM,CAAC,WAAW,CAAC,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC,CAAC;YAClE,MAAM,MAAM,CACV,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC,CACzE,CAAC,QAAQ,CAAC,aAAa,CAAC,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC,CAAC;QAClD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,kDAAkD,EAAE,KAAK,IAAI,EAAE;YAChE,MAAM,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,EAAE;gBAC7E,eAAe,EAAE,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,OAAO;aAC1C,CAAC,CAAC;YAEH,MAAM,CAAC,WAAW,CAAC,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC,CAAC;QACpE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;YAC/D,MAAM,MAAM,CACV,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,MAAM,CAAC,IAAI,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,CACxE,CAAC,OAAO,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC;YAE1B,MAAM,CAAC,WAAW,CAAC,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC,CAAC;YAClE,MAAM,MAAM,CAAC,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,OAAO,CAC3F,EAAE,MAAM,EAAE,SAAS,EAAE,KAAK,EAAE,OAAO,EAAE,CACtC,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,0CAA0C,EAAE,KAAK,IAAI,EAAE;YACxD,MAAM,QAAQ,GAAG,aAAa,EAAE,CAAC;YACjC,MAAM,OAAO,GAAG,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,QAAQ,CAAC,KAAK,CAAC,CAAC;YAC/D,QAAQ,CAAC,MAAM,EAAE,CAAC;YAClB,MAAM,MAAM,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC;YAE9C,MAAM,CAAC,WAAW,CAAC,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC,CAAC;QACpE,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,WAAW,EAAE,GAAG,EAAE;QACzB,EAAE,CAAC,2DAA2D,EAAE,KAAK,IAAI,EAAE;YACzE,MAAM,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC;YACpE,MAAM,QAAQ,GAAG,aAAa,EAAE,CAAC;YACjC,MAAM,QAAQ,GAAG,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,QAAQ,CAAC,KAAK,CAAC,CAAC;YAEhE,WAAW,CAAC,eAAe,CAAC,MAAM,CAAC,CAAC;YACpC,WAAW,CAAC,eAAe,CAAC,MAAM,CAAC,CAAC;YAEpC,MAAM,CAAC,WAAW,CAAC,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC,CAAC;YAClE,QAAQ,CAAC,OAAO,EAAE,CAAC;YACnB,MAAM,QAAQ,CAAC;QACjB,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;YACvD,MAAM,CAAC,GAAG,EAAE,CAAC,WAAW,CAAC,eAAe,CAAC,SAAS,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,EAAE,CAAC;YACnE,MAAM,CAAC,GAAG,EAAE,CAAC,WAAW,CAAC,eAAe,CAAC,SAAS,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,EAAE,CAAC;YACnE,MAAM,CAAC,WAAW,CAAC,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC,CAAC;QACpE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,sDAAsD,EAAE,KAAK,IAAI,EAAE;YACpE,MAAM,QAAQ,GAAG,aAAa,EAAE,CAAC;YACjC,MAAM,OAAO,GAAG,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,QAAQ,CAAC,KAAK,CAAC,CAAC;YAC/D,WAAW,CAAC,eAAe,CAAC,MAAM,CAAC,CAAC;YAEpC,QAAQ,CAAC,OAAO,EAAE,CAAC;YACnB,MAAM,OAAO,CAAC;YAEd,MAAM,CAAC,WAAW,CAAC,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC,CAAC;QACpE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,sCAAsC,EAAE,KAAK,IAAI,EAAE;YACpD,MAAM,QAAQ,GAAG,aAAa,EAAE,CAAC;YACjC,MAAM,SAAS,GAAG,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,QAAQ,CAAC,KAAK,CAAC,CAAC;YAEjE,EAAE,CAAC,mBAAmB,CAAC,gBAAgB,CAAC,CAAC;YAEzC,MAAM,WAAW,GAAG,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,aAAa,CAAC,CAAC,CAAC;YAChE,MAAM,MAAM,CAAC,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,WAAW,CAAC,CAAC,CAAC,QAAQ,CAAC,OAAO,CAAC;gBACzE,MAAM,EAAE,SAAS;gBACjB,KAAK,EAAE,aAAa;aACrB,CAAC,CAAC;YAEH,QAAQ,CAAC,OAAO,EAAE,CAAC;YACnB,MAAM,SAAS,CAAC;QAClB,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,wDAAwD,EAAE,KAAK,IAAI,EAAE;YACtE,MAAM,cAAc,GAAG,aAAa,EAAE,CAAC;YACvC,MAAM,SAAS,GAAG,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,cAAc,CAAC,KAAK,CAAC,CAAC;YAEvE,EAAE,CAAC,mBAAmB,CAAC,gBAAgB,CAAC,CAAC;YAEzC,MAAM,gBAAgB,GAAG,aAAa,EAAE,CAAC;YACzC,MAAM,WAAW,GAAG,WAAW,CAAC,UAAU,CAAC,MAAM,EAAE,gBAAgB,CAAC,KAAK,CAAC,CAAC;YAG3E,cAAc,CAAC,OAAO,EAAE,CAAC;YACzB,MAAM,SAAS,CAAC;YAChB,MAAM,CAAC,WAAW,CAAC,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC,CAAC;YAElE,gBAAgB,CAAC,OAAO,EAAE,CAAC;YAC3B,MAAM,WAAW,CAAC;YAClB,MAAM,CAAC,WAAW,CAAC,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC,CAAC;QACpE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,8DAA8D,EAAE,KAAK,IAAI,EAAE;YAG5E,MAAM,QAAQ,GAAG,aAAa,EAAE,CAAC;YACjC,MAAM,QAAQ,GAAG,WAAW,CAAC,UAAU,CAAC,SAAS,EAAE,QAAQ,CAAC,KAAK,CAAC,CAAC;YAEnE,KAAK,IAAI,KAAK,GAAG,CAAC,EAAE,KAAK,GAAG,WAAW,EAAE,KAAK,IAAI,CAAC,EAAE,CAAC;gBACpD,EAAE,CAAC,mBAAmB,CAAC,CAAC,CAAC,CAAC;gBAC1B,MAAM,WAAW,CAAC,UAAU,CAAC,aAAa,KAAK,EAAE,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC;YACpF,CAAC;YAED,MAAM,SAAS,GAAG,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,WAAW,CAAC,CAAC,CAAC;YAC5D,MAAM,MAAM,CAAC,WAAW,CAAC,UAAU,CAAC,SAAS,EAAE,SAAS,CAAC,CAAC,CAAC,QAAQ,CAAC,aAAa,CAAC;gBAChF,MAAM,EAAE,SAAS;aAClB,CAAC,CAAC;YACH,MAAM,CAAC,SAAS,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;YAEzC,QAAQ,CAAC,OAAO,EAAE,CAAC;YACnB,MAAM,QAAQ,CAAC;QACjB,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,sDAAsD,EAAE,KAAK,IAAI,EAAE;YACpE,MAAM,WAAW,CAAC,UAAU,CAAC,kBAAkB,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC;YAEhF,MAAM,IAAI,GAAG,EAAE,CAAC;YAChB,KAAK,IAAI,KAAK,GAAG,CAAC,EAAE,KAAK,GAAG,WAAW,EAAE,KAAK,IAAI,CAAC,EAAE,CAAC;gBACpD,EAAE,CAAC,mBAAmB,CAAC,CAAC,CAAC,CAAC;gBAC1B,MAAM,QAAQ,GAAG,aAAa,EAAE,CAAC;gBACjC,IAAI,CAAC,IAAI,CAAC;oBACR,QAAQ;oBACR,OAAO,EAAE,WAAW,CAAC,UAAU,CAAC,aAAa,KAAK,EAAE,EAAE,QAAQ,CAAC,KAAK,CAAC;iBACtE,CAAC,CAAC;YACL,CAAC;YAED,MAAM,CAAC,WAAW,CAAC,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,QAAQ,EAAE,WAAW,EAAE,CAAC,CAAC;YAE5E,KAAK,MAAM,EAAE,QAAQ,EAAE,OAAO,EAAE,IAAI,IAAI,EAAE,CAAC;gBACzC,QAAQ,CAAC,OAAO,EAAE,CAAC;gBACnB,MAAM,OAAO,CAAC;YAChB,CAAC;QACH,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/src/backend/services/github/service/pr-snapshot.service.d.ts

@@ -1,5 +1,5 @@
 import { EventEmitter } from 'node:events';
-import type { GitHubKanbanBridge, GitHubPRDiscoveryClaim, GitHubWorkspaceBridge } from './bridges.js';
+import type { GitHubPRDiscoveryClaim, GitHubWorkspaceBridge } from './bridges.js';
 import { githubCLIService } from './github-cli.service.js';
 type SnapshotData = {
     prNumber: number;
@@ -26,11 +26,7 @@ export type AttachAndRefreshResult = {
     reason: 'workspace_not_found' | 'fetch_failed' | 'claim_stale' | 'stale_observation' | 'error';
 };
 export declare const PR_SNAPSHOT_UPDATED: "pr_snapshot_updated";
-export declare const PR_DISPATCH_INVALIDATED: "pr_dispatch_invalidated";
-export interface PRDispatchInvalidatedEvent {
-    workspaceId: string;
-    prCiStatus: SnapshotData['prCiStatus'];
-}
+export declare const PR_URL_ATTACHED: "pr_url_attached";
 export interface PRSnapshotUpdatedEvent {
     workspaceId: string;
     prUrl?: string | null;
@@ -40,8 +36,16 @@ export interface PRSnapshotUpdatedEvent {
     prReviewState: string | null;
     ratchetDispatchChanged?: true;
 }
-interface CIObservationInput {
+export interface PRUrlAttachedEvent {
+    workspaceId: string;
+    prUrl: string;
+}
+interface PrObservationInput {
+    prNumber: number;
     ciStatus: SnapshotData['prCiStatus'];
+    prState: SnapshotData['prState'];
+    reviewState: string | null;
+    hasMergeConflict: boolean;
     failedAt?: Date | null;
     observedAt?: Date;
 }
@@ -55,17 +59,14 @@ interface ApplySnapshotOptions {
     branchName?: string;
 }
 declare class PRSnapshotService extends EventEmitter {
-    private kanbanBridge;
     private workspaceBridge;
     private readonly workspaceOperations;
     configure(bridges: {
-        kanban: GitHubKanbanBridge;
         workspace: GitHubWorkspaceBridge;
     }): void;
     private get workspace();
-    private get kanban();
     private runWorkspaceOperation;
-    recordCIObservation(workspaceId: string, input: CIObservationInput): Promise<void>;
+    recordPrObservation(workspaceId: string, input: PrObservationInput): Promise<void>;
     recordCINotification(workspaceId: string, notifiedAt?: Date): Promise<void>;
     recordReviewCheck(workspaceId: string, input?: ReviewCheckInput): Promise<void>;
     attachAndRefreshPR(workspaceId: string, prUrl: string): Promise<AttachAndRefreshResult>;

package/dist/src/backend/services/github/service/pr-snapshot.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"pr-snapshot.service.d.ts","sourceRoot":"","sources":["../../../../../../src/backend/services/github/service/pr-snapshot.service.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,YAAY,EAAE,MAAM,aAAa,CAAC;AAG3C,OAAO,KAAK,EAAE,kBAAkB,EAAE,sBAAsB,EAAE,qBAAqB,EAAE,MAAM,WAAW,CAAC;AACnG,OAAO,EAAE,gBAAgB,EAAE,MAAM,sBAAsB,CAAC;AAIxD,KAAK,YAAY,GAAG;IAClB,QAAQ,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,OAAO,CAAC,UAAU,CAAC,OAAO,gBAAgB,CAAC,sBAAsB,CAAC,CAAC,SAAS,MAAM,CAAC,GACxF,CAAC,SAAS;QAAE,OAAO,EAAE,MAAM,CAAC,CAAA;KAAE,GAC5B,CAAC,GACD,KAAK,GACP,KAAK,CAAC;IACV,aAAa,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7B,UAAU,EAAE,OAAO,CAAC,UAAU,CAAC,OAAO,gBAAgB,CAAC,sBAAsB,CAAC,CAAC,SAAS,MAAM,CAAC,GAC3F,CAAC,SAAS;QAAE,UAAU,EAAE,MAAM,CAAC,CAAA;KAAE,GAC/B,CAAC,GACD,KAAK,GACP,KAAK,CAAC;CACX,CAAC;AAEF,MAAM,MAAM,uBAAuB,GAC/B;IAAE,OAAO,EAAE,IAAI,CAAC;IAAC,QAAQ,EAAE,YAAY,CAAA;CAAE,GACzC;IACE,OAAO,EAAE,KAAK,CAAC;IACf,MAAM,EAAE,qBAAqB,GAAG,WAAW,GAAG,cAAc,GAAG,mBAAmB,GAAG,OAAO,CAAC;CAC9F,CAAC;AAEN,MAAM,MAAM,sBAAsB,GAC9B;IAAE,OAAO,EAAE,IAAI,CAAC;IAAC,QAAQ,EAAE,YAAY,CAAA;CAAE,GACzC;IACE,OAAO,EAAE,KAAK,CAAC;IACf,MAAM,EACF,qBAAqB,GACrB,cAAc,GACd,aAAa,GACb,mBAAmB,GACnB,OAAO,CAAC;CACb,CAAC;AAEN,eAAO,MAAM,mBAAmB,EAAG,qBAA8B,CAAC;AAClE,eAAO,MAAM,uBAAuB,EAAG,yBAAkC,CAAC;AAE1E,MAAM,WAAW,0BAA0B;IACzC,WAAW,EAAE,MAAM,CAAC;IACpB,UAAU,EAAE,YAAY,CAAC,YAAY,CAAC,CAAC;CACxC;AAED,MAAM,WAAW,sBAAsB;IACrC,WAAW,EAAE,MAAM,CAAC;IACpB,KAAK,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACtB,QAAQ,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,EAAE,MAAM,CAAC;IACnB,aAAa,EAAE,MAAM,GAAG,IAAI,CAAC;IAE7B,sBAAsB,CAAC,EAAE,IAAI,CAAC;CAC/B;AAED,UAAU,kBAAkB;IAC1B,QAAQ,EAAE,YAAY,CAAC,YAAY,CAAC,CAAC;IACrC,QAAQ,CAAC,EAAE,IAAI,GAAG,IAAI,CAAC;IACvB,UAAU,CAAC,EAAE,IAAI,CAAC;CACnB;AAED,UAAU,gBAAgB;IACxB,SAAS,CAAC,EAAE,IAAI,GAAG,IAAI,CAAC;IACxB,eAAe,CAAC,EAAE,MAAM,CAAC;CAC1B;AAED,UAAU,oBAAoB;IAC5B,UAAU,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,YAAY,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7B,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAED,cAAM,iBAAkB,SAAQ,YAAY;IAC1C,OAAO,CAAC,YAAY,CAAmC;IACvD,OAAO,CAAC,eAAe,CAAsC;IAC7D,OAAO,CAAC,QAAQ,CAAC,mBAAmB,CAAoC;IAExE,SAAS,CAAC,OAAO,EAAE;QAAE,MAAM,EAAE,kBAAkB,CAAC;QAAC,SAAS,EAAE,qBAAqB,CAAA;KAAE,GAAG,IAAI;IAK1F,OAAO,KAAK,SAAS,GAOpB;IAED,OAAO,KAAK,MAAM,GAOjB;IAED,OAAO,CAAC,qBAAqB;IAoBvB,mBAAmB,CAAC,WAAW,EAAE,MAAM,EAAE,KAAK,EAAE,kBAAkB,GAAG,OAAO,CAAC,IAAI,CAAC;IAuBlF,oBAAoB,CAAC,WAAW,EAAE,MAAM,EAAE,UAAU,OAAa,GAAG,OAAO,CAAC,IAAI,CAAC;IASjF,iBAAiB,CAAC,WAAW,EAAE,MAAM,EAAE,KAAK,GAAE,gBAAqB,GAAG,OAAO,CAAC,IAAI,CAAC;IAkBnF,kBAAkB,CAAC,WAAW,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC,sBAAsB,CAAC;YAM/E,qBAAqB;IAsF7B,4BAA4B,CAChC,WAAW,EAAE,MAAM,EACnB,KAAK,EAAE,MAAM,EACb,KAAK,EAAE,sBAAsB,GAC5B,OAAO,CAAC,sBAAsB,CAAC;YAMpB,+BAA+B;IA8DvC,gBAAgB,CACpB,WAAW,EAAE,MAAM,EACnB,aAAa,CAAC,EAAE,MAAM,GAAG,IAAI,GAC5B,OAAO,CAAC,uBAAuB,CAAC;YAMrB,mBAAmB;IAwD3B,aAAa,CACjB,WAAW,EAAE,MAAM,EACnB,QAAQ,EAAE,YAAY,EACtB,OAAO,GAAE,oBAAyB,GACjC,OAAO,CAAC,IAAI,CAAC;YAMF,gBAAgB;CAgC/B;AAED,eAAO,MAAM,iBAAiB,mBAA0B,CAAC"}
+{"version":3,"file":"pr-snapshot.service.d.ts","sourceRoot":"","sources":["../../../../../../src/backend/services/github/service/pr-snapshot.service.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,YAAY,EAAE,MAAM,aAAa,CAAC;AAG3C,OAAO,KAAK,EAAE,sBAAsB,EAAE,qBAAqB,EAAE,MAAM,WAAW,CAAC;AAC/E,OAAO,EAAE,gBAAgB,EAAE,MAAM,sBAAsB,CAAC;AAIxD,KAAK,YAAY,GAAG;IAClB,QAAQ,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,OAAO,CAAC,UAAU,CAAC,OAAO,gBAAgB,CAAC,sBAAsB,CAAC,CAAC,SAAS,MAAM,CAAC,GACxF,CAAC,SAAS;QAAE,OAAO,EAAE,MAAM,CAAC,CAAA;KAAE,GAC5B,CAAC,GACD,KAAK,GACP,KAAK,CAAC;IACV,aAAa,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7B,UAAU,EAAE,OAAO,CAAC,UAAU,CAAC,OAAO,gBAAgB,CAAC,sBAAsB,CAAC,CAAC,SAAS,MAAM,CAAC,GAC3F,CAAC,SAAS;QAAE,UAAU,EAAE,MAAM,CAAC,CAAA;KAAE,GAC/B,CAAC,GACD,KAAK,GACP,KAAK,CAAC;CACX,CAAC;AAEF,MAAM,MAAM,uBAAuB,GAC/B;IAAE,OAAO,EAAE,IAAI,CAAC;IAAC,QAAQ,EAAE,YAAY,CAAA;CAAE,GACzC;IACE,OAAO,EAAE,KAAK,CAAC;IACf,MAAM,EAAE,qBAAqB,GAAG,WAAW,GAAG,cAAc,GAAG,mBAAmB,GAAG,OAAO,CAAC;CAC9F,CAAC;AAEN,MAAM,MAAM,sBAAsB,GAC9B;IAAE,OAAO,EAAE,IAAI,CAAC;IAAC,QAAQ,EAAE,YAAY,CAAA;CAAE,GACzC;IACE,OAAO,EAAE,KAAK,CAAC;IACf,MAAM,EACF,qBAAqB,GACrB,cAAc,GACd,aAAa,GACb,mBAAmB,GACnB,OAAO,CAAC;CACb,CAAC;AAEN,eAAO,MAAM,mBAAmB,EAAG,qBAA8B,CAAC;AAClE,eAAO,MAAM,eAAe,EAAG,iBAA0B,CAAC;AAE1D,MAAM,WAAW,sBAAsB;IACrC,WAAW,EAAE,MAAM,CAAC;IACpB,KAAK,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACtB,QAAQ,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,EAAE,MAAM,CAAC;IACnB,aAAa,EAAE,MAAM,GAAG,IAAI,CAAC;IAE7B,sBAAsB,CAAC,EAAE,IAAI,CAAC;CAC/B;AAED,MAAM,WAAW,kBAAkB;IACjC,WAAW,EAAE,MAAM,CAAC;IACpB,KAAK,EAAE,MAAM,CAAC;CACf;AAGD,UAAU,kBAAkB;IAC1B,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,YAAY,CAAC,YAAY,CAAC,CAAC;IACrC,OAAO,EAAE,YAAY,CAAC,SAAS,CAAC,CAAC;IACjC,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;IAE3B,gBAAgB,EAAE,OAAO,CAAC;IAC1B,QAAQ,CAAC,EAAE,IAAI,GAAG,IAAI,CAAC;IACvB,UAAU,CAAC,EAAE,IAAI,CAAC;CACnB;AAED,UAAU,gBAAgB;IACxB,SAAS,CAAC,EAAE,IAAI,GAAG,IAAI,CAAC;IACxB,eAAe,CAAC,EAAE,MAAM,CAAC;CAC1B;AAED,UAAU,oBAAoB;IAC5B,UAAU,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,YAAY,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7B,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAED,cAAM,iBAAkB,SAAQ,YAAY;IAC1C,OAAO,CAAC,eAAe,CAAsC;IAC7D,OAAO,CAAC,QAAQ,CAAC,mBAAmB,CAAoC;IAExE,SAAS,CAAC,OAAO,EAAE;QAAE,SAAS,EAAE,qBAAqB,CAAA;KAAE,GAAG,IAAI;IAI9D,OAAO,KAAK,SAAS,GAOpB;IAED,OAAO,CAAC,qBAAqB;IA0BvB,mBAAmB,CAAC,WAAW,EAAE,MAAM,EAAE,KAAK,EAAE,kBAAkB,GAAG,OAAO,CAAC,IAAI,CAAC;IA8BlF,oBAAoB,CAAC,WAAW,EAAE,MAAM,EAAE,UAAU,OAAa,GAAG,OAAO,CAAC,IAAI,CAAC;IASjF,iBAAiB,CAAC,WAAW,EAAE,MAAM,EAAE,KAAK,GAAE,gBAAqB,GAAG,OAAO,CAAC,IAAI,CAAC;IAkBnF,kBAAkB,CAAC,WAAW,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC,sBAAsB,CAAC;YAM/E,qBAAqB;IAyF7B,4BAA4B,CAChC,WAAW,EAAE,MAAM,EACnB,KAAK,EAAE,MAAM,EACb,KAAK,EAAE,sBAAsB,GAC5B,OAAO,CAAC,sBAAsB,CAAC;YAMpB,+BAA+B;IAgEvC,gBAAgB,CACpB,WAAW,EAAE,MAAM,EACnB,aAAa,CAAC,EAAE,MAAM,GAAG,IAAI,GAC5B,OAAO,CAAC,uBAAuB,CAAC;YAMrB,mBAAmB;IAwD3B,aAAa,CACjB,WAAW,EAAE,MAAM,EACnB,QAAQ,EAAE,YAAY,EACtB,OAAO,GAAE,oBAAyB,GACjC,OAAO,CAAC,IAAI,CAAC;YAMF,gBAAgB;CA+B/B;AAED,eAAO,MAAM,iBAAiB,mBAA0B,CAAC"}

package/dist/src/backend/services/github/service/pr-snapshot.service.js

@@ -4,13 +4,11 @@ import { createLogger } from '../../../../backend/services/logger.service.js';
 import { githubCLIService } from './github-cli.service.js';
 const logger = createLogger('pr-snapshot');
 export const PR_SNAPSHOT_UPDATED = 'pr_snapshot_updated';
-export const PR_DISPATCH_INVALIDATED = 'pr_dispatch_invalidated';
+export const PR_URL_ATTACHED = 'pr_url_attached';
 class PRSnapshotService extends EventEmitter {
-    kanbanBridge = null;
     workspaceBridge = null;
     workspaceOperations = new Map();
     configure(bridges) {
-        this.kanbanBridge = bridges.kanban;
         this.workspaceBridge = bridges.workspace;
     }
     get workspace() {
@@ -19,12 +17,6 @@ class PRSnapshotService extends EventEmitter {
         }
         return this.workspaceBridge;
     }
-    get kanban() {
-        if (!this.kanbanBridge) {
-            throw new Error('PRSnapshotService not configured: kanban bridge missing. Call configure() first.');
-        }
-        return this.kanbanBridge;
-    }
     runWorkspaceOperation(workspaceId, operation) {
         const previous = this.workspaceOperations.get(workspaceId) ?? Promise.resolve();
         const result = previous.catch(() => undefined).then(operation);
@@ -37,23 +29,28 @@ class PRSnapshotService extends EventEmitter {
         });
         return result;
     }
-    async recordCIObservation(workspaceId, input) {
+    async recordPrObservation(workspaceId, input) {
         await this.runWorkspaceOperation(workspaceId, async () => {
-            const result = await this.workspace.applyCIObservationWithDispatchReset(workspaceId, {
+            const result = await this.workspace.applyPrObservationWithDispatchReset(workspaceId, {
+                expectedPrNumber: input.prNumber,
                 prCiStatus: input.ciStatus,
+                prState: input.prState,
+                prReviewState: input.reviewState,
+                prHasMergeConflict: input.hasMergeConflict,
                 prUpdatedAt: input.observedAt ?? new Date(),
                 ...(input.failedAt !== undefined ? { prCiFailedAt: input.failedAt ?? null } : {}),
             });
             if (!result.applied) {
                 return;
             }
-            if (result.dispatchReset) {
-                this.emit(PR_DISPATCH_INVALIDATED, {
-                    workspaceId,
-                    prCiStatus: input.ciStatus,
-                });
-            }
-            await this.kanban.updateCachedKanbanColumn(workspaceId);
+            this.emit(PR_SNAPSHOT_UPDATED, {
+                workspaceId,
+                prNumber: input.prNumber,
+                prState: input.prState,
+                prCiStatus: input.ciStatus,
+                prReviewState: input.reviewState,
+                ...(result.dispatchReset ? { ratchetDispatchChanged: true } : {}),
+            });
         });
     }
     async recordCINotification(workspaceId, notifiedAt = new Date()) {
@@ -85,7 +82,10 @@ class PRSnapshotService extends EventEmitter {
                     prUrl,
                     prUpdatedAt: new Date(),
                 });
-                await this.kanban.updateCachedKanbanColumn(workspaceId);
+                this.emit(PR_URL_ATTACHED, {
+                    workspaceId,
+                    prUrl,
+                });
                 logger.warn('Attached PR URL but could not fetch snapshot', { workspaceId, prUrl });
                 return { success: false, reason: 'fetch_failed' };
             }
@@ -146,7 +146,10 @@ class PRSnapshotService extends EventEmitter {
             }
             const snapshot = await githubCLIService.fetchAndComputePRState(prUrl);
             if (!snapshot) {
-                await this.kanban.updateCachedKanbanColumn(workspaceId);
+                this.emit(PR_URL_ATTACHED, {
+                    workspaceId,
+                    prUrl,
+                });
                 logger.warn('Attached discovered PR URL but could not fetch snapshot', {
                     workspaceId,
                     prUrl,
@@ -163,7 +166,6 @@ class PRSnapshotService extends EventEmitter {
             if (!persisted) {
                 return { success: false, reason: 'claim_stale' };
             }
-            await this.kanban.updateCachedKanbanColumn(workspaceId);
             this.emit(PR_SNAPSHOT_UPDATED, {
                 workspaceId,
                 prUrl,
@@ -254,7 +256,6 @@ class PRSnapshotService extends EventEmitter {
             prReviewState: snapshot.prReviewState,
             ...(result.dispatchReset ? { ratchetDispatchChanged: true } : {}),
         });
-        await this.kanban.updateCachedKanbanColumn(workspaceId);
         return true;
     }
 }