@atlaskit/eslint-plugin-platform 2.2.0 → 2.3.0

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # @atlaskit/eslint-plugin-platform
 
+## 2.3.0
+
+### Minor Changes
+
+- [#108485](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/108485)
+  [`0c642edaebbbb`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/0c642edaebbbb) -
+  AFB-822 Modifying expand-spacing-shorthand ESLint rule to not throw error for non-token
+  expressions in template strings
+
+## 2.2.1
+
+### Patch Changes
+
+- [#103444](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/103444)
+  [`5d5006fe81146`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/5d5006fe81146) -
+  AFB-822 Refactor ESLint rule and added an extra case for when ESLint rule should throw error but
+  not provide a fix
+
 ## 2.2.0
 
 ### Minor Changes

package/dist/cjs/index.js

@@ -74,7 +74,7 @@ var commonConfig = {
   // Compiled: rules that are not included via `@compiled/recommended
   '@atlaskit/platform/expand-border-shorthand': 'error',
   '@atlaskit/platform/expand-background-shorthand': 'error',
-  '@atlaskit/platform/expand-spacing-shorthand': 'off',
+  '@atlaskit/platform/expand-spacing-shorthand': 'error',
   '@compiled/jsx-pragma': ['error', {
     importSources: ['@atlaskit/css'],
     onlyRunIfImportingCompiled: true,

package/dist/cjs/rules/compiled/expand-spacing-shorthand/index.js

@@ -46,31 +46,75 @@ var parseTemplateLiteral = function parseTemplateLiteral(templateLiteral, contex
   }
   return propertyValues;
 };
-var checkValidPropertyValues = function checkValidPropertyValues(propertyValues) {
+
+/**
+ * Checks if the parsed property values are valid; values that we shouldn't handle with the fixer:
+ * 1. At least one expression in TemplateLiteral is not a token expression
+ * 2. Do not contain a token
+ * 3. Have length that are not in the range [1, 4] for different spacing directions
+ * 		- No more than 4 to exclude additional values such as `!important`
+ * 4. Includes `calc(...)`
+ * Then, the rule will return with no error
+ * @param propertyValues property values parsed as list of strings
+ * @returns boolean
+ */
+var isPropertyValueExempted = function isPropertyValueExempted(templateLiteral, propertyValues) {
+  var expressions = templateLiteral.expressions;
+  if (!expressions.every(function (expr) {
+    return expr.type === 'CallExpression' && isTokenCallExpression(expr);
+  })) {
+    return true;
+  }
   if (!propertyValues.some(function (str) {
     return str.includes('token(');
   })) {
-    return false;
+    return true;
   }
   if (propertyValues.length < 1 || propertyValues.length > 4) {
-    return false;
+    return true;
   }
   if (propertyValues.some(function (str) {
     return str.includes('calc(');
   })) {
-    return false;
+    return true;
   }
-  return true;
+  return false;
 };
 
-// Check that all expressions in TemplateLiteral are token expressions
-// If true: create an autofix
-// If false: report violation without autofix
-var hasOnlyTokens = function hasOnlyTokens(templateLiteral) {
-  var expressions = templateLiteral.expressions;
-  return expressions.every(function (expr) {
-    return expr.type === 'CallExpression' && isTokenCallExpression(expr);
-  });
+/**
+ *  Checks if the parsed property values are invalid (i.e. rule violation thrown) and autofix required. Cases are when
+ *  property values must have a format which includes -> e.g. 2, '2(rem|em|px)', auto, initial, inherit, token(...)
+ *  The rule will return with error and provide a fix
+ * @param templateLiteral TemplateLiteral AST Node
+ * @param propertyValues property values parsed as list of strings
+ * @returns boolean
+ */
+var isPropertyValuesInvalidFix = function isPropertyValuesInvalidFix(propertyValues) {
+  var _iterator = _createForOfIteratorHelper(propertyValues),
+    _step;
+  try {
+    for (_iterator.s(); !(_step = _iterator.n()).done;) {
+      var propValue = _step.value;
+      if (propValue === '0') {
+        continue;
+      }
+      if (['auto', 'initial', 'inherit'].includes(propValue.slice(1, -1))) {
+        continue;
+      }
+      if (/^token\(.*\)$/.test(propValue)) {
+        continue;
+      }
+      if (/^['"]\d+(\.\d+)?((rem)|(em)|(px))['"]$/.test(propValue)) {
+        continue;
+      }
+      return false;
+    }
+  } catch (err) {
+    _iterator.e(err);
+  } finally {
+    _iterator.f();
+  }
+  return true;
 };
 
 // To fix spacing shorthands, given a list of spacing property values, expands the spacing property and adds autofix fixes
@@ -92,11 +136,11 @@ var expandSpacingProperties = function expandSpacingProperties(_ref) {
   var fixes = [];
   var parentNode = node.parent;
   if (parentNode && parentNode.type === 'ObjectExpression') {
-    var _iterator = _createForOfIteratorHelper(parentNode.properties),
-      _step;
+    var _iterator2 = _createForOfIteratorHelper(parentNode.properties),
+      _step2;
     try {
-      for (_iterator.s(); !(_step = _iterator.n()).done;) {
-        var prop = _step.value;
+      for (_iterator2.s(); !(_step2 = _iterator2.n()).done;) {
+        var prop = _step2.value;
         if (prop.type !== 'Property') {
           continue;
         }
@@ -121,9 +165,9 @@ var expandSpacingProperties = function expandSpacingProperties(_ref) {
         }
       }
     } catch (err) {
-      _iterator.e(err);
+      _iterator2.e(err);
     } finally {
-      _iterator.f();
+      _iterator2.f();
     }
   }
   fixes.push(fixer.insertTextAfter(node, "".concat(propertyShorthand, "Top: ").concat(spacing[0], ",\n")));
@@ -138,8 +182,30 @@ var executeExpandSpacingRule = function executeExpandSpacingRule(context, node,
     return;
   }
   if (node.value.type === 'TemplateLiteral') {
-    // Value of spacing property is a TemplateLiteral type that contains a token, e.g. padding: `0 token('a')`
-    if (!hasOnlyTokens(node.value)) {
+    var propertyValues = parseTemplateLiteral(node.value, context);
+    if (isPropertyValueExempted(node.value, propertyValues)) {
+      // Valid, so no error should be thrown
+      return;
+    }
+    if (isPropertyValuesInvalidFix(propertyValues)) {
+      // Invalid, so error should be thrown and fix provided
+      context.report({
+        node: node,
+        messageId: 'expandSpacingShorthand',
+        data: {
+          property: propertyShorthand
+        },
+        fix: function fix(fixer) {
+          return expandSpacingProperties({
+            context: context,
+            node: node,
+            propertyValues: propertyValues,
+            fixer: fixer,
+            propertyShorthand: propertyShorthand
+          });
+        }
+      });
+    } else {
       context.report({
         node: node,
         messageId: 'expandSpacingShorthand',
@@ -149,26 +215,6 @@ var executeExpandSpacingRule = function executeExpandSpacingRule(context, node,
       });
       return;
     }
-    var propertyValues = parseTemplateLiteral(node.value, context);
-    if (!checkValidPropertyValues(propertyValues)) {
-      return;
-    }
-    context.report({
-      node: node,
-      messageId: 'expandSpacingShorthand',
-      data: {
-        property: propertyShorthand
-      },
-      fix: function fix(fixer) {
-        return expandSpacingProperties({
-          context: context,
-          node: node,
-          propertyValues: propertyValues,
-          fixer: fixer,
-          propertyShorthand: propertyShorthand
-        });
-      }
-    });
   } else if (node.value.type === 'CallExpression' && isTokenCallExpression(node.value)) {
     // Value of spacing property is a token CallExpression type, e.g. margin: token('space.100', '8px')
     var _propertyValues2 = [(0, _contextCompat.getSourceCode)(context).getText(node.value)];

package/dist/es2019/index.js

@@ -65,7 +65,7 @@ const commonConfig = {
   // Compiled: rules that are not included via `@compiled/recommended
   '@atlaskit/platform/expand-border-shorthand': 'error',
   '@atlaskit/platform/expand-background-shorthand': 'error',
-  '@atlaskit/platform/expand-spacing-shorthand': 'off',
+  '@atlaskit/platform/expand-spacing-shorthand': 'error',
   '@compiled/jsx-pragma': ['error', {
     importSources: ['@atlaskit/css'],
     onlyRunIfImportingCompiled: true,

package/dist/es2019/rules/compiled/expand-spacing-shorthand/index.js

@@ -35,25 +35,60 @@ const parseTemplateLiteral = (templateLiteral, context) => {
   }
   return propertyValues;
 };
-const checkValidPropertyValues = propertyValues => {
+
+/**
+ * Checks if the parsed property values are valid; values that we shouldn't handle with the fixer:
+ * 1. At least one expression in TemplateLiteral is not a token expression
+ * 2. Do not contain a token
+ * 3. Have length that are not in the range [1, 4] for different spacing directions
+ * 		- No more than 4 to exclude additional values such as `!important`
+ * 4. Includes `calc(...)`
+ * Then, the rule will return with no error
+ * @param propertyValues property values parsed as list of strings
+ * @returns boolean
+ */
+const isPropertyValueExempted = (templateLiteral, propertyValues) => {
+  const expressions = templateLiteral.expressions;
+  if (!expressions.every(expr => expr.type === 'CallExpression' && isTokenCallExpression(expr))) {
+    return true;
+  }
   if (!propertyValues.some(str => str.includes('token('))) {
-    return false;
+    return true;
   }
   if (propertyValues.length < 1 || propertyValues.length > 4) {
-    return false;
+    return true;
   }
   if (propertyValues.some(str => str.includes('calc('))) {
-    return false;
+    return true;
   }
-  return true;
+  return false;
 };
 
-// Check that all expressions in TemplateLiteral are token expressions
-// If true: create an autofix
-// If false: report violation without autofix
-const hasOnlyTokens = templateLiteral => {
-  const expressions = templateLiteral.expressions;
-  return expressions.every(expr => expr.type === 'CallExpression' && isTokenCallExpression(expr));
+/**
+ *  Checks if the parsed property values are invalid (i.e. rule violation thrown) and autofix required. Cases are when
+ *  property values must have a format which includes -> e.g. 2, '2(rem|em|px)', auto, initial, inherit, token(...)
+ *  The rule will return with error and provide a fix
+ * @param templateLiteral TemplateLiteral AST Node
+ * @param propertyValues property values parsed as list of strings
+ * @returns boolean
+ */
+const isPropertyValuesInvalidFix = propertyValues => {
+  for (const propValue of propertyValues) {
+    if (propValue === '0') {
+      continue;
+    }
+    if (['auto', 'initial', 'inherit'].includes(propValue.slice(1, -1))) {
+      continue;
+    }
+    if (/^token\(.*\)$/.test(propValue)) {
+      continue;
+    }
+    if (/^['"]\d+(\.\d+)?((rem)|(em)|(px))['"]$/.test(propValue)) {
+      continue;
+    }
+    return false;
+  }
+  return true;
 };
 
 // To fix spacing shorthands, given a list of spacing property values, expands the spacing property and adds autofix fixes
@@ -104,8 +139,30 @@ const executeExpandSpacingRule = (context, node, propertyShorthand) => {
     return;
   }
   if (node.value.type === 'TemplateLiteral') {
-    // Value of spacing property is a TemplateLiteral type that contains a token, e.g. padding: `0 token('a')`
-    if (!hasOnlyTokens(node.value)) {
+    const propertyValues = parseTemplateLiteral(node.value, context);
+    if (isPropertyValueExempted(node.value, propertyValues)) {
+      // Valid, so no error should be thrown
+      return;
+    }
+    if (isPropertyValuesInvalidFix(propertyValues)) {
+      // Invalid, so error should be thrown and fix provided
+      context.report({
+        node,
+        messageId: 'expandSpacingShorthand',
+        data: {
+          property: propertyShorthand
+        },
+        fix(fixer) {
+          return expandSpacingProperties({
+            context,
+            node,
+            propertyValues,
+            fixer,
+            propertyShorthand
+          });
+        }
+      });
+    } else {
       context.report({
         node,
         messageId: 'expandSpacingShorthand',
@@ -115,26 +172,6 @@ const executeExpandSpacingRule = (context, node, propertyShorthand) => {
       });
       return;
     }
-    const propertyValues = parseTemplateLiteral(node.value, context);
-    if (!checkValidPropertyValues(propertyValues)) {
-      return;
-    }
-    context.report({
-      node,
-      messageId: 'expandSpacingShorthand',
-      data: {
-        property: propertyShorthand
-      },
-      fix(fixer) {
-        return expandSpacingProperties({
-          context,
-          node,
-          propertyValues,
-          fixer,
-          propertyShorthand
-        });
-      }
-    });
   } else if (node.value.type === 'CallExpression' && isTokenCallExpression(node.value)) {
     // Value of spacing property is a token CallExpression type, e.g. margin: token('space.100', '8px')
     const propertyValues = [getSourceCode(context).getText(node.value)];

package/dist/esm/index.js

@@ -68,7 +68,7 @@ var commonConfig = {
   // Compiled: rules that are not included via `@compiled/recommended
   '@atlaskit/platform/expand-border-shorthand': 'error',
   '@atlaskit/platform/expand-background-shorthand': 'error',
-  '@atlaskit/platform/expand-spacing-shorthand': 'off',
+  '@atlaskit/platform/expand-spacing-shorthand': 'error',
   '@compiled/jsx-pragma': ['error', {
     importSources: ['@atlaskit/css'],
     onlyRunIfImportingCompiled: true,

package/dist/esm/rules/compiled/expand-spacing-shorthand/index.js

@@ -39,31 +39,75 @@ var parseTemplateLiteral = function parseTemplateLiteral(templateLiteral, contex
   }
   return propertyValues;
 };
-var checkValidPropertyValues = function checkValidPropertyValues(propertyValues) {
+
+/**
+ * Checks if the parsed property values are valid; values that we shouldn't handle with the fixer:
+ * 1. At least one expression in TemplateLiteral is not a token expression
+ * 2. Do not contain a token
+ * 3. Have length that are not in the range [1, 4] for different spacing directions
+ * 		- No more than 4 to exclude additional values such as `!important`
+ * 4. Includes `calc(...)`
+ * Then, the rule will return with no error
+ * @param propertyValues property values parsed as list of strings
+ * @returns boolean
+ */
+var isPropertyValueExempted = function isPropertyValueExempted(templateLiteral, propertyValues) {
+  var expressions = templateLiteral.expressions;
+  if (!expressions.every(function (expr) {
+    return expr.type === 'CallExpression' && isTokenCallExpression(expr);
+  })) {
+    return true;
+  }
   if (!propertyValues.some(function (str) {
     return str.includes('token(');
   })) {
-    return false;
+    return true;
   }
   if (propertyValues.length < 1 || propertyValues.length > 4) {
-    return false;
+    return true;
   }
   if (propertyValues.some(function (str) {
     return str.includes('calc(');
   })) {
-    return false;
+    return true;
   }
-  return true;
+  return false;
 };
 
-// Check that all expressions in TemplateLiteral are token expressions
-// If true: create an autofix
-// If false: report violation without autofix
-var hasOnlyTokens = function hasOnlyTokens(templateLiteral) {
-  var expressions = templateLiteral.expressions;
-  return expressions.every(function (expr) {
-    return expr.type === 'CallExpression' && isTokenCallExpression(expr);
-  });
+/**
+ *  Checks if the parsed property values are invalid (i.e. rule violation thrown) and autofix required. Cases are when
+ *  property values must have a format which includes -> e.g. 2, '2(rem|em|px)', auto, initial, inherit, token(...)
+ *  The rule will return with error and provide a fix
+ * @param templateLiteral TemplateLiteral AST Node
+ * @param propertyValues property values parsed as list of strings
+ * @returns boolean
+ */
+var isPropertyValuesInvalidFix = function isPropertyValuesInvalidFix(propertyValues) {
+  var _iterator = _createForOfIteratorHelper(propertyValues),
+    _step;
+  try {
+    for (_iterator.s(); !(_step = _iterator.n()).done;) {
+      var propValue = _step.value;
+      if (propValue === '0') {
+        continue;
+      }
+      if (['auto', 'initial', 'inherit'].includes(propValue.slice(1, -1))) {
+        continue;
+      }
+      if (/^token\(.*\)$/.test(propValue)) {
+        continue;
+      }
+      if (/^['"]\d+(\.\d+)?((rem)|(em)|(px))['"]$/.test(propValue)) {
+        continue;
+      }
+      return false;
+    }
+  } catch (err) {
+    _iterator.e(err);
+  } finally {
+    _iterator.f();
+  }
+  return true;
 };
 
 // To fix spacing shorthands, given a list of spacing property values, expands the spacing property and adds autofix fixes
@@ -85,11 +129,11 @@ var expandSpacingProperties = function expandSpacingProperties(_ref) {
   var fixes = [];
   var parentNode = node.parent;
   if (parentNode && parentNode.type === 'ObjectExpression') {
-    var _iterator = _createForOfIteratorHelper(parentNode.properties),
-      _step;
+    var _iterator2 = _createForOfIteratorHelper(parentNode.properties),
+      _step2;
     try {
-      for (_iterator.s(); !(_step = _iterator.n()).done;) {
-        var prop = _step.value;
+      for (_iterator2.s(); !(_step2 = _iterator2.n()).done;) {
+        var prop = _step2.value;
         if (prop.type !== 'Property') {
           continue;
         }
@@ -114,9 +158,9 @@ var expandSpacingProperties = function expandSpacingProperties(_ref) {
         }
       }
     } catch (err) {
-      _iterator.e(err);
+      _iterator2.e(err);
     } finally {
-      _iterator.f();
+      _iterator2.f();
     }
   }
   fixes.push(fixer.insertTextAfter(node, "".concat(propertyShorthand, "Top: ").concat(spacing[0], ",\n")));
@@ -131,8 +175,30 @@ var executeExpandSpacingRule = function executeExpandSpacingRule(context, node,
     return;
   }
   if (node.value.type === 'TemplateLiteral') {
-    // Value of spacing property is a TemplateLiteral type that contains a token, e.g. padding: `0 token('a')`
-    if (!hasOnlyTokens(node.value)) {
+    var propertyValues = parseTemplateLiteral(node.value, context);
+    if (isPropertyValueExempted(node.value, propertyValues)) {
+      // Valid, so no error should be thrown
+      return;
+    }
+    if (isPropertyValuesInvalidFix(propertyValues)) {
+      // Invalid, so error should be thrown and fix provided
+      context.report({
+        node: node,
+        messageId: 'expandSpacingShorthand',
+        data: {
+          property: propertyShorthand
+        },
+        fix: function fix(fixer) {
+          return expandSpacingProperties({
+            context: context,
+            node: node,
+            propertyValues: propertyValues,
+            fixer: fixer,
+            propertyShorthand: propertyShorthand
+          });
+        }
+      });
+    } else {
       context.report({
         node: node,
         messageId: 'expandSpacingShorthand',
@@ -142,26 +208,6 @@ var executeExpandSpacingRule = function executeExpandSpacingRule(context, node,
       });
       return;
     }
-    var propertyValues = parseTemplateLiteral(node.value, context);
-    if (!checkValidPropertyValues(propertyValues)) {
-      return;
-    }
-    context.report({
-      node: node,
-      messageId: 'expandSpacingShorthand',
-      data: {
-        property: propertyShorthand
-      },
-      fix: function fix(fixer) {
-        return expandSpacingProperties({
-          context: context,
-          node: node,
-          propertyValues: propertyValues,
-          fixer: fixer,
-          propertyShorthand: propertyShorthand
-        });
-      }
-    });
   } else if (node.value.type === 'CallExpression' && isTokenCallExpression(node.value)) {
     // Value of spacing property is a token CallExpression type, e.g. margin: token('space.100', '8px')
     var _propertyValues2 = [getSourceCode(context).getText(node.value)];

package/dist/types/index.d.ts

@@ -82,7 +82,7 @@ declare const plugin: {
                 '@atlaskit/platform/no-module-level-eval-nav4': "error";
                 '@atlaskit/platform/expand-border-shorthand': "error";
                 '@atlaskit/platform/expand-background-shorthand': "error";
-                '@atlaskit/platform/expand-spacing-shorthand': "off";
+                '@atlaskit/platform/expand-spacing-shorthand': "error";
                 '@compiled/jsx-pragma': ["error", {
                     importSources: string[];
                     onlyRunIfImportingCompiled: boolean;
@@ -114,7 +114,7 @@ declare const plugin: {
                 '@atlaskit/platform/no-module-level-eval-nav4': "error";
                 '@atlaskit/platform/expand-border-shorthand': "error";
                 '@atlaskit/platform/expand-background-shorthand': "error";
-                '@atlaskit/platform/expand-spacing-shorthand': "off";
+                '@atlaskit/platform/expand-spacing-shorthand': "error";
                 '@compiled/jsx-pragma': ["error", {
                     importSources: string[];
                     onlyRunIfImportingCompiled: boolean;
@@ -133,7 +133,7 @@ declare const plugin: {
                 '@atlaskit/platform/no-module-level-eval-nav4': "error";
                 '@atlaskit/platform/expand-border-shorthand': "error";
                 '@atlaskit/platform/expand-background-shorthand': "error";
-                '@atlaskit/platform/expand-spacing-shorthand': "off";
+                '@atlaskit/platform/expand-spacing-shorthand': "error";
                 '@compiled/jsx-pragma': ["error", {
                     importSources: string[];
                     onlyRunIfImportingCompiled: boolean;
@@ -155,7 +155,7 @@ declare const plugin: {
                 '@atlaskit/platform/no-module-level-eval-nav4': "error";
                 '@atlaskit/platform/expand-border-shorthand': "error";
                 '@atlaskit/platform/expand-background-shorthand': "error";
-                '@atlaskit/platform/expand-spacing-shorthand': "off";
+                '@atlaskit/platform/expand-spacing-shorthand': "error";
                 '@compiled/jsx-pragma': ["error", {
                     importSources: string[];
                     onlyRunIfImportingCompiled: boolean;
@@ -191,7 +191,7 @@ declare const configs: {
             '@atlaskit/platform/no-module-level-eval-nav4': "error";
             '@atlaskit/platform/expand-border-shorthand': "error";
             '@atlaskit/platform/expand-background-shorthand': "error";
-            '@atlaskit/platform/expand-spacing-shorthand': "off";
+            '@atlaskit/platform/expand-spacing-shorthand': "error";
             '@compiled/jsx-pragma': ["error", {
                 importSources: string[];
                 onlyRunIfImportingCompiled: boolean;
@@ -223,7 +223,7 @@ declare const configs: {
             '@atlaskit/platform/no-module-level-eval-nav4': "error";
             '@atlaskit/platform/expand-border-shorthand': "error";
             '@atlaskit/platform/expand-background-shorthand': "error";
-            '@atlaskit/platform/expand-spacing-shorthand': "off";
+            '@atlaskit/platform/expand-spacing-shorthand': "error";
             '@compiled/jsx-pragma': ["error", {
                 importSources: string[];
                 onlyRunIfImportingCompiled: boolean;
@@ -242,7 +242,7 @@ declare const configs: {
             '@atlaskit/platform/no-module-level-eval-nav4': "error";
             '@atlaskit/platform/expand-border-shorthand': "error";
             '@atlaskit/platform/expand-background-shorthand': "error";
-            '@atlaskit/platform/expand-spacing-shorthand': "off";
+            '@atlaskit/platform/expand-spacing-shorthand': "error";
             '@compiled/jsx-pragma': ["error", {
                 importSources: string[];
                 onlyRunIfImportingCompiled: boolean;
@@ -264,7 +264,7 @@ declare const configs: {
             '@atlaskit/platform/no-module-level-eval-nav4': "error";
             '@atlaskit/platform/expand-border-shorthand': "error";
             '@atlaskit/platform/expand-background-shorthand': "error";
-            '@atlaskit/platform/expand-spacing-shorthand': "off";
+            '@atlaskit/platform/expand-spacing-shorthand': "error";
             '@compiled/jsx-pragma': ["error", {
                 importSources: string[];
                 onlyRunIfImportingCompiled: boolean;

package/dist/types-ts4.5/index.d.ts

@@ -85,7 +85,7 @@ declare const plugin: {
                 '@atlaskit/platform/no-module-level-eval-nav4': "error";
                 '@atlaskit/platform/expand-border-shorthand': "error";
                 '@atlaskit/platform/expand-background-shorthand': "error";
-                '@atlaskit/platform/expand-spacing-shorthand': "off";
+                '@atlaskit/platform/expand-spacing-shorthand': "error";
                 '@compiled/jsx-pragma': [
                     "error",
                     {
@@ -123,7 +123,7 @@ declare const plugin: {
                 '@atlaskit/platform/no-module-level-eval-nav4': "error";
                 '@atlaskit/platform/expand-border-shorthand': "error";
                 '@atlaskit/platform/expand-background-shorthand': "error";
-                '@atlaskit/platform/expand-spacing-shorthand': "off";
+                '@atlaskit/platform/expand-spacing-shorthand': "error";
                 '@compiled/jsx-pragma': [
                     "error",
                     {
@@ -145,7 +145,7 @@ declare const plugin: {
                 '@atlaskit/platform/no-module-level-eval-nav4': "error";
                 '@atlaskit/platform/expand-border-shorthand': "error";
                 '@atlaskit/platform/expand-background-shorthand': "error";
-                '@atlaskit/platform/expand-spacing-shorthand': "off";
+                '@atlaskit/platform/expand-spacing-shorthand': "error";
                 '@compiled/jsx-pragma': [
                     "error",
                     {
@@ -170,7 +170,7 @@ declare const plugin: {
                 '@atlaskit/platform/no-module-level-eval-nav4': "error";
                 '@atlaskit/platform/expand-border-shorthand': "error";
                 '@atlaskit/platform/expand-background-shorthand': "error";
-                '@atlaskit/platform/expand-spacing-shorthand': "off";
+                '@atlaskit/platform/expand-spacing-shorthand': "error";
                 '@compiled/jsx-pragma': [
                     "error",
                     {
@@ -212,7 +212,7 @@ declare const configs: {
             '@atlaskit/platform/no-module-level-eval-nav4': "error";
             '@atlaskit/platform/expand-border-shorthand': "error";
             '@atlaskit/platform/expand-background-shorthand': "error";
-            '@atlaskit/platform/expand-spacing-shorthand': "off";
+            '@atlaskit/platform/expand-spacing-shorthand': "error";
             '@compiled/jsx-pragma': [
                 "error",
                 {
@@ -250,7 +250,7 @@ declare const configs: {
             '@atlaskit/platform/no-module-level-eval-nav4': "error";
             '@atlaskit/platform/expand-border-shorthand': "error";
             '@atlaskit/platform/expand-background-shorthand': "error";
-            '@atlaskit/platform/expand-spacing-shorthand': "off";
+            '@atlaskit/platform/expand-spacing-shorthand': "error";
             '@compiled/jsx-pragma': [
                 "error",
                 {
@@ -272,7 +272,7 @@ declare const configs: {
             '@atlaskit/platform/no-module-level-eval-nav4': "error";
             '@atlaskit/platform/expand-border-shorthand': "error";
             '@atlaskit/platform/expand-background-shorthand': "error";
-            '@atlaskit/platform/expand-spacing-shorthand': "off";
+            '@atlaskit/platform/expand-spacing-shorthand': "error";
             '@compiled/jsx-pragma': [
                 "error",
                 {
@@ -297,7 +297,7 @@ declare const configs: {
             '@atlaskit/platform/no-module-level-eval-nav4': "error";
             '@atlaskit/platform/expand-border-shorthand': "error";
             '@atlaskit/platform/expand-background-shorthand': "error";
-            '@atlaskit/platform/expand-spacing-shorthand': "off";
+            '@atlaskit/platform/expand-spacing-shorthand': "error";
             '@compiled/jsx-pragma': [
                 "error",
                 {

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "@atlaskit/eslint-plugin-platform",
 	"description": "The essential plugin for use with Atlassian frontend platform tools",
-	"version": "2.2.0",
+	"version": "2.3.0",
 	"author": "Atlassian Pty Ltd",
 	"atlassian": {
 		"team": "Build Infra",
@@ -31,7 +31,7 @@
 		".": "./src/index.tsx"
 	},
 	"dependencies": {
-		"@atlaskit/eslint-utils": "^1.8.0",
+		"@atlaskit/eslint-utils": "^1.9.0",
 		"@babel/runtime": "^7.0.0",
 		"@compiled/eslint-plugin": "^0.18.2",
 		"@manypkg/find-root": "^1.1.0",

package/src/index.tsx

@@ -73,7 +73,7 @@ const commonConfig = {
 	// Compiled: rules that are not included via `@compiled/recommended
 	'@atlaskit/platform/expand-border-shorthand': 'error',
 	'@atlaskit/platform/expand-background-shorthand': 'error',
-	'@atlaskit/platform/expand-spacing-shorthand': 'off',
+	'@atlaskit/platform/expand-spacing-shorthand': 'error',
 	'@compiled/jsx-pragma': [
 		'error',
 		{

package/src/rules/compiled/expand-spacing-shorthand/__tests__/rule.test.ts

@@ -79,6 +79,28 @@ const validTestCases = (property: string) => {
 			});
 			`,
 		},
+		// Template literal containing expressions that is not token should only throw error with no autofix
+		{
+			name: `${property}: property value template string where it contains an expression that is not a token`,
+			code: outdent`
+			import {css} from '@compiled/react';
+			const styles = css({
+				${property}: \`\${token('space.100', '8px')} \${gridSize * 2}px\`,
+			});
+			const styles2 = css({
+				padding: \`\${DROPDOWN_HEADER_VERTICAL_PADDING} \${token('space.075','6px')} 
+					\${DROPDOWN_HEADER_VERTICAL_PADDING} \${token('space.200', '16px')}\`,
+			});
+			const styles3 = css({
+				${property}: \`\${token('space.050', '4px')} \${token('space.150', '12px')} \${token('space.150', '12px')} 
+					\${({ isSummaryView }: EditFormContentWrapperProps) =>
+					isSummaryView ? token('space.0', '0') : token('space.150', '12px')}\`,
+			});
+			const styles4 = css({
+				${property}: \`\${token('space.100', '8px')} -\${gridSize}px 0\`,
+			});
+			`,
+		},
 	];
 };
 
@@ -252,25 +274,35 @@ const invalidTestCases = (property: string) => {
 			`,
 			errors: [{ messageId: 'expandSpacingShorthand' }],
 		},
-		// Template literal containing expressions that is not token should only throw error with no autofix
+		// Strings that are not valid property values should not be autofixed (e.g. !important)
 		{
-			name: `${property}: property value template string where it contains an expression that is not a token`,
+			name: `${property}: Don't autofix if not able to handle all the string values, e.g !important`,
 			code: outdent`
 			import {css} from '@compiled/react';
 			const styles = css({
-				${property}: \`\${token('space.100', '8px')} \${gridSize * 2}px\`,
+				${property}: \`0 \${token('space.200', '16px')} !important\`,
 			});
-			const styles2 = css({
-				padding: \`\${DROPDOWN_HEADER_VERTICAL_PADDING} \${token('space.075','6px')} 
-					\${DROPDOWN_HEADER_VERTICAL_PADDING} \${token('space.200', '16px')}\`,
+			`,
+			errors: [{ messageId: 'expandSpacingShorthand' }],
+		},
+		{
+			name: `${property}: Autofix if able to handle all string values`,
+			code: outdent`
+			import {css} from '@compiled/react';
+			const styles = css({
+				${property}: \`0 auto \${token('space.300', '24px')}\`,
 			});
-			const styles3 = css({
-				${property}: \`\${token('space.050', '4px')} \${token('space.150', '12px')} \${token('space.150', '12px')} 
-					\${({ isSummaryView }: EditFormContentWrapperProps) =>
-					isSummaryView ? token('space.0', '0') : token('space.150', '12px')}\`,
+			`,
+			output: outdent`
+			import {css} from '@compiled/react';
+			const styles = css({
+				${property}Top: 0,
+				${property}Right: 'auto',
+				${property}Bottom: token('space.300', '24px'),
+				${property}Left: 'auto',
 			});
 			`,
-			errors: Array.from(Array(3), () => ({ messageId: 'expandSpacingShorthand' })),
+			errors: [{ messageId: 'expandSpacingShorthand' }],
 		},
 		// Miscellaneous
 		{
@@ -383,7 +415,7 @@ const invalidTestCases = (property: string) => {
 			`,
 			errors: Array.from(Array(2), () => ({ messageId: 'expandSpacingShorthand' })),
 		},
-		// TODO (AFB-1022): Resolve this failing test
+		// // TODO (AFB-1022): Resolve this failing test
 		// {
 		//  name: `${property}: styled components with prop input`,
 		//  code: outdent`

package/src/rules/compiled/expand-spacing-shorthand/index.ts

@@ -49,25 +49,59 @@ const parseTemplateLiteral = (templateLiteral: TemplateLiteral, context: Rule.Ru
 	return propertyValues;
 };
 
-const checkValidPropertyValues = (propertyValues: string[]) => {
+/**
+ * Checks if the parsed property values are valid; values that we shouldn't handle with the fixer:
+ * 1. At least one expression in TemplateLiteral is not a token expression
+ * 2. Do not contain a token
+ * 3. Have length that are not in the range [1, 4] for different spacing directions
+ * 		- No more than 4 to exclude additional values such as `!important`
+ * 4. Includes `calc(...)`
+ * Then, the rule will return with no error
+ * @param propertyValues property values parsed as list of strings
+ * @returns boolean
+ */
+const isPropertyValueExempted = (templateLiteral: TemplateLiteral, propertyValues: string[]) => {
+	const expressions = templateLiteral.expressions;
+	if (!expressions.every((expr) => expr.type === 'CallExpression' && isTokenCallExpression(expr))) {
+		return true;
+	}
 	if (!propertyValues.some((str) => str.includes('token('))) {
-		return false;
+		return true;
 	}
 	if (propertyValues.length < 1 || propertyValues.length > 4) {
-		return false;
+		return true;
 	}
 	if (propertyValues.some((str) => str.includes('calc('))) {
-		return false;
+		return true;
 	}
-	return true;
+	return false;
 };
 
-// Check that all expressions in TemplateLiteral are token expressions
-// If true: create an autofix
-// If false: report violation without autofix
-const hasOnlyTokens = (templateLiteral: TemplateLiteral) => {
-	const expressions = templateLiteral.expressions;
-	return expressions.every((expr) => expr.type === 'CallExpression' && isTokenCallExpression(expr));
+/**
+ *  Checks if the parsed property values are invalid (i.e. rule violation thrown) and autofix required. Cases are when
+ *  property values must have a format which includes -> e.g. 2, '2(rem|em|px)', auto, initial, inherit, token(...)
+ *  The rule will return with error and provide a fix
+ * @param templateLiteral TemplateLiteral AST Node
+ * @param propertyValues property values parsed as list of strings
+ * @returns boolean
+ */
+const isPropertyValuesInvalidFix = (propertyValues: string[]) => {
+	for (const propValue of propertyValues) {
+		if (propValue === '0') {
+			continue;
+		}
+		if (['auto', 'initial', 'inherit'].includes(propValue.slice(1, -1))) {
+			continue;
+		}
+		if (/^token\(.*\)$/.test(propValue)) {
+			continue;
+		}
+		if (/^['"]\d+(\.\d+)?((rem)|(em)|(px))['"]$/.test(propValue)) {
+			continue;
+		}
+		return false;
+	}
+	return true;
 };
 
 // To fix spacing shorthands, given a list of spacing property values, expands the spacing property and adds autofix fixes
@@ -130,8 +164,30 @@ const executeExpandSpacingRule = (
 		return;
 	}
 	if (node.value.type === 'TemplateLiteral') {
-		// Value of spacing property is a TemplateLiteral type that contains a token, e.g. padding: `0 token('a')`
-		if (!hasOnlyTokens(node.value)) {
+		const propertyValues = parseTemplateLiteral(node.value, context);
+		if (isPropertyValueExempted(node.value, propertyValues)) {
+			// Valid, so no error should be thrown
+			return;
+		}
+		if (isPropertyValuesInvalidFix(propertyValues)) {
+			// Invalid, so error should be thrown and fix provided
+			context.report({
+				node,
+				messageId: 'expandSpacingShorthand',
+				data: {
+					property: propertyShorthand,
+				},
+				fix(fixer) {
+					return expandSpacingProperties({
+						context,
+						node,
+						propertyValues,
+						fixer,
+						propertyShorthand,
+					});
+				},
+			});
+		} else {
 			context.report({
 				node,
 				messageId: 'expandSpacingShorthand',
@@ -141,20 +197,6 @@ const executeExpandSpacingRule = (
 			});
 			return;
 		}
-		const propertyValues = parseTemplateLiteral(node.value, context);
-		if (!checkValidPropertyValues(propertyValues)) {
-			return;
-		}
-		context.report({
-			node,
-			messageId: 'expandSpacingShorthand',
-			data: {
-				property: propertyShorthand,
-			},
-			fix(fixer) {
-				return expandSpacingProperties({ context, node, propertyValues, fixer, propertyShorthand });
-			},
-		});
 	} else if (node.value.type === 'CallExpression' && isTokenCallExpression(node.value)) {
 		// Value of spacing property is a token CallExpression type, e.g. margin: token('space.100', '8px')
 		const propertyValues = [getSourceCode(context).getText(node.value)];

package/src/rules/feature-gating/README.md

@@ -1,7 +1,7 @@
 ### Notes
 
 - feature-gating/\* rules are copied from
-  [eslint-plugin-jira/rules/ff](https://stash.atlassian.com/projects/ATLASSIAN/repos/atlassian-frontend-monorepo/browse/jira/build-tools/eslint-plugin-jira/rules/ff)
+  [eslint-plugin-jira/rules/ff](https://stash.atlassian.com/projects/ATLASSIAN/repos/atlassian-frontend-monorepo/browse/jira/dev-tooling/packages/eslint-plugin-jira/rules/ff)
   with small variations as mentioned in this
   [PR](https://stash.atlassian.com/projects/ATLASSIAN/repos/atlassian-frontend-monorepo/pull-requests/115546/overview)
 - these rules could be a WIP since they are still targeting JFE libraries- see