@astryxdesign/cli 0.2.0 → 0.3.0

package/cli/commands/upgrade.integration-policy.test.mjs

@@ -1,204 +0,0 @@
-// Copyright (c) Meta Platforms, Inc. and affiliates.
-
-/**
- * @file Upgrade integration error-policy + --skip-codemod tests.
- *
- * These scaffold a real consumer project (astryx.config.mjs + an installed
- * integration package with codemods) under a REPO-LOCAL temp dir so Vite
- * permits dynamic import of the config/integration modules (it blocks /tmp).
- * They assert:
- *   - a broken integration codemod definition is SKIPPED (warned), not a hard
- *     fail of the upgrade (reverses the previous policy);
- *   - --skip-codemod excludes a named integration codemod.
- */
-
-import {describe, it, expect, beforeEach, afterEach, vi} from 'vitest';
-import * as fs from 'node:fs';
-import * as path from 'node:path';
-import {pathToFileURL} from 'node:url';
-import {Command} from 'commander';
-import {registerUpgrade} from './upgrade.mjs';
-
-let tmpDir;
-let originalCwd;
-let logCalls;
-let errCalls;
-let exitCode;
-
-const codemodModuleUrl = pathToFileURL(
-  path.resolve(process.cwd(), 'packages/cli/authoring/codemod.mjs'),
-).href;
-
-beforeEach(() => {
-  originalCwd = process.cwd();
-  // Repo-local temp dir: under cwd so Vite allows config/integration imports.
-  tmpDir = fs.mkdtempSync(path.join(process.cwd(), '.astryx-upgrade-policy-'));
-  process.chdir(tmpDir);
-  logCalls = [];
-  errCalls = [];
-  exitCode = undefined;
-  vi.spyOn(console, 'log').mockImplementation((...a) => logCalls.push(a.join(' ')));
-  vi.spyOn(console, 'error').mockImplementation((...a) => errCalls.push(a.join(' ')));
-  vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
-  vi.spyOn(process, 'exit').mockImplementation(code => {
-    exitCode = code;
-    throw new Error(`__exit ${code}`);
-  });
-});
-
-afterEach(() => {
-  process.chdir(originalCwd);
-  fs.rmSync(tmpDir, {recursive: true, force: true});
-  vi.restoreAllMocks();
-});
-
-function writeInstalledCore(version) {
-  const dir = path.join(tmpDir, 'node_modules', '@astryxdesign', 'core');
-  fs.mkdirSync(dir, {recursive: true});
-  fs.writeFileSync(
-    path.join(dir, 'package.json'),
-    JSON.stringify({name: '@astryxdesign/core', version}),
-  );
-}
-
-function writeSource() {
-  fs.mkdirSync(path.join(tmpDir, 'src'), {recursive: true});
-  fs.writeFileSync(path.join(tmpDir, 'src', 'index.ts'), 'const foo = 1;\n');
-}
-
-/**
- * Scaffold a consumer + an installed integration package with codemods.
- * @param {Object<string,string>} codemodFiles "<version>/<id>.mjs" -> body
- */
-function scaffoldIntegration(codemodFiles) {
-  fs.writeFileSync(
-    path.join(tmpDir, 'package.json'),
-    JSON.stringify({name: 'consumer'}),
-  );
-  fs.writeFileSync(
-    path.join(tmpDir, 'astryx.config.mjs'),
-    `export default { integrations: ['@acme/widgets'] };\n`,
-  );
-  const pkgDir = path.join(tmpDir, 'node_modules', '@acme', 'widgets');
-  fs.mkdirSync(pkgDir, {recursive: true});
-  fs.writeFileSync(
-    path.join(pkgDir, 'package.json'),
-    JSON.stringify({name: '@acme/widgets', version: '1.0.0'}),
-  );
-  fs.writeFileSync(
-    path.join(pkgDir, 'astryx.integration.mjs'),
-    `export default { codemods: './codemods' };\n`,
-  );
-  for (const [rel, body] of Object.entries(codemodFiles)) {
-    const full = path.join(pkgDir, 'codemods', rel);
-    fs.mkdirSync(path.dirname(full), {recursive: true});
-    fs.writeFileSync(full, body);
-  }
-}
-
-function createProgram() {
-  const program = new Command();
-  program.exitOverride();
-  program.option('--json', 'Output as typed JSON');
-  registerUpgrade(program);
-  return program;
-}
-
-async function runJson(args) {
-  const program = createProgram();
-  try {
-    await program.parseAsync(['node', 'astryx', ...args]);
-  } catch (err) {
-    if (!String(err?.message || '').startsWith('__exit')) throw err;
-  }
-  for (let i = logCalls.length - 1; i >= 0; i--) {
-    const line = logCalls[i];
-    if (line.startsWith('{')) {
-      try {
-        return JSON.parse(line);
-      } catch {
-        // keep looking
-      }
-    }
-  }
-  return null;
-}
-
-describe('upgrade integration error policy (skip + warn)', () => {
-  it('SKIPS a broken integration codemod instead of hard-failing the upgrade', async () => {
-    // A codemod module whose default export is not a valid codemod result —
-    // a DEFINITION error. The upgrade must NOT abort; it skips the broken
-    // integration's codemods and completes.
-    scaffoldIntegration({
-      '0.2.0/bad.mjs': `export default { not: 'a codemod' };\n`,
-    });
-    writeInstalledCore('0.2.0');
-    writeSource();
-
-    const result = await runJson([
-      '--json',
-      'upgrade',
-      '--from',
-      '0.1.0',
-      '--path',
-      'src',
-    ]);
-
-    // Did NOT hard-fail: no error envelope, and it ran to a status/run result.
-    expect(result).not.toBeNull();
-    expect(result.error).toBeUndefined();
-    expect(exitCode).not.toBe(1);
-  });
-
-  it('runs a healthy integration codemod for an applicable range', async () => {
-    scaffoldIntegration({
-      '0.2.0/drop-foo.mjs':
-        `import {createCodemod} from ${JSON.stringify(codemodModuleUrl)};\n` +
-        `export default createCodemod({ title: 'Drop foo', transform: (file) => file.source.replace(/foo/g, 'bar') });\n`,
-    });
-    writeInstalledCore('0.2.0');
-    writeSource();
-
-    const result = await runJson([
-      '--json',
-      'upgrade',
-      '--from',
-      '0.1.0',
-      '--path',
-      'src',
-      '--apply',
-    ]);
-    expect(result).not.toBeNull();
-    expect(result.error).toBeUndefined();
-    // The codemod rewrote foo -> bar.
-    const out = fs.readFileSync(path.join(tmpDir, 'src', 'index.ts'), 'utf-8');
-    expect(out).toContain('bar');
-  });
-
-  it('--skip-codemod excludes a named integration codemod', async () => {
-    scaffoldIntegration({
-      '0.2.0/drop-foo.mjs':
-        `import {createCodemod} from ${JSON.stringify(codemodModuleUrl)};\n` +
-        `export default createCodemod({ title: 'Drop foo', transform: (file) => file.source.replace(/foo/g, 'bar') });\n`,
-    });
-    writeInstalledCore('0.2.0');
-    writeSource();
-
-    const result = await runJson([
-      '--json',
-      'upgrade',
-      '--from',
-      '0.1.0',
-      '--path',
-      'src',
-      '--apply',
-      '--skip-codemod',
-      'drop-foo',
-    ]);
-    expect(result).not.toBeNull();
-    // Skipped: the source is unchanged (foo not rewritten to bar).
-    const out = fs.readFileSync(path.join(tmpDir, 'src', 'index.ts'), 'utf-8');
-    expect(out).toContain('foo');
-    expect(out).not.toContain('bar');
-  });
-});

