@house365/comate 0.2.6 → 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,19 +55,13 @@ export async function init(pkgRoot) {
     return 1;
   }
 
-  // ponytail: printed steps, not a \r spinner. npm's Windows global install
-  // routes `comate` through a comate.ps1 shim that makes stdout a pipe, where
-  // \r does not overwrite — every frame would stay on screen. A line per step
-  // shows progress in both a terminal and a pipe.
-  console.log('\nInitializing…');
-  const step = (name) => process.stdout.write(`  ${name}\n`);
+  const stopSpinner = startSpinner('Initializing…');
 
   // 1. OpenSpec CLI — a PATH command, so always global.
   const os = openSpecState();
   if (os.status === 'ready') {
     results.push(['OpenSpec CLI', 'skipped', `已安装 ${os.version}`]);
   } else {
-    step('OpenSpec CLI');
     const r = run(
       npmInvocation(['install', '-g', '@fission-ai/openspec@latest']),
       NPM_TIMEOUT_MS,
@@ -89,7 +84,6 @@ export async function init(pkgRoot) {
   if (superpowersInstalled('global') || superpowersInstalled('project')) {
     results.push(['Superpowers', 'skipped', '已安装']);
   } else {
-    step('Superpowers');
     const r = run(
       skillsInvocation(pkgRoot, ['add', 'obra/superpowers', '-g', '-y']),
       SKILLS_TIMEOUT_MS,
@@ -106,7 +100,6 @@ export async function init(pkgRoot) {
   // 3. comate itself. The skills CLI reports per-agent failures without ever
   // setting a non-zero exit code, and we swallow its output — so a zero status
   // is not evidence the skill landed. Check the file.
-  step('comate skills');
   const comateArgs = ['add', pkgRoot, '--skill', '*', '-g', '-y'];
   const r = run(skillsInvocation(pkgRoot, comateArgs), SKILLS_TIMEOUT_MS, { quiet: true });
   const landed = existsSync(join(home, '.agents', 'skills', 'comate', 'SKILL.md'));
@@ -122,7 +115,6 @@ export async function init(pkgRoot) {
   if (existsSync('openspec')) {
     results.push(['openspec/', 'skipped', '已存在']);
   } else {
-    step('openspec/ structure');
     // --tools is not optional in practice: openspec derives the tool list by
     // sniffing existing agent config dirs, so a fresh project (no .claude etc.)
     // aborts with "No tools detected and no --tools flag provided". comate's
@@ -140,13 +132,39 @@ export async function init(pkgRoot) {
     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');
@@ -164,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;
 }
 
@@ -204,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.6",
+  "version": "0.2.7",
   "description": "comate —— OpenSpec + Superpowers 规约驱动的 AI 开发编排器（Agent Skills）",
   "type": "module",
   "license": "MIT",