github-issue-tower-defence-management 1.53.0 → 1.54.0

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+# [1.54.0](https://github.com/HiromiShikata/npm-cli-github-issue-tower-defence-management/compare/v1.53.0...v1.54.0) (2026-05-23)
+
+
+### Features
+
+* **core:** change status ([b027776](https://github.com/HiromiShikata/npm-cli-github-issue-tower-defence-management/commit/b0277767b231cca3f5910998e63b1b2fc1166e0e))
+
 # [1.53.0](https://github.com/HiromiShikata/npm-cli-github-issue-tower-defence-management/compare/v1.52.1...v1.53.0) (2026-05-23)
 
 

package/README.md

@@ -73,7 +73,7 @@ npx github-issue-tower-defence-management notifyFinishedIssuePreparation --confi
 
 The `config.yaml` for the `schedule` command must match the input type of `HandleScheduledEventUseCase.run()`. Below is the structure:
 
-Workflow status names (`Unread`, `Awaiting Task Breakdown`, `Awaiting Workspace`, `Preparation`, `Failed Preparation`, `Awaiting Quality Check`, `Todo`, `PC Todo`, `In Tmux`, `Done`, `Icebox`) are fixed code constants and cannot be overridden from CLI options, config files, or project README. The `schedule` command automatically creates any missing required statuses on the target project on each run via `SetupTowerDefenceProjectUseCase`.
+Workflow status names (`Unread`, `Awaiting Task Breakdown`, `Awaiting Workspace`, `Preparation`, `Failed Preparation`, `Awaiting Quality Check`, `Todo by human`, `In Tmux by human`, `Done`, `Icebox`) are fixed code constants and cannot be overridden from CLI options, config files, or project README. The `schedule` command automatically creates any missing required statuses on the target project on each run via `SetupTowerDefenceProjectUseCase`. Projects with the legacy `Todo` and `In Tmux` status names are automatically migrated to `Todo by human` and `In Tmux by human` respectively by reusing the existing option IDs so that task associations are preserved. The legacy `PC Todo` status is removed from the required list and excluded from the project status list on the next setup run.
 
 ```yaml
 disabled: boolean # When true, skip all processing and return null
@@ -168,7 +168,7 @@ awLogDirectoryPath?: string # Optional: Directory path where aw log files named
 awLogStaleThresholdMinutes?: number # Optional: Minutes since last aw log mtime after which a Preparation issue is considered orphaned even when pgrep still returns 0. Requires awLogDirectoryPath
 ```
 
-Workflow status names (`Unread`, `Awaiting Task Breakdown`, `Awaiting Workspace`, `Preparation`, `Failed Preparation`, `Awaiting Quality Check`, `Todo`, `PC Todo`, `In Tmux`, `Done`, `Icebox`) are hardcoded constants and are not accepted via this config, the CLI, or the project README. To ensure they exist on the target project, run the `schedule` command — it invokes `SetupTowerDefenceProjectUseCase` automatically.
+Workflow status names (`Unread`, `Awaiting Task Breakdown`, `Awaiting Workspace`, `Preparation`, `Failed Preparation`, `Awaiting Quality Check`, `Todo by human`, `In Tmux by human`, `Done`, `Icebox`) are hardcoded constants and are not accepted via this config, the CLI, or the project README. To ensure they exist on the target project, run the `schedule` command — it invokes `SetupTowerDefenceProjectUseCase` automatically. Projects with the legacy `Todo` and `In Tmux` status names are automatically migrated; `PC Todo` is removed on the next setup run.
 
 Example:
 

package/bin/domain/entities/WorkflowStatus.js

@@ -1,17 +1,19 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.REQUIRED_WORKFLOW_STATUSES = exports.ICEBOX_STATUS_NAME = exports.DONE_STATUS_NAME = exports.IN_TMUX_STATUS_NAME = exports.PC_TODO_STATUS_NAME = exports.TODO_STATUS_NAME = exports.AWAITING_QUALITY_CHECK_STATUS_NAME = exports.FAILED_PREPARATION_STATUS_NAME = exports.PREPARATION_STATUS_NAME = exports.AWAITING_WORKSPACE_STATUS_NAME = exports.AWAITING_TASK_BREAKDOWN_STATUS_NAME = exports.DEFAULT_STATUS_NAME = void 0;
+exports.REQUIRED_WORKFLOW_STATUSES = exports.LEGACY_IN_TMUX_STATUS_NAME = exports.LEGACY_TODO_STATUS_NAME = exports.ICEBOX_STATUS_NAME = exports.DONE_STATUS_NAME = exports.IN_TMUX_STATUS_NAME = exports.PC_TODO_STATUS_NAME = exports.TODO_STATUS_NAME = exports.AWAITING_QUALITY_CHECK_STATUS_NAME = exports.FAILED_PREPARATION_STATUS_NAME = exports.PREPARATION_STATUS_NAME = exports.AWAITING_WORKSPACE_STATUS_NAME = exports.AWAITING_TASK_BREAKDOWN_STATUS_NAME = exports.DEFAULT_STATUS_NAME = void 0;
 exports.DEFAULT_STATUS_NAME = 'Unread';
 exports.AWAITING_TASK_BREAKDOWN_STATUS_NAME = 'Awaiting Task Breakdown';
 exports.AWAITING_WORKSPACE_STATUS_NAME = 'Awaiting Workspace';
 exports.PREPARATION_STATUS_NAME = 'Preparation';
 exports.FAILED_PREPARATION_STATUS_NAME = 'Failed Preparation';
 exports.AWAITING_QUALITY_CHECK_STATUS_NAME = 'Awaiting Quality Check';
-exports.TODO_STATUS_NAME = 'Todo';
+exports.TODO_STATUS_NAME = 'Todo by human';
 exports.PC_TODO_STATUS_NAME = 'PC Todo';
-exports.IN_TMUX_STATUS_NAME = 'In Tmux';
+exports.IN_TMUX_STATUS_NAME = 'In Tmux by human';
 exports.DONE_STATUS_NAME = 'Done';
 exports.ICEBOX_STATUS_NAME = 'Icebox';
+exports.LEGACY_TODO_STATUS_NAME = 'Todo';
+exports.LEGACY_IN_TMUX_STATUS_NAME = 'In Tmux';
 exports.REQUIRED_WORKFLOW_STATUSES = [
     {
         name: exports.DEFAULT_STATUS_NAME,
@@ -41,10 +43,6 @@ exports.REQUIRED_WORKFLOW_STATUSES = [
         name: exports.TODO_STATUS_NAME,
         color: 'PINK',
     },
-    {
-        name: exports.PC_TODO_STATUS_NAME,
-        color: 'PINK',
-    },
     {
         name: exports.IN_TMUX_STATUS_NAME,
         color: 'RED',

package/bin/domain/entities/WorkflowStatus.js.map

@@ -1 +1 @@
-{"version":3,"file":"WorkflowStatus.js","sourceRoot":"","sources":["../../../src/domain/entities/WorkflowStatus.ts"],"names":[],"mappings":";;;AAEa,QAAA,mBAAmB,GAAG,QAAQ,CAAC;AAC/B,QAAA,mCAAmC,GAAG,yBAAyB,CAAC;AAChE,QAAA,8BAA8B,GAAG,oBAAoB,CAAC;AACtD,QAAA,uBAAuB,GAAG,aAAa,CAAC;AACxC,QAAA,8BAA8B,GAAG,oBAAoB,CAAC;AACtD,QAAA,kCAAkC,GAAG,wBAAwB,CAAC;AAC9D,QAAA,gBAAgB,GAAG,MAAM,CAAC;AAC1B,QAAA,mBAAmB,GAAG,SAAS,CAAC;AAChC,QAAA,mBAAmB,GAAG,SAAS,CAAC;AAChC,QAAA,gBAAgB,GAAG,MAAM,CAAC;AAC1B,QAAA,kBAAkB,GAAG,QAAQ,CAAC;AAO9B,QAAA,0BAA0B,GAA+B;IACpE;QACE,IAAI,EAAE,2BAAmB;QACzB,KAAK,EAAE,QAAQ;KAChB;IACD;QACE,IAAI,EAAE,2CAAmC;QACzC,KAAK,EAAE,QAAQ;KAChB;IACD;QACE,IAAI,EAAE,sCAA8B;QACpC,KAAK,EAAE,MAAM;KACd;IACD;QACE,IAAI,EAAE,+BAAuB;QAC7B,KAAK,EAAE,QAAQ;KAChB;IACD;QACE,IAAI,EAAE,sCAA8B;QACpC,KAAK,EAAE,KAAK;KACb;IACD;QACE,IAAI,EAAE,0CAAkC;QACxC,KAAK,EAAE,OAAO;KACf;IACD;QACE,IAAI,EAAE,wBAAgB;QACtB,KAAK,EAAE,MAAM;KACd;IACD;QACE,IAAI,EAAE,2BAAmB;QACzB,KAAK,EAAE,MAAM;KACd;IACD;QACE,IAAI,EAAE,2BAAmB;QACzB,KAAK,EAAE,KAAK;KACb;IACD;QACE,IAAI,EAAE,wBAAgB;QACtB,KAAK,EAAE,QAAQ;KAChB;IACD;QACE,IAAI,EAAE,0BAAkB;QACxB,KAAK,EAAE,MAAM;KACd;CACF,CAAC"}
+{"version":3,"file":"WorkflowStatus.js","sourceRoot":"","sources":["../../../src/domain/entities/WorkflowStatus.ts"],"names":[],"mappings":";;;AAEa,QAAA,mBAAmB,GAAG,QAAQ,CAAC;AAC/B,QAAA,mCAAmC,GAAG,yBAAyB,CAAC;AAChE,QAAA,8BAA8B,GAAG,oBAAoB,CAAC;AACtD,QAAA,uBAAuB,GAAG,aAAa,CAAC;AACxC,QAAA,8BAA8B,GAAG,oBAAoB,CAAC;AACtD,QAAA,kCAAkC,GAAG,wBAAwB,CAAC;AAC9D,QAAA,gBAAgB,GAAG,eAAe,CAAC;AACnC,QAAA,mBAAmB,GAAG,SAAS,CAAC;AAChC,QAAA,mBAAmB,GAAG,kBAAkB,CAAC;AACzC,QAAA,gBAAgB,GAAG,MAAM,CAAC;AAC1B,QAAA,kBAAkB,GAAG,QAAQ,CAAC;AAE9B,QAAA,uBAAuB,GAAG,MAAM,CAAC;AACjC,QAAA,0BAA0B,GAAG,SAAS,CAAC;AAOvC,QAAA,0BAA0B,GAA+B;IACpE;QACE,IAAI,EAAE,2BAAmB;QACzB,KAAK,EAAE,QAAQ;KAChB;IACD;QACE,IAAI,EAAE,2CAAmC;QACzC,KAAK,EAAE,QAAQ;KAChB;IACD;QACE,IAAI,EAAE,sCAA8B;QACpC,KAAK,EAAE,MAAM;KACd;IACD;QACE,IAAI,EAAE,+BAAuB;QAC7B,KAAK,EAAE,QAAQ;KAChB;IACD;QACE,IAAI,EAAE,sCAA8B;QACpC,KAAK,EAAE,KAAK;KACb;IACD;QACE,IAAI,EAAE,0CAAkC;QACxC,KAAK,EAAE,OAAO;KACf;IACD;QACE,IAAI,EAAE,wBAAgB;QACtB,KAAK,EAAE,MAAM;KACd;IACD;QACE,IAAI,EAAE,2BAAmB;QACzB,KAAK,EAAE,KAAK;KACb;IACD;QACE,IAAI,EAAE,wBAAgB;QACtB,KAAK,EAAE,QAAQ;KAChB;IACD;QACE,IAAI,EAAE,0BAAkB;QACxB,KAAK,EAAE,MAAM;KACd;CACF,CAAC"}

package/bin/domain/usecases/SetupTowerDefenceProjectUseCase.js

@@ -8,14 +8,21 @@ class SetupTowerDefenceProjectUseCase {
         this.run = async (params) => {
             const project = await this.projectRepository.getByUrl(params.projectUrl);
             const existing = project.status.statuses;
-            if (SetupTowerDefenceProjectUseCase.hasRequiredStatusesInCanonicalOrder(existing)) {
+            const hasMigratedFromName = existing.some((s) => SetupTowerDefenceProjectUseCase.MIGRATED_FROM_NAMES.has(s.name));
+            if (!hasMigratedFromName &&
+                SetupTowerDefenceProjectUseCase.hasRequiredStatusesInCanonicalOrder(existing)) {
                 return;
             }
             const requiredNames = new Set(WorkflowStatus_1.REQUIRED_WORKFLOW_STATUSES.map((s) => s.name));
-            const others = existing.filter((status) => !requiredNames.has(status.name));
+            const others = existing.filter((status) => !requiredNames.has(status.name) &&
+                !SetupTowerDefenceProjectUseCase.MIGRATED_FROM_NAMES.has(status.name));
             const newStatusList = [
                 ...WorkflowStatus_1.REQUIRED_WORKFLOW_STATUSES.map((required) => {
-                    const found = existing.find((status) => status.name === required.name);
+                    const legacyName = SetupTowerDefenceProjectUseCase.LEGACY_STATUS_NAMES[required.name];
+                    const found = existing.find((status) => status.name === required.name) ??
+                        (legacyName !== undefined
+                            ? existing.find((status) => status.name === legacyName)
+                            : undefined);
                     return {
                         id: found ? found.id : null,
                         name: required.name,
@@ -35,6 +42,15 @@ class SetupTowerDefenceProjectUseCase {
     }
 }
 exports.SetupTowerDefenceProjectUseCase = SetupTowerDefenceProjectUseCase;
+SetupTowerDefenceProjectUseCase.LEGACY_STATUS_NAMES = {
+    [WorkflowStatus_1.TODO_STATUS_NAME]: WorkflowStatus_1.LEGACY_TODO_STATUS_NAME,
+    [WorkflowStatus_1.IN_TMUX_STATUS_NAME]: WorkflowStatus_1.LEGACY_IN_TMUX_STATUS_NAME,
+};
+SetupTowerDefenceProjectUseCase.MIGRATED_FROM_NAMES = new Set([
+    WorkflowStatus_1.LEGACY_TODO_STATUS_NAME,
+    WorkflowStatus_1.LEGACY_IN_TMUX_STATUS_NAME,
+    WorkflowStatus_1.PC_TODO_STATUS_NAME,
+]);
 SetupTowerDefenceProjectUseCase.hasRequiredStatusesInCanonicalOrder = (existing) => {
     if (existing.length < WorkflowStatus_1.REQUIRED_WORKFLOW_STATUSES.length) {
         return false;

package/bin/domain/usecases/SetupTowerDefenceProjectUseCase.js.map

@@ -1 +1 @@
-{"version":3,"file":"SetupTowerDefenceProjectUseCase.js","sourceRoot":"","sources":["../../../src/domain/usecases/SetupTowerDefenceProjectUseCase.ts"],"names":[],"mappings":";;;AAEA,+DAGoC;AAEpC,MAAa,+BAA+B;IAC1C,YACmB,iBAGhB;QAHgB,sBAAiB,GAAjB,iBAAiB,CAGjC;QAGH,QAAG,GAAG,KAAK,EAAE,MAA8B,EAAiB,EAAE;YAC5D,MAAM,OAAO,GAAG,MAAM,IAAI,CAAC,iBAAiB,CAAC,QAAQ,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC;YACzE,MAAM,QAAQ,GAAG,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC;YAEzC,IACE,+BAA+B,CAAC,mCAAmC,CACjE,QAAQ,CACT,EACD,CAAC;gBACD,OAAO;YACT,CAAC;YAED,MAAM,aAAa,GAAG,IAAI,GAAG,CAC3B,2CAA0B,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAC9C,CAAC;YACF,MAAM,MAAM,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC,aAAa,CAAC,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC;YAE5E,MAAM,aAAa,GAEZ;gBACL,GAAG,2CAA0B,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE,EAAE;oBAC7C,MAAM,KAAK,GAAG,QAAQ,CAAC,IAAI,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC,MAAM,CAAC,IAAI,KAAK,QAAQ,CAAC,IAAI,CAAC,CAAC;oBACvE,OAAO;wBACL,EAAE,EAAE,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI;wBAC3B,IAAI,EAAE,QAAQ,CAAC,IAAI;wBACnB,KAAK,EAAE,QAAQ,CAAC,KAAK;wBACrB,WAAW,EAAE,EAAE;qBAChB,CAAC;gBACJ,CAAC,CAAC;gBACF,GAAG,MAAM,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC;oBACxB,EAAE,EAAE,KAAK,CAAC,EAAE;oBACZ,IAAI,EAAE,KAAK,CAAC,IAAI;oBAChB,KAAK,EAAE,KAAK,CAAC,KAAK;oBAClB,WAAW,EAAE,KAAK,CAAC,WAAW;iBAC/B,CAAC,CAAC;aACJ,CAAC;YAEF,MAAM,IAAI,CAAC,iBAAiB,CAAC,gBAAgB,CAAC,OAAO,EAAE,aAAa,CAAC,CAAC;QACxE,CAAC,CAAC;IAxCC,CAAC;;AANN,0EAiEC;AAjBgB,mEAAmC,GAAG,CACnD,QAAuB,EACd,EAAE;IACX,IAAI,QAAQ,CAAC,MAAM,GAAG,2CAA0B,CAAC,MAAM,EAAE,CAAC;QACxD,OAAO,KAAK,CAAC;IACf,CAAC;IACD,OAAO,2CAA0B,CAAC,KAAK,CACrC,CAAC,QAAkC,EAAE,KAAa,EAAE,EAAE;QACpD,MAAM,MAAM,GAAG,QAAQ,CAAC,KAAK,CAAC,CAAC;QAC/B,OAAO,CACL,MAAM,CAAC,IAAI,KAAK,QAAQ,CAAC,IAAI;YAC7B,MAAM,CAAC,KAAK,KAAK,QAAQ,CAAC,KAAK;YAC/B,MAAM,CAAC,WAAW,KAAK,EAAE,CAC1B,CAAC;IACJ,CAAC,CACF,CAAC;AACJ,CAAC,AAhBiD,CAgBhD"}
+{"version":3,"file":"SetupTowerDefenceProjectUseCase.js","sourceRoot":"","sources":["../../../src/domain/usecases/SetupTowerDefenceProjectUseCase.ts"],"names":[],"mappings":";;;AAEA,+DAQoC;AAEpC,MAAa,+BAA+B;IAC1C,YACmB,iBAGhB;QAHgB,sBAAiB,GAAjB,iBAAiB,CAGjC;QAgBH,QAAG,GAAG,KAAK,EAAE,MAA8B,EAAiB,EAAE;YAC5D,MAAM,OAAO,GAAG,MAAM,IAAI,CAAC,iBAAiB,CAAC,QAAQ,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC;YACzE,MAAM,QAAQ,GAAG,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC;YAEzC,MAAM,mBAAmB,GAAG,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAC9C,+BAA+B,CAAC,mBAAmB,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,CAChE,CAAC;YACF,IACE,CAAC,mBAAmB;gBACpB,+BAA+B,CAAC,mCAAmC,CACjE,QAAQ,CACT,EACD,CAAC;gBACD,OAAO;YACT,CAAC;YAED,MAAM,aAAa,GAAG,IAAI,GAAG,CAC3B,2CAA0B,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAC9C,CAAC;YACF,MAAM,MAAM,GAAG,QAAQ,CAAC,MAAM,CAC5B,CAAC,MAAM,EAAE,EAAE,CACT,CAAC,aAAa,CAAC,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC;gBAC/B,CAAC,+BAA+B,CAAC,mBAAmB,CAAC,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC,CACxE,CAAC;YAEF,MAAM,aAAa,GAEZ;gBACL,GAAG,2CAA0B,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE,EAAE;oBAC7C,MAAM,UAAU,GACd,+BAA+B,CAAC,mBAAmB,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;oBACrE,MAAM,KAAK,GACT,QAAQ,CAAC,IAAI,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC,MAAM,CAAC,IAAI,KAAK,QAAQ,CAAC,IAAI,CAAC;wBACxD,CAAC,UAAU,KAAK,SAAS;4BACvB,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC,MAAM,CAAC,IAAI,KAAK,UAAU,CAAC;4BACvD,CAAC,CAAC,SAAS,CAAC,CAAC;oBACjB,OAAO;wBACL,EAAE,EAAE,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI;wBAC3B,IAAI,EAAE,QAAQ,CAAC,IAAI;wBACnB,KAAK,EAAE,QAAQ,CAAC,KAAK;wBACrB,WAAW,EAAE,EAAE;qBAChB,CAAC;gBACJ,CAAC,CAAC;gBACF,GAAG,MAAM,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC;oBACxB,EAAE,EAAE,KAAK,CAAC,EAAE;oBACZ,IAAI,EAAE,KAAK,CAAC,IAAI;oBAChB,KAAK,EAAE,KAAK,CAAC,KAAK;oBAClB,WAAW,EAAE,KAAK,CAAC,WAAW;iBAC/B,CAAC,CAAC;aACJ,CAAC;YAEF,MAAM,IAAI,CAAC,iBAAiB,CAAC,gBAAgB,CAAC,OAAO,EAAE,aAAa,CAAC,CAAC;QACxE,CAAC,CAAC;IAnEC,CAAC;;AANN,0EA4FC;AApFyB,mDAAmB,GAEvC;IACF,CAAC,iCAAgB,CAAC,EAAE,wCAAuB;IAC3C,CAAC,oCAAmB,CAAC,EAAE,2CAA0B;CAClD,AAL0C,CAKzC;AAEsB,mDAAmB,GAAwB,IAAI,GAAG,CAAC;IACzE,wCAAuB;IACvB,2CAA0B;IAC1B,oCAAmB;CACpB,CAAC,AAJyC,CAIxC;AAwDY,mEAAmC,GAAG,CACnD,QAAuB,EACd,EAAE;IACX,IAAI,QAAQ,CAAC,MAAM,GAAG,2CAA0B,CAAC,MAAM,EAAE,CAAC;QACxD,OAAO,KAAK,CAAC;IACf,CAAC;IACD,OAAO,2CAA0B,CAAC,KAAK,CACrC,CAAC,QAAkC,EAAE,KAAa,EAAE,EAAE;QACpD,MAAM,MAAM,GAAG,QAAQ,CAAC,KAAK,CAAC,CAAC;QAC/B,OAAO,CACL,MAAM,CAAC,IAAI,KAAK,QAAQ,CAAC,IAAI;YAC7B,MAAM,CAAC,KAAK,KAAK,QAAQ,CAAC,KAAK;YAC/B,MAAM,CAAC,WAAW,KAAK,EAAE,CAC1B,CAAC;IACJ,CAAC,CACF,CAAC;AACJ,CAAC,AAhBiD,CAgBhD"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "github-issue-tower-defence-management",
-  "version": "1.53.0",
+  "version": "1.54.0",
   "description": "",
   "main": "bin/index.js",
   "scripts": {

package/src/domain/entities/WorkflowStatus.ts

@@ -6,12 +6,15 @@ export const AWAITING_WORKSPACE_STATUS_NAME = 'Awaiting Workspace';
 export const PREPARATION_STATUS_NAME = 'Preparation';
 export const FAILED_PREPARATION_STATUS_NAME = 'Failed Preparation';
 export const AWAITING_QUALITY_CHECK_STATUS_NAME = 'Awaiting Quality Check';
-export const TODO_STATUS_NAME = 'Todo';
+export const TODO_STATUS_NAME = 'Todo by human';
 export const PC_TODO_STATUS_NAME = 'PC Todo';
-export const IN_TMUX_STATUS_NAME = 'In Tmux';
+export const IN_TMUX_STATUS_NAME = 'In Tmux by human';
 export const DONE_STATUS_NAME = 'Done';
 export const ICEBOX_STATUS_NAME = 'Icebox';
 
+export const LEGACY_TODO_STATUS_NAME = 'Todo';
+export const LEGACY_IN_TMUX_STATUS_NAME = 'In Tmux';
+
 export type WorkflowStatusDefinition = {
   name: string;
   color: FieldOption['color'];
@@ -46,10 +49,6 @@ export const REQUIRED_WORKFLOW_STATUSES: WorkflowStatusDefinition[] = [
     name: TODO_STATUS_NAME,
     color: 'PINK',
   },
-  {
-    name: PC_TODO_STATUS_NAME,
-    color: 'PINK',
-  },
   {
     name: IN_TMUX_STATUS_NAME,
     color: 'RED',

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

@@ -11,6 +11,8 @@ import {
   FAILED_PREPARATION_STATUS_NAME,
   ICEBOX_STATUS_NAME,
   IN_TMUX_STATUS_NAME,
+  LEGACY_IN_TMUX_STATUS_NAME,
+  LEGACY_TODO_STATUS_NAME,
   PC_TODO_STATUS_NAME,
   PREPARATION_STATUS_NAME,
   REQUIRED_WORKFLOW_STATUSES,
@@ -44,7 +46,7 @@ const buildCanonicalStatuses = (): FieldOption[] =>
   }));
 
 describe('SetupTowerDefenceProjectUseCase', () => {
-  it('should define exactly the 11 required statuses in the documented order with the documented colors and no descriptions', () => {
+  it('should define exactly the 10 required statuses in the documented order with the documented colors and no descriptions', () => {
     expect(REQUIRED_WORKFLOW_STATUSES).toEqual([
       { name: DEFAULT_STATUS_NAME, color: 'ORANGE' },
       { name: AWAITING_TASK_BREAKDOWN_STATUS_NAME, color: 'ORANGE' },
@@ -53,7 +55,6 @@ describe('SetupTowerDefenceProjectUseCase', () => {
       { name: FAILED_PREPARATION_STATUS_NAME, color: 'RED' },
       { name: AWAITING_QUALITY_CHECK_STATUS_NAME, color: 'GREEN' },
       { name: TODO_STATUS_NAME, color: 'PINK' },
-      { name: PC_TODO_STATUS_NAME, color: 'PINK' },
       { name: IN_TMUX_STATUS_NAME, color: 'RED' },
       { name: DONE_STATUS_NAME, color: 'PURPLE' },
       { name: ICEBOX_STATUS_NAME, color: 'GRAY' },
@@ -187,12 +188,6 @@ describe('SetupTowerDefenceProjectUseCase', () => {
           color: 'PINK',
           description: '',
         },
-        {
-          id: null,
-          name: PC_TODO_STATUS_NAME,
-          color: 'PINK',
-          description: '',
-        },
         {
           id: null,
           name: IN_TMUX_STATUS_NAME,
@@ -249,7 +244,6 @@ describe('SetupTowerDefenceProjectUseCase', () => {
       FAILED_PREPARATION_STATUS_NAME,
       AWAITING_QUALITY_CHECK_STATUS_NAME,
       TODO_STATUS_NAME,
-      PC_TODO_STATUS_NAME,
       IN_TMUX_STATUS_NAME,
       DONE_STATUS_NAME,
       ICEBOX_STATUS_NAME,
@@ -272,4 +266,178 @@ describe('SetupTowerDefenceProjectUseCase', () => {
     const [, payload] = mockProjectRepository.updateStatusList.mock.calls[0];
     expect(payload[2].color).toBe(REQUIRED_WORKFLOW_STATUSES[2].color);
   });
+
+  it('should rename legacy "Todo" to "Todo by human" by reusing the existing option ID', async () => {
+    const mockProjectRepository =
+      mock<Pick<ProjectRepository, 'getByUrl' | 'updateStatusList'>>();
+    const statuses: FieldOption[] = REQUIRED_WORKFLOW_STATUSES.map(
+      (required, index) => ({
+        id: `id-${index}`,
+        name:
+          required.name === TODO_STATUS_NAME
+            ? LEGACY_TODO_STATUS_NAME
+            : required.name,
+        color: required.color,
+        description: '',
+      }),
+    );
+    const project = buildProject(statuses);
+    mockProjectRepository.getByUrl.mockResolvedValue(project);
+    mockProjectRepository.updateStatusList.mockResolvedValue([]);
+
+    const useCase = new SetupTowerDefenceProjectUseCase(mockProjectRepository);
+    await useCase.run({ projectUrl: project.url });
+
+    expect(mockProjectRepository.updateStatusList).toHaveBeenCalledTimes(1);
+    const [, payload] = mockProjectRepository.updateStatusList.mock.calls[0];
+    const todoEntry = payload.find((s) => s.name === TODO_STATUS_NAME);
+    expect(todoEntry).toBeDefined();
+    expect(todoEntry?.id).toBe('id-6');
+    expect(payload.some((s) => s.name === LEGACY_TODO_STATUS_NAME)).toBe(false);
+  });
+
+  it('should rename legacy "In Tmux" to "In Tmux by human" by reusing the existing option ID', async () => {
+    const mockProjectRepository =
+      mock<Pick<ProjectRepository, 'getByUrl' | 'updateStatusList'>>();
+    const statuses: FieldOption[] = REQUIRED_WORKFLOW_STATUSES.map(
+      (required, index) => ({
+        id: `id-${index}`,
+        name:
+          required.name === IN_TMUX_STATUS_NAME
+            ? LEGACY_IN_TMUX_STATUS_NAME
+            : required.name,
+        color: required.color,
+        description: '',
+      }),
+    );
+    const project = buildProject(statuses);
+    mockProjectRepository.getByUrl.mockResolvedValue(project);
+    mockProjectRepository.updateStatusList.mockResolvedValue([]);
+
+    const useCase = new SetupTowerDefenceProjectUseCase(mockProjectRepository);
+    await useCase.run({ projectUrl: project.url });
+
+    expect(mockProjectRepository.updateStatusList).toHaveBeenCalledTimes(1);
+    const [, payload] = mockProjectRepository.updateStatusList.mock.calls[0];
+    const inTmuxEntry = payload.find((s) => s.name === IN_TMUX_STATUS_NAME);
+    expect(inTmuxEntry).toBeDefined();
+    expect(inTmuxEntry?.id).toBe('id-7');
+    expect(payload.some((s) => s.name === LEGACY_IN_TMUX_STATUS_NAME)).toBe(
+      false,
+    );
+  });
+
+  it('should remove "PC Todo" from the status list and not include it in others', async () => {
+    const mockProjectRepository =
+      mock<Pick<ProjectRepository, 'getByUrl' | 'updateStatusList'>>();
+    const statuses: FieldOption[] = [
+      ...buildCanonicalStatuses(),
+      {
+        id: 'pc-todo-id',
+        name: PC_TODO_STATUS_NAME,
+        color: 'PINK',
+        description: '',
+      },
+    ];
+    const project = buildProject(statuses);
+    mockProjectRepository.getByUrl.mockResolvedValue(project);
+    mockProjectRepository.updateStatusList.mockResolvedValue([]);
+
+    const useCase = new SetupTowerDefenceProjectUseCase(mockProjectRepository);
+    await useCase.run({ projectUrl: project.url });
+
+    expect(mockProjectRepository.updateStatusList).toHaveBeenCalledTimes(1);
+    const [, payload] = mockProjectRepository.updateStatusList.mock.calls[0];
+    expect(payload.some((s) => s.name === PC_TODO_STATUS_NAME)).toBe(false);
+  });
+
+  it('should migrate a project with legacy statuses: rename Todo and In Tmux by ID, remove PC Todo', async () => {
+    const mockProjectRepository =
+      mock<Pick<ProjectRepository, 'getByUrl' | 'updateStatusList'>>();
+    const statuses: FieldOption[] = [
+      {
+        id: 'id-0',
+        name: DEFAULT_STATUS_NAME,
+        color: 'ORANGE',
+        description: '',
+      },
+      {
+        id: 'id-1',
+        name: AWAITING_TASK_BREAKDOWN_STATUS_NAME,
+        color: 'ORANGE',
+        description: '',
+      },
+      {
+        id: 'id-2',
+        name: AWAITING_WORKSPACE_STATUS_NAME,
+        color: 'BLUE',
+        description: '',
+      },
+      {
+        id: 'id-3',
+        name: PREPARATION_STATUS_NAME,
+        color: 'YELLOW',
+        description: '',
+      },
+      {
+        id: 'id-4',
+        name: FAILED_PREPARATION_STATUS_NAME,
+        color: 'RED',
+        description: '',
+      },
+      {
+        id: 'id-5',
+        name: AWAITING_QUALITY_CHECK_STATUS_NAME,
+        color: 'GREEN',
+        description: '',
+      },
+      {
+        id: 'id-6',
+        name: LEGACY_TODO_STATUS_NAME,
+        color: 'PINK',
+        description: '',
+      },
+      { id: 'id-7', name: PC_TODO_STATUS_NAME, color: 'PINK', description: '' },
+      {
+        id: 'id-8',
+        name: LEGACY_IN_TMUX_STATUS_NAME,
+        color: 'RED',
+        description: '',
+      },
+      { id: 'id-9', name: DONE_STATUS_NAME, color: 'PURPLE', description: '' },
+      { id: 'id-10', name: ICEBOX_STATUS_NAME, color: 'GRAY', description: '' },
+    ];
+    const project = buildProject(statuses);
+    mockProjectRepository.getByUrl.mockResolvedValue(project);
+    mockProjectRepository.updateStatusList.mockResolvedValue([]);
+
+    const useCase = new SetupTowerDefenceProjectUseCase(mockProjectRepository);
+    await useCase.run({ projectUrl: project.url });
+
+    expect(mockProjectRepository.updateStatusList).toHaveBeenCalledTimes(1);
+    const [, payload] = mockProjectRepository.updateStatusList.mock.calls[0];
+
+    expect(payload.map((s) => s.name)).toEqual([
+      DEFAULT_STATUS_NAME,
+      AWAITING_TASK_BREAKDOWN_STATUS_NAME,
+      AWAITING_WORKSPACE_STATUS_NAME,
+      PREPARATION_STATUS_NAME,
+      FAILED_PREPARATION_STATUS_NAME,
+      AWAITING_QUALITY_CHECK_STATUS_NAME,
+      TODO_STATUS_NAME,
+      IN_TMUX_STATUS_NAME,
+      DONE_STATUS_NAME,
+      ICEBOX_STATUS_NAME,
+    ]);
+
+    expect(payload.find((s) => s.name === TODO_STATUS_NAME)?.id).toBe('id-6');
+    expect(payload.find((s) => s.name === IN_TMUX_STATUS_NAME)?.id).toBe(
+      'id-8',
+    );
+    expect(payload.some((s) => s.name === PC_TODO_STATUS_NAME)).toBe(false);
+    expect(payload.some((s) => s.name === LEGACY_TODO_STATUS_NAME)).toBe(false);
+    expect(payload.some((s) => s.name === LEGACY_IN_TMUX_STATUS_NAME)).toBe(
+      false,
+    );
+  });
 });

package/src/domain/usecases/SetupTowerDefenceProjectUseCase.ts

@@ -1,7 +1,12 @@
 import { FieldOption } from '../entities/Project';
 import { ProjectRepository } from './adapter-interfaces/ProjectRepository';
 import {
+  IN_TMUX_STATUS_NAME,
+  LEGACY_IN_TMUX_STATUS_NAME,
+  LEGACY_TODO_STATUS_NAME,
+  PC_TODO_STATUS_NAME,
   REQUIRED_WORKFLOW_STATUSES,
+  TODO_STATUS_NAME,
   WorkflowStatusDefinition,
 } from '../entities/WorkflowStatus';
 
@@ -13,11 +18,28 @@ export class SetupTowerDefenceProjectUseCase {
     >,
   ) {}
 
+  private static readonly LEGACY_STATUS_NAMES: Readonly<
+    Record<string, string>
+  > = {
+    [TODO_STATUS_NAME]: LEGACY_TODO_STATUS_NAME,
+    [IN_TMUX_STATUS_NAME]: LEGACY_IN_TMUX_STATUS_NAME,
+  };
+
+  private static readonly MIGRATED_FROM_NAMES: ReadonlySet<string> = new Set([
+    LEGACY_TODO_STATUS_NAME,
+    LEGACY_IN_TMUX_STATUS_NAME,
+    PC_TODO_STATUS_NAME,
+  ]);
+
   run = async (params: { projectUrl: string }): Promise<void> => {
     const project = await this.projectRepository.getByUrl(params.projectUrl);
     const existing = project.status.statuses;
 
+    const hasMigratedFromName = existing.some((s) =>
+      SetupTowerDefenceProjectUseCase.MIGRATED_FROM_NAMES.has(s.name),
+    );
     if (
+      !hasMigratedFromName &&
       SetupTowerDefenceProjectUseCase.hasRequiredStatusesInCanonicalOrder(
         existing,
       )
@@ -28,13 +50,23 @@ export class SetupTowerDefenceProjectUseCase {
     const requiredNames = new Set(
       REQUIRED_WORKFLOW_STATUSES.map((s) => s.name),
     );
-    const others = existing.filter((status) => !requiredNames.has(status.name));
+    const others = existing.filter(
+      (status) =>
+        !requiredNames.has(status.name) &&
+        !SetupTowerDefenceProjectUseCase.MIGRATED_FROM_NAMES.has(status.name),
+    );
 
     const newStatusList: (Omit<FieldOption, 'id'> & {
       id: FieldOption['id'] | null;
     })[] = [
       ...REQUIRED_WORKFLOW_STATUSES.map((required) => {
-        const found = existing.find((status) => status.name === required.name);
+        const legacyName =
+          SetupTowerDefenceProjectUseCase.LEGACY_STATUS_NAMES[required.name];
+        const found =
+          existing.find((status) => status.name === required.name) ??
+          (legacyName !== undefined
+            ? existing.find((status) => status.name === legacyName)
+            : undefined);
         return {
           id: found ? found.id : null,
           name: required.name,

package/types/domain/entities/WorkflowStatus.d.ts

@@ -5,11 +5,13 @@ export declare const AWAITING_WORKSPACE_STATUS_NAME = "Awaiting Workspace";
 export declare const PREPARATION_STATUS_NAME = "Preparation";
 export declare const FAILED_PREPARATION_STATUS_NAME = "Failed Preparation";
 export declare const AWAITING_QUALITY_CHECK_STATUS_NAME = "Awaiting Quality Check";
-export declare const TODO_STATUS_NAME = "Todo";
+export declare const TODO_STATUS_NAME = "Todo by human";
 export declare const PC_TODO_STATUS_NAME = "PC Todo";
-export declare const IN_TMUX_STATUS_NAME = "In Tmux";
+export declare const IN_TMUX_STATUS_NAME = "In Tmux by human";
 export declare const DONE_STATUS_NAME = "Done";
 export declare const ICEBOX_STATUS_NAME = "Icebox";
+export declare const LEGACY_TODO_STATUS_NAME = "Todo";
+export declare const LEGACY_IN_TMUX_STATUS_NAME = "In Tmux";
 export type WorkflowStatusDefinition = {
     name: string;
     color: FieldOption['color'];

package/types/domain/entities/WorkflowStatus.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"WorkflowStatus.d.ts","sourceRoot":"","sources":["../../../src/domain/entities/WorkflowStatus.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,WAAW,EAAE,MAAM,WAAW,CAAC;AAExC,eAAO,MAAM,mBAAmB,WAAW,CAAC;AAC5C,eAAO,MAAM,mCAAmC,4BAA4B,CAAC;AAC7E,eAAO,MAAM,8BAA8B,uBAAuB,CAAC;AACnE,eAAO,MAAM,uBAAuB,gBAAgB,CAAC;AACrD,eAAO,MAAM,8BAA8B,uBAAuB,CAAC;AACnE,eAAO,MAAM,kCAAkC,2BAA2B,CAAC;AAC3E,eAAO,MAAM,gBAAgB,SAAS,CAAC;AACvC,eAAO,MAAM,mBAAmB,YAAY,CAAC;AAC7C,eAAO,MAAM,mBAAmB,YAAY,CAAC;AAC7C,eAAO,MAAM,gBAAgB,SAAS,CAAC;AACvC,eAAO,MAAM,kBAAkB,WAAW,CAAC;AAE3C,MAAM,MAAM,wBAAwB,GAAG;IACrC,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,WAAW,CAAC,OAAO,CAAC,CAAC;CAC7B,CAAC;AAEF,eAAO,MAAM,0BAA0B,EAAE,wBAAwB,EA6ChE,CAAC"}
+{"version":3,"file":"WorkflowStatus.d.ts","sourceRoot":"","sources":["../../../src/domain/entities/WorkflowStatus.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,WAAW,EAAE,MAAM,WAAW,CAAC;AAExC,eAAO,MAAM,mBAAmB,WAAW,CAAC;AAC5C,eAAO,MAAM,mCAAmC,4BAA4B,CAAC;AAC7E,eAAO,MAAM,8BAA8B,uBAAuB,CAAC;AACnE,eAAO,MAAM,uBAAuB,gBAAgB,CAAC;AACrD,eAAO,MAAM,8BAA8B,uBAAuB,CAAC;AACnE,eAAO,MAAM,kCAAkC,2BAA2B,CAAC;AAC3E,eAAO,MAAM,gBAAgB,kBAAkB,CAAC;AAChD,eAAO,MAAM,mBAAmB,YAAY,CAAC;AAC7C,eAAO,MAAM,mBAAmB,qBAAqB,CAAC;AACtD,eAAO,MAAM,gBAAgB,SAAS,CAAC;AACvC,eAAO,MAAM,kBAAkB,WAAW,CAAC;AAE3C,eAAO,MAAM,uBAAuB,SAAS,CAAC;AAC9C,eAAO,MAAM,0BAA0B,YAAY,CAAC;AAEpD,MAAM,MAAM,wBAAwB,GAAG;IACrC,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,WAAW,CAAC,OAAO,CAAC,CAAC;CAC7B,CAAC;AAEF,eAAO,MAAM,0BAA0B,EAAE,wBAAwB,EAyChE,CAAC"}

package/types/domain/usecases/SetupTowerDefenceProjectUseCase.d.ts

@@ -2,6 +2,8 @@ import { ProjectRepository } from './adapter-interfaces/ProjectRepository';
 export declare class SetupTowerDefenceProjectUseCase {
     private readonly projectRepository;
     constructor(projectRepository: Pick<ProjectRepository, 'getByUrl' | 'updateStatusList'>);
+    private static readonly LEGACY_STATUS_NAMES;
+    private static readonly MIGRATED_FROM_NAMES;
     run: (params: {
         projectUrl: string;
     }) => Promise<void>;

package/types/domain/usecases/SetupTowerDefenceProjectUseCase.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"SetupTowerDefenceProjectUseCase.d.ts","sourceRoot":"","sources":["../../../src/domain/usecases/SetupTowerDefenceProjectUseCase.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,iBAAiB,EAAE,MAAM,wCAAwC,CAAC;AAM3E,qBAAa,+BAA+B;IAExC,OAAO,CAAC,QAAQ,CAAC,iBAAiB;gBAAjB,iBAAiB,EAAE,IAAI,CACtC,iBAAiB,EACjB,UAAU,GAAG,kBAAkB,CAChC;IAGH,GAAG,GAAU,QAAQ;QAAE,UAAU,EAAE,MAAM,CAAA;KAAE,KAAG,OAAO,CAAC,IAAI,CAAC,CAsCzD;IAEF,OAAO,CAAC,MAAM,CAAC,mCAAmC,CAgBhD;CACH"}
+{"version":3,"file":"SetupTowerDefenceProjectUseCase.d.ts","sourceRoot":"","sources":["../../../src/domain/usecases/SetupTowerDefenceProjectUseCase.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,iBAAiB,EAAE,MAAM,wCAAwC,CAAC;AAW3E,qBAAa,+BAA+B;IAExC,OAAO,CAAC,QAAQ,CAAC,iBAAiB;gBAAjB,iBAAiB,EAAE,IAAI,CACtC,iBAAiB,EACjB,UAAU,GAAG,kBAAkB,CAChC;IAGH,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,mBAAmB,CAKzC;IAEF,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,mBAAmB,CAIxC;IAEH,GAAG,GAAU,QAAQ;QAAE,UAAU,EAAE,MAAM,CAAA;KAAE,KAAG,OAAO,CAAC,IAAI,CAAC,CAoDzD;IAEF,OAAO,CAAC,MAAM,CAAC,mCAAmC,CAgBhD;CACH"}