@astryxdesign/cli 0.1.2 → 0.1.3

package/src/commands/blog.mjs

@@ -0,0 +1,60 @@
+// Copyright (c) Meta Platforms, Inc. and affiliates.
+
+/**
+ * @file blog command (hidden) — read the Astryx blog from the published feed.
+ *
+ * Hidden on purpose: it's not part of the documented command surface and does
+ * not appear in --help or the manifest. It reads the blog the same way any
+ * feed reader would — over the public RSS feed — and prints a post's plaintext
+ * (.txt) variant. Nothing about the blog's structure has to change for this to
+ * work; the CLI is just a consumer of the feed.
+ *
+ *   astryx blog                    List posts from the feed
+ *   astryx blog <slug>             Print a post as plaintext
+ */
+
+import {getRunPrefix} from '../utils/package-manager.mjs';
+import {humanLog} from '../lib/json.mjs';
+import {cliError} from '../lib/cli-error.mjs';
+import {blog as blogApi} from '../api/blog.mjs';
+
+function formatList({feedUrl, posts}, run) {
+  const lines = [`\nAstryx blog · feed: ${feedUrl}\n`];
+  if (posts.length === 0) {
+    lines.push('No posts found in the feed.');
+    return lines.join('\n');
+  }
+  for (const p of posts) {
+    lines.push(`  ${p.slug}`);
+    lines.push(`    ${p.title}`);
+    if (p.type) lines.push(`    ${p.type}`);
+    if (p.textUrl) lines.push(`    ${p.textUrl}`);
+    lines.push('');
+  }
+  lines.push(`Read one: ${run} astryx blog <slug>`);
+  return lines.join('\n');
+}
+
+export function registerBlog(program) {
+  program
+    .command('blog [slug]', {hidden: true})
+    .description('Read the Astryx blog from the published feed')
+    .action(async slug => {
+      const run = getRunPrefix();
+      let result;
+      try {
+        result = await blogApi(slug);
+      } catch (e) {
+        cliError(e.message, {suggestions: e.suggestions || [], code: e.code});
+        return;
+      }
+
+      if (result.type === 'blog.list') {
+        humanLog(formatList(result.data, run));
+      } else {
+        // blog.detail — print the feed URL, then the plaintext body.
+        humanLog(`Feed: ${result.data.feedUrl}\n`);
+        humanLog(result.data.text);
+      }
+    });
+}

package/src/commands/build-theme.import-path.test.mjs

@@ -18,14 +18,10 @@ import * as fs from 'node:fs';
 import * as path from 'node:path';
 import * as os from 'node:os';
 import {fileURLToPath} from 'node:url';
