@astryxdesign/cli 0.1.7 → 0.1.8-canary.07362a9

package/src/types/theme.d.ts

@@ -1,23 +0,0 @@
-// Copyright (c) Meta Platforms, Inc. and affiliates.
-
-/**
- * Theme build command JSON responses.
- *
- * Invocation                                 -> type discriminator
- * ------------------------------------------------------------------
- * xds --json theme build <file>             -> theme.build
- * (file not found / parse error)            -> CLIError
- */
-
-/** xds --json theme build <file> */
-export interface ThemeBuildResponse {
-  type: 'theme.build';
-  data: {
-    name: string;
-    tokenCount: number;
-    componentCount: number;
-    sizeKB: number;
-    outputs: {css: string; js: string; dts: string; variantsDts?: string};
-    warnings: string[];
-  };
-}

package/src/types/upgrade.d.ts

@@ -1,64 +0,0 @@
-// Copyright (c) Meta Platforms, Inc. and affiliates.
-
-/**
- * Upgrade command JSON responses.
- *
- * Invocation                                 -> type discriminator
- * ------------------------------------------------------------------
- * xds --json upgrade --list                 -> upgrade.list
- * xds --json upgrade [--apply]              -> upgrade.run
- * xds --json upgrade (status short-circuit) -> upgrade.status
- * (version detection failure)               -> CLIError
- */
-
-/** xds --json upgrade --list */
-export interface UpgradeListResponse {
-  type: 'upgrade.list';
-  data: UpgradeListEntry[];
-}
-
-export interface UpgradeListEntry {
-  name: string;
-  title: string;
-  version: string;
-}
-
-/** xds --json upgrade [--apply] */
-export interface UpgradeRunResponse {
-  type: 'upgrade.run';
-  data: {
-    from: string;
-    to: string;
-    codemods: number;
-    depsUpdated: string[];
-    agentDocsRefreshed: boolean;
-  };
-}
-
-/**
- * xds --json upgrade — short-circuit status results.
- *
- * - `up_to_date`: `--from` is >= installed target and `--force` was not passed.
- * - `no_codemods`: no codemods (core or integration) apply to the range.
- * - `config_fixable`: DRY-RUN ONLY. The consumer's astryx.config currently
- *   fails strict validation, but a pending core CONFIG codemod (in the selected
- *   range) would repair it. The dry run previews the fix without writing and
- *   reports the exact command to apply it; integrations are skipped for the
- *   preview (they will be processed on the `--apply` run).
- */
-export interface UpgradeStatusResponse {
-  type: 'upgrade.status';
-  data:
-    | {status: 'up_to_date'; from: string; to: string}
-    | {status: 'no_codemods'; from: string; to: string}
-    | {
-        status: 'config_fixable';
-        from: string;
-        to: string;
-        configError: string;
-        configCodemods: string[];
-        suggestedCommand: string;
-        message: string;
-        note: string;
-      };
-}

package/src/utils/interactive.mjs

