@xhulz/nina 0.52.0 → 0.53.0

package/src/commands/pills.mjs

@@ -336,43 +336,24 @@ export function validate(pill, text, roles, today) {
 }
 
 /**
- * `nina pills`.
+ * A project's pills, checked: what `nina pills` prints and the dashboard's pills tab shows — each pill's stage, status,
+ * count and trigger, and the problems and notes over the whole corpus. Null when the project has composed no agents.
  *
- * @param {string[]} argv - Command arguments; `--project <dir>` selects the target.
- * @param {{root: string}} [ctx] - CLI context; `root` is the NINA install directory.
- * @returns {Promise<number>} Process exit code.
+ * @param {string} target - The project.
+ * @param {{root?: string}} [ctx] - CLI context: where the layers are, to say where a recurring lesson belongs.
  */
-export async function pills(argv, ctx) {
-  const target = resolve(argv.includes('--project') ? argv[argv.indexOf('--project') + 1] : '.');
-  const quiet = argv.includes('--quiet');
+export async function pillsData(target, ctx) {
   const pillsDir = join(target, '.claude', 'pills');
   const agentsDir = join(target, '.claude', 'agents');
-
-  if (!existsSync(agentsDir)) {
-    console.error(`  no .claude/agents under ${target} — run \`nina compose\` first.\n`);
-    return 1;
-  }
+  if (!existsSync(agentsDir)) return null;
   const roles = (await readdir(agentsDir))
     .filter((f) => f.endsWith('.md'))
     .map((f) => f.replace(/\.md$/, ''))
     .sort();
-
   const today = new Date().toISOString().slice(0, 10);
-  if (argv.includes('--tidy')) {
-    const moved = await tidyRetired(target);
-    for (const m of moved) console.log(`  moved ${m} → ${RETIRED}/${m}`);
-    console.log(moved.length ? `\npills: ${moved.length} retired pill(s) moved where no agent opens them\n` : 'pills: no retired pill where agents read\n');
-    return 0;
-  }
   const files = existsSync(pillsDir) ? await pillFiles(pillsDir) : [];
 
-  // The directory itself is composed, so its presence says nothing. An empty corpus is the
-  // normal state of a new project and the suspicious state of an old one — say which, and
-  // leave the judgement to whoever knows how long the pipeline has been running.
-  if (files.length === 0) {
-    if (!quiet) console.log(`  no pill written yet under ${target} — either nothing has been learned the expensive way, or nothing was written down.\n`);
-    return 0;
-  }
+  if (files.length === 0) return { files: 0, active: 0, retired: 0, problems: [], notes: [], pills: [] };
   const problems = [];
   const notes = [];
   const active = [];
@@ -404,8 +385,21 @@ export async function pills(argv, ctx) {
   let resolved = 0;
   let cited = 0;
 
+  const each = [];
   for (const pill of files) {
     const result = validate(pill, await readFile(pill.path, 'utf8'), roles, today);
+    const f = result.fields ?? {};
+    each.push({
+      label: result.label,
+      status: f.status ?? null,
+      roles: list(f.applies_to),
+      occurrences: Number(f.occurrences ?? 1) || 1,
+      date: f.date ?? null,
+      last_seen: f.last_seen ?? null,
+      severity: f.severity ?? null,
+      trigger: f.trigger ?? null,
+      problems: result.problems.length,
+    });
     problems.push(...result.problems);
     notes.push(...result.notes);
     if (pill.dir && pill.dir !== 'shared' && pill.dir !== RETIRED) taught.add(pill.dir);
@@ -478,14 +472,47 @@ export async function pills(argv, ctx) {
   const dated = active.map((p) => p.date).filter(Boolean).sort();
   if (dated.length > 0) notes.push(`oldest active pill is ${dated[0]}, newest ${dated[dated.length - 1]}`);
 
+  return { files: files.length, active: active.length, retired, problems, notes, pills: each };
+}
+
+/**
+ * `nina pills`.
+ *
+ * @param {string[]} argv - Command arguments; `--project <dir>` selects the target.
+ * @param {{root: string}} [ctx] - CLI context; `root` is the NINA install directory.
+ * @returns {Promise<number>} Process exit code.
+ */
+export async function pills(argv, ctx) {
+  const target = resolve(argv.includes('--project') ? argv[argv.indexOf('--project') + 1] : '.');
+  const quiet = argv.includes('--quiet');
+  const agentsDir = join(target, '.claude', 'agents');
+
+  if (!existsSync(agentsDir)) {
+    console.error(`  no .claude/agents under ${target} — run \`nina compose\` first.\n`);
+    return 1;
+  }
+  if (argv.includes('--tidy')) {
+    const moved = await tidyRetired(target);
+    for (const m of moved) console.log(`  moved ${m} → ${RETIRED}/${m}`);
+    console.log(moved.length ? `\npills: ${moved.length} retired pill(s) moved where no agent opens them\n` : 'pills: no retired pill where agents read\n');
+    return 0;
+  }
+  const { files, active, retired, problems, notes } = await pillsData(target, ctx);
+  // The directory itself is composed, so its presence says nothing. An empty corpus is the
+  // normal state of a new project and the suspicious state of an old one — say which, and
+  // leave the judgement to whoever knows how long the pipeline has been running.
+  if (files === 0) {
+    if (!quiet) console.log(`  no pill written yet under ${target} — either nothing has been learned the expensive way, or nothing was written down.\n`);
+    return 0;
+  }
   if (!(quiet && problems.length === 0)) {
-    console.log(`  ${files.length} pill(s): ${active.length} active, ${retired} retired\n`);
+    console.log(`  ${files} pill(s): ${active} active, ${retired} retired\n`);
     for (const p of problems) console.log(`  ✗ ${p}`);
     for (const n of notes) console.log(`  · ${n}`);
     if (problems.length === 0 && notes.length === 0) console.log('  ✓ every pill is well formed, placed where it is read, and carries evidence');
     console.log(
       problems.length === 0
-        ? `\npills: all ${files.length} well formed${notes.length > 0 ? ` (${notes.length} note(s))` : ''}\n`
+        ? `\npills: all ${files} well formed${notes.length > 0 ? ` (${notes.length} note(s))` : ''}\n`
         : `\npills: ${problems.length} problem(s)\n`,
     );
   }

package/src/commands/pipeline.mjs

@@ -165,6 +165,54 @@ export function historyOf(records, since) {
   return byRole;
 }
 
+/**
+ * A composed pipeline, as data: what `nina pipeline` draws and the dashboard's pipeline tab shows — the line and where
+ * it ends, the gates, the stages that run as several agents, who sends work back to whom and how many rounds per
+ * issue, and each stage's model, effort, skills and what it ran since `since`; the project's own agents; and which
+ * chain each task shape takes. Null when the project has no composed graph.
+ *
+ * @param {string} dir - The project.
+ * @param {{since?: string}} [options] - The first day of the history each stage is measured over.
+ */
+export function pipelineData(dir, { since } = {}) {
+  const graphPath = join(dir, '.claude', 'graph.md');
+  if (!existsSync(graphPath)) return null;
+  const graphText = readFileSync(graphPath, 'utf8');
+  const graph = parseGraph(graphText);
+  const words = stageWords(graphText);
+  const shape = shapeOf(graph);
+  const claude = join(dir, 'CLAUDE.md');
+  const profile = readProfile(dir).profile ?? {};
+  const agentsDir = join(dir, '.claude', 'agents');
+  const specs = new Map();
+  for (const file of existsSync(agentsDir) ? readdirSync(agentsDir).filter((f) => f.endsWith('.md')).sort() : []) {
+    specs.set(file.replace(/\.md$/, ''), readFileSync(join(agentsDir, file), 'utf8'));
+  }
+  const from = since || localDay(new Date(Date.now() - HISTORY_DAYS * 86_400_000).toISOString());
+  const records = projectRecords(dir);
+  const history = records ? historyOf(records, from) : null;
+  return {
+    project: basename(dir),
+    core: profile.core ?? null,
+    surfaces: (profile.surfaces ?? []).slice().sort(),
+    graph,
+    line: shape.line,
+    ends: shape.ends,
+    gates: shape.gates,
+    many: [...(graph.many ?? new Map())].map(([stage, { when }]) => ({ stage, when })),
+    back: shape.back,
+    since: from,
+    measured: history !== null,
+    stages: [...graph.stages].map((stage) => {
+      const spec = specs.get(stage) ?? '';
+      const front = frontmatter(spec) ?? {};
+      return { stage, model: front.model ?? null, effort: front.effort ?? null, words: words.get(stage) ?? '', history: history?.get(stage) ?? null, skills: [...required(spec).skills].sort() };
+    }),
+    own: [...specs.keys()].filter((role) => !graph.stages.has(role)).map((role) => ({ role, description: (frontmatter(specs.get(role)) ?? {}).description ?? '' })),
+    shapes: existsSync(claude) ? chainsByShape(readFileSync(claude, 'utf8')) : [],
+  };
+}
+
 /**
  * Runs the command.
  *
@@ -200,12 +248,9 @@ export async function pipeline(argv) {
     }
     return viewOffice({ target: dir, wanted: arg('--session') || undefined, replay: argv.includes('--replay'), port: port === null ? undefined : Number(port), open: argv.includes('--open') });
   }
-  const graphText = readFileSync(graphPath, 'utf8');
-  const graph = parseGraph(graphText);
-  const words = stageWords(graphText);
-  const shape = shapeOf(graph);
-  const claude = join(dir, 'CLAUDE.md');
-  const shapes = existsSync(claude) ? chainsByShape(readFileSync(claude, 'utf8')) : [];
+  const data = pipelineData(dir, { since: arg('--since') || undefined });
+  const { graph, shapes } = data;
+  const shape = { line: data.line, ends: data.ends, gates: data.gates, back: data.back };
 
   // `--for`: one shape's chain, and only the stages on it.
   const query = arg('--for');
@@ -232,9 +277,7 @@ export async function pipeline(argv) {
   const padName = (s, n) => `${name(s)}${' '.repeat(Math.max(n - s.length, 1))}`;
 
   // A pipeline composed by hand draws all the same; only the header has less to say.
-  const profile = readProfile(dir).profile ?? {};
-  const surfaces = (profile.surfaces ?? []).slice().sort();
-  console.log(`  pipeline · ${basename(dir)}${profile.core ? ` · core ${profile.core}` : ''}${surfaces.length ? ` · surfaces ${surfaces.join(', ')}` : ''}`);
+  console.log(`  pipeline · ${data.project}${data.core ? ` · core ${data.core}` : ''}${data.surfaces.length ? ` · surfaces ${data.surfaces.join(', ')}` : ''}`);
 
   // The stages this drawing covers: all of them, or those the chosen shape's chain names.
   const onChain = chosen ? (chosen.chain.match(/[a-z][a-z-]*/g) ?? []).filter((w) => graph.stages.has(w)) : null;
@@ -300,30 +343,18 @@ export async function pipeline(argv) {
   }
 
   // Each stage: what it does, on which model and effort, with which skills, and what it did lately.
-  const agentsDir = join(dir, '.claude', 'agents');
-  const specs = new Map();
-  for (const file of existsSync(agentsDir) ? readdirSync(agentsDir).filter((f) => f.endsWith('.md')).sort() : []) {
-    specs.set(file.replace(/\.md$/, ''), readFileSync(join(agentsDir, file), 'utf8'));
-  }
-  const since = arg('--since') || localDay(new Date(Date.now() - HISTORY_DAYS * 86_400_000).toISOString());
-  const runs = projectRecords(dir);
-  const history = runs ? historyOf(runs, since) : null;
-  const stages = [...graph.stages].filter((s) => covered.has(s));
-  const col = Math.max(...stages.map((s) => s.length), 10) + 2;
-  console.log(`\n  ${dim(history ? `stages · what each ran since ${since}, from the measured history` : 'stages · no measured history for this project yet')}`);
-  for (const stage of stages) {
-    const spec = specs.get(stage) ?? '';
-    const front = frontmatter(spec) ?? {};
-    const on = [front.model, front.effort].filter(Boolean).join(' · ') || 'no spec';
-    console.log(`    ${padName(stage, col)}${on.padEnd(26)}${dim(fit(words.get(stage) ?? '', width - col - 32))}`);
-    if (history) {
-      const h = history.get(stage);
+  const stages = data.stages.filter((s) => covered.has(s.stage));
+  const col = Math.max(...stages.map((s) => s.stage.length), 10) + 2;
+  console.log(`\n  ${dim(data.measured ? `stages · what each ran since ${data.since}, from the measured history` : 'stages · no measured history for this project yet')}`);
+  for (const { stage, model, effort, words, history: h, skills } of stages) {
+    const on = [model, effort].filter(Boolean).join(' · ') || 'no spec';
+    console.log(`    ${padName(stage, col)}${on.padEnd(26)}${dim(fit(words, width - col - 32))}`);
+    if (data.measured) {
       const said = !h
         ? 'no run'
         : `${h.runs} run(s)${h.rounds > h.runs ? ` in ${h.rounds} rounds` : ''}${h.read === 0 ? ', no verdict read' : `, sent back ${h.back} of ${h.read} (${Math.round((100 * h.back) / h.read)}%)`}`;
       console.log(`    ${' '.repeat(col)}${dim('ran     ')}${said}`);
     }
-    const skills = [...required(spec).skills].sort();
     if (skills.length > 0) {
       const lines = [];
       let current = '';
@@ -341,12 +372,10 @@ export async function pipeline(argv) {
 
   if (!chosen) {
     // An agent the project wrote for itself is not a stage, and the graph does not route to it.
-    const own = [...specs.keys()].filter((role) => !graph.stages.has(role));
-    if (own.length > 0) {
+    if (data.own.length > 0) {
       console.log(`\n  ${dim("the project's own agents, outside the pipeline")}`);
-      for (const role of own) {
-        const front = frontmatter(specs.get(role)) ?? {};
-        console.log(`    ${role}${' '.repeat(Math.max(col - role.length, 2))}${dim(fit(front.description ?? '', width - col - 6))}`);
+      for (const { role, description } of data.own) {
+        console.log(`    ${role}${' '.repeat(Math.max(col - role.length, 2))}${dim(fit(description, width - col - 6))}`);
       }
     }
     if (shapes.length > 0) {

package/src/commands/runs.mjs

@@ -66,12 +66,25 @@ function summarize(cycle) {
     unpriced,
     name: String(rounds[0]?.desc ?? '').trim() || '(no description)',
     how: ENDS[cycle.end](rounds.at(-1)),
+    end_kind: cycle.end,
     stages: [...stages.values()].map((s) => ({ ...s, runs: s.runs.size })),
     files: files.length ? Math.max(...files) : null,
     peak: peaks.length ? Math.max(...peaks) : null,
   };
 }
 
+/**
+ * A project's pieces of work, newest first: what `nina runs` prints and the dashboard's tasks tab shows.
+ *
+ * @param {object[]} records - The project's measured rounds.
+ * @returns {ReturnType<typeof summarize>[]}
+ */
+export function taskList(records) {
+  return cyclesOf(records.filter((r) => r.status !== 'denied'))
+    .map(summarize)
+    .sort((a, b) => b.start.localeCompare(a.start));
+}
+
 /** A duration in minutes, as hours and minutes. */
 const span = (m) => (m === null ? '—' : m >= 60 ? `${Math.floor(m / 60)}h${String(m % 60).padStart(2, '0')}` : `${m}m`);
 
@@ -103,9 +116,7 @@ export async function runs(argv) {
     );
     return 1;
   }
-  const cycles = cyclesOf(records)
-    .map(summarize)
-    .sort((a, b) => b.start.localeCompare(a.start));
+  const cycles = taskList(records);
   const shown = cycles.slice(0, last);
   console.log(
     `  ${bold(pink(projectName(records[0].project)))} · ${cycles.length} cycle(s)` +