github-issue-tower-defence-management 1.38.0 → 1.38.1

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [1.38.1](https://github.com/HiromiShikata/npm-cli-github-issue-tower-defence-management/compare/v1.38.0...v1.38.1) (2026-05-04)
+
+
+### Bug Fixes
+
+* **core:** skip issues with story: label in SetNoStoryIssueToStoryUseCase ([9e325a3](https://github.com/HiromiShikata/npm-cli-github-issue-tower-defence-management/commit/9e325a3eaccc1e2c3ea5ca3288b459ed12d4d08e))
+
 # [1.38.0](https://github.com/HiromiShikata/npm-cli-github-issue-tower-defence-management/compare/v1.37.1...v1.38.0) (2026-05-04)
 
 

package/bin/domain/usecases/SetNoStoryIssueToStoryUseCase.js

@@ -13,11 +13,11 @@ class SetNoStoryIssueToStoryUseCase {
             }
             const isTargetIssue = (issue) => {
                 return (issue.story === null &&
+                    !issue.labels.some((label) => label.toLowerCase().startsWith('story:')) &&
                     (issue.nextActionDate === null ||
                         issue.nextActionDate.getTime() <= input.targetDates[0].getTime()) &&
                     issue.nextActionHour === null &&
-                    issue.state === 'OPEN' &&
-                    issue.story === null);
+                    issue.state === 'OPEN');
             };
             const firstStory = input.project.story?.stories[0];
             if (!firstStory) {

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

@@ -1 +1 @@
-{"version":3,"file":"SetNoStoryIssueToStoryUseCase.js","sourceRoot":"","sources":["../../../src/domain/usecases/SetNoStoryIssueToStoryUseCase.ts"],"names":[],"mappings":";;;AAIA,MAAa,6BAA6B;IACxC,YAAqB,eAAqD;QAArD,oBAAe,GAAf,eAAe,CAAsC;QAE1E,QAAG,GAAG,KAAK,EAAE,KAKZ,EAAiB,EAAE;YAClB,MAAM,KAAK,GAAG,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC;YAClC,IACE,CAAC,KAAK;gBACN,KAAK,CAAC,SAAS;gBACf,CAAC,KAAK,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC,UAAU,EAAE,EAAE,CAAC,UAAU,CAAC,UAAU,EAAE,KAAK,CAAC,CAAC,EACtE,CAAC;gBACD,OAAO;YACT,CAAC;YACD,MAAM,aAAa,GAAG,CAAC,KAAY,EAAW,EAAE;gBAC9C,OAAO,CACL,KAAK,CAAC,KAAK,KAAK,IAAI;oBACpB,CAAC,KAAK,CAAC,cAAc,KAAK,IAAI;wBAC5B,KAAK,CAAC,cAAc,CAAC,OAAO,EAAE,IAAI,KAAK,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC;oBACnE,KAAK,CAAC,cAAc,KAAK,IAAI;oBAC7B,KAAK,CAAC,KAAK,KAAK,MAAM;oBACtB,KAAK,CAAC,KAAK,KAAK,IAAI,CACrB,CAAC;YACJ,CAAC,CAAC;YACF,MAAM,UAAU,GAAG,KAAK,CAAC,OAAO,CAAC,KAAK,EAAE,OAAO,CAAC,CAAC,CAAC,CAAC;YACnD,IAAI,CAAC,UAAU,EAAE,CAAC;gBAChB,OAAO;YACT,CAAC;YACD,KAAK,MAAM,KAAK,IAAI,KAAK,CAAC,MAAM,EAAE,CAAC;gBACjC,IAAI,CAAC,aAAa,CAAC,KAAK,CAAC,EAAE,CAAC;oBAC1B,SAAS;gBACX,CAAC;gBACD,MAAM,IAAI,CAAC,eAAe,CAAC,WAAW,CACpC,EAAE,GAAG,KAAK,CAAC,OAAO,EAAE,KAAK,EAAE,EAC3B,KAAK,EACL,UAAU,CAAC,EAAE,CACd,CAAC;gBACF,MAAM,IAAI,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,UAAU,CAAC,OAAO,EAAE,IAAI,CAAC,CAAC,CAAC;YAC5D,CAAC;QACH,CAAC,CAAC;IAzC2E,CAAC;CA0C/E;AA3CD,sEA2CC"}
+{"version":3,"file":"SetNoStoryIssueToStoryUseCase.js","sourceRoot":"","sources":["../../../src/domain/usecases/SetNoStoryIssueToStoryUseCase.ts"],"names":[],"mappings":";;;AAIA,MAAa,6BAA6B;IACxC,YAAqB,eAAqD;QAArD,oBAAe,GAAf,eAAe,CAAsC;QAE1E,QAAG,GAAG,KAAK,EAAE,KAKZ,EAAiB,EAAE;YAClB,MAAM,KAAK,GAAG,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC;YAClC,IACE,CAAC,KAAK;gBACN,KAAK,CAAC,SAAS;gBACf,CAAC,KAAK,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC,UAAU,EAAE,EAAE,CAAC,UAAU,CAAC,UAAU,EAAE,KAAK,CAAC,CAAC,EACtE,CAAC;gBACD,OAAO;YACT,CAAC;YACD,MAAM,aAAa,GAAG,CAAC,KAAY,EAAW,EAAE;gBAC9C,OAAO,CACL,KAAK,CAAC,KAAK,KAAK,IAAI;oBACpB,CAAC,KAAK,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,EAAE,CAC3B,KAAK,CAAC,WAAW,EAAE,CAAC,UAAU,CAAC,QAAQ,CAAC,CACzC;oBACD,CAAC,KAAK,CAAC,cAAc,KAAK,IAAI;wBAC5B,KAAK,CAAC,cAAc,CAAC,OAAO,EAAE,IAAI,KAAK,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC;oBACnE,KAAK,CAAC,cAAc,KAAK,IAAI;oBAC7B,KAAK,CAAC,KAAK,KAAK,MAAM,CACvB,CAAC;YACJ,CAAC,CAAC;YACF,MAAM,UAAU,GAAG,KAAK,CAAC,OAAO,CAAC,KAAK,EAAE,OAAO,CAAC,CAAC,CAAC,CAAC;YACnD,IAAI,CAAC,UAAU,EAAE,CAAC;gBAChB,OAAO;YACT,CAAC;YACD,KAAK,MAAM,KAAK,IAAI,KAAK,CAAC,MAAM,EAAE,CAAC;gBACjC,IAAI,CAAC,aAAa,CAAC,KAAK,CAAC,EAAE,CAAC;oBAC1B,SAAS;gBACX,CAAC;gBACD,MAAM,IAAI,CAAC,eAAe,CAAC,WAAW,CACpC,EAAE,GAAG,KAAK,CAAC,OAAO,EAAE,KAAK,EAAE,EAC3B,KAAK,EACL,UAAU,CAAC,EAAE,CACd,CAAC;gBACF,MAAM,IAAI,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,UAAU,CAAC,OAAO,EAAE,IAAI,CAAC,CAAC,CAAC;YAC5D,CAAC;QACH,CAAC,CAAC;IA3C2E,CAAC;CA4C/E;AA7CD,sEA6CC"}

package/package.json

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

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

@@ -0,0 +1,372 @@
+import { mock } from 'jest-mock-extended';
+import { SetNoStoryIssueToStoryUseCase } from './SetNoStoryIssueToStoryUseCase';
+import { IssueRepository } from './adapter-interfaces/IssueRepository';
+import { Issue } from '../entities/Issue';
+import { Project } from '../entities/Project';
+
+describe('SetNoStoryIssueToStoryUseCase', () => {
+  const mockIssueRepository = mock<IssueRepository>();
+
+  const basicStory = {
+    name: 'Story Field',
+    fieldId: 'storyFieldId',
+    databaseId: 123,
+    stories: [
+      {
+        id: 'noStoryId',
+        name: 'regular / NO STORY',
+        color: 'GRAY' as const,
+        description: '',
+      },
+      {
+        id: 'highPriorityId',
+        name: 'regular / high priority',
+        color: 'RED' as const,
+        description: '',
+      },
+    ],
+    workflowManagementStory: {
+      id: 'workflowManagementStoryId',
+      name: 'workflow management',
+    },
+  };
+
+  const basicProject: Project = {
+    ...mock<Project>(),
+    story: basicStory,
+  };
+
+  const targetDate = new Date('2000-01-01T01:00:00Z');
+
+  let useCase: SetNoStoryIssueToStoryUseCase;
+
+  beforeEach(() => {
+    jest.clearAllMocks();
+    jest.useFakeTimers();
+    useCase = new SetNoStoryIssueToStoryUseCase(mockIssueRepository);
+  });
+
+  afterEach(() => {
+    jest.useRealTimers();
+  });
+
+  describe('run', () => {
+    it('should do nothing when project has no story field', async () => {
+      const projectWithoutStory: Project = { ...basicProject, story: null };
+
+      await useCase.run({
+        targetDates: [targetDate],
+        project: projectWithoutStory,
+        issues: [
+          {
+            ...mock<Issue>(),
+            labels: [],
+            story: null,
+            state: 'OPEN',
+            nextActionDate: null,
+            nextActionHour: null,
+          },
+        ],
+        cacheUsed: false,
+      });
+
+      expect(mockIssueRepository.updateStory).not.toHaveBeenCalled();
+    });
+
+    it('should do nothing when cacheUsed is true', async () => {
+      await useCase.run({
+        targetDates: [targetDate],
+        project: basicProject,
+        issues: [
+          {
+            ...mock<Issue>(),
+            labels: [],
+            story: null,
+            state: 'OPEN',
+            nextActionDate: null,
+            nextActionHour: null,
+          },
+        ],
+        cacheUsed: true,
+      });
+
+      expect(mockIssueRepository.updateStory).not.toHaveBeenCalled();
+    });
+
+    it('should do nothing when no target date has minutes === 0', async () => {
+      const nonHourDate = new Date('2000-01-01T01:30:00Z');
+
+      await useCase.run({
+        targetDates: [nonHourDate],
+        project: basicProject,
+        issues: [
+          {
+            ...mock<Issue>(),
+            labels: [],
+            story: null,
+            state: 'OPEN',
+            nextActionDate: null,
+            nextActionHour: null,
+          },
+        ],
+        cacheUsed: false,
+      });
+
+      expect(mockIssueRepository.updateStory).not.toHaveBeenCalled();
+    });
+
+    it('should do nothing when project story has no stories', async () => {
+      const projectWithEmptyStories: Project = {
+        ...basicProject,
+        story: {
+          ...basicStory,
+          stories: [],
+        },
+      };
+
+      await useCase.run({
+        targetDates: [targetDate],
+        project: projectWithEmptyStories,
+        issues: [
+          {
+            ...mock<Issue>(),
+            labels: [],
+            story: null,
+            state: 'OPEN',
+            nextActionDate: null,
+            nextActionHour: null,
+          },
+        ],
+        cacheUsed: false,
+      });
+
+      expect(mockIssueRepository.updateStory).not.toHaveBeenCalled();
+    });
+
+    it('should assign first story to eligible issue with no story and no story: labels', async () => {
+      const issue: Issue = {
+        ...mock<Issue>(),
+        labels: [],
+        story: null,
+        state: 'OPEN',
+        nextActionDate: null,
+        nextActionHour: null,
+      };
+
+      const promise = useCase.run({
+        targetDates: [targetDate],
+        project: basicProject,
+        issues: [issue],
+        cacheUsed: false,
+      });
+      await jest.runAllTimersAsync();
+      await promise;
+
+      expect(mockIssueRepository.updateStory.mock.calls).toEqual([
+        [{ ...basicProject, story: basicProject.story }, issue, 'noStoryId'],
+      ]);
+    });
+
+    it('should skip issue that has a story: label', async () => {
+      const issue: Issue = {
+        ...mock<Issue>(),
+        labels: ['story:high-priority'],
+        story: null,
+        state: 'OPEN',
+        nextActionDate: null,
+        nextActionHour: null,
+      };
+
+      await useCase.run({
+        targetDates: [targetDate],
+        project: basicProject,
+        issues: [issue],
+        cacheUsed: false,
+      });
+
+      expect(mockIssueRepository.updateStory).not.toHaveBeenCalled();
+    });
+
+    it('should skip issue that has a story:workflow-management label', async () => {
+      const issue: Issue = {
+        ...mock<Issue>(),
+        labels: ['story:workflow-management'],
+        story: null,
+        state: 'OPEN',
+        nextActionDate: null,
+        nextActionHour: null,
+      };
+
+      await useCase.run({
+        targetDates: [targetDate],
+        project: basicProject,
+        issues: [issue],
+        cacheUsed: false,
+      });
+
+      expect(mockIssueRepository.updateStory).not.toHaveBeenCalled();
+    });
+
+    it('should skip issue that has a story: label with uppercase prefix', async () => {
+      const issue: Issue = {
+        ...mock<Issue>(),
+        labels: ['STORY:high-priority'],
+        story: null,
+        state: 'OPEN',
+        nextActionDate: null,
+        nextActionHour: null,
+      };
+
+      await useCase.run({
+        targetDates: [targetDate],
+        project: basicProject,
+        issues: [issue],
+        cacheUsed: false,
+      });
+
+      expect(mockIssueRepository.updateStory).not.toHaveBeenCalled();
+    });
+
+    it('should skip issue that already has a story assigned', async () => {
+      const issue: Issue = {
+        ...mock<Issue>(),
+        labels: [],
+        story: 'regular / NO STORY',
+        state: 'OPEN',
+        nextActionDate: null,
+        nextActionHour: null,
+      };
+
+      await useCase.run({
+        targetDates: [targetDate],
+        project: basicProject,
+        issues: [issue],
+        cacheUsed: false,
+      });
+
+      expect(mockIssueRepository.updateStory).not.toHaveBeenCalled();
+    });
+
+    it('should skip CLOSED issue', async () => {
+      const issue: Issue = {
+        ...mock<Issue>(),
+        labels: [],
+        story: null,
+        state: 'CLOSED',
+        nextActionDate: null,
+        nextActionHour: null,
+      };
+
+      await useCase.run({
+        targetDates: [targetDate],
+        project: basicProject,
+        issues: [issue],
+        cacheUsed: false,
+      });
+
+      expect(mockIssueRepository.updateStory).not.toHaveBeenCalled();
+    });
+
+    it('should skip issue with nextActionDate in the future', async () => {
+      const futureDate = new Date('2000-01-02T00:00:00Z');
+      const issue: Issue = {
+        ...mock<Issue>(),
+        labels: [],
+        story: null,
+        state: 'OPEN',
+        nextActionDate: futureDate,
+        nextActionHour: null,
+      };
+
+      await useCase.run({
+        targetDates: [targetDate],
+        project: basicProject,
+        issues: [issue],
+        cacheUsed: false,
+      });
+
+      expect(mockIssueRepository.updateStory).not.toHaveBeenCalled();
+    });
+
+    it('should skip issue with nextActionHour set', async () => {
+      const issue: Issue = {
+        ...mock<Issue>(),
+        labels: [],
+        story: null,
+        state: 'OPEN',
+        nextActionDate: null,
+        nextActionHour: 9,
+      };
+
+      await useCase.run({
+        targetDates: [targetDate],
+        project: basicProject,
+        issues: [issue],
+        cacheUsed: false,
+      });
+
+      expect(mockIssueRepository.updateStory).not.toHaveBeenCalled();
+    });
+
+    it('should process issue with nextActionDate equal to or before target date', async () => {
+      const pastDate = new Date('2000-01-01T00:00:00Z');
+      const issue: Issue = {
+        ...mock<Issue>(),
+        labels: [],
+        story: null,
+        state: 'OPEN',
+        nextActionDate: pastDate,
+        nextActionHour: null,
+      };
+
+      const promise = useCase.run({
+        targetDates: [targetDate],
+        project: basicProject,
+        issues: [issue],
+        cacheUsed: false,
+      });
+      await jest.runAllTimersAsync();
+      await promise;
+
+      expect(mockIssueRepository.updateStory.mock.calls).toEqual([
+        [{ ...basicProject, story: basicProject.story }, issue, 'noStoryId'],
+      ]);
+    });
+
+    it('should process multiple eligible issues and skip those with story: labels', async () => {
+      const eligibleIssue: Issue = {
+        ...mock<Issue>(),
+        labels: [],
+        story: null,
+        state: 'OPEN',
+        nextActionDate: null,
+        nextActionHour: null,
+      };
+      const issueWithStoryLabel: Issue = {
+        ...mock<Issue>(),
+        labels: ['story:high-priority'],
+        story: null,
+        state: 'OPEN',
+        nextActionDate: null,
+        nextActionHour: null,
+      };
+
+      const promise = useCase.run({
+        targetDates: [targetDate],
+        project: basicProject,
+        issues: [eligibleIssue, issueWithStoryLabel],
+        cacheUsed: false,
+      });
+      await jest.runAllTimersAsync();
+      await promise;
+
+      expect(mockIssueRepository.updateStory.mock.calls).toEqual([
+        [
+          { ...basicProject, story: basicProject.story },
+          eligibleIssue,
+          'noStoryId',
+        ],
+      ]);
+    });
+  });
+});

package/src/domain/usecases/SetNoStoryIssueToStoryUseCase.ts

@@ -22,11 +22,13 @@ export class SetNoStoryIssueToStoryUseCase {
     const isTargetIssue = (issue: Issue): boolean => {
       return (
         issue.story === null &&
+        !issue.labels.some((label) =>
+          label.toLowerCase().startsWith('story:'),
+        ) &&
         (issue.nextActionDate === null ||
           issue.nextActionDate.getTime() <= input.targetDates[0].getTime()) &&
         issue.nextActionHour === null &&
-        issue.state === 'OPEN' &&
-        issue.story === null
+        issue.state === 'OPEN'
       );
     };
     const firstStory = input.project.story?.stories[0];

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

@@ -1 +1 @@
-{"version":3,"file":"SetNoStoryIssueToStoryUseCase.d.ts","sourceRoot":"","sources":["../../../src/domain/usecases/SetNoStoryIssueToStoryUseCase.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,mBAAmB,CAAC;AAC1C,OAAO,EAAE,eAAe,EAAE,MAAM,sCAAsC,CAAC;AACvE,OAAO,EAAE,OAAO,EAAE,MAAM,qBAAqB,CAAC;AAE9C,qBAAa,6BAA6B;IAC5B,QAAQ,CAAC,eAAe,EAAE,IAAI,CAAC,eAAe,EAAE,aAAa,CAAC;gBAArD,eAAe,EAAE,IAAI,CAAC,eAAe,EAAE,aAAa,CAAC;IAE1E,GAAG,GAAU,OAAO;QAClB,WAAW,EAAE,IAAI,EAAE,CAAC;QACpB,OAAO,EAAE,OAAO,CAAC;QACjB,MAAM,EAAE,KAAK,EAAE,CAAC;QAChB,SAAS,EAAE,OAAO,CAAC;KACpB,KAAG,OAAO,CAAC,IAAI,CAAC,CAkCf;CACH"}
+{"version":3,"file":"SetNoStoryIssueToStoryUseCase.d.ts","sourceRoot":"","sources":["../../../src/domain/usecases/SetNoStoryIssueToStoryUseCase.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,mBAAmB,CAAC;AAC1C,OAAO,EAAE,eAAe,EAAE,MAAM,sCAAsC,CAAC;AACvE,OAAO,EAAE,OAAO,EAAE,MAAM,qBAAqB,CAAC;AAE9C,qBAAa,6BAA6B;IAC5B,QAAQ,CAAC,eAAe,EAAE,IAAI,CAAC,eAAe,EAAE,aAAa,CAAC;gBAArD,eAAe,EAAE,IAAI,CAAC,eAAe,EAAE,aAAa,CAAC;IAE1E,GAAG,GAAU,OAAO;QAClB,WAAW,EAAE,IAAI,EAAE,CAAC;QACpB,OAAO,EAAE,OAAO,CAAC;QACjB,MAAM,EAAE,KAAK,EAAE,CAAC;QAChB,SAAS,EAAE,OAAO,CAAC;KACpB,KAAG,OAAO,CAAC,IAAI,CAAC,CAoCf;CACH"}