package/cli/commands/upgrade.mjs

@@ -1,67 +0,0 @@
-// Copyright (c) Meta Platforms, Inc. and affiliates.
-
-/**
- * @file upgrade command — thin wrapper around api/upgrade.
- *
- * `astryx upgrade` runs codemods that migrate source between Astryx versions and
- * refreshes the managed agent-docs block. All logic lives in api/upgrade; this
- * handler only parses flags, wires the progress logger (term output in human
- * mode, silent in --json), and renders the result / exit code.
- *
- * Consumers bump/install their Astryx packages first, then run:
- *   astryx upgrade --from <old-version> --path <source-dir> --apply
- */
-
-import {jsonOut, jsonError} from '../../lib/json.mjs';
-import {termLogger, noopLogger} from '../../lib/term-log.mjs';
-import {AstryxError} from '../../api/error.mjs';
-import {upgrade as upgradeApi} from '../../api/upgrade/upgrade.mjs';
-
-/**
- * @param {import('commander').Command} program
- */
-export function registerUpgrade(program) {
-  program
-    .command('upgrade')
-    .description('Run codemods to migrate between versions')
-    .option('--from <version>', 'Previous version before the dependency upgrade')
-    .option('--apply', 'Write changes to disk (default: dry-run)', false)
-    .option('--force', 'Run codemods even if --from is newer than the installed version', false)
-    .option('--codemod <name>', 'Run a specific transform only')
-    .option('--skip-codemod <name...>', 'Exclude named codemods (repeatable). Re-run past a failed codemod by skipping it.')
-    .option('--integration <package-or-file>', 'Explicit integration package name or integration file path (repeatable)',
-      /** @param {string} value @param {string[]} previous */
-      (value, previous) => [...(previous ?? []), value], [])
-    .option('--path <dir>', 'Source directory to scan', './src')
-    .option('--install-deps', 'Auto-install jscodeshift without prompting', false)
-    .option('--list', 'List available codemods', false)
-    .action(
-      /**
-       * @param {import('../../api/upgrade/upgrade.mjs').UpgradeOptions} options
-       */
-      async options => {
-        const json = program.opts().json || false;
-        const logger = json ? noopLogger : termLogger;
-
-        /** @type {import('../../types/upgrade').UpgradeListResponse | import('../../types/upgrade').UpgradeStatusResponse | import('../../types/upgrade').UpgradeRunResponse} */
-        let result;
-        try {
-          result = await upgradeApi(options, {cwd: process.cwd(), logger});
-        } catch (e) {
-          // Handled failures throw AstryxError: --json emits the structured
-          // envelope (byte-identical to the old inline jsonError) + exits 1;
-          // human mode already saw the error line + outro via the logger, so we
-          // just set the exit code. Anything else is unexpected — let the
-          // top-level boundary in bin/astryx.mjs format it as before.
-          if (e instanceof AstryxError) {
-            if (json) jsonError(e.message, undefined, e.code);
-            else process.exitCode = 1;
-            return;
-          }
-          throw e;
-        }
-
-        if (json) jsonOut(result);
-      },
-    );
-}

