factory-factory 0.4.7 → 0.4.9

package/dist/src/backend/services/env-schemas.d.ts

@@ -1,81 +1,81 @@
 import { z } from 'zod';
 declare const LogLevelSchema: z.ZodEnum<{
+    debug: "debug";
     error: "error";
-    warn: "warn";
     info: "info";
-    debug: "debug";
+    warn: "warn";
 }>;
 export declare const LoggerEnvSchema: z.ZodObject<{
     LOG_LEVEL: z.ZodCatch<z.ZodPreprocess<z.ZodEnum<{
+        debug: "debug";
         error: "error";
-        warn: "warn";
         info: "info";
-        debug: "debug";
-    }>>>;
-    SERVICE_NAME: z.ZodCatch<z.ZodPreprocess<z.ZodString>>;
+        warn: "warn";
+    }>, unknown>>;
+    SERVICE_NAME: z.ZodCatch<z.ZodPreprocess<z.ZodString, unknown>>;
     NODE_ENV: z.ZodCatch<z.ZodPreprocess<z.ZodEnum<{
         development: "development";
         production: "production";
         test: "test";
-    }>>>;
-    BASE_DIR: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString>>>;
+    }>, unknown>>;
+    BASE_DIR: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString, unknown>>>;
 }, z.core.$strip>;
 export declare const ConfigEnvSchema: z.ZodObject<{
-    DEFAULT_MODEL: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString>>>;
+    DEFAULT_MODEL: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString, unknown>>>;
     DEFAULT_PERMISSIONS: z.ZodCatch<z.ZodPreprocess<z.ZodEnum<{
-        strict: "strict";
         relaxed: "relaxed";
+        strict: "strict";
         yolo: "yolo";
-    }>>>;
+    }>, unknown>>;
     LOG_LEVEL: z.ZodCatch<z.ZodPreprocess<z.ZodEnum<{
+        debug: "debug";
         error: "error";
-        warn: "warn";
         info: "info";
-        debug: "debug";
-    }>>>;
-    SERVICE_NAME: z.ZodCatch<z.ZodPreprocess<z.ZodString>>;
-    ACP_STARTUP_TIMEOUT_MS: z.ZodCatch<z.ZodPreprocess<z.ZodNumber>>;
-    ACP_TRACE_LOGS_ENABLED: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodBoolean>>>;
-    ACP_TRACE_LOGS_PATH: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString>>>;
-    WS_LOGS_ENABLED: z.ZodCatch<z.ZodPreprocess<z.ZodBoolean>>;
-    WEB_CONCURRENCY: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodNumber>>>;
-    FF_RUN_SCRIPT_PROXY_ENABLED: z.ZodCatch<z.ZodPreprocess<z.ZodBoolean>>;
-    CLAUDE_CONFIG_DIR: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString>>>;
-    CLAUDE_RATE_LIMIT_PER_MINUTE: z.ZodCatch<z.ZodPreprocess<z.ZodNumber>>;
-    CLAUDE_RATE_LIMIT_PER_HOUR: z.ZodCatch<z.ZodPreprocess<z.ZodNumber>>;
-    RATE_LIMIT_QUEUE_SIZE: z.ZodCatch<z.ZodPreprocess<z.ZodNumber>>;
-    RATE_LIMIT_QUEUE_TIMEOUT_MS: z.ZodCatch<z.ZodPreprocess<z.ZodNumber>>;
-    NOTIFICATION_SOUND_ENABLED: z.ZodCatch<z.ZodPreprocess<z.ZodBoolean>>;
-    NOTIFICATION_PUSH_ENABLED: z.ZodCatch<z.ZodPreprocess<z.ZodBoolean>>;
-    NOTIFICATION_SOUND_FILE: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString>>>;
-    NOTIFICATION_QUIET_HOURS_START: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodNumber>>>;
-    NOTIFICATION_QUIET_HOURS_END: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodNumber>>>;
-    CORS_ALLOWED_ORIGINS: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString>>>;
-    TRUSTED_LOCAL_CIDRS: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString>>>;
-    TRUST_PROXY_HEADERS: z.ZodCatch<z.ZodPreprocess<z.ZodBoolean>>;
-    BASE_DIR: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString>>>;
-    WORKTREE_BASE_DIR: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString>>>;
-    REPOS_DIR: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString>>>;
+        warn: "warn";
+    }>, unknown>>;
+    SERVICE_NAME: z.ZodCatch<z.ZodPreprocess<z.ZodString, unknown>>;
+    ACP_STARTUP_TIMEOUT_MS: z.ZodCatch<z.ZodPreprocess<z.ZodNumber, unknown>>;
+    ACP_TRACE_LOGS_ENABLED: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodBoolean, unknown>>>;
+    ACP_TRACE_LOGS_PATH: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString, unknown>>>;
+    WS_LOGS_ENABLED: z.ZodCatch<z.ZodPreprocess<z.ZodBoolean, unknown>>;
+    WEB_CONCURRENCY: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodNumber, unknown>>>;
+    FF_RUN_SCRIPT_PROXY_ENABLED: z.ZodCatch<z.ZodPreprocess<z.ZodBoolean, unknown>>;
+    CLAUDE_CONFIG_DIR: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString, unknown>>>;
+    CLAUDE_RATE_LIMIT_PER_MINUTE: z.ZodCatch<z.ZodPreprocess<z.ZodNumber, unknown>>;
+    CLAUDE_RATE_LIMIT_PER_HOUR: z.ZodCatch<z.ZodPreprocess<z.ZodNumber, unknown>>;
+    RATE_LIMIT_QUEUE_SIZE: z.ZodCatch<z.ZodPreprocess<z.ZodNumber, unknown>>;
+    RATE_LIMIT_QUEUE_TIMEOUT_MS: z.ZodCatch<z.ZodPreprocess<z.ZodNumber, unknown>>;
+    NOTIFICATION_SOUND_ENABLED: z.ZodCatch<z.ZodPreprocess<z.ZodBoolean, unknown>>;
+    NOTIFICATION_PUSH_ENABLED: z.ZodCatch<z.ZodPreprocess<z.ZodBoolean, unknown>>;
+    NOTIFICATION_SOUND_FILE: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString, unknown>>>;
+    NOTIFICATION_QUIET_HOURS_START: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodNumber, unknown>>>;
+    NOTIFICATION_QUIET_HOURS_END: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodNumber, unknown>>>;
+    CORS_ALLOWED_ORIGINS: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString, unknown>>>;
+    TRUSTED_LOCAL_CIDRS: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString, unknown>>>;
+    TRUST_PROXY_HEADERS: z.ZodCatch<z.ZodPreprocess<z.ZodBoolean, unknown>>;
+    BASE_DIR: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString, unknown>>>;
+    WORKTREE_BASE_DIR: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString, unknown>>>;
+    REPOS_DIR: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString, unknown>>>;
     NODE_ENV: z.ZodCatch<z.ZodPreprocess<z.ZodEnum<{
         development: "development";
         production: "production";
         test: "test";
-    }>>>;
-    WS_LOGS_PATH: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString>>>;
-    FRONTEND_STATIC_PATH: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString>>>;
-    BACKEND_PORT: z.ZodCatch<z.ZodPreprocess<z.ZodNumber>>;
-    BACKEND_HOST: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString>>>;
-    SHELL: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString>>>;
-    DATABASE_PATH: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString>>>;
-    MIGRATIONS_PATH: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString>>>;
-    HEALTH_CHECK_INTERVAL_MS: z.ZodCatch<z.ZodPreprocess<z.ZodNumber>>;
-    MAX_SESSIONS_PER_WORKSPACE: z.ZodCatch<z.ZodPreprocess<z.ZodNumber>>;
-    PR_DISCOVERY_CANDIDATE_LIMIT: z.ZodCatch<z.ZodPreprocess<z.ZodNumber>>;
-    PR_DISCOVERY_REPOSITORY_LIMIT: z.ZodCatch<z.ZodPreprocess<z.ZodNumber>>;
-    DEBUG_CHAT_WS: z.ZodCatch<z.ZodPreprocess<z.ZodBoolean>>;
-    EVENT_COMPRESSION_ENABLED: z.ZodCatch<z.ZodPreprocess<z.ZodBoolean>>;
-    BRANCH_RENAME_MESSAGE_THRESHOLD: z.ZodCatch<z.ZodPreprocess<z.ZodNumber>>;
-    npm_package_version: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString>>>;
+    }>, unknown>>;
+    WS_LOGS_PATH: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString, unknown>>>;
+    FRONTEND_STATIC_PATH: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString, unknown>>>;
+    BACKEND_PORT: z.ZodCatch<z.ZodPreprocess<z.ZodNumber, unknown>>;
+    BACKEND_HOST: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString, unknown>>>;
+    SHELL: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString, unknown>>>;
+    DATABASE_PATH: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString, unknown>>>;
+    MIGRATIONS_PATH: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString, unknown>>>;
+    HEALTH_CHECK_INTERVAL_MS: z.ZodCatch<z.ZodPreprocess<z.ZodNumber, unknown>>;
+    MAX_SESSIONS_PER_WORKSPACE: z.ZodCatch<z.ZodPreprocess<z.ZodNumber, unknown>>;
+    PR_DISCOVERY_CANDIDATE_LIMIT: z.ZodCatch<z.ZodPreprocess<z.ZodNumber, unknown>>;
+    PR_DISCOVERY_REPOSITORY_LIMIT: z.ZodCatch<z.ZodPreprocess<z.ZodNumber, unknown>>;
+    DEBUG_CHAT_WS: z.ZodCatch<z.ZodPreprocess<z.ZodBoolean, unknown>>;
+    EVENT_COMPRESSION_ENABLED: z.ZodCatch<z.ZodPreprocess<z.ZodBoolean, unknown>>;
+    BRANCH_RENAME_MESSAGE_THRESHOLD: z.ZodCatch<z.ZodPreprocess<z.ZodNumber, unknown>>;
+    npm_package_version: z.ZodCatch<z.ZodOptional<z.ZodPreprocess<z.ZodString, unknown>>>;
 }, z.core.$strip>;
 export type LogLevel = z.infer<typeof LogLevelSchema>;
 export {};

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

