@atlaskit/eslint-plugin-design-system 8.19.1 → 8.20.0

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # @atlaskit/eslint-plugin-design-system
 
+## 8.20.0
+
+### Minor Changes
+
+- [#66409](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/66409) [`f6c48f4a67c1`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/f6c48f4a67c1) - Implemented functionality for the consistent-css-prop-usage rule to account for cssMap usages
+
+### Patch Changes
+
+- [#66604](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/66604) [`3205b1daf57f`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/3205b1daf57f) - Refactor internal logic of `use-primitives` to better handle cases it will not suggest transformations for, based on the value of CSS properties.
+
+## 8.19.2
+
+### Patch Changes
+
+- [#66118](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/66118) [`93988e6fd035`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/93988e6fd035) - `use-primitives` now handles tokenised padding/margin properties. This change is guarded by a config flag and not enabled by default.
+
 ## 8.19.1
 
 ### Patch Changes

package/constellation/index/usage.mdx

@@ -142,7 +142,7 @@ This rule comes with options to support different repository configurations.
 
 #### cssFunctions
 
-An array of function names the linting rule should target. Defaults to `['css', 'xcss']`.
+An array of function names the linting rule should target. Defaults to `['css', 'xcss']`. Functionality of cssMap will be linted regardless of the configuration of `cssFunctions` as it can be used with either attribute.
 
 #### stylesPlacement
 

package/dist/cjs/rules/consistent-css-prop-usage/index.js

@@ -6,6 +6,7 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
+var _toConsumableArray2 = _interopRequireDefault(require("@babel/runtime/helpers/toConsumableArray"));
 var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
 var _eslintCodemodUtils = require("eslint-codemod-utils");
 var _assign = _interopRequireDefault(require("lodash/assign"));
@@ -25,9 +26,9 @@ function findSpreadProperties(node) {
   });
 }
 function analyzeIdentifier(context, sourceIdentifier, configuration) {
-  var _getIdentifierInParen;
+  var _getIdentifierInParen, _getIdentifierInParen2;
   var scope = context.getScope();
-  var _ref = ((_getIdentifierInParen = (0, _eslintCodemodUtils.getIdentifierInParentScope)(scope, sourceIdentifier.name)) === null || _getIdentifierInParen === void 0 ? void 0 : _getIdentifierInParen.identifiers) || [],
+  var _ref = (_getIdentifierInParen = (_getIdentifierInParen2 = (0, _eslintCodemodUtils.getIdentifierInParentScope)(scope, sourceIdentifier.name)) === null || _getIdentifierInParen2 === void 0 ? void 0 : _getIdentifierInParen2.identifiers) !== null && _getIdentifierInParen !== void 0 ? _getIdentifierInParen : [],
     _ref2 = (0, _slicedToArray2.default)(_ref, 1),
     identifier = _ref2[0];
   if (!identifier || !identifier.parent) {
@@ -50,7 +51,7 @@ function analyzeIdentifier(context, sourceIdentifier, configuration) {
     });
     return;
   }
-  if (identifier.parent && identifier.parent.init && !isCssCallExpression(identifier.parent.init, configuration.cssFunctions)) {
+  if (identifier.parent && identifier.parent.init && !isCssCallExpression(identifier.parent.init, [].concat((0, _toConsumableArray2.default)(configuration.cssFunctions), ['cssMap']))) {
     // When variable value is not of type css({})
     context.report({
       node: identifier,
@@ -69,6 +70,13 @@ function analyzeIdentifier(context, sourceIdentifier, configuration) {
     });
   }
 }
+
+/**
+ * Handle different cases based on what's been passed in the css-related JSXAttribute
+ * @param context the context of the node
+ * @param expression the expression of the JSXAttribute value
+ * @param configuration what css-related functions to account for (eg. css, xcss, cssMap), and whether to detect bottom vs top expressions
+ */
 var traverseExpressionWithConfig = function traverseExpressionWithConfig(context, expression, configuration) {
   function traverseExpression(expression) {
     switch (expression.type) {
@@ -140,7 +148,8 @@ var rule = (0, _createRule.createLintRule)({
   create: function create(context) {
     var _ref3;
     var mergedConfig = (0, _assign.default)({}, defaultConfig, context.options[0]);
-    var callSelector = mergedConfig.cssFunctions.map(function (fn) {
+    var callSelectorFunctions = [].concat((0, _toConsumableArray2.default)(mergedConfig.cssFunctions), ['cssMap']);
+    var callSelector = callSelectorFunctions.map(function (fn) {
       return "CallExpression[callee.name=".concat(fn, "]");
     }).join(',');
     return _ref3 = {}, (0, _defineProperty2.default)(_ref3, callSelector, function (node) {

package/dist/cjs/rules/use-primitives/config/index.js

@@ -5,7 +5,7 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.getConfig = void 0;
 var defaults = {
-  patterns: ['div-object-css']
+  patterns: ['compiled-css-function']
 };
 var getConfig = exports.getConfig = function getConfig(overrides) {
   // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/assign

package/dist/cjs/rules/use-primitives/index.js

@@ -37,7 +37,7 @@ var rule = (0, _createRule.createLintRule)({
           return;
         }
         var styledComponentVariableRef = (0, _utils.findValidStyledComponentCall)(node);
-        if (!styledComponentVariableRef || !(0, _eslintCodemodUtils.isNodeOfType)(styledComponentVariableRef.id, 'Identifier') || !(0, _utils.isValidCssPropertiesToTransform)(node)) {
+        if (!styledComponentVariableRef || !(0, _eslintCodemodUtils.isNodeOfType)(styledComponentVariableRef.id, 'Identifier') || !(0, _utils.isValidCssPropertiesToTransform)(node, config)) {
           return;
         }
         var styledComponentJsxRef = (0, _utils.findValidJsxUsageToTransform)(styledComponentVariableRef.id.name, context.getScope());
@@ -60,7 +60,9 @@ var rule = (0, _createRule.createLintRule)({
       },
       // transforms <div css={...}> usages
       JSXOpeningElement: function JSXOpeningElement(node) {
-        var config = (0, _config.getConfig)(context.options[0]);
+        if (!config.patterns.includes('compiled-css-function')) {
+          return false;
+        }
         if (!(0, _eslintCodemodUtils.isNodeOfType)(node, 'JSXOpeningElement')) {
           return;
         }
@@ -135,9 +137,6 @@ var shouldSuggestBox = function shouldSuggestBox(node, context, config
   if (!cssVariableValue || !(0, _utils.isFunctionNamed)(cssVariableValue, 'css')) {
     return false;
   }
-  if (!config.patterns.includes('div-object-css')) {
-    return false;
-  }
-  return (0, _utils.isValidCssPropertiesToTransform)(cssVariableValue.node.init);
+  return (0, _utils.isValidCssPropertiesToTransform)(cssVariableValue.node.init, config);
 };
 var _default = exports.default = rule;

package/dist/cjs/rules/use-primitives/transformers/css-to-xcss.js

@@ -39,7 +39,7 @@ var cssToXcssTransformer = exports.cssToXcssTransformer = function cssToXcssTran
   fixer.replaceText(cssVariableValue.node.init.callee, (0, _eslintCodemodUtils.identifier)('xcss').toString())].concat((0, _toConsumableArray2.default)(styledObjectToXcssTokens(cssObjectExpression, fixer)));
 };
 
-// Update css object values to xcss values. e.g. `'8px'` -> `'space.100'`
+// Update css object values to xcss values
 // Note: `properties` in this context is a group of AST nodes that make up a key/value pair in an object.
 // e.g. `padding: '8px'`. For clarity, it's renamed to `entry` inside the `.map()`.
 var styledObjectToXcssTokens = exports.styledObjectToXcssTokens = function styledObjectToXcssTokens(styles, fixer) {
@@ -50,16 +50,30 @@ var styledObjectToXcssTokens = exports.styledObjectToXcssTokens = function style
     if (!(0, _eslintCodemodUtils.isNodeOfType)(entry.key, 'Identifier')) {
       return;
     }
-    if (!(0, _eslintCodemodUtils.isNodeOfType)(entry.value, 'Literal')) {
-      return;
+
+    // maps literal values like: 8px to 'space.100'
+    if ((0, _eslintCodemodUtils.isNodeOfType)(entry.value, 'Literal')) {
+      var value = entry.value.value;
+      if (typeof value !== 'string') {
+        return;
+      }
+      return fixer.replaceText(entry.value, (0, _eslintCodemodUtils.literal)("'".concat(supportedStylesMap[entry.key.name][value], "'")).toString());
     }
-    var value = entry.value.value;
-    if (typeof value !== 'string') {
-      return;
+    // maps token calls like: token('space.100') to 'space.100'
+    if ((0, _eslintCodemodUtils.isNodeOfType)(entry.value, 'CallExpression')) {
+      var callExpression = entry.value;
+      // skip if not a call to `token`
+      if (!(0, _eslintCodemodUtils.isNodeOfType)(callExpression.callee, 'Identifier') || callExpression.callee.name !== 'token' || !(0, _eslintCodemodUtils.isNodeOfType)(callExpression.arguments[0], 'Literal')) {
+        return;
+      }
+      // the first argument of `token` is the token name and
+      // can be given directly to `xcss` as it has been validated already.
+      return fixer.replaceText(entry.value, (0, _eslintCodemodUtils.literal)("'".concat(callExpression.arguments[0].value, "'")).toString());
     }
-    return fixer.replaceText(entry.value, (0, _eslintCodemodUtils.literal)("'".concat(supportedStylesMap[entry.key.name][value], "'")).toString());
   });
 };
+
+// TODO: https://product-fabric.atlassian.net/browse/DSP-16054
 var spaceTokenMap = exports.spaceTokenMap = {
   '0px': 'space.0',
   '2px': 'space.025',

package/dist/cjs/rules/use-primitives/utils/convert-ast-object-expression-to-js-object.js

@@ -3,31 +3,23 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.convertASTObjectExpressionToJSObject = void 0;
+exports.convertASTObjectExpressionToJSObject = exports.SPREAD_SYNTAX = void 0;
 var _eslintCodemodUtils = require("eslint-codemod-utils");
-// FIXME: This is only loosely typed
-
+var SPREAD_SYNTAX = exports.SPREAD_SYNTAX = Symbol('SPREAD_SYNTAX');
 /**
  * Note: Not recursive. Only handles top level key/value pairs
  */
 var convertASTObjectExpressionToJSObject = exports.convertASTObjectExpressionToJSObject = function convertASTObjectExpressionToJSObject(styles) {
-  var styleObj = {};
+  var styleObj = {
+    unsupported: []
+  };
 
-  // if we see any spread props we stop and return false to indicate this is unsupported
+  // if we see any spread props we indicate that as unsupported
   if (!styles.properties.every(function (prop) {
     return (0, _eslintCodemodUtils.isNodeOfType)(prop, 'Property');
   })) {
-    return false;
+    styleObj.unsupported.push(SPREAD_SYNTAX);
   }
-
-  // TODO: We need to harden this logic.
-  // It currently generates a false positive for:
-  // styled.div({
-  //  marginTop: "0px",
-  //  marginBottom: token("space.100", "8px"),
-  // })
-  // as the value for `marginBottom` is not a string, so it is just skipped
-  // from the resulting map and this causes the rule to trigger when it shouldn't
   styles.properties.forEach(function (prop) {
     if (!(0, _eslintCodemodUtils.isNodeOfType)(prop, 'Property')) {
       return;
@@ -35,13 +27,34 @@ var convertASTObjectExpressionToJSObject = exports.convertASTObjectExpressionToJ
     if (!(0, _eslintCodemodUtils.isNodeOfType)(prop.key, 'Identifier')) {
       return;
     }
-    if (!(0, _eslintCodemodUtils.isNodeOfType)(prop.value, 'Literal')) {
+
+    // a literal string value, the base case
+    if ((0, _eslintCodemodUtils.isNodeOfType)(prop.value, 'Literal') && typeof prop.value.value === 'string') {
+      styleObj[prop.key.name] = prop.value.value;
       return;
     }
-    if (typeof prop.value.value !== 'string') {
-      return;
+
+    // try to handle a direct call to `token`
+    if ((0, _eslintCodemodUtils.isNodeOfType)(prop.value, 'CallExpression')) {
+      var callExpression = prop.value;
+      // strictly handle calls to `token`
+      if ((0, _eslintCodemodUtils.isNodeOfType)(callExpression.callee, 'Identifier') && callExpression.callee.name === 'token') {
+        // only two valid cases are supported
+        // one argument  => token('space.100')
+        // two arguments => token('space.100', '8px')
+        if ((callExpression.arguments.length === 1 || callExpression.arguments.length === 2) && (0, _eslintCodemodUtils.isNodeOfType)(callExpression.arguments[0], 'Literal') && (typeof callExpression.arguments[1] === 'undefined' || (0, _eslintCodemodUtils.isNodeOfType)(callExpression.arguments[1], 'Literal'))) {
+          var _callExpression$argum;
+          styleObj[prop.key.name] = {
+            tokenName: String(callExpression.arguments[0].value),
+            fallbackValue: (_callExpression$argum = callExpression.arguments[1]) !== null && _callExpression$argum !== void 0 && _callExpression$argum.value ? String(callExpression.arguments[1].value) : undefined
+          };
+          return;
+        }
+      }
     }
-    styleObj[prop.key.name] = prop.value.value;
+
+    // if we get here we have an unsupported value
+    styleObj.unsupported.push(prop.key.name);
   });
   return styleObj;
 };

package/dist/cjs/rules/use-primitives/utils/find-valid-styled-component-call.js

@@ -34,6 +34,7 @@ var findValidStyledComponentCall = exports.findValidStyledComponentCall = functi
  *
  * In the future it could be enhanced to double check `styled` and `styled2`
  * are Compiled imports but as is should work for the majority of use cases
+ * https://product-fabric.atlassian.net/browse/DSP-16058
  */
 var isStyledCallExpression = function isStyledCallExpression(call) {
   if (!(0, _eslintCodemodUtils.isNodeOfType)(call, 'CallExpression')) {

package/dist/cjs/rules/use-primitives/utils/get-variable-usage-count.js

@@ -5,6 +5,7 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.getVariableUsagesCount = void 0;
 /**
+ * TODO: Update this logic: https://product-fabric.atlassian.net/browse/DSP-16059
  * Using Regex here because otherwise we'd need to traverse the entire AST
  * We should harden this logic as we go.
  */

package/dist/cjs/rules/use-primitives/utils/is-valid-css-properties-to-transform.js

@@ -1,21 +1,33 @@
 "use strict";
 
+var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.isValidCssPropertiesToTransform = void 0;
+var _typeof2 = _interopRequireDefault(require("@babel/runtime/helpers/typeof"));
+var _objectWithoutProperties2 = _interopRequireDefault(require("@babel/runtime/helpers/objectWithoutProperties"));
 var _eslintCodemodUtils = require("eslint-codemod-utils");
 var _cssToXcss = require("../transformers/css-to-xcss");
 var _convertAstObjectExpressionToJsObject = require("./convert-ast-object-expression-to-js-object");
-var isValidCssPropertiesToTransform = exports.isValidCssPropertiesToTransform = function isValidCssPropertiesToTransform(node) {
+var _excluded = ["unsupported"];
+var isValidCssPropertiesToTransform = exports.isValidCssPropertiesToTransform = function isValidCssPropertiesToTransform(node, config) {
   var cssObjectExpression = node.arguments[0];
   // Bail on empty object calls
   if (!cssObjectExpression || !(0, _eslintCodemodUtils.isNodeOfType)(cssObjectExpression, 'ObjectExpression')) {
     return false;
   }
-  var cssObject = (0, _convertAstObjectExpressionToJsObject.convertASTObjectExpressionToJSObject)(cssObjectExpression);
-  // Bail if there are less or more than 1 styles defined
-  if (!cssObject || Object.keys(cssObject).length !== 1) {
+  var _convertASTObjectExpr = (0, _convertAstObjectExpressionToJsObject.convertASTObjectExpressionToJSObject)(cssObjectExpression),
+    unsupported = _convertASTObjectExpr.unsupported,
+    cssObject = (0, _objectWithoutProperties2.default)(_convertASTObjectExpr, _excluded);
+  // Bail if there are any unsupported styles
+  if (unsupported.length > 0 || Object.keys(cssObject).length !== 1) {
+    return false;
+  }
+  // Short-circuit when token calls are found but pattern is not enabled in config
+  if (!config.patterns.includes('css-property-with-tokens') && Object.values(cssObject).some(function (value) {
+    return (0, _typeof2.default)(value) === 'object' && value.tokenName;
+  })) {
     return false;
   }
 
@@ -25,10 +37,22 @@ var isValidCssPropertiesToTransform = exports.isValidCssPropertiesToTransform =
   // than the rule reporting on things that can't be mapped.
   var containsOnlyValidStyles = Object.keys(cssObject).every(function (styleProperty) {
     var styleValue = cssObject[styleProperty];
-    return _cssToXcss.supportedStylesMap[styleProperty] &&
-    // Is the key something we can map
-    _cssToXcss.supportedStylesMap[styleProperty][styleValue] // Is the value something we can map
-    ;
+
+    // token function call
+    if ((0, _typeof2.default)(styleValue) === 'object') {
+      // if there is no fallback value, we just map to the token name, if one is found
+      if (!styleValue.fallbackValue) {
+        return _cssToXcss.supportedStylesMap[styleProperty] && Object.values(_cssToXcss.supportedStylesMap[styleProperty]).includes(styleValue.tokenName);
+      }
+      // token with fallback
+      return _cssToXcss.supportedStylesMap[styleProperty] && _cssToXcss.supportedStylesMap[styleProperty][styleValue.fallbackValue] === styleValue.tokenName;
+    } else {
+      // direct value used
+      return _cssToXcss.supportedStylesMap[styleProperty] &&
+      // Is the key something we can map
+      _cssToXcss.supportedStylesMap[styleProperty][styleValue] // Is the value something we can map
+      ;
+    }
   });
 
   if (!containsOnlyValidStyles) {

package/dist/es2019/rules/consistent-css-prop-usage/index.js

@@ -14,9 +14,9 @@ function findSpreadProperties(node) {
   property.type === 'ExperimentalSpreadProperty');
 }
 function analyzeIdentifier(context, sourceIdentifier, configuration) {
-  var _getIdentifierInParen;
+  var _getIdentifierInParen, _getIdentifierInParen2;
   const scope = context.getScope();
-  const [identifier] = ((_getIdentifierInParen = getIdentifierInParentScope(scope, sourceIdentifier.name)) === null || _getIdentifierInParen === void 0 ? void 0 : _getIdentifierInParen.identifiers) || [];
+  const [identifier] = (_getIdentifierInParen = (_getIdentifierInParen2 = getIdentifierInParentScope(scope, sourceIdentifier.name)) === null || _getIdentifierInParen2 === void 0 ? void 0 : _getIdentifierInParen2.identifiers) !== null && _getIdentifierInParen !== void 0 ? _getIdentifierInParen : [];
   if (!identifier || !identifier.parent) {
     // Identifier isn't in the module, skip!
     return;
@@ -37,7 +37,7 @@ function analyzeIdentifier(context, sourceIdentifier, configuration) {
     });
     return;
   }
-  if (identifier.parent && identifier.parent.init && !isCssCallExpression(identifier.parent.init, configuration.cssFunctions)) {
+  if (identifier.parent && identifier.parent.init && !isCssCallExpression(identifier.parent.init, [...configuration.cssFunctions, 'cssMap'])) {
     // When variable value is not of type css({})
     context.report({
       node: identifier,
@@ -56,6 +56,13 @@ function analyzeIdentifier(context, sourceIdentifier, configuration) {
     });
   }
 }
+
+/**
+ * Handle different cases based on what's been passed in the css-related JSXAttribute
+ * @param context the context of the node
+ * @param expression the expression of the JSXAttribute value
+ * @param configuration what css-related functions to account for (eg. css, xcss, cssMap), and whether to detect bottom vs top expressions
+ */
 const traverseExpressionWithConfig = (context, expression, configuration) => {
   function traverseExpression(expression) {
     switch (expression.type) {
@@ -124,7 +131,8 @@ const rule = createLintRule({
   },
   create(context) {
     const mergedConfig = assign({}, defaultConfig, context.options[0]);
-    const callSelector = mergedConfig.cssFunctions.map(fn => `CallExpression[callee.name=${fn}]`).join(',');
+    const callSelectorFunctions = [...mergedConfig.cssFunctions, 'cssMap'];
+    const callSelector = callSelectorFunctions.map(fn => `CallExpression[callee.name=${fn}]`).join(',');
     return {
       [callSelector]: node => {
         if (node.parent.type !== 'VariableDeclarator') {

package/dist/es2019/rules/use-primitives/config/index.js

@@ -1,5 +1,5 @@
 const defaults = {
-  patterns: ['div-object-css']
+  patterns: ['compiled-css-function']
 };
 export const getConfig = overrides => {
   // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/assign

package/dist/es2019/rules/use-primitives/index.js

@@ -31,7 +31,7 @@ const rule = createLintRule({
           return;
         }
         const styledComponentVariableRef = findValidStyledComponentCall(node);
-        if (!styledComponentVariableRef || !isNodeOfType(styledComponentVariableRef.id, 'Identifier') || !isValidCssPropertiesToTransform(node)) {
+        if (!styledComponentVariableRef || !isNodeOfType(styledComponentVariableRef.id, 'Identifier') || !isValidCssPropertiesToTransform(node, config)) {
           return;
         }
         const styledComponentJsxRef = findValidJsxUsageToTransform(styledComponentVariableRef.id.name, context.getScope());
@@ -54,7 +54,9 @@ const rule = createLintRule({
       },
       // transforms <div css={...}> usages
       JSXOpeningElement(node) {
-        const config = getConfig(context.options[0]);
+        if (!config.patterns.includes('compiled-css-function')) {
+          return false;
+        }
         if (!isNodeOfType(node, 'JSXOpeningElement')) {
           return;
         }
@@ -129,9 +131,6 @@ const shouldSuggestBox = (node, context, config
   if (!cssVariableValue || !isFunctionNamed(cssVariableValue, 'css')) {
     return false;
   }
-  if (!config.patterns.includes('div-object-css')) {
-    return false;
-  }
-  return isValidCssPropertiesToTransform(cssVariableValue.node.init);
+  return isValidCssPropertiesToTransform(cssVariableValue.node.init, config);
 };
 export default rule;

package/dist/es2019/rules/use-primitives/transformers/css-to-xcss.js

@@ -31,7 +31,7 @@ export const cssToXcssTransformer = (node, context, fixer) => {
   fixer.replaceText(cssVariableValue.node.init.callee, identifier('xcss').toString()), ...styledObjectToXcssTokens(cssObjectExpression, fixer)];
 };
 
-// Update css object values to xcss values. e.g. `'8px'` -> `'space.100'`
+// Update css object values to xcss values
 // Note: `properties` in this context is a group of AST nodes that make up a key/value pair in an object.
 // e.g. `padding: '8px'`. For clarity, it's renamed to `entry` inside the `.map()`.
 export const styledObjectToXcssTokens = (styles, fixer) => {
@@ -42,16 +42,30 @@ export const styledObjectToXcssTokens = (styles, fixer) => {
     if (!isNodeOfType(entry.key, 'Identifier')) {
       return;
     }
-    if (!isNodeOfType(entry.value, 'Literal')) {
-      return;
+
+    // maps literal values like: 8px to 'space.100'
+    if (isNodeOfType(entry.value, 'Literal')) {
+      const value = entry.value.value;
+      if (typeof value !== 'string') {
+        return;
+      }
+      return fixer.replaceText(entry.value, literal(`'${supportedStylesMap[entry.key.name][value]}'`).toString());
     }
-    const value = entry.value.value;
-    if (typeof value !== 'string') {
-      return;
+    // maps token calls like: token('space.100') to 'space.100'
+    if (isNodeOfType(entry.value, 'CallExpression')) {
+      const callExpression = entry.value;
+      // skip if not a call to `token`
+      if (!isNodeOfType(callExpression.callee, 'Identifier') || callExpression.callee.name !== 'token' || !isNodeOfType(callExpression.arguments[0], 'Literal')) {
+        return;
+      }
+      // the first argument of `token` is the token name and
+      // can be given directly to `xcss` as it has been validated already.
+      return fixer.replaceText(entry.value, literal(`'${callExpression.arguments[0].value}'`).toString());
     }
-    return fixer.replaceText(entry.value, literal(`'${supportedStylesMap[entry.key.name][value]}'`).toString());
   });
 };
+
+// TODO: https://product-fabric.atlassian.net/browse/DSP-16054
 export const spaceTokenMap = {
   '0px': 'space.0',
   '2px': 'space.025',

package/dist/es2019/rules/use-primitives/utils/convert-ast-object-expression-to-js-object.js

@@ -1,26 +1,17 @@
 import { isNodeOfType } from 'eslint-codemod-utils';
-
-// FIXME: This is only loosely typed
-
+export const SPREAD_SYNTAX = Symbol('SPREAD_SYNTAX');
 /**
  * Note: Not recursive. Only handles top level key/value pairs
  */
 export const convertASTObjectExpressionToJSObject = styles => {
-  const styleObj = {};
+  const styleObj = {
+    unsupported: []
+  };
 
-  // if we see any spread props we stop and return false to indicate this is unsupported
+  // if we see any spread props we indicate that as unsupported
   if (!styles.properties.every(prop => isNodeOfType(prop, 'Property'))) {
-    return false;
+    styleObj.unsupported.push(SPREAD_SYNTAX);
   }
-
-  // TODO: We need to harden this logic.
-  // It currently generates a false positive for:
-  // styled.div({
-  //  marginTop: "0px",
-  //  marginBottom: token("space.100", "8px"),
-  // })
-  // as the value for `marginBottom` is not a string, so it is just skipped
-  // from the resulting map and this causes the rule to trigger when it shouldn't
   styles.properties.forEach(prop => {
     if (!isNodeOfType(prop, 'Property')) {
       return;
@@ -28,13 +19,34 @@ export const convertASTObjectExpressionToJSObject = styles => {
     if (!isNodeOfType(prop.key, 'Identifier')) {
       return;
     }
-    if (!isNodeOfType(prop.value, 'Literal')) {
+
+    // a literal string value, the base case
+    if (isNodeOfType(prop.value, 'Literal') && typeof prop.value.value === 'string') {
+      styleObj[prop.key.name] = prop.value.value;
       return;
     }
-    if (typeof prop.value.value !== 'string') {
-      return;
+
+    // try to handle a direct call to `token`
+    if (isNodeOfType(prop.value, 'CallExpression')) {
+      const callExpression = prop.value;
+      // strictly handle calls to `token`
+      if (isNodeOfType(callExpression.callee, 'Identifier') && callExpression.callee.name === 'token') {
+        // only two valid cases are supported
+        // one argument  => token('space.100')
+        // two arguments => token('space.100', '8px')
+        if ((callExpression.arguments.length === 1 || callExpression.arguments.length === 2) && isNodeOfType(callExpression.arguments[0], 'Literal') && (typeof callExpression.arguments[1] === 'undefined' || isNodeOfType(callExpression.arguments[1], 'Literal'))) {
+          var _callExpression$argum;
+          styleObj[prop.key.name] = {
+            tokenName: String(callExpression.arguments[0].value),
+            fallbackValue: (_callExpression$argum = callExpression.arguments[1]) !== null && _callExpression$argum !== void 0 && _callExpression$argum.value ? String(callExpression.arguments[1].value) : undefined
+          };
+          return;
+        }
+      }
     }
-    styleObj[prop.key.name] = prop.value.value;
+
+    // if we get here we have an unsupported value
+    styleObj.unsupported.push(prop.key.name);
   });
   return styleObj;
 };

package/dist/es2019/rules/use-primitives/utils/find-valid-styled-component-call.js

@@ -29,6 +29,7 @@ export const findValidStyledComponentCall = node => {
  *
  * In the future it could be enhanced to double check `styled` and `styled2`
  * are Compiled imports but as is should work for the majority of use cases
+ * https://product-fabric.atlassian.net/browse/DSP-16058
  */
 const isStyledCallExpression = call => {
   if (!isNodeOfType(call, 'CallExpression')) {

package/dist/es2019/rules/use-primitives/utils/get-variable-usage-count.js

@@ -1,4 +1,5 @@
 /**
+ * TODO: Update this logic: https://product-fabric.atlassian.net/browse/DSP-16059
  * Using Regex here because otherwise we'd need to traverse the entire AST
  * We should harden this logic as we go.
  */

package/dist/es2019/rules/use-primitives/utils/is-valid-css-properties-to-transform.js

@@ -1,15 +1,22 @@
 import { isNodeOfType } from 'eslint-codemod-utils';
 import { supportedStylesMap } from '../transformers/css-to-xcss';
 import { convertASTObjectExpressionToJSObject } from './convert-ast-object-expression-to-js-object';
-export const isValidCssPropertiesToTransform = node => {
+export const isValidCssPropertiesToTransform = (node, config) => {
   const cssObjectExpression = node.arguments[0];
   // Bail on empty object calls
   if (!cssObjectExpression || !isNodeOfType(cssObjectExpression, 'ObjectExpression')) {
     return false;
   }
-  const cssObject = convertASTObjectExpressionToJSObject(cssObjectExpression);
-  // Bail if there are less or more than 1 styles defined
-  if (!cssObject || Object.keys(cssObject).length !== 1) {
+  const {
+    unsupported,
+    ...cssObject
+  } = convertASTObjectExpressionToJSObject(cssObjectExpression);
+  // Bail if there are any unsupported styles
+  if (unsupported.length > 0 || Object.keys(cssObject).length !== 1) {
+    return false;
+  }
+  // Short-circuit when token calls are found but pattern is not enabled in config
+  if (!config.patterns.includes('css-property-with-tokens') && Object.values(cssObject).some(value => typeof value === 'object' && value.tokenName)) {
     return false;
   }
 
@@ -19,10 +26,22 @@ export const isValidCssPropertiesToTransform = node => {
   // than the rule reporting on things that can't be mapped.
   const containsOnlyValidStyles = Object.keys(cssObject).every(styleProperty => {
     const styleValue = cssObject[styleProperty];
-    return supportedStylesMap[styleProperty] &&
-    // Is the key something we can map
-    supportedStylesMap[styleProperty][styleValue] // Is the value something we can map
-    ;
+
+    // token function call
+    if (typeof styleValue === 'object') {
+      // if there is no fallback value, we just map to the token name, if one is found
+      if (!styleValue.fallbackValue) {
+        return supportedStylesMap[styleProperty] && Object.values(supportedStylesMap[styleProperty]).includes(styleValue.tokenName);
+      }
+      // token with fallback
+      return supportedStylesMap[styleProperty] && supportedStylesMap[styleProperty][styleValue.fallbackValue] === styleValue.tokenName;
+    } else {
+      // direct value used
+      return supportedStylesMap[styleProperty] &&
+      // Is the key something we can map
+      supportedStylesMap[styleProperty][styleValue] // Is the value something we can map
+      ;
+    }
   });
 
   if (!containsOnlyValidStyles) {

package/dist/esm/rules/consistent-css-prop-usage/index.js

@@ -1,4 +1,5 @@
 import _defineProperty from "@babel/runtime/helpers/defineProperty";
+import _toConsumableArray from "@babel/runtime/helpers/toConsumableArray";
 import _slicedToArray from "@babel/runtime/helpers/slicedToArray";
 // eslint-disable-next-line import/no-extraneous-dependencies
 
@@ -18,9 +19,9 @@ function findSpreadProperties(node) {
   });
 }
 function analyzeIdentifier(context, sourceIdentifier, configuration) {
-  var _getIdentifierInParen;
+  var _getIdentifierInParen, _getIdentifierInParen2;
   var scope = context.getScope();
-  var _ref = ((_getIdentifierInParen = getIdentifierInParentScope(scope, sourceIdentifier.name)) === null || _getIdentifierInParen === void 0 ? void 0 : _getIdentifierInParen.identifiers) || [],
+  var _ref = (_getIdentifierInParen = (_getIdentifierInParen2 = getIdentifierInParentScope(scope, sourceIdentifier.name)) === null || _getIdentifierInParen2 === void 0 ? void 0 : _getIdentifierInParen2.identifiers) !== null && _getIdentifierInParen !== void 0 ? _getIdentifierInParen : [],
     _ref2 = _slicedToArray(_ref, 1),
     identifier = _ref2[0];
   if (!identifier || !identifier.parent) {
@@ -43,7 +44,7 @@ function analyzeIdentifier(context, sourceIdentifier, configuration) {
     });
     return;
   }
-  if (identifier.parent && identifier.parent.init && !isCssCallExpression(identifier.parent.init, configuration.cssFunctions)) {
+  if (identifier.parent && identifier.parent.init && !isCssCallExpression(identifier.parent.init, [].concat(_toConsumableArray(configuration.cssFunctions), ['cssMap']))) {
     // When variable value is not of type css({})
     context.report({
       node: identifier,
@@ -62,6 +63,13 @@ function analyzeIdentifier(context, sourceIdentifier, configuration) {
     });
   }
 }
+
+/**
+ * Handle different cases based on what's been passed in the css-related JSXAttribute
+ * @param context the context of the node
+ * @param expression the expression of the JSXAttribute value
+ * @param configuration what css-related functions to account for (eg. css, xcss, cssMap), and whether to detect bottom vs top expressions
+ */
 var traverseExpressionWithConfig = function traverseExpressionWithConfig(context, expression, configuration) {
   function traverseExpression(expression) {
     switch (expression.type) {
@@ -133,7 +141,8 @@ var rule = createLintRule({
   create: function create(context) {
     var _ref3;
     var mergedConfig = assign({}, defaultConfig, context.options[0]);
-    var callSelector = mergedConfig.cssFunctions.map(function (fn) {
+    var callSelectorFunctions = [].concat(_toConsumableArray(mergedConfig.cssFunctions), ['cssMap']);
+    var callSelector = callSelectorFunctions.map(function (fn) {
       return "CallExpression[callee.name=".concat(fn, "]");
     }).join(',');
     return _ref3 = {}, _defineProperty(_ref3, callSelector, function (node) {

package/dist/esm/rules/use-primitives/config/index.js

@@ -1,5 +1,5 @@
 var defaults = {
-  patterns: ['div-object-css']
+  patterns: ['compiled-css-function']
 };
 export var getConfig = function getConfig(overrides) {
   // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/assign

package/dist/esm/rules/use-primitives/index.js

@@ -31,7 +31,7 @@ var rule = createLintRule({
           return;
         }
         var styledComponentVariableRef = findValidStyledComponentCall(node);
-        if (!styledComponentVariableRef || !isNodeOfType(styledComponentVariableRef.id, 'Identifier') || !isValidCssPropertiesToTransform(node)) {
+        if (!styledComponentVariableRef || !isNodeOfType(styledComponentVariableRef.id, 'Identifier') || !isValidCssPropertiesToTransform(node, config)) {
           return;
         }
         var styledComponentJsxRef = findValidJsxUsageToTransform(styledComponentVariableRef.id.name, context.getScope());
@@ -54,7 +54,9 @@ var rule = createLintRule({
       },
       // transforms <div css={...}> usages
       JSXOpeningElement: function JSXOpeningElement(node) {
-        var config = getConfig(context.options[0]);
+        if (!config.patterns.includes('compiled-css-function')) {
+          return false;
+        }
         if (!isNodeOfType(node, 'JSXOpeningElement')) {
           return;
         }
@@ -129,9 +131,6 @@ var shouldSuggestBox = function shouldSuggestBox(node, context, config
   if (!cssVariableValue || !isFunctionNamed(cssVariableValue, 'css')) {
     return false;
   }
-  if (!config.patterns.includes('div-object-css')) {
-    return false;
-  }
-  return isValidCssPropertiesToTransform(cssVariableValue.node.init);
+  return isValidCssPropertiesToTransform(cssVariableValue.node.init, config);
 };
 export default rule;

package/dist/esm/rules/use-primitives/transformers/css-to-xcss.js

@@ -32,7 +32,7 @@ export var cssToXcssTransformer = function cssToXcssTransformer(node, context, f
   fixer.replaceText(cssVariableValue.node.init.callee, identifier('xcss').toString())].concat(_toConsumableArray(styledObjectToXcssTokens(cssObjectExpression, fixer)));
 };
 
-// Update css object values to xcss values. e.g. `'8px'` -> `'space.100'`
+// Update css object values to xcss values
 // Note: `properties` in this context is a group of AST nodes that make up a key/value pair in an object.
 // e.g. `padding: '8px'`. For clarity, it's renamed to `entry` inside the `.map()`.
 export var styledObjectToXcssTokens = function styledObjectToXcssTokens(styles, fixer) {
@@ -43,16 +43,30 @@ export var styledObjectToXcssTokens = function styledObjectToXcssTokens(styles,
     if (!isNodeOfType(entry.key, 'Identifier')) {
       return;
     }
-    if (!isNodeOfType(entry.value, 'Literal')) {
-      return;
+
+    // maps literal values like: 8px to 'space.100'
+    if (isNodeOfType(entry.value, 'Literal')) {
+      var value = entry.value.value;
+      if (typeof value !== 'string') {
+        return;
+      }
+      return fixer.replaceText(entry.value, literal("'".concat(supportedStylesMap[entry.key.name][value], "'")).toString());
     }
-    var value = entry.value.value;
-    if (typeof value !== 'string') {
-      return;
+    // maps token calls like: token('space.100') to 'space.100'
+    if (isNodeOfType(entry.value, 'CallExpression')) {
+      var callExpression = entry.value;
+      // skip if not a call to `token`
+      if (!isNodeOfType(callExpression.callee, 'Identifier') || callExpression.callee.name !== 'token' || !isNodeOfType(callExpression.arguments[0], 'Literal')) {
+        return;
+      }
+      // the first argument of `token` is the token name and
+      // can be given directly to `xcss` as it has been validated already.
+      return fixer.replaceText(entry.value, literal("'".concat(callExpression.arguments[0].value, "'")).toString());
     }
-    return fixer.replaceText(entry.value, literal("'".concat(supportedStylesMap[entry.key.name][value], "'")).toString());
   });
 };
+
+// TODO: https://product-fabric.atlassian.net/browse/DSP-16054
 export var spaceTokenMap = {
   '0px': 'space.0',
   '2px': 'space.025',

package/dist/esm/rules/use-primitives/utils/convert-ast-object-expression-to-js-object.js

@@ -1,28 +1,19 @@
 import { isNodeOfType } from 'eslint-codemod-utils';
-
-// FIXME: This is only loosely typed
-
+export var SPREAD_SYNTAX = Symbol('SPREAD_SYNTAX');
 /**
  * Note: Not recursive. Only handles top level key/value pairs
  */
 export var convertASTObjectExpressionToJSObject = function convertASTObjectExpressionToJSObject(styles) {
-  var styleObj = {};
+  var styleObj = {
+    unsupported: []
+  };
 
-  // if we see any spread props we stop and return false to indicate this is unsupported
+  // if we see any spread props we indicate that as unsupported
   if (!styles.properties.every(function (prop) {
     return isNodeOfType(prop, 'Property');
   })) {
-    return false;
+    styleObj.unsupported.push(SPREAD_SYNTAX);
   }
-
-  // TODO: We need to harden this logic.
-  // It currently generates a false positive for:
-  // styled.div({
-  //  marginTop: "0px",
-  //  marginBottom: token("space.100", "8px"),
-  // })
-  // as the value for `marginBottom` is not a string, so it is just skipped
-  // from the resulting map and this causes the rule to trigger when it shouldn't
   styles.properties.forEach(function (prop) {
     if (!isNodeOfType(prop, 'Property')) {
       return;
@@ -30,13 +21,34 @@ export var convertASTObjectExpressionToJSObject = function convertASTObjectExpre
     if (!isNodeOfType(prop.key, 'Identifier')) {
       return;
     }
-    if (!isNodeOfType(prop.value, 'Literal')) {
+
+    // a literal string value, the base case
+    if (isNodeOfType(prop.value, 'Literal') && typeof prop.value.value === 'string') {
+      styleObj[prop.key.name] = prop.value.value;
       return;
     }
-    if (typeof prop.value.value !== 'string') {
-      return;
+
+    // try to handle a direct call to `token`
+    if (isNodeOfType(prop.value, 'CallExpression')) {
+      var callExpression = prop.value;
+      // strictly handle calls to `token`
+      if (isNodeOfType(callExpression.callee, 'Identifier') && callExpression.callee.name === 'token') {
+        // only two valid cases are supported
+        // one argument  => token('space.100')
+        // two arguments => token('space.100', '8px')
+        if ((callExpression.arguments.length === 1 || callExpression.arguments.length === 2) && isNodeOfType(callExpression.arguments[0], 'Literal') && (typeof callExpression.arguments[1] === 'undefined' || isNodeOfType(callExpression.arguments[1], 'Literal'))) {
+          var _callExpression$argum;
+          styleObj[prop.key.name] = {
+            tokenName: String(callExpression.arguments[0].value),
+            fallbackValue: (_callExpression$argum = callExpression.arguments[1]) !== null && _callExpression$argum !== void 0 && _callExpression$argum.value ? String(callExpression.arguments[1].value) : undefined
+          };
+          return;
+        }
+      }
     }
-    styleObj[prop.key.name] = prop.value.value;
+
+    // if we get here we have an unsupported value
+    styleObj.unsupported.push(prop.key.name);
   });
   return styleObj;
 };

package/dist/esm/rules/use-primitives/utils/find-valid-styled-component-call.js

@@ -29,6 +29,7 @@ export var findValidStyledComponentCall = function findValidStyledComponentCall(
  *
  * In the future it could be enhanced to double check `styled` and `styled2`
  * are Compiled imports but as is should work for the majority of use cases
+ * https://product-fabric.atlassian.net/browse/DSP-16058
  */
 var isStyledCallExpression = function isStyledCallExpression(call) {
   if (!isNodeOfType(call, 'CallExpression')) {

package/dist/esm/rules/use-primitives/utils/get-variable-usage-count.js

@@ -1,4 +1,5 @@
 /**
+ * TODO: Update this logic: https://product-fabric.atlassian.net/browse/DSP-16059
  * Using Regex here because otherwise we'd need to traverse the entire AST
  * We should harden this logic as we go.
  */

package/dist/esm/rules/use-primitives/utils/is-valid-css-properties-to-transform.js

@@ -1,15 +1,26 @@
+import _typeof from "@babel/runtime/helpers/typeof";
+import _objectWithoutProperties from "@babel/runtime/helpers/objectWithoutProperties";
+var _excluded = ["unsupported"];
 import { isNodeOfType } from 'eslint-codemod-utils';
 import { supportedStylesMap } from '../transformers/css-to-xcss';
 import { convertASTObjectExpressionToJSObject } from './convert-ast-object-expression-to-js-object';
-export var isValidCssPropertiesToTransform = function isValidCssPropertiesToTransform(node) {
+export var isValidCssPropertiesToTransform = function isValidCssPropertiesToTransform(node, config) {
   var cssObjectExpression = node.arguments[0];
   // Bail on empty object calls
   if (!cssObjectExpression || !isNodeOfType(cssObjectExpression, 'ObjectExpression')) {
     return false;
   }
-  var cssObject = convertASTObjectExpressionToJSObject(cssObjectExpression);
-  // Bail if there are less or more than 1 styles defined
-  if (!cssObject || Object.keys(cssObject).length !== 1) {
+  var _convertASTObjectExpr = convertASTObjectExpressionToJSObject(cssObjectExpression),
+    unsupported = _convertASTObjectExpr.unsupported,
+    cssObject = _objectWithoutProperties(_convertASTObjectExpr, _excluded);
+  // Bail if there are any unsupported styles
+  if (unsupported.length > 0 || Object.keys(cssObject).length !== 1) {
+    return false;
+  }
+  // Short-circuit when token calls are found but pattern is not enabled in config
+  if (!config.patterns.includes('css-property-with-tokens') && Object.values(cssObject).some(function (value) {
+    return _typeof(value) === 'object' && value.tokenName;
+  })) {
     return false;
   }
 
@@ -19,10 +30,22 @@ export var isValidCssPropertiesToTransform = function isValidCssPropertiesToTran
   // than the rule reporting on things that can't be mapped.
   var containsOnlyValidStyles = Object.keys(cssObject).every(function (styleProperty) {
     var styleValue = cssObject[styleProperty];
-    return supportedStylesMap[styleProperty] &&
-    // Is the key something we can map
-    supportedStylesMap[styleProperty][styleValue] // Is the value something we can map
-    ;
+
+    // token function call
+    if (_typeof(styleValue) === 'object') {
+      // if there is no fallback value, we just map to the token name, if one is found
+      if (!styleValue.fallbackValue) {
+        return supportedStylesMap[styleProperty] && Object.values(supportedStylesMap[styleProperty]).includes(styleValue.tokenName);
+      }
+      // token with fallback
+      return supportedStylesMap[styleProperty] && supportedStylesMap[styleProperty][styleValue.fallbackValue] === styleValue.tokenName;
+    } else {
+      // direct value used
+      return supportedStylesMap[styleProperty] &&
+      // Is the key something we can map
+      supportedStylesMap[styleProperty][styleValue] // Is the value something we can map
+      ;
+    }
   });
 
   if (!containsOnlyValidStyles) {

package/dist/types/rules/use-primitives/config/index.d.ts

@@ -1,4 +1,6 @@
+type Pattern = 'compiled-css-function' | 'compiled-styled-object' | 'css-template-literal' | 'css-property-with-tokens' | 'css-property-multiple-values';
 export interface RuleConfig {
-    patterns: ('div-object-css' | 'compiled-styled-object' | 'compiled-template-literal' | 'div-object-css-with-tokens' | 'div-object-css-with-multiple-vals')[];
+    patterns: Pattern[];
 }
 export declare const getConfig: (overrides: Partial<RuleConfig>) => Required<RuleConfig>;
+export {};

package/dist/types/rules/use-primitives/utils/convert-ast-object-expression-to-js-object.d.ts

@@ -1,9 +1,18 @@
+import type { CSSProperties } from 'react';
 import type { Rule } from 'eslint';
 import { ObjectExpression } from 'eslint-codemod-utils';
+export declare const SPREAD_SYNTAX: unique symbol;
+type Token = {
+    tokenName: string;
+    fallbackValue: string | undefined;
+};
 export type CSSPropStyleObject = {
-    [key: string]: string | number;
+    [key in keyof CSSProperties]: string | number | Token;
+} & {
+    unsupported: (keyof CSSProperties | typeof SPREAD_SYNTAX)[];
 };
 /**
  * Note: Not recursive. Only handles top level key/value pairs
  */
-export declare const convertASTObjectExpressionToJSObject: (styles: ObjectExpression & Partial<Rule.NodeParentExtension>) => CSSPropStyleObject | false;
+export declare const convertASTObjectExpressionToJSObject: (styles: ObjectExpression & Partial<Rule.NodeParentExtension>) => CSSPropStyleObject;
+export {};

package/dist/types/rules/use-primitives/utils/get-variable-usage-count.d.ts

@@ -1,5 +1,6 @@
 import type { Rule } from 'eslint';
 /**
+ * TODO: Update this logic: https://product-fabric.atlassian.net/browse/DSP-16059
  * Using Regex here because otherwise we'd need to traverse the entire AST
  * We should harden this logic as we go.
  */

package/dist/types/rules/use-primitives/utils/is-valid-css-properties-to-transform.d.ts

@@ -1,3 +1,4 @@
 import type { Rule } from 'eslint';
 import { SimpleCallExpression } from 'eslint-codemod-utils';
-export declare const isValidCssPropertiesToTransform: (node: SimpleCallExpression & Rule.NodeParentExtension) => boolean;
+import { RuleConfig } from '../config';
+export declare const isValidCssPropertiesToTransform: (node: SimpleCallExpression & Rule.NodeParentExtension, config: RuleConfig) => boolean;

package/dist/types-ts4.5/rules/use-primitives/config/index.d.ts

@@ -1,4 +1,6 @@
+type Pattern = 'compiled-css-function' | 'compiled-styled-object' | 'css-template-literal' | 'css-property-with-tokens' | 'css-property-multiple-values';
 export interface RuleConfig {
-    patterns: ('div-object-css' | 'compiled-styled-object' | 'compiled-template-literal' | 'div-object-css-with-tokens' | 'div-object-css-with-multiple-vals')[];
+    patterns: Pattern[];
 }
 export declare const getConfig: (overrides: Partial<RuleConfig>) => Required<RuleConfig>;
+export {};

package/dist/types-ts4.5/rules/use-primitives/utils/convert-ast-object-expression-to-js-object.d.ts

@@ -1,9 +1,18 @@
+import type { CSSProperties } from 'react';
 import type { Rule } from 'eslint';
 import { ObjectExpression } from 'eslint-codemod-utils';
+export declare const SPREAD_SYNTAX: unique symbol;
+type Token = {
+    tokenName: string;
+    fallbackValue: string | undefined;
+};
 export type CSSPropStyleObject = {
-    [key: string]: string | number;
+    [key in keyof CSSProperties]: string | number | Token;
+} & {
+    unsupported: (keyof CSSProperties | typeof SPREAD_SYNTAX)[];
 };
 /**
  * Note: Not recursive. Only handles top level key/value pairs
  */
-export declare const convertASTObjectExpressionToJSObject: (styles: ObjectExpression & Partial<Rule.NodeParentExtension>) => CSSPropStyleObject | false;
+export declare const convertASTObjectExpressionToJSObject: (styles: ObjectExpression & Partial<Rule.NodeParentExtension>) => CSSPropStyleObject;
+export {};

package/dist/types-ts4.5/rules/use-primitives/utils/get-variable-usage-count.d.ts

@@ -1,5 +1,6 @@
 import type { Rule } from 'eslint';
 /**
+ * TODO: Update this logic: https://product-fabric.atlassian.net/browse/DSP-16059
  * Using Regex here because otherwise we'd need to traverse the entire AST
  * We should harden this logic as we go.
  */

package/dist/types-ts4.5/rules/use-primitives/utils/is-valid-css-properties-to-transform.d.ts

@@ -1,3 +1,4 @@
 import type { Rule } from 'eslint';
 import { SimpleCallExpression } from 'eslint-codemod-utils';
-export declare const isValidCssPropertiesToTransform: (node: SimpleCallExpression & Rule.NodeParentExtension) => boolean;
+import { RuleConfig } from '../config';
+export declare const isValidCssPropertiesToTransform: (node: SimpleCallExpression & Rule.NodeParentExtension, config: RuleConfig) => boolean;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@atlaskit/eslint-plugin-design-system",
   "description": "The essential plugin for use with the Atlassian Design System.",
-  "version": "8.19.1",
+  "version": "8.20.0",
   "author": "Atlassian Pty Ltd",
   "publishConfig": {
     "registry": "https://registry.npmjs.org/"