@astryxdesign/cli 0.6.3-canary.f22695a → 0.6.3

package/api/integration/pack-check.mjs

@@ -388,14 +388,6 @@ async function validatePackedComponentExports(integration, scratchBase) {
     components.map(component => component.specifier),
     scratchBase,
   );
-  issues.push(
-    ...validateSpecifierExtensions(
-      components.map(({name, specifier}) => ({
-        label: `Component "${name}"`,
-        specifier,
-      })),
-    ),
-  );
   for (const {name, specifier} of components) {
     const result = resolved.get(specifier);
     if (!result?.url || result.error) {
@@ -435,27 +427,19 @@ async function validatePackedComponentExports(integration, scratchBase) {
  */
 async function validatePackedTemplateExports(integration, scratchBase) {
   const {templates} = await discoverIntegrationTemplatesForOne(integration);
-  const entries = templates.map(template => {
-    const relPath = path
+  const entries = templates.map(template => ({
+    id: template.dirName,
+    specifier: `${integration.name}/${path
       .relative(integration.__packageDir, template.filePath)
       .split(path.sep)
-      .join('/');
-    const extensionless = relPath.replace(/\.tsx?$/u, '');
-    return {
-      id: template.dirName,
-      specifier: `${integration.name}/${extensionless}`,
-    };
-  });
+      .join('/')}`,
+  }));
   const resolved = resolveConsumerSpecifiers(
     entries.map(entry => entry.specifier),
     scratchBase,
   );
-  const issues = validateSpecifierExtensions(
-    entries.map(({id, specifier}) => ({
-      label: `Template "${id}"`,
-      specifier,
-    })),
-  );
+  /** @type {Issue[]} */
+  const issues = [];
   for (const {id, specifier} of entries) {
     const result = resolved.get(specifier);
     if (!result?.url || result.error) {
@@ -485,32 +469,6 @@ async function validatePackedTemplateExports(integration, scratchBase) {
   return issues;
 }
 
-const TS_EXTENSION_RE = /\.tsx?$/u;
-
-/**
- * Reject specifiers that end in a TypeScript extension — they fail under
- * default `moduleResolution` with TS5097/TS2307 unless the consumer enables
- * `allowImportingTsExtensions`.
- *
- * @param {Array<{label: string, specifier: string}>} entries
- * @returns {Issue[]}
- */
-function validateSpecifierExtensions(entries) {
-  /** @type {Issue[]} */
-  const issues = [];
-  for (const {label, specifier} of entries) {
-    if (TS_EXTENSION_RE.test(specifier)) {
-      issues.push(
-        error(
-          'typescript_extension_in_specifier',
-          `${label} advertises import "${specifier}", which ends in a TypeScript extension. A consumer with default moduleResolution will reject it (TS5097). Use an extensionless specifier mapped through the package exports.`,
-        ),
-      );
-    }
-  }
-  return issues;
-}
-
 /**
  * Verify an integration package is ready to publish.
  *

package/api/integration/pack-check.test.mjs

@@ -567,252 +567,3 @@ describe('pack-check mutation tests', () => {
     }
   });
 });
-
-describe('extensionless subpath resolution', () => {
-  it('rejects .tsx public specifiers before publishing', async () => {
-    // Manually construct a package with OLD-style .tsx exports keys
-    writePackage({
-      manifest: "export default {components: './components'};\n",
-      files: ['astryx.integration.mjs', 'components', 'index.mjs'],
-      themes: false,
-    });
-    const pkgFile = path.join(tmpDir, 'package.json');
-    const pkg = JSON.parse(fs.readFileSync(pkgFile, 'utf-8'));
-    pkg.exports = {
-      '.': './index.mjs',
-      './components/AcmeWidget.tsx': './components/AcmeWidget.tsx',
-    };
-    fs.writeFileSync(pkgFile, `${JSON.stringify(pkg, null, 2)}\n`);
-    fs.writeFileSync(
-      path.join(tmpDir, 'index.mjs'),
-      'export const existing = 1;\n',
-    );
-    fs.mkdirSync(path.join(tmpDir, 'components'));
-    fs.writeFileSync(
-      path.join(tmpDir, 'components', 'AcmeWidget.doc.mjs'),
-      `export default {type: 'component', name: 'AcmeWidget', import: '@acme/widgets/components/AcmeWidget.tsx', description: 'Widget.', props: []};\n`,
-    );
-    fs.writeFileSync(
-      path.join(tmpDir, 'components', 'AcmeWidget.tsx'),
-      'export function AcmeWidget() { return null; }\n',
-    );
-
-    const result = await integrationPackCheck({cwd: tmpDir});
-
-    // The .tsx extension in the specifier must be caught
-    expect(result.data.packable).toBe(false);
-    expect(result.data.issues).toContainEqual(
-      expect.objectContaining({
-        code: 'typescript_extension_in_specifier',
-        message: expect.stringContaining('.tsx'),
-      }),
-    );
-  });
-
-  it('generated extensionless component specifier resolves through exports map', async () => {
-    fs.writeFileSync(
-      path.join(tmpDir, 'package.json'),
-      `${JSON.stringify({
-        name: '@acme/widgets',
-        version: '1.0.0',
-        files: ['index.mjs'],
-        exports: {'.': './index.mjs'},
-      })}\n`,
-    );
-    fs.writeFileSync(
-      path.join(tmpDir, 'index.mjs'),
-      'export const existing = 1;\n',
-    );
-    await integrationAddComponent('AcmeWidget', {cwd: tmpDir});
-
-    // Verify the exports map has extensionless key → .tsx target
-    const pkg = JSON.parse(
-      fs.readFileSync(path.join(tmpDir, 'package.json'), 'utf-8'),
-    );
-    expect(pkg.exports['./components/AcmeWidget']).toBe(
-      './components/AcmeWidget.tsx',
-    );
-    expect(pkg.exports['./components/AcmeWidget.tsx']).toBeUndefined();
-
-    const result = await integrationPackCheck({cwd: tmpDir});
-
-    expect(result.data.packable).toBe(true);
-    expect(result.data.issues).not.toContainEqual(
-      expect.objectContaining({code: 'typescript_extension_in_specifier'}),
-    );
-    expect(result.data.issues).not.toContainEqual(
-      expect.objectContaining({code: 'component_import_unresolvable'}),
-    );
-  });
-
-  it('validates generated imports without project-local TypeScript', async () => {
-    fs.rmSync(tmpDir, {recursive: true, force: true});
-    tmpDir = fs.mkdtempSync(
-      path.join(os.tmpdir(), 'astryx-pack-check-no-typescript-'),
-    );
-    fs.writeFileSync(
-      path.join(tmpDir, 'package.json'),
-      `${JSON.stringify({
-        name: '@acme/widgets',
-        version: '1.0.0',
-        files: ['index.mjs'],
-        exports: {'.': './index.mjs'},
-      })}\n`,
-    );
-    fs.writeFileSync(
-      path.join(tmpDir, 'index.mjs'),
-      'export const existing = 1;\n',
-    );
-    await integrationAddComponent('AcmeWidget', {cwd: tmpDir});
-
-    const result = await integrationPackCheck({cwd: tmpDir});
-
-    expect(result.data.packable).toBe(true);
-    expect(result.data.issues).not.toContainEqual(
-      expect.objectContaining({code: 'component_import_unresolvable'}),
-    );
-  });
-
-  it('generated extensionless template specifiers resolve for page and block types', async () => {
-    fs.writeFileSync(
-      path.join(tmpDir, 'package.json'),
-      `${JSON.stringify({
-        name: '@acme/widgets',
-        version: '1.0.0',
-        files: ['index.mjs'],
-        exports: {'.': './index.mjs'},
-      })}\n`,
-    );
-    fs.writeFileSync(
-      path.join(tmpDir, 'index.mjs'),
-      'export const existing = 1;\n',
-    );
-    await integrationAddTemplate('account-page', {cwd: tmpDir, type: 'page'});
-    await integrationAddTemplate('info-card', {cwd: tmpDir, type: 'block'});
-
-    const pkg = JSON.parse(
-      fs.readFileSync(path.join(tmpDir, 'package.json'), 'utf-8'),
-    );
-    expect(pkg.exports['./templates/account-page']).toBe(
-      './templates/account-page.tsx',
-    );
-    expect(pkg.exports['./templates/info-card']).toBe(
-      './templates/info-card.tsx',
-    );
-
-    const result = await integrationPackCheck({cwd: tmpDir});
-
-    expect(result.data.packable).toBe(true);
-    expect(result.data.issues).not.toContainEqual(
-      expect.objectContaining({code: 'template_import_unresolvable'}),
-    );
-    expect(result.data.issues).not.toContainEqual(
-      expect.objectContaining({code: 'typescript_extension_in_specifier'}),
-    );
-  });
-
-  it('preserves condition-style exports sugar without creating a map', async () => {
-    // Package with condition-only exports (sugar, no subpaths)
-    fs.writeFileSync(
-      path.join(tmpDir, 'package.json'),
-      `${JSON.stringify({
-        name: '@acme/widgets',
-        version: '1.0.0',
-        files: ['index.mjs'],
-        exports: {import: './index.mjs', require: './index.cjs'},
-      })}\n`,
-    );
-    fs.writeFileSync(
-      path.join(tmpDir, 'index.mjs'),
-      'export const existing = 1;\n',
-    );
-    fs.writeFileSync(path.join(tmpDir, 'index.cjs'), 'exports.existing = 1;\n');
-    fs.writeFileSync(
-      path.join(tmpDir, 'astryx.integration.mjs'),
-      'export default {};\n',
-    );
-
-    await integrationAddComponent('AcmeWidget', {cwd: tmpDir});
-
-    const pkg = JSON.parse(
-      fs.readFileSync(path.join(tmpDir, 'package.json'), 'utf-8'),
-    );
-    // Condition sugar wrapped into subpath map with the component added
-    expect(pkg.exports['.']).toEqual({
-      import: './index.mjs',
-      require: './index.cjs',
-    });
-    expect(pkg.exports['./components/AcmeWidget']).toBe(
-      './components/AcmeWidget.tsx',
-    );
-  });
-
-  it('no-map: pack-check fails when generated component has no exports map', async () => {
-    fs.writeFileSync(
-      path.join(tmpDir, 'package.json'),
-      `${JSON.stringify({
-        name: '@acme/widgets',
-        version: '1.0.0',
-        files: ['dist'],
-      })}\n`,
-    );
-    fs.writeFileSync(
-      path.join(tmpDir, 'astryx.integration.mjs'),
-      'export default {};\n',
-    );
-
-    await integrationAddComponent('AcmeWidget', {cwd: tmpDir});
-    await integrationAddTemplate('hero-page', {cwd: tmpDir, type: 'page'});
-
-    const pkg = JSON.parse(
-      fs.readFileSync(path.join(tmpDir, 'package.json'), 'utf-8'),
-    );
-    // No exports map created
-    expect(pkg.exports).toBeUndefined();
-
-    const result = await integrationPackCheck({cwd: tmpDir});
-
-    // The extensionless import cannot resolve without an exports map —
-    // pack-check must fail closed, not false-green.
-    expect(result.data.packable).toBe(false);
-    expect(result.data.issues).toContainEqual(
-      expect.objectContaining({
-        severity: 'error',
-        message: expect.stringContaining('AcmeWidget'),
-      }),
-    );
-  });
-
-  it('idempotent when extensionless export already exists', async () => {
-    fs.writeFileSync(
-      path.join(tmpDir, 'package.json'),
-      `${JSON.stringify({
-        name: '@acme/widgets',
-        version: '1.0.0',
-        files: ['index.mjs'],
-        exports: {
-          '.': './index.mjs',
-          './components/AcmeWidget': './components/AcmeWidget.tsx',
-        },
-      })}\n`,
-    );
-    fs.writeFileSync(
-      path.join(tmpDir, 'index.mjs'),
-      'export const existing = 1;\n',
-    );
-    fs.writeFileSync(
-      path.join(tmpDir, 'astryx.integration.mjs'),
-      'export default {};\n',
-    );
-
-    await integrationAddComponent('AcmeWidget', {cwd: tmpDir});
-
-    const pkg = JSON.parse(
-      fs.readFileSync(path.join(tmpDir, 'package.json'), 'utf-8'),
-    );
-    // Export preserved, no duplicate
-    expect(
-      Object.keys(pkg.exports).filter(k => k.includes('AcmeWidget')),
-    ).toEqual(['./components/AcmeWidget']);
-  });
-});

package/api/search/search.d.mts

@@ -99,5 +99,5 @@ export type Candidate = {
     _import?: string | undefined;
     _title?: string | undefined;
     _displayName?: string | undefined;
-    _kind?: "page" | "block" | undefined;
+    _kind?: "block" | "page" | undefined;
 };

package/api/search/search.mjs

@@ -66,7 +66,7 @@ import {
 import {loadIntegrationsSafely} from '../component/_adapter.mjs';
 import {levenshteinDistance} from '../../foundation/text/string-utils.mjs';
 import {discoverTemplates, extractComponents} from '../template/template.mjs';
-import {loadDocsCatalog, lowerTopic} from '../docs/_adapter.mjs';
+import {loadDocsCatalog, loadTopicDoc} from '../docs/_adapter.mjs';
 import {AstryxError} from '../error.mjs';
 import {ERROR_CODES} from '../../foundation/response/error-codes.mjs';
 import {setResultCoverage} from './coverage.mjs';
@@ -732,16 +732,16 @@ async function gatherHooks(coreDir) {
 async function gatherDocs(cwd) {
   /** @type {Candidate[]} */
   const candidates = [];
-  let catalog;
+  let entries;
   try {
-    catalog = await loadDocsCatalog(cwd);
+    entries = (await loadDocsCatalog(cwd)).entries();
   } catch {
     return candidates;
   }
-  for (const entry of catalog.entries()) {
+  for (const entry of entries) {
     let doc = null;
     try {
-      doc = (await lowerTopic(catalog, entry)).doc;
+      doc = await loadTopicDoc(entry);
     } catch {
       // A topic that cannot be loaded is reported by the commands that own
       // integration issues; search just cannot index it.

package/api/search/search.type.d.mts

@@ -48,10 +48,10 @@ export type SearchResultEntry = {
     /**
      * - Template kind (`page` | `block`) — present for template results.
      */
-    kind?: "page" | "block" | undefined;
+    kind?: "block" | "page" | undefined;
 };
 /**
- * astryx --json search <query>
+ * xds --json search <query>
  */
 export type SearchResponse = {
     type: "search";

package/api/search/search.type.mjs

@@ -28,7 +28,7 @@
  */
 
 /**
- * astryx --json search <query>
+ * xds --json search <query>
  * @typedef {object} SearchResponse
  * @property {'search'} type
  * @property {object} data

package/api/swizzle/swizzle.type.d.mts

@@ -2,7 +2,7 @@
 // DO NOT EDIT — run `pnpm sync:api-types` to regenerate.
 
 /**
- * astryx --json swizzle [--list]
+ * xds --json swizzle [--list]
  */
 export type SwizzleListResponse = {
     type: "swizzle.list";
@@ -22,7 +22,7 @@ export type SwizzleFeedback = {
     ghCommand?: string | undefined;
 };
 /**
- * astryx --json swizzle <component>
+ * xds --json swizzle <component>
  */
 export type SwizzleCopyResponse = {
     type: "swizzle.copy";

package/api/swizzle/swizzle.type.mjs

@@ -7,7 +7,7 @@
  */
 
 /**
- * astryx --json swizzle [--list]
+ * xds --json swizzle [--list]
  *
  * @typedef {object} SwizzleListResponse
  * @property {'swizzle.list'} type
@@ -23,7 +23,7 @@
  */
 
 /**
- * astryx --json swizzle <component>
+ * xds --json swizzle <component>
  *
  * @typedef {object} SwizzleCopyResponse
  * @property {'swizzle.copy'} type

package/api/template/template.d.mts

@@ -31,7 +31,7 @@ export function template(name?: string, options?: {
     skeleton?: boolean | undefined;
     cdn?: string | boolean | undefined;
     show?: boolean | undefined;
-    type?: "page" | "block" | undefined;
+    type?: "block" | "page" | undefined;
     package?: string | undefined;
     cwd?: string | undefined;
 }): Promise<{

package/api/template/template.type.d.mts

@@ -2,7 +2,7 @@
 // DO NOT EDIT — run `pnpm sync:api-types` to regenerate.
 
 /**
- * astryx --json template [--list]
+ * xds --json template [--list]
  */
 export type TemplateListResponse = {
     type: "template.list";
@@ -53,7 +53,7 @@ export type TemplateListEntry = {
     scaffold?: boolean | undefined;
 };
 /**
- * astryx --json template <name>
+ * xds --json template <name>
  */
 export type TemplateShowResponse = {
     type: "template.show";
@@ -66,7 +66,7 @@ export type TemplateShowResponse = {
     };
 };
 /**
- * astryx --json template <name> --skeleton
+ * xds --json template <name> --skeleton
  */
 export type TemplateSkeletonResponse = {
     type: "template.skeleton";
@@ -78,7 +78,7 @@ export type TemplateSkeletonResponse = {
     };
 };
 /**
- * astryx --json template <name> [path]
+ * xds --json template <name> [path]
  */
 export type TemplateCopyResponse = {
     type: "template.copy";
@@ -90,7 +90,7 @@ export type TemplateCopyResponse = {
     };
 };
 /**
- * astryx --json template --cdn [path]
+ * xds --json template --cdn [path]
  * `written: false` with `reason: 'exists'` is a success: the command is safe to
  * re-run, and an edited page is the consumer's file to keep. `version` is the
  * Astryx version every CDN URL in the file was pinned to.
@@ -118,7 +118,7 @@ export type TemplateOptions = {
     /**
      * Filter templates by kind: 'page' or 'block'. Only applies to list views.
      */
-    type?: "page" | "block" | undefined;
+    type?: "block" | "page" | undefined;
     /**
      * Narrow to templates from a specific package (id-only lookups across packages are ambiguous).
      */

package/api/template/template.type.mjs

@@ -6,18 +6,18 @@
  *
  * Each template is exactly two files: page.tsx (code) + template.doc.mjs (metadata).
  *
- * Invocation                               -> type discriminator
+ * Invocation                                 -> type discriminator
  * ------------------------------------------------------------------
- * astryx --json template [--list]          -> template.list
- * astryx --json template <name>            -> template.show
- * astryx --json template <name> --skeleton -> template.skeleton
- * astryx --json template <name> [path]     -> template.copy
- * astryx --json template --cdn [path]      -> template.cdn
- * (unknown template)                       -> CLIError
+ * xds --json template [--list]              -> template.list
+ * xds --json template <name>               -> template.show
+ * xds --json template <name> --skeleton    -> template.skeleton
+ * xds --json template <name> [path]        -> template.copy
+ * xds --json template --cdn [path]         -> template.cdn
+ * (unknown template)                        -> CLIError
  */
 
 /**
- * astryx --json template [--list]
+ * xds --json template [--list]
  * @typedef {object} TemplateListResponse
  * @property {'template.list'} type
  * @property {TemplateListEntry[]} data
@@ -43,7 +43,7 @@
  */
 
 /**
- * astryx --json template <name>
+ * xds --json template <name>
  * @typedef {object} TemplateShowResponse
  * @property {'template.show'} type
  * @property {object} data
@@ -55,7 +55,7 @@
  */
 
 /**
- * astryx --json template <name> --skeleton
+ * xds --json template <name> --skeleton
  * @typedef {object} TemplateSkeletonResponse
  * @property {'template.skeleton'} type
  * @property {object} data
@@ -66,7 +66,7 @@
  */
 
 /**
- * astryx --json template <name> [path]
+ * xds --json template <name> [path]
  * @typedef {object} TemplateCopyResponse
  * @property {'template.copy'} type
  * @property {object} data
@@ -77,7 +77,7 @@
  */
 
 /**
- * astryx --json template --cdn [path]
+ * xds --json template --cdn [path]
  * `written: false` with `reason: 'exists'` is a success: the command is safe to
  * re-run, and an edited page is the consumer's file to keep. `version` is the
  * Astryx version every CDN URL in the file was pinned to.

package/api/theme/build/build.mjs

@@ -12,10 +12,9 @@
  * - A .d.ts (plus an optional .variants.d.ts for custom prop values)
  *
  * It performs the writes and returns a `theme.build` receipt — its `warnings`
- * carry override problems, every declaration core's generator dropped because
- * its value could not stay one CSS declaration, and any fonts the theme names
- * but does not load (font-warning.mjs) — or `null` when the theme produced no
- * CSS (nothing to build). Errors throw AstryxError (with
+ * carry override problems and any fonts the theme names but does not load
+ * (font-warning.mjs) — or `null` when the theme produced no CSS (nothing to
+ * build). Errors throw AstryxError (with
  * a stable code). Human progress is emitted through the shared `logger`
  * (silent by default), so the CLI keeps its exact output while a programmatic
  * caller stays quiet.
@@ -2304,15 +2303,7 @@ async function themeBuildInternal(
     const scopeSelector = themeScopeStart(themeDef.name);
     const scopeTo = THEME_SCOPE_TO;
 
-    // Older cores ignore this optional collector. Current core writes the same
-    // warning text here that runtime callers receive on the console.
-    /** @type {string[]} */
-    const droppedDeclarations = [];
-
-    const {component, prose} = _generateThemeRulesSplit(
-      resolvedTheme,
-      droppedDeclarations,
-    );
+    const {component, prose} = _generateThemeRulesSplit(resolvedTheme);
     const cssParts = [];
     // Prose element defaults always ship — the `<Theme>` runtime
     // (generateThemeCSS) always emits them, so the build must too, or the
@@ -2331,7 +2322,7 @@ async function themeBuildInternal(
     let adaptationCss;
     try {
       adaptationCss = _generateAdaptationCSS
-        ? _generateAdaptationCSS(resolvedTheme, droppedDeclarations)
+        ? _generateAdaptationCSS(resolvedTheme)
         : {component: '', prose: ''};
     } catch (error) {
       const message =
@@ -2377,16 +2368,11 @@ async function themeBuildInternal(
     // adaptations on the same resolved leaf.
     let onMediaCss = '';
     if (_generateOnMediaCSS) {
-      onMediaCss = _generateOnMediaCSS(resolvedTheme, droppedDeclarations);
+      onMediaCss = _generateOnMediaCSS(resolvedTheme);
       if (onMediaCss) {
         cssParts.push(`@layer astryx-theme {\n${onMediaCss}\n}`);
       }
     }
-    for (const message of droppedDeclarations) {
-      const w = `Declaration ${message}. The generated CSS omits it; fix the value in the theme source.`;
-      warningMessages.push(w);
-      logger.warn(`  ⚠ ${w}`);
-    }
     if (cssParts.length === 0) {
       logger.log('No overrides found — nothing to build.');
       return null;