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

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

@@ -1,207 +0,0 @@
-import * as fs from 'fs';
-import * as os from 'os';
-import * as path from 'path';
-import {
-  DiskBlocks,
-  ProcHostMetricsRepository,
-  cpuUsedPercentFromSamples,
-  cycleMinutesFromFetchTimestamps,
-  parseCpuSample,
-  parseDiskUsedPercent,
-  parseLoadAverages,
-  parseMemoryUsedPercent,
-} from './ProcHostMetricsRepository';
-
-describe('parseMemoryUsedPercent', () => {
-  it('computes (total - available) / total rounded to a whole percent', () => {
-    expect(
-      parseMemoryUsedPercent(
-        'MemTotal: 1000 kB\nMemFree: 100 kB\nMemAvailable: 380 kB\n',
-      ),
-    ).toBe(62);
-  });
-
-  it('throws when MemTotal is not positive', () => {
-    expect(() =>
-      parseMemoryUsedPercent('MemTotal: 0 kB\nMemAvailable: 0 kB\n'),
-    ).toThrow();
-  });
-});
-
-describe('parseCpuSample', () => {
-  it('uses the aggregate cpu line with idle = idle + iowait and total = sum of all fields', () => {
-    expect(
-      parseCpuSample('cpu  100 0 100 700 100 0 0 0 0 0\ncpu0 1 2 3 4\n'),
-    ).toEqual({ total: 1000, idle: 800 });
-  });
-
-  it('throws when there is no aggregate cpu line', () => {
-    expect(() => parseCpuSample('cpu0 1 2 3 4\n')).toThrow();
-  });
-});
-
-describe('cpuUsedPercentFromSamples', () => {
-  it('computes (total_delta - idle_delta) / total_delta', () => {
-    expect(
-      cpuUsedPercentFromSamples(
-        { total: 1000, idle: 700 },
-        { total: 1100, idle: 770 },
-      ),
-    ).toBe(30);
-  });
-
-  it('throws when the total delta is not positive', () => {
-    expect(() =>
-      cpuUsedPercentFromSamples(
-        { total: 1000, idle: 700 },
-        { total: 1000, idle: 700 },
-      ),
-    ).toThrow();
-  });
-});
-
-describe('parseLoadAverages', () => {
-  it('parses the first three floats from /proc/loadavg', () => {
-    expect(parseLoadAverages('1.20 0.98 0.75 1/123 4567\n')).toEqual({
-      oneMinute: 1.2,
-      fiveMinute: 0.98,
-      fifteenMinute: 0.75,
-    });
-  });
-
-  it('throws when fewer than three numeric values are present', () => {
-    expect(() => parseLoadAverages('1.20 0.98\n')).toThrow();
-  });
-});
-
-describe('parseDiskUsedPercent', () => {
-  it('excludes root-reserved blocks from the denominator like df', () => {
-    expect(parseDiskUsedPercent(1100, 200, 100)).toBe(90);
-  });
-
-  it('matches df when there is no root reservation (bfree equals bavail)', () => {
-    expect(parseDiskUsedPercent(1000, 100, 100)).toBe(90);
-  });
-
-  it('throws when the disk total is not positive', () => {
-    expect(() => parseDiskUsedPercent(100, 200, 0)).toThrow();
-  });
-});
-
-describe('cycleMinutesFromFetchTimestamps', () => {
-  it('rounds the gap between the previous and current fetch to whole minutes', () => {
-    expect(
-      cycleMinutesFromFetchTimestamps(
-        '2026-07-07T00:00:00.000Z',
-        '2026-07-07T00:14:00.000Z',
-      ),
-    ).toBe(14);
-  });
-
-  it('rounds a 1620 second gap to 27 minutes', () => {
-    expect(
-      cycleMinutesFromFetchTimestamps(
-        '2026-07-07T00:00:00.000Z',
-        '2026-07-07T00:27:00.000Z',
-      ),
-    ).toBe(27);
-  });
-
-  it('returns null when either timestamp is missing', () => {
-    expect(
-      cycleMinutesFromFetchTimestamps(null, '2026-07-07T00:00:00.000Z'),
-    ).toBeNull();
-    expect(
-      cycleMinutesFromFetchTimestamps('2026-07-07T00:00:00.000Z', null),
-    ).toBeNull();
-    expect(cycleMinutesFromFetchTimestamps(null, null)).toBeNull();
-  });
-
-  it('returns null when a timestamp is not parseable', () => {
-    expect(
-      cycleMinutesFromFetchTimestamps('not-a-date', '2026-07-07T00:00:00.000Z'),
-    ).toBeNull();
-  });
-});
-
-describe('ProcHostMetricsRepository', () => {
-  let procDirectory: string;
-
-  beforeEach(() => {
-    procDirectory = fs.mkdtempSync(path.join(os.tmpdir(), 'fake-proc-host-'));
-  });
-
-  afterEach(() => {
-    fs.rmSync(procDirectory, { recursive: true, force: true });
-  });
-
-  it('reads memory used percent from /proc/meminfo', () => {
-    fs.writeFileSync(
-      path.join(procDirectory, 'meminfo'),
-      'MemTotal: 1000 kB\nMemAvailable: 380 kB\n',
-    );
-    const repository = new ProcHostMetricsRepository(procDirectory);
-    expect(repository.readMemoryUsedPercent()).toBe(62);
-  });
-
-  it('reads load averages from /proc/loadavg', () => {
-    fs.writeFileSync(
-      path.join(procDirectory, 'loadavg'),
-      '2.00 1.00 0.50 1/100 200\n',
-    );
-    const repository = new ProcHostMetricsRepository(procDirectory);
-    expect(repository.readLoadAverages()).toEqual({
-      oneMinute: 2.0,
-      fiveMinute: 1.0,
-      fifteenMinute: 0.5,
-    });
-  });
-
-  it('reads the df-style disk used percent from the root filesystem statfs', () => {
-    const diskBlocks: DiskBlocks = { blocks: 1100, bfree: 200, bavail: 100 };
-    const repository = new ProcHostMetricsRepository(
-      procDirectory,
-      async () => {},
-      () => diskBlocks,
-    );
-    expect(repository.readDiskUsedPercent()).toBe(90);
-  });
-
-  it('reads the df-style disk used percent for a given mountpoint', () => {
-    const blocksByMountpoint: Record<string, DiskBlocks> = {
-      '/': { blocks: 1100, bfree: 200, bavail: 100 },
-      '/mountpoint-a': { blocks: 1000, bfree: 700, bavail: 700 },
-    };
-    const requestedMountpoints: string[] = [];
-    const repository = new ProcHostMetricsRepository(
-      procDirectory,
-      async () => {},
-      (mountpoint) => {
-        requestedMountpoints.push(mountpoint);
-        const blocks = blocksByMountpoint[mountpoint];
-        if (!blocks) {
-          throw new Error(`unexpected mountpoint ${mountpoint}`);
-        }
-        return blocks;
-      },
-    );
-    expect(repository.readDiskUsedPercentForMountpoint('/mountpoint-a')).toBe(
-      30,
-    );
-    expect(repository.readDiskUsedPercentForMountpoint('/')).toBe(90);
-    expect(requestedMountpoints).toEqual(['/mountpoint-a', '/']);
-  });
-
-  it('samples /proc/stat twice and computes the busy percent', async () => {
-    const statPath = path.join(procDirectory, 'stat');
-    let sampleIndex = 0;
-    fs.writeFileSync(statPath, 'cpu  700 0 0 300 0 0 0 0 0 0\n');
-    const sleep = async (): Promise<void> => {
-      sampleIndex += 1;
-      fs.writeFileSync(statPath, 'cpu  770 0 0 330 0 0 0 0 0 0\n');
-    };
-    const repository = new ProcHostMetricsRepository(procDirectory, sleep);
-    expect(await repository.readCpuUsedPercent()).toBe(70);
-    expect(sampleIndex).toBe(1);
-  });
-});

