github-issue-tower-defence-management 2.19.1 → 2.19.3

package/src/domain/usecases/ChangeStatusByStoryColorUseCase.ts

@@ -1,80 +0,0 @@
-import { IssueRepository } from './adapter-interfaces/IssueRepository';
-import { Project } from '../entities/Project';
-import { DateRepository } from './adapter-interfaces/DateRepository';
-import { StoryObjectMap } from '../entities/StoryObjectMap';
-import { ICEBOX_STATUS_NAME } from '../entities/WorkflowStatus';
-import { Member } from '../entities/Member';
-
-export class ChangeStatusByStoryColorUseCase {
-  constructor(
-    readonly dateRepository: Pick<DateRepository, 'now'>,
-    readonly issueRepository: Pick<
-      IssueRepository,
-      'updateStatus' | 'createComment'
-    >,
-  ) {}
-
-  run = async (input: {
-    project: Project;
-    cacheUsed: boolean;
-    org: string;
-    repo: string;
-    storyObjectMap: StoryObjectMap;
-    manager: Member['name'];
-  }): Promise<void> => {
-    const firstStatus = input.project.status.statuses[0];
-    if (!firstStatus) {
-      throw new Error('First status is not found');
-    } else if (input.cacheUsed) {
-      return;
-    }
-    const disabledStatusObject = input.project.status.statuses.find(
-      (status) => status.name === ICEBOX_STATUS_NAME,
-    );
-    if (!disabledStatusObject) {
-      throw new Error('Icebox status is not found');
-    }
-    for (const storyObject of Array.from(input.storyObjectMap.values())) {
-      const isStoryDisabled = storyObject.story.color === 'GRAY';
-      for (const issue of storyObject.issues) {
-        if (isStoryDisabled) {
-          if (issue.status && issue.status === ICEBOX_STATUS_NAME) {
-            continue;
-          }
-          await this.issueRepository.updateStatus(
-            input.project,
-            issue,
-            disabledStatusObject.id,
-          );
-          await this.issueRepository.createComment(
-            issue,
-            `This issue status is changed because the story is disabled.`,
-          );
-        } else if (!isStoryDisabled) {
-          if (issue.status && issue.status !== ICEBOX_STATUS_NAME) {
-            continue;
-          }
-          const hasNoStatus = !issue.status;
-          const isOwnedByNonManagerAssignee = issue.assignees.some(
-            (assignee) => assignee !== input.manager,
-          );
-          if (hasNoStatus && isOwnedByNonManagerAssignee) {
-            console.warn(
-              `ChangeStatusByStoryColorUseCase: skipping the first status write because the issue has no status and is assigned to someone other than the manager. issueUrl: ${issue.url} assignees: ${issue.assignees.join(', ')}`,
-            );
-            continue;
-          }
-          await this.issueRepository.updateStatus(
-            input.project,
-            issue,
-            firstStatus.id,
-          );
-          await this.issueRepository.createComment(
-            issue,
-            `This issue status is changed because the story is enabled.`,
-          );
-        }
-      }
-    }
-  };
-}

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

