@astryxdesign/cli 0.1.3-canary.73bcac9 → 0.1.3-canary.a18ca6b

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@astryxdesign/cli",
-  "version": "0.1.3-canary.73bcac9",
+  "version": "0.1.3-canary.a18ca6b",
   "displayName": "CLI",
   "description": "Scaffold projects, browse templates, generate themes, and get agent-ready docs from the command line.",
   "author": "Meta Open Source",
@@ -75,9 +75,9 @@
     "zod": "^4.4.3"
   },
   "peerDependencies": {
-    "@astryxdesign/core": "0.1.3-canary.73bcac9",
-    "@astryxdesign/lab": "0.1.3-canary.73bcac9",
-    "@astryxdesign/theme-neutral": "0.1.3-canary.73bcac9",
+    "@astryxdesign/core": "0.1.3-canary.a18ca6b",
+    "@astryxdesign/lab": "0.1.3-canary.a18ca6b",
+    "@astryxdesign/theme-neutral": "0.1.3-canary.a18ca6b",
     "gpt-tokenizer": "^2.0.0"
   },
   "peerDependenciesMeta": {
@@ -92,9 +92,9 @@
     }
   },
   "devDependencies": {
-    "@astryxdesign/core": "0.1.3-canary.73bcac9",
-    "@astryxdesign/lab": "0.1.3-canary.73bcac9",
-    "@astryxdesign/theme-neutral": "0.1.3-canary.73bcac9",
+    "@astryxdesign/core": "0.1.3-canary.a18ca6b",
+    "@astryxdesign/lab": "0.1.3-canary.a18ca6b",
+    "@astryxdesign/theme-neutral": "0.1.3-canary.a18ca6b",
     "gpt-tokenizer": "^2.0.0"
   },
   "scripts": {

package/src/lib/component-format.mjs

@@ -7,9 +7,20 @@
 import {discoverComponents, findComponentReadme, resolveImportPath} from './component-discovery.mjs';
 import {loadDocs} from './component-loader.mjs';
 
-/** Derive the theme component key from a theming target (strips 'xds-' prefix). */
+/**
+ * Derive the `defineTheme` component-override key from a theming target.
+ *
+ * Override keys are the component's stable class name with the `astryx-`
+ * namespace prefix stripped — `generateThemeRules` re-adds the prefix when it
+ * builds the `.astryx-*` selector. So `astryx-base-table` → key `base-table`
+ * (→ `.astryx-base-table`), and `astryx-button` → key `button`.
+ *
+ * Keep the `astryx-` literal in sync with packages/core/src/naming.ts
+ * (NAMESPACE / classPrefix), the same way build-theme.mjs mirrors it.
+ * <!-- SYNC: packages/core/src/naming.ts (namespace prefix source of truth) -->
+ */
 function targetKey(target) {
-  return target.className.replace(/^xds-/, '');
+  return target.className.replace(/^astryx-/, '');
 }
 
 function dataAttrForName(name) {

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

@@ -51,3 +51,45 @@ describe('formatFull sub-component rendering', () => {
     expect(out).not.toContain('astryx component XDSButtonGroup');
   });
 });
+
+describe('formatFull theming override keys', () => {
+  // Regression guard: the `defineTheme` component-override key is the stable
+  // class name with the `astryx-` prefix stripped — `generateThemeRules`
+  // re-adds the prefix to build the `.astryx-*` selector. targetKey() used to
+  // strip a dead `xds-` prefix (from before the astryx rename), so the printed
+  // example advertised `astryx-*` keys. Those double-prefix to
+  // `.astryx-astryx-*` at runtime and silently match nothing. See issue #3458.
+  it('prints override keys without the astryx- prefix (single-word component)', () => {
+    const docs = {
+      name: 'Button',
+      description: 'A button.',
+      theming: {targets: [{className: 'astryx-button', visualProps: ['variant']}]},
+    };
+    const out = formatFull(docs);
+
+    expect(out).toContain("'button': {");
+    // The full DOM class must not appear as an override key.
+    expect(out).not.toContain("'astryx-button': {");
+  });
+
+  it('prints override keys without the astryx- prefix (compound Table classes)', () => {
+    const docs = {
+      name: 'Table',
+      description: 'A table.',
+      theming: {
+        targets: [
+          {className: 'astryx-base-table'},
+          {className: 'astryx-table-cell'},
+        ],
+      },
+    };
+    const out = formatFull(docs);
+
+    // Correct keys: class name minus the astryx- prefix.
+    expect(out).toContain("'base-table': {");
+    expect(out).toContain("'table-cell': {");
+    // The verbatim DOM class names must not be advertised as override keys.
+    expect(out).not.toContain("'astryx-base-table': {");
+    expect(out).not.toContain("'astryx-table-cell': {");
+  });
+});