@@ -1,76 +0,0 @@
-// Copyright (c) Meta Platforms, Inc. and affiliates.
-
-/**
- * @file Interactivity contract for the CLI.
- *
- * A single source of truth for "can this process prompt the user?". Several
- * commands launch @clack/prompts wizards; in a non-interactive context (CI,
- * piped stdin/stdout, no TTY) those prompts block forever. Historically each
- * command answered this question differently — some checked `stdout.isTTY`,
- * one checked `stdin.isTTY`, one added `!process.env.CI`, and some had no
- * guard at all. This module centralizes the check so every command behaves
- * identically.
- *
- * Two entry points, for two situations:
- *
- *   - `requireInteractive()` — for commands whose prompt IS the work
- *     (e.g. `astryx init`, `astryx theme`). With no TTY there is nothing to do, so
- *     fail fast (exit 1) with actionable, non-interactive guidance.
- *
- *   - `isInteractive()` — for commands with an OPTIONAL secondary prompt
- *     that runs after the primary work has already succeeded; callers use
- *     this to skip the prompt gracefully in non-interactive contexts.
- */
-
-/**
- * True when the process can safely run an interactive prompt.
- *
- * Requires BOTH stdin and stdout to be TTYs (clack reads stdin and renders to
- * stdout) and that we are not in a CI environment. A CI runner may allocate a
- * pseudo-TTY, so `process.env.CI` is an explicit override: never prompt in CI.
- *
- * @param {object} [env] - Override hook for tests.
- * @param {boolean} [env.stdinTTY=process.stdin.isTTY]
- * @param {boolean} [env.stdoutTTY=process.stdout.isTTY]
- * @param {boolean} [env.ci=Boolean(process.env.CI)]
- * @returns {boolean}
- */
-export function isInteractive({
-  stdinTTY = Boolean(process.stdin && process.stdin.isTTY),
-  stdoutTTY = Boolean(process.stdout && process.stdout.isTTY),
-  ci = Boolean(process.env.CI),
-} = {}) {
-  if (ci) return false;
-  return stdinTTY && stdoutTTY;
-}
-
-/**
- * Guard for commands whose primary action is an interactive wizard. When the
- * process is non-interactive, prints an actionable error and exits 1 instead
- * of hanging on a prompt that will never receive input.
- *
- * @param {object} options
- * @param {string} options.command - Command name for the message, e.g. 'init'.
- * @param {string} options.hint - Concrete non-interactive invocation, e.g.
- *   '`pnpm astryx init --all` or `--features agents,theme,template`'.
- * @param {boolean} [options.json=false] - When true, the command does not
- *   support --json; we still exit 1 but skip the human-formatted guidance.
- * @param {object} [env] - Forwarded to isInteractive (test hook).
- * @returns {void} Returns when interactive; otherwise calls process.exit(1).
- */
-export function requireInteractive({command, hint, json = false} = {}, env) {
-  if (isInteractive(env)) return;
-  const name = command ? `astryx ${command}` : 'this command';
-  console.error(
-    `Error: \`${name}\` with no flags is interactive and requires a TTY.`,
-  );
-  if (hint) {
-    console.error(`Run non-interactively with: ${hint}`);
-  }
-  if (!json) {
-    console.error(
-      'Detected a non-interactive environment (no TTY, piped I/O, or CI=1).',
-    );
-  }
-  process.exit(1);
-}

package/src/utils/interactive.test.mjs

@@ -1,70 +0,0 @@
-// Copyright (c) Meta Platforms, Inc. and affiliates.
-
-/**
- * @file Unit tests for the interactivity contract.
- *
- * isInteractive() is pure given its injected env, so these are fast unit
- * tests. The "does the command actually fail fast instead of hanging" proof
- * lives in the per-command subprocess tests (init/theme non-interactive).
- */
-
-import {describe, it, expect, vi, afterEach} from 'vitest';
-import {isInteractive, requireInteractive} from './interactive.mjs';
-
-describe('isInteractive', () => {
-  it('is true only when stdin AND stdout are TTYs and not CI', () => {
-    expect(isInteractive({stdinTTY: true, stdoutTTY: true, ci: false})).toBe(true);
-  });
-
-  it('is false when stdin is not a TTY (piped input)', () => {
-    expect(isInteractive({stdinTTY: false, stdoutTTY: true, ci: false})).toBe(false);
-  });
-
-  it('is false when stdout is not a TTY (piped output)', () => {
-    expect(isInteractive({stdinTTY: true, stdoutTTY: false, ci: false})).toBe(false);
-  });
-
-  it('is false in CI even with a pseudo-TTY on both streams', () => {
-    expect(isInteractive({stdinTTY: true, stdoutTTY: true, ci: true})).toBe(false);
-  });
-});
-
-describe('requireInteractive', () => {
-  afterEach(() => {
-    vi.restoreAllMocks();
-  });
-
-  it('returns (does not exit) when interactive', () => {
-    const exit = vi.spyOn(process, 'exit').mockImplementation(() => {
-      throw new Error('exit should not be called');
-    });
-    expect(() =>
-      requireInteractive(
-        {command: 'init', hint: '`astryx init --all`'},
-        {stdinTTY: true, stdoutTTY: true, ci: false},
-      ),
-    ).not.toThrow();
-    expect(exit).not.toHaveBeenCalled();
-  });
-
-  it('exits 1 with actionable guidance when non-interactive', () => {
-    const exit = vi
-      .spyOn(process, 'exit')
-      .mockImplementation(() => {
-        throw new Error('__exit__');
-      });
-    const err = vi.spyOn(console, 'error').mockImplementation(() => {});
-    expect(() =>
-      requireInteractive(
-        {command: 'theme', hint: '`astryx theme <preset>`'},
-        {stdinTTY: false, stdoutTTY: false, ci: false},
-      ),
-    ).toThrow('__exit__');
-    expect(exit).toHaveBeenCalledWith(1);
-    const output = err.mock.calls.map(c => c.join(' ')).join('\n');
-    expect(output).toMatch(/requires a TTY/i);
-    expect(output).toMatch(/astryx theme <preset>/);
-    expect(output).toMatch(/`astryx theme`/);
-    expect(output).not.toMatch(/\bxds\b/);
-  });
-});

