claude-code-session-manager 0.54.0 → 0.55.1

package/dist/index.html

@@ -7,10 +7,10 @@
     <link rel="preconnect" href="https://fonts.googleapis.com">
     <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
     <link href="https://fonts.googleapis.com/css2?family=Newsreader:ital,opsz,wght@0,6..72,400;0,6..72,500;0,6..72,600;0,6..72,700;1,6..72,400&family=Geist:wght@300;400;500;600;700&family=IBM+Plex+Mono:wght@400;500;600&display=swap" rel="stylesheet">
-    <script type="module" crossorigin src="./assets/index-BYOscSQK.js"></script>
+    <script type="module" crossorigin src="./assets/index-tW8C4h-7.js"></script>
     <link rel="modulepreload" crossorigin href="./assets/monaco-editor-BW5C4Iv1.js">
     <link rel="stylesheet" crossorigin href="./assets/monaco-editor-BTnBOi8r.css">
-    <link rel="stylesheet" crossorigin href="./assets/index-BEyUC3v9.css">
+    <link rel="stylesheet" crossorigin href="./assets/index-CMQ3X1Y6.css">
   </head>
   <body class="bg-bg text-fg font-sans antialiased">
     <div id="root"></div>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-code-session-manager",
-  "version": "0.54.0",
+  "version": "0.55.1",
   "description": "Local cockpit for the Claude Code CLI — multi-tab terminal, full config surface, scheduler, voice dictation, and live observability.",
   "type": "module",
   "main": "src/main/index.cjs",

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

@@ -12,7 +12,7 @@ const fs = require('node:fs');
 const fsp = require('node:fs/promises');
 const os = require('node:os');
 const path = require('node:path');
