claude-code-session-manager 0.57.0 → 0.58.0

package/plugins/session-manager-dev/skills/propose-epic/SKILL.md

@@ -1,68 +0,0 @@
----
-name: propose-epic
-description: >-
-  File work you think should happen as a PROPOSED Epic — an Epic that does not
-  start until a human presses Approve. Replaces the retired /process-feedback
-  and /my-feedback intake pipeline: there is no feedback folder, no triage
-  pass, and no archiving convention, because the proposal IS the work item.
-  Use whenever you would previously have written a feedback file — for this
-  project or another one — or whenever the user says "/propose-epic",
-  "propose this", "file this for later", "suggest we do X", "send feedback to
-  X". Keywords: propose, proposal, feedback, enhancement request, file for
-  approval, cross-project request, backlog.
----
-
-# propose-epic
-
-**Role:** the one way an agent asks for work to happen that it is not doing right now.
-
-You do not implement, and you do not queue anything that runs. You file a **proposed
-Epic** and stop. A human approves it in the Epics workspace, at which point its objective
-becomes the first prompt of its own claude session — the same start path every hand-created
-Epic takes.
-
-## Why this replaced the feedback folder
-
-The old flow was: write a markdown file into `session-manager-operations/feedback/` →
-`/process-feedback` later reads it, evaluates it, dedupes it, queues PRDs via `/develop`,
-archives the file to `processed/`, and updates a README with lessons. Four moving parts and
-~400 lines of instructions existed to move an idea from "written down" to "being worked on".
-
-An Epic already is that thing. It has a goal, an intent tag, its own session, its own PRD
-directory, and a place in the UI. The only piece it was missing was "don't start yet" —
-that is the `proposed` status. So the intake pipeline collapses into one command.
-
-## How to file one
-
-```bash
-node "$SM_ROOT/scripts/propose-epic.cjs" <project-cwd> "<one-line title>" [feature|bug|discussion] <<'BODY'
-<the full objective — this is sent verbatim as the first prompt on approval>
-BODY
-```
-
-`$SM_ROOT` is session-manager's own package root. Resolve it once, in this order:
-1. You are working inside the session-manager repo → `.` (use `scripts/propose-epic.cjs`).
-2. Otherwise the installed package — this skill file lives at
-   `<SM_ROOT>/plugins/session-manager-dev/skills/propose-epic/SKILL.md`, so
-   `SM_ROOT` is four directories up from this file's own path.
-
-- **`<project-cwd>`** — the project the work belongs to. Cross-project requests are just a
-  proposal filed into *that* project's cwd; there is no separate outbound skill.
-- **title** — one line. It is what the Epics list shows. Keep it specific: `Scheduler rows
-  don't show their Epic`, not `Scheduler bug`.
-- **body** — write it as an instruction to the agent that will pick it up, not as a report
-  to a human. Include what you observed, where in the code it lives (file:line if you know
-  it), and what done looks like. It is the first prompt, so it should be enough to act on.
-- Re-proposing the same title into the same project **joins** the existing proposal and
-  enriches it rather than duplicating it. Say more in the body and re-run; that is the
-  update mechanism.
-
-Report back the Epic id and the title you filed. Do not queue PRDs, do not edit code, and
-do not ask whether you should file it — filing is the default action, and the approval gate
-is the user's control, not your prompt.
-
-## When NOT to propose
-
-- The user asked you to do the work now → do it (or `/develop` it), don't propose it.
-- It is a one-line fix you are already in the file for → just fix it.
-- Nothing actionable → say so plainly and file nothing.

package/scripts/propose-epic.cjs

