@llman-sdd/cli 0.1.2 → 0.1.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@llman-sdd/cli",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "Spec-driven development workflow CLI (provides both `llman-sdd` and `llmanspec` commands)",
   "license": "MIT",
   "repository": {
@@ -28,10 +28,11 @@
     "build": "bun run ./scripts/build-binary.ts"
   },
   "dependencies": {
-    "@llman-sdd/core": "0.1.2",
+    "@llman-sdd/core": "0.1.3",
     "commander": "^15.0.0"
   },
   "engines": {
-    "bun": ">=1.4.0"
+    "bun": ">=1.4.0",
+    "node": ">=24.0.0"
   }
 }

package/src/io.ts

@@ -55,8 +55,9 @@ export function makeIo(root: string): CliIo {
       try {
         process.kill(pid, 0);
         return true;
-      } catch {
-        return false;
+      } catch (error) {
+        // EPERM = 进程存在但无权发信号;只有 ESRCH 才能判定死亡。
+        return (error as NodeJS.ErrnoException).code !== 'ESRCH';
       }
     },
     mtimeMs: (p) => statSync(full(p)).mtimeMs,

package/src/main.ts

@@ -1,6 +1,6 @@
 #!/usr/bin/env bun
 import { spawnSync } from 'node:child_process';
-import { existsSync, readdirSync, readFileSync, statSync, writeFileSync } from 'node:fs';
+import { existsSync, readFileSync, writeFileSync } from 'node:fs';
 import { join } from 'node:path';
 
 import {
@@ -9,7 +9,10 @@ import {
   buildReview,
   collectChanges,
   collectSpecs,
+  discoverSpecs,
+  embeddedTemplates,
   graphMermaid,
+  makeEmbeddedTemplateIo,
   nextReqId,
   changeDiff,
   deriveChangeId,
@@ -17,10 +20,10 @@ import {
   loadConfig,
   loadTree,
   newChange,
-  parseCapability,
   renderChangesJson,
   runContextRetrieval,
   renderChangesList,
+  renderReviewHtml,
   renderSpecsJson,
   renderSpecsList,
   runInit,
@@ -34,10 +37,10 @@ import {
   runList,
   runThaw,
   validateAllSpecs,
-  type TagBinding,
-  type GitLike,
+  TEMPLATES_ROOT,
   resolveChatConfig,
   unavailableResult,
+  type TemplateIo,
 } from '@llman-sdd/core';
 import { Command } from 'commander';
 
@@ -47,23 +50,25 @@ import { makeCliGit, makeIo } from './io.ts';
 // package version when running from source.
 const version = process.env.LLMAN_SDD_VERSION ?? VERSION;
 
-function collectFeatureFiles(dir: string): string[] {
-  if (!existsSync(dir) || !statSync(dir).isDirectory()) return [];
-  const out: string[] = [];
-  for (const name of readdirSync(dir).toSorted()) {
-    const full = join(dir, name);
-    if (statSync(full).isDirectory()) out.push(...collectFeatureFiles(full));
-    else if (name.endsWith('.feature')) out.push(full);
-  }
-  return out;
+// Compiled single-file binaries have no on-disk templates and Bun <= 1.4.x has
+// no embedding mechanism, so build-binary.ts injects the template table via
+// define; every non-compiled run keeps reading the real filesystem (npm/source/
+// Node). Both init and `review --export-html` resolve through this one seam.
+const embedded = embeddedTemplates();
+const templateIo: TemplateIo = embedded
+  ? makeEmbeddedTemplateIo(embedded)
+  : {
+      exists: (p) => existsSync(p),
+      readText: (p) => readFileSync(p, 'utf8'),
+    };
+
+/** Parse all capability specs under llmanspec/specs via core discovery. */
+function loadSpecEntries(): ReturnType<typeof discoverSpecs> {
+  return discoverSpecs('llmanspec/specs', makeIo(process.cwd()));
 }
 
 function runValidateSpecs(options: { specs?: boolean; check: boolean }): number {
-  const entries = collectFeatureFiles('llmanspec/specs').map((path) => ({
-    fileName: path,
-    doc: parseCapability(readFileSync(path, 'utf8'), path),
-  }));
-  const report = validateAllSpecs(entries, makeIo(process.cwd()));
+  const report = validateAllSpecs(loadSpecEntries(), makeIo(process.cwd()));
   for (const line of report.lines) console.log(line);
 
   let failed = report.failed;
@@ -88,7 +93,7 @@ program
   .option('--update', 'refresh an existing installation')
   .option('--locale <locale>', 'locale for generated templates (defaults to config or en)')
   .action((options: { update?: boolean; locale?: string }) => {
-    const result = runInit(process.cwd(), {
+    const result = runInit(makeIo(process.cwd()), templateIo, {
       update: options.update ?? false,
       locale: options.locale,
       version,
@@ -103,7 +108,8 @@ program
   .option('--specs', 'validate specs (default and only scope for now)')
   .option('--no-check', 'skip the bdd.run_command check (structural validation only)')
   .action((options: { specs?: boolean; check: boolean }) => {
-    process.exit(runValidateSpecs(options));
+    // Node 下管道 stdout 写入异步,process.exit 会截断输出;exitCode 等价且安全。
+    process.exitCode = runValidateSpecs(options);
   });
 
 const change = program
@@ -118,7 +124,8 @@ change
   .action((id: string | undefined, options: { from?: string }) => {
     if ((id === undefined) === (options.from === undefined)) {
       console.error('<CHANGE> and --from are mutually exclusive; pass one or the other');
-      process.exit(1);
+      process.exitCode = 1;
+      return;
     }
     const io = makeIo(process.cwd());
     const result = newChange(io, { id, from: options.from });
@@ -132,7 +139,7 @@ change
   .argument('<id>')
   .option('--branch-prefix <prefix>', 'feature branch prefix', 'sdd/')
   .action((id: string, options: { branchPrefix: string }) => {
-    const git: GitLike = makeCliGit(process.cwd());
+    const git = makeCliGit(process.cwd());
     const result = startChange(git, makeIo(process.cwd()), id, {
       branchPrefix: options.branchPrefix,
     });
@@ -175,7 +182,8 @@ change
   .action((id: string, options: { into?: string; method: string }) => {
     if (options.method !== 'squash' && options.method !== 'ff') {
       console.error(`invalid --method: ${options.method}`);
-      process.exit(1);
+      process.exitCode = 1;
+      return;
     }
     const result = finalizeChange(makeCliGit(process.cwd()), makeIo(process.cwd()), id, {
       into: options.into,
@@ -194,11 +202,7 @@ program
   .option('--json', 'machine-readable output')
   .action((options: { specs?: boolean; json?: boolean }) => {
     if (options.specs) {
-      const entries = collectFeatureFiles('llmanspec/specs').map((path) => ({
-        fileName: path,
-        doc: parseCapability(readFileSync(path, 'utf8'), path),
-      }));
-      const summaries = collectSpecs(entries);
+      const summaries = collectSpecs(loadSpecEntries());
       console.log(
         options.json ? renderSpecsJson(summaries) : renderSpecsList(summaries).join('\n'),
       );
@@ -223,13 +227,10 @@ program
       const path = join('llmanspec', 'specs', `${item}.feature`);
       if (!existsSync(path)) {
         console.error(`spec not found: ${item}`);
-        process.exit(1);
+        process.exitCode = 1;
+        return;
       }
-      const entries = collectFeatureFiles('llmanspec/specs').map((specPath) => ({
-        fileName: specPath,
-        doc: parseCapability(readFileSync(specPath, 'utf8'), specPath),
-      }));
-      const summary = collectSpecs(entries).find((x) => x.id === item);
+      const summary = collectSpecs(loadSpecEntries()).find((x) => x.id === item);
       const morphology = summary
         ? `\n\n## Morphology\nruleCount=${summary.morphology.ruleCount} enforced=${summary.morphology.ruleEnforcedCount} manual=${summary.morphology.ruleManualCount} pending=${summary.morphology.rulePendingCount} acceptanceCount=${summary.morphology.acceptanceCount}`
         : '';
@@ -238,16 +239,15 @@ program
     }
     if (options.output !== 'json') {
       console.error('only --output json is supported for changes (text format pending)');
-      process.exit(1);
+      process.exitCode = 1;
+      return;
     }
-    const io = makeIo(process.cwd());
     const result = showChangeJson(
       {
-        io,
-        discovery: makeIo(process.cwd()),
+        io: makeIo(process.cwd()),
+        git: makeCliGit(process.cwd()),
         root: process.cwd(),
         specsDir: 'llmanspec/specs',
-        now: new Date(),
       },
       item,
     );
@@ -261,7 +261,8 @@ program
   .action((options: { format: string }) => {
     if (options.format !== 'mermaid') {
       console.error(`unsupported format: ${options.format}`);
-      process.exit(1);
+      process.exitCode = 1;
+      return;
     }
     console.log(graphMermaid(makeIo(process.cwd()), process.cwd()).join('\n'));
   });
@@ -351,7 +352,7 @@ archive
       for (const line of result.lines) console.log(line);
     } catch (error) {
       console.error((error as Error).message);
-      process.exit(1);
+      process.exitCode = 1;
     }
   });
 
@@ -364,30 +365,24 @@ review
   .option('--json', 'emit structured JSON (signals + summary)')
   .option('--export-html <path>', 'write a self-contained HTML report')
   .action((options: { capability?: string; json?: boolean; exportHtml?: string }) => {
-    const entries = collectFeatureFiles('llmanspec/specs').map((path) => ({
-      fileName: path,
-      doc: parseCapability(readFileSync(path, 'utf8'), path),
-    }));
     const config = existsSync('llmanspec/config.yaml')
       ? loadConfig(readFileSync('llmanspec/config.yaml', 'utf8'))
       : null;
     const bindings = config?.bdd?.bindings?.filter((b) => b.kind === 'tags') ?? [];
-    const changeIo = makeIo(process.cwd());
-    const boundCount = collectChanges(changeIo, process.cwd(), new Date()).filter(
-      (c) => c.hasBinding,
-    ).length;
-    const activeChanges = collectChanges(changeIo, process.cwd(), new Date());
+    const io = makeIo(process.cwd());
+    const activeChanges = collectChanges(io, process.cwd(), new Date());
     const result = buildReview(
       {
-        entries,
-        bindings: bindings as TagBinding[],
-        boundChangeCount: boundCount,
+        entries: loadSpecEntries(),
+        bindings: bindings.map((b) => ({ kind: 'tags', tags: b.tags })),
+        boundChangeCount: activeChanges.filter((c) => c.hasBinding).length,
         activeChanges,
       },
-      makeIo(process.cwd()),
+      io,
     );
     if (options.exportHtml !== undefined) {
-      writeFileSync(options.exportHtml, renderReviewHtml(result));
+      const template = templateIo.readText(join(TEMPLATES_ROOT, 'shared', 'review.html'));
+      writeFileSync(options.exportHtml, renderReviewHtml(template, result));
       console.log(`wrote ${options.exportHtml}`);
     }
     if (options.json) {
@@ -395,46 +390,9 @@ review
     } else {
       console.log(result.lines.join('\n'));
     }
-    if (result.exitCode !== 0) process.exit(result.exitCode);
+    if (result.exitCode !== 0) process.exitCode = result.exitCode;
   });
 
-/** Self-contained HTML report via the v1 shared/review.html template. */
-function renderReviewHtml(result: {
-  signals: { kind: string; capability: string; count: number; detail: string }[];
-  summary: { criticalCount: number; warningCount: number };
-}): string {
-  // main.ts sits at <root>/apps/cli/src — three levels up is the repo root.
-  const templatePath = join(
-    import.meta.dirname ?? '.',
-    '..',
-    '..',
-    '..',
-    'packages',
-    'core',
-    'templates',
-    'shared',
-    'review.html',
-  );
-  const template = readFileSync(templatePath, 'utf8');
-  const esc = (input: string): string =>
-    input.replaceAll('&', '&amp;').replaceAll('<', '&lt;').replaceAll('>', '&gt;');
-  let mermaid = 'graph TD\n';
-  const sigJson: unknown[] = [];
-  result.signals.forEach((s, idx) => {
-    const label = esc(
-      `${s.capability} [${s.kind}] = ${s.count} — ${s.detail === '' ? 'ok' : s.detail}`,
-    );
-    mermaid += `    s${idx}["${label}"]\n`;
-    sigJson.push({ kind: s.kind, capability: s.capability, count: s.count, detail: s.detail });
-  });
-  return template
-    .replaceAll('__CRITICAL__', String(result.summary.criticalCount))
-    .replaceAll('__WARNING__', String(result.summary.warningCount))
-    .replaceAll('__SIGNALS__', JSON.stringify(sigJson))
-    .replaceAll('__MERMAID__', mermaid)
-    .replaceAll('__GENERATED__', new Date().toISOString());
-}
-
 const indexCmd = program
   .command('index')
   .description('Index management commands (rebuild, check freshness)');
@@ -443,12 +401,7 @@ indexCmd
   .command('rebuild')
   .description('Rebuild the pageindex tree from spec IR (no LLM)')
   .action(() => {
-    const indexIo = makeIo(process.cwd());
-    const entries = collectFeatureFiles('llmanspec/specs').map((path) => ({
-      fileName: path,
-      doc: parseCapability(readFileSync(path, 'utf8'), path),
-    }));
-    const result = rebuildIndex(indexIo, process.cwd(), 'llmanspec/specs', entries, {
+    const result = rebuildIndex(makeIo(process.cwd()), 'llmanspec/specs', loadSpecEntries(), {
       chatModel: process.env.LLMAN_SDD_INDEX_CHAT_MODEL ?? '',
     });
     for (const line of result.lines) console.log(line);
@@ -458,9 +411,9 @@ indexCmd
   .command('check')
   .description('Check index freshness without rebuilding')
   .action(() => {
-    const result = checkIndexFreshness(makeIo(process.cwd()), process.cwd(), 'llmanspec/specs');
+    const result = checkIndexFreshness(makeIo(process.cwd()), 'llmanspec/specs');
     for (const line of result.lines) console.log(line);
-    if (!result.fresh) process.exit(1);
+    if (!result.fresh) process.exitCode = 1;
   });
 
 program
@@ -469,10 +422,11 @@ program
   .option('--task <task>', 'natural language task description')
   .option('--paths <paths>', 'comma-separated file paths')
   .option('--top <n>', 'max entries per tier', '5')
-  .action((options: { task?: string; paths?: string; top?: string }) => {
+  .action(async (options: { task?: string; paths?: string; top?: string }) => {
     if (!options.task && !options.paths) {
       console.error('at least one of --task or --paths is required');
-      process.exit(1);
+      process.exitCode = 1;
+      return;
     }
     const config = resolveChatConfig(process.env as Record<string, string | undefined>);
     if (config === null) {
@@ -480,14 +434,14 @@ program
       console.log(JSON.stringify(unavailableResult(), null, 2));
       return;
     }
-    const tree = loadTree(makeIo(process.cwd()), process.cwd());
+    const tree = loadTree(makeIo(process.cwd()));
     if (tree === null) {
       const missing = unavailableResult();
       missing.status.qualityNote = 'index missing — run `llman-sdd index rebuild` first';
       console.log(JSON.stringify(missing, null, 2));
       return;
     }
-    void runContextRetrieval({
+    const result = await runContextRetrieval({
       config,
       task: options.task ?? '',
       paths: options.paths,
@@ -495,9 +449,8 @@ program
       tree,
       readFile: (p) => readFileSync(p, 'utf8'),
       root: process.cwd(),
-    }).then((resolved) => {
-      console.log(JSON.stringify(resolved, null, 2));
     });
+    console.log(JSON.stringify(result, null, 2));
   });
 
 async function main(): Promise<void> {