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

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

@@ -1,242 +0,0 @@
-import * as fs from 'fs';
-import * as os from 'os';
-import * as path from 'path';
-import { ProcClaudeHandoverSessionRepository } from './ProcClaudeHandoverSessionRepository';
-
-type FakeProcess = {
-  pid: number;
-  comm: string;
-  cmdline: string;
-  environ: Record<string, string>;
-  cgroup?: string;
-};
-
-const issueUrl = 'https://github.com/HiromiShikata/example/issues/1';
-const argv = (...parts: string[]): string => `${parts.join('\0')}\0`;
-
-describe('ProcClaudeHandoverSessionRepository', () => {
-  let procDirectory: string;
-
-  beforeEach(() => {
-    procDirectory = fs.mkdtempSync(
-      path.join(os.tmpdir(), 'fake-proc-handover-'),
-    );
-  });
-
-  afterEach(() => {
-    fs.rmSync(procDirectory, { recursive: true, force: true });
-  });
-
-  const writeProcess = (fakeProcess: FakeProcess): void => {
-    const processDirectory = path.join(procDirectory, String(fakeProcess.pid));
-    fs.mkdirSync(processDirectory, { recursive: true });
-    fs.writeFileSync(
-      path.join(processDirectory, 'comm'),
-      `${fakeProcess.comm}\n`,
-    );
-    fs.writeFileSync(
-      path.join(processDirectory, 'cmdline'),
-      fakeProcess.cmdline,
-    );
-    const environBuffer = Object.entries(fakeProcess.environ)
-      .map(([key, value]) => `${key}=${value}\0`)
-      .join('');
-    fs.writeFileSync(path.join(processDirectory, 'environ'), environBuffer);
-    if (fakeProcess.cgroup !== undefined) {
-      fs.writeFileSync(
-        path.join(processDirectory, 'cgroup'),
-        `0::${fakeProcess.cgroup}\n`,
-      );
-    }
-  };
-
-  it('classifies an issue-URL leader from a cl --name url launch', () => {
-    writeProcess({
-      pid: 201,
-      comm: 'claude',
-      cmdline: argv('claude', '--model', 'opus', '--name', issueUrl),
-      environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-a' },
-    });
-
-    const repository = new ProcClaudeHandoverSessionRepository(procDirectory);
-
-    expect(repository.listHandoverSessions()).toEqual([
-      {
-        kind: 'issueUrlLeader',
-        pid: 201,
-        token: 'token-a',
-        sessionName: issueUrl.replace(/[.:]/g, '_'),
-        name: issueUrl,
-        issueUrl,
-        runsUnderWorkspacePreparationScript: false,
-      },
-    ]);
-  });
-
-  it('classifies a bare-name resident leader from a cl --name bare launch', () => {
-    writeProcess({
-      pid: 202,
-      comm: 'claude',
-      cmdline: argv('claude', '--name', 'app'),
-      environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-b' },
-    });
-
-    const repository = new ProcClaudeHandoverSessionRepository(procDirectory);
-
-    expect(repository.listHandoverSessions()).toEqual([
-      {
-        kind: 'bareNameLeader',
-        pid: 202,
-        token: 'token-b',
-        sessionName: 'app',
-        name: 'app',
-        issueUrl: null,
-        runsUnderWorkspacePreparationScript: false,
-      },
-    ]);
-  });
-
-  it('classifies an impl subagent from a -p task string containing an issue URL', () => {
-    writeProcess({
-      pid: 203,
-      comm: 'claude',
-      cmdline: argv(
-        'claude',
-        '--verbose',
-        '-p',
-        `Take ownership of ${issueUrl} and finish it`,
-      ),
-      environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-c' },
-    });
-
-    const repository = new ProcClaudeHandoverSessionRepository(procDirectory);
-
-    expect(repository.listHandoverSessions()).toEqual([
-      {
-        kind: 'implSubagent',
-        pid: 203,
-        token: 'token-c',
-        sessionName: null,
-        name: null,
-        issueUrl,
-        runsUnderWorkspacePreparationScript: false,
-      },
-    ]);
-  });
-
-  it('normalizes dots and colons in a bare name to the tmux session name', () => {
-    writeProcess({
-      pid: 204,
-      comm: 'claude',
-      cmdline: argv('claude', '--name', 'tdpm-cli.v2'),
-      environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-d' },
-    });
-
-    const repository = new ProcClaudeHandoverSessionRepository(procDirectory);
-
-    expect(repository.listHandoverSessions()[0].sessionName).toBe(
-      'tdpm-cli_v2',
-    );
-  });
-
-  it('ignores a non-claude process even when it has a token and a name', () => {
-    writeProcess({
-      pid: 205,
-      comm: 'node',
-      cmdline: argv('node', '--name', 'app'),
-      environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-e' },
-    });
-
-    const repository = new ProcClaudeHandoverSessionRepository(procDirectory);
-
-    expect(repository.listHandoverSessions()).toEqual([]);
-  });
-
-  it('ignores a claude process without an OAuth token', () => {
-    writeProcess({
-      pid: 206,
-      comm: 'claude',
-      cmdline: argv('claude', '--name', 'app'),
-      environ: {},
-    });
-
-    const repository = new ProcClaudeHandoverSessionRepository(procDirectory);
-
-    expect(repository.listHandoverSessions()).toEqual([]);
-  });
-
-  it('ignores a claude process with neither a name nor an issue URL', () => {
-    writeProcess({
-      pid: 207,
-      comm: 'claude',
-      cmdline: argv('claude', '--model', 'opus'),
-      environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-f' },
-    });
-
-    const repository = new ProcClaudeHandoverSessionRepository(procDirectory);
-
-    expect(repository.listHandoverSessions()).toEqual([]);
-  });
-
-  it('accepts a claude-agent comm', () => {
-    writeProcess({
-      pid: 208,
-      comm: 'claude-agent',
-      cmdline: argv('claude-agent', '--name', 'secretary'),
-      environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-g' },
-    });
-
-    const repository = new ProcClaudeHandoverSessionRepository(procDirectory);
-
-    expect(repository.listHandoverSessions()[0]).toMatchObject({
-      kind: 'bareNameLeader',
-      name: 'secretary',
-    });
-  });
-
-  it('marks a session whose cgroup scope is an aw scope as launched by the workspace preparation script', () => {
-    writeProcess({
-      pid: 209,
-      comm: 'claude',
-      cmdline: argv(
-        'claude',
-        '-p',
-        `Take ownership of ${issueUrl} and finish it`,
-      ),
-      environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-h' },
-      cgroup:
-        '/user.slice/user-1000.slice/user@1000.service/aiagents.slice/aw-HiromiShikata-example-1-4005530.scope',
-    });
-
-    const repository = new ProcClaudeHandoverSessionRepository(procDirectory);
-
-    expect(
-      repository.listHandoverSessions()[0].runsUnderWorkspacePreparationScript,
-    ).toBe(true);
-  });
-
-  it('does not mark a leader session whose cgroup scope is a cl scope as launched by the workspace preparation script', () => {
-    writeProcess({
-      pid: 210,
-      comm: 'claude',
-      cmdline: argv('claude', '--name', 'uminopm'),
-      environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-i' },
-      cgroup:
-        '/user.slice/user-1000.slice/user@1000.service/aiagents.slice/cl-uminopm.scope',
-    });
-
-    const repository = new ProcClaudeHandoverSessionRepository(procDirectory);
-
-    expect(
-      repository.listHandoverSessions()[0].runsUnderWorkspacePreparationScript,
-    ).toBe(false);
-  });
-
-  it('returns an empty list when the proc directory cannot be read', () => {
-    const repository = new ProcClaudeHandoverSessionRepository(
-      path.join(procDirectory, 'does-not-exist'),
-    );
-
-    expect(repository.listHandoverSessions()).toEqual([]);
-  });
-});

