github-issue-tower-defence-management 2.19.1 → 2.19.2

package/src/domain/usecases/IssueRejectionEvaluator.test.ts

@@ -1,1137 +0,0 @@
-import { IssueRejectionEvaluator } from './IssueRejectionEvaluator';
-import { RelatedPullRequest } from './adapter-interfaces/IssueRepository';
-
-const createReadyPr = (
-  url = 'https://github.com/user/repo/pull/1',
-  overrides: Partial<RelatedPullRequest> = {},
-): RelatedPullRequest => ({
-  url,
-  branchName: 'feature-branch',
-  createdAt: new Date('2000-01-01T00:00:00Z'),
-  isDraft: false,
-  isConflicted: false,
-  mergeable: null,
-  isPassedAllCiJob: true,
-  isCiStateSuccess: true,
-  isResolvedAllReviewComments: true,
-  isBranchOutOfDate: false,
-  missingRequiredCheckNames: [],
-  ...overrides,
-});
-
-describe('IssueRejectionEvaluator', () => {
-  let mockIssueRepository: {
-    findRelatedOpenPRs: jest.Mock;
-    getOpenPullRequest: jest.Mock;
-    getPullRequestChangedFilePaths: jest.Mock;
-    requestChangesWithInlineComment: jest.Mock;
-  };
-  let evaluator: IssueRejectionEvaluator;
-
-  beforeEach(() => {
-    jest.resetAllMocks();
-
-    mockIssueRepository = {
-      findRelatedOpenPRs: jest.fn(),
-      getOpenPullRequest: jest.fn(),
-      getPullRequestChangedFilePaths: jest.fn().mockResolvedValue([]),
-      requestChangesWithInlineComment: jest.fn().mockResolvedValue(undefined),
-    };
-
-    evaluator = new IssueRejectionEvaluator(mockIssueRepository);
-  });
-
-  describe('evaluate', () => {
-    it('should return no rejections when PR is ready and not draft', async () => {
-      mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-        createReadyPr(),
-      ]);
-
-      const result = await evaluator.evaluate({
-        url: 'https://github.com/user/repo/issues/1',
-        labels: [],
-        isPr: false,
-      });
-
-      expect(result.rejections).toHaveLength(0);
-      expect(result.approvedPrUrl).toBe('https://github.com/user/repo/pull/1');
-    });
-
-    it('should reject with PULL_REQUEST_IS_DRAFT when PR is in draft state', async () => {
-      mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-        createReadyPr('https://github.com/user/repo/pull/1', { isDraft: true }),
-      ]);
-
-      const result = await evaluator.evaluate({
-        url: 'https://github.com/user/repo/issues/1',
-        labels: [],
-        isPr: false,
-      });
-
-      expect(result.rejections).toHaveLength(1);
-      expect(result.rejections[0].type).toBe('PULL_REQUEST_IS_DRAFT');
-      expect(result.rejections[0].detail).toContain('PULL_REQUEST_IS_DRAFT');
-      expect(result.rejections[0].detail).toContain(
-        'https://github.com/user/repo/pull/1',
-      );
-      expect(result.approvedPrUrl).toBeNull();
-    });
-
-    it('should not approve a draft PR even when all other checks pass', async () => {
-      mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-        createReadyPr('https://github.com/user/repo/pull/1', { isDraft: true }),
-      ]);
-
-      const result = await evaluator.evaluate({
-        url: 'https://github.com/user/repo/issues/1',
-        labels: [],
-        isPr: false,
-      });
-
-      expect(result.approvedPrUrl).toBeNull();
-    });
-
-    it('should accumulate PULL_REQUEST_IS_DRAFT alongside PULL_REQUEST_CONFLICTED when draft PR is also conflicted', async () => {
-      mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-        createReadyPr('https://github.com/user/repo/pull/1', {
-          isDraft: true,
-          isConflicted: true,
-        }),
-      ]);
-
-      const result = await evaluator.evaluate({
-        url: 'https://github.com/user/repo/issues/1',
-        labels: [],
-        isPr: false,
-      });
-
-      const rejectionTypes = result.rejections.map((r) => r.type);
-      expect(rejectionTypes).toContain('PULL_REQUEST_IS_DRAFT');
-      expect(rejectionTypes).toContain('PULL_REQUEST_CONFLICTED');
-      expect(result.approvedPrUrl).toBeNull();
-    });
-
-    it('should reject with PULL_REQUEST_IS_DRAFT when PR item (isPr=true) is in draft state', async () => {
-      mockIssueRepository.getOpenPullRequest.mockResolvedValue(
-        createReadyPr('https://github.com/user/repo/pull/10', {
-          isDraft: true,
-        }),
-      );
-
-      const result = await evaluator.evaluate({
-        url: 'https://github.com/user/repo/pull/10',
-        labels: [],
-        isPr: true,
-      });
-
-      expect(mockIssueRepository.getOpenPullRequest).toHaveBeenCalledWith(
-        'https://github.com/user/repo/pull/10',
-      );
-      expect(result.rejections).toHaveLength(1);
-      expect(result.rejections[0].type).toBe('PULL_REQUEST_IS_DRAFT');
-      expect(result.approvedPrUrl).toBeNull();
-    });
-
-    it('should not require PR evaluation when issue has a non-developer agent field value', async () => {
-      const result = await evaluator.evaluate({
-        url: 'https://github.com/user/repo/issues/1',
-        labels: [],
-        isPr: false,
-        agent: 'chore',
-      });
-
-      expect(mockIssueRepository.findRelatedOpenPRs).not.toHaveBeenCalled();
-      expect(result.rejections).toHaveLength(0);
-      expect(result.approvedPrUrl).toBeNull();
-    });
-
-    it('should require PR evaluation when issue has agent field set to developer', async () => {
-      mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([]);
-
-      const result = await evaluator.evaluate({
-        url: 'https://github.com/user/repo/issues/1',
-        labels: [],
-        isPr: false,
-        agent: 'developer',
-      });
-
-      expect(mockIssueRepository.findRelatedOpenPRs).toHaveBeenCalled();
-      expect(result.rejections).toHaveLength(1);
-      expect(result.rejections[0].type).toBe('PULL_REQUEST_NOT_FOUND');
-    });
-
-    it('should require PR evaluation when issue has null agent field (default developer)', async () => {
-      mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([]);
-
-      const result = await evaluator.evaluate({
-        url: 'https://github.com/user/repo/issues/1',
-        labels: [],
-        isPr: false,
-        agent: null,
-      });
-
-      expect(mockIssueRepository.findRelatedOpenPRs).toHaveBeenCalled();
-      expect(result.rejections).toHaveLength(1);
-      expect(result.rejections[0].type).toBe('PULL_REQUEST_NOT_FOUND');
-    });
-
-    it('should require PR evaluation when issue has agent field set to developer even with llm-agent:developer label', async () => {
-      mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([]);
-
-      const result = await evaluator.evaluate({
-        url: 'https://github.com/user/repo/issues/1',
-        labels: ['llm-agent:developer'],
-        isPr: false,
-        agent: 'developer',
-      });
-
-      expect(mockIssueRepository.findRelatedOpenPRs).toHaveBeenCalled();
-      expect(result.rejections).toHaveLength(1);
-      expect(result.rejections[0].type).toBe('PULL_REQUEST_NOT_FOUND');
-    });
-
-    it('should not require PR evaluation when issue has accounting agent field value', async () => {
-      const result = await evaluator.evaluate({
-        url: 'https://github.com/user/repo/issues/1',
-        labels: [],
-        isPr: false,
-        agent: 'accounting',
-      });
-
-      expect(mockIssueRepository.findRelatedOpenPRs).not.toHaveBeenCalled();
-      expect(result.rejections).toHaveLength(0);
-      expect(result.approvedPrUrl).toBeNull();
-    });
-
-    it('should require PR evaluation for custom developer agent name when it matches issue agent', async () => {
-      mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([]);
-
-      const result = await evaluator.evaluate(
-        {
-          url: 'https://github.com/user/repo/issues/1',
-          labels: [],
-          isPr: false,
-          agent: 'my-developer',
-        },
-        [],
-        { developerAgentNames: ['my-developer'] },
-      );
-
-      expect(mockIssueRepository.findRelatedOpenPRs).toHaveBeenCalled();
-      expect(result.rejections).toHaveLength(1);
-      expect(result.rejections[0].type).toBe('PULL_REQUEST_NOT_FOUND');
-    });
-
-    it('should not require PR evaluation when issue agent does not match custom developer agent names', async () => {
-      const result = await evaluator.evaluate(
-        {
-          url: 'https://github.com/user/repo/issues/1',
-          labels: [],
-          isPr: false,
-          agent: 'other-agent',
-        },
-        [],
-        { developerAgentNames: ['my-developer'] },
-      );
-
-      expect(mockIssueRepository.findRelatedOpenPRs).not.toHaveBeenCalled();
-      expect(result.rejections).toHaveLength(0);
-      expect(result.approvedPrUrl).toBeNull();
-    });
-
-    it('should default to developer when developerAgentNames is null', async () => {
-      mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([]);
-
-      const result = await evaluator.evaluate(
-        {
-          url: 'https://github.com/user/repo/issues/1',
-          labels: [],
-          isPr: false,
-          agent: 'developer',
-        },
-        [],
-        { developerAgentNames: null },
-      );
-
-      expect(mockIssueRepository.findRelatedOpenPRs).toHaveBeenCalled();
-      expect(result.rejections).toHaveLength(1);
-      expect(result.rejections[0].type).toBe('PULL_REQUEST_NOT_FOUND');
-    });
-
-    it('should require PR evaluation when issue agent matches any of multiple developer agent names', async () => {
-      mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([]);
-
-      const result = await evaluator.evaluate(
-        {
-          url: 'https://github.com/user/repo/issues/1',
-          labels: [],
-          isPr: false,
-          agent: 'second-developer',
-        },
-        [],
-        { developerAgentNames: ['first-developer', 'second-developer'] },
-      );
-
-      expect(mockIssueRepository.findRelatedOpenPRs).toHaveBeenCalled();
-      expect(result.rejections).toHaveLength(1);
-      expect(result.rejections[0].type).toBe('PULL_REQUEST_NOT_FOUND');
-    });
-
-    it('should not require PR evaluation when issue agent does not match any of multiple developer agent names', async () => {
-      const result = await evaluator.evaluate(
-        {
-          url: 'https://github.com/user/repo/issues/1',
-          labels: [],
-          isPr: false,
-          agent: 'chore',
-        },
-        [],
-        { developerAgentNames: ['first-developer', 'second-developer'] },
-      );
-
-      expect(mockIssueRepository.findRelatedOpenPRs).not.toHaveBeenCalled();
-      expect(result.rejections).toHaveLength(0);
-      expect(result.approvedPrUrl).toBeNull();
-    });
-
-    it('should require PR evaluation and reject CI failure when issue agent is pr-reviewer', async () => {
-      mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-        createReadyPr('https://github.com/user/repo/pull/10', {
-          isPassedAllCiJob: false,
-          isCiStateSuccess: false,
-        }),
-      ]);
-
-      const result = await evaluator.evaluate({
-        url: 'https://github.com/user/repo/issues/1',
-        labels: [],
-        isPr: false,
-        agent: 'pr-reviewer',
-      });
-
-      expect(mockIssueRepository.findRelatedOpenPRs).toHaveBeenCalled();
-      expect(result.rejections.map((r) => r.type)).toContain(
-        'ANY_CI_JOB_FAILED_OR_IN_PROGRESS',
-      );
-    });
-
-    it('should require PR evaluation and reject conflict when issue agent is pr-reviewer', async () => {
-      mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-        createReadyPr('https://github.com/user/repo/pull/10', {
-          isConflicted: true,
-        }),
-      ]);
-
-      const result = await evaluator.evaluate({
-        url: 'https://github.com/user/repo/issues/1',
-        labels: [],
-        isPr: false,
-        agent: 'pr-reviewer',
-      });
-
-      expect(mockIssueRepository.findRelatedOpenPRs).toHaveBeenCalled();
-      expect(result.rejections.map((r) => r.type)).toContain(
-        'PULL_REQUEST_CONFLICTED',
-      );
-    });
-
-    it('should require PR evaluation and approve when issue agent is pr-reviewer with passing CI and no conflict', async () => {
-      mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-        createReadyPr('https://github.com/user/repo/pull/10'),
-      ]);
-
-      const result = await evaluator.evaluate({
-        url: 'https://github.com/user/repo/issues/1',
-        labels: [],
-        isPr: false,
-        agent: 'pr-reviewer',
-      });
-
-      expect(mockIssueRepository.findRelatedOpenPRs).toHaveBeenCalled();
-      expect(result.rejections).toHaveLength(0);
-      expect(result.approvedPrUrl).toBe('https://github.com/user/repo/pull/10');
-    });
-
-    it('should still require PR evaluation for pr-reviewer when custom developerAgentNames is set', async () => {
-      mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-        createReadyPr('https://github.com/user/repo/pull/10', {
-          isPassedAllCiJob: false,
-          isCiStateSuccess: false,
-        }),
-      ]);
-
-      const result = await evaluator.evaluate(
-        {
-          url: 'https://github.com/user/repo/issues/1',
-          labels: [],
-          isPr: false,
-          agent: 'pr-reviewer',
-        },
-        [],
-        { developerAgentNames: ['my-developer'] },
-      );
-
-      expect(mockIssueRepository.findRelatedOpenPRs).toHaveBeenCalled();
-      expect(result.rejections.map((r) => r.type)).toContain(
-        'ANY_CI_JOB_FAILED_OR_IN_PROGRESS',
-      );
-    });
-
-    it('should not reject for draft state when issue has non-e2e category label', async () => {
-      const result = await evaluator.evaluate({
-        url: 'https://github.com/user/repo/issues/1',
-        labels: ['category:frontend'],
-        isPr: false,
-      });
-
-      expect(mockIssueRepository.findRelatedOpenPRs).not.toHaveBeenCalled();
-      expect(result.rejections).toHaveLength(0);
-    });
-
-    it('should reject with PULL_REQUEST_NOT_FOUND when a normal issue has no related PR', async () => {
-      mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([]);
-
-      const result = await evaluator.evaluate({
-        url: 'https://github.com/user/repo/issues/1',
-        labels: [],
-        isPr: false,
-      });
-
-      expect(mockIssueRepository.findRelatedOpenPRs).toHaveBeenCalledWith(
-        'https://github.com/user/repo/issues/1',
-      );
-      expect(result.rejections).toHaveLength(1);
-      expect(result.rejections[0].type).toBe('PULL_REQUEST_NOT_FOUND');
-      expect(result.approvedPrUrl).toBeNull();
-    });
-
-    it('should not reject with PULL_REQUEST_NOT_FOUND when issue has a label listed in labelsAsLlmAgentName', async () => {
-      const result = await evaluator.evaluate(
-        {
-          url: 'https://github.com/user/repo/issues/1',
-          labels: ['story'],
-          isPr: false,
-        },
-        ['story'],
-      );
-
-      expect(mockIssueRepository.findRelatedOpenPRs).not.toHaveBeenCalled();
-      expect(result.rejections).toHaveLength(0);
-      expect(result.approvedPrUrl).toBeNull();
-    });
-
-    it('should still reject with PULL_REQUEST_NOT_FOUND when issue label is not in labelsAsLlmAgentName', async () => {
-      mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([]);
-
-      const result = await evaluator.evaluate(
-        {
-          url: 'https://github.com/user/repo/issues/1',
-          labels: ['story'],
-          isPr: false,
-        },
-        ['bug'],
-      );
-
-      expect(mockIssueRepository.findRelatedOpenPRs).toHaveBeenCalledWith(
-        'https://github.com/user/repo/issues/1',
-      );
-      expect(result.rejections).toHaveLength(1);
-      expect(result.rejections[0].type).toBe('PULL_REQUEST_NOT_FOUND');
-    });
-
-    describe('prebuilt relatedOpenPrUrls (in-memory derivation path)', () => {
-      it('should resolve PR status via getOpenPullRequest and not call findRelatedOpenPRs when relatedOpenPrUrls is provided', async () => {
-        mockIssueRepository.getOpenPullRequest.mockResolvedValue(
-          createReadyPr('https://github.com/user/repo/pull/7'),
-        );
-
-        const result = await evaluator.evaluate(
-          {
-            url: 'https://github.com/user/repo/issues/1',
-            labels: [],
-            isPr: false,
-          },
-          [],
-          { relatedOpenPrUrls: ['https://github.com/user/repo/pull/7'] },
-        );
-
-        expect(mockIssueRepository.findRelatedOpenPRs).not.toHaveBeenCalled();
-        expect(mockIssueRepository.getOpenPullRequest).toHaveBeenCalledWith(
-          'https://github.com/user/repo/pull/7',
-        );
-        expect(result.rejections).toHaveLength(0);
-        expect(result.approvedPrUrl).toBe(
-          'https://github.com/user/repo/pull/7',
-        );
-      });
-
-      it('should reject with PULL_REQUEST_NOT_FOUND when relatedOpenPrUrls is empty', async () => {
-        const result = await evaluator.evaluate(
-          {
-            url: 'https://github.com/user/repo/issues/1',
-            labels: [],
-            isPr: false,
-          },
-          [],
-          { relatedOpenPrUrls: [] },
-        );
-
-        expect(mockIssueRepository.findRelatedOpenPRs).not.toHaveBeenCalled();
-        expect(mockIssueRepository.getOpenPullRequest).not.toHaveBeenCalled();
-        expect(result.rejections).toHaveLength(1);
-        expect(result.rejections[0].type).toBe('PULL_REQUEST_NOT_FOUND');
-        expect(result.approvedPrUrl).toBeNull();
-      });
-
-      it('should drop a related URL whose PR is no longer open (getOpenPullRequest returns null), yielding PULL_REQUEST_NOT_FOUND', async () => {
-        mockIssueRepository.getOpenPullRequest.mockResolvedValue(null);
-
-        const result = await evaluator.evaluate(
-          {
-            url: 'https://github.com/user/repo/issues/1',
-            labels: [],
-            isPr: false,
-          },
-          [],
-          { relatedOpenPrUrls: ['https://github.com/user/repo/pull/9'] },
-        );
-
-        expect(mockIssueRepository.findRelatedOpenPRs).not.toHaveBeenCalled();
-        expect(mockIssueRepository.getOpenPullRequest).toHaveBeenCalledWith(
-          'https://github.com/user/repo/pull/9',
-        );
-        expect(result.rejections).toHaveLength(1);
-        expect(result.rejections[0].type).toBe('PULL_REQUEST_NOT_FOUND');
-      });
-
-      it('should reject with MULTIPLE_PULL_REQUESTS_FOUND when more than one related open PR resolves', async () => {
-        mockIssueRepository.getOpenPullRequest.mockImplementation(
-          (prUrl: string) => Promise.resolve(createReadyPr(prUrl)),
-        );
-
-        const result = await evaluator.evaluate(
-          {
-            url: 'https://github.com/user/repo/issues/1',
-            labels: [],
-            isPr: false,
-          },
-          [],
-          {
-            relatedOpenPrUrls: [
-              'https://github.com/user/repo/pull/1',
-              'https://github.com/user/repo/pull/2',
-            ],
-          },
-        );
-
-        expect(mockIssueRepository.findRelatedOpenPRs).not.toHaveBeenCalled();
-        expect(result.rejections).toHaveLength(1);
-        expect(result.rejections[0].type).toBe('MULTIPLE_PULL_REQUESTS_FOUND');
-      });
-
-      it('should de-duplicate repeated related URLs before resolving, mirroring findRelatedOpenPRs', async () => {
-        mockIssueRepository.getOpenPullRequest.mockResolvedValue(
-          createReadyPr('https://github.com/user/repo/pull/5'),
-        );
-
-        const result = await evaluator.evaluate(
-          {
-            url: 'https://github.com/user/repo/issues/1',
-            labels: [],
-            isPr: false,
-          },
-          [],
-          {
-            relatedOpenPrUrls: [
-              'https://github.com/user/repo/pull/5',
-              'https://github.com/user/repo/pull/5',
-            ],
-          },
-        );
-
-        expect(mockIssueRepository.getOpenPullRequest).toHaveBeenCalledTimes(1);
-        expect(result.rejections).toHaveLength(0);
-        expect(result.approvedPrUrl).toBe(
-          'https://github.com/user/repo/pull/5',
-        );
-      });
-
-      it('should produce the same draft rejection through the prebuilt path as through findRelatedOpenPRs', async () => {
-        mockIssueRepository.getOpenPullRequest.mockResolvedValue(
-          createReadyPr('https://github.com/user/repo/pull/3', {
-            isDraft: true,
-          }),
-        );
-
-        const result = await evaluator.evaluate(
-          {
-            url: 'https://github.com/user/repo/issues/1',
-            labels: [],
-            isPr: false,
-          },
-          [],
-          { relatedOpenPrUrls: ['https://github.com/user/repo/pull/3'] },
-        );
-
-        expect(result.rejections).toHaveLength(1);
-        expect(result.rejections[0].type).toBe('PULL_REQUEST_IS_DRAFT');
-        expect(result.approvedPrUrl).toBeNull();
-      });
-
-      it('should still use findRelatedOpenPRs when relatedOpenPrUrls is undefined (single-issue path unchanged)', async () => {
-        mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-          createReadyPr(),
-        ]);
-
-        const result = await evaluator.evaluate({
-          url: 'https://github.com/user/repo/issues/1',
-          labels: [],
-          isPr: false,
-        });
-
-        expect(mockIssueRepository.findRelatedOpenPRs).toHaveBeenCalledWith(
-          'https://github.com/user/repo/issues/1',
-        );
-        expect(result.rejections).toHaveLength(0);
-      });
-
-      it('should ignore relatedOpenPrUrls for a PR item (isPr=true) and use getOpenPullRequest on the PR URL itself', async () => {
-        mockIssueRepository.getOpenPullRequest.mockResolvedValue(
-          createReadyPr('https://github.com/user/repo/pull/10'),
-        );
-
-        const result = await evaluator.evaluate(
-          {
-            url: 'https://github.com/user/repo/pull/10',
-            labels: [],
-            isPr: true,
-          },
-          [],
-          { relatedOpenPrUrls: ['https://github.com/user/repo/pull/999'] },
-        );
-
-        expect(mockIssueRepository.findRelatedOpenPRs).not.toHaveBeenCalled();
-        expect(mockIssueRepository.getOpenPullRequest).toHaveBeenCalledWith(
-          'https://github.com/user/repo/pull/10',
-        );
-        expect(mockIssueRepository.getOpenPullRequest).not.toHaveBeenCalledWith(
-          'https://github.com/user/repo/pull/999',
-        );
-        expect(result.rejections).toHaveLength(0);
-      });
-    });
-
-    describe('transient getOpenPullRequest failure containment', () => {
-      let warnSpy: jest.SpyInstance;
-
-      beforeEach(() => {
-        warnSpy = jest.spyOn(console, 'warn').mockImplementation(() => {});
-      });
-
-      afterEach(() => {
-        warnSpy.mockRestore();
-      });
-
-      it('should not throw and should return no rejections when getOpenPullRequest rejects for a PR item (isPr=true)', async () => {
-        mockIssueRepository.getOpenPullRequest.mockRejectedValue(
-          new Error(
-            'GraphQL errors: [{"message":"Something went wrong while executing your query"}]',
-          ),
-        );
-
-        const result = await evaluator.evaluate({
-          url: 'https://github.com/user/repo/pull/10',
-          labels: [],
-          isPr: true,
-        });
-
-        expect(result.rejections).toHaveLength(0);
-        expect(result.approvedPrUrl).toBeNull();
-      });
-
-      it('should log one warning line containing the PR URL and error message when getOpenPullRequest rejects for a PR item', async () => {
-        mockIssueRepository.getOpenPullRequest.mockRejectedValue(
-          new Error('Something went wrong while executing your query'),
-        );
-
-        await evaluator.evaluate({
-          url: 'https://github.com/user/repo/pull/10',
-          labels: [],
-          isPr: true,
-        });
-
-        expect(warnSpy).toHaveBeenCalledTimes(1);
-        expect(warnSpy).toHaveBeenCalledWith(
-          expect.stringContaining('https://github.com/user/repo/pull/10'),
-        );
-        expect(warnSpy).toHaveBeenCalledWith(
-          expect.stringContaining(
-            'Something went wrong while executing your query',
-          ),
-        );
-      });
-
-      it('should skip the failing URL, log a warning, and still evaluate the healthy PR in the same batch (prebuilt path)', async () => {
-        mockIssueRepository.getOpenPullRequest.mockImplementation(
-          (prUrl: string) =>
-            prUrl === 'https://github.com/user/repo/pull/8'
-              ? Promise.reject(
-                  new Error('Something went wrong while executing your query'),
-                )
-              : Promise.resolve(createReadyPr(prUrl)),
-        );
-
-        const result = await evaluator.evaluate(
-          {
-            url: 'https://github.com/user/repo/issues/1',
-            labels: [],
-            isPr: false,
-          },
-          [],
-          {
-            relatedOpenPrUrls: [
-              'https://github.com/user/repo/pull/8',
-              'https://github.com/user/repo/pull/9',
-            ],
-          },
-        );
-
-        expect(warnSpy).toHaveBeenCalledTimes(1);
-        expect(warnSpy).toHaveBeenCalledWith(
-          expect.stringContaining('https://github.com/user/repo/pull/8'),
-        );
-        expect(result.rejections).toHaveLength(0);
-        expect(result.approvedPrUrl).toBe(
-          'https://github.com/user/repo/pull/9',
-        );
-      });
-
-      it('should not reject with PULL_REQUEST_NOT_FOUND when the only related URL fails transiently (state unknown, skipped)', async () => {
-        mockIssueRepository.getOpenPullRequest.mockRejectedValue(
-          new Error('Something went wrong while executing your query'),
-        );
-
-        const result = await evaluator.evaluate(
-          {
-            url: 'https://github.com/user/repo/issues/1',
-            labels: [],
-            isPr: false,
-          },
-          [],
-          { relatedOpenPrUrls: ['https://github.com/user/repo/pull/8'] },
-        );
-
-        expect(warnSpy).toHaveBeenCalledTimes(1);
-        expect(result.rejections).toHaveLength(0);
-        expect(result.approvedPrUrl).toBeNull();
-      });
-
-      it('should still evaluate the healthy remaining PR normally when it has a rejection reason (draft) after another URL fails', async () => {
-        mockIssueRepository.getOpenPullRequest.mockImplementation(
-          (prUrl: string) =>
-            prUrl === 'https://github.com/user/repo/pull/8'
-              ? Promise.reject(
-                  new Error('Something went wrong while executing your query'),
-                )
-              : Promise.resolve(createReadyPr(prUrl, { isDraft: true })),
-        );
-
-        const result = await evaluator.evaluate(
-          {
-            url: 'https://github.com/user/repo/issues/1',
-            labels: [],
-            isPr: false,
-          },
-          [],
-          {
-            relatedOpenPrUrls: [
-              'https://github.com/user/repo/pull/8',
-              'https://github.com/user/repo/pull/9',
-            ],
-          },
-        );
-
-        expect(result.rejections).toHaveLength(1);
-        expect(result.rejections[0].type).toBe('PULL_REQUEST_IS_DRAFT');
-        expect(result.approvedPrUrl).toBeNull();
-      });
-
-      it('should handle a non-Error rejection value without throwing', async () => {
-        mockIssueRepository.getOpenPullRequest.mockRejectedValue(
-          'string failure',
-        );
-
-        const result = await evaluator.evaluate({
-          url: 'https://github.com/user/repo/pull/10',
-          labels: [],
-          isPr: true,
-        });
-
-        expect(warnSpy).toHaveBeenCalledTimes(1);
-        expect(warnSpy).toHaveBeenCalledWith(
-          expect.stringContaining('string failure'),
-        );
-        expect(result.rejections).toHaveLength(0);
-      });
-    });
-
-    describe('detectConflictEvenIfEvaluationSkipped option', () => {
-      it('should call findRelatedOpenPRs and return PULL_REQUEST_CONFLICTED when relatedOpenPrUrls is null and PR is conflicted', async () => {
-        mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-          createReadyPr('https://github.com/user/repo/pull/1', {
-            isConflicted: true,
-          }),
-        ]);
-
-        const result = await evaluator.evaluate(
-          {
-            url: 'https://github.com/user/repo/issues/1',
-            labels: [],
-            isPr: false,
-            agent: 'chore',
-          },
-          [],
-          {
-            detectConflictEvenIfEvaluationSkipped: true,
-            relatedOpenPrUrls: null,
-          },
-        );
-
-        expect(mockIssueRepository.findRelatedOpenPRs).toHaveBeenCalledWith(
-          'https://github.com/user/repo/issues/1',
-        );
-        expect(result.rejections).toHaveLength(1);
-        expect(result.rejections[0].type).toBe('PULL_REQUEST_CONFLICTED');
-        expect(result.approvedPrUrl).toBeNull();
-      });
-
-      it('should call findRelatedOpenPRs and return no rejections when relatedOpenPrUrls is null and no PR is conflicted', async () => {
-        mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-          createReadyPr('https://github.com/user/repo/pull/1'),
-        ]);
-
-        const result = await evaluator.evaluate(
-          {
-            url: 'https://github.com/user/repo/issues/1',
-            labels: [],
-            isPr: false,
-            agent: 'chore',
-          },
-          [],
-          {
-            detectConflictEvenIfEvaluationSkipped: true,
-            relatedOpenPrUrls: null,
-          },
-        );
-
-        expect(mockIssueRepository.findRelatedOpenPRs).toHaveBeenCalledWith(
-          'https://github.com/user/repo/issues/1',
-        );
-        expect(result.rejections).toHaveLength(0);
-        expect(result.approvedPrUrl).toBeNull();
-      });
-
-      it('should use resolveOpenPrsFromUrls path and not call findRelatedOpenPRs when relatedOpenPrUrls is provided', async () => {
-        mockIssueRepository.getOpenPullRequest.mockResolvedValue(
-          createReadyPr('https://github.com/user/repo/pull/5', {
-            isConflicted: true,
-          }),
-        );
-
-        const result = await evaluator.evaluate(
-          {
-            url: 'https://github.com/user/repo/issues/1',
-            labels: [],
-            isPr: false,
-            agent: 'chore',
-          },
-          [],
-          {
-            detectConflictEvenIfEvaluationSkipped: true,
-            relatedOpenPrUrls: ['https://github.com/user/repo/pull/5'],
-          },
-        );
-
-        expect(mockIssueRepository.findRelatedOpenPRs).not.toHaveBeenCalled();
-        expect(mockIssueRepository.getOpenPullRequest).toHaveBeenCalledWith(
-          'https://github.com/user/repo/pull/5',
-        );
-        expect(result.rejections).toHaveLength(1);
-        expect(result.rejections[0].type).toBe('PULL_REQUEST_CONFLICTED');
-      });
-
-      it('should emit only PULL_REQUEST_CONFLICTED and not PULL_REQUEST_IS_DRAFT or other types when evaluation is skipped', async () => {
-        mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-          createReadyPr('https://github.com/user/repo/pull/1', {
-            isConflicted: true,
-            isDraft: true,
-            isPassedAllCiJob: false,
-            isCiStateSuccess: false,
-          }),
-        ]);
-
-        const result = await evaluator.evaluate(
-          {
-            url: 'https://github.com/user/repo/issues/1',
-            labels: [],
-            isPr: false,
-            agent: 'chore',
-          },
-          [],
-          {
-            detectConflictEvenIfEvaluationSkipped: true,
-            relatedOpenPrUrls: null,
-          },
-        );
-
-        expect(result.rejections).toHaveLength(1);
-        expect(result.rejections[0].type).toBe('PULL_REQUEST_CONFLICTED');
-        expect(result.rejections.map((r) => r.type)).not.toContain(
-          'PULL_REQUEST_IS_DRAFT',
-        );
-        expect(result.rejections.map((r) => r.type)).not.toContain(
-          'ANY_CI_JOB_FAILED_OR_IN_PROGRESS',
-        );
-      });
-
-      it('should resolve the pull request item itself and not call findRelatedOpenPRs when the item is a pull request', async () => {
-        mockIssueRepository.findRelatedOpenPRs.mockRejectedValue(
-          new Error(
-            'findRelatedOpenPRs only supports issue URLs, not pull request URLs',
-          ),
-        );
-        mockIssueRepository.getOpenPullRequest.mockResolvedValue(
-          createReadyPr('https://github.com/user/repo/pull/7', {
-            isConflicted: true,
-          }),
-        );
-
-        const result = await evaluator.evaluate(
-          {
-            url: 'https://github.com/user/repo/pull/7',
-            labels: [],
-            isPr: true,
-            agent: 'chore',
-          },
-          [],
-          {
-            detectConflictEvenIfEvaluationSkipped: true,
-            relatedOpenPrUrls: null,
-          },
-        );
-
-        expect(mockIssueRepository.findRelatedOpenPRs).not.toHaveBeenCalled();
-        expect(mockIssueRepository.getOpenPullRequest).toHaveBeenCalledWith(
-          'https://github.com/user/repo/pull/7',
-        );
-        expect(result.rejections).toHaveLength(1);
-        expect(result.rejections[0].type).toBe('PULL_REQUEST_CONFLICTED');
-      });
-
-      it('should return no rejections when the pull request item itself is not conflicted', async () => {
-        mockIssueRepository.findRelatedOpenPRs.mockRejectedValue(
-          new Error(
-            'findRelatedOpenPRs only supports issue URLs, not pull request URLs',
-          ),
-        );
-        mockIssueRepository.getOpenPullRequest.mockResolvedValue(
-          createReadyPr('https://github.com/user/repo/pull/7'),
-        );
-
-        const result = await evaluator.evaluate(
-          {
-            url: 'https://github.com/user/repo/pull/7',
-            labels: [],
-            isPr: true,
-            agent: 'chore',
-          },
-          [],
-          {
-            detectConflictEvenIfEvaluationSkipped: true,
-            relatedOpenPrUrls: null,
-          },
-        );
-
-        expect(mockIssueRepository.findRelatedOpenPRs).not.toHaveBeenCalled();
-        expect(result.rejections).toHaveLength(0);
-      });
-
-      it('should return no rejections when resolving the pull request item fails transiently', async () => {
-        mockIssueRepository.getOpenPullRequest.mockRejectedValue(
-          new Error('Something went wrong while executing your query.'),
-        );
-
-        const result = await evaluator.evaluate(
-          {
-            url: 'https://github.com/user/repo/pull/7',
-            labels: [],
-            isPr: true,
-            agent: 'chore',
-          },
-          [],
-          {
-            detectConflictEvenIfEvaluationSkipped: true,
-            relatedOpenPrUrls: null,
-          },
-        );
-
-        expect(result.rejections).toHaveLength(0);
-      });
-    });
-
-    describe('change-target-must: label behavior', () => {
-      const prUrl = 'https://github.com/user/repo/pull/1';
-
-      it('should not reject when required path contains at least one changed file', async () => {
-        mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-          createReadyPr(prUrl),
-        ]);
-        mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-          'src/domain/entities/Foo.ts',
-          'src/adapter/Bar.ts',
-        ]);
-
-        const result = await evaluator.evaluate({
-          url: 'https://github.com/user/repo/issues/1',
-          labels: ['change-target-must:src/domain'],
-          isPr: false,
-        });
-
-        expect(result.rejections).toHaveLength(0);
-        expect(result.approvedPrUrl).toBe(prUrl);
-        expect(
-          mockIssueRepository.requestChangesWithInlineComment,
-        ).not.toHaveBeenCalled();
-      });
-
-      it('should reject with CHANGE_TARGET_MUST_PATH_NOT_CHANGED and post inline comment when required path has no changed file', async () => {
-        mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-          createReadyPr(prUrl),
-        ]);
-        mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-          'src/adapter/Bar.ts',
-        ]);
-
-        const result = await evaluator.evaluate({
-          url: 'https://github.com/user/repo/issues/1',
-          labels: ['change-target-must:src/domain'],
-          isPr: false,
-        });
-
-        expect(result.rejections).toHaveLength(1);
-        expect(result.rejections[0].type).toBe(
-          'CHANGE_TARGET_MUST_PATH_NOT_CHANGED',
-        );
-        expect(result.rejections[0].detail).toContain('src/domain');
-        expect(result.approvedPrUrl).toBeNull();
-        expect(
-          mockIssueRepository.requestChangesWithInlineComment,
-        ).toHaveBeenCalledWith(
-          prUrl,
-          'src/adapter/Bar.ts',
-          expect.stringContaining('src/domain'),
-        );
-      });
-
-      it('should post PR-level comment as fallback when PR has no changed files', async () => {
-        mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-          createReadyPr(prUrl),
-        ]);
-        mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue(
-          [],
-        );
-
-        const result = await evaluator.evaluate({
-          url: 'https://github.com/user/repo/issues/1',
-          labels: ['change-target-must:src/domain'],
-          isPr: false,
-        });
-
-        expect(result.rejections).toHaveLength(1);
-        expect(result.rejections[0].type).toBe(
-          'CHANGE_TARGET_MUST_PATH_NOT_CHANGED',
-        );
-        expect(
-          mockIssueRepository.requestChangesWithInlineComment,
-        ).toHaveBeenCalledWith(prUrl, null, expect.any(String));
-      });
-
-      it('should match boundary-safely so that change-target-must:foo matches foo/bar.ts but not foobar/baz.ts', async () => {
-        mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-          createReadyPr(prUrl),
-        ]);
-        mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-          'foobar/baz.ts',
-        ]);
-
-        const result = await evaluator.evaluate({
-          url: 'https://github.com/user/repo/issues/1',
-          labels: ['change-target-must:foo'],
-          isPr: false,
-        });
-
-        expect(result.rejections).toHaveLength(1);
-        expect(result.rejections[0].type).toBe(
-          'CHANGE_TARGET_MUST_PATH_NOT_CHANGED',
-        );
-        expect(
-          mockIssueRepository.requestChangesWithInlineComment,
-        ).toHaveBeenCalledWith(prUrl, 'foobar/baz.ts', expect.any(String));
-      });
-
-      it('should not reject when changed file is exactly the required path (boundary-safe exact match)', async () => {
-        mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-          createReadyPr(prUrl),
-        ]);
-        mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-          'foo',
-        ]);
-
-        const result = await evaluator.evaluate({
-          url: 'https://github.com/user/repo/issues/1',
-          labels: ['change-target-must:foo'],
-          isPr: false,
-        });
-
-        expect(result.rejections).toHaveLength(0);
-        expect(result.approvedPrUrl).toBe(prUrl);
-      });
-
-      it('should not reject when changed file is under the change-target-must path', async () => {
-        mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-          createReadyPr(prUrl),
-        ]);
-        mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-          'src/domain/entities/Foo.ts',
-        ]);
-
-        const result = await evaluator.evaluate({
-          url: 'https://github.com/user/repo/issues/1',
-          labels: ['change-target-must:src/domain'],
-          isPr: false,
-        });
-
-        expect(result.rejections).toHaveLength(0);
-        expect(result.approvedPrUrl).toBe(prUrl);
-      });
-
-      it('should normalize leading slash in change-target-must label path', async () => {
-        mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([
-          createReadyPr(prUrl),
-        ]);
-        mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-          'src/domain/entities/Foo.ts',
-        ]);
-
-        const result = await evaluator.evaluate({
-          url: 'https://github.com/user/repo/issues/1',
-          labels: ['change-target-must:/src/domain'],
-          isPr: false,
-        });
-
-        expect(result.rejections).toHaveLength(0);
-        expect(result.approvedPrUrl).toBe(prUrl);
-      });
-    });
-  });
-});