github-issue-tower-defence-management 1.69.9 → 1.69.10

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [1.69.10](https://github.com/HiromiShikata/npm-cli-github-issue-tower-defence-management/compare/v1.69.9...v1.69.10) (2026-06-05)
+
+
+### Bug Fixes
+
+* **scheduler:** log per-item assignee update failures instead of aborting the slow sweep cycle ([be7a236](https://github.com/HiromiShikata/npm-cli-github-issue-tower-defence-management/commit/be7a23677c02365cb78f47f3b90fffae561c0665))
+
 ## [1.69.9](https://github.com/HiromiShikata/npm-cli-github-issue-tower-defence-management/compare/v1.69.8...v1.69.9) (2026-06-05)
 
 

package/bin/domain/usecases/AssignNoAssigneeIssueToManagerUseCase.js

@@ -19,7 +19,8 @@ class AssignNoAssigneeIssueToManagerUseCase {
                     if (!(e instanceof Error)) {
                         throw e;
                     }
-                    throw new Error(`Failed to update assignee for issue ${issue.url}: ${e.message}`);
+                    console.error(`Failed to update assignee for issue ${issue.url}: ${e.message}`);
+                    continue;
                 }
                 await new Promise((resolve) => setTimeout(resolve, 1000));
             }

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

@@ -1 +1 @@
-{"version":3,"file":"AssignNoAssigneeIssueToManagerUseCase.js","sourceRoot":"","sources":["../../../src/domain/usecases/AssignNoAssigneeIssueToManagerUseCase.ts"],"names":[],"mappings":";;;AAIA,MAAa,qCAAqC;IAChD,YACW,eAA4D;QAA5D,oBAAe,GAAf,eAAe,CAA6C;QAGvE,QAAG,GAAG,KAAK,EAAE,KAIZ,EAAiB,EAAE;YAClB,IAAI,KAAK,CAAC,SAAS,EAAE,CAAC;gBACpB,OAAO;YACT,CAAC;YACD,KAAK,MAAM,KAAK,IAAI,KAAK,CAAC,MAAM,EAAE,CAAC;gBACjC,IAAI,KAAK,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,IAAI,KAAK,CAAC,KAAK,KAAK,MAAM,EAAE,CAAC;oBACzD,SAAS;gBACX,CAAC;gBACD,IAAI,CAAC;oBACH,MAAM,IAAI,CAAC,eAAe,CAAC,kBAAkB,CAAC,KAAK,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC;gBACxE,CAAC;gBAAC,OAAO,CAAC,EAAE,CAAC;oBACX,IAAI,CAAC,CAAC,CAAC,YAAY,KAAK,CAAC,EAAE,CAAC;wBAC1B,MAAM,CAAC,CAAC;oBACV,CAAC;oBACD,MAAM,IAAI,KAAK,CACb,uCAAuC,KAAK,CAAC,GAAG,KAAK,CAAC,CAAC,OAAO,EAAE,CACjE,CAAC;gBACJ,CAAC;gBACD,MAAM,IAAI,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,UAAU,CAAC,OAAO,EAAE,IAAI,CAAC,CAAC,CAAC;YAC5D,CAAC;QACH,CAAC,CAAC;IA1BC,CAAC;CA2BL;AA9BD,sFA8BC"}
+{"version":3,"file":"AssignNoAssigneeIssueToManagerUseCase.js","sourceRoot":"","sources":["../../../src/domain/usecases/AssignNoAssigneeIssueToManagerUseCase.ts"],"names":[],"mappings":";;;AAIA,MAAa,qCAAqC;IAChD,YACW,eAA4D;QAA5D,oBAAe,GAAf,eAAe,CAA6C;QAGvE,QAAG,GAAG,KAAK,EAAE,KAIZ,EAAiB,EAAE;YAClB,IAAI,KAAK,CAAC,SAAS,EAAE,CAAC;gBACpB,OAAO;YACT,CAAC;YACD,KAAK,MAAM,KAAK,IAAI,KAAK,CAAC,MAAM,EAAE,CAAC;gBACjC,IAAI,KAAK,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,IAAI,KAAK,CAAC,KAAK,KAAK,MAAM,EAAE,CAAC;oBACzD,SAAS;gBACX,CAAC;gBACD,IAAI,CAAC;oBACH,MAAM,IAAI,CAAC,eAAe,CAAC,kBAAkB,CAAC,KAAK,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC;gBACxE,CAAC;gBAAC,OAAO,CAAC,EAAE,CAAC;oBACX,IAAI,CAAC,CAAC,CAAC,YAAY,KAAK,CAAC,EAAE,CAAC;wBAC1B,MAAM,CAAC,CAAC;oBACV,CAAC;oBACD,OAAO,CAAC,KAAK,CACX,uCAAuC,KAAK,CAAC,GAAG,KAAK,CAAC,CAAC,OAAO,EAAE,CACjE,CAAC;oBACF,SAAS;gBACX,CAAC;gBACD,MAAM,IAAI,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,UAAU,CAAC,OAAO,EAAE,IAAI,CAAC,CAAC,CAAC;YAC5D,CAAC;QACH,CAAC,CAAC;IA3BC,CAAC;CA4BL;AA/BD,sFA+BC"}

package/package.json

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

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

@@ -145,24 +145,57 @@ describe('AssignNoAssigneeIssueToManagerUseCase', () => {
       });
     });
 
