@house365/comate 0.2.0 → 0.2.2

package/bin/init-run.mjs

@@ -1,11 +1,12 @@
 // The `init` flow: install whatever the current project is missing.
-import { existsSync } from 'node:fs';
+import { existsSync, readFileSync } from 'node:fs';
 import { join, resolve } from 'node:path';
 
 import {
   MIN_OPENSPEC,
   NPM_TIMEOUT_MS,
   SKILLS_TIMEOUT_MS,
+  capture,
   npmInvocation,
   openSpecState,
   run,
@@ -82,8 +83,12 @@ export async function init(pkgRoot) {
   if (superpowersInstalled('global') || superpowersInstalled('project')) {
     results.push(['Superpowers', 'skipped', '已安装']);
   } else {
-    console.log('\n→ 安装 Superpowers (skills add obra/superpowers -g)');
-    const r = run(skillsInvocation(pkgRoot, ['add', 'obra/superpowers', '-g', '-y']), SKILLS_TIMEOUT_MS);
+    console.log('\n→ 安装 Superpowers');
+    const r = run(
+      skillsInvocation(pkgRoot, ['add', 'obra/superpowers', '-g', '-y']),
+      SKILLS_TIMEOUT_MS,
+      { quiet: true },
+    );
     // It clones from GitHub, so a bare exit code hides the usual cause: no
     // network. Say so rather than leaving the user with "exit code 1".
     const note = r.ok
@@ -92,11 +97,18 @@ export async function init(pkgRoot) {
     results.push(['Superpowers', r.ok ? 'installed' : 'failed', note]);
   }
 
-  // 3. comate itself.
+  // 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.
   console.log('\n→ 安装 comate skills');
   const comateArgs = ['add', pkgRoot, '--skill', '*', '-g', '-y'];
-  const r = run(skillsInvocation(pkgRoot, comateArgs), SKILLS_TIMEOUT_MS);
-  results.push(['comate', r.ok ? 'installed' : 'failed', r.ok ? '' : r.reason]);
+  const r = run(skillsInvocation(pkgRoot, comateArgs), SKILLS_TIMEOUT_MS, { quiet: true });
+  const landed = existsSync(join(home, '.agents', 'skills', 'comate', 'SKILL.md'));
+  if (r.ok && !landed) {
+    results.push(['comate', 'failed', `未装到 ${join(home, '.agents', 'skills')}，重跑看详情`]);
+  } else {
+    results.push(['comate', r.ok ? 'installed' : 'failed', r.ok ? '' : r.reason]);
+  }
 
   // 4. openspec/ project structure. Not optional: comate's open and archive
   // phases read it, so an install without it cannot actually run. An existing
@@ -105,8 +117,17 @@ export async function init(pkgRoot) {
     results.push(['openspec 结构', 'skipped', '已存在']);
   } else {
     console.log('\n→ 初始化 openspec/ 项目结构');
+    // --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
+    // skills are agent-agnostic and only openspec/ itself matters here, so ask
+    // for none rather than guessing which agents the user wants wired up.
     const r2 = run(
-      { command: 'openspec', args: ['init', '.'], shell: process.platform === 'win32' },
+      {
+        command: 'openspec',
+        args: ['init', '.', '--tools', 'none'],
+        shell: process.platform === 'win32',
+      },
       NPM_TIMEOUT_MS,
     );
     results.push(['openspec 结构', r2.ok ? 'installed' : 'failed', r2.ok ? '' : r2.reason]);
@@ -127,6 +148,54 @@ function report(results) {
     console.error(`\n${failed.length} 项失败，修好后重跑 comate init。`);
     return 1;
   }
-  console.log('\n完成。在 agent 中输入 /comate 开始使用，或跑 comate doctor 自检。');
+  console.log(banner(pkgVersion()));
+  console.log('在 agent 中输入 /comate 开始使用，或跑 comate doctor 自检。\n');
   return 0;
 }
+
+// ponytail: read package.json rather than hardcoding — npm version bumps the
+// file, and a stale literal in the banner is worse than no version at all.
+export function pkgVersion(root = resolve(import.meta.dirname, '..')) {
+  try {
+    return JSON.parse(readFileSync(join(root, 'package.json'), 'utf8')).version;
+  } catch {
+    return '';
+  }
+}
+
+// Box-drawing glyphs need a UTF-8 capable terminal. Windows consoles still
+// default to a legacy codepage where they render as mojibake, so the wordmark
+// only goes out when the codepage says otherwise; everything else gets the
+// plain line. Piped output is skipped too — logos belong on screens, not in
+// logs or CI transcripts.
+export function banner(version, { tty = process.stdout.isTTY, utf8 = unicodeSafe() } = {}) {
+  const tag = `comate${version ? ` v${version}` : ''}`;
+  const plain = `\n  ${tag} — OpenSpec + Superpowers 规约驱动开发编排器\n`;
+  if (!tty || !utf8) return plain;
+
+  const art = [
+    '██████╗ ██████╗ ███╗   ███╗ █████╗ ███████╗',
+    '██╔═══╝██╔═══██╗████╗ ████║██╔══██╗╚══██╔══╝',
+    '██║     ██║   ██║██╔████╔██║███████║   ██║',
+    '██║     ██║   ██║██║╚██╔╝██║██╔══██║   ██║',
+    '╚██████╗╚██████╔╝██║ ╚═╝ ██║██║  ██║   ██║',
+    ' ╚═════╝ ╚═════╝ ╚═╝     ╚═╝╚═╝  ╚═╝   ╚═╝',
+  ];
+  const cyan = (s) => (process.env.NO_COLOR ? s : `\x1B[36m${s}\x1B[0m`);
+  const dim = (s) => (process.env.NO_COLOR ? s : `\x1B[2m${s}\x1B[0m`);
+  return [
+    '',
+    ...art.map((line) => `  ${cyan(line)}`),
+    '',
+    `  ${dim(`${tag} — OpenSpec + Superpowers 规约驱动开发编排器`)}`,
+    '',
+  ].join('\n');
+}
+
+// POSIX terminals are UTF-8 in practice. Windows consoles are not: the legacy
+// codepage (936, 437, …) renders box-drawing glyphs as mojibake, so ask chcp
+// and only claim UTF-8 on 65001.
+function unicodeSafe() {
+  if (process.platform !== 'win32') return true;
+  return /65001/.test(capture('chcp', []) ?? '');
+}

package/bin/init.mjs

@@ -75,14 +75,24 @@ function parsePackOutput(stdout) {
   throw new Error(`no usable file list in npm pack output:\n${stdout}`);
 }
 
-function run({ command, args, shell }, timeout) {
-  const { status, error } = spawnSync(command, args, {
-    stdio: 'inherit',
+// quiet buffers the child's output and prints it only on failure. The skills
+// CLI reports a per-agent failure for every agent that has no user-level skills
+// dir (PromptScript et al.) — cosmetic, since it neither affects the exit code
+// nor the agents that did get installed, but it reads like a broken install.
+// init prints its own summary, so on success the child's output adds nothing.
+function run({ command, args, shell }, timeout, { quiet = false } = {}) {
+  const { status, error, stdout, stderr } = spawnSync(command, args, {
+    stdio: quiet ? ['ignore', 'pipe', 'pipe'] : 'inherit',
+    encoding: quiet ? 'utf8' : undefined,
     timeout,
     ...(shell ? { shell: true } : {}),
   });
   if (error) return { ok: false, reason: error.message };
-  if (status !== 0) return { ok: false, reason: `exit code ${String(status)}` };
+  if (status !== 0) {
+    // Without this the user gets a bare "exit code 1" and no way to diagnose.
+    if (quiet) process.stderr.write((stdout ?? '') + (stderr ?? ''));
+    return { ok: false, reason: `exit code ${String(status)}` };
+  }
   return { ok: true };
 }
 

package/package.json

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