@astryxdesign/cli 0.1.6-canary.fec872b → 0.1.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@astryxdesign/cli",
-  "version": "0.1.6-canary.fec872b",
+  "version": "0.1.6",
   "displayName": "CLI",
   "description": "Scaffold projects, browse templates, generate themes, and get agent-ready docs from the command line.",
   "author": "Meta Open Source",
@@ -47,10 +47,6 @@
       "types": "./src/types/integration.d.ts",
       "import": "./src/integration.mjs"
     },
-    "./doc": {
-      "types": "./src/types/doc.d.ts",
-      "import": "./src/doc.mjs"
-    },
     "./template": {
       "types": "./src/types/template-api.d.ts",
       "import": "./src/template.mjs"
@@ -79,10 +75,10 @@
     "zod": "^4.4.3"
   },
   "peerDependencies": {
-    "@astryxdesign/charts": "0.1.6-canary.fec872b",
-    "@astryxdesign/core": "0.1.6-canary.fec872b",
-    "@astryxdesign/lab": "0.1.6-canary.fec872b",
-    "@astryxdesign/theme-neutral": "0.1.6-canary.fec872b",
+    "@astryxdesign/charts": "*",
+    "@astryxdesign/core": "*",
+    "@astryxdesign/lab": "*",
+    "@astryxdesign/theme-neutral": "*",
     "gpt-tokenizer": "^3.4.0"
   },
   "peerDependenciesMeta": {
@@ -100,10 +96,10 @@
     }
   },
   "devDependencies": {
-    "@astryxdesign/charts": "0.1.6-canary.fec872b",
-    "@astryxdesign/core": "0.1.6-canary.fec872b",
-    "@astryxdesign/lab": "0.1.6-canary.fec872b",
-    "@astryxdesign/theme-neutral": "0.1.6-canary.fec872b",
+    "@astryxdesign/charts": "*",
+    "@astryxdesign/core": "*",
+    "@astryxdesign/lab": "*",
+    "@astryxdesign/theme-neutral": "*",
     "gpt-tokenizer": "^3.4.0"
   },
   "scripts": {

package/src/api/template.mjs

@@ -6,9 +6,8 @@
 
 import * as fs from 'node:fs';
 import * as path from 'node:path';
-import {createJiti} from 'jiti';
 import {loadModuleWithSchema} from '../lib/module-loader.mjs';
-import {TemplateEnvelopeSchema} from '../schemas/template-schema.mjs';
+import {TemplateEnvelopeSchema} from '../template.mjs';
 import {CLI_ROOT, discoverExternalPackages} from '../utils/paths.mjs';
 import {
   assertWithin,
@@ -22,54 +21,9 @@ import {Project} from '../lib/project.mjs';
 /** Identity used for core (built-in) templates in package-scoped listings. */
 const CORE_PACKAGE = '@astryxdesign/core';
 
-/**
- * Canonical basename suffixes for template-spec files, in precedence order.
- * A template spec exports a `createBlockTemplate`/`createPageTemplate` result
- * (a scaffoldable TEMPLATE), so `.template.*` is the descriptive family name.
- */
-const TEMPLATE_SUFFIXES = ['.template.ts', '.template.mjs', '.template.js'];
-
-/**
- * Legacy basename suffixes for template-spec files, in precedence order.
- * `.doc.*` was inherited from the component-doc convention before templates
- * had their own name; it is still accepted during the transition window.
- */
+/** Doc-file basename suffixes for integration templates, in precedence order. */
 const DOC_SUFFIXES = ['.doc.ts', '.doc.mjs', '.doc.js'];
 
-/**
- * The union of canonical + legacy template-spec suffixes, canonical first so
- * `.template.*` wins over `.doc.*` when both stems exist. All template
- * discovery matches this union so a `Foo.template.ts` file is treated exactly
- * like a legacy `Foo.doc.mjs`.
- */
-const ALL_TEMPLATE_SUFFIXES = [...TEMPLATE_SUFFIXES, ...DOC_SUFFIXES];
-
-/**
- * The template-spec suffix present on `file`, or null if none matches.
- * Recognizes both the canonical `.template.*` and legacy `.doc.*` families.
- * @param {string} file
- * @returns {string | null}
- */
-function matchedTemplateSuffix(file) {
-  return ALL_TEMPLATE_SUFFIXES.find(suffix => file.endsWith(suffix)) ?? null;
-}
-
-/**
- * RegExp matching either template-spec suffix family at end of a basename.
- * Used where a per-file regex is convenient (e.g. block discovery walks).
- */
-const TEMPLATE_SUFFIX_RE = /\.(template|doc)\.(ts|mjs|js)$/;
-
-let jitiInstance;
-/** Lazily-created jiti for loading `.ts` template specs (JSX-capable). */
-function getJiti() {
-  if (!jitiInstance) {
-    jitiInstance = createJiti(import.meta.url, {jsx: true});
-  }
-  return jitiInstance;
-}
-
-
 /**
  * Load an integration template doc module and validate it against the template
  * envelope at the load boundary. Default export only — `.ts` via jiti,
@@ -133,29 +87,12 @@ export function stripTemplateAssetRefs(source) {
     source,
   );
 }
-/**
- * Load a template-spec module and return its metadata object. Supports both
- * families of suffix:
- *   - Legacy `.doc.*` core/external specs export `export const doc = {...}`.
- *   - Specs authored with `createPageTemplate`/`createBlockTemplate` export the
- *     stamped object as the default export.
- * Prefers the default export, falling back to the named `doc` export, so a
- * `Foo.template.ts` (default export) is read identically to a legacy
- * `Foo.doc.mjs` (`doc` export). `.ts` is loaded via jiti; `.mjs`/`.js` via a
- * native dynamic import. Returns null if the file does not exist.
- *
- * @param {string} docPath absolute path to the spec file
- * @returns {Promise<Record<string, unknown> | undefined | null>}
- */
 async function loadDocModule(docPath) {
   if (!fs.existsSync(docPath)) return null;
-  const docModule = docPath.endsWith('.ts')
-    ? await getJiti().import(docPath)
-    : await import(`file://${docPath}`);
-  return docModule.default ?? docModule.doc;
+  const docModule = await import(`file://${docPath}`);
+  return docModule.doc;
 }
 
-
 export {discoverAll as discoverTemplates};
 
 export function listTemplates() {
@@ -185,20 +122,6 @@ function findDocFiles(dir, pattern) {
   return results;
 }
 
-/**
- * Resolve the template-spec file for a core page directory: the first existing
- * `template.<suffix>` in canonical-then-legacy precedence, or null.
- * @param {string} dirPath
- * @returns {string | null}
- */
-function findPageDocFile(dirPath) {
-  for (const suffix of ALL_TEMPLATE_SUFFIXES) {
-    const candidate = path.join(dirPath, `template${suffix}`);
-    if (fs.existsSync(candidate)) return candidate;
-  }
-  return null;
-}
-
 async function discoverPages() {
   if (!fs.existsSync(PAGES_DIR)) return [];
   const dirs = fs
@@ -208,9 +131,7 @@ async function discoverPages() {
   const templates = [];
   for (const dir of dirs) {
     const dirPath = path.join(PAGES_DIR, dir.name);
-    const docPath =
-      findPageDocFile(dirPath) ?? path.join(dirPath, 'template.doc.mjs');
-    const doc = await loadDocModule(docPath);
+    const doc = await loadDocModule(path.join(dirPath, 'template.doc.mjs'));
     templates.push({
       type: 'page',
       dirName: dir.name,
@@ -220,18 +141,17 @@ async function discoverPages() {
       isReady: doc?.isReady ?? true,
       scaffold: doc?.scaffold ?? false,
       filePath: path.join(dirPath, 'page.tsx'),
-      docPath,
+      docPath: path.join(dirPath, 'template.doc.mjs'),
     });
   }
   return templates;
 }
 
 async function discoverBlocks() {
-  const docFiles = findDocFiles(BLOCKS_DIR, TEMPLATE_SUFFIX_RE);
+  const docFiles = findDocFiles(BLOCKS_DIR, /\.doc\.mjs$/);
   const blocks = [];
   for (const docPath of docFiles) {
-    const suffix = matchedTemplateSuffix(docPath);
-    const basename = path.basename(docPath, suffix);
+    const basename = path.basename(docPath, '.doc.mjs');
     const tsxPath = path.join(path.dirname(docPath), basename + '.tsx');
     if (!fs.existsSync(tsxPath)) continue;
     const doc = await loadDocModule(docPath);
@@ -253,7 +173,6 @@ async function discoverBlocks() {
   return blocks;
 }
 
-
 /**
  * Discover blocks from external packages that declare `astryx.blocks` in
  * their package.json. Same shape as discoverBlocks() output.
@@ -266,10 +185,9 @@ async function discoverExternalBlocks(cwd = process.cwd()) {
 
   for (const ext of externals) {
     if (!ext.blocksDir || !fs.existsSync(ext.blocksDir)) continue;
-    const docFiles = findDocFiles(ext.blocksDir, TEMPLATE_SUFFIX_RE);
+    const docFiles = findDocFiles(ext.blocksDir, /\.doc\.mjs$/);
     for (const docPath of docFiles) {
-      const suffix = matchedTemplateSuffix(docPath);
-      const basename = path.basename(docPath, suffix);
+      const basename = path.basename(docPath, '.doc.mjs');
       const tsxPath = path.join(path.dirname(docPath), basename + '.tsx');
       if (!fs.existsSync(tsxPath)) continue;
       const doc = await loadDocModule(docPath);
@@ -340,9 +258,8 @@ async function discoverAllWithErrors(cwd = process.cwd()) {
 export {discoverAllWithErrors};
 
 /**
- * Recursively collect integration template-spec files under `root`.
- * Returns absolute paths to files ending in one of ALL_TEMPLATE_SUFFIXES
- * (canonical `.template.*` or legacy `.doc.*`).
+ * Recursively collect integration template doc files under `root`.
+ * Returns absolute paths to files ending in one of DOC_SUFFIXES.
  *
  * @param {string} root
  * @returns {string[]}
@@ -355,7 +272,7 @@ function findIntegrationDocFiles(root) {
       const full = path.join(dir, entry.name);
       if (entry.isDirectory()) {
         walk(full);
-      } else if (ALL_TEMPLATE_SUFFIXES.some(suffix => entry.name.endsWith(suffix))) {
+      } else if (DOC_SUFFIXES.some(suffix => entry.name.endsWith(suffix))) {
         results.push(full);
       }
     }
@@ -364,17 +281,24 @@ function findIntegrationDocFiles(root) {
   return results;
 }
 
+/**
+ * The doc-file suffix present on `file`, or null if none matches.
+ * @param {string} file
+ */
+function matchedDocSuffix(file) {
+  return DOC_SUFFIXES.find(suffix => file.endsWith(suffix)) ?? null;
+}
+
 /**
  * Discover templates contributed by configured integrations.
  *
  * For each integration with a resolved `templates` root, every
- * `<id>.template.{ts,mjs,js}` (or legacy `<id>.doc.{ts,mjs,js}`) file is a
- * template whose id is its path relative to the templates root with the
- * matched suffix stripped (kebab-case, may be nested). The doc's `type`
- * (page|block) decides scaffolding — there is no `/pages` vs `/blocks`
- * requirement. A same-stem sibling source file (`<id>.tsx`) is required; a doc
- * missing its source, or missing `type`, is an integration error and that
- * template is skipped (recorded in `errors`).
+ * `<id>.doc.{ts,mjs,js}` file is a template whose id is its path relative to
+ * the templates root with the `.doc.*` suffix stripped (kebab-case, may be
+ * nested). The doc's `type` (page|block) decides scaffolding — there is no
+ * `/pages` vs `/blocks` requirement. A same-stem sibling source file
+ * (`<id>.tsx`) is required; a doc missing its source, or missing `type`, is
+ * an integration error and that template is skipped (recorded in `errors`).
  *
  * @param {string} [cwd]
  * @returns {Promise<{templates: object[], errors: {package: string, template?: string, message: string}[]}>}
@@ -420,7 +344,7 @@ export async function discoverIntegrationTemplatesForOne(integration) {
   if (!root || !fs.existsSync(root)) return {templates, errors};
 
   for (const docPath of findIntegrationDocFiles(root)) {
-    const suffix = matchedTemplateSuffix(docPath);
+    const suffix = matchedDocSuffix(docPath);
     const id = path
       .relative(root, docPath)
       .slice(0, -suffix.length)

package/src/api/validate-integration.mjs

@@ -67,6 +67,11 @@ function error(code, message) {
   return {code, severity: 'error', message};
 }
 
+/** @param {string} code @param {string} message @returns {Issue} */
+function warning(code, message) {
+  return {code, severity: 'warning', message};
+}
+
 /**
  * Verify each declared contribution root exists on disk. A declared-but-missing
  * root is a `missing_root` error.
@@ -360,3 +365,6 @@ export function summarizeIssues(issues) {
   }
   return {errors, warnings};
 }
+
+// Re-export issue constructors for callers/tests that want them.
+export {error as integrationError, warning as integrationWarning};

package/src/codemods/__tests__/registry.test.mjs

@@ -20,7 +20,6 @@ describe('registry', () => {
         '0.1.2',
         '0.1.3',
         '0.1.5',
-        '0.1.7',
       ]);
     });
   });

package/src/codemods/registry.mjs

@@ -21,7 +21,6 @@ const registry = new Map([
   ['0.1.2', () => import('./transforms/v0.1.2/index.mjs')],
   ['0.1.3', () => import('./transforms/v0.1.3/index.mjs')],
   ['0.1.5', () => import('./transforms/v0.1.5/index.mjs')],
-  ['0.1.7', () => import('./transforms/v0.1.7/index.mjs')],
 ]);
 
 // Re-export from the shared utility so registry callers and other consumers

package/src/config.mjs

@@ -1,9 +1,18 @@
 // Copyright (c) Meta Platforms, Inc. and affiliates.
 
 /**
- * Re-export of the config-authoring helper, which now lives in
- * `@astryxdesign/core/config` so an app's config file gets type feedback
- * without depending on the CLI. Kept here so existing
- * `@astryxdesign/cli/config` imports continue to work unchanged.
+ * Type-preserving helper for the Astryx config file.
+ *
+ * This is an intentionally tiny runtime identity function: it returns its
+ * argument unchanged. Its value is the exported TypeScript surface from
+ * `@astryxdesign/cli/config`, so config files get editor/type feedback without
+ * coupling to CLI internals. Validation is NOT performed here — it happens at
+ * the load boundary (see `loadModuleWithSchema` + `AstryxConfigSchema`).
+ *
+ * @template {import('./types/config').AstryxConfig} T
+ * @param {T} config
+ * @returns {T}
  */
-export {createConfig} from '@astryxdesign/core/config';
+export function createConfig(config) {
+  return config;
+}

package/src/integration.mjs

@@ -1,8 +1,19 @@
 // Copyright (c) Meta Platforms, Inc. and affiliates.
 
 /**
- * Re-export of the integration-authoring helper, which now lives in
- * `@astryxdesign/core/authoring`. Kept here so existing
- * `@astryxdesign/cli/integration` imports continue to work unchanged.
+ * Type-preserving helper for an Astryx integration manifest.
+ *
+ * This is an intentionally tiny runtime identity function: it returns its
+ * argument unchanged. Its value is the exported TypeScript surface from
+ * `@astryxdesign/cli/integration`, so manifests get editor/type feedback
+ * without coupling to CLI internals. Validation is NOT performed here — it
+ * happens at the load boundary (see `loadModuleWithSchema` +
+ * `AstryxIntegrationSchema`).
+ *
+ * @template {import('./types/integration').AstryxIntegration} T
+ * @param {T} integration
+ * @returns {T}
  */
-export {createIntegration} from '@astryxdesign/core/authoring';
+export function createIntegration(integration) {
+  return integration;
+}

package/src/lib/component-discovery.mjs

@@ -273,24 +273,14 @@ export function findComponentReadme(coreDir, name) {
  * For "Button" finds src/Button/XDSButton.tsx
  * For "Layout" finds src/Layout/XDSLayout/XDSLayout.tsx
  * For "Card" finds src/Layout/Container/XDSCard.tsx (deep search fallback)
- *
- * Hooks and other functions are authored as `.ts` (e.g. `useMediaQuery.ts`,
- * `useResizable.ts`), so `.ts` candidates are searched alongside `.tsx`. Without
- * this, deriving the import path for a `.ts`-authored function falls back to the
- * bare `@astryxdesign/core` root instead of its tree-shakeable subpath.
  */
 export function findComponentSource(coreDir, name) {
   const srcDir = path.join(coreDir, 'src');
-  // Try the prefixed forms (`XDSButton.tsx`) first since that is the current
-  // on-disk convention, then the bare forms (`Button.tsx`) that the Astryx-prefix
-  // migration (P4) renames to. `.tsx` before `.ts` within each so a component's
-  // `.tsx` wins over a same-named `.ts` helper; `.ts` covers hooks/functions.
-  const candidateFiles = [
-    `XDS${name}.tsx`,
-    `${name}.tsx`,
-    `XDS${name}.ts`,
-    `${name}.ts`,
-  ];
+  // Try the prefixed form (`XDSButton.tsx`) first since that is the current
+  // on-disk convention, then the bare form (`Button.tsx`) that the Astryx-prefix
+  // migration (P4) renames to. Listing the prefixed name first keeps behavior
+  // identical until files are actually renamed.
+  const candidateFiles = [`XDS${name}.tsx`, `${name}.tsx`];
 
   function searchDir(dirPath) {
     if (!fs.existsSync(dirPath)) return null;

package/src/lib/component-format.mjs

@@ -34,30 +34,15 @@ function getTargetDataAttributes(target) {
   ];
 }
 
-/**
- * Escape a value for a markdown table cell.
- *
- * A prop type is a union, and a union is spelled with the same `|` that GFM
- * uses to separate cells — backticks do not protect it. A row with more cells
- * than the header has columns gets the excess *discarded*, so an unescaped
- * `gap: 0 | 0.5 | ...` silently eats its own Default and Description columns.
- * <!-- SYNC: packages/core/src/Markdown/parser.ts (splits on unescaped pipes) -->
- */
-export function mdCell(value) {
-  return String(value ?? '').replace(/\|/g, '\\|');
-}
-
 function formatPropsTable(props) {
   if (!props || props.length === 0) return '';
   const lines = [];
   lines.push('| Prop | Type | Default | Description |');
   lines.push('|------|------|---------|-------------|');
   for (const p of props) {
-    const def = p.default ? `\`${mdCell(p.default)}\`` : '—';
+    const def = p.default ? `\`${p.default}\`` : '—';
     const req = p.required ? ' **(required)**' : '';
-    lines.push(
-      `| \`${mdCell(p.name)}\` | \`${mdCell(p.type)}\` | ${def} | ${mdCell(p.description)}${req} |`,
-    );
+    lines.push(`| \`${p.name}\` | \`${p.type}\` | ${def} | ${p.description}${req} |`);
   }
   return lines.join('\n');
 }
@@ -192,7 +177,7 @@ export function formatFull(docs, options = {}) {
     sections.push('|---------|----------|-------------|');
     for (const el of docs.usage.anatomy) {
       const req = el.required ? 'Yes' : 'No';
-      sections.push(`| ${mdCell(el.name)} | ${req} | ${mdCell(el.description)} |`);
+      sections.push(`| ${el.name} | ${req} | ${el.description} |`);
     }
     sections.push('');
   }
@@ -297,9 +282,7 @@ export function formatFull(docs, options = {}) {
         varLines.push('| CSS Variable | Default | Description |');
         varLines.push('|-------------|---------|-------------|');
         for (const v of publicVars) {
-          varLines.push(
-            `| \`${mdCell(v.name)}\` | \`${mdCell(v.default)}\` | ${mdCell(v.description)} |`,
-          );
+          varLines.push(`| \`${v.name}\` | \`${v.default}\` | ${v.description} |`);
         }
         sections.push(varLines.join('\n') + '\n');
       }
@@ -395,8 +378,8 @@ export function formatCompact(docs, componentName, importHint) {
     propLines.push('| CSS Property | Sets |');
     propLines.push('|-------------|------|');
     for (const d of docs.theming.derived) {
-      const target = d.expand === 'container' ? 'container layout tokens' : (d.vars || []).map(v => `\`${mdCell(v)}\``).join(', ');
-      propLines.push(`| \`${mdCell(d.property)}\` | ${target} |`);
+      const target = d.expand === 'container' ? 'container layout tokens' : (d.vars || []).map(v => `\`${v}\``).join(', ');
+      propLines.push(`| \`${d.property}\` | ${target} |`);
     }
     sections.push(propLines.join('\n') + '\n');
   }
@@ -412,19 +395,6 @@ export function formatCompact(docs, componentName, importHint) {
  *
  * For multi-component docs, extracts the entry matching componentName.
  */
-/**
- * Longest union `--detail brief` will spell out inside the one-line signature.
- *
- * The signature is a glance, not a reference: a six-member enum like
- * `start|center|end|between|around|evenly` reads at a glance, an eleven-member
- * spacing scale does not — and Stack carries four of them (gap, padding,
- * paddingInline, paddingBlock), so it printed the same scale four times. Longer
- * unions fall through to the terse prop list, exactly as they did when the type
- * was still the bare name `SpacingStep`. The full values are always one
- * `astryx component <Name>` away.
- */
-const SIGNATURE_UNION_MAX_MEMBERS = 8;
-
 export function formatBrief(docs, componentName, importHint, options = {}) {
   const displayName = componentName.startsWith('XDS')
     ? componentName.slice(3)
@@ -451,15 +421,13 @@ export function formatBrief(docs, componentName, importHint, options = {}) {
   const otherProps = [];
 
   for (const prop of props) {
-    const values =
-      prop.type.includes('|') && !prop.type.includes('ReactNode')
-        ? prop.type
-            .replace(/['"]/g, '')
-            .split('|')
-            .map(v => v.trim())
-        : null;
-    if (values && values.length <= SIGNATURE_UNION_MAX_MEMBERS) {
-      signatureProps.push(`${prop.name}: ${values.join('|')}`);
+    if (prop.type.includes('|') && !prop.type.includes('ReactNode')) {
+      const values = prop.type
+        .replace(/['"]/g, '')
+        .split('|')
+        .map(v => v.trim())
+        .join('|');
+      signatureProps.push(`${prop.name}: ${values}`);
     } else if (prop.required) {
       otherProps.unshift(`${prop.name}: ${prop.type.split('|')[0].trim()}`);
     } else {

package/src/lib/component-format.test.mjs

@@ -1,7 +1,7 @@
 // Copyright (c) Meta Platforms, Inc. and affiliates.
 
 import {describe, it, expect} from 'vitest';
-import {formatBrief, formatFull} from './component-format.mjs';
+import {formatFull} from './component-format.mjs';
 
 describe('formatFull sub-component rendering', () => {
   // Regression guard: sub-components are sometimes declared as a bare
@@ -93,97 +93,3 @@ describe('formatFull theming override keys', () => {
     expect(out).not.toContain("'astryx-table-cell': {");
   });
 });
-
-/** Pipes that actually separate cells — a `\|` is content, not a separator. */
-function cellPipes(row) {
-  return (row.match(/(?<!\\)\|/g) || []).length;
-}
-
-describe('markdown table cells escape their pipes', () => {
-  // A prop type is a union, and a union is spelled with `|` — the very
-  // character GFM uses to separate cells. Backticks do not protect it: a row
-  // with more cells than the header has columns gets its excess *discarded*,
-  // so an unescaped `gap: 0 | 0.5 | ...` silently eats its own Default and
-  // Description columns. See packages/core/src/Markdown/parser.ts, which
-  // splits on unescaped pipes and preserves `\|` as literal.
-  it('keeps a union-typed prop row at four cells', () => {
-    const docs = {
-      name: 'Grid',
-      description: 'A grid.',
-      props: [
-        {
-          name: 'gap',
-          type: '0 | 0.5 | 1 | 1.5 | 2 | 3 | 4 | 5 | 6 | 8 | 10',
-          default: '2',
-          description: 'Spacing between all items.',
-        },
-      ],
-    };
-    const row = formatFull(docs)
-      .split('\n')
-      .find(l => l.startsWith('| `gap`'));
-
-    // A 4-column row has exactly 5 separators. Unescaped, this row had 15.
-    expect(cellPipes(row)).toBe(5);
-    expect(row).toContain('\\|');
-    // The columns the unescaped pipes were swallowing.
-    expect(row).toContain('`2`');
-    expect(row).toContain('Spacing between all items.');
-  });
-
-  it('escapes pipes in a description too', () => {
-    const docs = {
-      name: 'AppShell',
-      description: 'A shell.',
-      props: [
-        {
-          name: 'mobileNav',
-          type: 'ReactNode',
-          description: "Config. breakpoint is 'sm' | 'md' | 'lg' | 'none'.",
-        },
-      ],
-    };
-    const row = formatFull(docs)
-      .split('\n')
-      .find(l => l.startsWith('| `mobileNav`'));
-
-    expect(cellPipes(row)).toBe(5);
-  });
-});
-
-describe('formatBrief signature stays terse', () => {
-  // `--detail brief` exists to be token-cheap. It hoists union-typed props into
-  // the signature because a short enum reads at a glance — but an 11-member
-  // spacing scale does not, and Stack carries four of them (gap, padding,
-  // paddingInline, paddingBlock), so the same scale got printed four times.
-  // Long unions stay in the terse prop list, exactly as they did when the type
-  // was still the bare name `SpacingStep`.
-  it('hoists a short enum but not a long scale', () => {
-    const docs = {
-      name: 'HStack',
-      description: 'A stack.',
-      props: [
-        {
-          name: 'gap',
-          type: '0 | 0.5 | 1 | 1.5 | 2 | 3 | 4 | 5 | 6 | 8 | 10',
-          description: 'Gap.',
-        },
-        {name: 'wrap', type: "'nowrap' | 'wrap' | 'wrap-reverse'", description: 'Wrap.'},
-        {
-          name: 'hAlign',
-          type: "'start' | 'center' | 'end' | 'between' | 'around' | 'evenly'",
-          description: 'Align.',
-        },
-      ],
-    };
-    const signature = formatBrief(docs, 'HStack', '').split('\n')[0];
-
-    // Short enums still earn their place in the signature.
-    expect(signature).toContain('wrap: nowrap|wrap|wrap-reverse');
-    expect(signature).toContain('hAlign: start|center|end|between|around|evenly');
-    // The long scale does not.
-    expect(signature).not.toContain('0|0.5|1|1.5|2|3|4|5|6|8|10');
-    // But the prop is still named, so it is not lost.
-    expect(formatBrief(docs, 'HStack', '')).toContain('gap');
-  });
-});