package/src/adapter/repositories/ProcHostMetricsRepository.ts

@@ -1,181 +0,0 @@
-import * as fs from 'fs';
-import * as path from 'path';
-
-export type CpuSample = {
-  total: number;
-  idle: number;
-};
-
-export type LoadAverages = {
-  oneMinute: number;
-  fiveMinute: number;
-  fifteenMinute: number;
-};
-
-const DEFAULT_PROC_DIRECTORY = '/proc';
-const DEFAULT_ROOT_PATH = '/';
-const CPU_SAMPLE_INTERVAL_MS = 400;
-
-export type DiskBlocks = {
-  blocks: number;
-  bfree: number;
-  bavail: number;
-};
-
-export const parseMemoryUsedPercent = (meminfoText: string): number => {
-  const fields = new Map<string, number>();
-  for (const line of meminfoText.split('\n')) {
-    const separatorIndex = line.indexOf(':');
-    if (separatorIndex <= 0) {
-      continue;
-    }
-    const key = line.slice(0, separatorIndex).trim();
-    const valuePart = line
-      .slice(separatorIndex + 1)
-      .trim()
-      .split(/\s+/)[0];
-    const value = Number(valuePart);
-    if (Number.isFinite(value)) {
-      fields.set(key, value);
-    }
-  }
-  const total = fields.get('MemTotal');
-  const available = fields.get('MemAvailable');
-  if (total === undefined || available === undefined) {
-    throw new Error('MemTotal and MemAvailable are required');
-  }
-  if (total <= 0) {
-    throw new Error('MemTotal must be positive');
-  }
-  const used = total - available;
-  return Math.round((used / total) * 100);
-};
-
-export const parseCpuSample = (statText: string): CpuSample => {
-  for (const line of statText.split('\n')) {
-    if (line.startsWith('cpu ')) {
-      const values = line
-        .trim()
-        .split(/\s+/)
-        .slice(1)
-        .map((value) => Number(value));
-      if (values.some((value) => !Number.isFinite(value))) {
-        throw new Error('aggregate cpu line contains non-numeric values');
-      }
-      const idle = values[3] + (values.length > 4 ? values[4] : 0);
-      const total = values.reduce((sum, value) => sum + value, 0);
-      return { total, idle };
-    }
-  }
-  throw new Error('aggregate cpu line not found');
-};
-
-export const cpuUsedPercentFromSamples = (
-  first: CpuSample,
-  second: CpuSample,
-): number => {
-  const totalDelta = second.total - first.total;
-  const idleDelta = second.idle - first.idle;
-  if (totalDelta <= 0) {
-    throw new Error('total delta must be positive');
-  }
-  const busyDelta = totalDelta - idleDelta;
-  return Math.round((busyDelta / totalDelta) * 100);
-};
-
-export const parseDiskUsedPercent = (
-  blocks: number,
-  bfree: number,
-  bavail: number,
-): number => {
-  const total = blocks - bfree + bavail;
-  if (total <= 0) {
-    throw new Error('disk total must be positive');
-  }
-  const used = blocks - bfree;
-  return Math.round((used / total) * 100);
-};
-
-export const parseLoadAverages = (loadavgText: string): LoadAverages => {
-  const parts = loadavgText.trim().split(/\s+/);
-  const oneMinute = Number(parts[0]);
-  const fiveMinute = Number(parts[1]);
-  const fifteenMinute = Number(parts[2]);
-  if (
-    !Number.isFinite(oneMinute) ||
-    !Number.isFinite(fiveMinute) ||
-    !Number.isFinite(fifteenMinute)
-  ) {
-    throw new Error('loadavg must contain three numeric values');
-  }
-  return { oneMinute, fiveMinute, fifteenMinute };
-};
-
-export const cycleMinutesFromFetchTimestamps = (
-  previousFetchedAtIso: string | null,
-  currentFetchedAtIso: string | null,
-): number | null => {
-  if (previousFetchedAtIso === null || currentFetchedAtIso === null) {
-    return null;
-  }
-  const previousMs = new Date(previousFetchedAtIso).getTime();
-  const currentMs = new Date(currentFetchedAtIso).getTime();
-  if (!Number.isFinite(previousMs) || !Number.isFinite(currentMs)) {
-    return null;
-  }
-  return Math.round((currentMs - previousMs) / 60000);
-};
-
-export class ProcHostMetricsRepository {
-  constructor(
-    private readonly procDirectory: string = DEFAULT_PROC_DIRECTORY,
-    private readonly sleep: (milliseconds: number) => Promise<void> = (
-      milliseconds,
-    ) =>
-      new Promise((resolve) => {
-        setTimeout(resolve, milliseconds);
-      }),
-    private readonly readDiskBlocks: (rootPath: string) => DiskBlocks = (
-      rootPath,
-    ) => {
-      const stats = fs.statfsSync(rootPath);
-      return {
-        blocks: Number(stats.blocks),
-        bfree: Number(stats.bfree),
-        bavail: Number(stats.bavail),
-      };
-    },
-    private readonly rootPath: string = DEFAULT_ROOT_PATH,
-  ) {}
-
-  readMemoryUsedPercent = (): number =>
-    parseMemoryUsedPercent(
-      fs.readFileSync(path.join(this.procDirectory, 'meminfo'), 'utf8'),
-    );
-
-  readCpuUsedPercent = async (): Promise<number> => {
-    const first = parseCpuSample(
-      fs.readFileSync(path.join(this.procDirectory, 'stat'), 'utf8'),
-    );
-    await this.sleep(CPU_SAMPLE_INTERVAL_MS);
-    const second = parseCpuSample(
-      fs.readFileSync(path.join(this.procDirectory, 'stat'), 'utf8'),
-    );
-    return cpuUsedPercentFromSamples(first, second);
-  };
-
-  readLoadAverages = (): LoadAverages =>
-    parseLoadAverages(
-      fs.readFileSync(path.join(this.procDirectory, 'loadavg'), 'utf8'),
-    );
-
-  readDiskUsedPercent = (): number => {
-    const { blocks, bfree, bavail } = this.readDiskBlocks(this.rootPath);
-    return parseDiskUsedPercent(blocks, bfree, bavail);
-  };
-
-  readDiskUsedPercentForMountpoint = (mountpoint: string): number => {
-    const { blocks, bfree, bavail } = this.readDiskBlocks(mountpoint);
-    return parseDiskUsedPercent(blocks, bfree, bavail);
-  };
-}

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

