github-issue-tower-defence-management 2.6.1 → 2.7.0

package/src/domain/usecases/NotifyFinishedIssuePreparationUseCase.ts

@@ -15,16 +15,7 @@ import {
 } from './IssueRejectionEvaluator';
 import { ChangeTargetPullRequestApprover } from './ChangeTargetPullRequestApprover';
 import { resolveLabelsNotRequiringPullRequest } from './resolveLabelsNotRequiringPullRequest';
-import { isPullRequestDeclaredUnnecessary } from './isPullRequestDeclaredUnnecessary';
-import {
-  RETURNED_TO_AWAITING_WORKSPACE_MESSAGE,
-  RETURNED_TO_AWAITING_WORKSPACE_MESSAGE_HEAD,
-} from './returnedToAwaitingWorkspaceMessage';
-import { isWaitingForOwnerApproval } from './isWaitingForOwnerApproval';
-import {
-  AWAITING_OWNER_APPROVAL_MESSAGE,
-  AWAITING_OWNER_APPROVAL_MESSAGE_HEAD,
-} from './awaitingOwnerApprovalMessage';
+import { isTriagerAgentName } from './triagerAgentName';
 import {
   ConsoleListItem,
   ConsoleTabName,
@@ -127,7 +118,6 @@ export class NotifyFinishedIssuePreparationUseCase {
     manager?: string | null;
     developerAgentName?: string | null;
     deferPreparation?: boolean | null;
-    ownerApprovalTimeoutCycles?: number | null;
   }): Promise<void> => {
     const project = await this.projectRepository.getByUrl(params.projectUrl);
 
@@ -257,110 +247,12 @@ export class NotifyFinishedIssuePreparationUseCase {
     const nextStepAgent = lastAgentReport
       ? extractNextStepAgent(lastAgentReport.content)
       : null;
-    if (nextStepAgent !== null) {
-      if (
-        params.agents &&
-        params.agents.length > 0 &&
-        !params.agents.includes(nextStepAgent)
-      ) {
-        issue.status = FAILED_PREPARATION_STATUS_NAME;
-        await this.issueRepository.update(issue, project);
-        await this.issueRepository.updateStatus(
-          project,
-          issue,
-          failedPreparationStatusOption.id,
-        );
-        await this.patchConsoleTab(issue);
-        await this.issueCommentRepository.createComment(
-          issue,
-          `nextStepAgent '${nextStepAgent}' is not in the configured agents list. Update the configuration to include it.`,
-        );
-        return;
-      }
-      const repetition = resolveNextStepAgentDispatchRepetition({
-        agentFieldValue: issue.agent,
-        nextStepAgent,
-        comments,
-        isTrustedAuthor,
-        thresholdForAutoReject: params.thresholdForAutoReject,
-        thresholdForDispatchLoop:
-          params.thresholdForDispatchLoop ??
-          DEFAULT_THRESHOLD_FOR_DISPATCH_LOOP,
-      });
-      if (repetition.type === 'escalateToFailedPreparation') {
-        issue.status = FAILED_PREPARATION_STATUS_NAME;
-        await this.issueRepository.update(issue, project);
-        await this.issueRepository.updateStatus(
-          project,
-          issue,
-          failedPreparationStatusOption.id,
-        );
-        await this.patchConsoleTab(issue);
-        await this.issueCommentRepository.createComment(
-          issue,
-          repetition.comment,
-        );
-        await this.sendWorkflowBlockerNotification(
-          params.issueUrl,
-          params.workflowBlockerResolvedWebhookUrl,
-          project,
-        );
-        return;
-      }
-      const agentOptionId = await this.ensureAgentOptionAndGetId(
-        project,
-        nextStepAgent,
-      );
-      if (agentOptionId) {
-        await this.issueRepository.setIssueAgentField(
-          params.issueUrl,
-          project,
-          agentOptionId,
-        );
-      }
-      issue.status = AWAITING_WORKSPACE_STATUS_NAME;
-      await this.issueRepository.update(issue, project);
-      await this.issueRepository.updateStatus(
-        project,
-        issue,
-        awaitingWorkspaceStatusOption.id,
-      );
-      await this.patchConsoleTab(issue);
-      if (repetition.type === 'dispatchAgain') {
-        await this.issueCommentRepository.createComment(
-          issue,
-          repetition.comment,
-        );
-      }
-      return;
-    }
-
     if (
-      lastAgentReport !== null &&
-      isWaitingForOwnerApproval(lastAgentReport.content)
+      nextStepAgent !== null &&
+      params.agents &&
+      params.agents.length > 0 &&
+      !params.agents.includes(nextStepAgent)
     ) {
-      const ownerApprovalTimeoutCycles =
-        params.ownerApprovalTimeoutCycles ?? 12;
-      const awaitingOwnerApprovalCount = comments.filter(
-        (comment) =>
-          isTrustedAuthor(comment.author) &&
-          comment.content.startsWith(AWAITING_OWNER_APPROVAL_MESSAGE_HEAD),
-      ).length;
-      if (awaitingOwnerApprovalCount < ownerApprovalTimeoutCycles) {
-        issue.status = AWAITING_QUALITY_CHECK_STATUS_NAME;
-        await this.issueRepository.update(issue, project);
-        await this.issueRepository.updateStatus(
-          project,
-          issue,
-          awaitingQualityCheckStatusOption.id,
-        );
-        await this.patchConsoleTab(issue);
-        await this.issueCommentRepository.createComment(
-          issue,
-          AWAITING_OWNER_APPROVAL_MESSAGE,
-        );
-        return;
-      }
       issue.status = FAILED_PREPARATION_STATUS_NAME;
       await this.issueRepository.update(issue, project);
       await this.issueRepository.updateStatus(
@@ -371,12 +263,7 @@ export class NotifyFinishedIssuePreparationUseCase {
       await this.patchConsoleTab(issue);
       await this.issueCommentRepository.createComment(
         issue,
-        `Owner approval was not received after ${ownerApprovalTimeoutCycles} cycles. Moving to Failed Preparation.`,
-      );
-      await this.sendWorkflowBlockerNotification(
-        params.issueUrl,
-        params.workflowBlockerResolvedWebhookUrl,
-        project,
+        `nextStepAgent '${nextStepAgent}' is not in the configured agents list. Update the configuration to include it.`,
       );
       return;
     }
@@ -424,6 +311,7 @@ export class NotifyFinishedIssuePreparationUseCase {
       comments,
       isTrustedAuthor,
       resolveLabelsNotRequiringPullRequest(params),
+      nextStepAgent,
       params.developerAgentName,
     );
 
@@ -475,17 +363,48 @@ export class NotifyFinishedIssuePreparationUseCase {
       return;
     }
 
-    if (
-      rejections.length <= 0 &&
-      isPullRequestDeclaredUnnecessary(comments, isTrustedAuthor) &&
-      !comments.some(
-        (comment) =>
-          isTrustedAuthor(comment.author) &&
-          comment.content.startsWith(
-            RETURNED_TO_AWAITING_WORKSPACE_MESSAGE_HEAD,
-          ),
-      )
-    ) {
+    if (nextStepAgent !== null) {
+      const repetition = resolveNextStepAgentDispatchRepetition({
+        agentFieldValue: issue.agent,
+        nextStepAgent,
+        comments,
+        isTrustedAuthor,
+        thresholdForAutoReject: params.thresholdForAutoReject,
+        thresholdForDispatchLoop:
+          params.thresholdForDispatchLoop ??
+          DEFAULT_THRESHOLD_FOR_DISPATCH_LOOP,
+      });
+      if (repetition.type === 'escalateToFailedPreparation') {
+        issue.status = FAILED_PREPARATION_STATUS_NAME;
+        await this.issueRepository.update(issue, project);
+        await this.issueRepository.updateStatus(
+          project,
+          issue,
+          failedPreparationStatusOption.id,
+        );
+        await this.patchConsoleTab(issue);
+        await this.issueCommentRepository.createComment(
+          issue,
+          repetition.comment,
+        );
+        await this.sendWorkflowBlockerNotification(
+          params.issueUrl,
+          params.workflowBlockerResolvedWebhookUrl,
+          project,
+        );
+        return;
+      }
+      const agentOptionId = await this.ensureAgentOptionAndGetId(
+        project,
+        nextStepAgent,
+      );
+      if (agentOptionId) {
+        await this.issueRepository.setIssueAgentField(
+          params.issueUrl,
+          project,
+          agentOptionId,
+        );
+      }
       issue.status = AWAITING_WORKSPACE_STATUS_NAME;
       await this.issueRepository.update(issue, project);
       await this.issueRepository.updateStatus(
@@ -494,10 +413,23 @@ export class NotifyFinishedIssuePreparationUseCase {
         awaitingWorkspaceStatusOption.id,
       );
       await this.patchConsoleTab(issue);
-      await this.issueCommentRepository.createComment(
-        issue,
-        RETURNED_TO_AWAITING_WORKSPACE_MESSAGE,
-      );
+      if (rejections.length > 0) {
+        await this.setDependedIssueUrlForAllOpenPRs(
+          issue,
+          params.issueUrl,
+          project,
+        );
+        await this.issueCommentRepository.createComment(
+          issue,
+          rejectionStatusMessage,
+        );
+      }
+      if (repetition.type === 'dispatchAgain') {
+        await this.issueCommentRepository.createComment(
+          issue,
+          repetition.comment,
+        );
+      }
       return;
     }
 
@@ -657,6 +589,7 @@ export class NotifyFinishedIssuePreparationUseCase {
     comments: { author: string; content: string }[],
     isTrustedAuthor: (author: string) => boolean,
     labelsNotRequiringPullRequest: string[],
+    nextStepAgent: string | null,
     developerAgentName?: string | null,
   ): Promise<{
     rejections: { type: RejectedReasonType; detail: string }[];
@@ -687,10 +620,7 @@ export class NotifyFinishedIssuePreparationUseCase {
         labelsNotRequiringPullRequest,
         { developerAgentName },
       );
-    const requiredPrRejections = isPullRequestDeclaredUnnecessary(
-      comments,
-      isTrustedAuthor,
-    )
+    const requiredPrRejections = isTriagerAgentName(nextStepAgent)
       ? prRejections.filter(
           (rejection) => rejection.type !== 'PULL_REQUEST_NOT_FOUND',
         )

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

@@ -388,7 +388,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       );
     });
 
-    it('should not revert an issue whose last agent report declares pullRequestRequired as false', async () => {
+    it('should not revert an issue whose last agent report designates the triager', async () => {
       const issue = createMockIssue({
         status: 'Awaiting Quality Check',
       });
@@ -401,7 +401,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
         {
           author: 'owner',
           content:
-            'From: :robot: Agent report\n```json\n{"pullRequestRequired": false}\n```',
+            'From: :robot: Agent report\n```json\n{"nextStepAgent": "triager"}\n```',
           createdAt: new Date(),
         },
       ]);
@@ -416,7 +416,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
       expect(mockIssueCommentRepository.createComment).not.toHaveBeenCalled();
     });
 
-    it('should still revert an issue whose linked PR is conflicted even when the last agent report declares pullRequestRequired as false', async () => {
+    it('should still revert an issue whose linked PR is conflicted even when the last agent report designates the triager', async () => {
       const issue = createMockIssue({
         status: 'Awaiting Quality Check',
       });
@@ -427,7 +427,7 @@ describe('RevertNotReadyReviewQueueIssueUseCase', () => {
         {
           author: 'owner',
           content:
-            'From: :robot: Agent report\n```json\n{"pullRequestRequired": false}\n```',
+            'From: :robot: Agent report\n```json\n{"nextStepAgent": "triager"}\n```',
           createdAt: new Date(),
         },
       ]);

package/src/domain/usecases/RevertNotReadyReviewQueueIssueUseCase.ts

@@ -5,7 +5,8 @@ import { IssueCommentRepository } from './adapter-interfaces/IssueCommentReposit
 import { IssueRejectionEvaluator } from './IssueRejectionEvaluator';
 import { ChangeTargetPullRequestApprover } from './ChangeTargetPullRequestApprover';
 import { resolveLabelsNotRequiringPullRequest } from './resolveLabelsNotRequiringPullRequest';
-import { isPullRequestDeclaredUnnecessary } from './isPullRequestDeclaredUnnecessary';
+import { extractNextStepAgentFromComments } from './extractNextStepAgentFromComments';
+import { isTriagerAgentName } from './triagerAgentName';
 import { isAuthorAuthorizedForAutoStatusCheck } from './isAuthorAuthorizedForAutoStatusCheck';
 import {
   AWAITING_QUALITY_CHECK_STATUS_NAME,
@@ -132,10 +133,15 @@ export class RevertNotReadyReviewQueueIssueUseCase {
         if (
           rejections.length === 1 &&
           rejections[0].type === 'PULL_REQUEST_NOT_FOUND' &&
-          isPullRequestDeclaredUnnecessary(
-            await this.issueCommentRepository.getCommentsFromIssue(issue),
-            (author) =>
-              isAuthorAuthorizedForAutoStatusCheck(author, allowedIssueAuthors),
+          isTriagerAgentName(
+            extractNextStepAgentFromComments(
+              await this.issueCommentRepository.getCommentsFromIssue(issue),
+              (author) =>
+                isAuthorAuthorizedForAutoStatusCheck(
+                  author,
+                  allowedIssueAuthors,
+                ),
+            ),
           )
         ) {
           continue;

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

@@ -597,226 +597,6 @@ describe('RevertOrphanedPreparationUseCase', () => {
     expect(mockIssueRepository.updateStatus.mock.calls[0][2]).toBe('1');
   });
 
-  it('should return an orphaned issue to Awaiting Workspace without a rejection when the last report first declares that no pull request is required', async () => {
-    const stuckIssue = createMockIssue({
-      url: 'https://github.com/user/repo/issues/10',
-      status: 'Preparation',
-    });
-    mockIssueRepository.getAllIssues.mockResolvedValue({
-      project: mockProject,
-      issues: [stuckIssue],
-      cacheUsed: false,
-    });
-    mockLocalCommandRunner.runCommand.mockResolvedValue({
-      stdout: '',
-      stderr: '',
-      exitCode: 1,
-    });
-    mockIssueCommentRepository.getCommentsFromIssue.mockResolvedValue([
-      {
-        author: 'agent-bot',
-        content:
-          'From: :robot: agent report\n\n```json\n{ "pullRequestRequired": false, "nextStep": null }\n```\n',
-        createdAt: new Date(),
-      },
-    ]);
-    mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([]);
-
-    await useCase.run({
-      projectUrl: 'https://github.com/user/repo',
-      preparationProcessCheckCommand: 'pgrep -fa "claude-agent.*{URL}"',
-      thresholdForAutoReject: 3,
-      allowedIssueAuthors: ['agent-bot'],
-    });
-
-    expect(mockIssueRepository.findRelatedOpenPRs.mock.calls).toHaveLength(0);
-    expect(mockIssueRepository.updateStatus.mock.calls).toHaveLength(1);
-    expect(mockIssueRepository.updateStatus.mock.calls[0][2]).toBe('1');
-    expect(mockIssueCommentRepository.createComment.mock.calls).toHaveLength(1);
-    expect(mockIssueCommentRepository.createComment.mock.calls[0][1]).toContain(
-      'Auto Status Check: RETURNED_TO_AWAITING_WORKSPACE',
-    );
-  });
-
-  it('should advance an orphaned issue to Awaiting Quality Check when it was already returned to the workspace once for the same declaration', async () => {
-    const stuckIssue = createMockIssue({
-      url: 'https://github.com/user/repo/issues/10',
-      status: 'Preparation',
-    });
-    mockIssueRepository.getAllIssues.mockResolvedValue({
-      project: mockProject,
-      issues: [stuckIssue],
-      cacheUsed: false,
-    });
-    mockLocalCommandRunner.runCommand.mockResolvedValue({
-      stdout: '',
-      stderr: '',
-      exitCode: 1,
-    });
-    mockIssueCommentRepository.getCommentsFromIssue.mockResolvedValue([
-      {
-        author: 'agent-bot',
-        content:
-          'Auto Status Check: RETURNED_TO_AWAITING_WORKSPACE\nThe last report declared that this task needs no pull request.',
-        createdAt: new Date(),
-      },
-      {
-        author: 'agent-bot',
-        content:
-          'From: :robot: agent report\n\n```json\n{ "pullRequestRequired": false, "nextStep": null }\n```\n',
-        createdAt: new Date(),
-      },
-    ]);
-    mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([]);
-
-    await useCase.run({
-      projectUrl: 'https://github.com/user/repo',
-      preparationProcessCheckCommand: 'pgrep -fa "claude-agent.*{URL}"',
-      thresholdForAutoReject: 3,
-      allowedIssueAuthors: ['agent-bot'],
-    });
-
-    expect(mockIssueRepository.findRelatedOpenPRs.mock.calls).toHaveLength(0);
-    expect(mockIssueRepository.updateStatus.mock.calls).toHaveLength(1);
-    expect(mockIssueRepository.updateStatus.mock.calls[0][2]).toBe('4');
-    expect(mockIssueCommentRepository.createComment.mock.calls).toHaveLength(0);
-  });
-
-  it('should not re-dispatch the declared agent when a later report omits the next step agent an earlier report declared', async () => {
-    const stuckIssue = createMockIssue({
-      url: 'https://github.com/user/repo/issues/10',
-      status: 'Preparation',
-    });
-    mockIssueRepository.getAllIssues.mockResolvedValue({
-      project: mockProject,
-      issues: [stuckIssue],
-      cacheUsed: false,
-    });
-    mockLocalCommandRunner.runCommand.mockResolvedValue({
-      stdout: '',
-      stderr: '',
-      exitCode: 1,
-    });
-    mockIssueCommentRepository.getCommentsFromIssue.mockResolvedValue([
-      {
-        author: 'agent-bot',
-        content:
-          'From: :robot: agent report\n\n```json\n{ "nextStepAgent": "pr-reviewer" }\n```\n',
-        createdAt: new Date(),
-      },
-      {
-        author: 'agent-bot',
-        content:
-          'Auto Status Check: RETURNED_TO_AWAITING_WORKSPACE\nThe last report declared that this task needs no pull request.',
-        createdAt: new Date(),
-      },
-      {
-        author: 'agent-bot',
-        content:
-          'From: :robot: agent report\n\n```json\n{ "pullRequestRequired": false, "reviewResult": "PASS", "nextStep": null }\n```\n',
-        createdAt: new Date(),
-      },
-    ]);
-    mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([]);
-
-    await useCase.run({
-      projectUrl: 'https://github.com/user/repo',
-      preparationProcessCheckCommand: 'pgrep -fa "claude-agent.*{URL}"',
-      thresholdForAutoReject: 3,
-      allowedIssueAuthors: ['agent-bot'],
-    });
-
-    expect(mockIssueRepository.setIssueAgentField.mock.calls).toHaveLength(0);
-    expect(mockIssueRepository.updateStatus.mock.calls).toHaveLength(1);
-    expect(mockIssueRepository.updateStatus.mock.calls[0][2]).toBe('4');
-  });
-
-  it('should revert an orphaned issue to Awaiting Workspace when the no pull request declaration comes from an author outside allowedIssueAuthors', async () => {
-    const stuckIssue = createMockIssue({
-      url: 'https://github.com/user/repo/issues/10',
-      status: 'Preparation',
-    });
-    mockIssueRepository.getAllIssues.mockResolvedValue({
-      project: mockProject,
-      issues: [stuckIssue],
-      cacheUsed: false,
-    });
-    mockLocalCommandRunner.runCommand.mockResolvedValue({
-      stdout: '',
-      stderr: '',
-      exitCode: 1,
-    });
-    mockIssueCommentRepository.getCommentsFromIssue.mockResolvedValue([
-      {
-        author: 'stranger',
-        content:
-          'From: :robot: agent report\n\n```json\n{ "pullRequestRequired": false, "nextStep": null }\n```\n',
-        createdAt: new Date(),
-      },
-    ]);
-    mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([]);
-
-    await useCase.run({
-      projectUrl: 'https://github.com/user/repo',
-      preparationProcessCheckCommand: 'pgrep -fa "claude-agent.*{URL}"',
-      thresholdForAutoReject: 3,
-      allowedIssueAuthors: ['agent-bot'],
-    });
-
-    expect(mockIssueRepository.updateStatus.mock.calls).toHaveLength(1);
-    expect(mockIssueRepository.updateStatus.mock.calls[0][2]).toBe('1');
-    expect(mockIssueCommentRepository.createComment.mock.calls).toHaveLength(1);
-  });
-
-  it('should advance an orphaned issue to Awaiting Quality Check when its own status comments were written after the report declaring no pull request is required', async () => {
-    const stuckIssue = createMockIssue({
-      url: 'https://github.com/user/repo/issues/10',
-      status: 'Preparation',
-    });
-    mockIssueRepository.getAllIssues.mockResolvedValue({
-      project: mockProject,
-      issues: [stuckIssue],
-      cacheUsed: false,
-    });
-    mockLocalCommandRunner.runCommand.mockResolvedValue({
-      stdout: '',
-      stderr: '',
-      exitCode: 1,
-    });
-    mockIssueCommentRepository.getCommentsFromIssue.mockResolvedValue([
-      {
-        author: 'agent-bot',
-        content:
-          'From: :robot: agent report\n\n```json\n{ "pullRequestRequired": false, "nextStep": null }\n```\n',
-        createdAt: new Date(),
-      },
-      {
-        author: 'agent-bot',
-        content:
-          'Auto Status Check: RETURNED_TO_AWAITING_WORKSPACE\nThe last report declared that this task needs no pull request.',
-        createdAt: new Date(),
-      },
-      {
-        author: 'agent-bot',
-        content: 'Auto Status Check: REJECTED\n- ORPHANED_PREPARATION',
-        createdAt: new Date(),
-      },
-    ]);
-    mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([]);
-
-    await useCase.run({
-      projectUrl: 'https://github.com/user/repo',
-      preparationProcessCheckCommand: 'pgrep -fa "claude-agent.*{URL}"',
-      thresholdForAutoReject: 3,
-      allowedIssueAuthors: ['agent-bot'],
-    });
-
-    expect(mockIssueRepository.findRelatedOpenPRs.mock.calls).toHaveLength(0);
-    expect(mockIssueRepository.updateStatus.mock.calls).toHaveLength(1);
-    expect(mockIssueRepository.updateStatus.mock.calls[0][2]).toBe('4');
-    expect(mockIssueCommentRepository.createComment.mock.calls).toHaveLength(0);
-  });
-
   it('should keep rejecting an orphaned issue whose own status comments follow a report that declares nothing', async () => {
     const stuckIssue = createMockIssue({
       url: 'https://github.com/user/repo/issues/10',
@@ -2609,97 +2389,4 @@ describe('RevertOrphanedPreparationUseCase', () => {
       'Auto Status Check: REJECTED',
     );
   });
-
-  it('should move an orphaned issue to Awaiting Quality Check with AWAITING_OWNER_APPROVAL when last report declares waitingForOwnerApproval', async () => {
-    const stuckIssue = createMockIssue({
-      url: 'https://github.com/user/repo/issues/10',
-      status: 'Preparation',
-    });
-    mockIssueRepository.getAllIssues.mockResolvedValue({
-      project: mockProject,
-      issues: [stuckIssue],
-      cacheUsed: false,
-    });
-    mockLocalCommandRunner.runCommand.mockResolvedValue({
-      stdout: '',
-      stderr: '',
-      exitCode: 1,
-    });
-    mockIssueCommentRepository.getCommentsFromIssue.mockResolvedValue([
-      {
-        author: 'agent-bot',
-        content:
-          'From: :robot: systems-analyst (model)\n```json\n{"pullRequestRequired": false, "waitingForOwnerApproval": true}\n```',
-        createdAt: new Date(),
-      },
-    ]);
-    mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([]);
-
-    await useCase.run({
-      projectUrl: 'https://github.com/user/repo',
-      preparationProcessCheckCommand: 'pgrep -fa "claude-agent.*{URL}"',
-      thresholdForAutoReject: 3,
-      allowedIssueAuthors: ['agent-bot'],
-    });
-
-    expect(mockIssueRepository.updateStatus.mock.calls).toHaveLength(1);
-    expect(mockIssueRepository.updateStatus.mock.calls[0][2]).toBe('4');
-    expect(mockIssueCommentRepository.createComment.mock.calls).toHaveLength(1);
-    expect(mockIssueCommentRepository.createComment.mock.calls[0][1]).toContain(
-      'Auto Status Check: AWAITING_OWNER_APPROVAL',
-    );
-  });
-
-  it('should escalate to Failed Preparation after ownerApprovalTimeoutCycles AWAITING_OWNER_APPROVAL messages in RevertOrphaned', async () => {
-    const stuckIssue = createMockIssue({
-      url: 'https://github.com/user/repo/issues/10',
-      status: 'Preparation',
-    });
-    mockIssueRepository.getAllIssues.mockResolvedValue({
-      project: mockProject,
-      issues: [stuckIssue],
-      cacheUsed: false,
-    });
-    mockLocalCommandRunner.runCommand.mockResolvedValue({
-      stdout: '',
-      stderr: '',
-      exitCode: 1,
-    });
-    mockIssueCommentRepository.getCommentsFromIssue.mockResolvedValue([
-      {
-        author: 'agent-bot',
-        content:
-          'From: :robot: systems-analyst (model)\n```json\n{"pullRequestRequired": false, "waitingForOwnerApproval": true}\n```',
-        createdAt: new Date(),
-      },
-      {
-        author: 'agent-bot',
-        content:
-          'Auto Status Check: AWAITING_OWNER_APPROVAL\nThe last report declared that this task is waiting for owner approval. Returning to Awaiting Workspace for the next cycle.',
-        createdAt: new Date(),
-      },
-      {
-        author: 'agent-bot',
-        content:
-          'Auto Status Check: AWAITING_OWNER_APPROVAL\nThe last report declared that this task is waiting for owner approval. Returning to Awaiting Workspace for the next cycle.',
-        createdAt: new Date(),
-      },
-    ]);
-    mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([]);
-
-    await useCase.run({
-      projectUrl: 'https://github.com/user/repo',
-      preparationProcessCheckCommand: 'pgrep -fa "claude-agent.*{URL}"',
-      thresholdForAutoReject: 3,
-      allowedIssueAuthors: ['agent-bot'],
-      ownerApprovalTimeoutCycles: 2,
-    });
-
-    expect(mockIssueRepository.updateStatus.mock.calls).toHaveLength(1);
-    expect(mockIssueRepository.updateStatus.mock.calls[0][2]).toBe('5');
-    expect(mockIssueCommentRepository.createComment.mock.calls).toHaveLength(1);
-    expect(mockIssueCommentRepository.createComment.mock.calls[0][1]).toContain(
-      'Owner approval was not received after 2 cycles. Moving to Failed Preparation.',
-    );
-  });
 });