package/cli/commands/upgrade.test.mjs

@@ -1,314 +0,0 @@
-// Copyright (c) Meta Platforms, Inc. and affiliates.
-
-import {describe, it, expect, beforeEach, afterEach, vi} from 'vitest';
-import * as fs from 'node:fs';
-import * as path from 'node:path';
-import * as os from 'node:os';
-import {Command} from 'commander';
-import {registerUpgrade} from './upgrade.mjs';
-import {generateCompressedIndex} from '../../lib/agent-docs/agent-docs.mjs';
-
-let tmpDir;
-let originalCwd;
-let logCalls;
-let stdoutCalls;
-let exitCode;
-
-beforeEach(() => {
-  tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'astryx-upgrade-test-'));
-  originalCwd = process.cwd();
-  process.chdir(tmpDir);
-  logCalls = [];
-  stdoutCalls = [];
-  exitCode = undefined;
-  vi.spyOn(console, 'log').mockImplementation((...args) => {
-    logCalls.push(args.join(' '));
-  });
-  vi.spyOn(console, 'error').mockImplementation(() => {});
-  // Some human logs are written straight to process.stdout — capture that too.
-  vi.spyOn(process.stdout, 'write').mockImplementation((chunk) => {
-    stdoutCalls.push(typeof chunk === 'string' ? chunk : chunk.toString());
-    return true;
-  });
-  // jsonError() calls process.exit(1) directly. Trap it so tests can assert.
-  vi.spyOn(process, 'exit').mockImplementation((code) => {
-    exitCode = code;
-    throw new Error(`__exit ${code}`);
-  });
-});
-
-afterEach(() => {
-  process.chdir(originalCwd);
-  fs.rmSync(tmpDir, {recursive: true, force: true});
-  vi.restoreAllMocks();
-});
-
-function createProgram() {
-  const program = new Command();
-  program.exitOverride();
-  // Mirror the global --json flag from src/index.mjs so program.opts().json
-  // resolves the same way in tests.
-  program.option('--json', 'Output as typed JSON');
-  registerUpgrade(program);
-  return program;
-}
-
-function writePkg(deps = {}) {
-  fs.writeFileSync(
-    path.join(tmpDir, 'package.json'),
-    JSON.stringify({name: 'fixture', dependencies: deps}, null, 2),
-  );
-}
-
-function writeInstalledCore(version, packageName = '@astryxdesign/core') {
-  const parts = packageName.split('/');
-  const dir = path.join(tmpDir, 'node_modules', ...parts);
-  fs.mkdirSync(dir, {recursive: true});
-  fs.writeFileSync(
-    path.join(dir, 'package.json'),
-    JSON.stringify({name: packageName, version}, null, 2),
-  );
-}
-
-function writeSourceFile() {
-  fs.mkdirSync(path.join(tmpDir, 'src'), {recursive: true});
-  fs.writeFileSync(path.join(tmpDir, 'src', 'index.ts'), 'const x = 1;\n');
-}
-
-/** Run a command and capture the parsed JSON response (last printed JSON line). */
-async function runJson(args) {
-  const program = createProgram();
-  try {
-    await program.parseAsync(['node', 'astryx', ...args]);
-  } catch (err) {
-    if (!String(err?.message || '').startsWith('__exit')) throw err;
-  }
-  // Find the most recent stringified JSON envelope in console.log output.
-  for (let i = logCalls.length - 1; i >= 0; i--) {
-    const line = logCalls[i];
-    if (line.startsWith('{')) {
-      try {
-        return JSON.parse(line);
-      } catch {
-        // not JSON, keep looking
-      }
-    }
-  }
-  return null;
-}
-
-describe('upgrade gate (semver comparison)', () => {
-  it('does NOT block an upgrade from 0.0.9 to installed 0.0.10 (regression)', async () => {
-    // The original bug: string compare said '0.0.9' >= '0.0.10', so the
-    // gate told users "Already up to date" without --force.
-    writePkg();
-    writeInstalledCore('0.0.10');
-    writeSourceFile();
-
-    const result = await runJson(['--json', 'upgrade', '--from', '0.0.9', '--path', 'src']);
-    expect(result).not.toBeNull();
-    expect(result.type === 'upgrade.run' || result.error || logCalls.some(l => l.includes('No codemods'))).toBeTruthy();
-  });
-
-  it('blocks when --from >= installed target by semver (e.g. 0.0.10 → 0.0.9)', async () => {
-    writePkg();
-    writeInstalledCore('0.0.9');
-    const program = createProgram();
-    await program.parseAsync(['node', 'astryx', 'upgrade', '--from', '0.0.10']);
-    const output = stdoutCalls.join('') + logCalls.join('\n');
-    expect(output).toMatch(/up to date|Already/i);
-  });
-});
-
-describe('upgrade argument validation', () => {
-  it('requires --from for upgrade runs', async () => {
-    writePkg();
-    writeInstalledCore('0.0.15');
-    const result = await runJson(['--json', 'upgrade']);
-    expect(result).not.toBeNull();
-    expect(result.error).toMatch(/Missing required --from/);
-    expect(exitCode).toBe(1);
-  });
-
-  it('rejects bogus --from values', async () => {
-    writePkg();
-    writeInstalledCore('0.0.15');
-    const result = await runJson(['--json', 'upgrade', '--from', 'not-a-version']);
-    expect(result).not.toBeNull();
-    expect(result.error).toMatch(/Invalid --from/);
-    expect(exitCode).toBe(1);
-  });
-
-  it('detects the installed target version from @astryxdesign/core', async () => {
-    writePkg();
-    writeInstalledCore('0.0.15');
-    writeSourceFile();
-    const result = await runJson(['--json', 'upgrade', '--from', '0.0.14', '--path', 'src']);
-    expect(result?.error || '').not.toMatch(/Could not find installed/);
-  });
-});
-
-describe('upgrade --list dedup', () => {
-  it('lists each codemod exactly once', async () => {
-    const result = await runJson(['--json', 'upgrade', '--list']);
-    expect(result).not.toBeNull();
-    expect(result.type).toBe('upgrade.list');
-    const names = result.data.map((c) => c.name);
-    const unique = new Set(names);
-    // The bug: 31 unique codemods printed 9× → 201 entries.
-    expect(names.length).toBe(unique.size);
-  });
-});
-
-// Issue #4168 — `upgrade` must refresh the managed agent-docs block after a
-// version bump, on EVERY path (the block documents the installed library, not
-// the codemod outcome), and must NOT write during a dry run.
-describe('upgrade agent-docs refresh (#4168)', () => {
-  /** Write an agent-doc file with a managed block generated for `version`. */
-  function writeAgentBlock(rel, version) {
-    const filePath = path.join(tmpDir, rel);
-    fs.mkdirSync(path.dirname(filePath), {recursive: true});
-    fs.writeFileSync(filePath, `# Doc\n\n${generateCompressedIndex(version)}\n`);
-  }
-
-  it('refreshes a stale block with --apply, even when there are no codemods (up-to-date path)', async () => {
-    writePkg();
-    writeInstalledCore('0.0.15');
-    writeAgentBlock('AGENTS.md', '0.0.1');
-
-    // --from == installed → up-to-date short-circuit: no codemods run, yet the
-    // stale block must still be brought current. This is the exact gap in #4168.
-    const result = await runJson(['--json', 'upgrade', '--from', '0.0.15', '--apply']);
-
-    expect(result.type).toBe('upgrade.status');
-    expect(result.data.status).toBe('up_to_date');
-    expect(result.data.agentDocs.status).toBe('stale');
-    expect(result.data.agentDocs.action).toBe('refreshed');
-    expect(result.data.agentDocs.refreshed).toBe(true);
-    expect(result.data.agentDocs.fromVersions).toEqual(['0.0.1']);
-    expect(result.data.agentDocs.files).toContain('AGENTS.md');
-
-    const content = fs.readFileSync(path.join(tmpDir, 'AGENTS.md'), 'utf-8');
-    expect(content).toMatch(/Astryx v0\.0\.15 ·/);
-    expect(content).not.toMatch(/Astryx v0\.0\.1 ·/);
-  });
-
-  it('reports a stale block on a dry run WITHOUT writing', async () => {
-    writePkg();
-    writeInstalledCore('0.0.15');
-    writeAgentBlock('AGENTS.md', '0.0.1');
-    const before = fs.readFileSync(path.join(tmpDir, 'AGENTS.md'), 'utf-8');
-
-    const result = await runJson(['--json', 'upgrade', '--from', '0.0.15']); // no --apply
-
-    expect(result.data.agentDocs.status).toBe('stale');
-    expect(result.data.agentDocs.action).toBe('would-refresh');
-    expect(result.data.agentDocs.refreshed).toBe(false);
-    // The file must be untouched by a dry run.
-    expect(fs.readFileSync(path.join(tmpDir, 'AGENTS.md'), 'utf-8')).toBe(before);
-  });
-
-  it('refreshes a stale block on the no-codemods path (--apply)', async () => {
-    writePkg();
-    writeInstalledCore('0.0.11'); // (0.0.10, 0.0.11] has no registered codemods
-    writeSourceFile();
-    writeAgentBlock('CLAUDE.md', '0.0.1');
-
-    const result = await runJson(['--json', 'upgrade', '--from', '0.0.10', '--apply', '--path', 'src']);
-
-    expect(result.type).toBe('upgrade.status');
-    expect(result.data.status).toBe('no_codemods');
-    expect(result.data.agentDocs.refreshed).toBe(true);
-    expect(fs.readFileSync(path.join(tmpDir, 'CLAUDE.md'), 'utf-8')).toMatch(/Astryx v0\.0\.11 ·/);
-  });
-
-  it('nudges to run init when core is installed but no managed block exists', async () => {
-    writePkg();
-    writeInstalledCore('0.0.15');
-    // No agent-doc files at all.
-
-    const result = await runJson(['--json', 'upgrade', '--from', '0.0.15', '--apply']);
-
-    expect(result.data.agentDocs.status).toBe('missing');
-    expect(result.data.agentDocs.action).toBe('nudge-init');
-    expect(result.data.agentDocs.refreshed).toBe(false);
-    // Must NOT silently create docs during an upgrade.
-    expect(fs.existsSync(path.join(tmpDir, 'AGENTS.md'))).toBe(false);
-    expect(fs.existsSync(path.join(tmpDir, '.claude', 'CLAUDE.md'))).toBe(false);
-  });
-
-  it('treats an agent file without our markers as "never initialized" (missing)', async () => {
-    writePkg();
-    writeInstalledCore('0.0.15');
-    fs.writeFileSync(path.join(tmpDir, 'AGENTS.md'), '# Agents\n\nHand-written, no astryx block.\n');
-    const before = fs.readFileSync(path.join(tmpDir, 'AGENTS.md'), 'utf-8');
-
-    const result = await runJson(['--json', 'upgrade', '--from', '0.0.15', '--apply']);
-
-    expect(result.data.agentDocs.status).toBe('missing');
-    expect(result.data.agentDocs.action).toBe('nudge-init');
-    // An unmarked file is left exactly as-is.
-    expect(fs.readFileSync(path.join(tmpDir, 'AGENTS.md'), 'utf-8')).toBe(before);
-  });
-
-  it('does not silently no-op a stale block with malformed markers (START without END)', async () => {
-    writePkg();
-    writeInstalledCore('0.0.15');
-    // A corrupted block: START + version header but no matching END marker, so
-    // the writer can't safely splice it. This must not crash, corrupt the file,
-    // or falsely claim success — it should surface an error and nudge re-init.
-    const corrupted =
-      '# A\n\n<!-- ASTRYX:START -->\nAstryx v0.0.1 · 9 components\nguidance, but no end marker\n';
-    fs.writeFileSync(path.join(tmpDir, 'AGENTS.md'), corrupted);
-
-    const result = await runJson(['--json', 'upgrade', '--from', '0.0.15', '--apply']);
-
-    expect(result.data.agentDocs.status).toBe('stale');
-    expect(result.data.agentDocs.action).toBe('error');
-    expect(result.data.agentDocs.refreshed).toBe(false);
-    // File left byte-for-byte intact — never corrupted.
-    expect(fs.readFileSync(path.join(tmpDir, 'AGENTS.md'), 'utf-8')).toBe(corrupted);
-  });
-
-  it('stays silent when the block already matches the installed version', async () => {
-    writePkg();
-    writeInstalledCore('0.0.15');
-    writeAgentBlock('AGENTS.md', '0.0.15');
-
-    const result = await runJson(['--json', 'upgrade', '--from', '0.0.15', '--apply']);
-
-    expect(result.data.agentDocs.status).toBe('current');
-    expect(result.data.agentDocs.action).toBe('none');
-    expect(result.data.agentDocs.refreshed).toBe(false);
-  });
-
-  it('migrates a legacy XDS block to the current Astryx block (--apply)', async () => {
-    writePkg();
-    writeInstalledCore('0.0.15');
-    fs.writeFileSync(
-      path.join(tmpDir, 'CLAUDE.md'),
-      '# Claude\n\n<!-- XDS:START -->\nlegacy index\n<!-- XDS:END -->\n\nMore rules.\n',
-    );
-
-    const result = await runJson(['--json', 'upgrade', '--from', '0.0.15', '--apply']);
-
-    expect(result.data.agentDocs.status).toBe('stale');
-    expect(result.data.agentDocs.refreshed).toBe(true);
-    const content = fs.readFileSync(path.join(tmpDir, 'CLAUDE.md'), 'utf-8');
-    expect(content).toContain('<!-- ASTRYX:START -->');
-    expect(content).toMatch(/Astryx v0\.0\.15 ·/);
-    expect(content).not.toContain('legacy index');
-    expect(content).toContain('More rules.');
-  });
-
-  it('surfaces agentDocs in the human output on the up-to-date path', async () => {
-    writePkg();
-    writeInstalledCore('0.0.15');
-    writeAgentBlock('AGENTS.md', '0.0.1');
-
-    const program = createProgram();
-    await program.parseAsync(['node', 'astryx', 'upgrade', '--from', '0.0.15', '--apply']);
-    const output = stdoutCalls.join('') + logCalls.join('\n');
-    expect(output).toMatch(/Agent docs refreshed/i);
-  });
-});

