github-issue-tower-defence-management 1.60.1 → 1.63.1

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

@@ -187,6 +187,7 @@ describe('StartPreparationUseCase', () => {
       url: 'https://github.com/user/repo/pull/42',
       branchName: 'i1',
       createdAt: new Date('2024-01-01'),
+      isDraft: false,
       isConflicted: false,
       isPassedAllCiJob: false,
       isCiStateSuccess: false,
@@ -247,6 +248,7 @@ describe('StartPreparationUseCase', () => {
       url: 'https://github.com/user/repo/pull/354',
       branchName: 'dependabot/npm_and_yarn/multi-cc382f683c',
       createdAt: new Date('2024-01-01'),
+      isDraft: false,
       isConflicted: false,
       isPassedAllCiJob: false,
       isCiStateSuccess: false,
@@ -343,6 +345,7 @@ describe('StartPreparationUseCase', () => {
       url: 'https://github.com/user/repo/pull/999',
       branchName: null,
       createdAt: new Date('2024-01-01'),
+      isDraft: false,
       isConflicted: false,
       isPassedAllCiJob: false,
       isCiStateSuccess: false,
@@ -389,6 +392,7 @@ describe('StartPreparationUseCase', () => {
       url: 'https://github.com/user/repo/pull/999',
       branchName: 'evil$(rm -rf /)',
       createdAt: new Date('2024-01-01'),
+      isDraft: false,
       isConflicted: false,
       isPassedAllCiJob: false,
       isCiStateSuccess: false,
@@ -431,6 +435,7 @@ describe('StartPreparationUseCase', () => {
       url: 'https://github.com/user/repo/pull/42',
       branchName: 'i1',
       createdAt: new Date('2024-01-01T00:00:00Z'),
+      isDraft: false,
       isConflicted: false,
       isPassedAllCiJob: false,
       isCiStateSuccess: false,
@@ -442,6 +447,7 @@ describe('StartPreparationUseCase', () => {
       url: 'https://github.com/user/repo/pull/43',
       branchName: 'i1-fix',
       createdAt: new Date('2024-01-02T00:00:00Z'),
+      isDraft: false,
       isConflicted: false,
       isPassedAllCiJob: false,
       isCiStateSuccess: false,
@@ -519,6 +525,7 @@ describe('StartPreparationUseCase', () => {
       url: 'https://github.com/user/repo/pull/42',
       branchName: null,
       createdAt: new Date('2024-01-01T00:00:00Z'),
+      isDraft: false,
       isConflicted: false,
       isPassedAllCiJob: false,
       isCiStateSuccess: false,
@@ -530,6 +537,7 @@ describe('StartPreparationUseCase', () => {
       url: 'https://github.com/user/repo/pull/43',
       branchName: 'i1-fix',
       createdAt: new Date('2024-01-02T00:00:00Z'),
+      isDraft: false,
       isConflicted: false,
       isPassedAllCiJob: false,
       isCiStateSuccess: false,
@@ -592,6 +600,7 @@ describe('StartPreparationUseCase', () => {
       url: 'https://github.com/user/repo/pull/42',
       branchName: null,
       createdAt: new Date('2024-01-01'),
+      isDraft: false,
       isConflicted: false,
       isPassedAllCiJob: false,
       isCiStateSuccess: false,
@@ -1187,6 +1196,123 @@ describe('StartPreparationUseCase', () => {
     expect(mockIssueRepository.updateStatus.mock.calls).toHaveLength(6);
     expect(mockLocalCommandRunner.runCommand.mock.calls).toHaveLength(6);
   });
+  it('should allow six preparing processes per available Claude OAuth token when maximumPreparingIssuesCount is null', async () => {
+    const awaitingIssues: Issue[] = Array.from({ length: 20 }, (_, i) =>
+      createMockIssue({
+        url: `url${i + 1}`,
+        title: `Issue ${i + 1}`,
+        labels: [],
+        status: 'Awaiting Workspace',
+        number: i + 1,
+        itemId: `item-${i + 1}`,
+      }),
+    );
+    mockProjectRepository.getByUrl.mockResolvedValue(mockProject);
+    mockIssueRepository.getStoryObjectMap.mockResolvedValue(
+      createMockStoryObjectMap(awaitingIssues),
+    );
+    mockLocalCommandRunner.runCommand.mockResolvedValue({
+      stdout: '',
+      stderr: '',
+      exitCode: 0,
+    });
+    mockClaudeTokenUsageRepository.getAvailableTokenUsages.mockResolvedValue([
+      {
+        name: 'token-a',
+        token: 'token-a',
+        fiveHourUtilization: 0,
+        blocked: false,
+        rejected: false,
+        modelWeeklyLimits: {},
+      },
+      {
+        name: 'token-b',
+        token: 'token-b',
+        fiveHourUtilization: 0,
+        blocked: false,
+        rejected: false,
+        modelWeeklyLimits: {},
+      },
+    ]);
+
+    await useCase.run({
+      projectUrl: 'https://github.com/user/repo',
+      defaultAgentName: 'agent1',
+      defaultLlmModelName: 'claude-sonnet-4-6',
+      defaultLlmAgentName: null,
+      configFilePath: '/path/to/config.yml',
+      maximumPreparingIssuesCount: null,
+      utilizationPercentageThreshold: 90,
+      allowedIssueAuthors: null,
+      codexHomeCandidates: null,
+      allowIssueCacheMinutes: 0,
+    });
+
+    expect(mockLocalCommandRunner.runCommand.mock.calls).toHaveLength(12);
+    const spawnedTokens = mockLocalCommandRunner.runCommand.mock.calls.map(
+      (call) => call[2]?.env?.CLAUDE_CODE_OAUTH_TOKEN,
+    );
+    expect(spawnedTokens.filter((token) => token === 'token-a')).toHaveLength(
+      6,
+    );
+    expect(spawnedTokens.filter((token) => token === 'token-b')).toHaveLength(
+      6,
+    );
+  });
+  it('should cap configured maximumPreparingIssuesCount to six per available Claude OAuth token', async () => {
+    const awaitingIssues: Issue[] = Array.from({ length: 20 }, (_, i) =>
+      createMockIssue({
+        url: `url${i + 1}`,
+        title: `Issue ${i + 1}`,
+        labels: [],
+        status: 'Awaiting Workspace',
+        number: i + 1,
+        itemId: `item-${i + 1}`,
+      }),
+    );
+    mockProjectRepository.getByUrl.mockResolvedValue(mockProject);
+    mockIssueRepository.getStoryObjectMap.mockResolvedValue(
+      createMockStoryObjectMap(awaitingIssues),
+    );
+    mockLocalCommandRunner.runCommand.mockResolvedValue({
+      stdout: '',
+      stderr: '',
+      exitCode: 0,
+    });
+    mockClaudeTokenUsageRepository.getAvailableTokenUsages.mockResolvedValue([
+      {
+        name: 'token-a',
+        token: 'token-a',
+        fiveHourUtilization: 0,
+        blocked: false,
+        rejected: false,
+        modelWeeklyLimits: {},
+      },
+      {
+        name: 'token-b',
+        token: 'token-b',
+        fiveHourUtilization: 0,
+        blocked: false,
+        rejected: false,
+        modelWeeklyLimits: {},
+      },
+    ]);
+
+    await useCase.run({
+      projectUrl: 'https://github.com/user/repo',
+      defaultAgentName: 'agent1',
+      defaultLlmModelName: 'claude-sonnet-4-6',
+      defaultLlmAgentName: null,
+      configFilePath: '/path/to/config.yml',
+      maximumPreparingIssuesCount: 20,
+      utilizationPercentageThreshold: 90,
+      allowedIssueAuthors: null,
+      codexHomeCandidates: null,
+      allowIssueCacheMinutes: 0,
+    });
+
+    expect(mockLocalCommandRunner.runCommand.mock.calls).toHaveLength(12);
+  });
 
   it('should not skip issues from repositories with workflow blockers', async () => {
     const blockerIssue = createMockIssue({
@@ -2149,6 +2275,7 @@ describe('StartPreparationUseCase', () => {
     });
     mockClaudeTokenUsageRepository.getAvailableTokenUsages.mockResolvedValue([
       {
+        name: 'token-a',
         token: 'token-a',
         fiveHourUtilization: 0,
         blocked: false,
@@ -2223,6 +2350,7 @@ describe('StartPreparationUseCase', () => {
     });
     mockClaudeTokenUsageRepository.getAvailableTokenUsages.mockResolvedValue([
       {
+        name: 'token-a',
         token: 'token-a',
         fiveHourUtilization: 0,
         blocked: false,
@@ -2230,6 +2358,7 @@ describe('StartPreparationUseCase', () => {
         modelWeeklyLimits: {},
       },
       {
+        name: 'token-b',
         token: 'token-b',
         fiveHourUtilization: 0,
         blocked: false,
@@ -2334,6 +2463,7 @@ describe('StartPreparationUseCase', () => {
     });
     mockClaudeTokenUsageRepository.getAvailableTokenUsages.mockResolvedValue([
       {
+        name: 'token-high',
         token: 'token-high',
         fiveHourUtilization: 0.8,
         blocked: false,
@@ -2341,6 +2471,7 @@ describe('StartPreparationUseCase', () => {
         modelWeeklyLimits: {},
       },
       {
+        name: 'token-low',
         token: 'token-low',
         fiveHourUtilization: 0.1,
         blocked: false,
@@ -2391,6 +2522,7 @@ describe('StartPreparationUseCase', () => {
     });
     mockClaudeTokenUsageRepository.getAvailableTokenUsages.mockResolvedValue([
       {
+        name: 'token-blocked',
         token: 'token-blocked',
         fiveHourUtilization: 0.05,
         blocked: true,
@@ -2398,6 +2530,7 @@ describe('StartPreparationUseCase', () => {
         modelWeeklyLimits: {},
       },
       {
+        name: 'token-ok',
         token: 'token-ok',
         fiveHourUtilization: 0.5,
         blocked: false,
@@ -2448,6 +2581,7 @@ describe('StartPreparationUseCase', () => {
     });
     mockClaudeTokenUsageRepository.getAvailableTokenUsages.mockResolvedValue([
       {
+        name: 'token-a',
         token: 'token-a',
         fiveHourUtilization: 0.05,
         blocked: true,
@@ -2455,6 +2589,7 @@ describe('StartPreparationUseCase', () => {
         modelWeeklyLimits: {},
       },
       {
+        name: 'token-b',
         token: 'token-b',
         fiveHourUtilization: 0.08,
         blocked: true,
@@ -2491,7 +2626,72 @@ describe('StartPreparationUseCase', () => {
     consoleWarnSpy.mockRestore();
   });
 
-  it('should return all tokens sorted ascending when all are below the threshold', async () => {
+  it('should skip preparation when every configured token is at or above 95 percent 5h utilization', async () => {
+    const awaitingIssue = createMockIssue({
+      url: 'url1',
+      title: 'Issue 1',
+      labels: ['category:impl'],
+      status: 'Awaiting Workspace',
+      number: 1,
+      itemId: 'item-1',
+    });
+    mockProjectRepository.getByUrl.mockResolvedValue(mockProject);
+    mockIssueRepository.getStoryObjectMap.mockResolvedValue(
+      createMockStoryObjectMap([awaitingIssue]),
+    );
+    mockLocalCommandRunner.runCommand.mockResolvedValue({
+      stdout: '',
+      stderr: '',
+      exitCode: 0,
+    });
+    mockClaudeTokenUsageRepository.getAvailableTokenUsages.mockResolvedValue([
+      {
+        name: 'token-a',
+        token: 'token-a',
+        fiveHourUtilization: 0.95,
+        blocked: false,
+        rejected: false,
+        modelWeeklyLimits: {},
+      },
+      {
+        name: 'token-b',
+        token: 'token-b',
+        fiveHourUtilization: 0.97,
+        blocked: false,
+        rejected: false,
+        modelWeeklyLimits: {},
+      },
+    ]);
+    const consoleWarnSpy = jest
+      .spyOn(console, 'warn')
+      .mockImplementation(() => {});
+
+    await useCase.run({
+      projectUrl: 'https://github.com/user/repo',
+      defaultAgentName: 'agent1',
+      defaultLlmModelName: 'claude-opus',
+      defaultLlmAgentName: null,
+      configFilePath: '/path/to/config.yml',
+      maximumPreparingIssuesCount: null,
+      utilizationPercentageThreshold: 90,
+      allowedIssueAuthors: null,
+      codexHomeCandidates: null,
+      allowIssueCacheMinutes: 0,
+    });
+
+    expect(
+      mockClaudeTokenUsageRepository.ensureObservable.mock.calls,
+    ).toHaveLength(0);
+    expect(mockProjectRepository.getByUrl).not.toHaveBeenCalled();
+    expect(mockIssueRepository.updateStatus.mock.calls).toHaveLength(0);
+    expect(mockLocalCommandRunner.runCommand.mock.calls).toHaveLength(0);
+    expect(consoleWarnSpy).toHaveBeenCalledWith(
+      expect.stringContaining('5h utilization >= 95%'),
+    );
+    consoleWarnSpy.mockRestore();
+  });
+
+  it('should return all tokens sorted ascending when all have full process capacity', async () => {
     const awaitingIssues: Issue[] = [
       createMockIssue({
         url: 'url1',
@@ -2529,6 +2729,7 @@ describe('StartPreparationUseCase', () => {
     });
     mockClaudeTokenUsageRepository.getAvailableTokenUsages.mockResolvedValue([
       {
+        name: 'token-mid',
         token: 'token-mid',
         fiveHourUtilization: 0.5,
         blocked: false,
@@ -2536,6 +2737,7 @@ describe('StartPreparationUseCase', () => {
         modelWeeklyLimits: {},
       },
       {
+        name: 'token-low',
         token: 'token-low',
         fiveHourUtilization: 0.1,
         blocked: false,
@@ -2543,6 +2745,7 @@ describe('StartPreparationUseCase', () => {
         modelWeeklyLimits: {},
       },
       {
+        name: 'token-high',
         token: 'token-high',
         fiveHourUtilization: 0.8,
         blocked: false,
@@ -2576,18 +2779,20 @@ describe('StartPreparationUseCase', () => {
     });
   });
 
-  it('should exclude a token whose 5h utilization is at or above the threshold', async () => {
-    const awaitingIssue = createMockIssue({
-      url: 'url1',
-      title: 'Issue 1',
-      labels: ['category:impl'],
-      status: 'Awaiting Workspace',
-      number: 1,
-      itemId: 'item-1',
-    });
+  it('should reduce token process capacity exponentially above 80 percent 5h utilization', async () => {
+    const awaitingIssues: Issue[] = Array.from({ length: 10 }, (_, i) =>
+      createMockIssue({
+        url: `url${i + 1}`,
+        title: `Issue ${i + 1}`,
+        labels: ['category:impl'],
+        status: 'Awaiting Workspace',
+        number: i + 1,
+        itemId: `item-${i + 1}`,
+      }),
+    );
     mockProjectRepository.getByUrl.mockResolvedValue(mockProject);
     mockIssueRepository.getStoryObjectMap.mockResolvedValue(
-      createMockStoryObjectMap([awaitingIssue]),
+      createMockStoryObjectMap(awaitingIssues),
     );
     mockLocalCommandRunner.runCommand.mockResolvedValue({
       stdout: '',
@@ -2596,15 +2801,33 @@ describe('StartPreparationUseCase', () => {
     });
     mockClaudeTokenUsageRepository.getAvailableTokenUsages.mockResolvedValue([
       {
-        token: 'token-at-threshold',
+        name: 'token-full-capacity',
+        token: 'token-full-capacity',
+        fiveHourUtilization: 0.8,
+        blocked: false,
+        rejected: false,
+        modelWeeklyLimits: {},
+      },
+      {
+        name: 'token-two-slots',
+        token: 'token-two-slots',
+        fiveHourUtilization: 0.85,
+        blocked: false,
+        rejected: false,
+        modelWeeklyLimits: {},
+      },
+      {
+        name: 'token-one-slot',
+        token: 'token-one-slot',
         fiveHourUtilization: 0.9,
         blocked: false,
         rejected: false,
         modelWeeklyLimits: {},
       },
       {
-        token: 'token-below',
-        fiveHourUtilization: 0.4,
+        name: 'token-zero-slots',
+        token: 'token-zero-slots',
+        fiveHourUtilization: 0.95,
         blocked: false,
         rejected: false,
         modelWeeklyLimits: {},
@@ -2624,13 +2847,20 @@ describe('StartPreparationUseCase', () => {
       allowIssueCacheMinutes: 0,
     });
 
-    expect(mockLocalCommandRunner.runCommand.mock.calls).toHaveLength(1);
-    expect(mockLocalCommandRunner.runCommand.mock.calls[0][2]).toEqual({
-      env: {
-        CLAUDE_CODE_OAUTH_TOKEN: 'token-below',
-        ANTHROPIC_BASE_URL: 'http://127.0.0.1:8787',
-      },
-    });
+    expect(mockLocalCommandRunner.runCommand.mock.calls).toHaveLength(9);
+    const spawnedTokens = mockLocalCommandRunner.runCommand.mock.calls.map(
+      (call) => call[2]?.env?.CLAUDE_CODE_OAUTH_TOKEN,
+    );
+    expect(
+      spawnedTokens.filter((token) => token === 'token-full-capacity'),
+    ).toHaveLength(6);
+    expect(
+      spawnedTokens.filter((token) => token === 'token-two-slots'),
+    ).toHaveLength(2);
+    expect(
+      spawnedTokens.filter((token) => token === 'token-one-slot'),
+    ).toHaveLength(1);
+    expect(spawnedTokens).not.toContain('token-zero-slots');
   });
 
   it('should exclude a rejected token from rotation', async () => {
@@ -2653,6 +2883,7 @@ describe('StartPreparationUseCase', () => {
     });
     mockClaudeTokenUsageRepository.getAvailableTokenUsages.mockResolvedValue([
       {
+        name: 'token-rejected',
         token: 'token-rejected',
         fiveHourUtilization: 0.1,
         blocked: false,
@@ -2660,6 +2891,7 @@ describe('StartPreparationUseCase', () => {
         modelWeeklyLimits: {},
       },
       {
+        name: 'token-ok',
         token: 'token-ok',
         fiveHourUtilization: 0.5,
         blocked: false,
@@ -2710,6 +2942,7 @@ describe('StartPreparationUseCase', () => {
     });
     mockClaudeTokenUsageRepository.getAvailableTokenUsages.mockResolvedValue([
       {
+        name: 'token-reset',
         token: 'token-reset',
         fiveHourUtilization: 0,
         blocked: false,
@@ -2717,6 +2950,7 @@ describe('StartPreparationUseCase', () => {
         modelWeeklyLimits: {},
       },
       {
+        name: 'token-busy',
         token: 'token-busy',
         fiveHourUtilization: 0.5,
         blocked: false,
@@ -2767,6 +3001,7 @@ describe('StartPreparationUseCase', () => {
     });
     mockClaudeTokenUsageRepository.getAvailableTokenUsages.mockResolvedValue([
       {
+        name: 'token-saturated',
         token: 'token-saturated',
         fiveHourUtilization: 0.95,
         blocked: false,
@@ -2774,6 +3009,7 @@ describe('StartPreparationUseCase', () => {
         modelWeeklyLimits: {},
       },
       {
+        name: 'token-ok',
         token: 'token-ok',
         fiveHourUtilization: 0.2,
         blocked: false,
@@ -2824,6 +3060,7 @@ describe('StartPreparationUseCase', () => {
     });
     mockClaudeTokenUsageRepository.getAvailableTokenUsages.mockResolvedValue([
       {
+        name: 'token-a',
         token: 'token-a',
         fiveHourUtilization: 0.1,
         blocked: false,
@@ -2831,6 +3068,7 @@ describe('StartPreparationUseCase', () => {
         modelWeeklyLimits: {},
       },
       {
+        name: 'token-b',
         token: 'token-b',
         fiveHourUtilization: 0.2,
         blocked: false,
@@ -2931,6 +3169,7 @@ describe('StartPreparationUseCase', () => {
     const futureReset = Math.floor(Date.now() / 1000) + 3600;
     mockClaudeTokenUsageRepository.getAvailableTokenUsages.mockResolvedValue([
       {
+        name: 'token-sonnet-exhausted',
         token: 'token-sonnet-exhausted',
         fiveHourUtilization: 0.1,
         blocked: false,
@@ -2940,6 +3179,7 @@ describe('StartPreparationUseCase', () => {
         },
       },
       {
+        name: 'token-ok',
         token: 'token-ok',
         fiveHourUtilization: 0.5,
         blocked: false,
@@ -2991,6 +3231,7 @@ describe('StartPreparationUseCase', () => {
     const pastReset = Math.floor(Date.now() / 1000) - 3600;
     mockClaudeTokenUsageRepository.getAvailableTokenUsages.mockResolvedValue([
       {
+        name: 'token-recovered',
         token: 'token-recovered',
         fiveHourUtilization: 0.1,
         blocked: false,
@@ -3000,6 +3241,7 @@ describe('StartPreparationUseCase', () => {
         },
       },
       {
+        name: 'token-busy',
         token: 'token-busy',
         fiveHourUtilization: 0.5,
         blocked: false,
@@ -3051,6 +3293,7 @@ describe('StartPreparationUseCase', () => {
     const futureReset = Math.floor(Date.now() / 1000) + 3600;
     mockClaudeTokenUsageRepository.getAvailableTokenUsages.mockResolvedValue([
       {
+        name: 'token-sonnet-exhausted',
         token: 'token-sonnet-exhausted',
         fiveHourUtilization: 0.1,
         blocked: false,
@@ -3060,6 +3303,7 @@ describe('StartPreparationUseCase', () => {
         },
       },
       {
+        name: 'token-higher-util',
         token: 'token-higher-util',
         fiveHourUtilization: 0.5,
         blocked: false,
@@ -3111,6 +3355,7 @@ describe('StartPreparationUseCase', () => {
     const futureReset = Math.floor(Date.now() / 1000) + 3600;
     mockClaudeTokenUsageRepository.getAvailableTokenUsages.mockResolvedValue([
       {
+        name: 'token-weekly-exhausted',
         token: 'token-weekly-exhausted',
         fiveHourUtilization: 0.1,
         blocked: false,
@@ -3120,6 +3365,7 @@ describe('StartPreparationUseCase', () => {
         },
       },
       {
+        name: 'token-ok',
         token: 'token-ok',
         fiveHourUtilization: 0.5,
         blocked: false,
@@ -3150,3 +3396,140 @@ describe('StartPreparationUseCase', () => {
     });
   });
 });
+
+describe('StartPreparationUseCase.buildRotationOrder', () => {
+  const mockProjectRepositoryForRotation: Mocked<
+    Pick<ProjectRepository, 'getByUrl'>
+  > = {
+    getByUrl: jest.fn(),
+  };
+  const mockIssueRepositoryForRotation: Mocked<
+    Pick<
+      IssueRepository,
+      | 'getStoryObjectMap'
+      | 'updateStatus'
+      | 'findRelatedOpenPRs'
+      | 'getOpenPullRequest'
+      | 'closePullRequest'
+      | 'deletePullRequestBranch'
+      | 'createCommentByUrl'
+    >
+  > = {
+    getStoryObjectMap: jest.fn(),
+    updateStatus: jest.fn(),
+    findRelatedOpenPRs: jest.fn(),
+    getOpenPullRequest: jest.fn(),
+    closePullRequest: jest.fn(),
+    deletePullRequestBranch: jest.fn(),
+    createCommentByUrl: jest.fn(),
+  };
+  const mockLocalCommandRunnerForRotation: Mocked<LocalCommandRunner> = {
+    runCommand: jest.fn(),
+  };
+  const mockClaudeTokenUsageRepositoryForRotation: Mocked<ClaudeTokenUsageRepository> =
+    {
+      ensureObservable: jest.fn(),
+      getAvailableTokenUsages: jest.fn(),
+      proxyBaseUrl: jest.fn(),
+    };
+
+  const useCase = new StartPreparationUseCase(
+    mockProjectRepositoryForRotation,
+    mockIssueRepositoryForRotation,
+    mockLocalCommandRunnerForRotation,
+    mockClaudeTokenUsageRepositoryForRotation,
+  );
+
+  it('lists selected tokens first in ascending utilization order then excluded tokens', () => {
+    const tokenUsages = [
+      {
+        name: 'high-util',
+        token: 'sk-ant-high',
+        fiveHourUtilization: 0.8,
+        blocked: false,
+        rejected: false,
+        modelWeeklyLimits: {},
+      },
+      {
+        name: 'low-util',
+        token: 'sk-ant-low',
+        fiveHourUtilization: 0.1,
+        blocked: false,
+        rejected: false,
+        modelWeeklyLimits: {},
+      },
+      {
+        name: 'blocked-token',
+        token: 'sk-ant-blocked',
+        fiveHourUtilization: 0.0,
+        blocked: true,
+        rejected: false,
+        modelWeeklyLimits: {},
+      },
+    ];
+    const result = useCase.buildRotationOrder(tokenUsages, 90, null);
+
+    expect(result[0].name).toBe('low-util');
+    expect(result[1].name).toBe('high-util');
+    expect(result[2].name).toBe('blocked-token');
+    expect(result[2].blocked).toBe(true);
+    expect(result[2].thresholdExcluded).toBe(false);
+  });
+
+  it('does not include raw token strings in output entries', () => {
+    const tokenUsages = [
+      {
+        name: 'my-token',
+        token: 'sk-ant-secret-value',
+        fiveHourUtilization: 0.1,
+        blocked: false,
+        rejected: false,
+        modelWeeklyLimits: {},
+      },
+    ];
+    const result = useCase.buildRotationOrder(tokenUsages, 90, null);
+    const serialized = JSON.stringify(result);
+
+    expect(serialized).not.toContain('sk-ant-secret-value');
+    expect(result[0].name).toBe('my-token');
+  });
+
+  it('marks thresholdExcluded true when token is at or above 95 percent utilization', () => {
+    const tokenUsages = [
+      {
+        name: 'over-threshold',
+        token: 'sk-ant-over',
+        fiveHourUtilization: 0.95,
+        blocked: false,
+        rejected: false,
+        modelWeeklyLimits: {},
+      },
+    ];
+    const result = useCase.buildRotationOrder(tokenUsages, 90, null);
+
+    expect(result).toHaveLength(1);
+    expect(result[0].thresholdExcluded).toBe(true);
+    expect(result[0].blocked).toBe(false);
+    expect(result[0].rejected).toBe(false);
+  });
+
+  it('does not mark thresholdExcluded for tokens in the 90 to 94 percent utilization range because selectRotationTokens still assigns them slots', () => {
+    const tokenUsages = [
+      {
+        name: 'mid-util',
+        token: 'sk-ant-mid',
+        fiveHourUtilization: 0.92,
+        blocked: false,
+        rejected: false,
+        modelWeeklyLimits: {},
+      },
+    ];
+    const result = useCase.buildRotationOrder(tokenUsages, 90, null);
+
+    expect(result).toHaveLength(1);
+    expect(result[0].thresholdExcluded).toBe(false);
+    expect(result[0].blocked).toBe(false);
+    expect(result[0].rejected).toBe(false);
+    expect(result[0].fiveHourUtilization).toBe(0.92);
+  });
+});