package/src/adapter/repositories/ProcClaudeHandoverSessionRepository.ts

@@ -1,261 +0,0 @@
-import * as fs from 'fs';
-import * as path from 'path';
-import {
-  ClaudeHandoverSession,
-  ClaudeHandoverSessionKind,
-} from '../../domain/entities/ClaudeHandoverSession';
-import { ClaudeHandoverSessionRepository } from '../../domain/usecases/adapter-interfaces/ClaudeHandoverSessionRepository';
-
-const DEFAULT_PROC_DIRECTORY = '/proc';
-const OAUTH_TOKEN_ENVIRON_KEY = 'CLAUDE_CODE_OAUTH_TOKEN';
-const NAME_ARGUMENT = '--name';
-const NAME_ARGUMENT_PREFIX = '--name=';
-const PRINT_ARGUMENTS = ['-p', '--print'];
-const PRINT_ARGUMENT_PREFIXES = ['-p=', '--print='];
-const CLAUDE_COMM_PATTERN = /^claude(-agent)?$/;
-const WORKSPACE_PREPARATION_SCOPE_PATTERN = /\/aw-[^/]*\.scope(\/|$)/;
-const ISSUE_OR_PR_URL_PATTERN =
-  /https:\/\/github\.com\/[^\s\0]+?\/(?:issues|pull)\/\d+/;
-
-const normalizeName = (value: string): string => value.replace(/[.:]/g, '_');
-
-const isIssueUrl = (value: string): boolean =>
-  value.startsWith('http://') || value.startsWith('https://');
-
-const extractIssueUrlFromText = (text: string): string | null => {
-  const match = text.match(ISSUE_OR_PR_URL_PATTERN);
-  return match ? match[0] : null;
-};
-
-const parseName = (commandArguments: string[]): string | null => {
-  for (let index = 0; index < commandArguments.length; index += 1) {
-    const argument = commandArguments[index];
-    if (argument === NAME_ARGUMENT && index + 1 < commandArguments.length) {
-      const value = commandArguments[index + 1];
-      return value.length > 0 ? value : null;
-    }
-    if (argument.startsWith(NAME_ARGUMENT_PREFIX)) {
-      const value = argument.slice(NAME_ARGUMENT_PREFIX.length);
-      return value.length > 0 ? value : null;
-    }
-  }
-  return null;
-};
-
-const parseIssueUrlFromPrint = (commandArguments: string[]): string | null => {
-  for (let index = 0; index < commandArguments.length; index += 1) {
-    const argument = commandArguments[index];
-    if (
-      PRINT_ARGUMENTS.includes(argument) &&
-      index + 1 < commandArguments.length
-    ) {
-      const url = extractIssueUrlFromText(commandArguments[index + 1]);
-      if (url !== null) {
-        return url;
-      }
-    }
-    for (const prefix of PRINT_ARGUMENT_PREFIXES) {
-      if (argument.startsWith(prefix)) {
-        const url = extractIssueUrlFromText(argument.slice(prefix.length));
-        if (url !== null) {
-          return url;
-        }
-      }
-    }
-  }
-  return extractIssueUrlFromText(commandArguments.join(' '));
-};
-
-const classifySession = (
-  pid: number,
-  commandArguments: string[],
-  token: string | null,
-  runsUnderWorkspacePreparationScript: boolean,
-): ClaudeHandoverSession | null => {
-  if (token === null || token.length === 0) {
-    return null;
-  }
-  const name = parseName(commandArguments);
-  if (name !== null) {
-    const sessionName = normalizeName(name);
-    if (isIssueUrl(name)) {
-      const kind: ClaudeHandoverSessionKind = 'issueUrlLeader';
-      return {
-        kind,
-        pid,
-        token,
-        sessionName,
-        name,
-        issueUrl: extractIssueUrlFromText(name),
-        runsUnderWorkspacePreparationScript,
-      };
-    }
-    const kind: ClaudeHandoverSessionKind = 'bareNameLeader';
-    return {
-      kind,
-      pid,
-      token,
-      sessionName,
-      name,
-      issueUrl: null,
-      runsUnderWorkspacePreparationScript,
-    };
-  }
-  const issueUrl = parseIssueUrlFromPrint(commandArguments);
-  if (issueUrl === null) {
-    return null;
-  }
-  return {
-    kind: 'implSubagent',
-    pid,
-    token,
-    sessionName: null,
-    name: null,
-    issueUrl,
-    runsUnderWorkspacePreparationScript,
-  };
-};
-
-export class ProcClaudeHandoverSessionRepository implements ClaudeHandoverSessionRepository {
-  constructor(
-    private readonly procDirectory: string = DEFAULT_PROC_DIRECTORY,
-  ) {}
-
-  listHandoverSessions = (): ClaudeHandoverSession[] => {
-    const sessions: ClaudeHandoverSession[] = [];
-    for (const processIdDirectory of this.listProcessIdDirectories()) {
-      const session = this.readHandoverSession(processIdDirectory);
-      if (session !== null) {
-        sessions.push(session);
-      }
-    }
-    return sessions;
-  };
-
-  private listProcessIdDirectories = (): string[] => {
-    let entries: string[];
-    try {
-      entries = fs.readdirSync(this.procDirectory);
-    } catch {
-      return [];
-    }
-    return entries.filter((entry) => /^\d+$/.test(entry));
-  };
-
-  private readHandoverSession = (
-    processIdDirectory: string,
-  ): ClaudeHandoverSession | null => {
-    const comm = this.readComm(processIdDirectory);
-    if (comm === null || !CLAUDE_COMM_PATTERN.test(comm)) {
-      return null;
-    }
-    const commandArguments = this.readCommandArguments(processIdDirectory);
-    if (commandArguments === null) {
-      return null;
-    }
-    const environ = this.readEnviron(processIdDirectory);
-    if (environ === null) {
-      return null;
-    }
-    const token = environ.get(OAUTH_TOKEN_ENVIRON_KEY) ?? null;
-    return classifySession(
-      Number(processIdDirectory),
-      commandArguments,
-      token,
-      this.readRunsUnderWorkspacePreparationScript(processIdDirectory),
-    );
-  };
-
-  private readRunsUnderWorkspacePreparationScript = (
-    processIdDirectory: string,
-  ): boolean => {
-    const cgroupPath = path.join(
-      this.procDirectory,
-      processIdDirectory,
-      'cgroup',
-    );
-    let raw: string;
-    try {
-      raw = fs.readFileSync(cgroupPath, 'utf8');
-    } catch (error) {
-      console.error(
-        `ProcClaudeHandoverSessionRepository: could not read ${cgroupPath}: ${
-          error instanceof Error ? error.message : String(error)
-        }`,
-      );
-      return false;
-    }
-    for (const line of raw.split('\n')) {
-      const firstSeparatorIndex = line.indexOf(':');
-      if (firstSeparatorIndex < 0) {
-        continue;
-      }
-      const secondSeparatorIndex = line.indexOf(':', firstSeparatorIndex + 1);
-      if (secondSeparatorIndex < 0) {
-        continue;
-      }
-      const cgroupHierarchyPath = line.slice(secondSeparatorIndex + 1);
-      if (WORKSPACE_PREPARATION_SCOPE_PATTERN.test(cgroupHierarchyPath)) {
-        return true;
-      }
-    }
-    return false;
-  };
-
-  private readComm = (processIdDirectory: string): string | null => {
-    const commPath = path.join(this.procDirectory, processIdDirectory, 'comm');
-    try {
-      return fs.readFileSync(commPath, 'utf8').trim();
-    } catch {
-      return null;
-    }
-  };
-
-  private readCommandArguments = (
-    processIdDirectory: string,
-  ): string[] | null => {
-    const cmdlinePath = path.join(
-      this.procDirectory,
-      processIdDirectory,
-      'cmdline',
-    );
-    let raw: string;
-    try {
-      raw = fs.readFileSync(cmdlinePath, 'utf8');
-    } catch {
-      return null;
-    }
-    return raw.split('\0').filter((argument) => argument.length > 0);
-  };
-
-  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;
-      }
-      environ.set(
-        entry.slice(0, separatorIndex),
-        entry.slice(separatorIndex + 1),
-      );
-    }
-    return environ;
-  };
-}

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

