factory-factory 0.4.9 → 0.4.10

package/dist/src/backend/services/github/service/github-cli-full-details-buffer.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"github-cli-full-details-buffer.test.js","sourceRoot":"","sources":["../../../../../../src/backend/services/github/service/github-cli-full-details-buffer.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAwB,QAAQ,EAAE,MAAM,oBAAoB,CAAC;AACpE,OAAO,EAAE,OAAO,EAAE,EAAE,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAC1D,OAAO,EAAE,MAAM,EAAE,MAAM,SAAS,CAAC;AACjC,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EAAE,SAAS,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAEzE,EAAE,CAAC,IAAI,CAAC,oBAAoB,EAAE,GAAG,EAAE,CAAC,CAAC,EAAE,QAAQ,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,CAAC;AAC7D,EAAE,CAAC,IAAI,CAAC,WAAW,EAAE,GAAG,EAAE,CAAC,CAAC,EAAE,SAAS,EAAE,CAAC,EAAW,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC;AACjE,EAAE,CAAC,IAAI,CAAC,mCAAmC,EAAE,GAAG,EAAE,CAAC,CAAC;IAClD,YAAY,EAAE,GAAG,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,KAAK,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,KAAK,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC;CACvF,CAAC,CAAC,CAAC;AAEJ,OAAO,EAAE,gBAAgB,EAAE,MAAM,sBAAsB,CAAC;AAExD,MAAM,gBAAgB,GACpB,MAAM,EAAE,CAAC,YAAY,CAAsC,oBAAoB,CAAC,CAAC;AACnF,MAAM,QAAQ,GAAG,MAAM,EAAE,CAAC,YAAY,CAA6B,WAAW,CAAC,CAAC;AAChF,MAAM,WAAW,GAAG,QAAQ,CAAC,SAAS,CAAC,gBAAgB,CAAC,QAAQ,CAAC,CAAC;AAClE,IAAI,gBAAwB,CAAC;AAC7B,IAAI,WAAmB,CAAC;AACxB,IAAI,iBAAyB,CAAC;AAE9B,SAAS,kBAAkB;IACzB,MAAM,SAAS,GAAG,sBAAsB,CAAC;IACzC,OAAO;QACL,MAAM,EAAE,GAAG;QACX,KAAK,EAAE,UAAU;QACjB,GAAG,EAAE,wCAAwC;QAC7C,MAAM,EAAE,EAAE,KAAK,EAAE,QAAQ,EAAE;QAC3B,SAAS,EAAE,SAAS;QACpB,SAAS,EAAE,SAAS;QACpB,OAAO,EAAE,KAAK;QACd,KAAK,EAAE,MAAM;QACb,cAAc,EAAE,UAAU;QAC1B,OAAO,EAAE,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,GAAG,EAAE,EAAE,CAAC,CAAC,EAAE,KAAK,EAAE,EAAE,CAAC,CAAC;YAClD,EAAE,EAAE,UAAU,KAAK,EAAE;YACrB,MAAM,EAAE,EAAE,KAAK,EAAE,UAAU,EAAE;YAC7B,KAAK,EAAE,UAAU;YACjB,WAAW,EAAE,SAAS;YACtB,IAAI,EAAE,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC;SACvB,CAAC,CAAC;QACH,QAAQ,EAAE,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,GAAG,EAAE,EAAE,CAAC,CAAC,EAAE,KAAK,EAAE,EAAE,CAAC,CAAC;YACnD,EAAE,EAAE,WAAW,KAAK,EAAE;YACtB,MAAM,EAAE,EAAE,KAAK,EAAE,WAAW,EAAE;YAC9B,IAAI,EAAE,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC;YACtB,SAAS,EAAE,SAAS;YACpB,SAAS,EAAE,SAAS;YACpB,GAAG,EAAE,uDAAuD,KAAK,EAAE;SACpE,CAAC,CAAC;QACH,iBAAiB,EAAE,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,CAAC,CAAC,EAAE,KAAK,EAAE,EAAE,CAAC,CAAC;YAC3D,UAAU,EAAE,UAAU;YACtB,IAAI,EAAE,SAAS,KAAK,EAAE;YACtB,MAAM,EAAE,WAAW;YACnB,UAAU,EAAE,SAAS;SACtB,CAAC,CAAC;QACH,MAAM,EAAE,CAAC,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,QAAQ,EAAE,CAAC;QAC1C,SAAS,EAAE,GAAG;QACd,SAAS,EAAE,EAAE;QACb,YAAY,EAAE,CAAC;QACf,WAAW,EAAE,SAAS;QACtB,WAAW,EAAE,MAAM;QACnB,gBAAgB,EAAE,OAAO;KAC1B,CAAC;AACJ,CAAC;AAED,UAAU,CAAC,KAAK,IAAI,EAAE;IACpB,gBAAgB,CAAC,WAAW,EAAE,CAAC;IAC/B,gBAAgB,GAAG,MAAM,OAAO,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,kBAAkB,CAAC,CAAC,CAAC;IACrE,WAAW,GAAG,IAAI,CAAC,gBAAgB,EAAE,SAAS,CAAC,CAAC;IAChD,iBAAiB,GAAG,IAAI,CAAC,gBAAgB,EAAE,cAAc,CAAC,CAAC;IAG3D,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,kBAAkB,CAAC,CAAC,CACtC,OAAe,EACf,IAAc,EACd,OAAwB,EACxB,EAAE;QACF,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC3B,IAAI,IAAI,CAAC,CAAC,CAAC,KAAK,KAAK,EAAE,CAAC;YACtB,MAAM,CAAC,IAAI,CAAC,CAAC,OAAO,CAAC;gBACnB,KAAK;gBACL,iDAAiD;gBACjD,YAAY;gBACZ,SAAS;aACV,CAAC,CAAC;QACL,CAAC;aAAM,CAAC;YACN,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,MAAM,EAAE,KAAK,CAAC,CAAC,CAAC;QAC1D,CAAC;QACD,OAAO,WAAW,CAChB,OAAO,CAAC,QAAQ,EAChB;YACE,IAAI;YACJ,wEAAwE;YACxE,IAAI,CAAC,CAAC,CAAC,KAAK,KAAK,CAAC,CAAC,CAAC,iBAAiB,CAAC,CAAC,CAAC,WAAW;SACpD,EACD,OAAO,CACR,CAAC;IACJ,CAAC,CAAU,CAAC,CAAC;AACf,CAAC,CAAC,CAAC;AAEH,SAAS,CAAC,KAAK,IAAI,EAAE;IACnB,MAAM,EAAE,CAAC,gBAAgB,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;AAC/D,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,kCAAkC,EAAE,GAAG,EAAE;IAChD,EAAE,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,IAAI,CAAC,CAAC,CACpB,mDAAmD,EACnD,KAAK,EAAE,UAAU,EAAE,EAAE;QACnB,MAAM,OAAO,GAAG,kBAAkB,EAAE,CAAC;QACrC,MAAM,WAAW,GAAG,OAAO,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC;YACnD,OAAO,EAAE,MAAM,CAAC,EAAE;YAClB,IAAI,EAAE,MAAM,CAAC,MAAM;YACnB,KAAK,EAAE,MAAM,CAAC,KAAK;YACnB,YAAY,EAAE,MAAM,CAAC,WAAW;YAChC,IAAI,EAAE,MAAM,CAAC,IAAI;SAClB,CAAC,CAAC,CAAC;QACJ,MAAM,aAAa,GAAG,IAAI,CAAC,SAAS,CAAC,CAAC,WAAW,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,WAAW,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC;QAC1F,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,GAAG,QAAQ,EAAE,GAAG,OAAO,CAAC;QAClD,MAAM,OAAO,GAAG,IAAI,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAC;QACzC,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC,CAAC,eAAe,CAAC,IAAI,GAAG,IAAI,CAAC,CAAC;QAChE,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,aAAa,CAAC,CAAC,CAAC,eAAe,CAAC,IAAI,GAAG,IAAI,CAAC,CAAC;QACtE,MAAM,SAAS,CAAC,WAAW,EAAE,OAAO,CAAC,CAAC;QACtC,MAAM,SAAS,CAAC,iBAAiB,EAAE,aAAa,CAAC,CAAC;QAElD,MAAM,MAAM,GAAG,MAAM,gBAAgB,CAAC,gBAAgB,CACpD,YAAY,EACZ,GAAG,EACH,UAAU,CAAC,CAAC,CAAC,IAAI,eAAe,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,SAAS,CACtD,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAChC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC;QACzC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,aAAa,CAAC;YAC1C,EAAE,EAAE,YAAY;YAChB,IAAI,EAAE,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC;YACtB,kBAAkB,EAAE,GAAG;SACxB,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC;QAC1C,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,aAAa,CAAC,EAAE,EAAE,EAAE,aAAa,EAAE,IAAI,EAAE,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAC5F,MAAM,CAAC,MAAM,CAAC,iBAAiB,CAAC,CAAC,YAAY,CAAC,EAAE,CAAC,CAAC;QAClD,MAAM,CAAC,MAAM,CAAC,iBAAiB,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,aAAa,CAAC;YACrD,IAAI,EAAE,UAAU;YAChB,UAAU,EAAE,SAAS;SACtB,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,CAAC,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,QAAQ,EAAE,CAAC,CAAC,CAAC;QAClE,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IAC7C,CAAC,CACF,CAAC;IAEF,EAAE,CAAC,6DAA6D,EAAE,KAAK,IAAI,EAAE;QAC3E,MAAM,OAAO,GAAG,kBAAkB,EAAE,CAAC;QACrC,OAAO,CAAC,KAAK,GAAG,GAAG,CAAC,MAAM,CAAC,EAAE,GAAG,IAAI,GAAG,IAAI,CAAC,CAAC;QAC7C,MAAM,SAAS,CAAC,WAAW,EAAE,IAAI,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC,CAAC;QAEtD,MAAM,MAAM,CAAC,gBAAgB,CAAC,gBAAgB,CAAC,YAAY,EAAE,GAAG,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAChF,kCAAkC,CACnC,CAAC;IACJ,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/src/backend/services/github/service/github-cli-pr-details.test.d.ts

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

package/dist/src/backend/services/github/service/github-cli-pr-details.test.d.ts.map

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

package/dist/src/backend/services/github/service/github-cli-pr-details.test.js

@@ -0,0 +1,150 @@
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+const mockExecFile = vi.fn();
+const mockLoggerInfo = vi.fn();
+const mockLoggerDebug = vi.fn();
+const mockLoggerWarn = vi.fn();
+const mockLoggerError = vi.fn();
+vi.mock('node:child_process', () => ({
+    execFile: vi.fn(),
+}));
+vi.mock('node:util', () => ({
+    promisify: (fn) => fn,
+}));
+vi.mock('@/backend/services/logger.service', () => ({
+    createLogger: () => ({
+        get info() {
+            return mockLoggerInfo;
+        },
+        get debug() {
+            return mockLoggerDebug;
+        },
+        get warn() {
+            return mockLoggerWarn;
+        },
+        get error() {
+            return mockLoggerError;
+        },
+    }),
+}));
+import { execFile } from 'node:child_process';
+import { githubCLIService } from './github-cli.service.js';
+vi.mocked(execFile).mockImplementation(mockExecFile);
+describe('PR detail review fetch coordination', () => {
+    beforeEach(() => {
+        vi.clearAllMocks();
+        githubCLIService.clearCaches();
+    });
+    afterEach(() => {
+        vi.clearAllMocks();
+    });
+    it('passes abort signals to PR detail child processes', async () => {
+        const controller = new AbortController();
+        mockExecFile.mockResolvedValueOnce({
+            stdout: JSON.stringify({
+                number: 42,
+                title: 'PR',
+                url: 'https://github.com/owner/repo/pull/42',
+                author: { login: 'author' },
+                createdAt: '2026-01-01T00:00:00Z',
+                updatedAt: '2026-01-01T00:00:00Z',
+                isDraft: false,
+                state: 'OPEN',
+                reviewDecision: null,
+                statusCheckRollup: [],
+                reviews: [],
+                comments: [],
+                labels: [],
+                additions: 0,
+                deletions: 0,
+                changedFiles: 0,
+                headRefName: 'feature',
+                baseRefName: 'main',
+                mergeStateStatus: 'CLEAN',
+            }),
+            stderr: '',
+        });
+        mockExecFile.mockResolvedValueOnce({ stdout: '[[]]', stderr: '' });
+        await githubCLIService.getPRFullDetails('owner/repo', 42, controller.signal);
+        expect(mockExecFile).toHaveBeenCalledWith('gh', ['api', 'repos/owner/repo/pulls/42/reviews?per_page=100', '--paginate', '--slurp'], expect.objectContaining({ signal: controller.signal, maxBuffer: 10 * 1024 * 1024 }));
+    });
+    it('does not singleflight identical signal-bound PR reads', async () => {
+        const first = new AbortController();
+        const second = new AbortController();
+        const prDetails = {
+            number: 42,
+            title: 'PR',
+            url: 'https://github.com/owner/repo/pull/42',
+            author: { login: 'author' },
+            createdAt: '2026-01-01T00:00:00Z',
+            updatedAt: '2026-01-01T00:00:00Z',
+            isDraft: false,
+            state: 'OPEN',
+            reviewDecision: null,
+            statusCheckRollup: [],
+            reviews: [],
+            comments: [],
+            labels: [],
+            additions: 0,
+            deletions: 0,
+            changedFiles: 0,
+            headRefName: 'feature',
+            baseRefName: 'main',
+            mergeStateStatus: 'CLEAN',
+        };
+        mockExecFile.mockImplementation((_command, args) => Promise.resolve({
+            stdout: args[0] === 'api' ? '[[]]' : JSON.stringify(prDetails),
+            stderr: '',
+        }));
+        await Promise.all([
+            githubCLIService.getPRFullDetails('owner/repo', 42, first.signal),
+            githubCLIService.getPRFullDetails('owner/repo', 42, second.signal),
+        ]);
+        expect(mockExecFile).toHaveBeenCalledTimes(4);
+    });
+    it('does not spawn a signal-bound read that is cancelled while queued', async () => {
+        const prDetails = {
+            number: 42,
+            title: 'PR',
+            url: 'https://github.com/owner/repo/pull/42',
+            author: { login: 'author' },
+            createdAt: '2026-01-01T00:00:00Z',
+            updatedAt: '2026-01-01T00:00:00Z',
+            isDraft: false,
+            state: 'OPEN',
+            reviewDecision: null,
+            statusCheckRollup: [],
+            reviews: [],
+            comments: [],
+            labels: [],
+            additions: 0,
+            deletions: 0,
+            changedFiles: 0,
+            headRefName: 'feature',
+            baseRefName: 'main',
+            mergeStateStatus: 'CLEAN',
+        };
+        const releases = [];
+        mockExecFile.mockImplementation((_command, args) => args[0] === 'api'
+            ? Promise.resolve({ stdout: '[[]]', stderr: '' })
+            : new Promise((resolve) => {
+                releases.push(() => resolve({ stdout: JSON.stringify(prDetails), stderr: '' }));
+            }));
+        const blockers = Array.from({ length: 5 }, (_, index) => {
+            const controller = new AbortController();
+            return githubCLIService.getPRFullDetails('owner/repo', index + 1, controller.signal);
+        });
+        await vi.waitFor(() => expect(mockExecFile).toHaveBeenCalledTimes(5));
+        const queuedController = new AbortController();
+        const abortReason = new Error('queued request cancelled');
+        const queued = githubCLIService.getPRFullDetails('owner/repo', 99, queuedController.signal);
+        queuedController.abort(abortReason);
+        releases.shift()?.();
+        await expect(queued).rejects.toBe(abortReason);
+        expect(mockExecFile.mock.calls.filter(([, args]) => args[0] === 'pr')).toHaveLength(5);
+        for (const release of releases) {
+            release();
+        }
+        await Promise.all(blockers);
+    });
+});
+//# sourceMappingURL=github-cli-pr-details.test.js.map

package/dist/src/backend/services/github/service/github-cli-pr-details.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"github-cli-pr-details.test.js","sourceRoot":"","sources":["../../../../../../src/backend/services/github/service/github-cli-pr-details.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAGzE,MAAM,YAAY,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;AAC7B,MAAM,cAAc,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;AAC/B,MAAM,eAAe,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;AAChC,MAAM,cAAc,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;AAC/B,MAAM,eAAe,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;AAEhC,EAAE,CAAC,IAAI,CAAC,oBAAoB,EAAE,GAAG,EAAE,CAAC,CAAC;IACnC,QAAQ,EAAE,EAAE,CAAC,EAAE,EAAE;CAClB,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,WAAW,EAAE,GAAG,EAAE,CAAC,CAAC;IAC1B,SAAS,EAAE,CAAC,EAAW,EAAE,EAAE,CAAC,EAAE;CAC/B,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,mCAAmC,EAAE,GAAG,EAAE,CAAC,CAAC;IAClD,YAAY,EAAE,GAAG,EAAE,CAAC,CAAC;QACnB,IAAI,IAAI;YACN,OAAO,cAAc,CAAC;QACxB,CAAC;QACD,IAAI,KAAK;YACP,OAAO,eAAe,CAAC;QACzB,CAAC;QACD,IAAI,IAAI;YACN,OAAO,cAAc,CAAC;QACxB,CAAC;QACD,IAAI,KAAK;YACP,OAAO,eAAe,CAAC;QACzB,CAAC;KACF,CAAC;CACH,CAAC,CAAC,CAAC;AAGJ,OAAO,EAAE,QAAQ,EAAE,MAAM,oBAAoB,CAAC;AAC9C,OAAO,EAAE,gBAAgB,EAAE,MAAM,sBAAsB,CAAC;AAExD,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,kBAAkB,CAAC,YAAqB,CAAC,CAAC;AAE9D,QAAQ,CAAC,qCAAqC,EAAE,GAAG,EAAE;IACnD,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAC;QACnB,gBAAgB,CAAC,WAAW,EAAE,CAAC;IACjC,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,EAAE,CAAC,aAAa,EAAE,CAAC;IACrB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,KAAK,IAAI,EAAE;QACjE,MAAM,UAAU,GAAG,IAAI,eAAe,EAAE,CAAC;QACzC,YAAY,CAAC,qBAAqB,CAAC;YACjC,MAAM,EAAE,IAAI,CAAC,SAAS,CAAC;gBACrB,MAAM,EAAE,EAAE;gBACV,KAAK,EAAE,IAAI;gBACX,GAAG,EAAE,uCAAuC;gBAC5C,MAAM,EAAE,EAAE,KAAK,EAAE,QAAQ,EAAE;gBAC3B,SAAS,EAAE,sBAAsB;gBACjC,SAAS,EAAE,sBAAsB;gBACjC,OAAO,EAAE,KAAK;gBACd,KAAK,EAAE,MAAM;gBACb,cAAc,EAAE,IAAI;gBACpB,iBAAiB,EAAE,EAAE;gBACrB,OAAO,EAAE,EAAE;gBACX,QAAQ,EAAE,EAAE;gBACZ,MAAM,EAAE,EAAE;gBACV,SAAS,EAAE,CAAC;gBACZ,SAAS,EAAE,CAAC;gBACZ,YAAY,EAAE,CAAC;gBACf,WAAW,EAAE,SAAS;gBACtB,WAAW,EAAE,MAAM;gBACnB,gBAAgB,EAAE,OAAO;aAC1B,CAAC;YACF,MAAM,EAAE,EAAE;SACX,CAAC,CAAC;QAEH,YAAY,CAAC,qBAAqB,CAAC,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC,CAAC;QACnE,MAAM,gBAAgB,CAAC,gBAAgB,CAAC,YAAY,EAAE,EAAE,EAAE,UAAU,CAAC,MAAM,CAAC,CAAC;QAE7E,MAAM,CAAC,YAAY,CAAC,CAAC,oBAAoB,CACvC,IAAI,EACJ,CAAC,KAAK,EAAE,gDAAgD,EAAE,YAAY,EAAE,SAAS,CAAC,EAClF,MAAM,CAAC,gBAAgB,CAAC,EAAE,MAAM,EAAE,UAAU,CAAC,MAAM,EAAE,SAAS,EAAE,EAAE,GAAG,IAAI,GAAG,IAAI,EAAE,CAAC,CACpF,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uDAAuD,EAAE,KAAK,IAAI,EAAE;QACrE,MAAM,KAAK,GAAG,IAAI,eAAe,EAAE,CAAC;QACpC,MAAM,MAAM,GAAG,IAAI,eAAe,EAAE,CAAC;QACrC,MAAM,SAAS,GAAG;YAChB,MAAM,EAAE,EAAE;YACV,KAAK,EAAE,IAAI;YACX,GAAG,EAAE,uCAAuC;YAC5C,MAAM,EAAE,EAAE,KAAK,EAAE,QAAQ,EAAE;YAC3B,SAAS,EAAE,sBAAsB;YACjC,SAAS,EAAE,sBAAsB;YACjC,OAAO,EAAE,KAAK;YACd,KAAK,EAAE,MAAM;YACb,cAAc,EAAE,IAAI;YACpB,iBAAiB,EAAE,EAAE;YACrB,OAAO,EAAE,EAAE;YACX,QAAQ,EAAE,EAAE;YACZ,MAAM,EAAE,EAAE;YACV,SAAS,EAAE,CAAC;YACZ,SAAS,EAAE,CAAC;YACZ,YAAY,EAAE,CAAC;YACf,WAAW,EAAE,SAAS;YACtB,WAAW,EAAE,MAAM;YACnB,gBAAgB,EAAE,OAAO;SAC1B,CAAC;QACF,YAAY,CAAC,kBAAkB,CAAC,CAAC,QAAQ,EAAE,IAAc,EAAE,EAAE,CAC3D,OAAO,CAAC,OAAO,CAAC;YACd,MAAM,EAAE,IAAI,CAAC,CAAC,CAAC,KAAK,KAAK,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,SAAS,CAAC;YAC9D,MAAM,EAAE,EAAE;SACX,CAAC,CACH,CAAC;QAEF,MAAM,OAAO,CAAC,GAAG,CAAC;YAChB,gBAAgB,CAAC,gBAAgB,CAAC,YAAY,EAAE,EAAE,EAAE,KAAK,CAAC,MAAM,CAAC;YACjE,gBAAgB,CAAC,gBAAgB,CAAC,YAAY,EAAE,EAAE,EAAE,MAAM,CAAC,MAAM,CAAC;SACnE,CAAC,CAAC;QAEH,MAAM,CAAC,YAAY,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;IAChD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mEAAmE,EAAE,KAAK,IAAI,EAAE;QACjF,MAAM,SAAS,GAAG;YAChB,MAAM,EAAE,EAAE;YACV,KAAK,EAAE,IAAI;YACX,GAAG,EAAE,uCAAuC;YAC5C,MAAM,EAAE,EAAE,KAAK,EAAE,QAAQ,EAAE;YAC3B,SAAS,EAAE,sBAAsB;YACjC,SAAS,EAAE,sBAAsB;YACjC,OAAO,EAAE,KAAK;YACd,KAAK,EAAE,MAAM;YACb,cAAc,EAAE,IAAI;YACpB,iBAAiB,EAAE,EAAE;YACrB,OAAO,EAAE,EAAE;YACX,QAAQ,EAAE,EAAE;YACZ,MAAM,EAAE,EAAE;YACV,SAAS,EAAE,CAAC;YACZ,SAAS,EAAE,CAAC;YACZ,YAAY,EAAE,CAAC;YACf,WAAW,EAAE,SAAS;YACtB,WAAW,EAAE,MAAM;YACnB,gBAAgB,EAAE,OAAO;SAC1B,CAAC;QACF,MAAM,QAAQ,GAAsB,EAAE,CAAC;QACvC,YAAY,CAAC,kBAAkB,CAAC,CAAC,QAAQ,EAAE,IAAc,EAAE,EAAE,CAC3D,IAAI,CAAC,CAAC,CAAC,KAAK,KAAK;YACf,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC;YACjD,CAAC,CAAC,IAAI,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE;gBACtB,QAAQ,CAAC,IAAI,CAAC,GAAG,EAAE,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,IAAI,CAAC,SAAS,CAAC,SAAS,CAAC,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC,CAAC,CAAC;YAClF,CAAC,CAAC,CACP,CAAC;QAEF,MAAM,QAAQ,GAAG,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,EAAE,KAAK,EAAE,EAAE;YACtD,MAAM,UAAU,GAAG,IAAI,eAAe,EAAE,CAAC;YACzC,OAAO,gBAAgB,CAAC,gBAAgB,CAAC,YAAY,EAAE,KAAK,GAAG,CAAC,EAAE,UAAU,CAAC,MAAM,CAAC,CAAC;QACvF,CAAC,CAAC,CAAC;QACH,MAAM,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC,CAAC;QAEtE,MAAM,gBAAgB,GAAG,IAAI,eAAe,EAAE,CAAC;QAC/C,MAAM,WAAW,GAAG,IAAI,KAAK,CAAC,0BAA0B,CAAC,CAAC;QAC1D,MAAM,MAAM,GAAG,gBAAgB,CAAC,gBAAgB,CAAC,YAAY,EAAE,EAAE,EAAE,gBAAgB,CAAC,MAAM,CAAC,CAAC;QAC5F,gBAAgB,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC;QACpC,QAAQ,CAAC,KAAK,EAAE,EAAE,EAAE,CAAC;QAErB,MAAM,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;QAC/C,MAAM,CAAC,YAAY,CAAC,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,IAAI,CAAC,EAAE,EAAE,CAAC,IAAI,CAAC,CAAC,CAAC,KAAK,IAAI,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAEvF,KAAK,MAAM,OAAO,IAAI,QAAQ,EAAE,CAAC;YAC/B,OAAO,EAAE,CAAC;QACZ,CAAC;QACD,MAAM,OAAO,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC;IAC9B,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/src/backend/services/github/service/github-cli-review-authors.test.d.ts

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

package/dist/src/backend/services/github/service/github-cli-review-authors.test.d.ts.map

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

package/dist/src/backend/services/github/service/github-cli-review-authors.test.js

@@ -0,0 +1,57 @@
+import { execFile } from 'node:child_process';
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+vi.mock('node:child_process', () => ({ execFile: vi.fn() }));
+vi.mock('node:util', () => ({ promisify: (fn) => fn }));
+vi.mock('@/backend/services/logger.service', () => ({
+    createLogger: () => ({ info: vi.fn(), debug: vi.fn(), warn: vi.fn(), error: vi.fn() }),
+}));
+import { githubCLIService } from './github-cli.service.js';
+import { reviewCommentSchema } from './github-cli/schemas.js';
+function comment(id, user = { login: 'reviewer' }) {
+    return {
+        id,
+        user,
+        body: `Review ${id}`,
+        path: 'src/index.ts',
+        line: null,
+        created_at: new Date(id * 1000).toISOString(),
+        updated_at: new Date(id * 1000).toISOString(),
+        html_url: `https://github.com/owner/repo/pull/1#discussion_r${id}`,
+    };
+}
+beforeEach(() => {
+    vi.resetAllMocks();
+    githubCLIService.clearCaches();
+});
+describe('review comment authors', () => {
+    it('accepts an explicitly null user from a deleted account', () => {
+        expect(reviewCommentSchema.parse(comment(1, null)).user).toBeNull();
+    });
+    it.each([undefined, {}, { login: null }, { login: 123 }])('still rejects malformed users: %j', (user) => {
+        expect(reviewCommentSchema.safeParse({ ...comment(1), user }).success).toBe(false);
+    });
+    it('preserves feedback and named attribution across pages containing deleted authors', async () => {
+        const firstPage = Array.from({ length: 100 }, (_, index) => comment(101 - index));
+        firstPage[0] = comment(101, { login: 'named-reviewer' });
+        firstPage[99] = comment(2, null);
+        vi.mocked(execFile)
+            .mockResolvedValueOnce({ stdout: JSON.stringify(firstPage), stderr: '' })
+            .mockResolvedValueOnce({ stdout: JSON.stringify([comment(1)]), stderr: '' });
+        const result = await githubCLIService.getReviewComments('owner/repo', 1);
+        expect(result).toHaveLength(101);
+        expect(result.map((entry) => entry.id)).toEqual(Array.from({ length: 101 }, (_, index) => index + 1));
+        expect(result[0]?.author.login).toBe('reviewer');
+        expect(result.at(-1)?.author.login).toBe('named-reviewer');
+        expect(result[1]).toEqual({
+            id: 2,
+            author: { login: '' },
+            body: 'Review 2',
+            path: 'src/index.ts',
+            line: null,
+            createdAt: new Date(2000).toISOString(),
+            updatedAt: new Date(2000).toISOString(),
+            url: 'https://github.com/owner/repo/pull/1#discussion_r2',
+        });
+    });
+});
+//# sourceMappingURL=github-cli-review-authors.test.js.map

package/dist/src/backend/services/github/service/github-cli-review-authors.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"github-cli-review-authors.test.js","sourceRoot":"","sources":["../../../../../../src/backend/services/github/service/github-cli-review-authors.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,oBAAoB,CAAC;AAC9C,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAE9D,EAAE,CAAC,IAAI,CAAC,oBAAoB,EAAE,GAAG,EAAE,CAAC,CAAC,EAAE,QAAQ,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,CAAC;AAC7D,EAAE,CAAC,IAAI,CAAC,WAAW,EAAE,GAAG,EAAE,CAAC,CAAC,EAAE,SAAS,EAAE,CAAC,EAAW,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC;AACjE,EAAE,CAAC,IAAI,CAAC,mCAAmC,EAAE,GAAG,EAAE,CAAC,CAAC;IAClD,YAAY,EAAE,GAAG,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,KAAK,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,KAAK,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC;CACvF,CAAC,CAAC,CAAC;AAEJ,OAAO,EAAE,gBAAgB,EAAE,MAAM,sBAAsB,CAAC;AACxD,OAAO,EAAE,mBAAmB,EAAE,MAAM,sBAAsB,CAAC;AAE3D,SAAS,OAAO,CAAC,EAAU,EAAE,IAAI,GAA6B,EAAE,KAAK,EAAE,UAAU,EAAE;IACjF,OAAO;QACL,EAAE;QACF,IAAI;QACJ,IAAI,EAAE,UAAU,EAAE,EAAE;QACpB,IAAI,EAAE,cAAc;QACpB,IAAI,EAAE,IAAI;QACV,UAAU,EAAE,IAAI,IAAI,CAAC,EAAE,GAAG,IAAI,CAAC,CAAC,WAAW,EAAE;QAC7C,UAAU,EAAE,IAAI,IAAI,CAAC,EAAE,GAAG,IAAI,CAAC,CAAC,WAAW,EAAE;QAC7C,QAAQ,EAAE,oDAAoD,EAAE,EAAE;KACnE,CAAC;AACJ,CAAC;AAED,UAAU,CAAC,GAAG,EAAE;IACd,EAAE,CAAC,aAAa,EAAE,CAAC;IACnB,gBAAgB,CAAC,WAAW,EAAE,CAAC;AACjC,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,wBAAwB,EAAE,GAAG,EAAE;IACtC,EAAE,CAAC,wDAAwD,EAAE,GAAG,EAAE;QAChE,MAAM,CAAC,mBAAmB,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,QAAQ,EAAE,CAAC;IACtE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,IAAI,CAAC,CAAC,SAAS,EAAE,EAAE,EAAE,EAAE,KAAK,EAAE,IAAI,EAAE,EAAE,EAAE,KAAK,EAAE,GAAG,EAAE,CAAC,CAAC,CACvD,mCAAmC,EACnC,CAAC,IAAI,EAAE,EAAE;QACP,MAAM,CAAC,mBAAmB,CAAC,SAAS,CAAC,EAAE,GAAG,OAAO,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACrF,CAAC,CACF,CAAC;IAEF,EAAE,CAAC,kFAAkF,EAAE,KAAK,IAAI,EAAE;QAChG,MAAM,SAAS,GAAG,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,GAAG,EAAE,EAAE,CAAC,CAAC,EAAE,KAAK,EAAE,EAAE,CAAC,OAAO,CAAC,GAAG,GAAG,KAAK,CAAC,CAAC,CAAC;QAClF,SAAS,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,GAAG,EAAE,EAAE,KAAK,EAAE,gBAAgB,EAAE,CAAC,CAAC;QACzD,SAAS,CAAC,EAAE,CAAC,GAAG,OAAO,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC;QACjC,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC;aAChB,qBAAqB,CAAC,EAAE,MAAM,EAAE,IAAI,CAAC,SAAS,CAAC,SAAS,CAAC,EAAE,MAAM,EAAE,EAAE,EAAW,CAAC;aACjF,qBAAqB,CAAC,EAAE,MAAM,EAAE,IAAI,CAAC,SAAS,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,MAAM,EAAE,EAAE,EAAW,CAAC,CAAC;QAExF,MAAM,MAAM,GAAG,MAAM,gBAAgB,CAAC,iBAAiB,CAAC,YAAY,EAAE,CAAC,CAAC,CAAC;QAEzE,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC;QACjC,MAAM,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,CAC7C,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,GAAG,EAAE,EAAE,CAAC,CAAC,EAAE,KAAK,EAAE,EAAE,CAAC,KAAK,GAAG,CAAC,CAAC,CACrD,CAAC;QACF,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QACjD,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,EAAE,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC;QAC3D,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC;YACxB,EAAE,EAAE,CAAC;YACL,MAAM,EAAE,EAAE,KAAK,EAAE,EAAE,EAAE;YACrB,IAAI,EAAE,UAAU;YAChB,IAAI,EAAE,cAAc;YACpB,IAAI,EAAE,IAAI;YACV,SAAS,EAAE,IAAI,IAAI,CAAC,IAAI,CAAC,CAAC,WAAW,EAAE;YACvC,SAAS,EAAE,IAAI,IAAI,CAAC,IAAI,CAAC,CAAC,WAAW,EAAE;YACvC,GAAG,EAAE,oDAAoD;SAC1D,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

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

@@ -31,6 +31,9 @@ declare class GitHubCLIService {
         prReviewState: string | null;
         prCiStatus: CIStatus;
         headRefName: string | null;
+        title?: string | null;
+        baseRefName?: string | null;
+        prHasMergeConflict?: boolean;
     } | null>;
     listReviewRequests(): Promise<ReviewRequestedPR[]>;
     listOpenPRs(owner: string, repo: string): Promise<OpenPullRequest[]>;
@@ -42,7 +45,7 @@ declare class GitHubCLIService {
         assignee?: string;
     }): Promise<GitHubIssue[]>;
     private fetchIssues;
-    getReviewComments(repo: string, prNumber: number, since?: Date, signal?: AbortSignal): Promise<Array<{
+    getReviewComments(repo: string, prNumber: number, since?: Date, signal?: AbortSignal, onIncomplete?: () => void): Promise<Array<{
         id: number;
         author: {
             login: string;
@@ -54,7 +57,7 @@ declare class GitHubCLIService {
         updatedAt: string;
         url: string;
     }>>;
-    getResolvedReviewCommentIds(repo: string, prNumber: number, signal?: AbortSignal): Promise<Set<number>>;
+    getResolvedReviewCommentIds(repo: string, prNumber: number, signal?: AbortSignal, onIncomplete?: () => void): Promise<Set<number>>;
     private collectResolvedIdsFromThreadPages;
     private fetchReviewThreadsPage;
     private collectResolvedThreadCommentTail;

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

@@ -1 +1 @@
-{"version":3,"file":"github-cli.service.d.ts","sourceRoot":"","sources":["../../../../../../src/backend/services/github/service/github-cli.service.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,QAAQ,EAAE,OAAO,EAAE,MAAM,eAAe,CAAC;AACvD,OAAO,KAAK,EAAE,iBAAiB,EAAE,YAAY,EAAE,MAAM,uBAAuB,CAAC;AAuB7E,OAAO,KAAK,EACV,kBAAkB,EAClB,qBAAqB,EACrB,WAAW,EACX,eAAe,EACf,MAAM,EACN,kBAAkB,EAClB,iBAAiB,EAClB,MAAM,oBAAoB,CAAC;AAqD5B,cAAM,gBAAgB;IACpB,OAAO,CAAC,QAAQ,CAAC,SAAS,CAA0B;IACpD,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAA0C;IAGnE,OAAO,CAAC,YAAY,CAAqE;IACzF,OAAO,CAAC,qBAAqB,CAA+C;IAC5E,OAAO,CAAC,QAAQ,CAAC,mBAAmB,CAAU;IAE9C,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAmE;IAC9F,OAAO,CAAC,QAAQ,CAAC,oBAAoB,CAAqB;IAC1D,OAAO,CAAC,QAAQ,CAAC,kBAAkB,CAAU;IAG7C,OAAO,CAAC,gBAAgB,CAAuB;IAG/C,WAAW,IAAI,IAAI,CAMlB;IASD,OAAO,CAAC,IAAI;IA+CZ,OAAO,CAAC,YAAY;IA6Bd,wBAAwB,CAAC,MAAM,CAAC,EAAE,WAAW,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAW3E;IAGK,WAAW,CAAC,YAAY,UAAQ,GAAG,OAAO,CAAC,qBAAqB,CAAC,CAwBtE;YAEa,cAAc;IAqC5B,aAAa,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAW1C;IAKK,WAAW,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC,kBAAkB,GAAG,IAAI,CAAC,CAgCnE;IAED,eAAe,CAAC,iBAAiB,EAAE,kBAAkB,CAAC,mBAAmB,CAAC,YAEzE;IAED,cAAc,CAAC,MAAM,EAAE;QAAE,KAAK,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,OAAO,CAAC;QAAC,cAAc,EAAE,MAAM,GAAG,IAAI,CAAA;KAAE,WAExF;IAMK,sBAAsB,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC;QACnD,OAAO,EAAE,OAAO,CAAC;QACjB,QAAQ,EAAE,MAAM,CAAC;QACjB,aAAa,EAAE,MAAM,GAAG,IAAI,CAAC;QAC7B,UAAU,EAAE,QAAQ,CAAC;QACrB,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;KAC5B,GAAG,IAAI,CAAC,CAaR;IAMK,kBAAkB,IAAI,OAAO,CAAC,iBAAiB,EAAE,CAAC,CAwEvD;IAGK,WAAW,CAAC,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,eAAe,EAAE,CAAC,CA8CzE;IAKK,gBAAgB,CACpB,IAAI,EAAE,MAAM,EACZ,QAAQ,EAAE,MAAM,EAChB,MAAM,CAAC,EAAE,WAAW,GACnB,OAAO,CAAC,iBAAiB,CAAC,CAwE5B;IAKK,SAAS,CAAC,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAqB/D;IAKK,YAAY,CAChB,IAAI,EAAE,MAAM,EACZ,QAAQ,EAAE,MAAM,EAChB,MAAM,EAAE,YAAY,EACpB,IAAI,CAAC,EAAE,MAAM,GACZ,OAAO,CAAC,IAAI,CAAC,CA8Bf;IAOK,UAAU,CACd,KAAK,EAAE,MAAM,EACb,IAAI,EAAE,MAAM,EACZ,OAAO,GAAE;QAAE,KAAK,CAAC,EAAE,MAAM,CAAC;QAAC,QAAQ,CAAC,EAAE,MAAM,CAAA;KAAO,GAClD,OAAO,CAAC,WAAW,EAAE,CAAC,CA4BxB;YAEa,WAAW;IA8CnB,iBAAiB,CACrB,IAAI,EAAE,MAAM,EACZ,QAAQ,EAAE,MAAM,EAChB,KAAK,CAAC,EAAE,IAAI,EACZ,MAAM,CAAC,EAAE,WAAW,GACnB,OAAO,CACR,KAAK,CAAC;QACJ,EAAE,EAAE,MAAM,CAAC;QACX,MAAM,EAAE;YAAE,KAAK,EAAE,MAAM,CAAA;SAAE,CAAC;QAC1B,IAAI,EAAE,MAAM,CAAC;QACb,IAAI,EAAE,MAAM,CAAC;QACb,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;QACpB,SAAS,EAAE,MAAM,CAAC;QAClB,SAAS,EAAE,MAAM,CAAC;QAClB,GAAG,EAAE,MAAM,CAAC;KACb,CAAC,CACH,CA6EA;IAOK,2BAA2B,CAC/B,IAAI,EAAE,MAAM,EACZ,QAAQ,EAAE,MAAM,EAChB,MAAM,CAAC,EAAE,WAAW,GACnB,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,CA8BtB;YAEa,iCAAiC;YAgDjC,sBAAsB;YA+CtB,gCAAgC;YA8ChC,uBAAuB;IAoC/B,YAAY,CAAC,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAmB9E;IAKK,eAAe,CACnB,KAAK,EAAE,MAAM,EACb,IAAI,EAAE,MAAM,EACZ,WAAW,EAAE,MAAM,EACnB,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,IAAI,CAAC,CAqBf;IAKK,QAAQ,CAAC,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,WAAW,GAAG,IAAI,CAAC,CA8B5F;CACF;AAED,YAAY,EACV,kBAAkB,EAClB,qBAAqB,EACrB,WAAW,EACX,eAAe,EACf,MAAM,EACN,kBAAkB,EAClB,iBAAiB,GAClB,CAAC;AAEF,eAAO,MAAM,gBAAgB,kBAAyB,CAAC"}
+{"version":3,"file":"github-cli.service.d.ts","sourceRoot":"","sources":["../../../../../../src/backend/services/github/service/github-cli.service.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,QAAQ,EAAE,OAAO,EAAE,MAAM,eAAe,CAAC;AACvD,OAAO,KAAK,EAAE,iBAAiB,EAAE,YAAY,EAAE,MAAM,uBAAuB,CAAC;AA2B7E,OAAO,KAAK,EACV,kBAAkB,EAClB,qBAAqB,EACrB,WAAW,EACX,eAAe,EACf,MAAM,EACN,kBAAkB,EAClB,iBAAiB,EAClB,MAAM,oBAAoB,CAAC;AAmB5B,cAAM,gBAAgB;IACpB,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAe;IACzC,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAA0C;IAEnE,OAAO,CAAC,YAAY,CAAqE;IACzF,OAAO,CAAC,qBAAqB,CAA+C;IAC5E,OAAO,CAAC,QAAQ,CAAC,mBAAmB,CAAU;IAE9C,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAmE;IAC9F,OAAO,CAAC,QAAQ,CAAC,oBAAoB,CAAqB;IAC1D,OAAO,CAAC,QAAQ,CAAC,kBAAkB,CAAU;IAG7C,OAAO,CAAC,gBAAgB,CAAuB;IAE/C,WAAW,IAAI,IAAI,CAMlB;IAED,OAAO,CAAC,IAAI;IA+CZ,OAAO,CAAC,YAAY;IA6Bd,wBAAwB,CAAC,MAAM,CAAC,EAAE,WAAW,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAW3E;IAGK,WAAW,CAAC,YAAY,UAAQ,GAAG,OAAO,CAAC,qBAAqB,CAAC,CAwBtE;YAEa,cAAc;IAqC5B,aAAa,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAW1C;IAKK,WAAW,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC,kBAAkB,GAAG,IAAI,CAAC,CAgCnE;IAED,eAAe,CAAC,iBAAiB,EAAE,kBAAkB,CAAC,mBAAmB,CAAC,YAEzE;IAED,cAAc,CAAC,MAAM,EAAE;QAAE,KAAK,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,OAAO,CAAC;QAAC,cAAc,EAAE,MAAM,GAAG,IAAI,CAAA;KAAE,WAExF;IAMK,sBAAsB,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC;QACnD,OAAO,EAAE,OAAO,CAAC;QACjB,QAAQ,EAAE,MAAM,CAAC;QACjB,aAAa,EAAE,MAAM,GAAG,IAAI,CAAC;QAC7B,UAAU,EAAE,QAAQ,CAAC;QACrB,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;QAC3B,KAAK,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;QACtB,WAAW,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;QAC5B,kBAAkB,CAAC,EAAE,OAAO,CAAC;KAC9B,GAAG,IAAI,CAAC,CAgBR;IAMK,kBAAkB,IAAI,OAAO,CAAC,iBAAiB,EAAE,CAAC,CAwEvD;IAGK,WAAW,CAAC,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,eAAe,EAAE,CAAC,CA8CzE;IAKK,gBAAgB,CACpB,IAAI,EAAE,MAAM,EACZ,QAAQ,EAAE,MAAM,EAChB,MAAM,CAAC,EAAE,WAAW,GACnB,OAAO,CAAC,iBAAiB,CAAC,CA0E5B;IAKK,SAAS,CAAC,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAqB/D;IAKK,YAAY,CAChB,IAAI,EAAE,MAAM,EACZ,QAAQ,EAAE,MAAM,EAChB,MAAM,EAAE,YAAY,EACpB,IAAI,CAAC,EAAE,MAAM,GACZ,OAAO,CAAC,IAAI,CAAC,CA8Bf;IAOK,UAAU,CACd,KAAK,EAAE,MAAM,EACb,IAAI,EAAE,MAAM,EACZ,OAAO,GAAE;QAAE,KAAK,CAAC,EAAE,MAAM,CAAC;QAAC,QAAQ,CAAC,EAAE,MAAM,CAAA;KAAO,GAClD,OAAO,CAAC,WAAW,EAAE,CAAC,CA4BxB;YAEa,WAAW;IA8CnB,iBAAiB,CACrB,IAAI,EAAE,MAAM,EACZ,QAAQ,EAAE,MAAM,EAChB,KAAK,CAAC,EAAE,IAAI,EACZ,MAAM,CAAC,EAAE,WAAW,EACpB,YAAY,CAAC,EAAE,MAAM,IAAI,GACxB,OAAO,CACR,KAAK,CAAC;QACJ,EAAE,EAAE,MAAM,CAAC;QACX,MAAM,EAAE;YAAE,KAAK,EAAE,MAAM,CAAA;SAAE,CAAC;QAC1B,IAAI,EAAE,MAAM,CAAC;QACb,IAAI,EAAE,MAAM,CAAC;QACb,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;QACpB,SAAS,EAAE,MAAM,CAAC;QAClB,SAAS,EAAE,MAAM,CAAC;QAClB,GAAG,EAAE,MAAM,CAAC;KACb,CAAC,CACH,CA8EA;IAOK,2BAA2B,CAC/B,IAAI,EAAE,MAAM,EACZ,QAAQ,EAAE,MAAM,EAChB,MAAM,CAAC,EAAE,WAAW,EACpB,YAAY,CAAC,EAAE,MAAM,IAAI,GACxB,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,CA+BtB;YAEa,iCAAiC;YA0DjC,sBAAsB;YA+CtB,gCAAgC;YAkDhC,uBAAuB;IAoC/B,YAAY,CAAC,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAmB9E;IAKK,eAAe,CACnB,KAAK,EAAE,MAAM,EACb,IAAI,EAAE,MAAM,EACZ,WAAW,EAAE,MAAM,EACnB,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,IAAI,CAAC,CAqBf;IAKK,QAAQ,CAAC,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,WAAW,GAAG,IAAI,CAAC,CA8B5F;CACF;AAED,YAAY,EACV,kBAAkB,EAClB,qBAAqB,EACrB,WAAW,EACX,eAAe,EACf,MAAM,EACN,kBAAkB,EAClB,iBAAiB,GAClB,CAAC;AAEF,eAAO,MAAM,gBAAgB,kBAAyB,CAAC"}

package/dist/src/backend/services/github/service/github-cli.service.js

@@ -1,38 +1,19 @@
 import { execFile } from 'node:child_process';
 import { promisify } from 'node:util';
-import pLimit from 'p-limit';
 import { createLogger } from '../../logger.service.js';
 import { isRateLimitMessage } from '../../rate-limit-backoff.js';
-import { GH_CONCURRENCY, GH_MAX_BUFFER_BYTES, GH_TIMEOUT_MS } from './github-cli/constants.js';
+import { GH_MAX_BUFFER_BYTES, GH_TIMEOUT_MS, ghExecLimit } from './github-cli/constants.js';
 import { classifyError, logGitHubCLIError } from './github-cli/errors.js';
-import { computeCIStatus, computePRState, mapComments, mapLabels, mapReviews, mapStatusChecks, } from './github-cli/mappers.js';
+import { computeCIStatus, computePRState, mapComments, mapLabels, mapStatusChecks, } from './github-cli/mappers.js';
+import { collectResolvedReviewCommentIds, } from './github-cli/resolved-review-comments.js';
+import { getChronologicalReviews } from './github-cli/reviews.js';
 import { fullPRDetailsSchema, issueSchema, openPullRequestsGraphQLSchema, prStatusSchema, resolvedReviewThreadsGraphQLSchema, reviewCommentSchema, reviewRequestedPRGraphQLSchema, reviewThreadCommentsGraphQLSchema, } from './github-cli/schemas.js';
 import { parseGhJson } from './github-cli/utils.js';
 const execFileAsync = promisify(execFile);
 const logger = createLogger('github-cli');
-function collectResolvedReviewCommentIds(threads, resolvedIds) {
-    const truncatedThreads = [];
-    for (const thread of threads) {
-        if (!thread.isResolved) {
-            continue;
-        }
-        for (const comment of thread.comments.nodes) {
-            if (comment.fullDatabaseId !== null) {
-                resolvedIds.add(comment.fullDatabaseId);
-            }
-        }
-        if (thread.comments.pageInfo.hasNextPage) {
-            truncatedThreads.push({
-                threadId: thread.id,
-                afterCursor: thread.comments.pageInfo.endCursor,
-            });
-        }
-    }
-    return truncatedThreads;
-}
 const RATE_LIMIT_FAST_FAIL_MS = 60_000;
 class GitHubCLIService {
-    execLimit = pLimit(GH_CONCURRENCY);
+    execLimit = ghExecLimit;
     inflight = new Map();
     cachedHealth = null;
     healthRefreshInFlight = null;
@@ -190,7 +171,7 @@ class GitHubCLIService {
                 '--repo',
                 `${prInfo.owner}/${prInfo.repo}`,
                 '--json',
-                'number,state,isDraft,reviewDecision,statusCheckRollup,headRefName',
+                'number,state,isDraft,reviewDecision,statusCheckRollup,headRefName,title,baseRefName,mergeStateStatus',
             ], { timeout: GH_TIMEOUT_MS.default });
             return parseGhJson(prStatusSchema, stdout, 'getPRStatus');
         }
@@ -221,6 +202,9 @@ class GitHubCLIService {
             prReviewState: status.reviewDecision,
             prCiStatus: this.computeCIStatus(status.statusCheckRollup),
             headRefName: status.headRefName ?? null,
+            title: status.title ?? null,
+            baseRefName: status.baseRefName ?? null,
+            prHasMergeConflict: status.mergeStateStatus === 'DIRTY',
         };
     }
     async listReviewRequests() {
@@ -342,27 +326,27 @@ class GitHubCLIService {
             'state',
             'reviewDecision',
             'statusCheckRollup',
-            'reviews',
             'comments',
             'labels',
             'additions',
             'deletions',
             'changedFiles',
             'headRefName',
+            'headRefOid',
             'baseRefName',
             'mergeStateStatus',
         ].join(',');
         try {
-            const { stdout } = await this.exec(['pr', 'view', String(prNumber), '--repo', repo, '--json', fields], { timeout: GH_TIMEOUT_MS.default, signal });
+            const { stdout } = await this.exec(['pr', 'view', String(prNumber), '--repo', repo, '--json', fields], { timeout: GH_TIMEOUT_MS.default, maxBuffer: GH_MAX_BUFFER_BYTES.fullPRDetails, signal });
             const data = parseGhJson(fullPRDetailsSchema, stdout, 'getPRFullDetails');
-            const [, repoName] = repo.split('/');
+            const reviews = await getChronologicalReviews(repo, prNumber, (args, options) => this.exec(args, { ...options, timeout: GH_TIMEOUT_MS.default, signal }));
             return {
                 number: data.number,
                 title: data.title,
                 url: data.url,
                 author: data.author,
                 repository: {
-                    name: repoName,
+                    name: repo.split('/')[1] ?? '',
                     nameWithOwner: repo,
                 },
                 createdAt: data.createdAt,
@@ -371,12 +355,13 @@ class GitHubCLIService {
                 state: data.state,
                 reviewDecision: data.reviewDecision,
                 statusCheckRollup: data.statusCheckRollup ? mapStatusChecks(data.statusCheckRollup) : null,
-                reviews: mapReviews(data.reviews),
+                reviews,
                 comments: mapComments(data.comments),
                 labels: mapLabels(data.labels),
                 additions: data.additions || 0,
                 deletions: data.deletions || 0,
                 changedFiles: data.changedFiles || 0,
+                headRefOid: data.headRefOid,
                 headRefName: data.headRefName || '',
                 baseRefName: data.baseRefName || '',
                 mergeStateStatus: data.mergeStateStatus || 'UNKNOWN',
@@ -500,7 +485,7 @@ class GitHubCLIService {
             throw new Error(`Failed to list issues: ${errorMessage}`);
         }
     }
-    async getReviewComments(repo, prNumber, since, signal) {
+    async getReviewComments(repo, prNumber, since, signal, onIncomplete) {
         try {
             const PAGE_SIZE = 100;
             const MAX_PAGES = 20;
@@ -522,7 +507,7 @@ class GitHubCLIService {
                 for (const comment of pageComments) {
                     allComments.push({
                         id: comment.id,
-                        author: { login: comment.user.login },
+                        author: { login: comment.user?.login ?? '' },
                         body: comment.body,
                         path: comment.path,
                         line: comment.line,
@@ -535,6 +520,7 @@ class GitHubCLIService {
                     break;
                 }
                 if (page === MAX_PAGES) {
+                    onIncomplete?.();
                     logger.warn('getReviewComments: reached MAX_PAGES limit, results may be incomplete', {
                         repo,
                         prNumber,
@@ -558,7 +544,7 @@ class GitHubCLIService {
             throw new Error(`Failed to fetch PR review comments: ${errorMessage}`);
         }
     }
-    async getResolvedReviewCommentIds(repo, prNumber, signal) {
+    async getResolvedReviewCommentIds(repo, prNumber, signal, onIncomplete) {
         const [owner, name] = repo.split('/');
         if (!(owner && name)) {
             throw new Error(`Invalid repo format for getResolvedReviewCommentIds: ${repo}`);
@@ -566,7 +552,7 @@ class GitHubCLIService {
         try {
             signal?.throwIfAborted();
             const resolvedIds = new Set();
-            await this.collectResolvedIdsFromThreadPages({ owner, name, repo, prNumber }, resolvedIds, signal);
+            await this.collectResolvedIdsFromThreadPages({ owner, name, repo, prNumber }, resolvedIds, signal, onIncomplete);
             signal?.throwIfAborted();
             return resolvedIds;
         }
@@ -583,7 +569,7 @@ class GitHubCLIService {
             throw new Error(`Failed to fetch resolved review threads: ${errorMessage}`);
         }
     }
-    async collectResolvedIdsFromThreadPages(ctx, resolvedIds, signal) {
+    async collectResolvedIdsFromThreadPages(ctx, resolvedIds, signal, onIncomplete) {
         const MAX_PAGES = 20;
         const logContext = { repo: ctx.repo, prNumber: ctx.prNumber };
         let afterCursor = null;
@@ -591,22 +577,25 @@ class GitHubCLIService {
             signal?.throwIfAborted();
             const reviewThreads = await this.fetchReviewThreadsPage(ctx.owner, ctx.name, ctx.prNumber, afterCursor, signal);
             if (!reviewThreads) {
+                onIncomplete?.();
                 logger.warn('getResolvedReviewCommentIds: repository or PR not found', logContext);
                 return;
             }
             const truncatedThreads = collectResolvedReviewCommentIds(reviewThreads.nodes, resolvedIds);
             for (const thread of truncatedThreads) {
-                await this.collectResolvedThreadCommentTail(thread, resolvedIds, logContext, signal);
+                await this.collectResolvedThreadCommentTail(thread, resolvedIds, logContext, signal, onIncomplete);
             }
             if (!reviewThreads.pageInfo.hasNextPage) {
                 return;
             }
             afterCursor = reviewThreads.pageInfo.endCursor;
             if (!afterCursor) {
+                onIncomplete?.();
                 logger.warn('getResolvedReviewCommentIds: review thread page is missing an end cursor', logContext);
                 return;
             }
         }
+        onIncomplete?.();
         logger.warn('getResolvedReviewCommentIds: reached MAX_PAGES limit, results may be incomplete', {
             ...logContext,
             totalResolved: resolvedIds.size,
@@ -643,17 +632,19 @@ class GitHubCLIService {
         const parsed = parseGhJson(resolvedReviewThreadsGraphQLSchema, stdout, 'getResolvedReviewCommentIds');
         return parsed.data.repository?.pullRequest?.reviewThreads ?? null;
     }
-    async collectResolvedThreadCommentTail(thread, resolvedIds, logContext, signal) {
+    async collectResolvedThreadCommentTail(thread, resolvedIds, logContext, signal, onIncomplete) {
         const MAX_PAGES = 20;
         let afterCursor = thread.afterCursor;
         for (let page = 1; page <= MAX_PAGES; page++) {
             signal?.throwIfAborted();
             if (!afterCursor) {
+                onIncomplete?.();
                 logger.warn('getResolvedReviewCommentIds: truncated thread comments are missing an end cursor', logContext);
                 return;
             }
             const comments = await this.fetchThreadCommentsPage(thread.threadId, afterCursor, signal);
             if (!comments) {
+                onIncomplete?.();
                 logger.warn('getResolvedReviewCommentIds: review thread not found while paging comments', {
                     ...logContext,
                     threadId: thread.threadId,
@@ -670,6 +661,7 @@ class GitHubCLIService {
             }
             afterCursor = comments.pageInfo.endCursor;
         }
+        onIncomplete?.();
         logger.warn('getResolvedReviewCommentIds: reached MAX_PAGES limit while paging thread comments', { ...logContext, threadId: thread.threadId, maxPages: MAX_PAGES });
     }
     async fetchThreadCommentsPage(threadId, afterCursor, signal) {