package/src/utils/package-manager.mjs

@@ -1,74 +0,0 @@
-// Copyright (c) Meta Platforms, Inc. and affiliates.
-
-/**
- * @file Detect the project's package manager from lockfiles.
- *
- * Returns the correct command prefix for running package binaries
- * (e.g. 'npx astryx', 'yarn astryx', 'pnpm exec astryx').
- */
-
-import * as fs from 'node:fs';
-import * as path from 'node:path';
-
-/**
- * Detect the package manager used in a project directory.
- * Walks up from targetDir looking for lockfiles.
- *
- * @param {string} [targetDir=process.cwd()]
- * @returns {'yarn'|'pnpm'|'bun'|'npm'}
- */
-export function detectPackageManager(targetDir = process.cwd()) {
-  const KNOWN_PMS = new Set(['yarn', 'pnpm', 'bun', 'npm']);
-  let dir = path.resolve(targetDir);
-  const root = path.parse(dir).root;
-
-  while (dir !== root) {
-    // 1. Lockfiles (highest priority)
-    if (fs.existsSync(path.join(dir, 'yarn.lock'))) return 'yarn';
-    if (fs.existsSync(path.join(dir, 'pnpm-lock.yaml'))) return 'pnpm';
-    if (fs.existsSync(path.join(dir, 'bun.lockb')) || fs.existsSync(path.join(dir, 'bun.lock'))) return 'bun';
-    if (fs.existsSync(path.join(dir, 'package-lock.json'))) return 'npm';
-
-    // 2. packageManager field in package.json
-    const pkgPath = path.join(dir, 'package.json');
-    if (fs.existsSync(pkgPath)) {
-      try {
-        const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf-8'));
-        if (pkg.packageManager) {
-          const name = pkg.packageManager.split('@')[0];
-          if (KNOWN_PMS.has(name)) return name;
-        }
-      } catch {
-        // Best-effort: unreadable/invalid package.json — keep walking up.
-      }
-    }
-
-    dir = path.dirname(dir);
-  }
-
-  // 3. npm_config_user_agent env var (set by all PMs when running scripts)
-  const ua = process.env.npm_config_user_agent;
-  if (ua) {
-    const name = ua.split('/')[0];
-    if (KNOWN_PMS.has(name)) return name;
-  }
-
-  return 'npx';
-}
-
-/**
- * Get the command prefix for running a package binary.
- *
- * @param {string} [targetDir]
- * @returns {string} e.g. 'npx', 'yarn', 'pnpm exec', 'bunx'
- */
-export function getRunPrefix(targetDir) {
-  const pm = detectPackageManager(targetDir);
-  switch (pm) {
-    case 'yarn': return 'yarn';
-    case 'pnpm': return 'pnpm exec';
-    case 'bun': return 'bunx';
-    case 'npm':
-    default: return 'npx';
-  }
-}

package/src/utils/package-manager.test.mjs

