@astryxdesign/cli 0.6.3-canary.45a9bf5 → 0.6.3-canary.54be197

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;

package/api/theme/build/build.test.mjs

@@ -206,133 +206,6 @@ describe('themeBuild() — receipt', () => {
   });
 });
 
-describe('themeBuild() — dropped declarations reach the receipt', () => {
-  // Core's generator refuses a declaration whose value would end it early
-  // (an unquoted `;` or brace, an unclosed string/comment/url). The runtime
-  // says so on the console; a build has a receipt, so every drop must land in
-  // `warnings` — a programmatic caller otherwise sees `warnings: []` while the
-  // CSS silently omits a value the generated JS still carries.
-  it('reports every dropped declaration, from every generator path, in warnings', async () => {
-    const themeFile = path.join(tmpDir, 'dropped.mjs');
-    fs.writeFileSync(
-      themeFile,
-      `export default {
-        name: 'dropped',
-        tokens: {
-          '--color-bg': '#0a0a0a } body { background: url(https://example.com/token) ',
-        },
-        components: {
-          button: {
-            'variant:secondary': {
-              backgroundColor: 'red; background-image: url(https://example.com/component)',
-              ':hover': {color: '"https://example.com/pseudo'},
-            },
-          },
-        },
-        onDark: {
-          tokens: {'--color-bg': 'url(https://example.com/dark'},
-        },
-        adaptations: {
-          rules: [
-            {
-              when: {pointer: 'coarse'},
-              value: {tokens: {'--color-bg': 'red /* https://example.com/adaptation'}},
-            },
-          ],
-        },
-      };\n`,
-    );
-
-    const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {});
-    let result;
-    try {
-      result = await themeBuild('dropped.mjs', {}, {cwd: tmpDir});
-      // Drops are collected, not printed: the programmatic API stays silent.
-      expect(warnSpy).not.toHaveBeenCalled();
-    } finally {
-      warnSpy.mockRestore();
-    }
-
-    const css = fs.readFileSync(path.join(tmpDir, 'dropped.css'), 'utf8');
-    expect(css).not.toContain('example.com');
-
-    const warnings = result?.data.warnings ?? [];
-    expect(warnings).toHaveLength(5);
-    expect(warnings).toEqual(
-      expect.arrayContaining([
-        expect.stringMatching(
-          /^Declaration dropped "--color-bg" in tokens: an unquoted "}" would close the rule/,
-        ),
-        expect.stringMatching(
-          /^Declaration dropped "background-color" in components\.button\["variant:secondary"\]: an unquoted ";" would end the declaration/,
-        ),
-        expect.stringMatching(
-          /^Declaration dropped "color" in components\.button\["variant:secondary"\]\[":hover"\]: an unclosed " string/,
-        ),
-        expect.stringMatching(
-          /^Declaration dropped "--color-bg" in onDark\.tokens: an unclosed url\(/,
-        ),
-        expect.stringMatching(
-          /^Declaration dropped "--color-bg" in adaptations\[0\]\.tokens: an unclosed \/\* comment/,
-        ),
-      ]),
-    );
-    for (const w of warnings) {
-      expect(w).toContain('The generated CSS omits it');
-    }
-  });
-
-  it('generates CSS for legacy null tokens without losing neighboring declarations', async () => {
-    fs.writeFileSync(
-      path.join(tmpDir, 'legacy.mjs'),
-      `export default {
-        name: 'legacy',
-        tokens: {'--color-background-body': null, '--spacing-4': 12},
-        onDark: {tokens: {'--color-background-body': null}},
-        components: {button: {base: {borderRadius: '4px'}}},
-      };\n`,
-    );
-
-    const result = await themeBuild('legacy.mjs', {}, {cwd: tmpDir});
-    const css = fs.readFileSync(path.join(tmpDir, 'legacy.css'), 'utf8');
-    expect(css).toContain('--color-background-body: null;');
-    expect(css).toContain('--spacing-4: 12;');
-    expect(css).toContain('border-radius: 4px;');
-    expect(result?.data.warnings).toEqual([]);
-  });
-
-  it('keeps valid CSS that carries semicolons, and reports nothing', async () => {
-    const themeFile = path.join(tmpDir, 'kept.mjs');
-    fs.writeFileSync(
-      themeFile,
-      `export default {
-        name: 'kept',
-        tokens: {'--font-family-body': 'Gill\\\\ Sans, "Segoe;UI", serif /* ; */'},
-        components: {
-          button: {
-            'variant:secondary': {
-              backgroundImage: 'URL(data:image/svg+xml;base64,PHN2Zz4=)',
-              WebkitLineClamp: '2',
-            },
-          },
-        },
-      };\n`,
-    );
-
-    const result = await themeBuild('kept.mjs', {}, {cwd: tmpDir});
-    const css = fs.readFileSync(path.join(tmpDir, 'kept.css'), 'utf8');
-
-    expect(css).toContain(
-      '--font-family-body: Gill\\ Sans, "Segoe;UI", serif /* ; */;',
-    );
-    expect(css).toContain(
-      'background-image: URL(data:image/svg+xml;base64,PHN2Zz4=);',
-    );
-    expect(css).toContain('-webkit-line-clamp: 2;');
-    expect(result?.data.warnings).toEqual([]);
-  });
-});
-
 describe('themeBuild() — nothing to build', () => {
   it('returns null and writes nothing when the generator yields no CSS', async () => {
     const themeFile = path.join(tmpDir, 'empty.mjs');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@astryxdesign/cli",
-  "version": "0.6.3-canary.45a9bf5",
+  "version": "0.6.3-canary.54be197",
   "displayName": "CLI",
   "description": "Scaffold projects, browse templates, generate themes, and get agent-ready docs from the command line.",
   "author": "Meta Open Source",
@@ -100,10 +100,10 @@
     "zod": "^4.4.3"
   },
   "peerDependencies": {
-    "@astryxdesign/charts": "0.6.3-canary.45a9bf5",
-    "@astryxdesign/core": "0.6.3-canary.45a9bf5",
-    "@astryxdesign/lab": "0.6.3-canary.45a9bf5",
-    "@astryxdesign/theme-neutral": "0.6.3-canary.45a9bf5",
+    "@astryxdesign/charts": "0.6.3-canary.54be197",
+    "@astryxdesign/core": "0.6.3-canary.54be197",
+    "@astryxdesign/lab": "0.6.3-canary.54be197",
+    "@astryxdesign/theme-neutral": "0.6.3-canary.54be197",
     "gpt-tokenizer": "^3.4.0"
   },
   "peerDependenciesMeta": {
@@ -121,10 +121,10 @@
     }
   },
   "devDependencies": {
-    "@astryxdesign/charts": "0.6.3-canary.45a9bf5",
-    "@astryxdesign/core": "0.6.3-canary.45a9bf5",
-    "@astryxdesign/lab": "0.6.3-canary.45a9bf5",
-    "@astryxdesign/theme-neutral": "0.6.3-canary.45a9bf5",
+    "@astryxdesign/charts": "0.6.3-canary.54be197",
+    "@astryxdesign/core": "0.6.3-canary.54be197",
+    "@astryxdesign/lab": "0.6.3-canary.54be197",
+    "@astryxdesign/theme-neutral": "0.6.3-canary.54be197",
     "@heroicons/react": "^2.2.0",
     "@stylexjs/stylex": "^0.19.0",
     "@types/babel__core": "^7.20.5",