@house365/comate 0.2.5 → 0.2.7

package/bin/init-run.mjs

@@ -1,4 +1,5 @@
 // The `init` flow: install whatever the current project is missing.
+import { spawn } from 'node:child_process';
 import { existsSync, readFileSync } from 'node:fs';
 import { join, resolve } from 'node:path';
 
@@ -54,10 +55,7 @@ export async function init(pkgRoot) {
     return 1;
   }
 
-  // ponytail: a plain line, not a spinner. \r-erasing it needs a real terminal,
-  // and through npm's .ps1 shim stdout is a pipe where \r leaves the padding
-  // visible — the failed cleanup looked worse than just keeping the line.
-  console.log('\n安装中…');
+  const stopSpinner = startSpinner('Initializing…');
 
   // 1. OpenSpec CLI — a PATH command, so always global.
   const os = openSpecState();
@@ -115,7 +113,7 @@ export async function init(pkgRoot) {
   // phases read it, so an install without it cannot actually run. An existing
   // one is left alone, which covers projects that already keep it elsewhere.
   if (existsSync('openspec')) {
-    results.push(['openspec 结构', 'skipped', '已存在']);
+    results.push(['openspec/', 'skipped', '已存在']);
   } else {
     // --tools is not optional in practice: openspec derives the tool list by
     // sniffing existing agent config dirs, so a fresh project (no .claude etc.)
@@ -131,16 +129,42 @@ export async function init(pkgRoot) {
       NPM_TIMEOUT_MS,
       { quiet: true },
     );
-    results.push(['openspec 结构', r2.ok ? 'installed' : 'failed', r2.ok ? '' : r2.reason, r2.output]);
+    results.push(['openspec/', r2.ok ? 'installed' : 'failed', r2.ok ? '' : r2.reason, r2.output]);
   }
 
-  return report(results, { home, cwd });
+  stopSpinner();
+  return report(results);
+}
+
+// The spinner runs in its own process because every step above is a synchronous
+// spawnSync — an interval on this thread would never fire. In a pipe \r does not
+// overwrite, so animating there would leave every frame on screen; print one
+// static line instead.
+function startSpinner(label) {
+  if (!process.stdout.isTTY) {
+    console.log(`\n${label}`);
+    return () => {};
+  }
+
+  const child = spawn(process.execPath, [join(import.meta.dirname, 'spinner.mjs'), label], {
+    stdio: ['ignore', 'inherit', 'ignore'],
+  });
+  let stopped = false;
+  return () => {
+    if (stopped) return;
+    stopped = true;
+    child.kill();
+    // The parent erases the line, not the child: on Windows SIGTERM is not a
+    // real signal — Node terminates the process outright, so a handler in the
+    // child never runs and its last frame would sit under the logo.
+    process.stdout.write(`\r${' '.repeat(label.length + 6)}\r`);
+  };
 }
 
 // Success is one screen: the wordmark and where things landed. The per-item
 // breakdown only earns its space when something failed, and then it carries the
 // reason — a bare "失败" would leave the user with nowhere to go.
-function report(results, { home, cwd }) {
+function report(results) {
   const failed = results.filter(([, s]) => s === 'failed');
   if (failed.length > 0) {
     console.error('\n安装失败\n');
@@ -158,9 +182,7 @@ function report(results, { home, cwd }) {
   }
 
   console.log(banner(pkgVersion()));
-  console.log(`  skills      ${join(home, '.agents', 'skills')}`);
-  console.log(`  openspec    ${join(cwd, 'openspec')}`);
-  console.log('\n  输入 /comate 开始使用，comate doctor 可自检\n');
+  console.log('  Type /comate in your agent to start, or run comate doctor\n');
   return 0;
 }
 
@@ -198,13 +220,16 @@ export function banner(version, { ci = isCI() } = {}) {
     '╚██████╗╚██████╔╝██║ ╚═╝ ██║██║  ██║   ██║   ███████╗',
     ' ╚═════╝ ╚═════╝ ╚═╝     ╚═╝╚═╝  ╚═╝   ╚═╝   ╚══════╝',
   ];
-  const cyan = (s) => (process.env.NO_COLOR ? s : `\x1B[36m${s}\x1B[0m`);
+  // #ed6d00 as a 24-bit SGR sequence. Truecolor is not universal, but the two
+  // terminals that matter here (Windows Terminal, VS Code) both support it, and
+  // one that doesn't degrades to its nearest palette entry rather than breaking.
+  const brand = (s) => (process.env.NO_COLOR ? s : `\x1B[38;2;237;109;0m${s}\x1B[0m`);
   const dim = (s) => (process.env.NO_COLOR ? s : `\x1B[2m${s}\x1B[0m`);
   return [
     '',
     // Trailing padding keeps the source rows aligned; strip it so the colour
     // reset doesn't sit behind spaces the terminal may highlight on select.
-    ...art.map((line) => `  ${cyan(line.replace(/\s+$/, ''))}`),
+    ...art.map((line) => `  ${brand(line.replace(/\s+$/, ''))}`),
     '',
     `  ${dim(`${tag} - AI coding almost there!`)}`,
     '',

package/bin/spinner.mjs

@@ -0,0 +1,28 @@
+#!/usr/bin/env node
+// The init spinner, as its own process.
+//
+// It cannot run on init's main thread: every install step is a synchronous
+// spawnSync, which blocks the event loop, so an interval would not fire and the
+// frame would sit frozen for the whole install — worse than plain text.
+//
+// Animating means redrawing with \r, which only overwrites on a real terminal.
+// In a pipe every frame would stay on screen, so there the parent prints a
+// static line instead and never starts this process.
+const FRAMES = ['⠋', '⠙', '⠹', '⠸', '⠼', '⠴', '⠦', '⠧', '⠇', '⠏'];
+const INTERVAL_MS = 80;
+
+const label = process.argv[2] ?? 'Initializing…';
+let i = 0;
+
+process.stdout.write('\n');
+const timer = setInterval(() => {
+  process.stdout.write(`\r  ${FRAMES[i++ % FRAMES.length]} ${label}`);
+}, INTERVAL_MS);
+
+// No cleanup handler here on purpose: on Windows SIGTERM is not a real signal,
+// Node terminates the process outright and a handler would never run. The parent
+// erases the line after killing us, which works on every platform.
+//
+// timer is the only thing holding the loop open, so nothing else is needed to
+// keep animating until that kill arrives.
+void timer;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@house365/comate",
-  "version": "0.2.5",
+  "version": "0.2.7",
   "description": "comate —— OpenSpec + Superpowers 规约驱动的 AI 开发编排器（Agent Skills）",
   "type": "module",
   "license": "MIT",