+import {ensureCoreBuilt} from './ensure-core-built.mjs';
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 const CLI_BIN = path.resolve(__dirname, '../../bin/astryx.mjs');
-const REPO_ROOT = path.resolve(__dirname, '../../../..');
-const CORE_THEME_ENTRY = path.join(
-  REPO_ROOT,
-  'packages/core/dist/theme/index.js',
-);
 
 function runCli(args, cwd) {
   try {
@@ -59,13 +55,7 @@ function writeTheme(dir, name) {
 // in-CLI fallback generator). Build core once if it isn't already present so
 // the suite works in any CI job, regardless of job ordering.
 beforeAll(() => {
-  if (!fs.existsSync(CORE_THEME_ENTRY)) {
-    execFileSync('pnpm', ['-F', '@astryxdesign/core', 'build'], {
-      cwd: REPO_ROOT,
-      stdio: 'pipe',
-      timeout: 180_000,
-    });
-  }
+  ensureCoreBuilt();
 }, 200_000);
 
 let tmpDir;

package/src/commands/build-theme.mjs

@@ -15,6 +15,7 @@
 import * as fs from 'node:fs';
 import * as path from 'node:path';
 import {pathToFileURL, fileURLToPath} from 'node:url';
+import {spawn} from 'node:child_process';
 import {createJiti} from 'jiti';
 import {getRunPrefix} from '../utils/package-manager.mjs';
 import {
@@ -597,6 +598,104 @@ function validatePrivateVars(themeDef) {
   return errors;
 }
 
+/**
+ * Path to this CLI's real entry (bin/astryx.mjs), resolved from this module's
+ * location (src/commands/build-theme.mjs → ../../bin/astryx.mjs). Used to
+ * re-invoke `theme build` as a child process in watch mode.
+ */
+function resolveCliBin() {
+  const commandsDir = path.dirname(fileURLToPath(import.meta.url));
+  return path.resolve(commandsDir, '../../bin/astryx.mjs');
+}
+
+/**
+ * Run a single `theme build` as a child process, reusing the exact
+ * single-build code path (and its error handling) rather than duplicating it.
+ * Resolves with the child's exit code; never rejects.
+ *
+ * @param {string} file - The theme file argument, as the user passed it.
+ * @param {object} options - Parsed command options (only `out` is forwarded).
+ * @returns {Promise<number>}
+ */
+function runThemeBuildOnceChild(file, options) {
+  const cliBin = resolveCliBin();
+  const args = [cliBin, 'theme', 'build', file];
+  if (options.out) args.push('--out', options.out);
+  return new Promise(resolve => {
+    const child = spawn(process.execPath, args, {
+      stdio: 'inherit',
+      env: process.env,
+    });
+    child.on('close', code => resolve(code ?? 0));
+    child.on('error', () => resolve(1));
+  });
+}
+
+/**
+ * Watch a theme file and rebuild on change. Runs an initial build, then
+ * rebuilds (debounced) whenever the file changes, until interrupted with
+ * Ctrl-C. Each rebuild runs in a child process so a build error (which the
+ * single-build path reports via a hard exit) is contained and the watcher
+ * keeps running.
+ *
+ * @param {string} file - The theme file argument, as the user passed it.
+ * @param {string} filePath - Absolute path to the theme file.
+ * @param {object} options - Parsed command options.
+ * @returns {Promise<void>} Resolves when the watcher is stopped (Ctrl-C).
+ */
+async function runThemeBuildWatch(file, filePath, options) {
+  const rel = path.relative(process.cwd(), filePath);
+
+  // Initial build.
+  await runThemeBuildOnceChild(file, options);
+
+  humanLog(`\n👀 Watching ${rel} for changes — press Ctrl-C to stop.`);
+
+  let building = false;
+  let queued = false;
+  let debounce = null;
+
+  const rebuild = async () => {
+    if (building) {
+      // Coalesce changes that land mid-build into a single follow-up run.
+      queued = true;
+      return;
+    }
+    building = true;
+    humanLog(`\n♻️  Change detected — rebuilding ${rel}...`);
+    await runThemeBuildOnceChild(file, options);
+    building = false;
+    humanLog(`\n👀 Watching ${rel} for changes — press Ctrl-C to stop.`);
+    if (queued) {
+      queued = false;
+      rebuild();
+    }
+  };
+
+  // Some editors replace the file (rename) rather than writing in place, which
+  // can drop the watch. Watch the containing directory and filter to our file
+  // so edits survive atomic-save/rename.
+  const watchDir = path.dirname(filePath);
+  const baseName = path.basename(filePath);
+  const watcher = fs.watch(watchDir, (_eventType, changed) => {
+    if (changed && changed !== baseName) return;
+    clearTimeout(debounce);
+    // Debounce: editors often emit several events per save.
+    debounce = setTimeout(rebuild, 100);
+  });
+
+  await new Promise(resolve => {
+    const stop = () => {
+      clearTimeout(debounce);
+      watcher.close();
+      humanLog('\nStopped watching.');
+      resolve();
+    };
+    process.once('SIGINT', stop);
+    process.once('SIGTERM', stop);
+  });
+}
+
 export function registerTheme(program) {
   const theme = program
     .command('theme')
@@ -621,6 +720,10 @@ export function registerTheme(program) {
     .command('build <file>')
     .description('Compile a defineTheme file to CSS + JS')
     .option('-o, --out <path>', 'Output CSS file path')
+    .option(
+      '-w, --watch',
+      'Rebuild automatically when the theme file changes (Ctrl-C to stop)',
+    )
     .action(async (file, options) => {
       const filePath = path.resolve(process.cwd(), file);
       const json = program.opts().json || false;
@@ -630,6 +733,20 @@ export function registerTheme(program) {
         return;
       }
 
+      // Watch mode: run an initial build, then rebuild on every change to the
+      // theme file. Watch is a human-interactive, long-running mode — it is not
+      // supported in --json (machine) mode, which expects a single envelope.
+      if (options.watch) {
+        if (json) {
+          cliError('--watch is not supported with --json', {
+            code: ERROR_CODES.ERR_THEME_INVALID,
+          });
+          return;
+        }
+        await runThemeBuildWatch(file, filePath, options);
+        return;
+      }
+
       if (!json) humanLog(`\nBuilding theme from ${path.relative(process.cwd(), filePath)}...`);
 
       // Extract theme definition

package/src/commands/build-theme.prose.test.mjs

@@ -18,9 +18,9 @@
  *   - paragraphs use the body font, not the heading font.
  *
  * Building `astryx theme build` requires a compiled @astryxdesign/core (there is no in-CLI
- * fallback generator), so this suite builds core once in beforeAll — mirroring
- * scripts/build-css.test.mjs — to stay self-sufficient regardless of CI job
- * ordering.
+ * fallback generator), so this suite builds core once in beforeAll via the
+ * shared ensureCoreBuilt() helper — which serializes concurrent Vitest workers
+ * behind a lock — to stay self-sufficient regardless of CI job ordering.
  */
 
 import {describe, it, expect, beforeAll, beforeEach, afterEach} from 'vitest';
@@ -29,14 +29,10 @@ import * as fs from 'node:fs';
 import * as path from 'node:path';
 import * as os from 'node:os';
 import {fileURLToPath} from 'node:url';
+import {ensureCoreBuilt} from './ensure-core-built.mjs';
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 const CLI_BIN = path.resolve(__dirname, '../../bin/astryx.mjs');
-const REPO_ROOT = path.resolve(__dirname, '../../../..');
-const CORE_THEME_ENTRY = path.join(
-  REPO_ROOT,
-  'packages/core/dist/theme/index.js',
-);
 
 function runCli(args, cwd) {
   try {
@@ -71,13 +67,7 @@ function writeTheme(dir, name) {
 // `astryx theme build` imports the compiled @astryxdesign/core/theme entry. Build core
 // once if it isn't already present so the suite works in any CI job.
 beforeAll(() => {
-  if (!fs.existsSync(CORE_THEME_ENTRY)) {
-    execFileSync('pnpm', ['-F', '@astryxdesign/core', 'build'], {
-      cwd: REPO_ROOT,
-      stdio: 'pipe',
-      timeout: 180_000,
-    });
-  }
+  ensureCoreBuilt();
 }, 200_000);
 
 let tmpDir;

package/src/commands/build-theme.watch.test.mjs

@@ -0,0 +1,149 @@
+// Copyright (c) Meta Platforms, Inc. and affiliates.
+
+/**
+ * @file Tests for `astryx theme build --watch` (#3375).
+ *
+ * Watch mode runs an initial build, then rebuilds whenever the theme file
+ * changes, until interrupted. Each rebuild runs in a child process so a build
+ * error (reported by the single-build path via a hard exit) is contained and
+ * the watcher keeps running.
+ *
+ * Building `astryx theme build` requires a compiled @astryxdesign/core, so this
+ * suite builds core once in beforeAll via the shared ensureCoreBuilt() helper —
+ * which serializes concurrent Vitest workers behind a lock — to stay
+ * self-sufficient regardless of CI job ordering.
+ */
+
+import {describe, it, expect, beforeAll, beforeEach, afterEach} from 'vitest';
+import {execFileSync, spawn} from 'node:child_process';
+import * as fs from 'node:fs';
+import * as path from 'node:path';
+import * as os from 'node:os';
+import {fileURLToPath} from 'node:url';
+import {ensureCoreBuilt} from './ensure-core-built.mjs';
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const CLI_BIN = path.resolve(__dirname, '../../bin/astryx.mjs');
+
+function runCli(args, cwd) {
+  try {
+    const out = execFileSync('node', [CLI_BIN, ...args], {
+      cwd,
+      encoding: 'utf-8',
+      stdio: ['ignore', 'pipe', 'pipe'],
+      env: {...process.env, FORCE_COLOR: '0'},
+    });
+    return {code: 0, stdout: out, stderr: ''};
+  } catch (e) {
+    return {
+      code: e.status ?? 1,
+      stdout: e.stdout?.toString() ?? '',
+      stderr: e.stderr?.toString() ?? '',
+    };
+  }
+}
+
+/** Poll until `predicate()` is true or the timeout elapses. */
+async function waitFor(predicate, {timeout = 8000, interval = 50} = {}) {
+  const start = Date.now();
+  for (;;) {
+    if (predicate()) return true;
+    if (Date.now() - start > timeout) return false;
+    await new Promise(r => setTimeout(r, interval));
+  }
+}
+
+beforeAll(() => {
+  ensureCoreBuilt();
+}, 200_000);
+
+let tmpDir;
+beforeEach(() => {
+  tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'astryx-build-theme-watch-'));
+});
+afterEach(() => {
+  fs.rmSync(tmpDir, {recursive: true, force: true});
+});
+
+describe('theme build --watch', () => {
+  it('advertises the flag in --help', () => {
+    const result = runCli(['theme', 'build', '--help'], process.cwd());
+    const out = result.stdout + result.stderr;
+    expect(out).toMatch(/--watch/);
+  });
+
+  it('rejects --watch together with --json (single-envelope contract)', () => {
+    const themeFile = path.join(tmpDir, 'wt.mjs');
+    fs.writeFileSync(
+      themeFile,
+      `export default { name: 'wt', tokens: { '--color-bg': '#fff' } };\n`,
+    );
+    const result = runCli(
+      ['--json', 'theme', 'build', path.relative(tmpDir, themeFile), '--watch'],
+      tmpDir,
+    );
+    expect(result.code).not.toBe(0);
+    expect(result.stdout + result.stderr).toMatch(/watch/i);
+  });
+
+  it('builds initially, rebuilds on change, and stops cleanly on SIGINT', async () => {
+    const themeFile = path.join(tmpDir, 'wt.mjs');
+    const cssFile = path.join(tmpDir, 'wt.css');
+    fs.writeFileSync(
+      themeFile,
+      `export default { name: 'wt', tokens: { '--color-bg': '#ffffff' } };\n`,
+    );
+
+    const child = spawn(
+      process.execPath,
+      [CLI_BIN, 'theme', 'build', 'wt.mjs', '--watch'],
+      {cwd: tmpDir, env: {...process.env, FORCE_COLOR: '0'}},
+    );
+    let stdout = '';
+    child.stdout.on('data', d => (stdout += d.toString()));
+    child.stderr.on('data', d => (stdout += d.toString()));
+
+    try {
+      // Initial build produces the CSS.
+      const built = await waitFor(() => fs.existsSync(cssFile));
+      expect(built).toBe(true);
+      const firstCss = fs.readFileSync(cssFile, 'utf-8');
+      expect(firstCss).toMatch(/#ffffff/);
+
+      // Wait until the watcher is actually watching before editing, so the
+      // change isn't missed.
+      await waitFor(() => /Watching/i.test(stdout));
+
+      // Change the theme — the token value changes so the CSS must change.
+      fs.writeFileSync(
+        themeFile,
+        `export default { name: 'wt', tokens: { '--color-bg': '#010203' } };\n`,
+      );
+
+      // The rebuilt CSS should reflect the new value.
+      const rebuilt = await waitFor(() => {
+        try {
+          return fs.readFileSync(cssFile, 'utf-8').includes('#010203');
+        } catch {
+          return false;
+        }
+      });
+      expect(rebuilt).toBe(true);
+      expect(stdout).toMatch(/rebuild/i);
+    } finally {
+      // SIGINT must stop the watcher and exit cleanly.
+      child.kill('SIGINT');
+    }
+
+    const exited = await new Promise(resolve => {
+      let done = false;
+      child.on('exit', () => {
+        done = true;
+        resolve(true);
+      });
+      setTimeout(() => resolve(done), 4000);
+    });
+    expect(exited).toBe(true);
+    expect(stdout).toMatch(/Stopped watching/);
+  }, 30_000);
+});

package/src/commands/component/index.mjs

@@ -24,6 +24,8 @@ import {cliError} from '../../lib/cli-error.mjs';
 import {ERROR_CODES} from '../../lib/error-codes.mjs';
 import {component as componentApi} from '../../api/component.mjs';
 import {findRelatedBlocks} from '../../api/template.mjs';
+import {Project} from '../../lib/project.mjs';
+import {warnOnIntegrationIssues} from '../../lib/integration-warnings.mjs';
 
 export function registerComponent(program) {
   program
@@ -55,6 +57,16 @@ export function registerComponent(program) {
         return;
       }
 
+      // Non-blocking nudge: if any configured integration has validation
+      // issues, print one compact line to stderr pointing at
+      // validate-integration. Best-effort; suppressed in --json mode.
+      try {
+        const project = await Project.load(process.cwd());
+        await warnOnIntegrationIssues(project.loadedIntegrations, {json});
+      } catch {
+        // Never let the nudge break the command.
+      }
+
       let result;
       try {
         result = await componentApi(name, {
@@ -82,27 +94,49 @@ export function registerComponent(program) {
 
       switch (result.type) {
         case 'component.list': {
-          // --detail brief (default for list views) — names with import paths.
+          // --detail brief (default for list views). The API now returns
+          // package-qualified entries ({name, package}); the human view omits
+          // the core package label for readability but ALWAYS shows the package
+          // for integration components (and whenever names collide).
+          const CORE_PKG = '@astryxdesign/core';
+          // Names that appear under more than one package across the whole
+          // listing — these must always be package-qualified to disambiguate.
+          const nameCounts = new Map();
+          for (const items of Object.values(result.data)) {
+            for (const item of items) {
+              const set = nameCounts.get(item.name) ?? new Set();
+              set.add(item.package);
+              nameCounts.set(item.name, set);
+            }
+          }
+          const isCollision = n => (nameCounts.get(n)?.size ?? 0) > 1;
+          const pkgSuffix = item => {
+            if (item.package !== CORE_PKG) return `  [${item.package}]`;
+            if (isCollision(item.name)) return `  [${item.package}]`;
+            return '';
+          };
+
           if (options.category) {
             const [cat, comps] = Object.entries(result.data)[0];
             humanLog(`\n${cat}:`);
-            for (const comp of comps) {
-              const importPath = resolveImportPath(coreDir, comp);
-              humanLog(`  ${comp}  ← ${importPath}`);
+            for (const item of comps) {
+              const importPath = resolveImportPath(coreDir, item.name);
+              humanLog(`  ${item.name}  ← ${importPath}${pkgSuffix(item)}`);
             }
             humanLog('');
           } else {
             humanLog('');
             for (const [key, comps] of Object.entries(result.data)) {
-              const isUngrouped = comps.length === 1 && comps[0] === key;
+              const isUngrouped = comps.length === 1 && comps[0]?.name === key;
               if (isUngrouped) {
-                const importPath = resolveImportPath(coreDir, key);
-                humanLog(`${key}  ← ${importPath}`);
+                const item = comps[0];
+                const importPath = resolveImportPath(coreDir, item.name);
+                humanLog(`${item.name}  ← ${importPath}${pkgSuffix(item)}`);
               } else {
                 humanLog(`${key} (group)`);
-                for (const comp of comps) {
-                  const importPath = resolveImportPath(coreDir, comp);
-                  humanLog(`  ${comp}  ← ${importPath}`);
+                for (const item of comps) {
+                  const importPath = resolveImportPath(coreDir, item.name);
+                  humanLog(`  ${item.name}  ← ${importPath}${pkgSuffix(item)}`);
                 }
               }
             }