@@ -1,56 +0,0 @@
-#!/usr/bin/env node
-/**
- * propose-epic.cjs — file an Epic that does NOT start.
- *
- * This is the replacement for the feedback-folder intake. Instead of writing
- * a markdown file into `session-manager-operations/feedback/` for a separate
- * skill to later parse, triage, dedupe and re-queue, an agent files a
- * `proposed` Epic: it appears in the Epics workspace with an Approve & start
- * gate, and nothing runs or is spent until a human presses it.
- *
- * One mechanism replaces the whole intake pipeline — the proposal IS the
- * work item, already in the place work is done, already carrying its own
- * session and PRD directory.
- *
- * Usage:
- *   node scripts/propose-epic.cjs <cwd> "<one-line title>" [feature|bug|discussion]
- *     # full body (the first prompt sent on approval) is read from stdin;
- *     # when stdin is empty the title is used as the body.
- *
- * Prints the Epic id on stdout. Reuse-by-title: re-proposing the same title
- * into the same project joins the existing proposal instead of duplicating it.
- */
-'use strict';
-
-const { ensureEpic } = require('../src/main/lib/epicMint.cjs');
-
-const [cwd, title, tag] = process.argv.slice(2);
-if (!cwd || !title) {
-  process.stderr.write('usage: propose-epic.cjs <cwd> "<title>" [feature|bug|discussion]  (body on stdin)\n');
-  process.exit(1);
-}
-
-function readStdin() {
-  return new Promise((resolve) => {
-    if (process.stdin.isTTY) { resolve(''); return; }
-    let buf = '';
-    process.stdin.setEncoding('utf8');
-    process.stdin.on('data', (d) => { buf += d; });
-    process.stdin.on('end', () => resolve(buf));
-    process.stdin.on('error', () => resolve(''));
-  });
-}
-
-(async () => {
-  const body = (await readStdin()).trim();
-  const { epicId, created } = await ensureEpic(cwd, {
-    goalText: title,
-    tag: tag || 'feature',
-    status: 'proposed',
-    reuseByGoal: true,
-    openingPrompt: body || title,
-    source: { producer: 'propose-epic' },
-  });
-  process.stderr.write(`${created ? 'proposed' : 'joined existing proposal'} epic ${epicId}\n`);
-  process.stdout.write(`${epicId}\n`);
-})();

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

