github-issue-tower-defence-management 1.103.1 → 1.104.1

package/src/adapter/repositories/issue/ApiV3CheerioRestIssueRepository.ts

@@ -543,6 +543,7 @@ export class ApiV3CheerioRestIssueRepository
       isClosed: item.state !== 'OPEN',
       createdAt: new Date(item.createdAt || '2000-01-01'),
       author: item.author,
+      closingIssueReferenceUrls: item.closingIssueReferenceUrls,
     };
   };
   getAllIssuesFromCache = async (
@@ -578,6 +579,14 @@ export class ApiV3CheerioRestIssueRepository
               !('createdAt' in issue) || typeof issue.createdAt !== 'string'
                 ? new Date()
                 : new Date(issue.createdAt);
+            const closingIssueReferenceUrls =
+              'closingIssueReferenceUrls' in issue &&
+              Array.isArray(issue.closingIssueReferenceUrls) &&
+              issue.closingIssueReferenceUrls.every(
+                (url): url is string => typeof url === 'string',
+              )
+                ? issue.closingIssueReferenceUrls
+                : [];
 
             return {
               ...issue,
@@ -585,6 +594,7 @@ export class ApiV3CheerioRestIssueRepository
               completionDate50PercentConfidence:
                 completionDate50PercentConfidence,
               createdAt: createdAt,
+              closingIssueReferenceUrls: closingIssueReferenceUrls,
             };
           });
 

package/src/adapter/repositories/issue/GraphqlProjectItemRepository.ts

@@ -14,6 +14,7 @@ export type ProjectItem = {
   assignees: string[];
   createdAt: string;
   author: string;
+  closingIssueReferenceUrls: string[];
   customFields: {
     name: string;
     value: string | null;
@@ -214,6 +215,11 @@ query GetProjectItems($projectId: ID!, $after: String, $first: Int!) {
               repository {
                 nameWithOwner
               }
+              closingIssuesReferences(first: 50) {
+                nodes {
+                  url
+                }
+              }
             }
           }
         }
@@ -257,6 +263,7 @@ query GetProjectItems($projectId: ID!, $after: String, $first: Int!) {
               author: { login: string } | null;
               labels: { nodes: { name: string }[] };
               assignees: { nodes: { login: string }[] };
+              closingIssuesReferences?: { nodes: { url: string }[] };
             };
           }[];
         };
@@ -309,6 +316,7 @@ query GetProjectItems($projectId: ID!, $after: String, $first: Int!) {
                     author: { login: string } | null;
                     labels: { nodes: { name: string }[] };
                     assignees: { nodes: { login: string }[] };
+                    closingIssuesReferences?: { nodes: { url: string }[] };
                   };
                 }[];
               };
@@ -364,6 +372,7 @@ query GetProjectItems($projectId: ID!, $after: String, $first: Int!) {
               author: { login: string } | null;
               labels: { nodes: { name: string }[] };
               assignees: { nodes: { login: string }[] };
+              closingIssuesReferences?: { nodes: { url: string }[] };
             };
           }[];
         };