-const { listPersonas, openProjects, parseTools } = require('../agentLibrary.cjs');
+const { listPersonas, openProjects, parseTools, savePersona, deletePersona, removeOverride } = require('../agentLibrary.cjs');
 
 const tmpDirs = [];
 afterEach(async () => {
@@ -33,6 +33,15 @@ async function mkTmp(prefix) {
 // home directory config.cjs's validatePath would otherwise enforce.
 const identityValidatePath = (p) => p;
 
+// savePersona/deletePersona/removeOverride take an injectable writeTextAtomic
+// too, since the real config.cjs one enforces its own home-dir validatePath
+// regardless of what's injected for the `validatePath` param here.
+async function fakeWriteTextAtomic(abs, text) {
+  await fsp.mkdir(path.dirname(abs), { recursive: true });
+  await fsp.writeFile(abs, text, 'utf8');
+  return { ok: true, mtimeMs: Date.now() };
+}
+
 test('parseTools splits + trims a comma-separated tools frontmatter value', () => {
   expect(parseTools('Read, Grep, Glob, Bash')).toEqual(['Read', 'Grep', 'Glob', 'Bash']);
   expect(parseTools('')).toEqual([]);
@@ -74,6 +83,7 @@ test('listPersonas parses frontmatter and reports overridingProjects for open ta
       'name: builder',
       'description: Watch git history and drive the next publish.',
       'tools: Read, Grep, Glob, Bash',
+      'tags: feature, bug',
       '---',
       '',
       'You are the Builder agent.',
@@ -103,6 +113,8 @@ test('listPersonas parses frontmatter and reports overridingProjects for open ta
 
   const byName = Object.fromEntries(personas.map((p) => [p.name, p]));
   expect(byName.builder.description).toBe('Watch git history and drive the next publish.');
+  expect(byName.builder.tags).toEqual(['feature', 'bug']);
+  expect(byName.debugger.tags).toEqual([]);
   expect(byName.builder.tools).toEqual(['Read', 'Grep', 'Glob', 'Bash']);
   expect(byName.builder.body).toContain('You are the Builder agent.');
   expect(byName.builder.overridingProjects).toEqual([path.basename(projectWithOverlay)]);
@@ -131,3 +143,73 @@ test('listPersonas skips a project cwd that validatePath rejects, rather than th
   expect(personas).toHaveLength(1);
   expect(personas[0].overridingProjects).toEqual([]);
 });
+
+test('savePersona writes frontmatter + body, and rejects a non-slug name', async () => {
+  const globalDir = await mkTmp('sm-agent-library-save-');
+  await savePersona({
+    name: 'my-agent',
+    description: 'Does a thing.',
+    tools: ['Read', 'Grep'],
+    model: 'opus',
+    color: 'blue',
+    tags: ['feature', 'bug'],
+    body: 'You are my-agent.',
+    globalDir,
+    validatePath: identityValidatePath,
+    writeTextAtomic: fakeWriteTextAtomic,
+  });
+  const text = await fsp.readFile(path.join(globalDir, 'my-agent.md'), 'utf8');
+  expect(text).toContain('name: my-agent');
+  expect(text).toContain('description: Does a thing.');
+  expect(text).toContain('tools: Read, Grep');
+  expect(text).toContain('model: opus');
+  expect(text).toContain('color: blue');
+  expect(text).toContain('tags: feature, bug');
+  expect(text).toContain('You are my-agent.');
+
+  await expect(
+    savePersona({ name: 'Not Valid', body: '', globalDir, validatePath: identityValidatePath, writeTextAtomic: fakeWriteTextAtomic }),
+  ).rejects.toThrow(/lowercase, hyphenated/);
+});
+
+test('savePersona with originalName renames — writes the new file and removes the old one', async () => {
+  const globalDir = await mkTmp('sm-agent-library-rename-');
+  await fsp.writeFile(path.join(globalDir, 'old-name.md'), '---\nname: old-name\n---\nBody.\n');
+  await savePersona({
+    name: 'new-name',
+    originalName: 'old-name',
+    description: '',
+    tools: [],
+    model: 'inherit',
+    color: '',
+    body: 'Body.',
+    globalDir,
+    validatePath: identityValidatePath,
+    writeTextAtomic: fakeWriteTextAtomic,
+  });
+  expect(fs.existsSync(path.join(globalDir, 'new-name.md'))).toBe(true);
+  expect(fs.existsSync(path.join(globalDir, 'old-name.md'))).toBe(false);
+});
+
+test('deletePersona removes the file, and is a no-op when it is already gone', async () => {
+  const globalDir = await mkTmp('sm-agent-library-delete-');
+  await fsp.writeFile(path.join(globalDir, 'gone-soon.md'), '---\nname: gone-soon\n---\nBody.\n');
+  await deletePersona({ name: 'gone-soon', globalDir, validatePath: identityValidatePath });
+  expect(fs.existsSync(path.join(globalDir, 'gone-soon.md'))).toBe(false);
+  await expect(deletePersona({ name: 'gone-soon', globalDir, validatePath: identityValidatePath })).resolves.toEqual({ ok: true });
+});
+
+test('removeOverride deletes a project overlay resolved by project name, and rejects an unknown project', async () => {
+  const project = await mkTmp('sm-agent-library-override-');
+  await fsp.mkdir(path.join(project, '.claude', 'agents'), { recursive: true });
+  const overlayPath = path.join(project, '.claude', 'agents', 'builder.md');
+  await fsp.writeFile(overlayPath, 'overlay\n');
+  const loadSessions = async () => ({ tabs: [{ cwd: project }] });
+
+  await removeOverride({ name: 'builder', projectName: path.basename(project), loadSessions, validatePath: identityValidatePath });
+  expect(fs.existsSync(overlayPath)).toBe(false);
+
+  await expect(
+    removeOverride({ name: 'builder', projectName: 'not-open', loadSessions, validatePath: identityValidatePath }),
+  ).rejects.toThrow(/project not open/);
+});

package/src/main/agentLibrary.cjs

@@ -31,6 +31,101 @@ function parseTools(raw) {
   return raw.split(',').map((t) => t.trim()).filter(Boolean);
 }
 
+/** Filename-safe persona name: lowercase, hyphenated, matches the `.md` files on disk. */
+const PERSONA_NAME_RE = /^[a-z][a-z0-9-]*$/;
+
+/**
+ * Serializes a persona's frontmatter + body back to `.md` file text.
+ * `tags` is a Claude-Code-agnostic extension of the frontmatter (Claude Code
+ * itself never reads it) — the Epic intent tags (tagLibrary.ts's TAG_LIBRARY)
+ * this persona is associated with, so Agent Library and Tag Library can each
+ * assign/remove the relationship from either side. Stored the same
+ * comma-list way as `tools` for one parser to cover both.
+ */
+function serializePersona({ name, description, tools, model, color, tags, body }) {
+  const lines = ['---', `name: ${name}`];
+  if (description) lines.push(`description: ${description}`);
+  if (tools && tools.length) lines.push(`tools: ${tools.join(', ')}`);
+  if (model && model !== 'inherit') lines.push(`model: ${model}`);
+  if (color) lines.push(`color: ${color}`);
+  if (tags && tags.length) lines.push(`tags: ${tags.join(', ')}`);
+  lines.push('---', '');
+  return lines.join('\n') + (body || '').trim() + '\n';
+}
+
+/**
+ * Writes a global persona `.md` file — `~/.claude/agents/<name>.md`. Pass
+ * `originalName` when renaming an existing persona so the old file is
+ * removed once the new one is written (no-op if names match).
+ */
+async function savePersona({
+  name,
+  originalName,
+  description,
+  tools,
+  model,
+  color,
+  tags,
+  body,
+  globalDir = path.join(os.homedir(), '.claude', 'agents'),
+  validatePath = configMgr.validatePath,
+  writeTextAtomic = configMgr.writeTextAtomic,
+} = {}) {
+  if (!PERSONA_NAME_RE.test(name || '')) {
+    throw new Error('agent name must be lowercase, hyphenated (e.g. "my-agent")');
+  }
+  const target = validatePath(path.join(globalDir, `${name}.md`));
+  const text = serializePersona({ name, description, tools, model, color, tags, body });
+  await writeTextAtomic(target, text);
+  if (originalName && originalName !== name) {
+    const oldReal = validatePath(path.join(globalDir, `${originalName}.md`));
+    try {
+      await fsp.unlink(oldReal);
+    } catch (e) {
+      if (e.code !== 'ENOENT') throw e;
+    }
+  }
+  return { ok: true, path: target };
+}
+
+/** Deletes a global persona `.md` file. Idempotent — missing file is not an error. */
+async function deletePersona({
+  name,
+  globalDir = path.join(os.homedir(), '.claude', 'agents'),
+  validatePath = configMgr.validatePath,
+} = {}) {
+  const target = validatePath(path.join(globalDir, `${name}.md`));
+  try {
+    await fsp.unlink(target);
+  } catch (e) {
+    if (e.code !== 'ENOENT') throw e;
+  }
+  return { ok: true };
+}
+
+/**
+ * Drops a project's local override (`<cwd>/.claude/agents/<name>.md`) so the
+ * global definition applies there again. `projectName` is resolved against
+ * currently-open tabs (same source `listPersonas` uses for overridingProjects).
+ */
+async function removeOverride({
+  name,
+  projectName,
+  loadSessions = sessionsStore.load,
+  validatePath = configMgr.validatePath,
+} = {}) {
+  const projects = await openProjects({ loadSessions });
+  const project = projects.find((p) => p.name === projectName);
+  if (!project) throw new Error(`project not open: ${projectName}`);
+  const target = validatePath(path.join(project.cwd, '.claude', 'agents', `${name}.md`));
+  try {
+    await fsp.unlink(target);
+  } catch (e) {
+    if (e.code !== 'ENOENT') throw e;
+  }
+  return { ok: true };
+}
+
 /**
  * Currently-open project tabs, deduped by cwd, as `{ cwd, name }` — `name`
  * is the last path segment (matches the `projectNameFromCwd` convention used
@@ -103,6 +198,9 @@ async function listPersonas({
       name: fm.name || fallbackName,
       description: fm.description || null,
       tools: parseTools(fm.tools),
+      model: fm.model || null,
+      color: fm.color || null,
+      tags: parseTools(fm.tags),
       path: real,
       body: body.trim(),
       overridingProjects,
@@ -112,4 +210,12 @@ async function listPersonas({
   return personas;
 }
 
-module.exports = { listPersonas, openProjects, parseTools };
+module.exports = {
+  listPersonas,
+  openProjects,
+  parseTools,
+  savePersona,
+  deletePersona,
+  removeOverride,
+  PERSONA_NAME_RE,
+};

package/src/main/index.cjs

@@ -33,6 +33,7 @@ const chatRunner = require('./chatRunner.cjs');
 const promptSessionEvents = require('./promptSessionEvents.cjs');
 const activeIndexMerge = require('./lib/activeIndexMerge.cjs');
 const agentLibrary = require('./agentLibrary.cjs');
+const { sendIfAlive } = require('./lib/sendToRenderer.cjs');
 const { resolveBuildTarget } = require('./lib/buildTarget.cjs');
 const adminHttp = createAdminHttp();
 scheduler.registerAdminRoutes(adminHttp);
@@ -454,6 +455,32 @@ ipcMain.handle('mcp:status', () => probeMcpStatus());
 // personas plus, per currently-open project tab, whether that project
 // overlays the same agent name at `<cwd>/.claude/agents/<name>.md`. Read-only.
 ipcMain.handle('agents:list-personas', () => agentLibrary.listPersonas());
+// Agent Library CRUD — global personas only (`~/.claude/agents/*.md`); a
+// project's local overlay can only be removed here (removeOverride), never
+// created, since Agent Library is a Home-face, machine-wide surface with no
+// per-tab cwd context of its own. Each mutation broadcasts `agents:changed`
+// so Agent Library and Tag Library — two independently-mounted dockview
+// panels, each with its own component-local `listPersonas()` snapshot —
+// both refresh off the one file on disk instead of one panel going stale
+// after the other edits the agent<->tag relationship.
+function broadcastAgentsChanged() {
+  sendIfAlive(mainWindow, 'agents:changed', {});
+}
+ipcMain.handle('agents:save-persona', async (_e, payload) => {
+  const result = await agentLibrary.savePersona(payload);
+  broadcastAgentsChanged();
+  return result;
+});
+ipcMain.handle('agents:delete-persona', async (_e, payload) => {
+  const result = await agentLibrary.deletePersona(payload);
+  broadcastAgentsChanged();
+  return result;
+});
+ipcMain.handle('agents:remove-override', async (_e, payload) => {
+  const result = await agentLibrary.removeOverride(payload);
+  broadcastAgentsChanged();
+  return result;
+});
 
 ipcMain.handle('app:engage-rules-path', () => process.env.SESSION_MANAGER_ENGAGE_RULES || null);
 

package/src/preload/api.d.ts

@@ -263,12 +263,29 @@ export interface AgentPersona {
   name: string;
   description: string | null;
   tools: string[];
+  model: string | null;
+  color: string | null;
+  /** Epic intent tags (tagLibrary.ts's TAG_LIBRARY) this persona is associated with. */
+  tags: string[];
   path: string;
   body: string;
   /** Project names (basename of cwd) whose currently-open tab overlays this agent. */
   overridingProjects: string[];
 }
 
+/** Payload for `agents.savePersona` — creates or overwrites a global persona file. */
+export interface AgentPersonaSaveInput {
+  name: string;
+  /** Previous filename, when renaming an existing persona. Omit when creating. */
+  originalName?: string;
+  description: string;
+  tools: string[];
+  model: string;
+  color: string;
+  tags: string[];
+  body: string;
+}
+
 export interface VoiceHotkeyConfig {
   accelerator: string;
   mode: 'hold' | 'toggle';
@@ -1329,6 +1346,11 @@ export interface SessionManagerAPI {
   };
   agents: {
     listPersonas: () => Promise<AgentPersona[]>;
+    savePersona: (payload: AgentPersonaSaveInput) => Promise<{ ok: boolean; path: string }>;
+    deletePersona: (payload: { name: string }) => Promise<{ ok: boolean }>;
+    removeOverride: (payload: { name: string; projectName: string }) => Promise<{ ok: boolean }>;
+    /** Fires after any save/delete/removeOverride — subscribers should re-fetch listPersonas(). */
+    onChanged: (handler: () => void) => () => void;
   };
   logs: {
     write: (

package/src/preload/index.cjs

@@ -148,6 +148,16 @@ contextBridge.exposeInMainWorld('api', {
   },
   agents: {
     listPersonas: () => ipcRenderer.invoke('agents:list-personas'),
+    savePersona: (payload) => ipcRenderer.invoke('agents:save-persona', payload),
+    deletePersona: (payload) => ipcRenderer.invoke('agents:delete-persona', payload),
+    removeOverride: (payload) => ipcRenderer.invoke('agents:remove-override', payload),
+    // Fired after any save/delete/removeOverride so every mounted
+    // subscriber (Agent Library, Tag Library) re-fetches the same file.
+    onChanged: (handler) => {
+      const listener = () => handler();
+      ipcRenderer.on('agents:changed', listener);
+      return () => ipcRenderer.removeListener('agents:changed', listener);
+    },
   },
   logs: {
     // ctx is optional: { cwd, tabId, epicId, tags } — when present and