@@ -1 +1 @@
-{"version":3,"file":"constants.d.ts","sourceRoot":"","sources":["../../../../../../../src/backend/services/github/service/github-cli/constants.ts"],"names":[],"mappings":"AACA,eAAO,MAAM,cAAc,IAAI,CAAC;AAEhC,eAAO,MAAM,aAAa;;;;;;;EAOf,CAAC;AAEZ,eAAO,MAAM,mBAAmB;;;EAGrB,CAAC"}
+{"version":3,"file":"constants.d.ts","sourceRoot":"","sources":["../../../../../../../src/backend/services/github/service/github-cli/constants.ts"],"names":[],"mappings":"AACA,eAAO,MAAM,cAAc,IAAI,CAAC;AAEhC,eAAO,MAAM,aAAa;4BACT,IAAI;yBACP,KAAM;yBACN,KAAM;sBACT,KAAM;4BACA,KAAM;mBACf,KAAM;EACH,CAAC;AAEZ,eAAO,MAAM,mBAAmB;;;EAGrB,CAAC"}

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

@@ -1,5 +1,5 @@
-import { createLogger } from '../../../../../backend/services/logger.service.js';
-import { isRateLimitMessage } from '../../../../../backend/services/rate-limit-backoff.js';
+import { createLogger } from '../../../logger.service.js';
+import { isRateLimitMessage } from '../../../rate-limit-backoff.js';
 const logger = createLogger('github-cli');
 function isCliNotInstalledError(message) {
     return message.includes('enoent') || message.includes('not found');

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

@@ -2,16 +2,16 @@ import { z } from 'zod';
 export declare const prStatusSchema: z.ZodObject<{
     number: z.ZodNumber;
     state: z.ZodEnum<{
-        OPEN: "OPEN";
-        MERGED: "MERGED";
         CLOSED: "CLOSED";
+        MERGED: "MERGED";
+        OPEN: "OPEN";
     }>;
     isDraft: z.ZodBoolean;
     reviewDecision: z.ZodPreprocess<z.ZodNullable<z.ZodEnum<{
-        CHANGES_REQUESTED: "CHANGES_REQUESTED";
         APPROVED: "APPROVED";
+        CHANGES_REQUESTED: "CHANGES_REQUESTED";
         REVIEW_REQUIRED: "REVIEW_REQUIRED";
-    }>>>;
+    }>>, unknown>;
     statusCheckRollup: z.ZodNullable<z.ZodArray<z.ZodObject<{
         name: z.ZodOptional<z.ZodString>;
         workflowName: z.ZodOptional<z.ZodString>;
@@ -64,15 +64,15 @@ export declare const fullPRDetailsSchema: z.ZodObject<{
     updatedAt: z.ZodString;
     isDraft: z.ZodBoolean;
     state: z.ZodEnum<{
-        OPEN: "OPEN";
-        MERGED: "MERGED";
         CLOSED: "CLOSED";
+        MERGED: "MERGED";
+        OPEN: "OPEN";
     }>;
     reviewDecision: z.ZodPreprocess<z.ZodNullable<z.ZodEnum<{
-        CHANGES_REQUESTED: "CHANGES_REQUESTED";
         APPROVED: "APPROVED";
+        CHANGES_REQUESTED: "CHANGES_REQUESTED";
         REVIEW_REQUIRED: "REVIEW_REQUIRED";
-    }>>>;
+    }>>, unknown>;
     statusCheckRollup: z.ZodNullable<z.ZodArray<z.ZodUnion<readonly [z.ZodObject<{
         __typename: z.ZodOptional<z.ZodEnum<{
             CheckRun: "CheckRun";
@@ -124,13 +124,13 @@ export declare const fullPRDetailsSchema: z.ZodObject<{
     headRefName: z.ZodOptional<z.ZodString>;
     baseRefName: z.ZodOptional<z.ZodString>;
     mergeStateStatus: z.ZodOptional<z.ZodEnum<{
-        DRAFT: "DRAFT";
-        UNKNOWN: "UNKNOWN";
         BEHIND: "BEHIND";
         BLOCKED: "BLOCKED";
         CLEAN: "CLEAN";
         DIRTY: "DIRTY";
+        DRAFT: "DRAFT";
         HAS_HOOKS: "HAS_HOOKS";
+        UNKNOWN: "UNKNOWN";
         UNSTABLE: "UNSTABLE";
     }>>;
 }, z.core.$strip>;
@@ -140,8 +140,8 @@ export declare const issueSchema: z.ZodObject<{
     body: z.ZodString;
     url: z.ZodString;
     state: z.ZodEnum<{
-        OPEN: "OPEN";
         CLOSED: "CLOSED";
+        OPEN: "OPEN";
     }>;
     createdAt: z.ZodString;
     author: z.ZodObject<{
@@ -168,10 +168,10 @@ export declare const reviewRequestedPRGraphQLSchema: z.ZodObject<{
                     nameWithOwner: z.ZodString;
                 }, z.core.$strip>;
                 reviewDecision: z.ZodOptional<z.ZodPreprocess<z.ZodNullable<z.ZodEnum<{
-                    CHANGES_REQUESTED: "CHANGES_REQUESTED";
                     APPROVED: "APPROVED";
+                    CHANGES_REQUESTED: "CHANGES_REQUESTED";
                     REVIEW_REQUIRED: "REVIEW_REQUIRED";
-                }>>>>;
+                }>>, unknown>>;
                 additions: z.ZodOptional<z.ZodNumber>;
                 deletions: z.ZodOptional<z.ZodNumber>;
                 changedFiles: z.ZodOptional<z.ZodNumber>;

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

@@ -1,5 +1,5 @@
 import { z } from 'zod';
-import { createLogger } from '../../../../../backend/services/logger.service.js';
+import { createLogger } from '../../../logger.service.js';
 const logger = createLogger('github-cli');
 export function parseGhJson(schema, stdout, context) {
     try {

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

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

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

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

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

@@ -0,0 +1,45 @@
+import { beforeEach, expect, it, vi } from 'vitest';
+const mockExecFile = vi.hoisted(() => vi.fn());
+vi.mock('node:child_process', () => ({ execFile: mockExecFile }));
+vi.mock('node:util', () => ({ promisify: (fn) => fn }));
+vi.mock('@/backend/services/logger.service', () => ({
+    createLogger: () => ({ debug: vi.fn(), info: vi.fn(), warn: vi.fn(), error: vi.fn() }),
+}));
+import { githubCLIService } from './github-cli.service.js';
+beforeEach(() => {
+    mockExecFile.mockReset();
+    githubCLIService.clearCaches();
+});
+it('refreshes cached authentication immediately when explicitly requested', async () => {
+    mockExecFile
+        .mockResolvedValueOnce({ stdout: 'gh version 2.20.0', stderr: '' })
+        .mockRejectedValueOnce(new Error('not logged in'));
+    expect((await githubCLIService.checkHealth()).isAuthenticated).toBe(false);
+    mockExecFile.mockResolvedValue({ stdout: 'gh version 2.20.0', stderr: '' });
+    expect((await githubCLIService.checkHealth()).isAuthenticated).toBe(false);
+    expect((await githubCLIService.checkHealth(true)).isAuthenticated).toBe(true);
+    expect((await githubCLIService.checkHealth()).isAuthenticated).toBe(true);
+});
+it('waits for pre-login background checks before forcing a new authentication check', async () => {
+    const now = vi.spyOn(Date, 'now').mockReturnValue(0);
+    mockExecFile
+        .mockResolvedValueOnce({ stdout: 'gh version 2.20.0', stderr: '' })
+        .mockRejectedValueOnce(new Error('not logged in'));
+    await githubCLIService.checkHealth();
+    let rejectOldAuth;
+    mockExecFile
+        .mockResolvedValueOnce({ stdout: 'gh version 2.20.0', stderr: '' })
+        .mockImplementationOnce(() => new Promise((_resolve, reject) => {
+        rejectOldAuth = reject;
+    }))
+        .mockResolvedValue({ stdout: 'gh version 2.20.0', stderr: '' });
+    now.mockReturnValue(30_001);
+    expect((await githubCLIService.checkHealth()).isAuthenticated).toBe(false);
+    await vi.waitFor(() => expect(rejectOldAuth).toBeTypeOf('function'));
+    const forced = githubCLIService.checkHealth(true);
+    await new Promise((resolve) => setImmediate(resolve));
+    rejectOldAuth(new Error('pre-login result'));
+    expect((await forced).isAuthenticated).toBe(true);
+    expect((await githubCLIService.checkHealth()).isAuthenticated).toBe(true);
+});
+//# sourceMappingURL=github-cli-health-cache.test.js.map

package/dist/src/backend/services/github/service/github-cli-health-cache.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"github-cli-health-cache.test.js","sourceRoot":"","sources":["../../../../../../src/backend/services/github/service/github-cli-health-cache.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAEpD,MAAM,YAAY,GAAG,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;AAC/C,EAAE,CAAC,IAAI,CAAC,oBAAoB,EAAE,GAAG,EAAE,CAAC,CAAC,EAAE,QAAQ,EAAE,YAAY,EAAE,CAAC,CAAC,CAAC;AAClE,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,KAAK,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,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,UAAU,CAAC,GAAG,EAAE;IACd,YAAY,CAAC,SAAS,EAAE,CAAC;IACzB,gBAAgB,CAAC,WAAW,EAAE,CAAC;AACjC,CAAC,CAAC,CAAC;AAEH,EAAE,CAAC,uEAAuE,EAAE,KAAK,IAAI,EAAE;IACrF,YAAY;SACT,qBAAqB,CAAC,EAAE,MAAM,EAAE,mBAAmB,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC;SAClE,qBAAqB,CAAC,IAAI,KAAK,CAAC,eAAe,CAAC,CAAC,CAAC;IACrD,MAAM,CAAC,CAAC,MAAM,gBAAgB,CAAC,WAAW,EAAE,CAAC,CAAC,eAAe,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAE3E,YAAY,CAAC,iBAAiB,CAAC,EAAE,MAAM,EAAE,mBAAmB,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC,CAAC;IAC5E,MAAM,CAAC,CAAC,MAAM,gBAAgB,CAAC,WAAW,EAAE,CAAC,CAAC,eAAe,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAC3E,MAAM,CAAC,CAAC,MAAM,gBAAgB,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC,CAAC,eAAe,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC9E,MAAM,CAAC,CAAC,MAAM,gBAAgB,CAAC,WAAW,EAAE,CAAC,CAAC,eAAe,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC5E,CAAC,CAAC,CAAC;AAEH,EAAE,CAAC,iFAAiF,EAAE,KAAK,IAAI,EAAE;IAC/F,MAAM,GAAG,GAAG,EAAE,CAAC,KAAK,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC;IACrD,YAAY;SACT,qBAAqB,CAAC,EAAE,MAAM,EAAE,mBAAmB,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC;SAClE,qBAAqB,CAAC,IAAI,KAAK,CAAC,eAAe,CAAC,CAAC,CAAC;IACrD,MAAM,gBAAgB,CAAC,WAAW,EAAE,CAAC;IAErC,IAAI,aAAsC,CAAC;IAC3C,YAAY;SACT,qBAAqB,CAAC,EAAE,MAAM,EAAE,mBAAmB,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC;SAClE,sBAAsB,CACrB,GAAG,EAAE,CACH,IAAI,OAAO,CAAC,CAAC,QAAQ,EAAE,MAAM,EAAE,EAAE;QAC/B,aAAa,GAAG,MAAM,CAAC;IACzB,CAAC,CAAC,CACL;SACA,iBAAiB,CAAC,EAAE,MAAM,EAAE,mBAAmB,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC,CAAC;IAClE,GAAG,CAAC,eAAe,CAAC,MAAM,CAAC,CAAC;IAC5B,MAAM,CAAC,CAAC,MAAM,gBAAgB,CAAC,WAAW,EAAE,CAAC,CAAC,eAAe,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAC3E,MAAM,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,MAAM,CAAC,aAAa,CAAC,CAAC,UAAU,CAAC,UAAU,CAAC,CAAC,CAAC;IACrE,MAAM,MAAM,GAAG,gBAAgB,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC;IAElD,MAAM,IAAI,OAAO,CAAO,CAAC,OAAO,EAAE,EAAE,CAAC,YAAY,CAAC,OAAO,CAAC,CAAC,CAAC;IAC5D,aAAa,CAAC,IAAI,KAAK,CAAC,kBAAkB,CAAC,CAAC,CAAC;IAC7C,MAAM,CAAC,CAAC,MAAM,MAAM,CAAC,CAAC,eAAe,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAClD,MAAM,CAAC,CAAC,MAAM,gBAAgB,CAAC,WAAW,EAAE,CAAC,CAAC,eAAe,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC5E,CAAC,CAAC,CAAC"}

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

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

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

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

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

@@ -0,0 +1,58 @@
+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';
+function comment(id, updatedAt = id) {
+    return {
+        id,
+        user: { login: 'reviewer' },
+        body: `Review ${id}`,
+        path: 'src/index.ts',
+        line: 1,
+        created_at: new Date(id * 1000).toISOString(),
+        updated_at: new Date(updatedAt * 1000).toISOString(),
+        html_url: `https://github.com/owner/repo/pull/1#discussion_r${id}`,
+    };
+}
+beforeEach(() => {
+    vi.resetAllMocks();
+    githubCLIService.clearCaches();
+});
+describe('review comment pagination', () => {
+    it('keeps the latest activity, including edits to old comments, at the page cap', async () => {
+        const comments = Array.from({ length: 2001 }, (_, index) => comment(index + 1));
+        comments[0] = comment(1, 3000);
+        vi.mocked(execFile).mockImplementation(((_command, args) => {
+            const url = new URL(`https://api.github.com/${args[1]}`);
+            const newestFirst = url.searchParams.get('sort') === 'updated' && url.searchParams.get('direction') === 'desc';
+            const ordered = newestFirst
+                ? [...comments].sort((a, b) => b.updated_at.localeCompare(a.updated_at))
+                : comments;
+            const page = Number(url.searchParams.get('page'));
+            return Promise.resolve({
+                stdout: JSON.stringify(ordered.slice((page - 1) * 100, page * 100)),
+                stderr: '',
+            });
+        }));
+        const result = await githubCLIService.getReviewComments('owner/repo', 1);
+        expect(execFile).toHaveBeenCalledTimes(20);
+        expect(result).toHaveLength(2000);
+        expect(result.map((entry) => entry.id)).toContain(2001);
+        expect(result.map((entry) => entry.id)).toContain(1);
+        expect(result.map((entry) => entry.id)).not.toContain(2);
+        const updateTimes = result.map((entry) => entry.updatedAt);
+        expect(updateTimes).toEqual([...updateTimes].sort());
+        expect(result.at(-1)?.updatedAt).toBe(comments[0]?.updated_at);
+    });
+    it('retains the since filter while fetching newest updates first', async () => {
+        vi.mocked(execFile).mockResolvedValue({ stdout: '[]', stderr: '' });
+        const since = new Date('2026-09-01T00:00:00Z');
+        await githubCLIService.getReviewComments('owner/repo', 1, since);
+        expect(execFile).toHaveBeenCalledWith('gh', ['api', expect.stringContaining(`&since=${since.toISOString()}`)], expect.any(Object));
+    });
+});
+//# sourceMappingURL=github-cli-review-pagination.test.js.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"github-cli-review-pagination.test.js","sourceRoot":"","sources":["../../../../../../src/backend/services/github/service/github-cli-review-pagination.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;AAExD,SAAS,OAAO,CAAC,EAAU,EAAE,SAAS,GAAW,EAAE;IACjD,OAAO;QACL,EAAE;QACF,IAAI,EAAE,EAAE,KAAK,EAAE,UAAU,EAAE;QAC3B,IAAI,EAAE,UAAU,EAAE,EAAE;QACpB,IAAI,EAAE,cAAc;QACpB,IAAI,EAAE,CAAC;QACP,UAAU,EAAE,IAAI,IAAI,CAAC,EAAE,GAAG,IAAI,CAAC,CAAC,WAAW,EAAE;QAC7C,UAAU,EAAE,IAAI,IAAI,CAAC,SAAS,GAAG,IAAI,CAAC,CAAC,WAAW,EAAE;QACpD,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,2BAA2B,EAAE,GAAG,EAAE;IACzC,EAAE,CAAC,6EAA6E,EAAE,KAAK,IAAI,EAAE;QAC3F,MAAM,QAAQ,GAAG,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,EAAE,CAAC,CAAC,EAAE,KAAK,EAAE,EAAE,CAAC,OAAO,CAAC,KAAK,GAAG,CAAC,CAAC,CAAC,CAAC;QAChF,QAAQ,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC;QAC/B,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,kBAAkB,CAAC,CAAC,CAAC,QAAgB,EAAE,IAAc,EAAE,EAAE;YAC3E,MAAM,GAAG,GAAG,IAAI,GAAG,CAAC,0BAA0B,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;YACzD,MAAM,WAAW,GACf,GAAG,CAAC,YAAY,CAAC,GAAG,CAAC,MAAM,CAAC,KAAK,SAAS,IAAI,GAAG,CAAC,YAAY,CAAC,GAAG,CAAC,WAAW,CAAC,KAAK,MAAM,CAAC;YAC7F,MAAM,OAAO,GAAG,WAAW;gBACzB,CAAC,CAAC,CAAC,GAAG,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,UAAU,CAAC,aAAa,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC;gBACxE,CAAC,CAAC,QAAQ,CAAC;YACb,MAAM,IAAI,GAAG,MAAM,CAAC,GAAG,CAAC,YAAY,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,CAAC;YAClD,OAAO,OAAO,CAAC,OAAO,CAAC;gBACrB,MAAM,EAAE,IAAI,CAAC,SAAS,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC,IAAI,GAAG,CAAC,CAAC,GAAG,GAAG,EAAE,IAAI,GAAG,GAAG,CAAC,CAAC;gBACnE,MAAM,EAAE,EAAE;aACX,CAAC,CAAC;QACL,CAAC,CAAU,CAAC,CAAC;QAEb,MAAM,MAAM,GAAG,MAAM,gBAAgB,CAAC,iBAAiB,CAAC,YAAY,EAAE,CAAC,CAAC,CAAC;QAEzE,MAAM,CAAC,QAAQ,CAAC,CAAC,qBAAqB,CAAC,EAAE,CAAC,CAAC;QAC3C,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,IAAI,CAAC,CAAC;QAClC,MAAM,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QACxD,MAAM,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC;QACrD,MAAM,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC;QACzD,MAAM,WAAW,GAAG,MAAM,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC;QAC3D,MAAM,CAAC,WAAW,CAAC,CAAC,OAAO,CAAC,CAAC,GAAG,WAAW,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC;QACrD,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,EAAE,SAAS,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,CAAC,EAAE,UAAU,CAAC,CAAC;IACjE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,KAAK,IAAI,EAAE;QAC5E,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,iBAAiB,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,EAAE,EAAW,CAAC,CAAC;QAC7E,MAAM,KAAK,GAAG,IAAI,IAAI,CAAC,sBAAsB,CAAC,CAAC;QAC/C,MAAM,gBAAgB,CAAC,iBAAiB,CAAC,YAAY,EAAE,CAAC,EAAE,KAAK,CAAC,CAAC;QACjE,MAAM,CAAC,QAAQ,CAAC,CAAC,oBAAoB,CACnC,IAAI,EACJ,CAAC,KAAK,EAAE,MAAM,CAAC,gBAAgB,CAAC,UAAU,KAAK,CAAC,WAAW,EAAE,EAAE,CAAC,CAAC,EACjE,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,CACnB,CAAC;IACJ,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

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

@@ -15,7 +15,7 @@ declare class GitHubCLIService {
     private exec;
     private execMutating;
     getAuthenticatedUsername(signal?: AbortSignal): Promise<string | null>;
-    checkHealth(): Promise<GitHubCLIHealthStatus>;
+    checkHealth(forceRefresh?: boolean): Promise<GitHubCLIHealthStatus>;
     private runCheckHealth;
     extractPRInfo(prUrl: string): PRInfo | null;
     getPRStatus(prUrl: string): Promise<PRStatusFromGitHub | null>;

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,CAAS;IACtC,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;IAenB,OAAO,CAAC,IAAI;IA+CZ,OAAO,CAAC,YAAY;IA6Bd,wBAAwB,CAAC,MAAM,CAAC,EAAE,WAAW,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC;IAiBtE,WAAW,IAAI,OAAO,CAAC,qBAAqB,CAAC;YAyBrC,cAAc;IAqC5B,aAAa,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI;IAgBrC,WAAW,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC,kBAAkB,GAAG,IAAI,CAAC;IAkCpE,eAAe,CAAC,iBAAiB,EAAE,kBAAkB,CAAC,mBAAmB,CAAC;IAI1E,cAAc,CAAC,MAAM,EAAE;QAAE,KAAK,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,OAAO,CAAC;QAAC,cAAc,EAAE,MAAM,GAAG,IAAI,CAAA;KAAE;IAQnF,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;IAmBH,kBAAkB,IAAI,OAAO,CAAC,iBAAiB,EAAE,CAAC;IA2ElD,WAAW,CAAC,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,eAAe,EAAE,CAAC;IAmDpE,gBAAgB,CACpB,IAAI,EAAE,MAAM,EACZ,QAAQ,EAAE,MAAM,EAChB,MAAM,CAAC,EAAE,WAAW,GACnB,OAAO,CAAC,iBAAiB,CAAC;IA6EvB,SAAS,CAAC,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC;IA0B1D,YAAY,CAChB,IAAI,EAAE,MAAM,EACZ,QAAQ,EAAE,MAAM,EAChB,MAAM,EAAE,YAAY,EACpB,IAAI,CAAC,EAAE,MAAM,GACZ,OAAO,CAAC,IAAI,CAAC;IAqCV,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;YA8BX,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;IAoFK,2BAA2B,CAC/B,IAAI,EAAE,MAAM,EACZ,QAAQ,EAAE,MAAM,EAChB,MAAM,CAAC,EAAE,WAAW,GACnB,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC;YAgCT,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;IAwBzE,eAAe,CACnB,KAAK,EAAE,MAAM,EACb,IAAI,EAAE,MAAM,EACZ,WAAW,EAAE,MAAM,EACnB,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,IAAI,CAAC;IA0BV,QAAQ,CAAC,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,WAAW,GAAG,IAAI,CAAC;CA+B9F;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":"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"}

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

@@ -1,8 +1,8 @@
 import { execFile } from 'node:child_process';
 import { promisify } from 'node:util';
 import pLimit from 'p-limit';
-import { createLogger } from '../../../../backend/services/logger.service.js';
-import { isRateLimitMessage } from '../../../../backend/services/rate-limit-backoff.js';
+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 { classifyError, logGitHubCLIError } from './github-cli/errors.js';
 import { computeCIStatus, computePRState, mapComments, mapLabels, mapReviews, mapStatusChecks, } from './github-cli/mappers.js';
@@ -35,7 +35,7 @@ class GitHubCLIService {
     execLimit = pLimit(GH_CONCURRENCY);
     inflight = new Map();
     cachedHealth = null;
-    healthRefreshInFlight = false;
+    healthRefreshInFlight = null;
     HEALTH_CACHE_TTL_MS = 30_000;
     issueCache = new Map();
     issueRefreshInFlight = new Set();
@@ -43,7 +43,7 @@ class GitHubCLIService {
     rateLimitedUntil = null;
     clearCaches() {
         this.cachedHealth = null;
-        this.healthRefreshInFlight = false;
+        this.healthRefreshInFlight = null;
         this.issueCache.clear();
         this.issueRefreshInFlight.clear();
         this.rateLimitedUntil = null;
@@ -71,7 +71,7 @@ class GitHubCLIService {
         }
         const key = args.join('\0');
         const existing = this.inflight.get(key);
-        if (existing) {
+        if (existing !== undefined) {
             return existing;
         }
         const promise = execute().finally(() => {
@@ -108,27 +108,29 @@ class GitHubCLIService {
             return null;
         }
     }
-    async checkHealth() {
-        const now = Date.now();
-        if (this.cachedHealth) {
-            const isStale = now - this.cachedHealth.fetchedAt >= this.HEALTH_CACHE_TTL_MS;
-            if (isStale && !this.healthRefreshInFlight) {
-                this.healthRefreshInFlight = true;
-                this.runCheckHealth()
-                    .then((result) => {
-                    this.cachedHealth = { result, fetchedAt: Date.now() };
-                })
-                    .catch(() => {
-                })
-                    .finally(() => {
-                    this.healthRefreshInFlight = false;
-                });
-            }
-            return this.cachedHealth.result;
+    async checkHealth(forceRefresh = false) {
+        if (forceRefresh && this.healthRefreshInFlight) {
+            await this.healthRefreshInFlight.catch(() => undefined);
+        }
+        const cached = this.cachedHealth;
+        if (!forceRefresh && cached && Date.now() - cached.fetchedAt < this.HEALTH_CACHE_TTL_MS) {
+            return cached.result;
+        }
+        if (!this.healthRefreshInFlight) {
+            const refresh = this.runCheckHealth()
+                .then((result) => {
+                this.cachedHealth = { result, fetchedAt: Date.now() };
+                return result;
+            })
+                .finally(() => {
+                if (this.healthRefreshInFlight === refresh) {
+                    this.healthRefreshInFlight = null;
+                }
+            });
+            this.healthRefreshInFlight = refresh;
         }
-        const result = await this.runCheckHealth();
-        this.cachedHealth = { result, fetchedAt: Date.now() };
-        return result;
+        void this.healthRefreshInFlight.catch(() => undefined);
+        return !forceRefresh && cached ? cached.result : this.healthRefreshInFlight;
     }
     async runCheckHealth() {
         try {
@@ -506,7 +508,7 @@ class GitHubCLIService {
             for (let page = 1; page <= MAX_PAGES; page++) {
                 signal?.throwIfAborted();
                 const sinceParam = since ? `&since=${since.toISOString()}` : '';
-                const path = `repos/${repo}/pulls/${prNumber}/comments?per_page=${PAGE_SIZE}&page=${page}${sinceParam}`;
+                const path = `repos/${repo}/pulls/${prNumber}/comments?per_page=${PAGE_SIZE}&page=${page}${sinceParam}&sort=updated&direction=desc`;
                 const { stdout } = await this.exec(['api', path], {
                     timeout: GH_TIMEOUT_MS.default,
                     maxBuffer: GH_MAX_BUFFER_BYTES.reviewComments,
@@ -541,7 +543,7 @@ class GitHubCLIService {
                     });
                 }
             }
-            return allComments;
+            return allComments.reverse();
         }
         catch (error) {
             signal?.throwIfAborted();