claude-code-session-manager 0.98.0 → 0.101.0

package/src/main/__tests__/seedStatus.test.cjs

@@ -1,100 +0,0 @@
-/**
- * seedStatus.test.cjs — unit tests for src/main/seedStatus.cjs.
- *
- * Run: timeout 120 npx vitest run src/main/__tests__/seedStatus.test.cjs
- */
-
-'use strict';
-
-import { test, expect, beforeEach, afterEach } from 'vitest';
-const fs = require('node:fs');
-const os = require('node:os');
-const path = require('node:path');
-
-let tmpHome;
-let realHome;
-let getSeedStatus;
-
-// config.cjs (via seedSchedulerMcp/seedAgentPersonas) and each seed*.cjs
-// module bake os.homedir() into top-level consts at first require — purge
-// all of them alongside seedStatus.cjs so each test's tmpHome takes effect.
-const MODULES_TO_RELOAD = [
-  '../seedStatus.cjs',
-  '../seedDevPlugin.cjs',
-  '../seedSchedulerMcp.cjs',
-  '../seedAgentPersonas.cjs',
-  '../config.cjs',
-];
-
-function purgeRequireCache() {
-  for (const m of MODULES_TO_RELOAD) {
-    try { delete require.cache[require.resolve(m)]; } catch { /* not loaded yet */ }
-  }
-}
-
-beforeEach(() => {
-  realHome = process.env.HOME;
-  tmpHome = fs.mkdtempSync(path.join(os.tmpdir(), 'seed-status-test-'));
-  process.env.HOME = tmpHome;
-  purgeRequireCache();
-  ({ getSeedStatus } = require('../seedStatus.cjs'));
-});
-
-afterEach(() => {
-  process.env.HOME = realHome;
-  fs.rmSync(tmpHome, { recursive: true, force: true });
-  purgeRequireCache();
-});
-
-const MARKER_NAMES = {
-  'dev-plugin': '.dev-plugin-seeded',
-  'scheduler-mcp': '.scheduler-mcp-seeded',
-  'agent-personas': '.agent-personas-seeded',
-};
-
-function markerPath(id) {
-  return path.join(tmpHome, '.claude', 'session-manager', MARKER_NAMES[id]);
-}
-
-function writeMarker(id, contents) {
-  const p = markerPath(id);
-  fs.mkdirSync(path.dirname(p), { recursive: true });
-  fs.writeFileSync(p, contents);
-}
-
-test('no marker files at all → every seeder reports pending with a fix string', () => {
-  const status = getSeedStatus();
-  for (const id of Object.keys(MARKER_NAMES)) {
-    expect(status[id].status).toBe('pending');
-    expect(typeof status[id].fix).toBe('string');
-    expect(status[id].fix.length).toBeGreaterThan(0);
-  }
-});
-
-for (const id of Object.keys(MARKER_NAMES)) {
-  test(`${id}: done:true marker → done`, () => {
-    writeMarker(id, JSON.stringify({ done: true, attempts: 1, ts: '2026-01-01T00:00:00.000Z' }));
-    expect(getSeedStatus()[id].status).toBe('done');
-  });
-
-  test(`${id}: done:false, attempts below max → pending`, () => {
-    writeMarker(id, JSON.stringify({ done: false, attempts: 1, ts: '2026-01-01T00:00:00.000Z' }));
-    expect(getSeedStatus()[id].status).toBe('pending');
-  });
-
-  test(`${id}: done:false, attempts >= max → exhausted`, () => {
-    writeMarker(id, JSON.stringify({ done: false, attempts: 3, ts: '2026-01-01T00:00:00.000Z' }));
-    expect(getSeedStatus()[id].status).toBe('exhausted');
-  });
-
-  test(`${id}: unparseable marker → pending, never throws`, () => {
-    writeMarker(id, 'not json at all {{{');
-    expect(() => getSeedStatus()).not.toThrow();
-    expect(getSeedStatus()[id].status).toBe('pending');
-  });
-
-  test(`${id}: missing marker file → pending`, () => {
-    // No file written for this id — directory may not even exist.
-    expect(getSeedStatus()[id].status).toBe('pending');
-  });
-}

package/src/main/__tests__/seedValidatorPersona.test.cjs

