@stylexjs/babel-plugin 0.17.2 → 0.17.4

package/lib/index.js

@@ -283,6 +283,45 @@ function getProgramStatement(path) {
   }
   return programPath;
 }
+function isVariableNamedExported(path) {
+  const idPath = path.get('id');
+  if (!idPath.isIdentifier()) return false;
+  const variableName = idPath.node.name;
+  const binding = path.scope.getBinding(variableName);
+  if (!binding) return false;
+  const programPath = getProgramPath(path);
+  if (programPath == null) {
+    return false;
+  }
+  let exported = false;
+  programPath.traverse({
+    ExportNamedDeclaration(p) {
+      const node = p.node;
+      if (node.source != null) return;
+      if (node.declaration) {
+        if (node.declaration.type === 'VariableDeclaration' && node.declaration.declarations.some(d => d.id.type === 'Identifier' && p.scope.getBinding(d.id.name) === binding)) {
+          exported = true;
+          p.stop();
+        }
+        return;
+      }
+      for (const s of node.specifiers ?? []) {
+        if (s.type !== 'ExportSpecifier') continue;
+        if (s.local.type !== 'Identifier') continue;
+        if (s.exported.type !== 'Identifier') continue;
+        if (s.exportKind === 'type') continue;
+        if (s.local.name !== s.exported.name) continue;
+        const localBinding = p.scope.getBinding(s.local.name);
+        if (localBinding === binding) {
+          exported = true;
+          p.stop();
+          return;
+        }
+      }
+    }
+  });
+  return exported;
+}
 
 const defaultOptions = {
   classNamePrefix: 'x',
@@ -602,15 +641,14 @@ class StateManager {
         ltr,
         rtl
       } = styleObj;
-      const args = [t__namespace.stringLiteral(ltr), t__namespace.numericLiteral(priority), ...(rtl != null ? [t__namespace.stringLiteral(rtl)] : [])];
-      if ('constKey' in styleObj && 'constVal' in styleObj) {
-        const {
-          constKey,
-          constVal
-        } = styleObj;
-        args.push(t__namespace.stringLiteral(constKey), typeof constVal === 'number' ? t__namespace.numericLiteral(constVal) : t__namespace.stringLiteral(String(constVal)));
+      let constKey = null;
+      let constVal = null;
+      if (styleObj.constKey != null && styleObj.constVal != null) {
+        constKey = styleObj.constKey;
+        constVal = styleObj.constVal;
       }
-      statementPath.insertBefore(t__namespace.expressionStatement(t__namespace.callExpression(injectName, args)));
+      const properties = [t__namespace.objectProperty(t__namespace.identifier('ltr'), t__namespace.stringLiteral(ltr)), ...(rtl != null ? [t__namespace.objectProperty(t__namespace.identifier('rtl'), t__namespace.stringLiteral(rtl))] : []), t__namespace.objectProperty(t__namespace.identifier('priority'), t__namespace.numericLiteral(priority)), ...(constKey != null ? [t__namespace.objectProperty(t__namespace.identifier('constKey'), t__namespace.stringLiteral(constKey)), t__namespace.objectProperty(t__namespace.identifier('constVal'), typeof constVal === 'number' ? t__namespace.numericLiteral(constVal) : t__namespace.stringLiteral(String(constVal)))] : [])];
+      statementPath.insertBefore(t__namespace.expressionStatement(t__namespace.callExpression(injectName, [t__namespace.objectExpression(properties)])));
     }
   }
   markComposedNamespace(memberExpression) {
@@ -5669,7 +5707,6 @@ function deopt(path, state, reason) {
   state.deoptReason = reason;
 }
 function evaluateImportedFile(filePath, namedExport, state, bindingPath) {
-  const fs = require('node:fs');
   const fileContents = fs.readFileSync(filePath, 'utf8');
   const ast = core.parseSync(fileContents, {
     babelrc: true
@@ -6951,11 +6988,10 @@ function transformStyleXDefineVars(callExpressionPath, state) {
 }
 function validateStyleXDefineVars(callExpressionPath) {
   const variableDeclaratorPath = callExpressionPath.parentPath;
-  const exportNamedDeclarationPath = variableDeclaratorPath.parentPath?.parentPath;
   if (variableDeclaratorPath == null || variableDeclaratorPath.isExpressionStatement() || !variableDeclaratorPath.isVariableDeclarator() || variableDeclaratorPath.node.id.type !== 'Identifier') {
     throw callExpressionPath.buildCodeFrameError(messages.unboundCallValue('defineVars'), SyntaxError);
   }
-  if (exportNamedDeclarationPath == null || !exportNamedDeclarationPath.isExportNamedDeclaration()) {
+  if (!isVariableNamedExported(variableDeclaratorPath)) {
     throw callExpressionPath.buildCodeFrameError(messages.nonExportNamedDeclaration('defineVars'), SyntaxError);
   }
   if (callExpressionPath.node.arguments.length !== 1) {
@@ -7010,11 +7046,10 @@ function transformStyleXDefineConsts(callExpressionPath, state) {
 }
 function validateStyleXDefineConsts(callExpressionPath) {
   const variableDeclaratorPath = callExpressionPath.parentPath;
-  const exportNamedDeclarationPath = variableDeclaratorPath.parentPath?.parentPath;
   if (variableDeclaratorPath == null || !variableDeclaratorPath.isVariableDeclarator() || variableDeclaratorPath.node.id.type !== 'Identifier') {
     throw callExpressionPath.buildCodeFrameError(messages.unboundCallValue('defineConsts'), SyntaxError);
   }
-  if (exportNamedDeclarationPath == null || !exportNamedDeclarationPath.isExportNamedDeclaration()) {
+  if (!isVariableNamedExported(variableDeclaratorPath)) {
     throw callExpressionPath.buildCodeFrameError(messages.nonExportNamedDeclaration('defineConsts'), SyntaxError);
   }
   if (callExpressionPath.node.arguments.length !== 1) {
@@ -7857,11 +7892,10 @@ function transformStyleXDefineMarker(path, state) {
 }
 function validateStyleXDefineMarker(path) {
   const variableDeclaratorPath = path.parentPath;
-  const exportNamedDeclarationPath = variableDeclaratorPath.parentPath?.parentPath;
   if (variableDeclaratorPath == null || variableDeclaratorPath.isExpressionStatement() || !variableDeclaratorPath.isVariableDeclarator() || variableDeclaratorPath.node.id.type !== 'Identifier') {
     throw path.buildCodeFrameError(unboundCallValue('defineMarker'), SyntaxError);
   }
-  if (exportNamedDeclarationPath == null || !exportNamedDeclarationPath.isExportNamedDeclaration()) {
+  if (!isVariableNamedExported(variableDeclaratorPath)) {
     throw path.buildCodeFrameError(nonExportNamedDeclaration('defineMarker'), SyntaxError);
   }
 }

package/lib/shared/types/index.d.ts

@@ -187,7 +187,6 @@ export declare class Num<T extends NumberValue>
 export declare const number: <T extends NumberValue = NumberValue>(
   value: NestedWithNumbers,
 ) => Num<T>;
-export declare type number = typeof number;
 type ResolutionValue = string | 0;
 export declare class Resolution<T extends ResolutionValue>
   extends BaseCSSType

package/lib/utils/ast-helpers.d.ts

@@ -35,3 +35,14 @@ export declare function addDefaultImport(
 ): t.Identifier;
 export declare function isProgramLevel(path: NodePath): boolean;
 export declare function getProgramStatement(path: NodePath): NodePath;
+/**
+ * Checks if a variable with the given name is named exported in the program.
+ * This handles both:
+ * - Direct named exports: `export const x = ...`
+ * - Locally declared named exports: `const x = ...; export { x }`
+ *
+ * Default exports, aliasing and re-exports from other files (e.g., `export { x } from './other'`) are NOT allowed.
+ */
+export declare function isVariableNamedExported(
+  path: NodePath<t.VariableDeclarator>,
+): boolean;

package/lib/utils/ast-helpers.js.flow

@@ -42,3 +42,15 @@ declare export function addDefaultImport(
 declare export function isProgramLevel(path: NodePath<>): boolean;
 
 declare export function getProgramStatement(path: NodePath<>): NodePath<>;
+
+/**
+ * Checks if a variable with the given name is named exported in the program.
+ * This handles both:
+ * - Direct named exports: `export const x = ...`
+ * - Locally declared named exports: `const x = ...; export { x }`
+ *
+ * Default exports, aliasing and re-exports from other files (e.g., `export { x } from './other'`) are NOT allowed.
+ */
+declare export function isVariableNamedExported(
+  path: NodePath<t.VariableDeclarator>,
+): boolean;

package/lib/visitors/stylex-define-consts.js.flow

@@ -11,7 +11,6 @@ import type { NodePath } from '@babel/traverse';
 
 import * as t from '@babel/types';
 import StateManager from '../utils/state-manager';
-
 /// This function looks for `stylex.defineConsts` calls and transforms them.
 /// 1. It finds and validates the first argument to `stylex.defineConsts`.
 /// 2. It evaluates the style const object to a JS value, erroring on non-static or non-object values.

package/package.json

@@ -1,8 +1,10 @@
 {
   "name": "@stylexjs/babel-plugin",
-  "version": "0.17.2",
+  "version": "0.17.4",
   "description": "StyleX babel plugin.",
   "main": "lib/index.js",
+  "browser": "lib/index.browser.js",
+  "types": "./lib/index.d.ts",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/facebook/stylex.git"
@@ -21,8 +23,8 @@
     "@babel/traverse": "^7.26.8",
     "@babel/types": "^7.26.8",
     "@dual-bundle/import-meta-resolve": "^4.1.0",
-    "@stylexjs/shared": "0.17.2",
-    "@stylexjs/stylex": "0.17.2",
+    "@stylexjs/shared": "0.17.4",
+    "@stylexjs/stylex": "0.17.4",
     "postcss-value-parser": "^4.1.0"
   },
   "devDependencies": {
@@ -33,8 +35,9 @@
     "@rollup/plugin-node-resolve": "^15.3.0",
     "@rollup/plugin-replace": "^6.0.1",
     "babel-plugin-syntax-hermes-parser": "^0.32.1",
+    "path-browserify": "^1.0.1",
     "rollup": "^4.24.0",
-    "scripts": "0.17.2"
+    "scripts": "0.17.4"
   },
   "files": [
     "flow_modules/*",