@@ -1,113 +0,0 @@
-// Copyright (c) Meta Platforms, Inc. and affiliates.
-
-import {describe, it, expect, afterEach, vi} from 'vitest';
-import * as fs from 'node:fs';
-import * as path from 'node:path';
-import * as os from 'node:os';
-import {detectPackageManager} from './package-manager.mjs';
-
-let tmpDir;
-
-afterEach(() => {
-  if (tmpDir) {
-    fs.rmSync(tmpDir, {recursive: true, force: true});
-    tmpDir = null;
-  }
-  vi.restoreAllMocks();
-  delete process.env.npm_config_user_agent;
-});
-
-function makeTmpDir() {
-  tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'astryx-pm-test-'));
-  return tmpDir;
-}
-
-describe('detectPackageManager', () => {
-  it('detects yarn from yarn.lock', () => {
-    const dir = makeTmpDir();
-    fs.writeFileSync(path.join(dir, 'yarn.lock'), '');
-    expect(detectPackageManager(dir)).toBe('yarn');
-  });
-
-  it('detects pnpm from pnpm-lock.yaml', () => {
-    const dir = makeTmpDir();
-    fs.writeFileSync(path.join(dir, 'pnpm-lock.yaml'), '');
-    expect(detectPackageManager(dir)).toBe('pnpm');
-  });
-
-  it('detects npm from package-lock.json', () => {
-    const dir = makeTmpDir();
-    fs.writeFileSync(path.join(dir, 'package-lock.json'), '{}');
-    expect(detectPackageManager(dir)).toBe('npm');
-  });
-
-  it('detects bun from bun.lockb', () => {
-    const dir = makeTmpDir();
-    fs.writeFileSync(path.join(dir, 'bun.lockb'), '');
-    expect(detectPackageManager(dir)).toBe('bun');
-  });
-
-  it('detects from packageManager field in package.json (no lockfile)', () => {
-    const dir = makeTmpDir();
-    fs.writeFileSync(
-      path.join(dir, 'package.json'),
-      JSON.stringify({packageManager: 'yarn@4.1.0'}),
-    );
-    expect(detectPackageManager(dir)).toBe('yarn');
-  });
-
-  it('detects pnpm from packageManager field', () => {
-    const dir = makeTmpDir();
-    fs.writeFileSync(
-      path.join(dir, 'package.json'),
-      JSON.stringify({packageManager: 'pnpm@8.0.0'}),
-    );
-    expect(detectPackageManager(dir)).toBe('pnpm');
-  });
-
-  it('ignores unknown packageManager values', () => {
-    const dir = makeTmpDir();
-    fs.writeFileSync(
-      path.join(dir, 'package.json'),
-      JSON.stringify({packageManager: 'unknown@1.0.0'}),
-    );
-    expect(detectPackageManager(dir)).toBe('npx');
-  });
-
-  it('lockfile takes priority over packageManager field', () => {
-    const dir = makeTmpDir();
-    fs.writeFileSync(path.join(dir, 'yarn.lock'), '');
-    fs.writeFileSync(
-      path.join(dir, 'package.json'),
-      JSON.stringify({packageManager: 'pnpm@8.0.0'}),
-    );
-    expect(detectPackageManager(dir)).toBe('yarn');
-  });
-
-  it('detects from npm_config_user_agent env var', () => {
-    const dir = makeTmpDir();
-    process.env.npm_config_user_agent = 'yarn/1.22.22 node/v20.0.0';
-    expect(detectPackageManager(dir)).toBe('yarn');
-  });
-
-  it('detects pnpm from npm_config_user_agent env var', () => {
-    const dir = makeTmpDir();
-    process.env.npm_config_user_agent = 'pnpm/8.15.0 node/v20.0.0';
-    expect(detectPackageManager(dir)).toBe('pnpm');
-  });
-
-  it('falls back to npx when no signals present', () => {
-    const dir = makeTmpDir();
-    expect(detectPackageManager(dir)).toBe('npx');
-  });
-
-  it('packageManager field takes priority over npm_config_user_agent', () => {
-    const dir = makeTmpDir();
-    fs.writeFileSync(
-      path.join(dir, 'package.json'),
-      JSON.stringify({packageManager: 'bun@1.0.0'}),
-    );
-    process.env.npm_config_user_agent = 'npm/10.0.0 node/v20.0.0';
-    expect(detectPackageManager(dir)).toBe('bun');
-  });
-});