@@ -1,248 +0,0 @@
-import { ChangeTargetPullRequestApprover } from './ChangeTargetPullRequestApprover';
-
-describe('ChangeTargetPullRequestApprover', () => {
-  let mockIssueRepository: {
-    getPullRequestChangedFilePaths: jest.Mock;
-    approvePullRequest: jest.Mock;
-  };
-  let approver: ChangeTargetPullRequestApprover;
-  const prUrl = 'https://github.com/user/repo/pull/1';
-
-  beforeEach(() => {
-    jest.resetAllMocks();
-    mockIssueRepository = {
-      getPullRequestChangedFilePaths: jest.fn().mockResolvedValue([]),
-      approvePullRequest: jest.fn().mockResolvedValue(undefined),
-    };
-    approver = new ChangeTargetPullRequestApprover(mockIssueRepository);
-  });
-
-  it('should do nothing when approvedPrUrl is null', async () => {
-    await approver.approveIfConfined(['change-target:src/domain'], null);
-
-    expect(
-      mockIssueRepository.getPullRequestChangedFilePaths,
-    ).not.toHaveBeenCalled();
-    expect(mockIssueRepository.approvePullRequest).not.toHaveBeenCalled();
-  });
-
-  it('should do nothing when issue has no change-target label', async () => {
-    await approver.approveIfConfined(['category:e2e'], prUrl);
-
-    expect(
-      mockIssueRepository.getPullRequestChangedFilePaths,
-    ).not.toHaveBeenCalled();
-    expect(mockIssueRepository.approvePullRequest).not.toHaveBeenCalled();
-  });
-
-  it('should approve when all files are confined to the labeled path', async () => {
-    mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-      'src/domain/entities/Foo.ts',
-      'src/domain/usecases/Bar.ts',
-    ]);
-
-    await approver.approveIfConfined(['change-target:src/domain'], prUrl);
-
-    expect(
-      mockIssueRepository.getPullRequestChangedFilePaths,
-    ).toHaveBeenCalledWith(prUrl);
-    expect(mockIssueRepository.approvePullRequest).toHaveBeenCalledWith(prUrl);
-  });
-
-  it('should not approve when any file is outside the labeled path', async () => {
-    mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-      'src/domain/entities/Foo.ts',
-      'src/adapter/Outside.ts',
-    ]);
-
-    await approver.approveIfConfined(['change-target:src/domain'], prUrl);
-
-    expect(mockIssueRepository.approvePullRequest).not.toHaveBeenCalled();
-  });
-
-  it('should approve when files are confined under any of multiple labels', async () => {
-    mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-      'src/domain/Foo.ts',
-      'docs/intro.md',
-    ]);
-
-    await approver.approveIfConfined(
-      ['change-target:src/domain', 'change-target:docs'],
-      prUrl,
-    );
-
-    expect(mockIssueRepository.approvePullRequest).toHaveBeenCalledWith(prUrl);
-  });
-
-  it('should match boundary-safely (foo matches foo/bar.ts but not foobar/baz.ts)', async () => {
-    mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-      'foo/bar.ts',
-      'foobar/baz.ts',
-    ]);
-
-    await approver.approveIfConfined(['change-target:foo'], prUrl);
-
-    expect(mockIssueRepository.approvePullRequest).not.toHaveBeenCalled();
-  });
-
-  it('should approve when changed file equals the labeled path exactly', async () => {
-    mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-      'README.md',
-    ]);
-
-    await approver.approveIfConfined(['change-target:README.md'], prUrl);
-
-    expect(mockIssueRepository.approvePullRequest).toHaveBeenCalledWith(prUrl);
-  });
-
-  it('should not approve when there are zero changed files', async () => {
-    mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([]);
-
-    await approver.approveIfConfined(['change-target:src/domain'], prUrl);
-
-    expect(
-      mockIssueRepository.getPullRequestChangedFilePaths,
-    ).toHaveBeenCalledWith(prUrl);
-    expect(mockIssueRepository.approvePullRequest).not.toHaveBeenCalled();
-  });
-
-  it('should normalize trailing slashes in change-target label paths', async () => {
-    mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-      'src/domain/entities/Foo.ts',
-    ]);
-
-    await approver.approveIfConfined(['change-target:src/domain/'], prUrl);
-
-    expect(mockIssueRepository.approvePullRequest).toHaveBeenCalledWith(prUrl);
-  });
-
-  it('should normalize leading slashes in change-target label paths', async () => {
-    mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-      'src/domain/entities/Foo.ts',
-    ]);
-
-    await approver.approveIfConfined(['change-target:/src/domain'], prUrl);
-
-    expect(mockIssueRepository.approvePullRequest).toHaveBeenCalledWith(prUrl);
-  });
-
-  it('should normalize both leading and trailing slashes in change-target label paths', async () => {
-    mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-      'src/domain/entities/Foo.ts',
-    ]);
-
-    await approver.approveIfConfined(['change-target:/src/domain/'], prUrl);
-
-    expect(mockIssueRepository.approvePullRequest).toHaveBeenCalledWith(prUrl);
-  });
-
-  it('should expand a path alias when pathAliases map is provided', async () => {
-    mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-      'src/domain/usecases/adapter-interfaces/IssueRepository.ts',
-    ]);
-
-    await approver.approveIfConfined(['change-target:adapters'], prUrl, {
-      adapters: 'src/domain/usecases/adapter-interfaces',
-    });
-
-    expect(mockIssueRepository.approvePullRequest).toHaveBeenCalledWith(prUrl);
-  });
-
-  it('should not approve when file is outside the expanded alias path', async () => {
-    mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-      'src/domain/usecases/SomeOtherUseCase.ts',
-    ]);
-
-    await approver.approveIfConfined(['change-target:adapters'], prUrl, {
-      adapters: 'src/domain/usecases/adapter-interfaces',
-    });
-
-    expect(mockIssueRepository.approvePullRequest).not.toHaveBeenCalled();
-  });
-
-  it('should treat label value as literal path when it does not match any alias key', async () => {
-    mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-      'src/domain/entities/Foo.ts',
-    ]);
-
-    await approver.approveIfConfined(['change-target:src/domain'], prUrl, {
-      adapters: 'src/domain/usecases/adapter-interfaces',
-    });
-
-    expect(mockIssueRepository.approvePullRequest).toHaveBeenCalledWith(prUrl);
-  });
-
-  it('should normalize leading slash in alias expanded value', async () => {
-    mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-      'src/domain/entities/Foo.ts',
-    ]);
-
-    await approver.approveIfConfined(['change-target:domain'], prUrl, {
-      domain: '/src/domain',
-    });
-
-    expect(mockIssueRepository.approvePullRequest).toHaveBeenCalledWith(prUrl);
-  });
-
-  it('should ignore empty change-target label values', async () => {
-    mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-      'src/domain/Foo.ts',
-    ]);
-
-    await approver.approveIfConfined(['change-target:'], prUrl);
-
-    expect(
-      mockIssueRepository.getPullRequestChangedFilePaths,
-    ).not.toHaveBeenCalled();
-    expect(mockIssueRepository.approvePullRequest).not.toHaveBeenCalled();
-  });
-
-  it('should normalize leading slashes in change-target-must label paths', async () => {
-    mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-      'src/domain/entities/Foo.ts',
-    ]);
-
-    await approver.approveIfConfined(['change-target-must:/src/domain'], prUrl);
-
-    expect(mockIssueRepository.approvePullRequest).toHaveBeenCalledWith(prUrl);
-  });
-
-  it('should treat change-target-must: path as an allowed confinement path and approve when all files are confined to it', async () => {
-    mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-      'src/domain/entities/Foo.ts',
-      'src/domain/usecases/Bar.ts',
-    ]);
-
-    await approver.approveIfConfined(['change-target-must:src/domain'], prUrl);
-
-    expect(
-      mockIssueRepository.getPullRequestChangedFilePaths,
-    ).toHaveBeenCalledWith(prUrl);
-    expect(mockIssueRepository.approvePullRequest).toHaveBeenCalledWith(prUrl);
-  });
-
-  it('should not approve when a file is outside the change-target-must path', async () => {
-    mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-      'src/domain/entities/Foo.ts',
-      'src/adapter/Outside.ts',
-    ]);
-
-    await approver.approveIfConfined(['change-target-must:src/domain'], prUrl);
-
-    expect(mockIssueRepository.approvePullRequest).not.toHaveBeenCalled();
-  });
-
-  it('should not approve when a file is outside both change-target and change-target-must allowed paths', async () => {
-    mockIssueRepository.getPullRequestChangedFilePaths.mockResolvedValue([
-      'src/domain/Foo.ts',
-      'unrelated/Outside.ts',
-    ]);
-
-    await approver.approveIfConfined(
-      ['change-target:src/domain', 'change-target-must:docs'],
-      prUrl,
-    );
-
-    expect(mockIssueRepository.approvePullRequest).not.toHaveBeenCalled();
-  });
-});