package/cli/commands/validate-integration.mjs

@@ -1,90 +0,0 @@
-// Copyright (c) Meta Platforms, Inc. and affiliates.
-
-/**
- * @file `astryx validate-integration` command — validate ONE integration
- * package's manifest and contributions and report findings using the
- * AstryxIntegrationIssue model.
- *
- *   astryx validate-integration            validate the local package (cwd)
- *   astryx validate-integration <pkg>      validate an installed package
- *
- * Exit code is the contract: 0 when there are no severity:'error' issues
- * (warnings are fine), 1 when any error issue is present — so it works as a CI
- * gate. The no-arg + no-local-manifest case prints guidance and exits 0 (not
- * an integration package is not a failure).
- */
-
-import {jsonOut, humanLog} from '../../lib/json.mjs';
-import {
-  validateIntegration,
-  summarizeIssues,
-} from '../../api/integration/validate-integration.mjs';
-
-/**
- * Render a validation result for humans.
- * @param {import('../../types/validate-integration').ValidateIntegrationResponse['data']} data
- */
-function printHuman(data) {
-  const label =
-    data.version != null ? `${data.name}@${data.version}` : data.name;
-  humanLog(`Validating integration: ${label}`);
-
-  if (data.issues.length === 0) {
-    humanLog('\n\u2713 No issues found.');
-    return;
-  }
-
-  humanLog('');
-  for (const issue of data.issues) {
-    humanLog(`  ${issue.severity} ${issue.code}: ${issue.message}`);
-  }
-
-  const {errors, warnings} = summarizeIssues(data.issues);
-  humanLog(
-    `\n${data.issues.length} issue(s): ${errors} error(s), ${warnings} warning(s)`,
-  );
-}
-
-const NO_MANIFEST_GUIDANCE =
-  'No astryx.integration.* found next to package.json. ' +
-  'To validate an installed integration: astryx validate-integration <package>';
-
-/**
- * Register the `astryx validate-integration` command.
- * @param {import('commander').Command} program
- */
-export function registerValidateIntegration(program) {
-  program
-    .command('validate-integration [package]')
-    .description(
-      'Validate an Astryx integration package (manifest + contributions)',
-    )
-    .addHelpText(
-      'after',
-      '\nWith no argument, validates the integration package rooted at the\n' +
-        'current directory. Pass a package name to validate an installed\n' +
-        'integration resolved from ./node_modules.\n\n' +
-        'Exit code:\n' +
-        '  0  no error issues (warnings are allowed) — safe as a CI gate\n' +
-        '  1  one or more error issues\n',
-    )
-    .action(async pkg => {
-      const json = program.opts().json || false;
-
-      const result = await validateIntegration(pkg);
-
-      if (json) {
-        jsonOut(result);
-      } else if (result.data.name === null) {
-        // No-arg + no local manifest: guidance, not an error.
-        humanLog(NO_MANIFEST_GUIDANCE);
-      } else {
-        printHuman(result.data);
-      }
-
-      const {errors} = summarizeIssues(result.data.issues);
-      if (errors > 0) {
-        process.exitCode = 1;
-      }
-    });
-}