crewx 0.8.1 → 0.8.2-rc.2

package/scripts/smoke-tarball.mjs

@@ -1,285 +1,285 @@
-#!/usr/bin/env node
-/**
- * smoke-tarball.mjs — Tarball-isolated smoke test for crewx CLI.
- *
- * Packs the current project into a tarball, installs it in a temp directory
- * (no global install), and runs a suite of smoke checks against the binary.
- * This avoids workspace symlink false-positives that npm install -g . can produce.
- *
- * Usage:
- *   node scripts/smoke-tarball.mjs [--version <expected>] [--skip-web]
- *
- * Flags:
- *   --version <ver>   Expected crewx --version output. Defaults to package.json version.
- *   --skip-web        Skip the NestJS web server startup check (useful in CI).
- *   --timeout <ms>    Per-command timeout in ms. Default: 15000.
- *
- * Exit code: 0 = ALL PASS, 1 = one or more FAILs.
- */
-
-import { execFileSync, spawnSync } from 'child_process';
-import { mkdtempSync, rmSync, writeFileSync, mkdirSync } from 'fs';
-import { join, resolve } from 'path';
-import { tmpdir } from 'os';
-import { createRequire } from 'module';
-
-const require = createRequire(import.meta.url);
-const rootPkg = require('../package.json');
-
-// ── CLI args ──────────────────────────────────────────────────────────────────
-const args = process.argv.slice(2);
-const getArg = (flag) => {
-  const i = args.indexOf(flag);
-  return i !== -1 ? args[i + 1] : null;
-};
-const hasFlag = (flag) => args.includes(flag);
-
-const expectedVersion = getArg('--version') ?? rootPkg.version;
-const skipWeb = hasFlag('--skip-web');
-const cmdTimeout = parseInt(getArg('--timeout') ?? '15000', 10);
-
-// ── Result tracking ───────────────────────────────────────────────────────────
-const results = [];
-let anyFail = false;
-
-function pass(label) {
-  results.push({ label, status: 'PASS' });
-  console.log(`  ✓ PASS  ${label}`);
-}
-
-function fail(label, detail = '') {
-  results.push({ label, status: 'FAIL', detail });
-  console.error(`  ✗ FAIL  ${label}${detail ? ': ' + detail : ''}`);
-  anyFail = true;
-}
-
-function run(cmd, cmdArgs, opts = {}) {
-  const result = spawnSync(cmd, cmdArgs, {
-    timeout: opts.timeout ?? cmdTimeout,
-    encoding: 'utf8',
-    cwd: opts.cwd,
-    env: { ...process.env, ...opts.env },
-  });
-  return {
-    stdout: result.stdout ?? '',
-    stderr: result.stderr ?? '',
-    status: result.status ?? -1,
-    timedOut: result.signal === 'SIGTERM',
-  };
-}
-
-// ── Step 1: Pack ──────────────────────────────────────────────────────────────
-const packDest = tmpdir();
-console.log('\n[1/3] Packing tarball...');
-let tarball;
-try {
-  const out = execFileSync('pnpm', ['pack', '--pack-destination', packDest], {
-    cwd: resolve('.'),
-    encoding: 'utf8',
-  }).trim();
-  tarball = out.split('\n').pop().trim();
-  console.log(`      Tarball: ${tarball}`);
-} catch (e) {
-  console.error('FATAL: pnpm pack failed:', e.message);
-  process.exit(1);
-}
-
-// ── Step 2: Install in temp dir ───────────────────────────────────────────────
-const installDir = mkdtempSync(join(tmpdir(), 'crewx-smoke-install-'));
-console.log(`\n[2/3] Installing into ${installDir}...`);
-try {
-  execFileSync('npm', ['install', tarball], {
-    cwd: installDir,
-    encoding: 'utf8',
-    stdio: 'inherit',
-  });
-} catch (e) {
-  console.error('FATAL: npm install failed:', e.message);
-  rmSync(installDir, { recursive: true, force: true });
-  process.exit(1);
-}
-
-const binDir = join(installDir, 'node_modules', '.bin');
-const env = { ...process.env, PATH: `${binDir}:${process.env.PATH}` };
-
-// ── Step 3: Test dir (clean workspace) ────────────────────────────────────────
-const testDir = mkdtempSync(join(tmpdir(), 'crewx-smoke-test-'));
-mkdirSync(join(testDir, '.git')); // minimal git marker
-
-// Minimal crewx.yaml for agent tests
-writeFileSync(join(testDir, 'crewx.yaml'), `
-agents:
-  - id: crewx
-    name: "CrewX Assistant"
-    inline:
-      provider: "cli/claude"
-      prompt: "You are a test agent. Reply with OK only."
-`.trim());
-
-console.log(`\n[3/3] Running smoke checks from ${testDir}...\n`);
-
-// ── Check 1: version ──────────────────────────────────────────────────────────
-{
-  const r = run('crewx', ['--version'], { cwd: testDir, env });
-  const actual = r.stdout.trim();
-  if (actual === expectedVersion) {
-    pass(`crewx --version == ${expectedVersion}`);
-  } else {
-    fail('crewx --version', `expected "${expectedVersion}", got "${actual}"`);
-  }
-}
-
-// ── Check 2: help ─────────────────────────────────────────────────────────────
-{
-  const r = run('crewx', ['--help'], { cwd: testDir, env });
-  if (r.status === 0 && r.stdout.includes(expectedVersion)) {
-    pass('crewx --help shows correct version');
-  } else {
-    fail('crewx --help', `exit=${r.status}, version in output: ${r.stdout.includes(expectedVersion)}`);
-  }
-}
-
-// ── Check 3: agent list ───────────────────────────────────────────────────────
-{
-  const r = run('crewx', ['agent', 'list'], { cwd: testDir, env });
-  if (r.status === 0) {
-    pass('crewx agent list');
-  } else {
-    fail('crewx agent list', r.stderr.slice(0, 200));
-  }
-}
-
-// ── Check 4: skill list ───────────────────────────────────────────────────────
-{
-  const r = run('crewx', ['skill', 'list'], { cwd: testDir, env });
-  if (r.status === 0 || r.stdout.includes('No skills')) {
-    pass('crewx skill list');
-  } else {
-    fail('crewx skill list', r.stderr.slice(0, 200));
-  }
-}
-
-// ── Check 5: memory --help ────────────────────────────────────────────────────
-{
-  const r = run('crewx', ['memory', '--help'], { cwd: testDir, env });
-  if (r.status === 0) {
-    pass('crewx memory --help');
-  } else {
-    fail('crewx memory --help', r.stderr.slice(0, 200));
-  }
-}
-
-// ── Check 6: workflow --help ──────────────────────────────────────────────────
-{
-  const r = run('crewx', ['workflow', '--help'], { cwd: testDir, env });
-  if (r.status === 0) {
-    pass('crewx workflow --help');
-  } else {
-    fail('crewx workflow --help', r.stderr.slice(0, 200));
-  }
-}
-
-// ── Check 7: search ───────────────────────────────────────────────────────────
-{
-  const r = run('crewx', ['search', 'test', '.'], { cwd: testDir, env, timeout: 10000 });
-  const noError = !r.stderr.includes('ERR_MODULE_NOT_FOUND') && !r.stderr.includes('is not a function');
-  if (noError) {
-    pass('crewx search (no crash)');
-  } else {
-    fail('crewx search', r.stderr.slice(0, 300));
-  }
-}
-
-// ── Check 8: crewx q "hi" (SDK loading check) ────────────────────────────────
-{
-  const r = run('crewx', ['q', 'hi'], { cwd: testDir, env, timeout: 15000 });
-  const combined = r.stdout + r.stderr;
-  const sdkErrors = ['ERR_MODULE_NOT_FOUND', 'is not a function', 'Cannot find module'];
-  const hasError = sdkErrors.some(e => combined.includes(e));
-  if (!hasError) {
-    pass('crewx q "hi" (no SDK import errors)');
-  } else {
-    const found = sdkErrors.find(e => combined.includes(e));
-    fail('crewx q "hi"', `SDK error: ${found}`);
-  }
-}
-
-// ── Check 9: crewx web startup ────────────────────────────────────────────────
-if (!skipWeb) {
-  const r = run('crewx', [], { cwd: testDir, env, timeout: 10000 });
-  const combined = r.stdout + r.stderr;
-  const hasModuleError = combined.includes('ERR_MODULE_NOT_FOUND');
-  if (!hasModuleError) {
-    pass('crewx (web) startup — no ERR_MODULE_NOT_FOUND');
-  } else {
-    fail('crewx (web) startup', 'ERR_MODULE_NOT_FOUND detected');
-  }
-} else {
-  console.log('  - SKIP  crewx web startup (--skip-web)');
-}
-
-// ── Check 10: SDK templates load (CJS require) ───────────────────────────────
-{
-  const sdkDir = join(installDir, 'node_modules', '@crewx', 'sdk');
-  const r = run('node', ['-e', `
-    const { existsSync, readdirSync } = require('fs');
-    const { join, dirname } = require('path');
-    const sdkMain = require.resolve('${sdkDir.replace(/\\/g, '/')}/dist/index.js');
-    const distDir = dirname(sdkMain);
-    const templatesPath = join(distDir, '../templates/agents');
-    if (!existsSync(templatesPath)) { process.stdout.write('FAIL: ' + templatesPath + '\\n'); process.exit(1); }
-    const files = readdirSync(templatesPath);
-    if (!files.includes('default.yaml')) { process.stdout.write('FAIL: default.yaml missing\\n'); process.exit(1); }
-    process.stdout.write('OK: ' + templatesPath + '\\n');
-  `], { cwd: testDir, env, timeout: 5000 });
-  if (r.status === 0 && r.stdout.includes('OK:')) {
-    pass('SDK templates load (CJS) — dist/index.js → ../templates/agents ✓');
-  } else {
-    fail('SDK templates load (CJS)', r.stderr.slice(0, 200) || r.stdout.slice(0, 200));
-  }
-}
-
-// ── Check 11: SDK templates load (ESM import — same dist/index.js via exports) ─
-{
-  const sdkDir = join(installDir, 'node_modules', '@crewx', 'sdk');
-  const r = run('node', ['--input-type=module', '--eval', `
-    import { existsSync, readdirSync } from 'fs';
-    import { join, dirname } from 'path';
-    import { createRequire } from 'module';
-    const require = createRequire(import.meta.url);
-    // "import" condition in package.json exports also points to dist/index.js (CJS)
-    const sdkMain = require.resolve('${sdkDir.replace(/\\/g, '/')}/dist/index.js');
-    const distDir = dirname(sdkMain);
-    const templatesPath = join(distDir, '../templates/agents');
-    if (!existsSync(templatesPath)) { process.stdout.write('FAIL: ' + templatesPath + '\\n'); process.exit(1); }
-    const files = readdirSync(templatesPath);
-    if (!files.includes('default.yaml')) { process.stdout.write('FAIL: default.yaml missing\\n'); process.exit(1); }
-    process.stdout.write('OK: ' + templatesPath + '\\n');
-  `], { cwd: testDir, env, timeout: 5000 });
-  if (r.status === 0 && r.stdout.includes('OK:')) {
-    pass('SDK templates load (ESM) — dist/index.js → ../templates/agents ✓');
-  } else {
-    fail('SDK templates load (ESM)', r.stderr.slice(0, 200) || r.stdout.slice(0, 200));
-  }
-}
-
-// ── Cleanup ───────────────────────────────────────────────────────────────────
-rmSync(testDir, { recursive: true, force: true });
-rmSync(installDir, { recursive: true, force: true });
-try { rmSync(tarball, { force: true }); } catch {}
-
-// ── Summary ───────────────────────────────────────────────────────────────────
-console.log('\n' + '─'.repeat(60));
-const passed = results.filter(r => r.status === 'PASS').length;
-const failed = results.filter(r => r.status === 'FAIL').length;
-
-if (!anyFail) {
-  console.log(`✅ ALL PASS (${passed}/${results.length})`);
-} else {
-  console.log(`❌ ${failed} FAIL / ${passed} PASS`);
-  results.filter(r => r.status === 'FAIL').forEach(r => {
-    console.error(`   FAIL: ${r.label}${r.detail ? ' — ' + r.detail : ''}`);
-  });
-}
-
-process.exit(anyFail ? 1 : 0);
+#!/usr/bin/env node
+/**
+ * smoke-tarball.mjs — Tarball-isolated smoke test for crewx CLI.
+ *
+ * Packs the current project into a tarball, installs it in a temp directory
+ * (no global install), and runs a suite of smoke checks against the binary.
+ * This avoids workspace symlink false-positives that npm install -g . can produce.
+ *
+ * Usage:
+ *   node scripts/smoke-tarball.mjs [--version <expected>] [--skip-web]
+ *
+ * Flags:
+ *   --version <ver>   Expected crewx --version output. Defaults to package.json version.
+ *   --skip-web        Skip the NestJS web server startup check (useful in CI).
+ *   --timeout <ms>    Per-command timeout in ms. Default: 15000.
+ *
+ * Exit code: 0 = ALL PASS, 1 = one or more FAILs.
+ */
+
+import { execFileSync, spawnSync } from 'child_process';
+import { mkdtempSync, rmSync, writeFileSync, mkdirSync } from 'fs';
+import { join, resolve } from 'path';
+import { tmpdir } from 'os';
+import { createRequire } from 'module';
+
+const require = createRequire(import.meta.url);
+const rootPkg = require('../package.json');
+
+// ── CLI args ──────────────────────────────────────────────────────────────────
+const args = process.argv.slice(2);
+const getArg = (flag) => {
+  const i = args.indexOf(flag);
+  return i !== -1 ? args[i + 1] : null;
+};
+const hasFlag = (flag) => args.includes(flag);
+
+const expectedVersion = getArg('--version') ?? rootPkg.version;
+const skipWeb = hasFlag('--skip-web');
+const cmdTimeout = parseInt(getArg('--timeout') ?? '15000', 10);
+
+// ── Result tracking ───────────────────────────────────────────────────────────
+const results = [];
+let anyFail = false;
+
+function pass(label) {
+  results.push({ label, status: 'PASS' });
+  console.log(`  ✓ PASS  ${label}`);
+}
+
+function fail(label, detail = '') {
+  results.push({ label, status: 'FAIL', detail });
+  console.error(`  ✗ FAIL  ${label}${detail ? ': ' + detail : ''}`);
+  anyFail = true;
+}
+
+function run(cmd, cmdArgs, opts = {}) {
+  const result = spawnSync(cmd, cmdArgs, {
+    timeout: opts.timeout ?? cmdTimeout,
+    encoding: 'utf8',
+    cwd: opts.cwd,
+    env: { ...process.env, ...opts.env },
+  });
+  return {
+    stdout: result.stdout ?? '',
+    stderr: result.stderr ?? '',
+    status: result.status ?? -1,
+    timedOut: result.signal === 'SIGTERM',
+  };
+}
+
+// ── Step 1: Pack ──────────────────────────────────────────────────────────────
+const packDest = tmpdir();
+console.log('\n[1/3] Packing tarball...');
+let tarball;
+try {
+  const out = execFileSync('pnpm', ['pack', '--pack-destination', packDest], {
+    cwd: resolve('.'),
+    encoding: 'utf8',
+  }).trim();
+  tarball = out.split('\n').pop().trim();
+  console.log(`      Tarball: ${tarball}`);
+} catch (e) {
+  console.error('FATAL: pnpm pack failed:', e.message);
+  process.exit(1);
+}
+
+// ── Step 2: Install in temp dir ───────────────────────────────────────────────
+const installDir = mkdtempSync(join(tmpdir(), 'crewx-smoke-install-'));
+console.log(`\n[2/3] Installing into ${installDir}...`);
+try {
+  execFileSync('npm', ['install', tarball], {
+    cwd: installDir,
+    encoding: 'utf8',
+    stdio: 'inherit',
+  });
+} catch (e) {
+  console.error('FATAL: npm install failed:', e.message);
+  rmSync(installDir, { recursive: true, force: true });
+  process.exit(1);
+}
+
+const binDir = join(installDir, 'node_modules', '.bin');
+const env = { ...process.env, PATH: `${binDir}:${process.env.PATH}` };
+
+// ── Step 3: Test dir (clean workspace) ────────────────────────────────────────
+const testDir = mkdtempSync(join(tmpdir(), 'crewx-smoke-test-'));
+mkdirSync(join(testDir, '.git')); // minimal git marker
+
+// Minimal crewx.yaml for agent tests
+writeFileSync(join(testDir, 'crewx.yaml'), `
+agents:
+  - id: crewx
+    name: "CrewX Assistant"
+    inline:
+      provider: "cli/claude"
+      prompt: "You are a test agent. Reply with OK only."
+`.trim());
+
+console.log(`\n[3/3] Running smoke checks from ${testDir}...\n`);
+
+// ── Check 1: version ──────────────────────────────────────────────────────────
+{
+  const r = run('crewx', ['--version'], { cwd: testDir, env });
+  const actual = r.stdout.trim();
+  if (actual === expectedVersion) {
+    pass(`crewx --version == ${expectedVersion}`);
+  } else {
+    fail('crewx --version', `expected "${expectedVersion}", got "${actual}"`);
+  }
+}
+
+// ── Check 2: help ─────────────────────────────────────────────────────────────
+{
+  const r = run('crewx', ['--help'], { cwd: testDir, env });
+  if (r.status === 0 && r.stdout.includes(expectedVersion)) {
+    pass('crewx --help shows correct version');
+  } else {
+    fail('crewx --help', `exit=${r.status}, version in output: ${r.stdout.includes(expectedVersion)}`);
+  }
+}
+
+// ── Check 3: agent list ───────────────────────────────────────────────────────
+{
+  const r = run('crewx', ['agent', 'list'], { cwd: testDir, env });
+  if (r.status === 0) {
+    pass('crewx agent list');
+  } else {
+    fail('crewx agent list', r.stderr.slice(0, 200));
+  }
+}
+
+// ── Check 4: skill list ───────────────────────────────────────────────────────
+{
+  const r = run('crewx', ['skill', 'list'], { cwd: testDir, env });
+  if (r.status === 0 || r.stdout.includes('No skills')) {
+    pass('crewx skill list');
+  } else {
+    fail('crewx skill list', r.stderr.slice(0, 200));
+  }
+}
+
+// ── Check 5: memory --help ────────────────────────────────────────────────────
+{
+  const r = run('crewx', ['memory', '--help'], { cwd: testDir, env });
+  if (r.status === 0) {
+    pass('crewx memory --help');
+  } else {
+    fail('crewx memory --help', r.stderr.slice(0, 200));
+  }
+}
+
+// ── Check 6: workflow --help ──────────────────────────────────────────────────
+{
+  const r = run('crewx', ['workflow', '--help'], { cwd: testDir, env });
+  if (r.status === 0) {
+    pass('crewx workflow --help');
+  } else {
+    fail('crewx workflow --help', r.stderr.slice(0, 200));
+  }
+}
+
+// ── Check 7: search ───────────────────────────────────────────────────────────
+{
+  const r = run('crewx', ['search', 'test', '.'], { cwd: testDir, env, timeout: 10000 });
+  const noError = !r.stderr.includes('ERR_MODULE_NOT_FOUND') && !r.stderr.includes('is not a function');
+  if (noError) {
+    pass('crewx search (no crash)');
+  } else {
+    fail('crewx search', r.stderr.slice(0, 300));
+  }
+}
+
+// ── Check 8: crewx q "hi" (SDK loading check) ────────────────────────────────
+{
+  const r = run('crewx', ['q', 'hi'], { cwd: testDir, env, timeout: 15000 });
+  const combined = r.stdout + r.stderr;
+  const sdkErrors = ['ERR_MODULE_NOT_FOUND', 'is not a function', 'Cannot find module'];
+  const hasError = sdkErrors.some(e => combined.includes(e));
+  if (!hasError) {
+    pass('crewx q "hi" (no SDK import errors)');
+  } else {
+    const found = sdkErrors.find(e => combined.includes(e));
+    fail('crewx q "hi"', `SDK error: ${found}`);
+  }
+}
+
+// ── Check 9: crewx web startup ────────────────────────────────────────────────
+if (!skipWeb) {
+  const r = run('crewx', [], { cwd: testDir, env, timeout: 10000 });
+  const combined = r.stdout + r.stderr;
+  const hasModuleError = combined.includes('ERR_MODULE_NOT_FOUND');
+  if (!hasModuleError) {
+    pass('crewx (web) startup — no ERR_MODULE_NOT_FOUND');
+  } else {
+    fail('crewx (web) startup', 'ERR_MODULE_NOT_FOUND detected');
+  }
+} else {
+  console.log('  - SKIP  crewx web startup (--skip-web)');
+}
+
+// ── Check 10: SDK templates load (CJS require) ───────────────────────────────
+{
+  const sdkDir = join(installDir, 'node_modules', '@crewx', 'sdk');
+  const r = run('node', ['-e', `
+    const { existsSync, readdirSync } = require('fs');
+    const { join, dirname } = require('path');
+    const sdkMain = require.resolve('${sdkDir.replace(/\\/g, '/')}/dist/index.js');
+    const distDir = dirname(sdkMain);
+    const templatesPath = join(distDir, '../templates/agents');
+    if (!existsSync(templatesPath)) { process.stdout.write('FAIL: ' + templatesPath + '\\n'); process.exit(1); }
+    const files = readdirSync(templatesPath);
+    if (!files.includes('default.yaml')) { process.stdout.write('FAIL: default.yaml missing\\n'); process.exit(1); }
+    process.stdout.write('OK: ' + templatesPath + '\\n');
+  `], { cwd: testDir, env, timeout: 5000 });
+  if (r.status === 0 && r.stdout.includes('OK:')) {
+    pass('SDK templates load (CJS) — dist/index.js → ../templates/agents ✓');
+  } else {
+    fail('SDK templates load (CJS)', r.stderr.slice(0, 200) || r.stdout.slice(0, 200));
+  }
+}
+
+// ── Check 11: SDK templates load (ESM import — same dist/index.js via exports) ─
+{
+  const sdkDir = join(installDir, 'node_modules', '@crewx', 'sdk');
+  const r = run('node', ['--input-type=module', '--eval', `
+    import { existsSync, readdirSync } from 'fs';
+    import { join, dirname } from 'path';
+    import { createRequire } from 'module';
+    const require = createRequire(import.meta.url);
+    // "import" condition in package.json exports also points to dist/index.js (CJS)
+    const sdkMain = require.resolve('${sdkDir.replace(/\\/g, '/')}/dist/index.js');
+    const distDir = dirname(sdkMain);
+    const templatesPath = join(distDir, '../templates/agents');
+    if (!existsSync(templatesPath)) { process.stdout.write('FAIL: ' + templatesPath + '\\n'); process.exit(1); }
+    const files = readdirSync(templatesPath);
+    if (!files.includes('default.yaml')) { process.stdout.write('FAIL: default.yaml missing\\n'); process.exit(1); }
+    process.stdout.write('OK: ' + templatesPath + '\\n');
+  `], { cwd: testDir, env, timeout: 5000 });
+  if (r.status === 0 && r.stdout.includes('OK:')) {
+    pass('SDK templates load (ESM) — dist/index.js → ../templates/agents ✓');
+  } else {
+    fail('SDK templates load (ESM)', r.stderr.slice(0, 200) || r.stdout.slice(0, 200));
+  }
+}
+
+// ── Cleanup ───────────────────────────────────────────────────────────────────
+rmSync(testDir, { recursive: true, force: true });
+rmSync(installDir, { recursive: true, force: true });
+try { rmSync(tarball, { force: true }); } catch {}
+
+// ── Summary ───────────────────────────────────────────────────────────────────
+console.log('\n' + '─'.repeat(60));
+const passed = results.filter(r => r.status === 'PASS').length;
+const failed = results.filter(r => r.status === 'FAIL').length;
+
+if (!anyFail) {
+  console.log(`✅ ALL PASS (${passed}/${results.length})`);
+} else {
+  console.log(`❌ ${failed} FAIL / ${passed} PASS`);
+  results.filter(r => r.status === 'FAIL').forEach(r => {
+    console.error(`   FAIL: ${r.label}${r.detail ? ' — ' + r.detail : ''}`);
+  });
+}
+
+process.exit(anyFail ? 1 : 0);