@@ -1,280 +0,0 @@
-/**
- * rcaFeedbackHook.test.cjs — unit tests for src/main/lib/rcaFeedbackHook.cjs.
- *
- * Run: timeout 120 npx vitest run src/main/__tests__/rcaFeedbackHook.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 rcaFeedbackHook;
-
-// config.cjs and rcaFeedbackHook.cjs both bake os.homedir() into top-level
-// consts at first require (allowedRoots/WRITE_PREFIXES, SM_REPO_ROOT).
-// Node's require cache must be purged per test so each test's tmpHome takes
-// effect — vi.resetModules() only resets vitest's ESM graph, not require().
-const MODULES_TO_RELOAD = ['../lib/rcaFeedbackHook.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(), 'rca-feedback-hook-test-'));
-  process.env.HOME = tmpHome;
-  delete process.env.SM_RCA_DISABLE;
-  purgeRequireCache();
-  rcaFeedbackHook = require('../lib/rcaFeedbackHook.cjs');
-  // The session-manager-repo fallback destination must exist as a real dir.
-  fs.mkdirSync(path.join(tmpHome, 'Projects', 'session-manager', 'session-manager-operations', 'feedback'), { recursive: true });
-});
-
-afterEach(() => {
-  process.env.HOME = realHome;
-  delete process.env.SM_RCA_DISABLE;
-  fs.rmSync(tmpHome, { recursive: true, force: true });
-  purgeRequireCache();
-});
-
-// ─── helpers ──────────────────────────────────────────────────────────────────
-
-function makeProjectWithInbox(name = 'proj') {
-  const cwd = path.join(tmpHome, name);
-  fs.mkdirSync(path.join(cwd, 'session-manager-operations', 'feedback'), { recursive: true });
-  return cwd;
-}
-
-function makeProjectWithoutInbox(name = 'bare-proj') {
-  const cwd = path.join(tmpHome, name);
-  fs.mkdirSync(cwd, { recursive: true });
-  return cwd;
-}
-
-function writeRun(runDir, slug, { logLines = ['line1', 'line2'], exitCode = 1, durationMs = 1234 } = {}) {
-  fs.mkdirSync(runDir, { recursive: true });
-  fs.writeFileSync(path.join(runDir, `${slug}.log`), logLines.join('\n') + '\n');
-  fs.writeFileSync(path.join(runDir, `${slug}.meta.json`), JSON.stringify({ exitCode, durationMs }));
-}
-
-function writePrd(cwd, slug, { acBody = '- [ ] `timeout 10 echo ok` passes.' } = {}) {
-  const prdsDir = path.join(cwd, 'session-manager-operations', 'scheduler', 'prds');
-  fs.mkdirSync(prdsDir, { recursive: true });
-  const body = ['---', `title: ${slug}`, '---', '', '# Goal', '', 'Do the thing.', '', '# Acceptance criteria', '', acBody, '', '# Out of scope', '', '- N/A'].join('\n');
-  fs.writeFileSync(path.join(prdsDir, `${slug}.md`), body);
-}
-
-function baseJob(overrides = {}) {
-  return { slug: 'testslug', runId: '2026-07-24T00-00-00-000Z', exitCode: 1, error: 'something broke', ...overrides };
-}
-
-// ─── dedupe / new-runId ───────────────────────────────────────────────────────
-
-// ─── proposal filing (replaced the feedback-file inbox) ──────────────────────
-
-function readSessions(cwd) {
-  const p = path.join(cwd, 'session-manager-operations', 'prompt-sessions', 'active-index.json');
-  if (!fs.existsSync(p)) return {};
-  return JSON.parse(fs.readFileSync(p, 'utf8')).sessions ?? {};
-}
-
-test('fileRcaFeedback: files a PROPOSED Epic carrying the RCA as its opening prompt', async () => {
-  const cwd = makeProjectWithInbox();
-  const runDir = path.join(tmpHome, 'run1');
-  writeRun(runDir, 'testslug');
-  writePrd(cwd, 'testslug');
-
-  const res = await rcaFeedbackHook.fileRcaFeedback({ job: baseJob({ cwd }), runDir, verdict: 'transcript_errors' });
-  expect(res.filed).toBe(true);
-  expect(res.created).toBe(true);
-
-  const sessions = readSessions(cwd);
-  const epic = sessions[res.epicId];
-  // Nothing may auto-run: the whole point is a human gate.
-  expect(epic.status).toBe('proposed');
-  expect(epic.tag).toBe('bug');
-  expect(epic.goalText).toContain('testslug');
-  // Short title for the list, full RCA body for the first prompt.
-  expect(epic.goalText.split('\n').length).toBe(1);
-  expect(epic.openingPrompt).toContain('Root cause');
-  expect(epic.openingPrompt).toContain('prdSlug: testslug');
-});
-
-test('fileRcaFeedback: a re-trigger for the same failure joins the proposal instead of duplicating it', async () => {
-  const cwd = makeProjectWithInbox();
-  const runDir = path.join(tmpHome, 'run1');
-  writeRun(runDir, 'testslug');
-  writePrd(cwd, 'testslug');
-  const job = baseJob({ cwd });
-
-  const first = await rcaFeedbackHook.fileRcaFeedback({ job, runDir, verdict: 'transcript_errors' });
-  const second = await rcaFeedbackHook.fileRcaFeedback({ job, runDir, verdict: 'transcript_errors' });
-
-  expect(first.created).toBe(true);
-  expect(second.created).toBe(false);
-  expect(second.epicId).toBe(first.epicId);
-  expect(Object.keys(readSessions(cwd)).length).toBe(1);
-});
-
-test('fileRcaFeedback: a different verdict for the same slug is its own proposal', async () => {
-  const cwd = makeProjectWithInbox();
-  const runDir = path.join(tmpHome, 'run1');
-  writeRun(runDir, 'testslug');
-  writePrd(cwd, 'testslug');
-
-  const a = await rcaFeedbackHook.fileRcaFeedback({ job: baseJob({ cwd }), runDir, verdict: 'transcript_errors' });
-  const b = await rcaFeedbackHook.fileRcaFeedback({ job: baseJob({ cwd }), runDir, verdict: 'pass_no_commit' });
-
-  expect(a.epicId).not.toBe(b.epicId);
-  expect(Object.keys(readSessions(cwd)).length).toBe(2);
-});
-
-test('fileRcaFeedback: files into the job\'s own project, no feedback dir required', async () => {
-  const cwd = makeProjectWithoutInbox();
-  const runDir = path.join(tmpHome, 'run1');
-  writeRun(runDir, 'testslug');
-
-  const res = await rcaFeedbackHook.fileRcaFeedback({ job: baseJob({ cwd }), runDir, verdict: 'transcript_errors' });
-  expect(res.filed).toBe(true);
-  expect(readSessions(cwd)[res.epicId].status).toBe('proposed');
-});
-
-// ─── epic targeting (job.epicId → preferred join target) ───────────────────
-
-function writeActiveIndex(cwd, sessions) {
-  const dir = path.join(cwd, 'session-manager-operations', 'prompt-sessions');
-  fs.mkdirSync(dir, { recursive: true });
-  const events = {};
-  for (const id of Object.keys(sessions)) {
-    events[id] = [{ id: `${id}-e0`, promptSessionId: id, kind: 'prompt', causedByEventId: null, at: new Date(0).toISOString(), text: sessions[id].goalText }];
-  }
-  fs.writeFileSync(path.join(dir, 'active-index.json'), JSON.stringify({ sessions, events }));
-}
-
-test('fileRcaFeedback: joins the job\'s known origin Epic when it is still open, and tags source', async () => {
-  const cwd = makeProjectWithInbox();
-  const runDir = path.join(tmpHome, 'run1');
-  writeRun(runDir, 'testslug');
-  writePrd(cwd, 'testslug');
-
-  writeActiveIndex(cwd, {
-    'origin-epic-1': { id: 'origin-epic-1', cwd, goalText: 'Unrelated title text', status: 'proposed', createdAt: new Date(0).toISOString(), completedAt: null },
-  });
-
-  const res = await rcaFeedbackHook.fileRcaFeedback({
-    job: baseJob({ cwd, epicId: 'origin-epic-1' }), runDir, verdict: 'transcript_errors',
-  });
-
-  expect(res.filed).toBe(true);
-  expect(res.epicId).toBe('origin-epic-1');
-  expect(res.created).toBe(false);
-  expect(Object.keys(readSessions(cwd)).length).toBe(1);
-});
-
-test('fileRcaFeedback: does not join a completed origin Epic, falls through to mint', async () => {
-  const cwd = makeProjectWithInbox();
-  const runDir = path.join(tmpHome, 'run1');
-  writeRun(runDir, 'testslug');
-  writePrd(cwd, 'testslug');
-
-  writeActiveIndex(cwd, {
-    'origin-epic-1': { id: 'origin-epic-1', cwd, goalText: 'Unrelated title text', status: 'completed', createdAt: new Date(0).toISOString(), completedAt: new Date(0).toISOString() },
-  });
-
-  const res = await rcaFeedbackHook.fileRcaFeedback({
-    job: baseJob({ cwd, epicId: 'origin-epic-1' }), runDir, verdict: 'transcript_errors',
-  });
-
-  expect(res.filed).toBe(true);
-  expect(res.epicId).not.toBe('origin-epic-1');
-  expect(res.created).toBe(true);
-  const sessions = readSessions(cwd);
-  expect(sessions[res.epicId].source).toEqual({ producer: 'rca-hook', prdSlug: 'testslug', runId: baseJob().runId });
-});
-
-test('fileRcaFeedback: a nonexistent job.epicId falls through to mint without erroring', async () => {
-  const cwd = makeProjectWithInbox();
-  const runDir = path.join(tmpHome, 'run1');
-  writeRun(runDir, 'testslug');
-  writePrd(cwd, 'testslug');
-
-  const res = await rcaFeedbackHook.fileRcaFeedback({
-    job: baseJob({ cwd, epicId: 'does-not-exist' }), runDir, verdict: 'transcript_errors',
-  });
-
-  expect(res.filed).toBe(true);
-  expect(res.epicId).not.toBe('does-not-exist');
-  expect(res.created).toBe(true);
-});
-
-test('fileRcaFeedback: no job.epicId preserves current mint-or-reuseByGoal behavior', async () => {
-  const cwd = makeProjectWithInbox();
-  const runDir = path.join(tmpHome, 'run1');
-  writeRun(runDir, 'testslug');
-  writePrd(cwd, 'testslug');
-
-  const res = await rcaFeedbackHook.fileRcaFeedback({ job: baseJob({ cwd }), runDir, verdict: 'transcript_errors' });
-  expect(res.filed).toBe(true);
-  expect(res.created).toBe(true);
-  const sessions = readSessions(cwd);
-  expect(sessions[res.epicId].source).toEqual({ producer: 'rca-hook', prdSlug: 'testslug', runId: baseJob().runId });
-});
-
-test('fileRcaFeedback: rejects a slug containing path-traversal segments', async () => {
-  const cwd = makeProjectWithInbox();
-  const runDir = path.join(tmpHome, 'run1');
-  fs.mkdirSync(runDir, { recursive: true });
-
-  const res = await rcaFeedbackHook.fileRcaFeedback({ job: baseJob({ cwd, slug: '../../etc/passwd' }), runDir, verdict: 'transcript_errors' });
-  expect(res.filed).toBe(false);
-  expect(res.reason).toBe('unsafe-slug');
-});
-
-// ─── kill-switch ────────────────────────────────────────────────────────────────
-
-test('fileRcaFeedback: SM_RCA_DISABLE=1 skips without writing', async () => {
-  process.env.SM_RCA_DISABLE = '1';
-  const cwd = makeProjectWithInbox();
-  const runDir = path.join(tmpHome, 'run1');
-  writeRun(runDir, 'testslug');
-  writePrd(cwd, 'testslug');
-
-  const res = await rcaFeedbackHook.fileRcaFeedback({ job: baseJob({ cwd }), runDir, verdict: 'transcript_errors' });
-  expect(res.filed).toBe(false);
-  expect(res.reason).toBe('disabled');
-  expect(fs.readdirSync(path.join(cwd, 'session-manager-operations', 'feedback')).length).toBe(0);
-});
-
-// ─── investigation-text enrichment ──────────────────────────────────────────────
-
-test('fileRcaFeedback: investigationText enriches the pending proposal in place', async () => {
-  const cwd = makeProjectWithInbox();
-  const runDir = path.join(tmpHome, 'run1');
-  writeRun(runDir, 'testslug');
-  writePrd(cwd, 'testslug');
-  const job = baseJob({ cwd });
-
-  const first = await rcaFeedbackHook.fileRcaFeedback({ job, runDir, verdict: 'transcript_errors' });
-  expect(first.filed).toBe(true);
-  expect(readSessions(cwd)[first.epicId].openingPrompt).not.toContain('## Investigation analysis');
-
-  const second = await rcaFeedbackHook.fileRcaFeedback({
-    job, runDir, verdict: 'transcript_errors', investigationText: 'Root cause: forgot to bound the poll loop.',
-  });
-  expect(second.epicId).toBe(first.epicId);
-  expect(second.updated).toBe(true);
-
-  const enriched = readSessions(cwd)[second.epicId];
-  expect(enriched.openingPrompt).toContain('## Investigation analysis');
-  expect(enriched.openingPrompt).toContain('forgot to bound the poll loop');
-  expect(enriched.status).toBe('proposed');
-  expect(Object.keys(readSessions(cwd)).length).toBe(1);
-});