claude-code-session-manager 0.56.0 → 0.57.1

package/src/main/agentLibrary.cjs

@@ -42,13 +42,14 @@ const PERSONA_NAME_RE = /^[a-z][a-z0-9-]*$/;
  * 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 }) {
+function serializePersona({ name, description, tools, model, color, tags, title, 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(', ')}`);
+  if (title) lines.push(`title: ${title}`);
   lines.push('---', '');
   return lines.join('\n') + (body || '').trim() + '\n';
 }
@@ -66,6 +67,7 @@ async function savePersona({
   model,
   color,
   tags,
+  title,
   body,
   globalDir = path.join(os.homedir(), '.claude', 'agents'),
   validatePath = configMgr.validatePath,
@@ -75,7 +77,7 @@ async function savePersona({
     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 });
+  const text = serializePersona({ name, description, tools, model, color, tags, title, body });
   await writeTextAtomic(target, text);
   if (originalName && originalName !== name) {
     const oldReal = validatePath(path.join(globalDir, `${originalName}.md`));
@@ -201,6 +203,7 @@ async function listPersonas({
       model: fm.model || null,
       color: fm.color || null,
       tags: parseTools(fm.tags),
+      title: fm.title || null,
       path: real,
       body: body.trim(),
       overridingProjects,

package/src/main/config.cjs

@@ -136,14 +136,6 @@ function validateWrite(realAbs) {
       if (realAbs === browserSub || realAbs.startsWith(browserSub + path.sep)) {
         return;
       }
-      // Feedback inbox writes (scheduler RCA hook — rcaFeedbackHook.cjs — and
-      // any interactive feedback filing): narrowly scoped to
-      // session-manager-operations/feedback/, this repo's existing
-      // per-project intake convention.
-      const feedbackSub = path.join(realRoot, 'session-manager-operations', 'feedback');
-      if (realAbs === feedbackSub || realAbs.startsWith(feedbackSub + path.sep)) {
-        return;
-      }
       // PromptSession persistence (active-index.json + per-session archives,
       // promptSessions.ts) and the scheduler's own read-modify-write of that
       // same active index (promptSessionEvents.cjs, PRD 814) — narrowly

package/src/main/lib/epicMint.cjs

@@ -215,6 +215,12 @@ function withPathLock(lockPath, task) {
  */
 function ensureEpic(cwd, { goalText, tag, reuseByGoal = false, epicId: explicitEpicId, status = 'proposed', openingPrompt = null, mintIfMissing = true, source = null, forceNewEpic = false } = {}) {
   if (!cwd || typeof cwd !== 'string') throw new Error('ensureEpic: cwd is required');
+  // A relative cwd (e.g. a caller passing '.') would otherwise get stored
+  // verbatim on the minted Epic's `cwd` field — the renderer's EpicsWorkspace
+  // matches Epics to a tab by exact `s.cwd === effectiveCwd` string equality
+  // against the tab's absolute path, so a relative value silently produces an
+  // Epic that exists on disk but never renders in any project's queue.
+  cwd = path.resolve(cwd);
   return withPathLock(activeIndexPath(cwd), () => {
     const index = readActiveIndex(cwd);
 

package/src/main/lib/opsOwnership.cjs

@@ -48,8 +48,6 @@ const OWNERS = Object.freeze({
   'scheduler': 'scheduler',
   // Project Home owns the synthesized Brief (generate + hand-edit).
   'project-brief': 'project-home',
-  // The feedback intake folder — written by the RCA hook and feedback filing.
-  'feedback': 'feedback',
   // Browser tab scratch saves (DOM captures, screenshots, recorded flows).
   'browser': 'browser',
   // Structured per-tab error log lines (JSONL), tagged for tracing/analysis.

package/src/main/lib/queueHistory.cjs

@@ -21,10 +21,22 @@ const { projectHistoryPath, stateCwds } = require('./queueStore.cjs');
 // resolved via queueStore.cjs). Old rows stay readable from here; new appends
 // go to the owning project's shard (entries without a cwd still land here so
 // nothing is ever dropped).
-const HISTORY_PATH = path.join(os.homedir(), '.claude', 'session-manager', 'scheduled-plans', 'history.jsonl');
+//
+// Overridable via SM_HISTORY_PATH_OVERRIDE so queueHistory.test.cjs can point
+// this at an isolated tmpdir path instead of racing a live Electron
+// instance's scheduler.cjs, which writes this same machine-global file.
+const HISTORY_PATH = process.env.SM_HISTORY_PATH_OVERRIDE
+  || path.join(os.homedir(), '.claude', 'session-manager', 'scheduled-plans', 'history.jsonl');
 
-/** Every history file currently in play: each project's shard + the legacy global. */
+/**
+ * Every history file currently in play: each project's shard + the legacy
+ * global. Under SM_HISTORY_PATH_OVERRIDE (queueHistory.test.cjs), skip the
+ * federated per-project shards entirely — they're real files under whatever
+ * projects exist on this machine's ~/.claude/projects, so including them
+ * would leak real production history rows into an otherwise-isolated test.
+ */
 function historyPaths() {
+  if (process.env.SM_HISTORY_PATH_OVERRIDE) return [HISTORY_PATH];
   const paths = [];
   for (const cwd of stateCwds()) {
     try { paths.push(projectHistoryPath(cwd)); } catch { /* unusable cwd */ }

package/src/main/scheduler.cjs

@@ -1138,11 +1138,26 @@ async function reconcile(state) {
 
   const next = [];
   const seen = new Set();
+  // Terminal (completed/failed) jobs whose .md vanished from disk BEFORE
+  // their row ever crossed HISTORY_RETENTION_MS in partitionJobs — i.e. the
+  // common case, since archiveCompletedPrd() (called right when a job
+  // finishes, scheduler.cjs's `newlyCompletedPrds` handling) moves the PRD
+  // file immediately, while partitionJobs' appendHistory only fires after 7
+  // days. Bug found 2026-08-02: the old code below dropped these rows on the
+  // very next reconcile() pass with NO history write at all — the file-gone
+  // check couldn't tell "already recorded to history.jsonl on a prior pass"
+  // apart from "never recorded anywhere", so a job that completed and had
+  // its file auto-archived within minutes lost its history entirely. Both
+  // Queue and History tabs then read as empty even for projects with real,
+  // successfully-completed work. Collected here; reconciled against
+  // historyTerminalBySlug() below and backfilled before being dropped.
+  const terminalDroppedNeedingHistoryCheck = [];
   for (const job of state.jobs) {
     const p = onDisk.get(job.slug);
     if (!p) {
       // A terminal job whose .md is gone was archived on purpose — dropping
-      // its row is the intended end of the auto-archive flow.
+      // its row is the intended end of the auto-archive flow, PROVIDED it's
+      // already durably recorded in history.jsonl (checked below).
       //
       // A PENDING or RUNNING job whose .md is merely not VISIBLE is a
       // different thing entirely, and dropping it destroys queued work: the
@@ -1163,6 +1178,8 @@ async function reconcile(state) {
         seen.add(job.slug);
         next.push({ ...job });
         console.warn(`[scheduler] reconcile: keeping ${job.status} job ${job.slug} — PRD source not visible in any candidate dir`);
+      } else if (job.status === 'completed' || job.status === 'failed') {
+        terminalDroppedNeedingHistoryCheck.push(job);
       }
       continue;
     }
@@ -1198,10 +1215,24 @@ async function reconcile(state) {
   for (const [slug] of onDisk) {
     if (!seen.has(slug)) unmatchedSlugs.push(slug);
   }
-  const historyBySlug = unmatchedSlugs.length > 0
+  const historyBySlug = (unmatchedSlugs.length > 0 || terminalDroppedNeedingHistoryCheck.length > 0)
     ? await queueHistory.historyTerminalBySlug()
     : new Map();
 
+  // Backfill: any terminal job dropped above whose slug isn't already in
+  // history.jsonl gets written now, before its row is gone for good. This is
+  // the fix for the bug described where the loop collects
+  // terminalDroppedNeedingHistoryCheck — without it, a job whose PRD file was
+  // auto-archived immediately on completion (the normal case) is lost from
+  // both jobs[] and history.jsonl in the same reconcile pass.
+  if (terminalDroppedNeedingHistoryCheck.length > 0) {
+    const missing = terminalDroppedNeedingHistoryCheck.filter((j) => !historyBySlug.has(j.slug));
+    if (missing.length > 0) {
+      await queueHistory.appendHistory(missing);
+      console.log(`[scheduler] reconcile: backfilled ${missing.length} completed/failed job(s) into history.jsonl (PRD file already archived, row about to drop)`);
+    }
+  }
+
   // Terminal-in-history slugs whose .md file is still on disk: fed into the
   // auto-archive selection pass below (as synthetic completed entries) so
   // their file can still be swept, without ever creating a live job row

package/src/preload/api.d.ts

@@ -133,7 +133,7 @@ export interface ImportRef {
 /** Declared owner ids for the single-writer law over a project's
  *  session-manager-operations/ root. Mirrors OWNERS in
  *  src/main/lib/opsOwnership.cjs — one writer per namespace; everyone reads. */
-export type OpsWriter = 'epics' | 'scheduler' | 'project-home' | 'feedback' | 'browser';
+export type OpsWriter = 'epics' | 'scheduler' | 'project-home' | 'browser';
 
 export interface WriteResult {
   ok: boolean;
@@ -267,6 +267,9 @@ export interface AgentPersona {
   color: string | null;
   /** Epic intent tags (tagLibrary.ts's TAG_LIBRARY) this persona is associated with. */
   tags: string[];
+  /** "<Department> — <Human title>" (e.g. "Engineering — Architect"), free text — drives
+   *  department grouping in the New Epic Agent picker. Optional/null when not set. */
+  title?: string | null;
   path: string;
   body: string;
   /** Project names (basename of cwd) whose currently-open tab overlays this agent. */
@@ -283,6 +286,7 @@ export interface AgentPersonaSaveInput {
   model: string;
   color: string;
   tags: string[];
+  title?: string;
   body: string;
 }