@@ -435,6 +444,7 @@ query GetProjectItems($projectId: ID!, $after: String, $first: Int!) {
           author: { login: string } | null;
           labels: { nodes: { name: string }[] };
           assignees: { nodes: { login: string }[] };
+          closingIssuesReferences?: { nodes: { url: string }[] };
         };
       }[] = pageNodes;
       projectItems.forEach((item) => {
@@ -453,6 +463,10 @@ query GetProjectItems($projectId: ID!, $after: String, $first: Int!) {
           assignees: item.content.assignees?.nodes?.map((a) => a.login) || [],
           createdAt: item.content.createdAt || new Date().toISOString(),
           author: item.content.author?.login || '',
+          closingIssueReferenceUrls:
+            item.content.closingIssuesReferences?.nodes
+              ?.map((node) => node.url)
+              .filter((url) => url.length > 0) || [],
           customFields: item.fieldValues.nodes
             .filter((field) => !!field.field)
             .map((field) => {
@@ -768,6 +782,11 @@ query GetProjectFields($owner: String!, $repository: String!, $issueNumber: Int!
       repository {
         nameWithOwner
       }
+      closingIssuesReferences(first: 50) {
+        nodes {
+          url
+        }
+      }
       projectItems(first: 10) {
         nodes {
           id
@@ -843,6 +862,7 @@ query GetProjectFields($owner: String!, $repository: String!, $issueNumber: Int!
       labels: { nodes: { name: string }[] };
       assignees: { nodes: { login: string }[] };
       repository: { nameWithOwner: string };
+      closingIssuesReferences?: { nodes: { url: string }[] };
       projectItems: {
         nodes: {
           id: string;
@@ -914,6 +934,10 @@ query GetProjectFields($owner: String!, $repository: String!, $issueNumber: Int!
       assignees: content.assignees?.nodes?.map((a) => a.login) || [],
       createdAt: content.createdAt || new Date().toISOString(),
       author: content.author?.login || '',
+      closingIssueReferenceUrls:
+        content.closingIssuesReferences?.nodes
+          ?.map((node) => node.url)
+          .filter((url) => url.length > 0) || [],
       customFields: item.fieldValues.nodes
         .filter((field) => !!field.field)
         .map((field) => {

package/src/adapter/repositories/issue/RestIssueRepository.test.ts

@@ -50,6 +50,7 @@ const buildIssue = (overrides: Partial<Issue> = {}): Issue => ({
   isClosed: false,
   createdAt: new Date(),
   author: '',
+  closingIssueReferenceUrls: [],
   ...overrides,
 });
 

package/src/domain/entities/Issue.ts

@@ -24,4 +24,5 @@ export type Issue = {
   isClosed: boolean;
   createdAt: Date;
   author: string;
+  closingIssueReferenceUrls: string[];
 };

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

@@ -27,6 +27,7 @@ const createMockIssue = (overrides: Partial<Issue> = {}): Issue => ({
   isClosed: false,
   createdAt: new Date('2000-01-01T00:00:00Z'),
   author: 'test-user',
+  closingIssueReferenceUrls: [],
   ...overrides,
 });
 

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

@@ -67,6 +67,7 @@ describe('GetStoryObjectMapUseCase', () => {
     isClosed: false,
     createdAt: new Date(),
     author: '',
+    closingIssueReferenceUrls: [],
     ...overrides,
   });
 

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

@@ -379,6 +379,115 @@ describe('HandleScheduledEventUseCase', () => {
       ).toHaveBeenCalledTimes(1);
     });
 
+    it('should pass a top-level allowedIssueAuthors to revertNotReadyReviewQueueIssueUseCase', async () => {
+      const input = {
+        projectName: 'test-project',
+        org: 'test-org',
+        projectUrl: 'https://github.com/test-org/test-project',
+        manager: 'test-manager',
+        workingReport: {
+          repo: 'test-repo',
+          members: ['member1'],
+          spreadsheetUrl: 'https://docs.google.com/spreadsheets/test',
+        },
+        urlOfStoryView: 'https://github.com/test-org/test-project/issues',
+        disabled: false,
+        allowIssueCacheMinutes: 60,
+        allowedIssueAuthors: ['top-level-author'],
+      };
+
+      mockProjectRepository.getProject.mockResolvedValue(mock<Project>());
+      await useCase.run(input);
+
+      expect(
+        mockRevertNotReadyReviewQueueIssueUseCase.run,
+      ).toHaveBeenCalledWith(
+        expect.objectContaining({
+          allowedIssueAuthors: ['top-level-author'],
+        }),
+      );
+    });
+
+    it('should prefer a top-level allowedIssueAuthors over the startPreparation one', async () => {
+      const input = {
+        projectName: 'test-project',
+        org: 'test-org',
+        projectUrl: 'https://github.com/test-org/test-project',
+        manager: 'test-manager',
+        workingReport: {
+          repo: 'test-repo',
+          members: ['member1'],
+          spreadsheetUrl: 'https://docs.google.com/spreadsheets/test',
+        },
+        urlOfStoryView: 'https://github.com/test-org/test-project/issues',
+        disabled: false,
+        allowIssueCacheMinutes: 60,
+        allowedIssueAuthors: ['top-level-author'],
+        startPreparation: {
+          defaultAgentName: 'agent1',
+          configFilePath: '/path/to/config.yml',
+          maximumPreparingIssuesCount: null,
+          allowedIssueAuthors: ['nested-author'],
+        },
+      };
+
+      mockProjectRepository.getProject.mockResolvedValue(mock<Project>());
+      mockStartPreparationUseCase.run.mockResolvedValue({
+        rotationOrder: null,
+      });
+      await useCase.run(input);
+
+      expect(
+        mockRevertNotReadyReviewQueueIssueUseCase.run,
+      ).toHaveBeenCalledWith(
+        expect.objectContaining({
+          allowedIssueAuthors: ['top-level-author'],
+        }),
+      );
+      expect(mockStartPreparationUseCase.run).toHaveBeenCalledWith(
+        expect.objectContaining({
+          allowedIssueAuthors: ['top-level-author'],
+        }),
+      );
+    });
+
+    it('should fall back to startPreparation allowedIssueAuthors when no top-level value is set', async () => {
+      const input = {
+        projectName: 'test-project',
+        org: 'test-org',
+        projectUrl: 'https://github.com/test-org/test-project',
+        manager: 'test-manager',
+        workingReport: {
+          repo: 'test-repo',
+          members: ['member1'],
+          spreadsheetUrl: 'https://docs.google.com/spreadsheets/test',
+        },
+        urlOfStoryView: 'https://github.com/test-org/test-project/issues',
+        disabled: false,
+        allowIssueCacheMinutes: 60,
+        startPreparation: {
+          defaultAgentName: 'agent1',
+          configFilePath: '/path/to/config.yml',
+          maximumPreparingIssuesCount: null,
+          allowedIssueAuthors: ['nested-author'],
+        },
+      };
+
+      mockProjectRepository.getProject.mockResolvedValue(mock<Project>());
+      mockStartPreparationUseCase.run.mockResolvedValue({
+        rotationOrder: null,
+      });
+      await useCase.run(input);
+
+      expect(
+        mockRevertNotReadyReviewQueueIssueUseCase.run,
+      ).toHaveBeenCalledWith(
+        expect.objectContaining({
+          allowedIssueAuthors: ['nested-author'],
+        }),
+      );
+    });
+
     it('should invoke UpdateRateLimitCacheUseCase before StartPreparationUseCase when startPreparation is configured', async () => {
       const callOrder: string[] = [];
       mockUpdateRateLimitCacheUseCase.run.mockImplementation(async () => {

package/src/domain/usecases/HandleScheduledEventUseCase.ts

@@ -27,6 +27,7 @@ import {
 import { RevertOrphanedPreparationUseCase } from './RevertOrphanedPreparationUseCase';
 import { RevertNotReadyReviewQueueIssueUseCase } from './RevertNotReadyReviewQueueIssueUseCase';
 import { resolveLabelsAsLlmAgentName } from './resolveLabelsAsLlmAgentName';
+import { resolveAllowedIssueAuthors } from './resolveAllowedIssueAuthors';
 import { SetupTowerDefenceProjectUseCase } from './SetupTowerDefenceProjectUseCase';
 import { UpdateRateLimitCacheUseCase } from './UpdateRateLimitCacheUseCase';
 import {
@@ -86,6 +87,7 @@ export class HandleScheduledEventUseCase {
     allowIssueCacheMinutes: number;
     labelsAsLlmAgentName?: string[] | null;
     changeTargetPathAliases?: Record<string, string> | null;
+    allowedIssueAuthors?: string[] | null;
     startPreparation?: {
       defaultAgentName: string;
       defaultLlmModelName?: string | null;
@@ -293,12 +295,16 @@ ${JSON.stringify(e)}
       topLevel: input.labelsAsLlmAgentName,
       startPreparation: input.startPreparation?.labelsAsLlmAgentName,
     });
+    const allowedIssueAuthors = resolveAllowedIssueAuthors({
+      topLevel: input.allowedIssueAuthors,
+      startPreparation: input.startPreparation?.allowedIssueAuthors,
+    });
     await this.revertNotReadyReviewQueueIssueUseCase.run({
       projectUrl: input.projectUrl,
       allowIssueCacheMinutes: input.allowIssueCacheMinutes,
       labelsAsLlmAgentName,
       changeTargetPathAliases: input.changeTargetPathAliases,
-      allowedIssueAuthors: input.startPreparation?.allowedIssueAuthors ?? null,
+      allowedIssueAuthors,
     });
     if (this.dailySecurityScanUseCase !== null && input.dailySecurityScan) {
       await this.dailySecurityScanUseCase.run({
@@ -341,7 +347,7 @@ ${JSON.stringify(e)}
           input.startPreparation.maximumPreparingIssuesCount,
         utilizationPercentageThreshold:
           input.startPreparation.utilizationPercentageThreshold ?? 90,
-        allowedIssueAuthors: input.startPreparation.allowedIssueAuthors ?? null,
+        allowedIssueAuthors,
         codexHomeCandidates: input.startPreparation.codexHomeCandidates ?? null,
         allowIssueCacheMinutes: input.allowIssueCacheMinutes,
         labelsAsLlmAgentName,

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

@@ -47,6 +47,7 @@ const issue = (url: string, status: string): Issue => ({
   isClosed: false,
   createdAt: new Date('2026-01-01T00:00:00Z'),
   author: 'hiromi',
+  closingIssueReferenceUrls: [],
 });
 
 describe('InTmuxByHumanSessionTokenCountUseCase', () => {

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

@@ -72,6 +72,7 @@ const createMockIssue = (overrides: Partial<Issue> = {}): Issue => ({
   isClosed: false,
   createdAt: new Date(),
   author: '',
+  closingIssueReferenceUrls: [],
   ...overrides,
 });
 

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

@@ -79,7 +79,8 @@ const createMockIssue = (overrides: Partial<Issue> = {}): Issue => ({
   isInProgress: false,
   isClosed: false,
   createdAt: new Date(),
-  author: '',
+  author: 'owner',
+  closingIssueReferenceUrls: [],
   ...overrides,
 });
 
@@ -170,6 +171,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).not.toHaveBeenCalled();
@@ -189,6 +191,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).not.toHaveBeenCalled();
@@ -208,6 +211,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).toHaveBeenCalledWith(
@@ -240,6 +244,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
         labelsAsLlmAgentName: ['story', 'chore', 'accounting'],
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).not.toHaveBeenCalled();
@@ -261,6 +266,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
         labelsAsLlmAgentName: ['story', 'chore'],
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).not.toHaveBeenCalled();
@@ -281,6 +287,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).toHaveBeenCalledWith(
@@ -309,6 +316,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
         labelsAsLlmAgentName: null,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).toHaveBeenCalledWith(
@@ -333,6 +341,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).not.toHaveBeenCalled();
@@ -354,6 +363,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).toHaveBeenCalledWith(
@@ -390,6 +400,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).toHaveBeenCalledWith(
@@ -422,6 +433,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).toHaveBeenCalledWith(
@@ -454,6 +466,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).toHaveBeenCalledWith(
@@ -487,6 +500,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).toHaveBeenCalledWith(
@@ -524,6 +538,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).toHaveBeenCalledWith(
@@ -561,6 +576,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
         useCase.run({
           projectUrl: 'https://github.com/users/user/projects/1',
           allowIssueCacheMinutes: 10,
+          allowedIssueAuthors: ['owner'],
         });
 
       it('should not approve PR when issue has no change-target label', async () => {
@@ -772,6 +788,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
         await useCase.run({
           projectUrl: 'https://github.com/users/user/projects/1',
           allowIssueCacheMinutes: 10,
+          allowedIssueAuthors: ['owner'],
           changeTargetPathAliases: {
             adapters: 'src/domain/usecases/adapter-interfaces',
           },
@@ -791,6 +808,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
         await useCase.run({
           projectUrl: 'https://github.com/users/user/projects/1',
           allowIssueCacheMinutes: 10,
+          allowedIssueAuthors: ['owner'],
           changeTargetPathAliases: {
             adapters: 'src/domain/usecases/adapter-interfaces',
           },
@@ -814,6 +832,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).not.toHaveBeenCalled();
@@ -835,6 +854,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).not.toHaveBeenCalled();
@@ -855,6 +875,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).not.toHaveBeenCalled();
@@ -875,6 +896,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).not.toHaveBeenCalled();
@@ -898,6 +920,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).toHaveBeenCalledWith(
@@ -937,6 +960,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).toHaveBeenCalledWith(
@@ -971,6 +995,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).toHaveBeenCalledWith(
@@ -1007,6 +1032,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).toHaveBeenCalledWith(
@@ -1041,6 +1067,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).toHaveBeenCalledWith(
@@ -1072,6 +1099,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).toHaveBeenCalledWith(
@@ -1109,6 +1137,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).toHaveBeenCalledWith(
@@ -1153,6 +1182,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       await useCase.run({
         projectUrl: 'https://github.com/users/user/projects/1',
         allowIssueCacheMinutes: 10,
+        allowedIssueAuthors: ['owner'],
       });
 
       expect(mockIssueRepository.updateStatus).not.toHaveBeenCalled();
@@ -1222,7 +1252,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
         );
       });
 
-      it('should act on an Unread pull request authored by a dependency-update bot not in allowedIssueAuthors', async () => {
+      it('should skip an Unread pull request authored by a dependency-update bot not in allowedIssueAuthors', async () => {
         const pullRequest = createMockPullRequest({
           status: 'Unread',
           author: 'dependabot[bot]',
@@ -1242,6 +1272,31 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
           allowedIssueAuthors: ['owner'],
         });
 
+        expect(mockIssueRepository.updateStatus).not.toHaveBeenCalled();
+        expect(mockIssueRepository.updateStory).not.toHaveBeenCalled();
+        expect(mockIssueCommentRepository.createComment).not.toHaveBeenCalled();
+      });
+
+      it('should act on a dependency-update bot only when it is explicitly listed in allowedIssueAuthors', async () => {
+        const pullRequest = createMockPullRequest({
+          status: 'Unread',
+          author: 'dependabot[bot]',
+        });
+        mockIssueRepository.getAllIssues.mockResolvedValue({
+          issues: [pullRequest],
+          cacheUsed: false,
+        });
+        mockIssueRepository.getOpenPullRequest.mockResolvedValue({
+          ...createReadyPr(),
+          isConflicted: true,
+        });
+
+        await useCase.run({
+          projectUrl: 'https://github.com/users/user/projects/1',
+          allowIssueCacheMinutes: 10,
+          allowedIssueAuthors: ['owner', 'dependabot[bot]'],
+        });
+
         expect(mockIssueRepository.updateStatus).toHaveBeenCalledWith(
           mockProject,
           pullRequest,
@@ -1253,7 +1308,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
         );
       });
 
-      it('should act on every author when allowedIssueAuthors is null', async () => {
+      it('should process no author when allowedIssueAuthors is null', async () => {
         const pullRequest = createMockPullRequest({
           status: 'Unread',
           author: 'outside-contributor',
@@ -1273,15 +1328,58 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
           allowedIssueAuthors: null,
         });
 
-        expect(mockIssueRepository.updateStatus).toHaveBeenCalledWith(
-          mockProject,
-          pullRequest,
-          'awaiting-workspace-id',
-        );
-        expect(mockIssueCommentRepository.createComment).toHaveBeenCalledWith(
-          pullRequest,
-          expect.stringContaining('Auto Status Check: REJECTED'),
-        );
+        expect(mockIssueRepository.updateStatus).not.toHaveBeenCalled();
+        expect(mockIssueRepository.updateStory).not.toHaveBeenCalled();
+        expect(mockIssueCommentRepository.createComment).not.toHaveBeenCalled();
+      });
+
+      it('should process no author when allowedIssueAuthors is undefined', async () => {
+        const pullRequest = createMockPullRequest({
+          status: 'Unread',
+          author: 'owner',
+        });
+        mockIssueRepository.getAllIssues.mockResolvedValue({
+          issues: [pullRequest],
+          cacheUsed: false,
+        });
+        mockIssueRepository.getOpenPullRequest.mockResolvedValue({
+          ...createReadyPr(),
+          isConflicted: true,
+        });
+
+        await useCase.run({
+          projectUrl: 'https://github.com/users/user/projects/1',
+          allowIssueCacheMinutes: 10,
+        });
+
+        expect(mockIssueRepository.updateStatus).not.toHaveBeenCalled();
+        expect(mockIssueRepository.updateStory).not.toHaveBeenCalled();
+        expect(mockIssueCommentRepository.createComment).not.toHaveBeenCalled();
+      });
+
+      it('should process no author when allowedIssueAuthors is an empty list', async () => {
+        const pullRequest = createMockPullRequest({
+          status: 'Unread',
+          author: 'owner',
+        });
+        mockIssueRepository.getAllIssues.mockResolvedValue({
+          issues: [pullRequest],
+          cacheUsed: false,
+        });
+        mockIssueRepository.getOpenPullRequest.mockResolvedValue({
+          ...createReadyPr(),
+          isConflicted: true,
+        });
+
+        await useCase.run({
+          projectUrl: 'https://github.com/users/user/projects/1',
+          allowIssueCacheMinutes: 10,
+          allowedIssueAuthors: [],
+        });
+
+        expect(mockIssueRepository.updateStatus).not.toHaveBeenCalled();
+        expect(mockIssueRepository.updateStory).not.toHaveBeenCalled();
+        expect(mockIssueCommentRepository.createComment).not.toHaveBeenCalled();
       });
     });
   });