claude-code-session-manager 0.87.0 → 0.88.1

package/scripts/scheduler-mcp-server.cjs

@@ -11,10 +11,7 @@
  *   feedback_list_projects()       -> GET  /admin/feedback/targets
  *   feedback_open_session({ ... }) -> POST /admin/feedback/open-session
  *
- *   project_home_get_contract({ cwd? })              -> GET  /admin/project-home/contract
- *   project_home_validate_summary({ cwd?, summary }) -> POST /admin/project-home/validate-summary
- *   project_home_render({ cwd?, summary, picks })    -> POST /admin/project-home/render
- *   project_home_status({ cwd? })                    -> GET  /admin/project-home/status
+ *   project_home_write({ cwd?, html }) -> POST /admin/project-home/write
  *
  * This is a separate process from the Electron app — it only ever reaches
  * it over the token-authed loopback HTTP API in admin-api.json, never by
@@ -24,7 +21,9 @@
 
 'use strict';
 
-const os = require('node:os');
+// Name this process first (pn-03): otherwise it shows as a bare `node`.
+const { PROC_NAMES, setProcessTitle } = require('../src/main/lib/smProcNames.cjs');
+setProcessTitle(PROC_NAMES.mcpServer);
 const path = require('node:path');
 const fsp = require('node:fs/promises');
 const { Server } = require('@modelcontextprotocol/sdk/server/index.js');
@@ -44,7 +43,7 @@ function descriptionFor(toolName) {
   return composeDescription(entry);
 }
 
-const TOKEN_PATH = path.join(os.homedir(), '.claude', 'session-manager', 'admin-api.json');
+const { adminTokenPath } = require('../src/main/lib/schedulerPaths.cjs');
 
 // Defined once so every failure path points to the same next call, verbatim
 // — never paste this sentence at each return site (PRD: session_manager_help).
