github-issue-tower-defence-management 1.40.0 → 1.41.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,8 +73,12 @@ 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;
     } | null;
     notifyFinishedPreparation?: {
@@ -344,10 +348,14 @@ ${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,
       });
     }
     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);
   });
 
@@ -295,8 +304,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,