@astryxdesign/cli 0.6.3 → 0.6.4-canary.078fd25

package/foundation/xle/expand.test.mjs

@@ -0,0 +1,54 @@
+// Copyright (c) Meta Platforms, Inc. and affiliates.
+
+/**
+ * @file Expander tests for the repeat cap. Every path that expands a `*N`
+ * repeat — emitted children, table rows, the root count — honours MAX_REPEAT.
+ * Uses the real @astryxdesign/core registry.
+ */
+
+import {describe, it, expect, beforeAll} from 'vitest';
+import {parse} from './parse.mjs';
+import {validate} from './validate.mjs';
+import {buildRegistry} from './registry.mjs';
+import {expand, MAX_REPEAT} from './expand.mjs';
+
+/** @type {import('./xle-ast').Registry} */
+let registry;
+
+// The registry imports every component doc on first use; warm it once.
+beforeAll(async () => {
+  registry = /** @type {import('./xle-ast').Registry} */ (
+    /** @type {unknown} */ (await buildRegistry())
+  );
+}, 120_000);
+
+/**
+ * @param {string} expression
+ * @returns {string}
+ */
+function expandCode(expression) {
+  const doc = parse(expression);
+  const {errors} = validate(doc, registry, []);
+  expect(errors).toEqual([]);
+  return expand(doc, registry).code;
+}
+
+/**
+ * @param {string} code
+ * @param {string} component
+ */
+function countOpening(code, component) {
+  return (code.match(new RegExp(`<(?:XDS)?${component}\\b`, 'g')) || []).length;
+}
+
+describe('expand — repeat cap', () => {
+  it('caps a repeated child at MAX_REPEAT copies', () => {
+    expect(countOpening(expandCode(`V > B*${MAX_REPEAT + 5}`), 'Button')).toBe(MAX_REPEAT);
+  });
+
+  it('caps repeated table rows at MAX_REPEAT copies too', () => {
+    expect(countOpening(expandCode(`T > TR*${MAX_REPEAT + 5} > TC`), 'TableRow')).toBe(
+      MAX_REPEAT,
+    );
+  });
+});

package/foundation/xle/xle.test.mjs

@@ -10,6 +10,7 @@ import {describe, it, expect} from 'vitest';
 import {parse, parseCompact, parseOutline, detectForm, XLEParseError} from './parse.mjs';
 import {toCompact, toOutline} from './print.mjs';
 import {validate} from './validate.mjs';
+import {expand} from './expand.mjs';
 import {parseEnumValues, SPACING_STEPS} from './registry.mjs';
 
 // ─── parser: compact ───────────────────────────────────────────────────────
@@ -309,6 +310,18 @@ describe('validate', () => {
     expect(doc.roots[0].hint.block.name).toBe('CardCallout');
   });
 
