github-issue-tower-defence-management 1.40.0 → 1.42.0

package/src/adapter/repositories/issue/RestIssueRepository.test.ts

@@ -55,6 +55,7 @@ describe('RestIssueRepository', () => {
         isInProgress: false,
         isClosed: false,
         createdAt: new Date(),
+        author: '',
       };
 
       await restIssueRepository.updateLabels(issue, ['default']);
@@ -92,6 +93,7 @@ describe('RestIssueRepository', () => {
         isInProgress: false,
         isClosed: false,
         createdAt: new Date(),
+        author: '',
       };
 
       await restIssueRepository.updateLabels(issue, ['test', 'to-remove']);
@@ -130,6 +132,7 @@ describe('RestIssueRepository', () => {
         isInProgress: false,
         isClosed: false,
         createdAt: new Date(),
+        author: '',
       };
       await restIssueRepository.updateAssigneeList(issue, ['HiromiShikata']);
       const issueWithAssignee = await restIssueRepository.getIssue(issue.url);

package/src/domain/entities/Issue.ts

@@ -23,4 +23,5 @@ export type Issue = {
   isInProgress: boolean;
   isClosed: boolean;
   createdAt: Date;
+  author: string;
 };

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

@@ -66,6 +66,7 @@ describe('GetStoryObjectMapUseCase', () => {
     isInProgress: false,
     isClosed: false,
     createdAt: new Date(),
+    author: '',
     ...overrides,
   });
 

package/src/domain/usecases/HandleScheduledEventUseCase.ts

@@ -73,9 +73,14 @@ export class HandleScheduledEventUseCase {
       awaitingWorkspaceStatus: string;
       preparationStatus: string;
       defaultAgentName: string;
-      logFilePath?: string;
+      defaultLlmModelName?: string | null;
+      defaultLlmAgentName?: string | null;
+      configFilePath: string;
       maximumPreparingIssuesCount: number | null;
+      utilizationPercentageThreshold?: number;
+      allowedIssueAuthors?: string[] | null;
       preparationProcessCheckCommand?: string;
+      codexHomeCandidates?: string[] | null;
     } | null;
     notifyFinishedPreparation?: {
       preparationStatus: string;
@@ -344,10 +349,15 @@ ${JSON.stringify(e)}
         awaitingWorkspaceStatus: input.startPreparation.awaitingWorkspaceStatus,
         preparationStatus: input.startPreparation.preparationStatus,
         defaultAgentName: input.startPreparation.defaultAgentName,
-        logFilePath: input.startPreparation.logFilePath,
+        defaultLlmModelName: input.startPreparation.defaultLlmModelName ?? null,
+        defaultLlmAgentName: input.startPreparation.defaultLlmAgentName ?? null,
+        configFilePath: input.startPreparation.configFilePath,
         maximumPreparingIssuesCount:
           input.startPreparation.maximumPreparingIssuesCount,
-        allowIssueCacheMinutes: input.allowIssueCacheMinutes,
+        utilizationPercentageThreshold:
+          input.startPreparation.utilizationPercentageThreshold ?? 90,
+        allowedIssueAuthors: input.startPreparation.allowedIssueAuthors ?? null,
+        codexHomeCandidates: input.startPreparation.codexHomeCandidates ?? null,
       });
     }
     if (input.notifyFinishedPreparation) {

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

@@ -46,6 +46,7 @@ const createMockIssue = (overrides: Partial<Issue> = {}): Issue => ({
   isInProgress: false,
   isClosed: false,
   createdAt: new Date(),
+  author: '',
   ...overrides,
 });
 

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

@@ -30,6 +30,7 @@ const createMockIssue = (overrides: Partial<Issue> = {}): Issue => ({
   isInProgress: false,
   isClosed: false,
   createdAt: new Date(),
+  author: '',
   ...overrides,
 });
 
@@ -136,9 +137,13 @@ describe('RevertOrphanedPreparationUseCase', () => {
     expect(mockIssueRepository.createComment.mock.calls).toHaveLength(1);
     expect(mockIssueRepository.createComment.mock.calls[0][0]).toBe(stuckIssue);
     expect(mockLocalCommandRunner.runCommand.mock.calls).toHaveLength(1);
-    expect(mockLocalCommandRunner.runCommand.mock.calls[0][0]).toBe(
-      'pgrep -fa "claude-agent.*https://github.com/user/repo/issues/10"',
-    );
+    expect(mockLocalCommandRunner.runCommand.mock.calls[0][0]).toBe('sh');
+    expect(mockLocalCommandRunner.runCommand.mock.calls[0][1]).toEqual([
+      '-c',
+      'pgrep -fa "claude-agent.*$1"',
+      '--',
+      'https://github.com/user/repo/issues/10',
+    ]);
   });
 
   it('should leave in-flight Preparation issue untouched when check command exits zero', async () => {
@@ -196,9 +201,13 @@ describe('RevertOrphanedPreparationUseCase', () => {
     });
 
     expect(mockLocalCommandRunner.runCommand.mock.calls).toHaveLength(1);
-    expect(mockLocalCommandRunner.runCommand.mock.calls[0][0]).toBe(
-      'check https://github.com/user/repo/issues/10',
-    );
+    expect(mockLocalCommandRunner.runCommand.mock.calls[0][0]).toBe('sh');
+    expect(mockLocalCommandRunner.runCommand.mock.calls[0][1]).toEqual([
+      '-c',
+      'check $1',
+      '--',
+      'https://github.com/user/repo/issues/10',
+    ]);
     expect(mockIssueRepository.updateStatus.mock.calls).toHaveLength(1);
   });
 
@@ -250,6 +259,48 @@ describe('RevertOrphanedPreparationUseCase', () => {
     ).rejects.toThrow('Project not found');
   });
 
+  it('should throw when getProject returns null after findProjectIdByUrl succeeds', async () => {
+    mockProjectRepository.findProjectIdByUrl.mockResolvedValue('project-1');
+    mockProjectRepository.getProject.mockResolvedValue(null);
+
+    await expect(
+      useCase.run({
+        projectUrl: 'https://github.com/user/repo',
+        preparationStatus: 'Preparation',
+        awaitingWorkspaceStatus: 'Awaiting Workspace',
+        allowIssueCacheMinutes: 0,
+        preparationProcessCheckCommand: 'check {URL}',
+      }),
+    ).rejects.toThrow('Project not found. projectId: project-1');
+  });
+
+  it('should do nothing when awaitingWorkspaceStatus is not found in project statuses', async () => {
+    const preparationIssue = createMockIssue({
+      url: 'https://github.com/user/repo/issues/10',
+      status: 'Preparation',
+    });
+    mockIssueRepository.getAllIssues.mockResolvedValue({
+      issues: [preparationIssue],
+      cacheUsed: false,
+    });
+    mockLocalCommandRunner.runCommand.mockResolvedValue({
+      stdout: '',
+      stderr: '',
+      exitCode: 1,
+    });
+
+    await useCase.run({
+      projectUrl: 'https://github.com/user/repo',
+      preparationStatus: 'Preparation',
+      awaitingWorkspaceStatus: 'NonExistentStatus',
+      allowIssueCacheMinutes: 0,
+      preparationProcessCheckCommand: 'check {URL}',
+    });
+
+    expect(mockIssueRepository.updateStatus.mock.calls).toHaveLength(0);
+    expect(mockIssueRepository.createComment.mock.calls).toHaveLength(0);
+  });
+
   it('should do nothing when there are no Preparation issues', async () => {
     mockIssueRepository.getAllIssues.mockResolvedValue({
       issues: [
@@ -295,8 +346,12 @@ describe('RevertOrphanedPreparationUseCase', () => {
     });
 
     expect(mockLocalCommandRunner.runCommand.mock.calls).toHaveLength(1);
-    expect(mockLocalCommandRunner.runCommand.mock.calls[0][0]).toBe(
-      'pgrep -fa "claude-agent.*https://github.com/org/project/issues/99"',
-    );
+    expect(mockLocalCommandRunner.runCommand.mock.calls[0][0]).toBe('sh');
+    expect(mockLocalCommandRunner.runCommand.mock.calls[0][1]).toEqual([
+      '-c',
+      'pgrep -fa "claude-agent.*$1"',
+      '--',
+      'https://github.com/org/project/issues/99',
+    ]);
   });
 });

package/src/domain/usecases/RevertOrphanedPreparationUseCase.ts

@@ -51,11 +51,16 @@ export class RevertOrphanedPreparationUseCase {
     }
 
     for (const issue of preparationIssues) {
-      const command = params.preparationProcessCheckCommand.replace(
+      const commandTemplate = params.preparationProcessCheckCommand.replace(
         '{URL}',
-        issue.url,
+        '$1',
       );
-      const { exitCode } = await this.localCommandRunner.runCommand(command);
+      const { exitCode } = await this.localCommandRunner.runCommand('sh', [
+        '-c',
+        commandTemplate,
+        '--',
+        issue.url,
+      ]);
       if (exitCode !== 0) {
         await this.issueRepository.updateStatus(
           project,