ghagga 3.0.0 → 3.1.0

package/dist/lib/pr-postback.test.js

@@ -0,0 +1,196 @@
+/**
+ * R-CLI: the `--pr` post-back routes through the REAL GitHubForgeAdapter folding
+ * over the CLI's own GitHubClientPort. We mock ONLY `fetch` (the network seam),
+ * so this exercises the genuine adapter find→delete→post idempotency path and
+ * the genuine CLI port REST calls — a faithful simulation against a mocked
+ * GitHub.
+ */
+import { REVIEW_COMMENT_MARKER } from 'ghagga-core';
+import { GitHubForgeAdapter } from 'ghagga-forge';
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+import { createCliGitHubClientPort } from './cli-github-client-port.js';
+import { postSummaryComment } from './pr-postback.js';
+const mockFetch = vi.fn();
+vi.stubGlobal('fetch', mockFetch);
+function jsonResponse(status, body) {
+    return {
+        ok: status >= 200 && status < 300,
+        status,
+        statusText: status === 200 ? 'OK' : status === 201 ? 'Created' : 'ERR',
+        text: () => Promise.resolve(JSON.stringify(body)),
+        json: () => Promise.resolve(body),
+    };
+}
+const MARKER = { html: REVIEW_COMMENT_MARKER };
+const REF = {
+    repo: { kind: 'github', nativeId: 'acme/widgets', path: 'acme/widgets' },
+    iid: 42,
+};
+function buildAdapter(token = 'tok') {
+    return new GitHubForgeAdapter({
+        client: createCliGitHubClientPort(),
+        token,
+        owner: 'acme',
+        repo: 'widgets',
+    });
+}
+describe('postSummaryComment via CLI port + GitHubForgeAdapter', () => {
+    beforeEach(() => mockFetch.mockReset());
+    it('posts a fresh comment when none exists (no stale to delete)', async () => {
+        // findExistingComment: empty list → null
+        mockFetch.mockResolvedValueOnce(jsonResponse(200, []));
+        // postComment → { id }
+        mockFetch.mockResolvedValueOnce(jsonResponse(201, { id: 9001 }));
+        const adapter = buildAdapter();
+        const body = `${REVIEW_COMMENT_MARKER}\n## body`;
+        const result = await postSummaryComment(adapter, REF, body, MARKER);
+        expect(result.createdNativeId).toBe(9001);
+        expect(result.deletedNativeIds).toEqual([]);
+        // 1 list call + 1 post call.
+        expect(mockFetch).toHaveBeenCalledTimes(2);
+        const [listUrl] = mockFetch.mock.calls[0];
+        expect(listUrl).toContain('/repos/acme/widgets/issues/42/comments');
+        const [postUrl, postInit] = mockFetch.mock.calls[1];
+        expect(postUrl).toBe('https://api.github.com/repos/acme/widgets/issues/42/comments');
+        expect(postInit.method).toBe('POST');
+        expect(JSON.parse(postInit.body)).toEqual({ body });
+    });
+    it('finds stale → deletes ALL → reposts fresh (idempotent upsert + marker)', async () => {
+        // list returns two GHAGGA comments (with marker) + one foreign comment
+        mockFetch.mockResolvedValueOnce(jsonResponse(200, [
+            { id: 100, body: `old ${REVIEW_COMMENT_MARKER}` },
+            { id: 200, body: 'someone else' },
+            { id: 300, body: `newer ${REVIEW_COMMENT_MARKER}` },
+        ]));
+        // deleteComment(latest=300), deleteComment(stale=100)
+        mockFetch.mockResolvedValueOnce(jsonResponse(204, {}));
+        mockFetch.mockResolvedValueOnce(jsonResponse(204, {}));
+        // postComment fresh
+        mockFetch.mockResolvedValueOnce(jsonResponse(201, { id: 400 }));
+        const adapter = buildAdapter();
+        const result = await postSummaryComment(adapter, REF, 'body', MARKER);
+        expect(result.createdNativeId).toBe(400);
+        // delete order: latest first, then stale.
+        expect(result.deletedNativeIds).toEqual([300, 100]);
+        const delCall1 = mockFetch.mock.calls[1];
+        const delCall2 = mockFetch.mock.calls[2];
+        expect(delCall1[0]).toContain('/issues/comments/300');
+        expect(delCall1[1].method).toBe('DELETE');
+        expect(delCall2[0]).toContain('/issues/comments/100');
+        // foreign comment (200) is never deleted.
+        const allUrls = mockFetch.mock.calls.map((c) => c[0]).join('\n');
+        expect(allUrls).not.toContain('/issues/comments/200');
+    });
+    it('a 401 with the static token is fatal (no invalidate-retry)', async () => {
+        // findExistingComment → 401
+        mockFetch.mockResolvedValueOnce(jsonResponse(401, { message: 'bad creds' }));
+        const adapter = buildAdapter();
+        await expect(postSummaryComment(adapter, REF, 'body', MARKER)).rejects.toThrow();
+        // exactly one call — no retry loop.
+        expect(mockFetch).toHaveBeenCalledTimes(1);
+    });
+    it('passes an AbortSignal (timeout) on every fetch so CI cannot hang', async () => {
+        mockFetch.mockResolvedValueOnce(jsonResponse(200, []));
+        mockFetch.mockResolvedValueOnce(jsonResponse(201, { id: 1 }));
+        const adapter = buildAdapter();
+        await postSummaryComment(adapter, REF, 'body', MARKER);
+        // Both the list (find) and the post fetch carry an abort signal.
+        for (const call of mockFetch.mock.calls) {
+            const init = call[1];
+            expect(init.signal).toBeInstanceOf(AbortSignal);
+        }
+    });
+    it('sends Bearer auth + version headers on the post', async () => {
+        mockFetch.mockResolvedValueOnce(jsonResponse(200, []));
+        mockFetch.mockResolvedValueOnce(jsonResponse(201, { id: 1 }));
+        const adapter = buildAdapter('secret-token');
+        await postSummaryComment(adapter, REF, 'body', MARKER);
+        const [, postInit] = mockFetch.mock.calls[1];
+        const headers = postInit.headers;
+        expect(headers.Authorization).toBe('Bearer secret-token');
+        expect(headers['X-GitHub-Api-Version']).toBe('2022-11-28');
+    });
+});
+describe('CLI GitHub port — findExistingComment pagination (backlog #6)', () => {
+    beforeEach(() => mockFetch.mockReset());
+    function makeComments(count, startId = 0, markerAt) {
+        return Array.from({ length: count }, (_, i) => ({
+            id: startId + i,
+            body: i === markerAt ? `summary ${startId + i} ${REVIEW_COMMENT_MARKER}` : `noise ${startId + i}`,
+        }));
+    }
+    it('single page (< 100 comments) → exactly one list call, no extra fetch', async () => {
+        const port = createCliGitHubClientPort();
+        mockFetch.mockResolvedValueOnce(jsonResponse(200, makeComments(20, 0)));
+        const result = await port.findExistingComment('acme', 'widgets', 42, 'tok');
+        expect(result).toBeNull();
+        expect(mockFetch).toHaveBeenCalledTimes(1);
+        expect(mockFetch.mock.calls[0][0]).toContain('page=1');
+    });
+    it('multi-page → full page 1 forces page 2 fetch; stale-on-page-2 found + deleted (no duplicate)', async () => {
+        // page 1: 100 comments, NO marker → must fetch page 2; page 2 carries the stale marker.
+        mockFetch.mockResolvedValueOnce(jsonResponse(200, makeComments(100, 0)));
+        mockFetch.mockResolvedValueOnce(jsonResponse(200, makeComments(15, 100, 5))); // marker id 105
+        // adapter deletes the found stale (105) then reposts fresh.
+        mockFetch.mockResolvedValueOnce(jsonResponse(204, {})); // delete 105
+        mockFetch.mockResolvedValueOnce(jsonResponse(201, { id: 777 })); // repost
+        const adapter = buildAdapter();
+        const result = await postSummaryComment(adapter, REF, 'body', MARKER);
+        expect(result.deletedNativeIds).toEqual([105]);
+        expect(result.createdNativeId).toBe(777);
+        expect(mockFetch.mock.calls[0][0]).toContain('page=1');
+        expect(mockFetch.mock.calls[1][0]).toContain('page=2');
+        const delCall = mockFetch.mock.calls[2];
+        expect(delCall[0]).toContain('/issues/comments/105');
+        expect(delCall[1].method).toBe('DELETE');
+    });
+    it('safety bound → stops at MAX_PAGES (50) and warns instead of looping forever', async () => {
+        for (let page = 0; page < 50; page++) {
+            mockFetch.mockResolvedValueOnce(jsonResponse(200, makeComments(100, page * 100)));
+        }
+        const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => { });
+        const port = createCliGitHubClientPort();
+        const result = await port.findExistingComment('acme', 'widgets', 42, 'tok');
+        expect(result).toBeNull();
+        expect(mockFetch).toHaveBeenCalledTimes(50);
+        expect(mockFetch.mock.calls[49][0]).toContain('page=50');
+        expect(warnSpy).toHaveBeenCalledWith(expect.stringContaining('MAX_PAGES (50)'));
+        warnSpy.mockRestore();
+    });
+});
+describe('CLI port stub safety (read members never hit in --pr flow)', () => {
+    it('the --pr post-back never invokes any stubbed read member', async () => {
+        mockFetch.mockResolvedValueOnce(jsonResponse(200, []));
+        mockFetch.mockResolvedValueOnce(jsonResponse(201, { id: 5 }));
+        const port = createCliGitHubClientPort();
+        // Spy on the stubs — if the post-back path touched them they'd throw, but
+        // assert explicitly they were never called.
+        const fetchPRDiff = vi.spyOn(port, 'fetchPRDiff');
+        const getPRFileList = vi.spyOn(port, 'getPRFileList');
+        const getPRCommitMessages = vi.spyOn(port, 'getPRCommitMessages');
+        const fetchPRDetails = vi.spyOn(port, 'fetchPRDetails');
+        const addCommentReaction = vi.spyOn(port, 'addCommentReaction');
+        const fetchGraphFromBranch = vi.spyOn(port, 'fetchGraphFromBranch');
+        const fetchGraphMetadata = vi.spyOn(port, 'fetchGraphMetadata');
+        const adapter = new GitHubForgeAdapter({ client: port, token: 't', owner: 'a', repo: 'b' });
+        await postSummaryComment(adapter, REF, 'body', MARKER);
+        expect(fetchPRDiff).not.toHaveBeenCalled();
+        expect(getPRFileList).not.toHaveBeenCalled();
+        expect(getPRCommitMessages).not.toHaveBeenCalled();
+        expect(fetchPRDetails).not.toHaveBeenCalled();
+        expect(addCommentReaction).not.toHaveBeenCalled();
+        expect(fetchGraphFromBranch).not.toHaveBeenCalled();
+        expect(fetchGraphMetadata).not.toHaveBeenCalled();
+    });
+    it('each stubbed read member throws a clear "not supported in CLI" error', async () => {
+        const port = createCliGitHubClientPort();
+        await expect(port.fetchPRDiff('a', 'b', 1, 't')).rejects.toThrow(/not supported in CLI/);
+        await expect(port.fetchPRDetails('a', 'b', 1, 't')).rejects.toThrow(/not supported in CLI/);
+        await expect(port.getPRFileList('a', 'b', 1, 't')).rejects.toThrow(/not supported in CLI/);
+        await expect(port.getPRCommitMessages('a', 'b', 1, 't')).rejects.toThrow(/not supported in CLI/);
+        await expect(port.addCommentReaction('a', 'b', 1, '+1', 't')).rejects.toThrow(/not supported in CLI/);
+        await expect(port.fetchGraphFromBranch('a', 'b', 't')).rejects.toThrow(/not supported in CLI/);
+        await expect(port.fetchGraphMetadata('a', 'b', 't')).rejects.toThrow(/not supported in CLI/);
+    });
+});
+//# sourceMappingURL=pr-postback.test.js.map