+  it('includes --type block when a known block source is unavailable', () => {
+    const doc = parse('Bx{card-callout}');
+    expect(validate(doc, registry, BLOCKS).errors).toEqual([]);
+    const result = expand(doc, registry, {
+      componentName: 'Demo',
+      blockModules: new Map(),
+    });
+    expect(result.code).toContain(
+      'astryx template CardCallout --type block',
+    );
+  });
+
   it('enforces structural pairings', () => {
     const doc = parse('Bx > Sh');
     const {errors} = validate(doc, registry, BLOCKS);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@astryxdesign/cli",
-  "version": "0.6.3",
+  "version": "0.6.4-canary.078fd25",
   "displayName": "CLI",
   "description": "Scaffold projects, browse templates, generate themes, and get agent-ready docs from the command line.",
   "author": "Meta Open Source",
@@ -91,6 +91,7 @@
     "@babel/types": "7.29.8",
     "ast-types": "0.16.3",
     "commander": "^12.1.0",
+    "ignore": "^7.0.5",
     "jiti": "^2.7.0",
     "jscodeshift": "^17.3.0",
     "node-diff3": "3.2.1",
@@ -100,11 +101,10 @@
     "zod": "^4.4.3"
   },
   "peerDependencies": {
-    "@astryxdesign/charts": "*",
-    "@astryxdesign/core": "*",
-    "@astryxdesign/lab": "*",
-    "@astryxdesign/theme-neutral": "*",
-    "gpt-tokenizer": "^3.4.0"
+    "@astryxdesign/charts": "0.6.4-canary.078fd25",
+    "@astryxdesign/core": "0.6.4-canary.078fd25",
+    "@astryxdesign/lab": "0.6.4-canary.078fd25",
+    "@astryxdesign/theme-neutral": "0.6.4-canary.078fd25"
   },
   "peerDependenciesMeta": {
     "@astryxdesign/charts": {
@@ -121,14 +121,13 @@
     }
   },
   "devDependencies": {
-    "@astryxdesign/charts": "*",
-    "@astryxdesign/core": "*",
-    "@astryxdesign/lab": "*",
-    "@astryxdesign/theme-neutral": "*",
+    "@astryxdesign/charts": "0.6.4-canary.078fd25",
+    "@astryxdesign/core": "0.6.4-canary.078fd25",
+    "@astryxdesign/lab": "0.6.4-canary.078fd25",
+    "@astryxdesign/theme-neutral": "0.6.4-canary.078fd25",
     "@heroicons/react": "^2.2.0",
     "@stylexjs/stylex": "^0.19.0",
     "@types/babel__core": "^7.20.5",
-    "gpt-tokenizer": "^3.4.0",
     "lucide-react": "^1.18.0",
     "recharts": "^3.9.2"
   },

package/api/docs/docs.test.mjs

@@ -1,83 +0,0 @@
-// Copyright (c) Meta Platforms, Inc. and affiliates.
-
-/**
- * @file Dispatcher-level tests for docs() — the argument-shape routing that
- * sits above the list/detail/section leaves. The leaves have their own tests;
- * this pins the router: docs() -> list, docs(topic) -> detail,
- * docs(topic, section) -> section, plus the unknown-topic/section error codes.
- * Runs against the real CLI-bundled docs (no cwd/project needed).
- */
-
-import {describe, it, expect} from 'vitest';
-import {docs} from './docs.mjs';
-import {AstryxError} from '../error.mjs';
-
-const SLOW = 30_000;
-
-describe('docs() dispatcher routing', () => {
-  it('no topic -> docs.list', async () => {
-    const r = await docs();
-    expect(r.type).toBe('docs.list');
-    expect(Array.isArray(r.data)).toBe(true);
-    expect(r.data.length).toBeGreaterThan(0);
-  }, SLOW);
-
-  it('empty topic -> docs.list (falsy topic routes to list)', async () => {
-    expect((await docs('')).type).toBe('docs.list');
-  }, SLOW);
-
-  it('topic only -> docs.detail', async () => {
-    const {data} = await docs();
-    const topic = data[0].topic;
-    const r = await docs(topic);
-    expect(r.type).toBe('docs.detail');
-  }, SLOW);
-
-  it('topic + section -> docs.detail.section', async () => {
-    const {data} = await docs();
-    let routed = null;
-    for (const {topic} of data) {
-      const detail = await docs(topic);
-      const sections = detail.data.sections;
-      if (Array.isArray(sections) && sections.length > 0) {
-        routed = await docs(topic, sections[0].title);
-        break;
-      }
-    }
-    expect(routed).not.toBeNull();
-    expect(routed.type).toBe('docs.detail.section');
-  }, SLOW);
-
-  it('unknown topic -> ERR_UNKNOWN_TOPIC', async () => {
-    await expect(docs('zzz-not-a-real-topic')).rejects.toBeInstanceOf(AstryxError);
-    await expect(docs('zzz-not-a-real-topic')).rejects.toMatchObject({
-      code: 'ERR_UNKNOWN_TOPIC',
-    });
-  }, SLOW);
-
-  it('known topic + unknown section -> ERR_UNKNOWN_SECTION', async () => {
-    const {data} = await docs();
-    const topic = data[0].topic;
-    await expect(docs(topic, 'zzz-not-a-real-section')).rejects.toMatchObject({
-      code: 'ERR_UNKNOWN_SECTION',
-    });
-  }, SLOW);
-
-  it('non-string topic -> ERR_UNKNOWN_TOPIC (not a raw TypeError)', async () => {
-    for (const bad of [123, {}, ['tokens'], true]) {
-      const err = await docs(/** @type {any} */ (bad)).catch(e => e);
-      expect(err).toBeInstanceOf(AstryxError);
-      expect(err.code).toBe('ERR_UNKNOWN_TOPIC');
-    }
-  }, SLOW);
-
-  it('non-string section -> ERR_UNKNOWN_SECTION (not a raw TypeError)', async () => {
-    const {data} = await docs();
-    const topic = data[0].topic;
-    for (const bad of [456, {}, ['x']]) {
-      const err = await docs(topic, /** @type {any} */ (bad)).catch(e => e);
-      expect(err).toBeInstanceOf(AstryxError);
-      expect(err.code).toBe('ERR_UNKNOWN_SECTION');
-    }
-  }, SLOW);
-});

package/api/docs/integrationDocs.test.mjs

@@ -1,208 +0,0 @@
-// Copyright (c) Meta Platforms, Inc. and affiliates.
-
-/**
- * @file End-to-end tests for integration-contributed doc topics: a scaffolded
- * consumer project whose configured integration ships a `docs` root, read back
- * through the public surfaces — `docs()`, `search()`, and the agent-docs block.
- *
- * The unit-level rules (what a docs root contributes, what the catalog does
- * with `replaces`/`extends`) are covered in
- * foundation/discovery/docs-discovery.test.mjs. What is pinned here is that a
- * contributed topic is indistinguishable from a built-in one at the surfaces
- * an agent actually reads.
- *
- * Fixtures live under a repo-local temp dir, not /tmp, because Vite refuses to
- * dynamically import a module from outside the project root.
- */
-
-import {afterEach, beforeEach, describe, expect, it} from 'vitest';
-import * as fs from 'node:fs';
-import * as path from 'node:path';
-import {docs} from './docs.mjs';
-import {search} from '../search/search.mjs';
-import {loadDocsCatalog} from './_adapter.mjs';
-import {AstryxError} from '../error.mjs';
-
-const SLOW = 30_000;
-
-let tmpDir;
-
-/** A minimal, valid topic. */
-function topic(fields) {
-  return {
-    type: 'generic',
-    name: 'deploying',
-    title: 'Deploying',
-    description: 'How to ship an app built with Acme widgets.',
-    category: 'guide',
-    sections: [
-      {title: 'Overview', content: [{type: 'prose', text: 'Push the button.'}]},
-    ],
-    ...fields,
-  };
-}
-
-/**
- * A consumer project that configures one integration, optionally with a docs
- * root holding the given topics.
- * @param {Record<string, object|string>} [topics] file name → doc
- * @param {{config?: string}} [options]
- */
-function scaffold(topics, {config} = {}) {
-  fs.writeFileSync(path.join(tmpDir, 'package.json'), JSON.stringify({name: 'consumer'}));
-  fs.writeFileSync(
-    path.join(tmpDir, 'astryx.config.mjs'),
-    config ?? "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 ${JSON.stringify(topics ? {docs: './docs'} : {})};\n`,
-  );
-
-  if (topics) {
-    const docsDir = path.join(pkgDir, 'docs');
-    fs.mkdirSync(docsDir, {recursive: true});
-    for (const [file, doc] of Object.entries(topics)) {
-      fs.writeFileSync(
-        path.join(docsDir, file),
-        typeof doc === 'string'
-          ? doc
-          : `export const docs = ${JSON.stringify(doc, null, 2)};\n`,
-      );
-    }
-  }
-}
-
-beforeEach(() => {
-  tmpDir = fs.mkdtempSync(path.join(process.cwd(), '.astryx-integration-docs-test-'));
-});
-
-afterEach(() => {
-  fs.rmSync(tmpDir, {recursive: true, force: true});
-});
-
-describe('integration-contributed topics', () => {
-  it('lists and reads like a built-in topic, naming its owner', async () => {
-    scaffold({'deploying.doc.mjs': topic()});
-
-    const listed = await docs(undefined, undefined, {cwd: tmpDir});
-    const entry = listed.data.find(t => t.topic === 'deploying');
-    expect(entry).toMatchObject({
-      topic: 'deploying',
-      description: 'How to ship an app built with Acme widgets.',
-      package: '@acme/widgets',
-    });
-    // The built-in topics keep their own owner.
-    expect(listed.data.find(t => t.topic === 'tokens').package).toBe('@astryxdesign/cli');
-
-    const detail = await docs('deploying', undefined, {cwd: tmpDir});
-    expect(detail.type).toBe('docs.detail');
-    expect(detail.data.sections[0].content[0].text).toBe('Push the button.');
-
-    const section = await docs('deploying', 'overview', {cwd: tmpDir});
-    expect(section.data.title).toBe('Overview');
-  }, SLOW);
-
-  it('is invisible to a project that does not configure the integration', async () => {
-    scaffold({'deploying.doc.mjs': topic()}, {config: 'export default {};\n'});
-    const listed = await docs(undefined, undefined, {cwd: tmpDir});
-    expect(listed.data.some(t => t.topic === 'deploying')).toBe(false);
-  }, SLOW);
-
-  it('serves the replacement of a built-in topic, and says what it replaced', async () => {
-    scaffold({
-      'getting-started.doc.mjs': topic({
-        name: 'getting-started',
-        replaces: 'getting-started',
-        title: 'Getting started',
-        description: 'Install Acme widgets.',
-        sections: [
-          {title: 'Install', content: [{type: 'prose', text: 'yarn add @acme/widgets'}]},
-        ],
-      }),
-    });
-
-    const detail = await docs('getting-started', undefined, {cwd: tmpDir});
-    expect(detail.data.sections.map(s => s.title)).toEqual(['Install']);
-    expect(detail.data.sections[0].content[0].text).toBe('yarn add @acme/widgets');
-
-    const listed = await docs(undefined, undefined, {cwd: tmpDir});
-    const entries = listed.data.filter(t => t.topic === 'getting-started');
-    expect(entries).toHaveLength(1);
-    expect(entries[0]).toMatchObject({
-      package: '@acme/widgets',
-      replaces: 'getting-started',
-    });
-  }, SLOW);
-
-  it('keeps the replaced name resolving when the replacement renames it', async () => {
-    scaffold({
-      'setup.doc.mjs': topic({name: 'setup', replaces: 'getting-started'}),
-    });
-    const byOldName = await docs('getting-started', undefined, {cwd: tmpDir});
-    const byNewName = await docs('setup', undefined, {cwd: tmpDir});
-    expect(byOldName.data.name).toBe('setup');
-    expect(byNewName.data.name).toBe('setup');
-  }, SLOW);
-
-  it('merges an extension into the topic it extends', async () => {
-    const builtin = await docs('theme');
-    const baseSectionTitle = builtin.data.sections[0].title;
-    scaffold({
-      'theme-internal.doc.mjs': topic({
-        name: 'theme-internal',
-        extends: 'theme',
-        sections: [
-          {title: baseSectionTitle, content: [{type: 'prose', text: 'Use the Acme theme.'}]},
-          {title: 'Acme themes', content: [{type: 'prose', text: 'Three of them.'}]},
-        ],
-      }),
-    });
-
-    const extended = await docs('theme', undefined, {cwd: tmpDir});
-    // The extension is not a topic of its own.
-    expect((await docs(undefined, undefined, {cwd: tmpDir})).data.some(
-      t => t.topic === 'theme-internal',
-    )).toBe(false);
-    expect(extended.data.sections[0].content[0].text).toBe('Use the Acme theme.');
-    expect(extended.data.sections.at(-1).title).toBe('Acme themes');
-    // Everything the extension did not name is still the base doc's.
-    expect(extended.data.sections.length).toBe(builtin.data.sections.length + 1);
-  }, SLOW);
-
-  it('offers the contributed topics as suggestions on an unknown one', async () => {
-    scaffold({'deploying.doc.mjs': topic()});
-    await expect(docs('nope-not-a-topic', undefined, {cwd: tmpDir})).rejects.toBeInstanceOf(
-      AstryxError,
-    );
-    try {
-      await docs('nope-not-a-topic', undefined, {cwd: tmpDir});
-    } catch (err) {
-      expect(err.suggestions.map(s => s.name)).toContain('deploying');
-    }
-  }, SLOW);
-
-  it('indexes a contributed topic in search', async () => {
-    scaffold({'deploying.doc.mjs': topic()});
-    const {data} = await search('deploying', {cwd: tmpDir, type: 'doc'});
-    expect(data.results[0]).toMatchObject({
-      domain: 'doc',
-      name: 'deploying',
-      command: 'astryx docs deploying',
-    });
-  }, SLOW);
-
-  it("falls back to the CLI's own topics when the project config is unreadable", async () => {
-    scaffold({'deploying.doc.mjs': topic()}, {config: 'export default {integrations: 42};\n'});
-    const catalog = await loadDocsCatalog(tmpDir);
-    expect(catalog.resolve('tokens')).toBeTruthy();
-    expect(catalog.resolve('deploying')).toBeUndefined();
-  }, SLOW);
-});