@@ -1,231 +0,0 @@
-import * as fs from 'fs';
-import * as os from 'os';
-import * as path from 'path';
-import { ProcTakeOwnershipSpawnRepository } from './ProcTakeOwnershipSpawnRepository';
-
-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('ProcTakeOwnershipSpawnRepository', () => {
-  let procDirectory: string;
-
-  beforeEach(() => {
-    procDirectory = fs.mkdtempSync(path.join(os.tmpdir(), 'fake-proc-spawn-'));
-  });
-
-  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 the token and log path of a Take ownership spawn', () => {
-    writeProcess({
-      pid: 301,
-      cmdline: argv(
-        'bash',
-        '-c',
-        `claude-agent -p "Take ownership of ${issueUrl}" | tee /home/user/logs-aw/20260626_120000_abc.log`,
-      ),
-      environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-a' },
-    });
-
-    const repository = new ProcTakeOwnershipSpawnRepository(procDirectory);
-
-    expect(repository.listSpawns()).toEqual([
-      { token: 'token-a', logPath: '/logs-aw/20260626_120000_abc.log' },
-    ]);
-  });
-
-  it('ignores a process that is not a Take ownership spawn', () => {
-    writeProcess({
-      pid: 302,
-      cmdline: argv('claude', '--name', issueUrl),
-      environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-b' },
-    });
-
-    const repository = new ProcTakeOwnershipSpawnRepository(procDirectory);
-
-    expect(repository.listSpawns()).toEqual([]);
-  });
-
-  it('ignores a Take ownership process whose cmdline carries no log path', () => {
-    writeProcess({
-      pid: 303,
-      cmdline: argv('claude-agent', '-p', `Take ownership of ${issueUrl}`),
-      environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-c' },
-    });
-
-    const repository = new ProcTakeOwnershipSpawnRepository(procDirectory);
-
-    expect(repository.listSpawns()).toEqual([]);
-  });
-
-  it('ignores a Take ownership process without an oauth token', () => {
-    writeProcess({
-      pid: 304,
-      cmdline: argv(
-        'bash',
-        '-c',
-        `Take ownership of ${issueUrl} | tee /home/user/logs-aw/x.log`,
-      ),
-      environ: { ANTHROPIC_API_KEY: 'api-key' },
-    });
-
-    const repository = new ProcTakeOwnershipSpawnRepository(procDirectory);
-
-    expect(repository.listSpawns()).toEqual([]);
-  });
-
-  it('returns one entry per child process so the use case can dedupe by log path', () => {
-    const fullLogPath = '/home/user/logs-aw/20260626_120000_dup.log';
-    const capturedLogPath = '/logs-aw/20260626_120000_dup.log';
-    writeProcess({
-      pid: 305,
-      cmdline: argv(
-        'bash',
-        '-c',
-        `Take ownership of ${issueUrl} | tee ${fullLogPath}`,
-      ),
-      environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-d' },
-    });
-    writeProcess({
-      pid: 306,
-      cmdline: argv('tee', fullLogPath, 'Take ownership marker'),
-      environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-d' },
-    });
-
-    const repository = new ProcTakeOwnershipSpawnRepository(procDirectory);
-
-    expect(repository.listSpawns()).toEqual([
-      { token: 'token-d', logPath: capturedLogPath },
-      { token: 'token-d', logPath: capturedLogPath },
-    ]);
-  });
-
-  it('returns an empty list when the proc directory does not exist', () => {
-    const repository = new ProcTakeOwnershipSpawnRepository(
-      path.join(procDirectory, 'missing'),
-    );
-
-    expect(repository.listSpawns()).toEqual([]);
-  });
-
-  describe('listRunningIssueUrls', () => {
-    it('extracts issue URL from a bash-c style Take ownership spawn', () => {
-      writeProcess({
-        pid: 401,
-        cmdline: argv(
-          'bash',
-          '-c',
-          `claude-agent -p "Take ownership of ${issueUrl}" | tee /home/user/logs-aw/20260626_120000_abc.log`,
-        ),
-        environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-a' },
-      });
-
-      const repository = new ProcTakeOwnershipSpawnRepository(procDirectory);
-
-      expect(repository.listRunningIssueUrls()).toEqual([issueUrl]);
-    });
-
-    it('extracts issue URL from a null-separated claude-agent spawn', () => {
-      writeProcess({
-        pid: 402,
-        cmdline: argv(
-          'timeout',
-          '--kill-after=60s',
-          '3h',
-          'claude-agent',
-          '--agent',
-          'impl',
-          '-p',
-          `Take ownership of ${issueUrl} and finish it`,
-          '--model',
-          'claude-sonnet-4-6',
-        ),
-        environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-b' },
-      });
-
-      const repository = new ProcTakeOwnershipSpawnRepository(procDirectory);
-
-      expect(repository.listRunningIssueUrls()).toEqual([issueUrl]);
-    });
-
-    it('returns empty list when no Take ownership process is running', () => {
-      writeProcess({
-        pid: 403,
-        cmdline: argv('claude', '--name', issueUrl),
-        environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-c' },
-      });
-
-      const repository = new ProcTakeOwnershipSpawnRepository(procDirectory);
-
-      expect(repository.listRunningIssueUrls()).toEqual([]);
-    });
-
-    it('returns URLs from all matching processes', () => {
-      const issueUrl2 = 'https://github.com/HiromiShikata/example/issues/2';
-      writeProcess({
-        pid: 404,
-        cmdline: argv(
-          'timeout',
-          '--kill-after=60s',
-          '3h',
-          'claude-agent',
-          '--agent',
-          'impl',
-          '-p',
-          `Take ownership of ${issueUrl} and finish it`,
-          '--model',
-          'claude-sonnet-4-6',
-        ),
-        environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-d' },
-      });
-      writeProcess({
-        pid: 405,
-        cmdline: argv(
-          'timeout',
-          '--kill-after=60s',
-          '3h',
-          'claude-agent',
-          '--agent',
-          'impl',
-          '-p',
-          `Take ownership of ${issueUrl2} and finish it`,
-          '--model',
-          'claude-sonnet-4-6',
-        ),
-        environ: { CLAUDE_CODE_OAUTH_TOKEN: 'token-e' },
-      });
-
-      const repository = new ProcTakeOwnershipSpawnRepository(procDirectory);
-
-      expect(repository.listRunningIssueUrls()).toEqual([issueUrl, issueUrl2]);
-    });
-
-    it('returns empty list when the proc directory does not exist', () => {
-      const repository = new ProcTakeOwnershipSpawnRepository(
-        path.join(procDirectory, 'missing'),
-      );
-
-      expect(repository.listRunningIssueUrls()).toEqual([]);
-    });
-  });
-});