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

package/src/adapter/repositories/ProcTakeOwnershipSpawnRepository.ts

@@ -1,134 +0,0 @@
-import * as fs from 'fs';
-import * as path from 'path';
-import {
-  TakeOwnershipSpawn,
-  TakeOwnershipSpawnRepository,
-} from '../../domain/usecases/adapter-interfaces/TakeOwnershipSpawnRepository';
-
-const DEFAULT_PROC_DIRECTORY = '/proc';
-const OAUTH_TOKEN_ENVIRON_KEY = 'CLAUDE_CODE_OAUTH_TOKEN';
-const TAKE_OWNERSHIP_MARKER = 'Take ownership';
-const LOG_PATH_PATTERN = /\/logs-aw\/[^\0"]+\.log/;
-const ISSUE_URL_PATTERN = /Take ownership of (https:\/\/github\.com\/[^ \0"]+)/;
-
-const parseCommandLineArguments = (cmdline: string): string[] =>
-  cmdline.split('\0').filter((argument) => argument.length > 0);
-
-const isTakeOwnershipSpawn = (commandArguments: string[]): boolean =>
-  commandArguments.some((argument) => argument.includes(TAKE_OWNERSHIP_MARKER));
-
-const extractLogPath = (cmdline: string): string | null => {
-  const match = cmdline.match(LOG_PATH_PATTERN);
-  return match === null ? null : match[0];
-};
-
-export class ProcTakeOwnershipSpawnRepository implements TakeOwnershipSpawnRepository {
-  constructor(
-    private readonly procDirectory: string = DEFAULT_PROC_DIRECTORY,
-  ) {}
-
-  listSpawns = (): TakeOwnershipSpawn[] => {
-    const spawns: TakeOwnershipSpawn[] = [];
-    for (const pidDirectory of this.listProcessIdDirectories()) {
-      const spawn = this.readSpawn(pidDirectory);
-      if (spawn !== null) {
-        spawns.push(spawn);
-      }
-    }
-    return spawns;
-  };
-
-  listRunningIssueUrls = (): string[] => {
-    const urls: string[] = [];
-    for (const pidDirectory of this.listProcessIdDirectories()) {
-      const rawCmdline = this.readRawCmdline(pidDirectory);
-      if (rawCmdline === null) {
-        continue;
-      }
-      const match = rawCmdline.match(ISSUE_URL_PATTERN);
-      if (match !== null) {
-        urls.push(match[1]);
-      }
-    }
-    return urls;
-  };
-
-  private listProcessIdDirectories = (): string[] => {
-    let entries: string[];
-    try {
-      entries = fs.readdirSync(this.procDirectory);
-    } catch {
-      return [];
-    }
-    return entries.filter((entry) => /^\d+$/.test(entry));
-  };
-
-  private readSpawn = (
-    processIdDirectory: string,
-  ): TakeOwnershipSpawn | null => {
-    const rawCmdline = this.readRawCmdline(processIdDirectory);
-    if (rawCmdline === null) {
-      return null;
-    }
-    const commandArguments = parseCommandLineArguments(rawCmdline);
-    if (!isTakeOwnershipSpawn(commandArguments)) {
-      return null;
-    }
-    const logPath = extractLogPath(rawCmdline);
-    if (logPath === null) {
-      return null;
-    }
-    const environ = this.readEnviron(processIdDirectory);
-    if (environ === null) {
-      return null;
-    }
-    const token = environ.get(OAUTH_TOKEN_ENVIRON_KEY);
-    if (token === undefined || token.length === 0) {
-      return null;
-    }
-    return { token, logPath };
-  };
-
-  private readRawCmdline = (processIdDirectory: string): string | null => {
-    const cmdlinePath = path.join(
-      this.procDirectory,
-      processIdDirectory,
-      'cmdline',
-    );
-    try {
-      return fs.readFileSync(cmdlinePath, 'utf8');
-    } catch {
-      return null;
-    }
-  };
-
-  private readEnviron = (
-    processIdDirectory: string,
-  ): Map<string, string> | null => {
-    const environPath = path.join(
-      this.procDirectory,
-      processIdDirectory,
-      'environ',
-    );
-    let raw: string;
-    try {
-      raw = fs.readFileSync(environPath, 'utf8');
-    } catch {
-      return null;
-    }
-    const environ = new Map<string, string>();
-    for (const entry of raw.split('\0')) {
-      if (entry.length === 0) {
-        continue;
-      }
-      const separatorIndex = entry.indexOf('=');
-      if (separatorIndex <= 0) {
-        continue;
-      }
-      const key = entry.slice(0, separatorIndex);
-      const value = entry.slice(separatorIndex + 1);
-      environ.set(key, value);
-    }
-    return environ;
-  };
-}

package/src/adapter/repositories/ProcessListSessionSubAgentActivityRepository.test.ts

@@ -1,121 +0,0 @@
-import { ProcessListSessionSubAgentActivityRepository } from './ProcessListSessionSubAgentActivityRepository';
-import { SubAgentProcessLister } from '../../domain/usecases/adapter-interfaces/SubAgentProcessLister';
-import { SubAgentSilentSecondsResolver } from '../../domain/usecases/adapter-interfaces/SubAgentSilentSecondsResolver';
-
-type Mocked<T> = jest.Mocked<T> & jest.MockedObject<T>;
-
-const createLister = (
-  processes: { commandLine: string; elapsedSeconds: number }[],
-): Mocked<SubAgentProcessLister> => ({
-  listProcesses: jest.fn().mockResolvedValue(processes),
-});
-
-const createResolver = (
-  silentSeconds: number,
-): Mocked<SubAgentSilentSecondsResolver> => ({
-  resolveSilentSeconds: jest.fn().mockReturnValue(silentSeconds),
-});
-
-describe('ProcessListSessionSubAgentActivityRepository', () => {
-  it('returns an empty map when the match pattern is not configured', async () => {
-    const repository = new ProcessListSessionSubAgentActivityRepository(
-      null,
-      createLister([
-        { commandLine: 'anything session=s1', elapsedSeconds: 10 },
-      ]),
-      createResolver(0),
-    );
-    const result = await repository.listSubAgentActivitiesBySessionName(['s1']);
-    expect(result.size).toBe(0);
-  });
-
-  it('returns an empty map when the match pattern is an empty string', async () => {
-    const repository = new ProcessListSessionSubAgentActivityRepository(
-      '',
-      createLister([
-        { commandLine: 'anything session=s1', elapsedSeconds: 10 },
-      ]),
-      createResolver(0),
-    );
-    const result = await repository.listSubAgentActivitiesBySessionName(['s1']);
-    expect(result.size).toBe(0);
-  });
-
-  it('associates a matched process to a monitored session using the session capture group', async () => {
-    const repository = new ProcessListSessionSubAgentActivityRepository(
-      'session=(?<session>[^ ]+) label=(?<label>[^ ]+)',
-      createLister([
-        {
-          commandLine: 'worker session=s1 label=task-a',
-          elapsedSeconds: 1200,
-        },
-      ]),
-      createResolver(180),
-    );
-    const result = await repository.listSubAgentActivitiesBySessionName(['s1']);
-    expect(result.get('s1')).toEqual([
-      {
-        label: 'task-a',
-        silentSeconds: 180,
-        runningSeconds: 1200,
-        waitingOnExternalProcess: false,
-        finishedResultUnconsumed: false,
-      },
-    ]);
-  });
-
-  it('ignores processes whose captured session is not monitored', async () => {
-    const repository = new ProcessListSessionSubAgentActivityRepository(
-      'session=(?<session>[^ ]+)',
-      createLister([
-        { commandLine: 'worker session=other', elapsedSeconds: 1200 },
-      ]),
-      createResolver(0),
-    );
-    const result = await repository.listSubAgentActivitiesBySessionName(['s1']);
-    expect(result.size).toBe(0);
-  });
-
-  it('falls back to the session name as the label when no label group is captured', async () => {
-    const repository = new ProcessListSessionSubAgentActivityRepository(
-      'session=(?<session>[^ ]+)',
-      createLister([{ commandLine: 'worker session=s1', elapsedSeconds: 30 }]),
-      createResolver(0),
-    );
-    const result = await repository.listSubAgentActivitiesBySessionName(['s1']);
-    expect(result.get('s1')).toEqual([
-      {
-        label: 's1',
-        silentSeconds: 0,
-        runningSeconds: 30,
-        waitingOnExternalProcess: false,
-        finishedResultUnconsumed: false,
-      },
-    ]);
-  });
-
-  it('groups multiple matched processes under the same session', async () => {
-    const repository = new ProcessListSessionSubAgentActivityRepository(
-      'session=(?<session>[^ ]+) label=(?<label>[^ ]+)',
-      createLister([
-        { commandLine: 'worker session=s1 label=task-a', elapsedSeconds: 10 },
-        { commandLine: 'worker session=s1 label=task-b', elapsedSeconds: 20 },
-      ]),
-      createResolver(0),
-    );
-    const result = await repository.listSubAgentActivitiesBySessionName(['s1']);
-    expect(result.get('s1')).toHaveLength(2);
-  });
-
-  it('ignores processes that do not match the pattern', async () => {
-    const repository = new ProcessListSessionSubAgentActivityRepository(
-      'session=(?<session>[^ ]+)',
-      createLister([
-        { commandLine: 'unrelated process', elapsedSeconds: 1200 },
-      ]),
-      createResolver(0),
-    );
-    const result = await repository.listSubAgentActivitiesBySessionName(['s1']);
-    expect(result.size).toBe(0);
-  });
-});

package/src/adapter/repositories/ProcessListSessionSubAgentActivityRepository.ts

@@ -1,58 +0,0 @@
-import { SubAgentActivity } from '../../domain/entities/LiveSessionActivitySnapshot';
-import { SessionSubAgentActivityRepository } from '../../domain/usecases/adapter-interfaces/SessionSubAgentActivityRepository';
-import { SubAgentProcessLister } from '../../domain/usecases/adapter-interfaces/SubAgentProcessLister';
-import { SubAgentSilentSecondsResolver } from '../../domain/usecases/adapter-interfaces/SubAgentSilentSecondsResolver';
-
-export class ProcessListSessionSubAgentActivityRepository implements SessionSubAgentActivityRepository {
-  private readonly matchRegExp: RegExp | null;
-
-  constructor(
-    matchPattern: string | null,
-    private readonly processLister: SubAgentProcessLister,
-    private readonly silentSecondsResolver: SubAgentSilentSecondsResolver,
-  ) {
-    this.matchRegExp =
-      matchPattern === null || matchPattern.length === 0
-        ? null
-        : new RegExp(matchPattern);
-  }
-
-  listSubAgentActivitiesBySessionName = async (
-    sessionNames: string[],
-  ): Promise<Map<string, SubAgentActivity[]>> => {
-    const result = new Map<string, SubAgentActivity[]>();
-    if (this.matchRegExp === null) {
-      return result;
-    }
-    const monitoredSessionNames = new Set(sessionNames);
-    const processes = await this.processLister.listProcesses();
-    for (const process of processes) {
-      const match = this.matchRegExp.exec(process.commandLine);
-      if (match === null || match.groups === undefined) {
-        continue;
-      }
-      const sessionName = match.groups.session;
-      if (
-        sessionName === undefined ||
-        !monitoredSessionNames.has(sessionName)
-      ) {
-        continue;
-      }
-      const label = match.groups.label ?? sessionName;
-      const activity: SubAgentActivity = {
-        label,
-        silentSeconds: this.silentSecondsResolver.resolveSilentSeconds(label),
-        runningSeconds: process.elapsedSeconds,
-        waitingOnExternalProcess: false,
-        finishedResultUnconsumed: false,
-      };
-      const existing = result.get(sessionName);
-      if (existing === undefined) {
-        result.set(sessionName, [activity]);
-      } else {
-        existing.push(activity);
-      }
-    }
-    return result;
-  };
-}

package/src/adapter/repositories/ProjectIssuesCacheRepository.test.ts

@@ -1,218 +0,0 @@
-const mockPost = jest.fn();
-
-jest.mock('ky', () => ({
-  default: {
-    post: mockPost,
-    get: jest.fn(),
-    put: jest.fn(),
-    patch: jest.fn(),
-    delete: jest.fn(),
-    extend: jest.fn(),
-    create: jest.fn(),
-    stop: jest.fn(),
-  },
-  __esModule: true,
-}));
-
-import { mock } from 'jest-mock-extended';
-import { ProjectIssuesCacheRepository } from './ProjectIssuesCacheRepository';
-import { GraphqlProjectRepository } from './GraphqlProjectRepository';
-import { LocalStorageCacheRepository } from './LocalStorageCacheRepository';
-import { LocalStorageRepository } from './LocalStorageRepository';
-import { ApiV3CheerioRestIssueRepository } from './issue/ApiV3CheerioRestIssueRepository';
-import { ApiV3IssueRepository } from './issue/ApiV3IssueRepository';
-import { RestIssueRepository } from './issue/RestIssueRepository';
-import { GraphqlProjectItemRepository } from './issue/GraphqlProjectItemRepository';
-import { ProjectRepository } from '../../domain/usecases/adapter-interfaces/ProjectRepository';
-import { DateRepository } from '../../domain/usecases/adapter-interfaces/DateRepository';
-import { FieldOption, Project } from '../../domain/entities/Project';
-
-const projectId = 'PVT_project123';
-const storyFieldId = 'PVTSSF_story';
-const statusFieldId = 'PVTSSF_status';
-
-const cachedProject: Project = {
-  id: projectId,
-  url: 'https://github.com/users/owner/projects/49',
-  databaseId: 1,
-  name: 'A project',
-  status: {
-    name: 'Status',
-    fieldId: statusFieldId,
-    statuses: [{ id: 'st1', name: 'Todo', color: 'GRAY', description: '' }],
-  },
-  nextActionDate: null,
-  nextActionHour: null,
-  story: {
-    name: 'story',
-    fieldId: storyFieldId,
-    databaseId: 2,
-    stories: [
-      { id: 'story1', name: 'First Story', color: 'BLUE', description: '' },
-    ],
-    workflowManagementStory: { id: 'workflow1', name: 'Workflow Story' },
-  },
-  remainingEstimationMinutes: null,
-  dependedIssueUrlSeparatedByComma: null,
-  completionDate50PercentConfidence: null,
-  agent: null,
-};
-
-const savedStoryOptions: FieldOption[] = [
-  { id: 'story1', name: 'First Story', color: 'BLUE', description: '' },
-  {
-    id: 'newStoryId1',
-    name: 'New Feature Request',
-    color: 'RED',
-    description: '',
-  },
-];
-
-const buildSharedCache = (): Pick<
-  LocalStorageCacheRepository,
-  'getLatest' | 'set' | 'getSingle' | 'setSingle'
-> => {
-  const store = new Map<string, unknown>();
-  return {
-    getLatest: async () => null,
-    set: async () => undefined,
-    getSingle: async (key: string) => store.get(key) ?? null,
-    setSingle: async (key: string, value: unknown) => {
-      store.set(key, value);
-    },
-  };
-};
-
-const buildIssueRepository = (
-  cache: Pick<LocalStorageCacheRepository, 'getSingle' | 'setSingle'>,
-  localStorageRepository: LocalStorageRepository,
-): ApiV3CheerioRestIssueRepository =>
-  new ApiV3CheerioRestIssueRepository(
-    mock<ApiV3IssueRepository>(),
-    mock<RestIssueRepository>(),
-    mock<GraphqlProjectItemRepository>(),
-    cache,
-    mock<ProjectRepository>(),
-    mock<DateRepository>(),
-    localStorageRepository,
-    'dummy',
-  );
-
-const seedCache = async (
-  cache: Pick<LocalStorageCacheRepository, 'getSingle' | 'setSingle'>,
-): Promise<void> => {
-  await new ProjectIssuesCacheRepository(cache).write(projectId, {
-    lastFetchedAt: '2026-01-01T00:00:00.000Z',
-    lastFullFetchAt: '2026-01-01T00:00:00.000Z',
-    project: cachedProject,
-    issues: [],
-  });
-};
-
-describe('project issues cache shared by the project repository and the issue repository', () => {
-  const localStorageRepository = new LocalStorageRepository();
-
-  beforeEach(() => {
-    mockPost.mockReset();
-  });
-
-  it('serves the story options written by updateStoryList to the issue repository reader', async () => {
-    const cache = buildSharedCache();
-    await seedCache(cache);
-    mockPost.mockReturnValueOnce({
-      json: jest.fn().mockResolvedValue({
-        data: {
-          updateProjectV2Field: {
-            projectV2Field: { options: savedStoryOptions },
-          },
-        },
-      }),
-    });
-    const projectRepository = new GraphqlProjectRepository(
-      localStorageRepository,
-      'dummy',
-      cache,
-    );
-
-    await projectRepository.updateStoryList(cachedProject, [
-      { id: 'story1', name: 'First Story', color: 'BLUE', description: '' },
-      { id: null, name: 'New Feature Request', color: 'RED', description: '' },
-    ]);
-
-    const project = await buildIssueRepository(
-      cache,
-      localStorageRepository,
-    ).getCachedProject(projectId);
-
-    expect(project?.story?.stories).toEqual(savedStoryOptions);
-  });
-
-  it('leaves the cached issues and the fetch timestamps untouched when it writes the story options through', async () => {
-    const cache = buildSharedCache();
-    await seedCache(cache);
-    mockPost.mockReturnValueOnce({
-      json: jest.fn().mockResolvedValue({
-        data: {
-          updateProjectV2Field: {
-            projectV2Field: { options: savedStoryOptions },
-          },
-        },
-      }),
-    });
-    const projectRepository = new GraphqlProjectRepository(
-      localStorageRepository,
-      'dummy',
-      cache,
-    );
-
-    await projectRepository.updateStoryList(cachedProject, [
-      { id: null, name: 'New Feature Request', color: 'RED', description: '' },
-    ]);
-
-    const cached = await new ProjectIssuesCacheRepository(cache).read(
-      projectId,
-    );
-
-    expect(cached?.lastFetchedAt).toBe('2026-01-01T00:00:00.000Z');
-    expect(cached?.lastFullFetchAt).toBe('2026-01-01T00:00:00.000Z');
-    expect(cached?.issues).toEqual([]);
-    expect(cached?.project.status.statuses).toEqual(
-      cachedProject.status.statuses,
-    );
-  });
-
-  it('serves the status options written by updateStatusList to the issue repository reader', async () => {
-    const savedStatusOptions: FieldOption[] = [
-      { id: 'st1', name: 'Todo', color: 'GRAY', description: '' },
-      { id: 'st2', name: 'In Progress', color: 'BLUE', description: '' },
-    ];
-    const cache = buildSharedCache();
-    await seedCache(cache);
-    mockPost.mockReturnValueOnce({
-      json: jest.fn().mockResolvedValue({
-        data: {
-          updateProjectV2Field: {
-            projectV2Field: { options: savedStatusOptions },
-          },
-        },
-      }),
-    });
-    const projectRepository = new GraphqlProjectRepository(
-      localStorageRepository,
-      'dummy',
-      cache,
-    );
-
-    await projectRepository.updateStatusList(cachedProject, [
-      { id: 'st1', name: 'Todo', color: 'GRAY', description: '' },
-      { id: null, name: 'In Progress', color: 'BLUE', description: '' },
-    ]);
-
-    const project = await buildIssueRepository(
-      cache,
-      localStorageRepository,
-    ).getCachedProject(projectId);
-
-    expect(project?.status.statuses).toEqual(savedStatusOptions);
-  });
-});

package/src/adapter/repositories/ProjectIssuesCacheRepository.ts

@@ -1,141 +0,0 @@
-import { Issue } from '../../domain/entities/Issue';
-import { FieldOption, Project } from '../../domain/entities/Project';
-import { LocalStorageCacheRepository } from './LocalStorageCacheRepository';
-
-export type CachedProjectIssues = {
-  lastFetchedAt: string;
-  lastFullFetchAt: string;
-  project: Project;
-  issues: Issue[];
-};
-
-export const isProject = (value: unknown): value is Project => {
-  if (typeof value !== 'object' || value === null) return false;
-  if (!('id' in value) || typeof value.id !== 'string') return false;
-  if (!('url' in value) || typeof value.url !== 'string') return false;
-  if (!('databaseId' in value) || typeof value.databaseId !== 'number')
-    return false;
-  if (!('name' in value) || typeof value.name !== 'string') return false;
-  if (
-    !('status' in value) ||
-    typeof value.status !== 'object' ||
-    value.status === null
-  )
-    return false;
-  return true;
-};
-
-export const isIssueArray = (value: unknown): value is Issue[] =>
-  Array.isArray(value) &&
-  value.every(
-    (item: unknown) =>
-      typeof item === 'object' &&
-      item !== null &&
-      'nameWithOwner' in item &&
-      typeof item.nameWithOwner === 'string' &&
-      'number' in item &&
-      typeof item.number === 'number' &&
-      'title' in item &&
-      typeof item.title === 'string' &&
-      'url' in item &&
-      typeof item.url === 'string',
-  );
-
-export class ProjectIssuesCacheRepository {
-  constructor(
-    readonly localStorageCacheRepository: Pick<
-      LocalStorageCacheRepository,
-      'getSingle' | 'setSingle'
-    >,
-  ) {}
-
-  cacheKey = (projectId: Project['id']): string => `allIssues-${projectId}`;
-
-  readRaw = async (projectId: Project['id']): Promise<unknown> =>
-    this.localStorageCacheRepository.getSingle(this.cacheKey(projectId));
-
-  read = async (
-    projectId: Project['id'],
-  ): Promise<CachedProjectIssues | null> => {
-    const raw = await this.readRaw(projectId);
-    if (typeof raw !== 'object' || raw === null) {
-      return null;
-    }
-    if (
-      !('lastFetchedAt' in raw) ||
-      typeof raw.lastFetchedAt !== 'string' ||
-      !('lastFullFetchAt' in raw) ||
-      typeof raw.lastFullFetchAt !== 'string' ||
-      !('project' in raw) ||
-      !('issues' in raw)
-    ) {
-      return null;
-    }
-    if (!isProject(raw.project) || !isIssueArray(raw.issues)) {
-      return null;
-    }
-    return {
-      lastFetchedAt: raw.lastFetchedAt,
-      lastFullFetchAt: raw.lastFullFetchAt,
-      project: raw.project,
-      issues: raw.issues,
-    };
-  };
-
-  readProject = async (projectId: Project['id']): Promise<Project | null> => {
-    const raw = await this.readRaw(projectId);
-    if (typeof raw !== 'object' || raw === null || !('project' in raw)) {
-      return null;
-    }
-    if (!isProject(raw.project)) {
-      return null;
-    }
-    return raw.project;
-  };
-
-  write = async (
-    projectId: Project['id'],
-    cached: CachedProjectIssues,
-  ): Promise<void> => {
-    await this.localStorageCacheRepository.setSingle(
-      this.cacheKey(projectId),
-      cached,
-    );
-  };
-
-  updateFieldOptions = async (
-    projectId: Project['id'],
-    fieldId: string,
-    options: FieldOption[],
-  ): Promise<void> => {
-    const raw = await this.readRaw(projectId);
-    if (typeof raw !== 'object' || raw === null || !('project' in raw)) {
-      return;
-    }
-    if (!isProject(raw.project)) {
-      return;
-    }
-    const project = this.projectWithFieldOptions(raw.project, fieldId, options);
-    if (project === null) {
-      return;
-    }
-    await this.localStorageCacheRepository.setSingle(this.cacheKey(projectId), {
-      ...raw,
-      project,
-    });
-  };
-
-  private projectWithFieldOptions = (
-    project: Project,
-    fieldId: string,
-    options: FieldOption[],
-  ): Project | null => {
-    if (project.story !== null && project.story.fieldId === fieldId) {
-      return { ...project, story: { ...project.story, stories: options } };
-    }
-    if (project.status.fieldId === fieldId) {
-      return { ...project, status: { ...project.status, statuses: options } };
-    }
-    return null;
-  };
-}