github-issue-tower-defence-management 1.46.0 → 1.48.0

package/src/adapter/entry-points/cli/projectConfig.ts

@@ -3,8 +3,6 @@ import * as fs from 'fs';
 
 export type ConfigFile = {
   projectUrl?: string;
-  awaitingWorkspaceStatus?: string;
-  preparationStatus?: string;
   defaultAgentName?: string;
   defaultLlmModelName?: string;
   defaultLlmAgentName?: string;
@@ -12,7 +10,6 @@ export type ConfigFile = {
   allowIssueCacheMinutes?: number;
   utilizationPercentageThreshold?: number;
   allowedIssueAuthors?: string;
-  awaitingQualityCheckStatus?: string;
   thresholdForAutoReject?: number;
   workflowBlockerResolvedWebhookUrl?: string;
   projectName?: string;
@@ -68,11 +65,6 @@ export const loadConfigFile = (configFilePath: string): ConfigFile => {
     }
     return {
       projectUrl: getStringValue(parsed, 'projectUrl'),
-      awaitingWorkspaceStatus: getStringValue(
-        parsed,
-        'awaitingWorkspaceStatus',
-      ),
-      preparationStatus: getStringValue(parsed, 'preparationStatus'),
       defaultAgentName: getStringValue(parsed, 'defaultAgentName'),
       defaultLlmModelName: getStringValue(parsed, 'defaultLlmModelName'),
       defaultLlmAgentName: getStringValue(parsed, 'defaultLlmAgentName'),
@@ -86,10 +78,6 @@ export const loadConfigFile = (configFilePath: string): ConfigFile => {
         'utilizationPercentageThreshold',
       ),
       allowedIssueAuthors: getStringValue(parsed, 'allowedIssueAuthors'),
-      awaitingQualityCheckStatus: getStringValue(
-        parsed,
-        'awaitingQualityCheckStatus',
-      ),
       thresholdForAutoReject: getNumberValue(parsed, 'thresholdForAutoReject'),
       workflowBlockerResolvedWebhookUrl: getStringValue(
         parsed,
@@ -133,11 +121,6 @@ export const parseProjectReadmeConfig = (readme: string): ConfigFile => {
       return {};
     }
     return {
-      awaitingWorkspaceStatus: getStringValue(
-        parsed,
-        'awaitingWorkspaceStatus',
-      ),
-      preparationStatus: getStringValue(parsed, 'preparationStatus'),
       defaultAgentName: getStringValue(parsed, 'defaultAgentName'),
       defaultLlmModelName: getStringValue(parsed, 'defaultLlmModelName'),
       defaultLlmAgentName: getStringValue(parsed, 'defaultLlmAgentName'),
@@ -151,10 +134,6 @@ export const parseProjectReadmeConfig = (readme: string): ConfigFile => {
         'utilizationPercentageThreshold',
       ),
       allowedIssueAuthors: getStringValue(parsed, 'allowedIssueAuthors'),
-      awaitingQualityCheckStatus: getStringValue(
-        parsed,
-        'awaitingQualityCheckStatus',
-      ),
       thresholdForAutoReject: getNumberValue(parsed, 'thresholdForAutoReject'),
       workflowBlockerResolvedWebhookUrl: getStringValue(
         parsed,
@@ -183,14 +162,6 @@ export const mergeConfigs = (
   readmeOverrides: ConfigFile,
 ): ConfigFile => ({
   projectUrl: cliOverrides.projectUrl ?? configFile.projectUrl,
-  awaitingWorkspaceStatus:
-    readmeOverrides.awaitingWorkspaceStatus ??
-    cliOverrides.awaitingWorkspaceStatus ??
-    configFile.awaitingWorkspaceStatus,
-  preparationStatus:
-    readmeOverrides.preparationStatus ??
-    cliOverrides.preparationStatus ??
-    configFile.preparationStatus,
   defaultAgentName:
     readmeOverrides.defaultAgentName ??
     cliOverrides.defaultAgentName ??
@@ -219,10 +190,6 @@ export const mergeConfigs = (
     readmeOverrides.allowedIssueAuthors ??
     cliOverrides.allowedIssueAuthors ??
     configFile.allowedIssueAuthors,
-  awaitingQualityCheckStatus:
-    readmeOverrides.awaitingQualityCheckStatus ??
-    cliOverrides.awaitingQualityCheckStatus ??
-    configFile.awaitingQualityCheckStatus,
   thresholdForAutoReject:
     readmeOverrides.thresholdForAutoReject ??
     cliOverrides.thresholdForAutoReject ??

package/src/adapter/entry-points/handlers/HandleScheduledEventUseCaseHandler.test.ts

@@ -113,8 +113,12 @@ jest.mock('../../repositories/GitHubIssueCommentRepository', () => ({
 jest.mock('../../repositories/FetchWebhookRepository', () => ({
   FetchWebhookRepository: jest.fn().mockImplementation(() => ({})),
 }));
+jest.mock('./situationFileWriter', () => ({
+  writeSituationFile: jest.fn().mockResolvedValue(undefined),
+}));
 
 import { HandleScheduledEventUseCaseHandler } from './HandleScheduledEventUseCaseHandler';
+import { writeSituationFile } from './situationFileWriter';
 import { GraphqlProjectRepository } from '../../repositories/GraphqlProjectRepository';
 import { ApiV3IssueRepository } from '../../repositories/issue/ApiV3IssueRepository';
 import { RestIssueRepository } from '../../repositories/issue/RestIssueRepository';
@@ -137,8 +141,6 @@ const validConfig = {
   projectUrl: 'https://github.com/users/TestOrg/projects/1',
   manager: 'TestManager',
   urlOfStoryView: 'https://github.com/users/TestOrg/projects/1/views/1',
-  disabledStatus: 'Icebox',
-  defaultStatus: 'Unread',
   disabled: false,
   allowIssueCacheMinutes: 1,
   workingReport: {
@@ -258,22 +260,17 @@ describe('HandleScheduledEventUseCaseHandler', () => {
     }
   });
 
-  it('should write runtimeConfig-{projectId}.json atomically after successful run', async () => {
+  it('should write situation file after successful run with resolved config values', async () => {
     const configWithPreparation = {
       ...validConfig,
       allowIssueCacheMinutes: 5,
       startPreparation: {
-        awaitingWorkspaceStatus: 'Awaiting',
-        preparationStatus: 'Preparing',
         defaultAgentName: 'agent1',
         configFilePath: './config.yml',
         maximumPreparingIssuesCount: 10,
         utilizationPercentageThreshold: 97,
       },
       notifyFinishedPreparation: {
-        preparationStatus: 'Preparing',
-        awaitingWorkspaceStatus: 'Awaiting',
-        awaitingQualityCheckStatus: 'Awaiting QC',
         thresholdForAutoReject: 30,
         workflowBlockerResolvedWebhookUrl: null,
       },
@@ -286,64 +283,40 @@ describe('HandleScheduledEventUseCaseHandler', () => {
     await handler.handle('config.yml', false);
 
     const expectedCachePath = `./tmp/cache/${validConfig.projectName}`;
-    const expectedFinalPath = `${expectedCachePath}/runtimeConfig-PVT_kwHOtest123.json`;
-    const expectedTmpPath = `${expectedFinalPath}.tmp`;
-
-    expect(jest.mocked(fs.mkdirSync)).toHaveBeenCalledWith(expectedCachePath, {
-      recursive: true,
-    });
-    expect(jest.mocked(fs.writeFileSync)).toHaveBeenCalledWith(
-      expectedTmpPath,
-      expect.stringContaining('"maximumPreparingIssuesCount":10'),
-    );
-    expect(jest.mocked(fs.writeFileSync)).toHaveBeenCalledWith(
-      expectedTmpPath,
-      expect.stringContaining('"utilizationPercentageThreshold":97'),
-    );
-    expect(jest.mocked(fs.writeFileSync)).toHaveBeenCalledWith(
-      expectedTmpPath,
-      expect.stringContaining('"allowIssueCacheMinutes":5'),
-    );
-    expect(jest.mocked(fs.writeFileSync)).toHaveBeenCalledWith(
-      expectedTmpPath,
-      expect.stringContaining('"thresholdForAutoReject":30'),
+    const firstCallArg = jest.mocked(writeSituationFile).mock.calls[0][0];
+    expect(firstCallArg.cachePath).toBe(expectedCachePath);
+    expect(firstCallArg.projectId).toBe('PVT_kwHOtest123');
+    expect(firstCallArg.config.maximumPreparingIssuesCount).toBe(10);
+    expect(firstCallArg.config.utilizationPercentageThreshold).toBe(97);
+    expect(firstCallArg.config.allowIssueCacheMinutes).toBe(5);
+    expect(firstCallArg.config.thresholdForAutoReject).toBe(30);
+    expect(firstCallArg.statusNames.awaitingQualityCheckStatus).toBe(
+      'Awaiting Quality Check',
     );
-    expect(jest.mocked(fs.renameSync)).toHaveBeenCalledWith(
-      expectedTmpPath,
-      expectedFinalPath,
+    expect(firstCallArg.statusNames.preparationStatus).toBe('Preparation');
+    expect(firstCallArg.statusNames.awaitingWorkspaceStatus).toBe(
+      'Awaiting Workspace',
     );
   });
 
-  it('should write runtimeConfig with numeric defaults when optional fields are absent', async () => {
+  it('should write situation file with numeric defaults when optional fields are absent', async () => {
     const handler = new HandleScheduledEventUseCaseHandler();
     await handler.handle('config.yml', false);
 
-    const expectedCachePath = `./tmp/cache/${validConfig.projectName}`;
-    const expectedTmpPath = `${expectedCachePath}/runtimeConfig-PVT_kwHOtest123.json.tmp`;
-
-    expect(jest.mocked(fs.writeFileSync)).toHaveBeenCalledWith(
-      expectedTmpPath,
-      expect.stringContaining('"utilizationPercentageThreshold":90'),
-    );
-    expect(jest.mocked(fs.writeFileSync)).toHaveBeenCalledWith(
-      expectedTmpPath,
-      expect.stringContaining('"thresholdForAutoReject":3'),
-    );
-    expect(jest.mocked(fs.writeFileSync)).toHaveBeenCalledWith(
-      expectedTmpPath,
-      expect.stringContaining('"maximumPreparingIssuesCount":null'),
-    );
+    const firstCallArg = jest.mocked(writeSituationFile).mock.calls[0][0];
+    expect(firstCallArg.config.utilizationPercentageThreshold).toBe(90);
+    expect(firstCallArg.config.thresholdForAutoReject).toBe(3);
+    expect(firstCallArg.config.maximumPreparingIssuesCount).toBeNull();
   });
 
-  it('should not write runtimeConfig when run returns null', async () => {
+  it('should not write situation file when run returns null', async () => {
     mockRun.mockResolvedValueOnce(null);
     jest.mocked(fs.readFileSync).mockReturnValue(YAML.stringify(validConfig));
 
     const handler = new HandleScheduledEventUseCaseHandler();
     await handler.handle('config.yml', false);
 
-    expect(jest.mocked(fs.writeFileSync)).not.toHaveBeenCalled();
-    expect(jest.mocked(fs.renameSync)).not.toHaveBeenCalled();
+    expect(jest.mocked(writeSituationFile)).not.toHaveBeenCalled();
   });
 
   describe('README config overrides', () => {
@@ -351,8 +324,6 @@ describe('HandleScheduledEventUseCaseHandler', () => {
       ...validConfig,
       allowIssueCacheMinutes: 5,
       startPreparation: {
-        awaitingWorkspaceStatus: 'Awaiting workspace',
-        preparationStatus: 'Preparation',
         defaultAgentName: 'yaml-agent',
         configFilePath: '/path/to/config.yml',
         maximumPreparingIssuesCount: 10,
@@ -364,8 +335,6 @@ describe('HandleScheduledEventUseCaseHandler', () => {
       const readmeContent = `<details>
 <summary>config</summary>
 maximumPreparingIssuesCount: 0
-awaitingWorkspaceStatus: README Awaiting
-preparationStatus: README Preparation
 defaultAgentName: readme-agent
 utilizationPercentageThreshold: 80
 </details>`;
@@ -380,8 +349,6 @@ utilizationPercentageThreshold: 80
       expect(capturedRunInputs[0][0]).toMatchObject({
         startPreparation: {
           maximumPreparingIssuesCount: 0,
-          awaitingWorkspaceStatus: 'README Awaiting',
-          preparationStatus: 'README Preparation',
           defaultAgentName: 'readme-agent',
           utilizationPercentageThreshold: 80,
         },
@@ -441,7 +408,6 @@ allowedIssueAuthors: 'user1, user2, user3'
         allowIssueCacheMinutes: 5,
         startPreparation: {
           maximumPreparingIssuesCount: 10,
-          awaitingWorkspaceStatus: 'Awaiting workspace',
           defaultAgentName: 'yaml-agent',
         },
       });

package/src/adapter/entry-points/handlers/HandleScheduledEventUseCaseHandler.ts

@@ -1,6 +1,7 @@
 import YAML from 'yaml';
 import TYPIA from 'typia';
 import fs from 'fs';
+import { writeSituationFile } from './situationFileWriter';
 import {
   fetchProjectReadme,
   parseProjectReadmeConfig,
@@ -29,7 +30,6 @@ import { ConvertCheckboxToIssueInStoryIssueUseCase } from '../../../domain/useca
 import { ChangeStatusByStoryColorUseCase } from '../../../domain/usecases/ChangeStatusByStoryColorUseCase';
 import { SetNoStoryIssueToStoryUseCase } from '../../../domain/usecases/SetNoStoryIssueToStoryUseCase';
 import { CreateNewStoryByLabelUseCase } from '../../../domain/usecases/CreateNewStoryByLabelUseCase';
-import { ProjectRepository } from '../../../domain/usecases/adapter-interfaces/ProjectRepository';
 import { AssignNoAssigneeIssueToManagerUseCase } from '../../../domain/usecases/AssignNoAssigneeIssueToManagerUseCase';
 import { UpdateIssueStatusByLabelUseCase } from '../../../domain/usecases/UpdateIssueStatusByLabelUseCase';
 import { StartPreparationUseCase } from '../../../domain/usecases/StartPreparationUseCase';
@@ -39,6 +39,12 @@ import { NotifyFinishedIssuePreparationUseCase } from '../../../domain/usecases/
 import { RevertOrphanedPreparationUseCase } from '../../../domain/usecases/RevertOrphanedPreparationUseCase';
 import { GitHubIssueCommentRepository } from '../../repositories/GitHubIssueCommentRepository';
 import { FetchWebhookRepository } from '../../repositories/FetchWebhookRepository';
+import { SetupTowerDefenceProjectUseCase } from '../../../domain/usecases/SetupTowerDefenceProjectUseCase';
+import {
+  AWAITING_QUALITY_CHECK_STATUS_NAME,
+  AWAITING_WORKSPACE_STATUS_NAME,
+  PREPARATION_STATUS_NAME,
+} from '../../../domain/entities/WorkflowStatus';
 
 export class HandleScheduledEventUseCaseHandler {
   handle = async (
@@ -96,12 +102,6 @@ export class HandleScheduledEventUseCaseHandler {
       startPreparation: input.startPreparation
         ? {
             ...input.startPreparation,
-            awaitingWorkspaceStatus:
-              readmeConfig.awaitingWorkspaceStatus ??
-              input.startPreparation.awaitingWorkspaceStatus,
-            preparationStatus:
-              readmeConfig.preparationStatus ??
-              input.startPreparation.preparationStatus,
             defaultAgentName:
               readmeConfig.defaultAgentName ??
               input.startPreparation.defaultAgentName,
@@ -134,15 +134,6 @@ export class HandleScheduledEventUseCaseHandler {
       notifyFinishedPreparation: input.notifyFinishedPreparation
         ? {
             ...input.notifyFinishedPreparation,
-            awaitingWorkspaceStatus:
-              readmeConfig.awaitingWorkspaceStatus ??
-              input.notifyFinishedPreparation.awaitingWorkspaceStatus,
-            preparationStatus:
-              readmeConfig.preparationStatus ??
-              input.notifyFinishedPreparation.preparationStatus,
-            awaitingQualityCheckStatus:
-              readmeConfig.awaitingQualityCheckStatus ??
-              input.notifyFinishedPreparation.awaitingQualityCheckStatus,
             thresholdForAutoReject:
               readmeConfig.thresholdForAutoReject ??
               input.notifyFinishedPreparation.thresholdForAutoReject,
@@ -174,15 +165,9 @@ export class HandleScheduledEventUseCaseHandler {
       input.credentials.bot.github.password,
       input.credentials.bot.github.authenticatorKey,
     ];
-    const projectRepository: ProjectRepository = {
-      ...new GraphqlProjectRepository(...githubRepositoryParams),
-      prepareStatus: async (
-        _name: string,
-        project: Project,
-      ): Promise<Project> => {
-        return project;
-      },
-    };
+    const projectRepository = new GraphqlProjectRepository(
+      ...githubRepositoryParams,
+    );
     const apiV3IssueRepository = new ApiV3IssueRepository(
       ...githubRepositoryParams,
     );
@@ -199,6 +184,9 @@ export class HandleScheduledEventUseCaseHandler {
       localStorageCacheRepository,
       ...githubRepositoryParams,
     );
+    const setupTowerDefenceProjectUseCase = new SetupTowerDefenceProjectUseCase(
+      projectRepository,
+    );
     const actionAnnouncement = new ActionAnnouncementUseCase(issueRepository);
     const setWorkflowManagementIssueToStoryUseCase =
       new SetWorkflowManagementIssueToStoryUseCase(issueRepository);
@@ -267,6 +255,7 @@ export class HandleScheduledEventUseCaseHandler {
       );
 
     const handleScheduledEventUseCase = new HandleScheduledEventUseCase(
+      setupTowerDefenceProjectUseCase,
       actionAnnouncement,
       setWorkflowManagementIssueToStoryUseCase,
       clearPastNextActionUseCase,
@@ -291,22 +280,28 @@ export class HandleScheduledEventUseCaseHandler {
 
     const result = await handleScheduledEventUseCase.run(mergedInput);
     if (result) {
-      const projectId = result.project.id;
-      const runtimeConfig = {
-        resolvedAt: new Date().toISOString(),
-        maximumPreparingIssuesCount:
-          mergedInput.startPreparation?.maximumPreparingIssuesCount ?? null,
-        utilizationPercentageThreshold:
-          mergedInput.startPreparation?.utilizationPercentageThreshold ?? 90,
-        allowIssueCacheMinutes: mergedInput.allowIssueCacheMinutes,
-        thresholdForAutoReject:
-          mergedInput.notifyFinishedPreparation?.thresholdForAutoReject ?? 3,
-      };
-      const finalPath = `${cachePath}/runtimeConfig-${projectId}.json`;
-      const tmpPath = `${finalPath}.tmp`;
-      fs.mkdirSync(cachePath, { recursive: true });
-      fs.writeFileSync(tmpPath, JSON.stringify(runtimeConfig));
-      fs.renameSync(tmpPath, finalPath);
+      await writeSituationFile({
+        cachePath,
+        projectId: result.project.id,
+        issues: result.issues,
+        statusNames: {
+          awaitingQualityCheckStatus: AWAITING_QUALITY_CHECK_STATUS_NAME,
+          preparationStatus: PREPARATION_STATUS_NAME,
+          awaitingWorkspaceStatus: AWAITING_WORKSPACE_STATUS_NAME,
+        },
+        config: {
+          maximumPreparingIssuesCount:
+            mergedInput.startPreparation?.maximumPreparingIssuesCount ?? null,
+          utilizationPercentageThreshold:
+            mergedInput.startPreparation?.utilizationPercentageThreshold ?? 90,
+          allowIssueCacheMinutes: mergedInput.allowIssueCacheMinutes,
+          thresholdForAutoReject:
+            mergedInput.notifyFinishedPreparation?.thresholdForAutoReject ?? 3,
+        },
+        preparationProcessCheckCommand:
+          mergedInput.startPreparation?.preparationProcessCheckCommand ?? null,
+        localCommandRunner: nodeLocalCommandRunner,
+      });
     }
     return result;
   };