@csszyx/compiler 0.10.7 → 0.10.8

package/dist/index.cjs

@@ -1,7 +1,7 @@
 'use strict';
 
 const core = require('@csszyx/core');
-const transformCore = require('./shared/compiler.DIYC1vdY.cjs');
+const transformCore = require('./shared/compiler.mibv6qPF.cjs');
 const oxcParser = require('oxc-parser');
 const t = require('@babel/types');
 const node_crypto = require('node:crypto');
@@ -245,6 +245,9 @@ function transformSourceCode(source, filename, options) {
               }
               const createMergedClassNameValue = (szExpr) => {
                 if (!existingClassExpr) {
+                  if (t__namespace.isStringLiteral(szExpr) && szExpr.value === "") {
+                    return t__namespace.jsxExpressionContainer(t__namespace.identifier("undefined"));
+                  }
                   return t__namespace.isStringLiteral(szExpr) ? szExpr : t__namespace.jsxExpressionContainer(szExpr);
                 }
                 if (existingClassNameNode && path.parentPath?.isJSXOpeningElement()) {
@@ -826,6 +829,9 @@ function parseStyleStringToObjectExpr(styleStr) {
   }
   return t__namespace.objectExpression(objProps);
 }
+function emptyClassToUndefined(node) {
+  return t__namespace.isStringLiteral(node) && node.value === "" ? t__namespace.identifier("undefined") : node;
+}
 function tryStaticTransformNode(node, getBinding) {
   if (t__namespace.isTSAsExpression(node) || t__namespace.isTSSatisfiesExpression(node)) {
     return tryStaticTransformNode(node.expression, getBinding);
@@ -862,7 +868,11 @@ function tryStaticTransformNode(node, getBinding) {
     const consequent = tryStaticTransformNode(node.consequent, getBinding);
     const alternate = tryStaticTransformNode(node.alternate, getBinding);
     if (consequent !== null && alternate !== null) {
-      return t__namespace.conditionalExpression(node.test, consequent, alternate);
+      return t__namespace.conditionalExpression(
+        node.test,
+        emptyClassToUndefined(consequent),
+        emptyClassToUndefined(alternate)
+      );
     }
     return null;
   }
@@ -899,7 +909,11 @@ function tryHoistConditionalSpread(node, getBinding) {
   if (!t__namespace.isStringLiteral(resolvedA) || !t__namespace.isStringLiteral(resolvedB)) {
     return null;
   }
-  return t__namespace.conditionalExpression(conditionalExpr.test, resolvedA, resolvedB);
+  return t__namespace.conditionalExpression(
+    conditionalExpr.test,
+    emptyClassToUndefined(resolvedA),
+    emptyClassToUndefined(resolvedB)
+  );
 }
 function readStaticConfigObject(configExpr, key, scope) {
   for (const prop of configExpr.properties) {
@@ -1029,20 +1043,20 @@ function buildConditionalClassExpr(baseClasses, conditionalClasses) {
   if (conditionalClasses.length === 0) {
     return t__namespace.stringLiteral(baseClasses);
   }
-  const makeCondExpr = (cc) => t__namespace.conditionalExpression(
+  const makeCondExpr = (cc, bare) => t__namespace.conditionalExpression(
     cc.test,
-    t__namespace.stringLiteral(cc.consequent),
-    t__namespace.stringLiteral(cc.alternate)
+    bare && cc.consequent === "" ? t__namespace.identifier("undefined") : t__namespace.stringLiteral(cc.consequent),
+    bare && cc.alternate === "" ? t__namespace.identifier("undefined") : t__namespace.stringLiteral(cc.alternate)
   );
   if (conditionalClasses.length === 1 && !baseClasses) {
-    return makeCondExpr(conditionalClasses[0]);
+    return makeCondExpr(conditionalClasses[0], true);
   }
   const quasis = [];
   const exprs = [];
   for (let i = 0; i < conditionalClasses.length; i++) {
     const prefix = i === 0 ? baseClasses ? `${baseClasses} ` : "" : " ";
     quasis.push(t__namespace.templateElement({ raw: prefix, cooked: prefix }, false));
-    exprs.push(makeCondExpr(conditionalClasses[i]));
+    exprs.push(makeCondExpr(conditionalClasses[i], false));
   }
   quasis.push(t__namespace.templateElement({ raw: "", cooked: "" }, true));
   return t__namespace.templateLiteral(quasis, exprs);
@@ -2769,7 +2783,7 @@ function transformOxc(source, filename, options) {
       ...existingRaw ? existingRaw.split(/\s+/).filter(Boolean) : [],
       ...szDerived
     ];
-    const mergedAttr = `className="${mergedClasses.join(" ")}"`;
+    const mergedAttr = mergedClasses.length === 0 ? "className={undefined}" : `className="${mergedClasses.join(" ")}"`;
     if (classNameAttr) {
       edits.overwrite(classNameAttr.start, classNameAttr.end, mergedAttr);
       for (const szAttr of szAttrs) {
@@ -3381,7 +3395,8 @@ function buildConditionalSpreadClassExpression(node, filename, bindings, source,
     }
   }
   const testSource = source.slice(conditionalSpread.test.start, conditionalSpread.test.end);
-  return `${testSource} ? ${JSON.stringify(consequent)} : ${JSON.stringify(alternate)}`;
+  const branch = (cls) => cls === "" ? "undefined" : JSON.stringify(cls);
+  return `${testSource} ? ${branch(consequent)} : ${branch(alternate)}`;
 }
 function compileConditionalSpreadBranch(branch, otherProps, sourceNode, filename, bindings, globalVarAliases, cssVariableMap) {
   const branchObject = resolveObjectExpression(branch, bindings);
@@ -3444,7 +3459,11 @@ function buildPartialObjectTransform(node, filename, bindings, source, options,
     classParts.push(entry.consequent, entry.alternate);
   }
   const className = classParts.filter(Boolean).join(" ");
-  const classNameAttr = partial.conditionalClasses.length > 0 ? `className={${buildConditionalClassSource(classParts, partial.conditionalClasses, source)}}` : `className="${className}"`;
+  const classNameAttr = partial.conditionalClasses.length > 0 ? `className={${buildConditionalClassSource(classParts, partial.conditionalClasses, source)}}` : className === "" ? (
+    // An sz that lowers to zero classes emits `className={undefined}` so the
+    // DOM has no `class` attribute, instead of the noisy `class=""`.
+    "className={undefined}"
+  ) : `className="${className}"`;
   const styleProps = [...partial.dynamicProps.entries()].filter(([id]) => !hoistedNames?.has(id)).map(
     ([, info]) => `${JSON.stringify(info.varName)}: ${generateStyleValueSource(info, source)}`
   );
@@ -4018,9 +4037,11 @@ function buildCSSVarClassName(info) {
 function buildConditionalClassSource(classParts, conditionals, source) {
   if (conditionals.length === 1) {
     const [entry] = conditionals;
-    const ternary = `${source.slice(entry.test.start, entry.test.end)} ? ${JSON.stringify(entry.consequent)} : ${JSON.stringify(entry.alternate)}`;
     const staticParts = classParts.slice(0, -2).filter(Boolean);
-    if (staticParts.length === 0) {
+    const bare = staticParts.length === 0;
+    const branch = (cls) => bare && cls === "" ? "undefined" : JSON.stringify(cls);
+    const ternary = `${source.slice(entry.test.start, entry.test.end)} ? ${branch(entry.consequent)} : ${branch(entry.alternate)}`;
+    if (bare) {
       return ternary;
     }
     return `\`${staticParts.join(" ")} \${${ternary}}\``;
@@ -4072,7 +4093,8 @@ function buildStaticConditionalClassExpression(node, filename, bindings, source,
     }
   }
   const testSource = source.slice(node.test.start, node.test.end);
-  return `${testSource} ? ${JSON.stringify(consequent)} : ${JSON.stringify(alternate)}`;
+  const branch = (cls) => cls === "" ? "undefined" : JSON.stringify(cls);
+  return `${testSource} ? ${branch(consequent)} : ${branch(alternate)}`;
 }
 function resolveStaticClassString(node, filename, bindings, globalVarAliases, cssVariableMap) {
   const unwrapped = unwrapExpression(node);

package/dist/index.mjs

@@ -1,6 +1,6 @@
 import { init, version, transform_sz, encode } from '@csszyx/core';
-import { t as transform, s as setSzWarnLocation, f as formatSzWarnLocation, C as COLOR_PROPERTIES, P as PROPERTY_MAP, g as getCSSVariableName, a as PropertyCategory, K as KNOWN_VARIANTS, b as getVariantPrefix, c as getPropertyCategory, d as stripInvalidColorStrings } from './shared/compiler.DINxoUCF.mjs';
-export { B as BOOLEAN_SHORTHANDS, e as PROPERTY_CATEGORY_MAP, R as REMOVED_BOOLEAN_SUGAR, S as SPECIAL_VARIANTS, h as SUGGESTION_MAP, i as isValidSzProp, n as normalizeClassName } from './shared/compiler.DINxoUCF.mjs';
+import { t as transform, s as setSzWarnLocation, f as formatSzWarnLocation, C as COLOR_PROPERTIES, P as PROPERTY_MAP, g as getCSSVariableName, a as PropertyCategory, K as KNOWN_VARIANTS, b as getVariantPrefix, c as getPropertyCategory, d as stripInvalidColorStrings } from './shared/compiler.CghwJ6p5.mjs';
+export { B as BOOLEAN_SHORTHANDS, e as PROPERTY_CATEGORY_MAP, R as REMOVED_BOOLEAN_SUGAR, S as SPECIAL_VARIANTS, h as SUGGESTION_MAP, i as isValidSzProp, n as normalizeClassName } from './shared/compiler.CghwJ6p5.mjs';
 import { parseSync } from 'oxc-parser';
 import * as t from '@babel/types';
 import { createHash } from 'node:crypto';
@@ -226,6 +226,9 @@ function transformSourceCode(source, filename, options) {
               }
               const createMergedClassNameValue = (szExpr) => {
                 if (!existingClassExpr) {
+                  if (t.isStringLiteral(szExpr) && szExpr.value === "") {
+                    return t.jsxExpressionContainer(t.identifier("undefined"));
+                  }
                   return t.isStringLiteral(szExpr) ? szExpr : t.jsxExpressionContainer(szExpr);
                 }
                 if (existingClassNameNode && path.parentPath?.isJSXOpeningElement()) {
@@ -807,6 +810,9 @@ function parseStyleStringToObjectExpr(styleStr) {
   }
   return t.objectExpression(objProps);
 }
+function emptyClassToUndefined(node) {
+  return t.isStringLiteral(node) && node.value === "" ? t.identifier("undefined") : node;
+}
 function tryStaticTransformNode(node, getBinding) {
   if (t.isTSAsExpression(node) || t.isTSSatisfiesExpression(node)) {
     return tryStaticTransformNode(node.expression, getBinding);
@@ -843,7 +849,11 @@ function tryStaticTransformNode(node, getBinding) {
     const consequent = tryStaticTransformNode(node.consequent, getBinding);
     const alternate = tryStaticTransformNode(node.alternate, getBinding);
     if (consequent !== null && alternate !== null) {
-      return t.conditionalExpression(node.test, consequent, alternate);
+      return t.conditionalExpression(
+        node.test,
+        emptyClassToUndefined(consequent),
+        emptyClassToUndefined(alternate)
+      );
     }
     return null;
   }
@@ -880,7 +890,11 @@ function tryHoistConditionalSpread(node, getBinding) {
   if (!t.isStringLiteral(resolvedA) || !t.isStringLiteral(resolvedB)) {
     return null;
   }
-  return t.conditionalExpression(conditionalExpr.test, resolvedA, resolvedB);
+  return t.conditionalExpression(
+    conditionalExpr.test,
+    emptyClassToUndefined(resolvedA),
+    emptyClassToUndefined(resolvedB)
+  );
 }
 function readStaticConfigObject(configExpr, key, scope) {
   for (const prop of configExpr.properties) {
@@ -1010,20 +1024,20 @@ function buildConditionalClassExpr(baseClasses, conditionalClasses) {
   if (conditionalClasses.length === 0) {
     return t.stringLiteral(baseClasses);
   }
-  const makeCondExpr = (cc) => t.conditionalExpression(
+  const makeCondExpr = (cc, bare) => t.conditionalExpression(
     cc.test,
-    t.stringLiteral(cc.consequent),
-    t.stringLiteral(cc.alternate)
+    bare && cc.consequent === "" ? t.identifier("undefined") : t.stringLiteral(cc.consequent),
+    bare && cc.alternate === "" ? t.identifier("undefined") : t.stringLiteral(cc.alternate)
   );
   if (conditionalClasses.length === 1 && !baseClasses) {
-    return makeCondExpr(conditionalClasses[0]);
+    return makeCondExpr(conditionalClasses[0], true);
   }
   const quasis = [];
   const exprs = [];
   for (let i = 0; i < conditionalClasses.length; i++) {
     const prefix = i === 0 ? baseClasses ? `${baseClasses} ` : "" : " ";
     quasis.push(t.templateElement({ raw: prefix, cooked: prefix }, false));
-    exprs.push(makeCondExpr(conditionalClasses[i]));
+    exprs.push(makeCondExpr(conditionalClasses[i], false));
   }
   quasis.push(t.templateElement({ raw: "", cooked: "" }, true));
   return t.templateLiteral(quasis, exprs);
@@ -2750,7 +2764,7 @@ function transformOxc(source, filename, options) {
       ...existingRaw ? existingRaw.split(/\s+/).filter(Boolean) : [],
       ...szDerived
     ];
-    const mergedAttr = `className="${mergedClasses.join(" ")}"`;
+    const mergedAttr = mergedClasses.length === 0 ? "className={undefined}" : `className="${mergedClasses.join(" ")}"`;
     if (classNameAttr) {
       edits.overwrite(classNameAttr.start, classNameAttr.end, mergedAttr);
       for (const szAttr of szAttrs) {
@@ -3362,7 +3376,8 @@ function buildConditionalSpreadClassExpression(node, filename, bindings, source,
     }
   }
   const testSource = source.slice(conditionalSpread.test.start, conditionalSpread.test.end);
-  return `${testSource} ? ${JSON.stringify(consequent)} : ${JSON.stringify(alternate)}`;
+  const branch = (cls) => cls === "" ? "undefined" : JSON.stringify(cls);
+  return `${testSource} ? ${branch(consequent)} : ${branch(alternate)}`;
 }
 function compileConditionalSpreadBranch(branch, otherProps, sourceNode, filename, bindings, globalVarAliases, cssVariableMap) {
   const branchObject = resolveObjectExpression(branch, bindings);
@@ -3425,7 +3440,11 @@ function buildPartialObjectTransform(node, filename, bindings, source, options,
     classParts.push(entry.consequent, entry.alternate);
   }
   const className = classParts.filter(Boolean).join(" ");
-  const classNameAttr = partial.conditionalClasses.length > 0 ? `className={${buildConditionalClassSource(classParts, partial.conditionalClasses, source)}}` : `className="${className}"`;
+  const classNameAttr = partial.conditionalClasses.length > 0 ? `className={${buildConditionalClassSource(classParts, partial.conditionalClasses, source)}}` : className === "" ? (
+    // An sz that lowers to zero classes emits `className={undefined}` so the
+    // DOM has no `class` attribute, instead of the noisy `class=""`.
+    "className={undefined}"
+  ) : `className="${className}"`;
   const styleProps = [...partial.dynamicProps.entries()].filter(([id]) => !hoistedNames?.has(id)).map(
     ([, info]) => `${JSON.stringify(info.varName)}: ${generateStyleValueSource(info, source)}`
   );
@@ -3999,9 +4018,11 @@ function buildCSSVarClassName(info) {
 function buildConditionalClassSource(classParts, conditionals, source) {
   if (conditionals.length === 1) {
     const [entry] = conditionals;
-    const ternary = `${source.slice(entry.test.start, entry.test.end)} ? ${JSON.stringify(entry.consequent)} : ${JSON.stringify(entry.alternate)}`;
     const staticParts = classParts.slice(0, -2).filter(Boolean);
-    if (staticParts.length === 0) {
+    const bare = staticParts.length === 0;
+    const branch = (cls) => bare && cls === "" ? "undefined" : JSON.stringify(cls);
+    const ternary = `${source.slice(entry.test.start, entry.test.end)} ? ${branch(entry.consequent)} : ${branch(entry.alternate)}`;
+    if (bare) {
       return ternary;
     }
     return `\`${staticParts.join(" ")} \${${ternary}}\``;
@@ -4053,7 +4074,8 @@ function buildStaticConditionalClassExpression(node, filename, bindings, source,
     }
   }
   const testSource = source.slice(node.test.start, node.test.end);
-  return `${testSource} ? ${JSON.stringify(consequent)} : ${JSON.stringify(alternate)}`;
+  const branch = (cls) => cls === "" ? "undefined" : JSON.stringify(cls);
+  return `${testSource} ? ${branch(consequent)} : ${branch(alternate)}`;
 }
 function resolveStaticClassString(node, filename, bindings, globalVarAliases, cssVariableMap) {
   const unwrapped = unwrapExpression(node);

package/dist/shared/{compiler.DINxoUCF.mjs → compiler.CghwJ6p5.mjs}

@@ -1496,6 +1496,16 @@ function handleSupports(supportsObj, prefix) {
 }
 let szTransformDepth = 0;
 let szWarnLocation;
+let szHintedProjectScan = false;
+function hintProjectScanOnce(location) {
+  if (szHintedProjectScan || !location || process.env.CSSZYX_NO_PROJECT_SCAN_HINT === "1") {
+    return;
+  }
+  szHintedProjectScan = true;
+  console.warn(
+    "[csszyx] Tip: run `npx @csszyx/cli check` to scan every file for sz key issues at once (dev warnings only surface files as you open them)."
+  );
+}
 function setSzWarnLocation(location) {
   szWarnLocation = location;
 }
@@ -2370,6 +2380,7 @@ function transformImpl(szProp, prefix, mangleMap) {
             `[csszyx] Unknown property "${rawKey}" in sz prop${at}. This will be ignored. Check for typos.`
           );
         }
+        hintProjectScanOnce(szWarnLocation);
       }
     }
     if (value === true) {

package/dist/shared/{compiler.DIYC1vdY.cjs → compiler.mibv6qPF.cjs}

@@ -1498,6 +1498,16 @@ function handleSupports(supportsObj, prefix) {
 }
 let szTransformDepth = 0;
 let szWarnLocation;
+let szHintedProjectScan = false;
+function hintProjectScanOnce(location) {
+  if (szHintedProjectScan || !location || process.env.CSSZYX_NO_PROJECT_SCAN_HINT === "1") {
+    return;
+  }
+  szHintedProjectScan = true;
+  console.warn(
+    "[csszyx] Tip: run `npx @csszyx/cli check` to scan every file for sz key issues at once (dev warnings only surface files as you open them)."
+  );
+}
 function setSzWarnLocation(location) {
   szWarnLocation = location;
 }
@@ -2372,6 +2382,7 @@ function transformImpl(szProp, prefix, mangleMap) {
             `[csszyx] Unknown property "${rawKey}" in sz prop${at}. This will be ignored. Check for typos.`
           );
         }
+        hintProjectScanOnce(szWarnLocation);
       }
     }
     if (value === true) {

package/dist/transform-core.cjs

@@ -1,6 +1,6 @@
 'use strict';
 
-const transformCore = require('./shared/compiler.DIYC1vdY.cjs');
+const transformCore = require('./shared/compiler.mibv6qPF.cjs');
 
 
 

package/dist/transform-core.mjs

@@ -1 +1 @@
-export { B as BOOLEAN_SHORTHANDS, K as KNOWN_VARIANTS, M as MAX_SZ_DEPTH, P as PROPERTY_MAP, R as REMOVED_BOOLEAN_SUGAR, S as SPECIAL_VARIANTS, h as SUGGESTION_MAP, j as SzDepthError, V as VARIANT_MAP, f as formatSzWarnLocation, b as getVariantPrefix, k as isForbiddenSzKey, i as isValidSzProp, l as normalizeArbitraryValue, m as normalizeArbitraryVariant, n as normalizeClassName, s as setSzWarnLocation, t as transform } from './shared/compiler.DINxoUCF.mjs';
+export { B as BOOLEAN_SHORTHANDS, K as KNOWN_VARIANTS, M as MAX_SZ_DEPTH, P as PROPERTY_MAP, R as REMOVED_BOOLEAN_SUGAR, S as SPECIAL_VARIANTS, h as SUGGESTION_MAP, j as SzDepthError, V as VARIANT_MAP, f as formatSzWarnLocation, b as getVariantPrefix, k as isForbiddenSzKey, i as isValidSzProp, l as normalizeArbitraryValue, m as normalizeArbitraryVariant, n as normalizeClassName, s as setSzWarnLocation, t as transform } from './shared/compiler.CghwJ6p5.mjs';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@csszyx/compiler",
-  "version": "0.10.7",
+  "version": "0.10.8",
   "description": "Core compiler and transformation logic for csszyx",
   "keywords": [
     "csszyx",
@@ -65,7 +65,7 @@
     "@babel/types": "^7.23.6",
     "magic-string": "0.30.21",
     "oxc-parser": "0.131.0",
-    "@csszyx/core": "0.10.7"
+    "@csszyx/core": "0.10.8"
   },
   "devDependencies": {
     "@types/babel__core": "^7.20.5",