@primer/stylelint-config 13.5.0 → 13.6.0-rc.55219cf

package/dist/index.cjs

@@ -10,13 +10,16 @@ var matchAll = require('string.prototype.matchall');
 var _documentCurrentScript = typeof document !== 'undefined' ? document.currentScript : null;
 const require$2 = node_module.createRequire((typeof document === 'undefined' ? require('u' + 'rl').pathToFileURL(__filename).href : (_documentCurrentScript && _documentCurrentScript.tagName.toUpperCase() === 'SCRIPT' && _documentCurrentScript.src || new URL('index.cjs', document.baseURI).href)));
 
-function primitivesVariables(type) {
+function primitivesVariables(type, options = {}) {
   const variables = [];
 
   const files = [];
   switch (type) {
     case 'spacing':
       files.push('base/size/size.json');
+      if (options.includeFunctional) {
+        files.push('functional/size/size.json');
+      }
       break
     case 'border':
       files.push('functional/size/border.json');
@@ -669,8 +672,18 @@ const messages$2 = ruleMessages$1(ruleName$2, {
 const propList$1 = ['padding', 'margin', 'top', 'right', 'bottom', 'left'];
 // Values that we want to ignore
 const valueList = ['${'];
+// Exact keyword values that we want to ignore (remain as-is, e.g. `padding: unset`)
+const ignoredExactValues = ['*', '+', '-', '/', '0', 'auto', 'inherit', 'initial', 'unset'];
 
 const sizes = primitivesVariables('spacing');
+const allowedSizeVars = primitivesVariables('spacing', {includeFunctional: true});
+
+const escapeForRegExp = string => string.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+
+const allowedSizeVarMatchers = allowedSizeVars.map(variable => {
+  const escapedName = escapeForRegExp(variable['name']);
+  return new RegExp(`${escapedName}\\b`)
+});
 
 // Add +-1px to each value
 for (const size of sizes) {
@@ -705,7 +718,7 @@ const ruleFunction$1 = primary => {
         }
 
         // Exact values to ignore.
-        if (['*', '+', '-', '/', '0', 'auto', 'inherit', 'initial'].includes(node.value)) {
+        if (ignoredExactValues.includes(node.value)) {
           return
         }
 
@@ -721,11 +734,7 @@ const ruleFunction$1 = primary => {
         }
 
         // If the variable is found in the value, skip it.
-        if (
-          sizes.some(variable =>
-            new RegExp(`${variable['name'].replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}\\b`).test(node.value),
-          )
-        ) {
+        if (allowedSizeVarMatchers.some(matcher => matcher.test(node.value))) {
           return
         }
 

package/dist/index.mjs

@@ -7,13 +7,16 @@ import matchAll from 'string.prototype.matchall';
 
 const require$2 = createRequire(import.meta.url);
 
-function primitivesVariables(type) {
+function primitivesVariables(type, options = {}) {
   const variables = [];
 
   const files = [];
   switch (type) {
     case 'spacing':
       files.push('base/size/size.json');
+      if (options.includeFunctional) {
+        files.push('functional/size/size.json');
+      }
       break
     case 'border':
       files.push('functional/size/border.json');
@@ -666,8 +669,18 @@ const messages$2 = ruleMessages$1(ruleName$2, {
 const propList$1 = ['padding', 'margin', 'top', 'right', 'bottom', 'left'];
 // Values that we want to ignore
 const valueList = ['${'];
+// Exact keyword values that we want to ignore (remain as-is, e.g. `padding: unset`)
+const ignoredExactValues = ['*', '+', '-', '/', '0', 'auto', 'inherit', 'initial', 'unset'];
 
 const sizes = primitivesVariables('spacing');
+const allowedSizeVars = primitivesVariables('spacing', {includeFunctional: true});
+
+const escapeForRegExp = string => string.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+
+const allowedSizeVarMatchers = allowedSizeVars.map(variable => {
+  const escapedName = escapeForRegExp(variable['name']);
+  return new RegExp(`${escapedName}\\b`)
+});
 
 // Add +-1px to each value
 for (const size of sizes) {
@@ -702,7 +715,7 @@ const ruleFunction$1 = primary => {
         }
 
         // Exact values to ignore.
-        if (['*', '+', '-', '/', '0', 'auto', 'inherit', 'initial'].includes(node.value)) {
+        if (ignoredExactValues.includes(node.value)) {
           return
         }
 
@@ -718,11 +731,7 @@ const ruleFunction$1 = primary => {
         }
 
         // If the variable is found in the value, skip it.
-        if (
-          sizes.some(variable =>
-            new RegExp(`${variable['name'].replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}\\b`).test(node.value),
-          )
-        ) {
+        if (allowedSizeVarMatchers.some(matcher => matcher.test(node.value))) {
           return
         }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@primer/stylelint-config",
-  "version": "13.5.0",
+  "version": "13.6.0-rc.55219cf",
   "description": "Sharable stylelint config used by GitHub's CSS",
   "author": "GitHub, Inc.",
   "license": "MIT",

package/plugins/lib/utils.js

@@ -2,13 +2,16 @@ import {createRequire} from 'node:module'
 
 const require = createRequire(import.meta.url)
 
-export function primitivesVariables(type) {
+export function primitivesVariables(type, options = {}) {
   const variables = []
 
   const files = []
   switch (type) {
     case 'spacing':
       files.push('base/size/size.json')
+      if (options.includeFunctional) {
+        files.push('functional/size/size.json')
+      }
       break
     case 'border':
       files.push('functional/size/border.json')

package/plugins/spacing.js

@@ -23,8 +23,18 @@ export const messages = ruleMessages(ruleName, {
 const propList = ['padding', 'margin', 'top', 'right', 'bottom', 'left']
 // Values that we want to ignore
 const valueList = ['${']
+// Exact keyword values that we want to ignore (remain as-is, e.g. `padding: unset`)
+const ignoredExactValues = ['*', '+', '-', '/', '0', 'auto', 'inherit', 'initial', 'unset']
 
 const sizes = primitivesVariables('spacing')
+const allowedSizeVars = primitivesVariables('spacing', {includeFunctional: true})
+
+const escapeForRegExp = string => string.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')
+
+const allowedSizeVarMatchers = allowedSizeVars.map(variable => {
+  const escapedName = escapeForRegExp(variable['name'])
+  return new RegExp(`${escapedName}\\b`)
+})
 
 // Add +-1px to each value
 for (const size of sizes) {
@@ -59,7 +69,7 @@ const ruleFunction = primary => {
         }
 
         // Exact values to ignore.
-        if (['*', '+', '-', '/', '0', 'auto', 'inherit', 'initial'].includes(node.value)) {
+        if (ignoredExactValues.includes(node.value)) {
           return
         }
 
@@ -75,11 +85,7 @@ const ruleFunction = primary => {
         }
 
         // If the variable is found in the value, skip it.
-        if (
-          sizes.some(variable =>
-            new RegExp(`${variable['name'].replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}\\b`).test(node.value),
-          )
-        ) {
+        if (allowedSizeVarMatchers.some(matcher => matcher.test(node.value))) {
           return
         }