@@ -1,184 +0,0 @@
-import * as fs from 'fs';
-import * as os from 'os';
-import * as path from 'path';
-import { ProcClaudeInteractiveSessionRepository } from './ProcClaudeInteractiveSessionRepository';
-
-type FakeProcess = {
-  pid: number;
-  cmdline: string;
-  environ: Record<string, string>;
-};
-
-const issueUrl = 'https://github.com/HiromiShikata/example/issues/1';
-
-const argv = (...parts: string[]): string => `${parts.join('\0')}\0`;
-
-describe('ProcClaudeInteractiveSessionRepository', () => {
-  let procDirectory: string;
-
-  beforeEach(() => {
-    procDirectory = fs.mkdtempSync(path.join(os.tmpdir(), 'fake-proc-int-'));
-  });
-
-  afterEach(() => {
-    fs.rmSync(procDirectory, { recursive: true, force: true });
-  });
-
-  const writeProcess = (fakeProcess: FakeProcess): void => {
-    const processDirectory = path.join(procDirectory, String(fakeProcess.pid));
-    fs.mkdirSync(processDirectory, { recursive: true });
-    fs.writeFileSync(
-      path.join(processDirectory, 'cmdline'),
-      fakeProcess.cmdline,
-    );
-    const environBuffer = Object.entries(fakeProcess.environ)
-      .map(([key, value]) => `${key}=${value}\0`)
-      .join('');
-    fs.writeFileSync(path.join(processDirectory, 'environ'), environBuffer);
-  };
-
-  it('reads token, session id and issue url from a cl-launched interactive process', () => {
-    writeProcess({
-      pid: 201,
-      cmdline: argv('claude', '--model', 'opus', '--name', issueUrl),
-      environ: {
-        CLAUDE_CODE_OAUTH_TOKEN: 'token-a',
-        CLAUDE_CODE_SESSION_ID: 'session-a',
-      },
-    });
-
-    const repository = new ProcClaudeInteractiveSessionRepository(
-      procDirectory,
-    );
-
-    expect(repository.listInteractiveSessions()).toEqual([
-      { token: 'token-a', sessionId: 'session-a', issueUrl },
-    ]);
-  });
-
-  it('ignores a process without a --name issue url argument', () => {
-    writeProcess({
-      pid: 202,
-      cmdline: argv('claude', '--model', 'opus'),
-      environ: {
-        CLAUDE_CODE_OAUTH_TOKEN: 'token-b',
-        CLAUDE_CODE_SESSION_ID: 'session-b',
-      },
-    });
-
-    const repository = new ProcClaudeInteractiveSessionRepository(
-      procDirectory,
-    );
-
-    expect(repository.listInteractiveSessions()).toEqual([]);
-  });
-
-  it('excludes a Take ownership aw spawn even when it carries the token', () => {
-    writeProcess({
-      pid: 203,
-      cmdline: argv(
-        'claude-agent',
-        '--agent',
-        'impl',
-        '-p',
-        `Take ownership of ${issueUrl} and finish it`,
-      ),
-      environ: {
-        CLAUDE_CODE_OAUTH_TOKEN: 'token-c',
-        CLAUDE_CODE_SESSION_ID: 'session-c',
-      },
-    });
-
-    const repository = new ProcClaudeInteractiveSessionRepository(
-      procDirectory,
-    );
-
-    expect(repository.listInteractiveSessions()).toEqual([]);
-  });
-
-  it('ignores a --name process without an oauth token', () => {
-    writeProcess({
-      pid: 204,
-      cmdline: argv('claude', '--name', issueUrl),
-      environ: {
-        CLAUDE_CODE_SESSION_ID: 'session-d',
-        ANTHROPIC_API_KEY: 'api-key',
-      },
-    });
-
-    const repository = new ProcClaudeInteractiveSessionRepository(
-      procDirectory,
-    );
-
-    expect(repository.listInteractiveSessions()).toEqual([]);
-  });
-
-  it('ignores a --name process without a session id', () => {
-    writeProcess({
-      pid: 205,
-      cmdline: argv('claude', '--name', issueUrl),
-      environ: {
-        CLAUDE_CODE_OAUTH_TOKEN: 'token-e',
-      },
-    });
-
-    const repository = new ProcClaudeInteractiveSessionRepository(
-      procDirectory,
-    );
-
-    expect(repository.listInteractiveSessions()).toEqual([]);
-  });
-
-  it('ignores a --name value that is not an http url', () => {
-    writeProcess({
-      pid: 206,
-      cmdline: argv('claude', '--name', 'just-a-label'),
-      environ: {
-        CLAUDE_CODE_OAUTH_TOKEN: 'token-f',
-        CLAUDE_CODE_SESSION_ID: 'session-f',
-      },
-    });
-
-    const repository = new ProcClaudeInteractiveSessionRepository(
-      procDirectory,
-    );
-
-    expect(repository.listInteractiveSessions()).toEqual([]);
-  });
-
-  it('returns one entry per child process so the use case can dedupe by session id', () => {
-    writeProcess({
-      pid: 207,
-      cmdline: argv('claude', '--name', issueUrl),
-      environ: {
-        CLAUDE_CODE_OAUTH_TOKEN: 'token-g',
-        CLAUDE_CODE_SESSION_ID: 'session-g',
-      },
-    });
-    writeProcess({
-      pid: 208,
-      cmdline: argv('claude', '--name', issueUrl),
-      environ: {
-        CLAUDE_CODE_OAUTH_TOKEN: 'token-g',
-        CLAUDE_CODE_SESSION_ID: 'session-g',
-      },
-    });
-
-    const repository = new ProcClaudeInteractiveSessionRepository(
-      procDirectory,
-    );
-
-    expect(repository.listInteractiveSessions()).toEqual([
-      { token: 'token-g', sessionId: 'session-g', issueUrl },
-      { token: 'token-g', sessionId: 'session-g', issueUrl },
-    ]);
-  });
-
-  it('returns an empty list when the proc directory does not exist', () => {
-    const repository = new ProcClaudeInteractiveSessionRepository(
-      path.join(procDirectory, 'missing'),
-    );
-
-    expect(repository.listInteractiveSessions()).toEqual([]);
-  });
-});