@@ -71,7 +70,7 @@ function resolveCwdArg(args) {
 }
 
 async function readAdminConfig() {
-  const raw = await fsp.readFile(TOKEN_PATH, 'utf8');
+  const raw = await fsp.readFile(adminTokenPath(), 'utf8');
   const parsed = JSON.parse(raw);
   if (!parsed || typeof parsed.port !== 'number' || typeof parsed.token !== 'string') {
     throw new Error(NOT_RUNNING_ERROR);
@@ -140,6 +139,16 @@ const TOOLS = [
       required: ['slug'],
     },
   },
+  {
+    name: 'scheduler_pause',
+    description: descriptionFor('scheduler_pause'),
+    inputSchema: { type: 'object', properties: {} },
+  },
+  {
+    name: 'scheduler_resume',
+    description: descriptionFor('scheduler_resume'),
+    inputSchema: { type: 'object', properties: {} },
+  },
   {
     name: 'scheduler_list_jobs',
     description: descriptionFor('scheduler_list_jobs'),
@@ -347,48 +356,15 @@ const TOOLS = [
     },
   },
   {
-    name: 'project_home_get_contract',
-    description: descriptionFor('project_home_get_contract'),
+    name: 'project_home_write',
+    description: descriptionFor('project_home_write'),
     inputSchema: {
       type: 'object',
       properties: {
         cwd: { type: 'string', description: "Optional: absolute path to the target project. Defaults to the calling session's own project root (SM_PROJECT_ROOT or process.cwd()) when omitted." },
+        html: { type: 'string', description: 'The complete self-contained HTML document (<= 1MB; no <script src>, <link href=http*>, @import, or url(http*)).' },
       },
-    },
-  },
-  {
-    name: 'project_home_validate_summary',
-    description: descriptionFor('project_home_validate_summary'),
-    inputSchema: {
-      type: 'object',
-      properties: {
-        cwd: { type: 'string', description: "Optional: absolute path to the target project. Defaults to the calling session's own project root when omitted." },
-        summary: { type: 'object', description: 'ProjectPageSummary object matching the summarySchema from project_home_get_contract' },
-      },
-      required: ['summary'],
-    },
-  },
-  {
-    name: 'project_home_render',
-    description: descriptionFor('project_home_render'),
-    inputSchema: {
-      type: 'object',
-      properties: {
-        cwd: { type: 'string', description: "Optional: absolute path to the target project. Defaults to the calling session's own project root when omitted." },
-        summary: { type: 'object', description: 'ProjectPageSummary object matching the summarySchema from project_home_get_contract' },
-        picks: { type: 'object', description: 'ProjectPagePicks object (lensId -> slotId -> variantId) matching the picksSchema from project_home_get_contract' },
-      },
-      required: ['summary', 'picks'],
-    },
-  },
-  {
-    name: 'project_home_status',
-    description: descriptionFor('project_home_status'),
-    inputSchema: {
-      type: 'object',
-      properties: {
-        cwd: { type: 'string', description: "Optional: absolute path to the target project. Defaults to the calling session's own project root when omitted." },
-      },
+      required: ['html'],
     },
   },
   {
@@ -428,6 +404,10 @@ async function handleCallTool(request) {
       const result = await adminRequest('POST', '/admin/scheduler/reset-job', { slug, force, cwd });
       return { content: [{ type: 'text', text: JSON.stringify(result) }] };
     }
+    if (name === 'scheduler_pause' || name === 'scheduler_resume') {
+      const result = await adminRequest('POST', `/admin/scheduler/${name === 'scheduler_pause' ? 'pause' : 'resume'}`, {});
+      return { content: [{ type: 'text', text: JSON.stringify(result) }] };
+    }
     if (name === 'scheduler_list_jobs') {
       const result = await adminRequest('GET', '/admin/scheduler/jobs');
       return { content: [{ type: 'text', text: JSON.stringify(result) }] };
@@ -575,35 +555,12 @@ async function handleCallTool(request) {
       const result = await adminRequest('POST', '/admin/chat/send-prompt', { tabId, prompt });
       return { content: [{ type: 'text', text: JSON.stringify(result) }] };
     }
-    if (name === 'project_home_get_contract') {
-      const cwd = resolveCwdArg(args);
-      const qs = new URLSearchParams({ cwd });
-      const result = await adminRequest('GET', `/admin/project-home/contract?${qs.toString()}`);
-      return { content: [{ type: 'text', text: JSON.stringify(result) }] };
-    }
-    if (name === 'project_home_validate_summary') {
-      if (!args || typeof args.summary !== 'object' || args.summary === null) {
-        return errorResult('missing required argument: summary');
-      }
-      const cwd = resolveCwdArg(args);
-      const result = await adminRequest('POST', '/admin/project-home/validate-summary', { cwd, summary: args.summary });
-      return { content: [{ type: 'text', text: JSON.stringify(result) }] };
-    }
-    if (name === 'project_home_render') {
-      if (!args || typeof args.summary !== 'object' || args.summary === null) {
-        return errorResult('missing required argument: summary');
-      }
-      if (typeof args.picks !== 'object' || args.picks === null) {
-        return errorResult('missing required argument: picks');
+    if (name === 'project_home_write') {
+      if (!args || typeof args.html !== 'string' || args.html.length === 0) {
+        return errorResult('missing required argument: html');
       }
       const cwd = resolveCwdArg(args);
-      const result = await adminRequest('POST', '/admin/project-home/render', { cwd, summary: args.summary, picks: args.picks });
-      return { content: [{ type: 'text', text: JSON.stringify(result) }] };
-    }
-    if (name === 'project_home_status') {
-      const cwd = resolveCwdArg(args);
-      const qs = new URLSearchParams({ cwd });
-      const result = await adminRequest('GET', `/admin/project-home/status?${qs.toString()}`);
+      const result = await adminRequest('POST', '/admin/project-home/write', { cwd, html: args.html });
       return { content: [{ type: 'text', text: JSON.stringify(result) }] };
     }
     if (name === 'session_manager_help') {

package/src/main/__tests__/bilkoHost-integration.test.cjs

@@ -14,8 +14,8 @@ function makeProject() {
   config.addAllowedRoot(root);
   fs.mkdirSync(path.join(root, 'session-manager-operations', 'project-pages', 'output'), { recursive: true });
   fs.writeFileSync(
-    path.join(root, 'session-manager-operations', 'project-pages', 'output', 'marketing.html'),
-    '<html><title>Demo</title>Marketing</html>',
+    path.join(root, 'session-manager-operations', 'project-pages', 'home.html'),
+    '<html><title>Demo</title>Home</html>',
   );
   fs.writeFileSync(
     path.join(root, 'session-manager-operations', 'project-pages', 'output', 'feature.html'),
@@ -29,7 +29,7 @@ test('prepareBundle seeds a root document and writes dist/index.html + manifest.
   const root = makeProject();
   const result = await bilkoHost.prepareBundle({ cwd: root, slug: 'demo-app' });
   assert.equal(fs.existsSync(path.join(result.distPath, 'index.html')), true);
-  assert.equal(fs.readFileSync(path.join(result.distPath, 'index.html'), 'utf8'), '<html><title>Demo</title>Marketing</html>');
+  assert.equal(fs.readFileSync(path.join(result.distPath, 'index.html'), 'utf8'), '<html><title>Demo</title>Home</html>');
   assert.equal(result.manifest.slug, 'demo-app');
   assert.equal(result.manifest.bundle.fileCount, 1);
   fs.rmSync(root, { recursive: true, force: true });

package/src/main/__tests__/chat-cancel-terminal.test.cjs

@@ -13,14 +13,14 @@
  * path (via SM_CLAUDE_BIN pointing at a stub process) and asserts a terminal
  * event is broadcast on cancel.
  *
- * Run: timeout 120 node --test src/main/__tests__/chat-cancel-terminal.test.cjs
+ * Run: timeout 300 npx vitest run src/main/__tests__/chat-cancel-terminal.test.cjs
  */
 
 'use strict';
 
 delete process.env.SM_CHAT_CONCURRENCY;
 
-const { test } = require('node:test');
+import { test, afterAll } from 'vitest';
 const assert = require('node:assert/strict');
 const fs = require('node:fs');
 const os = require('node:os');
@@ -30,12 +30,9 @@ const path = require('node:path');
 // mimics a long-running headless run so cancel() has something to SIGTERM.
 // Shebang + chmod makes it directly spawnable as a single executable (spawn
 // does not split a path on spaces, so SM_CLAUDE_BIN must be one binary).
-const stubPath = path.join(os.tmpdir(), `sm-claude-stub-${process.pid}.cjs`);
-fs.writeFileSync(
-  stubPath,
-  `#!${process.execPath}\nsetInterval(() => {}, 1000);\n`,
-  { mode: 0o755 },
-);
+const stubPath = require('../../../tests/helpers/claudeStub.cjs').writeClaudeStub({
+  body: 'setInterval(() => {}, 1000);',
+});
 process.env.SM_CLAUDE_BIN = stubPath;
 
 const cr = require('../chatRunner.cjs');
@@ -114,7 +111,7 @@ function isTerminal(channel) {
   );
 }
 
-test.after(() => {
+afterAll(() => {
   try { fs.unlinkSync(stubPath); } catch { /* already gone */ }
   delete process.env.SM_CLAUDE_BIN;
 });

package/src/main/__tests__/chat-exit-close-race.test.cjs

@@ -21,14 +21,14 @@
  * arrives, THEN close fires) rather than relying on real OS/pipe timing,
  * which is not reliably reproducible from a real child process.
  *
- * Run: timeout 120 node --test src/main/__tests__/chat-exit-close-race.test.cjs
+ * Run: timeout 300 npx vitest run src/main/__tests__/chat-exit-close-race.test.cjs
  */
 
 'use strict';
 
 delete process.env.SM_CHAT_CONCURRENCY;
 
-const { test } = require('node:test');
+import { test, afterAll } from 'vitest';
 const assert = require('node:assert/strict');
 const EventEmitter = require('node:events');
 const cp = require('node:child_process');
@@ -134,7 +134,7 @@ function isTerminal(channel) {
   );
 }
 
-test.after(() => {
+afterAll(() => {
   cp.spawn = originalSpawn;
   delete process.env.SM_CHAT_CONCURRENCY;
 });

package/src/main/__tests__/chat-mcp-consent-notice.test.cjs

@@ -6,24 +6,23 @@
  * asserts a `chat:run:notice` event is broadcast — informational, not a
  * terminal event.
  *
- * Run: timeout 120 node --test src/main/__tests__/chat-mcp-consent-notice.test.cjs
+ * Run: timeout 300 npx vitest run src/main/__tests__/chat-mcp-consent-notice.test.cjs
  */
 
 'use strict';
 
 delete process.env.SM_CHAT_CONCURRENCY;
 
-const { test } = require('node:test');
+import { test } from 'vitest';
 const assert = require('node:assert/strict');
 const fs = require('node:fs');
 const os = require('node:os');
 const path = require('node:path');
+const claudeStub = require('../../../tests/helpers/claudeStub.cjs');
 
 function writeStub(lines) {
-  const stubPath = path.join(os.tmpdir(), `sm-claude-stub-${process.pid}-${Math.floor(Math.random() * 1e9)}.cjs`);
   const body = lines.map((l) => `process.stdout.write(${JSON.stringify(JSON.stringify(l))} + "\\n");`).join('\n');
-  fs.writeFileSync(stubPath, `#!${process.execPath}\n${body}\nprocess.exit(0);\n`, { mode: 0o755 });
-  return stubPath;
+  return claudeStub.writeClaudeStub({ body: `${body}\nprocess.exit(0);` });
 }
 
 function isTerminal(channel) {

package/src/main/__tests__/chat-queue.test.cjs

@@ -6,7 +6,7 @@
  * this cap/FIFO governs silent runs only — manual runs bypass it entirely
  * (see chatRunner.spec.ts's "manual runs are uncapped" tests).
  *
- * Run: timeout 120 node --test src/main/__tests__/chat-queue.test.cjs
+ * Run: timeout 300 npx vitest run src/main/__tests__/chat-queue.test.cjs
  */
 
 'use strict';
@@ -14,7 +14,7 @@
 // Force the default cap (2) regardless of the developer's shell env.
 delete process.env.SM_CHAT_CONCURRENCY;
 
-const { test } = require('node:test');
+import { test } from 'vitest';
 const assert = require('node:assert/strict');
 const cr = require('../chatRunner.cjs');
 

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

@@ -2,12 +2,12 @@
  * chat-stop-signal.test.cjs — unit tests for chatRunner.parseStopSignal, the
  * single source of truth for the terminal-chat stop-signal protocol (PRD 318).
  *
- * Run: timeout 120 node --test src/main/__tests__/chat-stop-signal.test.cjs
+ * Run: timeout 300 npx vitest run src/main/__tests__/chat-stop-signal.test.cjs
  */
 
 'use strict';
 
-const { test } = require('node:test');
+import { test } from 'vitest';
 const assert = require('node:assert/strict');
 const { parseStopSignal, STOP_SENTINEL } = require('../chatRunner.cjs');
 

package/src/main/__tests__/dep-orphan-archive-health.test.cjs

@@ -0,0 +1,77 @@
+/**
+ * PRD 1286 — archiving a row must never orphan its dependents, and a fully
+ * blocked queue must never read healthy.
+ *
+ * Option (b): the depends-gate (computeDepHistorySatisfaction) treats a dep
+ * whose PRD file was archived — including the manual-archive layout
+ * prds-archived/<ISO-ts>/<slug>.md — as satisfied. A dep with no row, no
+ * history and no archived file still HOLDS.
+ *
+ * Run: timeout 120 npx vitest run src/main/__tests__/dep-orphan-archive-health.test.cjs
+ */
+
+'use strict';
+
+import { test, expect } from 'vitest';
+import fs from 'node:fs';
+import os from 'node:os';
+import path from 'node:path';
+
+const tmpHome = fs.mkdtempSync(path.join(os.tmpdir(), 'sm-1286-home-'));
+process.env.HOME = tmpHome;
+
+const scheduler = require('../scheduler.cjs');
+const { pickNextBatch } = require('../lib/schedulerBatch.cjs');
+const { evaluateUnresolvableDepHealth } = require('../health.cjs');
+
+function projectWithDependent(dep) {
+  const cwd = fs.mkdtempSync(path.join(os.tmpdir(), 'sm-1286-proj-'));
+  const jobs = [{ slug: '902-next', status: 'pending', cwd, dependsOn: [dep] }];
+  return { cwd, jobs };
+}
+
+async function eligible(jobs) {
+  const satisfiedSlugsByCwd = await scheduler.computeDepHistorySatisfaction({ jobs });
+  const { batch: picked } = pickNextBatch(jobs, new Set(), 3, { satisfiedSlugsByCwd });
+  return { batch: picked.map((j) => j.slug ?? j.job?.slug ?? j), satisfiedSlugsByCwd };
+}
+
+test('a dep archived via the manual-archive layout (prds-archived/<ts>/) leaves its dependent eligible', async () => {
+  const { cwd, jobs } = projectWithDependent('fo-01-base'); // bare slug, as authored
+  const dir = path.join(cwd, 'session-manager-operations', 'scheduler', 'epics', 'e1', 'prds-archived', '2026-09-18T01-00-00-000Z');
+  fs.mkdirSync(dir, { recursive: true });
+  fs.writeFileSync(path.join(dir, '900-fo-01-base.md'), '# Goal\n');
+  const { batch } = await eligible(jobs);
+  expect(batch).toContain('902-next');
+});
+
+test('a dep that never ran (no row, history or archived file) keeps its dependent held', async () => {
+  const { jobs } = projectWithDependent('fo-01-base');
+  const { batch } = await eligible(jobs);
+  expect(batch).not.toContain('902-next');
+});
+
+test('health: every pending row held behind one unresolvable root is non-GREEN and names the root', async () => {
+  const { cwd } = projectWithDependent('fo-01-base');
+  const jobs = [
+    { slug: '902-a', status: 'pending', cwd, dependsOn: ['fo-01-base'] },
+    { slug: '903-b', status: 'pending', cwd, dependsOn: ['a'] }, // transitive, bare
+  ];
+  const satisfied = await scheduler.computeDepHistorySatisfaction({ jobs });
+  const res = evaluateUnresolvableDepHealth(jobs, satisfied);
+  expect(res.ok).toBe(false);
+  expect(res.projects[0].roots).toEqual(['fo-01-base']);
+  expect(res.message).toContain('fo-01-base');
+});
+
+test('health: a project with any dispatchable pending row, or a satisfied root, stays GREEN', async () => {
+  const { cwd } = projectWithDependent('x');
+  const mixed = [
+    { slug: '902-a', status: 'pending', cwd, dependsOn: ['fo-01-base'] },
+    { slug: '903-free', status: 'pending', cwd, dependsOn: [] },
+  ];
+  const satisfied = await scheduler.computeDepHistorySatisfaction({ jobs: mixed });
+  expect(evaluateUnresolvableDepHealth(mixed, satisfied).ok).toBe(true);
+  const satisfiedRoot = new Map([[cwd, new Set(['900-fo-01-base'])]]);
+  expect(evaluateUnresolvableDepHealth([mixed[0]], satisfiedRoot).ok).toBe(true);
+});

package/src/main/__tests__/dod-batchkey.test.cjs

@@ -1,12 +1,12 @@
 /**
  * dod-batchkey.test.cjs — unit tests for definitionOfDone.cjs helpers.
  *
- * Run: timeout 120 node --test src/main/__tests__/dod-batchkey.test.cjs
+ * Run: timeout 300 npx vitest run src/main/__tests__/dod-batchkey.test.cjs
  */
 
 'use strict';
 
-const { test } = require('node:test');
+import { test } from 'vitest';
 const assert = require('node:assert/strict');
 const os = require('node:os');
 const fs = require('node:fs');

package/src/main/__tests__/dod-drain-hook.test.cjs

@@ -1,7 +1,7 @@
 /**
  * dod-drain-hook.test.cjs — drives the DoD drain handler.
  *
- * Run: timeout 120 node --test src/main/__tests__/dod-drain-hook.test.cjs
+ * Run: timeout 300 npx vitest run src/main/__tests__/dod-drain-hook.test.cjs
  *
  * Scenarios:
  *   1. Drain with completed jobs → writes report once.
@@ -13,7 +13,7 @@
 
 'use strict';
 
-const { test, beforeEach, afterEach } = require('node:test');
+import { test, beforeEach, afterEach } from 'vitest';
 const assert = require('node:assert/strict');
 const os = require('node:os');
 const fs = require('node:fs');

package/src/main/__tests__/dod-report.test.cjs

@@ -1,14 +1,14 @@
 /**
  * dod-report.test.cjs — unit tests for flagRiskySurfaces + writeReport.
  *
- * Run: timeout 120 node --test src/main/__tests__/dod-report.test.cjs
+ * Run: timeout 300 npx vitest run src/main/__tests__/dod-report.test.cjs
  *
  * Fixtures: os.tmpdir() only — never writes into real runs/.
  */
 
 'use strict';
 
-const { test } = require('node:test');
+import { test } from 'vitest';
 const assert = require('node:assert/strict');
 const os = require('node:os');
 const fs = require('node:fs');

package/src/main/__tests__/dod-reverify.test.cjs

@@ -1,14 +1,14 @@
 /**
  * dod-reverify.test.cjs — unit tests for extractAcCommand / reverifyAc / reverifyBatch.
  *
- * Run: timeout 180 node --test src/main/__tests__/dod-reverify.test.cjs
+ * Run: timeout 300 npx vitest run src/main/__tests__/dod-reverify.test.cjs
  *
  * Fixtures: os.tmpdir() only — never touches the real prds dir or scheduler queue.
  */
 
 'use strict';
 
-const { test } = require('node:test');
+import { test } from 'vitest';
 const assert = require('node:assert/strict');
 const os = require('node:os');
 const fs = require('node:fs');

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

@@ -42,8 +42,8 @@ function writeIndex(cwd, index) {
 }
 
 function readAuditRecordsForCwd(auditLog, cwd) {
-  if (!fs.existsSync(auditLog.AUDIT_LOG_PATH)) return [];
-  return fs.readFileSync(auditLog.AUDIT_LOG_PATH, 'utf8')
+  if (!fs.existsSync(auditLog.auditLogPath())) return [];
+  return fs.readFileSync(auditLog.auditLogPath(), 'utf8')
     .split('\n')
     .filter(Boolean)
     .map((line) => {

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

@@ -6,10 +6,10 @@
  * Runs against a tmp HOME so no real files are touched.
  * Stubs summarize.cjs to avoid any network calls.
  *
- * Run: timeout 120 node --test src/main/__tests__/exchanges.test.cjs
+ * Run: timeout 300 npx vitest run src/main/__tests__/exchanges.test.cjs
  */
 
-const { test, describe, before, after } = require('node:test');
+import { test, describe, beforeAll as before, afterAll as after } from 'vitest';
 const assert = require('node:assert/strict');
 const fs = require('node:fs');
 const fsp = require('node:fs/promises');

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

@@ -3,12 +3,12 @@
  * brace-matching JSON extractor used by memoryAggregate.cjs and
  * chatRunner.cjs's parseStopSignal.
  *
- * Run: timeout 120 node --test src/main/__tests__/extractJson.test.cjs
+ * Run: timeout 300 npx vitest run src/main/__tests__/extractJson.test.cjs
  */
 
 'use strict';
 
-const { test } = require('node:test');
+import { test } from 'vitest';
 const assert = require('node:assert/strict');
 const { extractJson } = require('../lib/extractJson.cjs');
 

package/src/main/__tests__/files-reject-credentials.test.cjs

@@ -3,7 +3,7 @@
 // rejectCredentials — write-side denylist for files:* mutations (write/create/
 // rename/delete all call it). Pure path logic; no fs touched.
 
-const { test } = require('node:test');
+import { test } from 'vitest';
 const assert = require('node:assert/strict');
 const os = require('node:os');
 const path = require('node:path');