package/dist/lib/pr-postback.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"pr-postback.test.js","sourceRoot":"","sources":["../../src/lib/pr-postback.test.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EAAE,qBAAqB,EAAE,MAAM,aAAa,CAAC;AACpD,OAAO,EAAE,kBAAkB,EAAE,MAAM,cAAc,CAAC;AAClD,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAC9D,OAAO,EAAE,yBAAyB,EAAE,MAAM,6BAA6B,CAAC;AACxE,OAAO,EAAE,kBAAkB,EAAE,MAAM,kBAAkB,CAAC;AAEtD,MAAM,SAAS,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;AAC1B,EAAE,CAAC,UAAU,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;AAElC,SAAS,YAAY,CAAC,MAAc,EAAE,IAAa;IACjD,OAAO;QACL,EAAE,EAAE,MAAM,IAAI,GAAG,IAAI,MAAM,GAAG,GAAG;QACjC,MAAM;QACN,UAAU,EAAE,MAAM,KAAK,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,MAAM,KAAK,GAAG,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,KAAK;QACtE,IAAI,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QACjD,IAAI,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC;KACtB,CAAC;AAChB,CAAC;AAED,MAAM,MAAM,GAAG,EAAE,IAAI,EAAE,qBAAqB,EAAE,CAAC;AAC/C,MAAM,GAAG,GAAG;IACV,IAAI,EAAE,EAAE,IAAI,EAAE,QAAiB,EAAE,QAAQ,EAAE,cAAc,EAAE,IAAI,EAAE,cAAc,EAAE;IACjF,GAAG,EAAE,EAAE;CACR,CAAC;AAEF,SAAS,YAAY,CAAC,KAAK,GAAG,KAAK;IACjC,OAAO,IAAI,kBAAkB,CAAC;QAC5B,MAAM,EAAE,yBAAyB,EAAE;QACnC,KAAK;QACL,KAAK,EAAE,MAAM;QACb,IAAI,EAAE,SAAS;KAChB,CAAC,CAAC;AACL,CAAC;AAED,QAAQ,CAAC,sDAAsD,EAAE,GAAG,EAAE;IACpE,UAAU,CAAC,GAAG,EAAE,CAAC,SAAS,CAAC,SAAS,EAAE,CAAC,CAAC;IAExC,EAAE,CAAC,6DAA6D,EAAE,KAAK,IAAI,EAAE;QAC3E,yCAAyC;QACzC,SAAS,CAAC,qBAAqB,CAAC,YAAY,CAAC,GAAG,EAAE,EAAE,CAAC,CAAC,CAAC;QACvD,uBAAuB;QACvB,SAAS,CAAC,qBAAqB,CAAC,YAAY,CAAC,GAAG,EAAE,EAAE,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC;QAEjE,MAAM,OAAO,GAAG,YAAY,EAAE,CAAC;QAC/B,MAAM,IAAI,GAAG,GAAG,qBAAqB,WAAW,CAAC;QACjD,MAAM,MAAM,GAAG,MAAM,kBAAkB,CAAC,OAAO,EAAE,GAAG,EAAE,IAAI,EAAE,MAAM,CAAC,CAAC;QAEpE,MAAM,CAAC,MAAM,CAAC,eAAe,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC1C,MAAM,CAAC,MAAM,CAAC,gBAAgB,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QAE5C,6BAA6B;QAC7B,MAAM,CAAC,SAAS,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QAC3C,MAAM,CAAC,OAAO,CAAC,GAAG,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAA0B,CAAC;QACnE,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,wCAAwC,CAAC,CAAC;QAEpE,MAAM,CAAC,OAAO,EAAE,QAAQ,CAAC,GAAG,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAA0B,CAAC;QAC7E,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,8DAA8D,CAAC,CAAC;QACrF,MAAM,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QACrC,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,IAAc,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC;IAChE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wEAAwE,EAAE,KAAK,IAAI,EAAE;QACtF,uEAAuE;QACvE,SAAS,CAAC,qBAAqB,CAC7B,YAAY,CAAC,GAAG,EAAE;YAChB,EAAE,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE,OAAO,qBAAqB,EAAE,EAAE;YACjD,EAAE,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE,cAAc,EAAE;YACjC,EAAE,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE,SAAS,qBAAqB,EAAE,EAAE;SACpD,CAAC,CACH,CAAC;QACF,sDAAsD;QACtD,SAAS,CAAC,qBAAqB,CAAC,YAAY,CAAC,GAAG,EAAE,EAAE,CAAC,CAAC,CAAC;QACvD,SAAS,CAAC,qBAAqB,CAAC,YAAY,CAAC,GAAG,EAAE,EAAE,CAAC,CAAC,CAAC;QACvD,oBAAoB;QACpB,SAAS,CAAC,qBAAqB,CAAC,YAAY,CAAC,GAAG,EAAE,EAAE,EAAE,EAAE,GAAG,EAAE,CAAC,CAAC,CAAC;QAEhE,MAAM,OAAO,GAAG,YAAY,EAAE,CAAC;QAC/B,MAAM,MAAM,GAAG,MAAM,kBAAkB,CAAC,OAAO,EAAE,GAAG,EAAE,MAAM,EAAE,MAAM,CAAC,CAAC;QAEtE,MAAM,CAAC,MAAM,CAAC,eAAe,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACzC,0CAA0C;QAC1C,MAAM,CAAC,MAAM,CAAC,gBAAgB,CAAC,CAAC,OAAO,CAAC,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC,CAAC;QAEpD,MAAM,QAAQ,GAAG,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAA0B,CAAC;QAClE,MAAM,QAAQ,GAAG,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAA0B,CAAC;QAClE,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,sBAAsB,CAAC,CAAC;QACtD,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QAC1C,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,sBAAsB,CAAC,CAAC;QACtD,0CAA0C;QAC1C,MAAM,OAAO,GAAG,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAW,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC3E,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,sBAAsB,CAAC,CAAC;IACxD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4DAA4D,EAAE,KAAK,IAAI,EAAE;QAC1E,4BAA4B;QAC5B,SAAS,CAAC,qBAAqB,CAAC,YAAY,CAAC,GAAG,EAAE,EAAE,OAAO,EAAE,WAAW,EAAE,CAAC,CAAC,CAAC;QAC7E,MAAM,OAAO,GAAG,YAAY,EAAE,CAAC;QAC/B,MAAM,MAAM,CAAC,kBAAkB,CAAC,OAAO,EAAE,GAAG,EAAE,MAAM,EAAE,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,EAAE,CAAC;QACjF,oCAAoC;QACpC,MAAM,CAAC,SAAS,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kEAAkE,EAAE,KAAK,IAAI,EAAE;QAChF,SAAS,CAAC,qBAAqB,CAAC,YAAY,CAAC,GAAG,EAAE,EAAE,CAAC,CAAC,CAAC;QACvD,SAAS,CAAC,qBAAqB,CAAC,YAAY,CAAC,GAAG,EAAE,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QAC9D,MAAM,OAAO,GAAG,YAAY,EAAE,CAAC;QAC/B,MAAM,kBAAkB,CAAC,OAAO,EAAE,GAAG,EAAE,MAAM,EAAE,MAAM,CAAC,CAAC;QACvD,iEAAiE;QACjE,KAAK,MAAM,IAAI,IAAI,SAAS,CAAC,IAAI,CAAC,KAAK,EAAE,CAAC;YACxC,MAAM,IAAI,GAAG,IAAI,CAAC,CAAC,CAAgB,CAAC;YACpC,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,cAAc,CAAC,WAAW,CAAC,CAAC;QAClD,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;QAC/D,SAAS,CAAC,qBAAqB,CAAC,YAAY,CAAC,GAAG,EAAE,EAAE,CAAC,CAAC,CAAC;QACvD,SAAS,CAAC,qBAAqB,CAAC,YAAY,CAAC,GAAG,EAAE,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QAC9D,MAAM,OAAO,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;QAC7C,MAAM,kBAAkB,CAAC,OAAO,EAAE,GAAG,EAAE,MAAM,EAAE,MAAM,CAAC,CAAC;QACvD,MAAM,CAAC,EAAE,QAAQ,CAAC,GAAG,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAA0B,CAAC;QACtE,MAAM,OAAO,GAAG,QAAQ,CAAC,OAAiC,CAAC;QAC3D,MAAM,CAAC,OAAO,CAAC,aAAa,CAAC,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAC;QAC1D,MAAM,CAAC,OAAO,CAAC,sBAAsB,CAAC,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;IAC7D,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,+DAA+D,EAAE,GAAG,EAAE;IAC7E,UAAU,CAAC,GAAG,EAAE,CAAC,SAAS,CAAC,SAAS,EAAE,CAAC,CAAC;IAExC,SAAS,YAAY,CAAC,KAAa,EAAE,OAAO,GAAG,CAAC,EAAE,QAAiB;QACjE,OAAO,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,KAAK,EAAE,EAAE,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;YAC9C,EAAE,EAAE,OAAO,GAAG,CAAC;YACf,IAAI,EACF,CAAC,KAAK,QAAQ,CAAC,CAAC,CAAC,WAAW,OAAO,GAAG,CAAC,IAAI,qBAAqB,EAAE,CAAC,CAAC,CAAC,SAAS,OAAO,GAAG,CAAC,EAAE;SAC9F,CAAC,CAAC,CAAC;IACN,CAAC;IAED,EAAE,CAAC,sEAAsE,EAAE,KAAK,IAAI,EAAE;QACpF,MAAM,IAAI,GAAG,yBAAyB,EAAE,CAAC;QACzC,SAAS,CAAC,qBAAqB,CAAC,YAAY,CAAC,GAAG,EAAE,YAAY,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC;QACxE,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,mBAAmB,CAAC,MAAM,EAAE,SAAS,EAAE,EAAE,EAAE,KAAK,CAAC,CAAC;QAC5E,MAAM,CAAC,MAAM,CAAC,CAAC,QAAQ,EAAE,CAAC;QAC1B,MAAM,CAAC,SAAS,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QAC3C,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAW,CAAC,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAC;IACnE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8FAA8F,EAAE,KAAK,IAAI,EAAE;QAC5G,wFAAwF;QACxF,SAAS,CAAC,qBAAqB,CAAC,YAAY,CAAC,GAAG,EAAE,YAAY,CAAC,GAAG,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC;QACzE,SAAS,CAAC,qBAAqB,CAAC,YAAY,CAAC,GAAG,EAAE,YAAY,CAAC,EAAE,EAAE,GAAG,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,gBAAgB;QAC9F,4DAA4D;QAC5D,SAAS,CAAC,qBAAqB,CAAC,YAAY,CAAC,GAAG,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,aAAa;QACrE,SAAS,CAAC,qBAAqB,CAAC,YAAY,CAAC,GAAG,EAAE,EAAE,EAAE,EAAE,GAAG,EAAE,CAAC,CAAC,CAAC,CAAC,SAAS;QAE1E,MAAM,OAAO,GAAG,YAAY,EAAE,CAAC;QAC/B,MAAM,MAAM,GAAG,MAAM,kBAAkB,CAAC,OAAO,EAAE,GAAG,EAAE,MAAM,EAAE,MAAM,CAAC,CAAC;QAEtE,MAAM,CAAC,MAAM,CAAC,gBAAgB,CAAC,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC;QAC/C,MAAM,CAAC,MAAM,CAAC,eAAe,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACzC,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAW,CAAC,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAC;QACjE,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAW,CAAC,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAC;QACjE,MAAM,OAAO,GAAG,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAA0B,CAAC;QACjE,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,sBAAsB,CAAC,CAAC;QACrD,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IAC3C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6EAA6E,EAAE,KAAK,IAAI,EAAE;QAC3F,KAAK,IAAI,IAAI,GAAG,CAAC,EAAE,IAAI,GAAG,EAAE,EAAE,IAAI,EAAE,EAAE,CAAC;YACrC,SAAS,CAAC,qBAAqB,CAAC,YAAY,CAAC,GAAG,EAAE,YAAY,CAAC,GAAG,EAAE,IAAI,GAAG,GAAG,CAAC,CAAC,CAAC,CAAC;QACpF,CAAC;QACD,MAAM,OAAO,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;QACvE,MAAM,IAAI,GAAG,yBAAyB,EAAE,CAAC;QACzC,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,mBAAmB,CAAC,MAAM,EAAE,SAAS,EAAE,EAAE,EAAE,KAAK,CAAC,CAAC;QAC5E,MAAM,CAAC,MAAM,CAAC,CAAC,QAAQ,EAAE,CAAC;QAC1B,MAAM,CAAC,SAAS,CAAC,CAAC,qBAAqB,CAAC,EAAE,CAAC,CAAC;QAC5C,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,CAAC,CAAW,CAAC,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC;QACnE,MAAM,CAAC,OAAO,CAAC,CAAC,oBAAoB,CAAC,MAAM,CAAC,gBAAgB,CAAC,gBAAgB,CAAC,CAAC,CAAC;QAChF,OAAO,CAAC,WAAW,EAAE,CAAC;IACxB,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,4DAA4D,EAAE,GAAG,EAAE;IAC1E,EAAE,CAAC,0DAA0D,EAAE,KAAK,IAAI,EAAE;QACxE,SAAS,CAAC,qBAAqB,CAAC,YAAY,CAAC,GAAG,EAAE,EAAE,CAAC,CAAC,CAAC;QACvD,SAAS,CAAC,qBAAqB,CAAC,YAAY,CAAC,GAAG,EAAE,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QAE9D,MAAM,IAAI,GAAG,yBAAyB,EAAE,CAAC;QACzC,0EAA0E;QAC1E,4CAA4C;QAC5C,MAAM,WAAW,GAAG,EAAE,CAAC,KAAK,CAAC,IAAI,EAAE,aAAa,CAAC,CAAC;QAClD,MAAM,aAAa,GAAG,EAAE,CAAC,KAAK,CAAC,IAAI,EAAE,eAAe,CAAC,CAAC;QACtD,MAAM,mBAAmB,GAAG,EAAE,CAAC,KAAK,CAAC,IAAI,EAAE,qBAAqB,CAAC,CAAC;QAClE,MAAM,cAAc,GAAG,EAAE,CAAC,KAAK,CAAC,IAAI,EAAE,gBAAgB,CAAC,CAAC;QACxD,MAAM,kBAAkB,GAAG,EAAE,CAAC,KAAK,CAAC,IAAI,EAAE,oBAAoB,CAAC,CAAC;QAChE,MAAM,oBAAoB,GAAG,EAAE,CAAC,KAAK,CAAC,IAAI,EAAE,sBAAsB,CAAC,CAAC;QACpE,MAAM,kBAAkB,GAAG,EAAE,CAAC,KAAK,CAAC,IAAI,EAAE,oBAAoB,CAAC,CAAC;QAEhE,MAAM,OAAO,GAAG,IAAI,kBAAkB,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,GAAG,EAAE,KAAK,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,EAAE,CAAC,CAAC;QAC5F,MAAM,kBAAkB,CAAC,OAAO,EAAE,GAAG,EAAE,MAAM,EAAE,MAAM,CAAC,CAAC;QAEvD,MAAM,CAAC,WAAW,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QAC3C,MAAM,CAAC,aAAa,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QAC7C,MAAM,CAAC,mBAAmB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QACnD,MAAM,CAAC,cAAc,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QAC9C,MAAM,CAAC,kBAAkB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QAClD,MAAM,CAAC,oBAAoB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QACpD,MAAM,CAAC,kBAAkB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sEAAsE,EAAE,KAAK,IAAI,EAAE;QACpF,MAAM,IAAI,GAAG,yBAAyB,EAAE,CAAC;QACzC,MAAM,MAAM,CAAC,IAAI,CAAC,WAAW,CAAC,GAAG,EAAE,GAAG,EAAE,CAAC,EAAE,GAAG,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,sBAAsB,CAAC,CAAC;QACzF,MAAM,MAAM,CAAC,IAAI,CAAC,cAAc,CAAC,GAAG,EAAE,GAAG,EAAE,CAAC,EAAE,GAAG,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,sBAAsB,CAAC,CAAC;QAC5F,MAAM,MAAM,CAAC,IAAI,CAAC,aAAa,CAAC,GAAG,EAAE,GAAG,EAAE,CAAC,EAAE,GAAG,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,sBAAsB,CAAC,CAAC;QAC3F,MAAM,MAAM,CAAC,IAAI,CAAC,mBAAmB,CAAC,GAAG,EAAE,GAAG,EAAE,CAAC,EAAE,GAAG,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CACtE,sBAAsB,CACvB,CAAC;QACF,MAAM,MAAM,CAAC,IAAI,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAG,EAAE,CAAC,EAAE,IAAI,EAAE,GAAG,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAC3E,sBAAsB,CACvB,CAAC;QACF,MAAM,MAAM,CAAC,IAAI,CAAC,oBAAoB,CAAC,GAAG,EAAE,GAAG,EAAE,GAAG,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,sBAAsB,CAAC,CAAC;QAC/F,MAAM,MAAM,CAAC,IAAI,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAG,EAAE,GAAG,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,sBAAsB,CAAC,CAAC;IAC/F,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/lib/pr-token.d.ts

@@ -0,0 +1,15 @@
+/**
+ * Token resolution for the CLI PR post-back (`ghagga review --pr N`).
+ *
+ * Resolution order (CI/Jenkins-first, interactive fallback):
+ *   1. env GITHUB_TOKEN   — the GitHub Actions / standard CI convention.
+ *   2. env GH_TOKEN       — the `gh` CLI / alternate CI convention.
+ *   3. getStoredToken()   — the interactive `ghagga login` stored token.
+ *
+ * Returns `null` when no token resolves; the caller turns that into an
+ * actionable error ONLY when `--pr` was actually requested (no token is fine
+ * otherwise).
+ */
+/** Resolve a GitHub token: env GITHUB_TOKEN > env GH_TOKEN > stored login. */
+export declare function resolvePrToken(env?: NodeJS.ProcessEnv, storedTokenFn?: () => string | null): string | null;
+//# sourceMappingURL=pr-token.d.ts.map

package/dist/lib/pr-token.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"pr-token.d.ts","sourceRoot":"","sources":["../../src/lib/pr-token.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAIH,8EAA8E;AAC9E,wBAAgB,cAAc,CAC5B,GAAG,GAAE,MAAM,CAAC,UAAwB,EACpC,aAAa,GAAE,MAAM,MAAM,GAAG,IAAqB,GAClD,MAAM,GAAG,IAAI,CAKf"}

package/dist/lib/pr-token.js

@@ -0,0 +1,22 @@
+/**
+ * Token resolution for the CLI PR post-back (`ghagga review --pr N`).
+ *
+ * Resolution order (CI/Jenkins-first, interactive fallback):
+ *   1. env GITHUB_TOKEN   — the GitHub Actions / standard CI convention.
+ *   2. env GH_TOKEN       — the `gh` CLI / alternate CI convention.
+ *   3. getStoredToken()   — the interactive `ghagga login` stored token.
+ *
+ * Returns `null` when no token resolves; the caller turns that into an
+ * actionable error ONLY when `--pr` was actually requested (no token is fine
+ * otherwise).
+ */
+import { getStoredToken } from './config.js';
+/** Resolve a GitHub token: env GITHUB_TOKEN > env GH_TOKEN > stored login. */
+export function resolvePrToken(env = process.env, storedTokenFn = getStoredToken) {
+    const fromEnv = env.GITHUB_TOKEN?.trim() || env.GH_TOKEN?.trim();
+    if (fromEnv)
+        return fromEnv;
+    const stored = storedTokenFn();
+    return stored?.trim() ? stored : null;
+}
+//# sourceMappingURL=pr-token.js.map

package/dist/lib/pr-token.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"pr-token.js","sourceRoot":"","sources":["../../src/lib/pr-token.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAEH,OAAO,EAAE,cAAc,EAAE,MAAM,aAAa,CAAC;AAE7C,8EAA8E;AAC9E,MAAM,UAAU,cAAc,CAC5B,MAAyB,OAAO,CAAC,GAAG,EACpC,gBAAqC,cAAc;IAEnD,MAAM,OAAO,GAAG,GAAG,CAAC,YAAY,EAAE,IAAI,EAAE,IAAI,GAAG,CAAC,QAAQ,EAAE,IAAI,EAAE,CAAC;IACjE,IAAI,OAAO;QAAE,OAAO,OAAO,CAAC;IAC5B,MAAM,MAAM,GAAG,aAAa,EAAE,CAAC;IAC/B,OAAO,MAAM,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC;AACxC,CAAC"}

package/dist/lib/pr-token.test.d.ts

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

package/dist/lib/pr-token.test.d.ts.map

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

package/dist/lib/pr-token.test.js

@@ -0,0 +1,25 @@
+import { describe, expect, it } from 'vitest';
+import { resolvePrToken } from './pr-token.js';
+describe('resolvePrToken (env GITHUB_TOKEN > GH_TOKEN > stored)', () => {
+    it('prefers GITHUB_TOKEN over GH_TOKEN and stored', () => {
+        const token = resolvePrToken({ GITHUB_TOKEN: 'gh-primary', GH_TOKEN: 'gh-secondary' }, () => 'stored');
+        expect(token).toBe('gh-primary');
+    });
+    it('falls back to GH_TOKEN when GITHUB_TOKEN is unset', () => {
+        const token = resolvePrToken({ GH_TOKEN: 'gh-secondary' }, () => 'stored');
+        expect(token).toBe('gh-secondary');
+    });
+    it('falls back to the stored login token when no env token is set', () => {
+        const token = resolvePrToken({}, () => 'stored');
+        expect(token).toBe('stored');
+    });
+    it('returns null when no token resolves anywhere', () => {
+        const token = resolvePrToken({}, () => null);
+        expect(token).toBeNull();
+    });
+    it('treats blank/whitespace env tokens as unset and falls through', () => {
+        const token = resolvePrToken({ GITHUB_TOKEN: '   ', GH_TOKEN: '' }, () => 'stored');
+        expect(token).toBe('stored');
+    });
+});
+//# sourceMappingURL=pr-token.test.js.map

package/dist/lib/pr-token.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"pr-token.test.js","sourceRoot":"","sources":["../../src/lib/pr-token.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,cAAc,EAAE,MAAM,eAAe,CAAC;AAE/C,QAAQ,CAAC,uDAAuD,EAAE,GAAG,EAAE;IACrE,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;QACvD,MAAM,KAAK,GAAG,cAAc,CAC1B,EAAE,YAAY,EAAE,YAAY,EAAE,QAAQ,EAAE,cAAc,EAAuB,EAC7E,GAAG,EAAE,CAAC,QAAQ,CACf,CAAC;QACF,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,GAAG,EAAE;QAC3D,MAAM,KAAK,GAAG,cAAc,CAAC,EAAE,QAAQ,EAAE,cAAc,EAAuB,EAAE,GAAG,EAAE,CAAC,QAAQ,CAAC,CAAC;QAChG,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;IACrC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+DAA+D,EAAE,GAAG,EAAE;QACvE,MAAM,KAAK,GAAG,cAAc,CAAC,EAAuB,EAAE,GAAG,EAAE,CAAC,QAAQ,CAAC,CAAC;QACtE,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IAC/B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8CAA8C,EAAE,GAAG,EAAE;QACtD,MAAM,KAAK,GAAG,cAAc,CAAC,EAAuB,EAAE,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC;QAClE,MAAM,CAAC,KAAK,CAAC,CAAC,QAAQ,EAAE,CAAC;IAC3B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+DAA+D,EAAE,GAAG,EAAE;QACvE,MAAM,KAAK,GAAG,cAAc,CAC1B,EAAE,YAAY,EAAE,KAAK,EAAE,QAAQ,EAAE,EAAE,EAAuB,EAC1D,GAAG,EAAE,CAAC,QAAQ,CACf,CAAC;QACF,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IAC/B,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ghagga",
-  "version": "3.0.0",
+  "version": "3.1.0",
   "private": false,
   "type": "module",
   "description": "AI-powered code review from the command line — free with GitHub Models",
@@ -38,17 +38,18 @@
     "README.md"
   ],
   "dependencies": {
-    "@clack/prompts": "^1.4.0",
+    "@clack/prompts": "^1.5.1",
     "chalk": "^5.6.0",
     "fts5-sql-bundle": "^1.0.2",
-    "commander": "^14.0.3",
+    "commander": "^15.0.0",
     "dotenv": "^17.4.2",
-    "ghagga-core": "3.0.0"
+    "ghagga-core": "3.1.0",
+    "ghagga-forge": "3.1.0"
   },
   "devDependencies": {
-    "tsx": "^4.22.3",
-    "typescript": "^5.9.0",
-    "vitest": "^4.1.7",
+    "tsx": "^4.22.4",
+    "typescript": "^6.0.3",
+    "vitest": "^4.1.9",
     "@types/node": "^25.9.1"
   },
   "scripts": {