-    it('should include issue url in error message when updateAssigneeList fails', async () => {
-      const issueWithUrl = {
+    it('should log error and continue to next issue when updateAssigneeList fails', async () => {
+      const failingIssue = {
         ...basicIssue,
         url: 'https://github.com/testOrg/testRepo/issues/42',
       };
+      const subsequentIssue = {
+        ...basicIssue,
+        url: 'https://github.com/testOrg/testRepo/issues/43',
+      };
       mockIssueRepository.updateAssigneeList.mockRejectedValueOnce(
         new Error('Request failed with status code 403 Forbidden'),
       );
+      const consoleErrorSpy = jest
+        .spyOn(console, 'error')
+        .mockImplementation(() => undefined);
 
       await expect(
         useCase.run({
-          issues: [issueWithUrl],
+          issues: [failingIssue, subsequentIssue],
           manager: 'manager1',
           cacheUsed: false,
         }),
-      ).rejects.toThrow(
+      ).resolves.toBeUndefined();
+
+      expect(consoleErrorSpy).toHaveBeenCalledWith(
         'Failed to update assignee for issue https://github.com/testOrg/testRepo/issues/42: Request failed with status code 403 Forbidden',
       );
+      expect(mockIssueRepository.updateAssigneeList.mock.calls).toEqual([
+        [failingIssue, ['manager1']],
+        [subsequentIssue, ['manager1']],
+      ]);
+
+      consoleErrorSpy.mockRestore();
+    });
+
+    it('should rethrow non-Error thrown values without logging', async () => {
+      const failingIssue = {
+        ...basicIssue,
+        url: 'https://github.com/testOrg/testRepo/issues/44',
+      };
+      mockIssueRepository.updateAssigneeList.mockImplementationOnce(() => {
+        throw 'string-failure';
+      });
+
+      await expect(
+        useCase.run({
+          issues: [failingIssue],
+          manager: 'manager1',
+          cacheUsed: false,
+        }),
+      ).rejects.toBe('string-failure');
     });
   });
 });

package/src/domain/usecases/AssignNoAssigneeIssueToManagerUseCase.ts

@@ -25,9 +25,10 @@ export class AssignNoAssigneeIssueToManagerUseCase {
         if (!(e instanceof Error)) {
           throw e;
         }
-        throw new Error(
+        console.error(
           `Failed to update assignee for issue ${issue.url}: ${e.message}`,
         );
+        continue;
       }
       await new Promise((resolve) => setTimeout(resolve, 1000));
     }

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

@@ -1 +1 @@
-{"version":3,"file":"AssignNoAssigneeIssueToManagerUseCase.d.ts","sourceRoot":"","sources":["../../../src/domain/usecases/AssignNoAssigneeIssueToManagerUseCase.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,mBAAmB,CAAC;AAC1C,OAAO,EAAE,eAAe,EAAE,MAAM,sCAAsC,CAAC;AACvE,OAAO,EAAE,MAAM,EAAE,MAAM,oBAAoB,CAAC;AAE5C,qBAAa,qCAAqC;IAE9C,QAAQ,CAAC,eAAe,EAAE,IAAI,CAAC,eAAe,EAAE,oBAAoB,CAAC;gBAA5D,eAAe,EAAE,IAAI,CAAC,eAAe,EAAE,oBAAoB,CAAC;IAGvE,GAAG,GAAU,OAAO;QAClB,MAAM,EAAE,KAAK,EAAE,CAAC;QAChB,OAAO,EAAE,MAAM,CAAC,MAAM,CAAC,CAAC;QACxB,SAAS,EAAE,OAAO,CAAC;KACpB,KAAG,OAAO,CAAC,IAAI,CAAC,CAoBf;CACH"}
+{"version":3,"file":"AssignNoAssigneeIssueToManagerUseCase.d.ts","sourceRoot":"","sources":["../../../src/domain/usecases/AssignNoAssigneeIssueToManagerUseCase.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,mBAAmB,CAAC;AAC1C,OAAO,EAAE,eAAe,EAAE,MAAM,sCAAsC,CAAC;AACvE,OAAO,EAAE,MAAM,EAAE,MAAM,oBAAoB,CAAC;AAE5C,qBAAa,qCAAqC;IAE9C,QAAQ,CAAC,eAAe,EAAE,IAAI,CAAC,eAAe,EAAE,oBAAoB,CAAC;gBAA5D,eAAe,EAAE,IAAI,CAAC,eAAe,EAAE,oBAAoB,CAAC;IAGvE,GAAG,GAAU,OAAO;QAClB,MAAM,EAAE,KAAK,EAAE,CAAC;QAChB,OAAO,EAAE,MAAM,CAAC,MAAM,CAAC,CAAC;QACxB,SAAS,EAAE,OAAO,CAAC;KACpB,KAAG,OAAO,CAAC,IAAI,CAAC,CAqBf;CACH"}