package/src/domain/usecases/ChangeTargetPullRequestApprover.ts

@@ -1,69 +0,0 @@
-import { IssueRepository } from './adapter-interfaces/IssueRepository';
-
-export class ChangeTargetPullRequestApprover {
-  constructor(
-    private readonly issueRepository: Pick<
-      IssueRepository,
-      'getPullRequestChangedFilePaths' | 'approvePullRequest'
-    >,
-  ) {}
-
-  approveIfConfined = async (
-    issueLabels: string[],
-    approvedPrUrl: string | null,
-    pathAliases?: Record<string, string> | null,
-  ): Promise<void> => {
-    if (approvedPrUrl === null) {
-      return;
-    }
-    const changeTargetPaths = this.extractChangeTargetPaths(
-      issueLabels,
-      pathAliases,
-    );
-    if (changeTargetPaths.length === 0) {
-      return;
-    }
-    const changedFilePaths =
-      await this.issueRepository.getPullRequestChangedFilePaths(approvedPrUrl);
-    if (changedFilePaths.length === 0) {
-      return;
-    }
-    const allConfined = changedFilePaths.every((filePath) =>
-      this.isFilePathConfinedToAllowedPaths(filePath, changeTargetPaths),
-    );
-    if (!allConfined) {
-      return;
-    }
-    await this.issueRepository.approvePullRequest(approvedPrUrl);
-  };
-
-  private extractChangeTargetPaths = (
-    labels: string[],
-    pathAliases?: Record<string, string> | null,
-  ): string[] => {
-    const prefixes = ['change-target:', 'change-target-must:'];
-    const paths: string[] = [];
-    for (const label of labels) {
-      const matchedPrefix = prefixes.find((p) => label.startsWith(p));
-      if (!matchedPrefix) continue;
-      const raw = label.slice(matchedPrefix.length).trim();
-      if (raw.length === 0) continue;
-      const normalized = raw.replace(/^\/+/, '').replace(/\/+$/, '');
-      if (normalized.length === 0) continue;
-      const aliasExpanded = pathAliases?.[normalized] ?? normalized;
-      const finalPath = aliasExpanded.replace(/^\/+/, '').replace(/\/+$/, '');
-      if (finalPath.length === 0) continue;
-      paths.push(finalPath);
-    }
-    return paths;
-  };
-
-  private isFilePathConfinedToAllowedPaths = (
-    filePath: string,
-    allowedPaths: string[],
-  ): boolean =>
-    allowedPaths.some(
-      (allowedPath) =>
-        filePath === allowedPath || filePath.startsWith(`${allowedPath}/`),
-    );
-}