@@ -1,116 +0,0 @@
-/**
- * seedValidatorPersona.test.cjs — covers the `validator` persona's delivery
- * through seedAgentPersonas.cjs's existing "names not yet in `seeded`" path.
- *
- * Run: timeout 120 npx vitest run src/main/__tests__/seedValidatorPersona.test.cjs
- */
-
-'use strict';
-
-import { test, expect, beforeEach, afterEach } from 'vitest';
-const fs = require('node:fs');
-const os = require('node:os');
-const path = require('node:path');
-
-let tmpHome;
-let realHome;
-let seedAgentPersonas;
-
-// config.cjs bakes os.homedir() into top-level consts at first require
-// (allowedRoots/WRITE_PREFIXES, used by writeJsonSync) — must be purged
-// alongside seedAgentPersonas.cjs so each test's tmpHome takes effect.
-const MODULES_TO_RELOAD = ['../seedAgentPersonas.cjs', '../config.cjs'];
-
-function purgeRequireCache() {
-  for (const m of MODULES_TO_RELOAD) {
-    try { delete require.cache[require.resolve(m)]; } catch { /* not loaded yet */ }
-  }
-}
-
-beforeEach(() => {
-  realHome = process.env.HOME;
-  tmpHome = fs.mkdtempSync(path.join(os.tmpdir(), 'seed-validator-persona-test-'));
-  process.env.HOME = tmpHome;
-  delete process.env.SM_SEED_AGENT_PERSONAS_DISABLE;
-  purgeRequireCache();
-  ({ seedAgentPersonas } = require('../seedAgentPersonas.cjs'));
-});
-
-afterEach(() => {
-  process.env.HOME = realHome;
-  delete process.env.SM_SEED_AGENT_PERSONAS_DISABLE;
-  fs.rmSync(tmpHome, { recursive: true, force: true });
-  purgeRequireCache();
-});
-
-function agentsDir() {
-  return path.join(tmpHome, '.claude', 'agents');
-}
-
-function markerPath() {
-  return path.join(tmpHome, '.claude', 'session-manager', '.agent-personas-seeded');
-}
-
-function silentLogger() {
-  return { log: () => {}, warn: () => {} };
-}
-
-const ALL_PERSONAS = ['architect', 'dev-lead', 'project-home-builder', 'validator'];
-const seedSrc = (name) => path.join(__dirname, '..', '..', 'seed', 'agents', `${name}.md`);
-
-test('a fresh homedir gets all four persona files', async () => {
-  await seedAgentPersonas({ logger: silentLogger() });
-
-  for (const name of ALL_PERSONAS) {
-    const dest = path.join(agentsDir(), `${name}.md`);
-    expect(fs.existsSync(dest)).toBe(true);
-    expect(fs.readFileSync(dest, 'utf8')).toBe(fs.readFileSync(seedSrc(name), 'utf8'));
-  }
-
-  const marker = JSON.parse(fs.readFileSync(markerPath(), 'utf8'));
-  expect(marker.seeded.sort()).toEqual([...ALL_PERSONAS].sort());
-});
-
-test('a homedir whose marker already lists the first three receives validator.md, and the other three files are byte-identical before and after', async () => {
-  const mPath = markerPath();
-  fs.mkdirSync(path.dirname(mPath), { recursive: true });
-  fs.writeFileSync(
-    mPath,
-    JSON.stringify({ seeded: ['architect', 'dev-lead', 'project-home-builder'], attempts: 0, ts: '2026-01-01T00:00:00.000Z' })
-  );
-  fs.mkdirSync(agentsDir(), { recursive: true });
-  const preExisting = {};
-  for (const name of ['architect', 'dev-lead', 'project-home-builder']) {
-    const content = `existing ${name} content\n`;
-    fs.writeFileSync(path.join(agentsDir(), `${name}.md`), content);
-    preExisting[name] = content;
-  }
-
-  expect(fs.existsSync(path.join(agentsDir(), 'validator.md'))).toBe(false);
-
-  await seedAgentPersonas({ logger: silentLogger() });
-
-  expect(fs.readFileSync(path.join(agentsDir(), 'validator.md'), 'utf8')).toBe(fs.readFileSync(seedSrc('validator'), 'utf8'));
-
-  for (const name of ['architect', 'dev-lead', 'project-home-builder']) {
-    expect(fs.readFileSync(path.join(agentsDir(), `${name}.md`), 'utf8')).toBe(preExisting[name]);
-  }
-
-  const marker = JSON.parse(fs.readFileSync(mPath, 'utf8'));
-  expect(marker.seeded.sort()).toEqual([...ALL_PERSONAS].sort());
-});
-
-test('the seeded validator body is under AGENT_BODY_CHAR_CAP and contains both required sentinel lines', () => {
-  const { AGENT_BODY_CHAR_CAP } = require('../lib/agentModelResolve.cjs');
-  const { splitFrontmatter } = require('../lib/prdFrontmatter.cjs');
-  const raw = fs.readFileSync(seedSrc('validator'), 'utf8');
-  const { body } = splitFrontmatter(raw);
-  const trimmed = body.trim();
-
-  expect(trimmed.length).toBeLessThan(AGENT_BODY_CHAR_CAP);
-  expect(trimmed).toContain('VALIDATION: <slug> VERIFIED');
-  expect(trimmed).toContain('SCHEDULER_VERDICT: PASS');
-  expect(trimmed).not.toContain('Do not touch:');
-  expect(trimmed).not.toContain('Report explicitly:');
-  expect(trimmed.endsWith('report them.')).toBe(true);
-});

package/src/main/__tests__/stop-signal-anchor.test.cjs

@@ -1,75 +0,0 @@
-/**
- * stop-signal-anchor.test.cjs — drift guard between chatRunner's
- * STOP_SENTINEL/splitStopSignal and the renderer's duplicated copy.
- *
- * `src/renderer/lib/stopSignal.ts` cannot import this main-process `.cjs`
- * (CommonJS main / ESM renderer split), so it carries its own literal
- * `STOP_SENTINEL` + a re-implementation of `splitStopSignal`. This test
- * requires the REAL exported constant and asserts the renderer's copy is
- * byte-identical, and that both splitters agree on `body` for the same
- * composed sample — so a rework of either constant/function fails here
- * loudly instead of the two forms of a stop-signal reply silently drifting
- * back out of sync (which is exactly what caused the doubled-message bug
- * this module exists to fix).
- *
- * Run: timeout 120 npx vitest run src/main/__tests__/stop-signal-anchor.test.cjs
- */
-
-'use strict';
-
-const path = require('path');
-
-import { test, expect } from 'vitest';
-
-const { STOP_SENTINEL, splitStopSignal } = require('../chatRunner.cjs');
-
-async function loadRendererCopy() {
-  const mod = await import(
-    /* @vite-ignore */ path.resolve(__dirname, '..', '..', 'renderer', 'lib', 'stopSignal.ts')
-  );
-  return mod;
-}
-
-test('the renderer STOP_SENTINEL literal is byte-identical to chatRunner.cjs', async () => {
-  const rendererCopy = await loadRendererCopy();
-  expect(rendererCopy.STOP_SENTINEL).toBe(STOP_SENTINEL);
-});
-
-test('both splitters agree on body for a composed stop-signal reply', async () => {
-  const rendererCopy = await loadRendererCopy();
-  const body = 'Here is what I found and what I still need from you.';
-  const full = `${body}\n\n${STOP_SENTINEL}\n${JSON.stringify({ questions: ['Deploy to prod now?'] })}`;
-
-  const mainSplit = splitStopSignal(full);
-  const rendererSplit = rendererCopy.splitStopSignal(full);
-
-  expect(mainSplit).not.toBeNull();
-  expect(rendererSplit).not.toBeNull();
-  expect(rendererSplit.body).toBe(mainSplit.answerBody);
-  expect(rendererSplit.body).toBe(body);
-  expect(rendererSplit.questions ?? mainSplit.questions).toBeDefined();
-});
-
-test('both splitters return null for text with no sentinel', async () => {
-  const rendererCopy = await loadRendererCopy();
-  const plain = 'Just a normal reply, no protocol block.';
-  expect(splitStopSignal(plain)).toBeNull();
-  expect(rendererCopy.splitStopSignal(plain)).toBeNull();
-});
-
-test('both splitters refuse a quoted sentinel with no valid JSON tail', async () => {
-  const rendererCopy = await loadRendererCopy();
-  const quoting = `Why does chat use ${STOP_SENTINEL} internally? Not real JSON after it.`;
-  expect(splitStopSignal(quoting)).toBeNull();
-  expect(rendererCopy.splitStopSignal(quoting)).toBeNull();
-});
-
-test('both splitters anchor on the LAST sentinel occurrence', async () => {
-  const rendererCopy = await loadRendererCopy();
-  const body = 'first mention of the sentinel text, then the real one';
-  const full = `${STOP_SENTINEL} mentioned earlier in prose.\n\n${body}\n\n${STOP_SENTINEL}\n${JSON.stringify({ questions: ['q'] })}`;
-
-  const mainSplit = splitStopSignal(full);
-  const rendererSplit = rendererCopy.splitStopSignal(full);
-  expect(mainSplit.answerBody).toBe(rendererSplit.body);
-});