Add support for stylelint 16 (#187)

Co-authored-by: Krister Kari <[email protected]>

.github/workflows/test.yml

@@ -11,7 +11,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        node: [18, 20]
+        node: [20]
         os: [ubuntu-latest, windows-latest]
 
     steps:

package.json

@@ -33,8 +33,8 @@
     "prepublish": "npm run build",
     "pretest": "npm run lint",
     "release": "np",
-    "jest": "jest",
-    "test": "jest --coverage"
+    "jest": "cross-env NODE_OPTIONS=\"--experimental-vm-modules --no-warnings\" jest",
+    "test": "cross-env NODE_OPTIONS=\"--experimental-vm-modules --no-warnings\" jest --coverage"
   },
   "repository": {
     "type": "git",
@@ -46,7 +46,7 @@
     "url": "https://github.com/kristerkari/stylelint-react-native/issues"
   },
   "engines": {
-    "node": ">=6"
+    "node": ">=18.12.0"
   },
   "homepage": "https://github.com/kristerkari/stylelint-react-native#readme",
   "jest": {
@@ -68,6 +68,7 @@
         "statements": 75
       }
     },
+    "preset": "jest-preset-stylelint",
     "setupFiles": [
       "./jest-setup.js"
     ],
@@ -78,7 +79,7 @@
     "testRegex": ".*\\.test\\.js$|src/.*/__tests__/.*\\.js$"
   },
   "peerDependencies": {
-    "stylelint": "^8.0.0 || ^9.0.0 || ^10.0.0 || ^11.0.0 || ^12.0.0 || ^13.0.0 || ^14.0.0 || ^15.0.0"
+    "stylelint": "^8.0.0 || ^9.0.0 || ^10.0.0 || ^11.0.0 || ^12.0.0 || ^13.0.0 || ^14.0.0 || ^15.0.0 || ^16.0.0"
   },
   "lint-staged": {
     "**/*.{js,json,md}": [
@@ -128,21 +129,22 @@
     "@stylelint/postcss-css-in-js": "^0.38.0",
     "babel-plugin-istanbul": "^6.0.0",
     "babel-preset-jest": "^29.2.0",
+    "cross-env": "^7.0.3",
     "eslint": "^8.31.0",
     "eslint-config-stylelint": "^20.0.0",
     "eslint-plugin-sort-requires": "^2.1.0",
     "husky": "^7.0.1",
-    "jest": "^29.4.2",
-    "jest-cli": "^29.4.2",
-    "jest-preset-stylelint": "^6.0.0",
+    "jest": "^29.7.0",
+    "jest-cli": "^29.7.0",
+    "jest-preset-stylelint": "^7.0.0",
     "lint-staged": "^13.1.0",
     "np": "^8.0.4",
     "postcss-less": "^5.0.0",
     "postcss-scss": "^4.0.2",
     "postcss-syntax": "^0.36.2",
     "prettier": "^3.0.0",
     "rimraf": "^5.0.1",
-    "stylelint": "^15.1.0"
+    "stylelint": "^16.2.0"
   },
   "prettier": {
     "semi": true,

src/index.js

@@ -6,4 +6,4 @@ const rulesPlugins = Object.keys(rules).map(ruleName => {
   return createPlugin(namespace(ruleName), rules[ruleName]);
 });
 
-export default rulesPlugins;
+module.exports = rulesPlugins;

src/rules/font-weight-no-ignored-values/index.js

@@ -1,7 +1,7 @@
 import { utils } from "stylelint";
 import { namespace } from "../../utils";
 
-const declarationValueIndex = require("stylelint/lib/utils/declarationValueIndex");
+const declarationValueIndex = require("../../utils/declarationValueIndex");
 
 export const ruleName = namespace("font-weight-no-ignored-values");
 

src/rules/index.js

@@ -2,8 +2,10 @@ import cssPropertyNoUnknown from "./css-property-no-unknown";
 import stylePropertyNoUnknown from "./style-property-no-unknown";
 import fontWeightNoIgnoredValues from "./font-weight-no-ignored-values";
 
-export default {
+const rules = {
   "font-weight-no-ignored-values": fontWeightNoIgnoredValues,
   "css-property-no-unknown": cssPropertyNoUnknown,
   "style-property-no-unknown": stylePropertyNoUnknown
 };
+
+module.exports = rules;

src/utils/declarationValueIndex.js

@@ -0,0 +1,28 @@
+const validateTypes = require("./validateTypes");
+
+/**
+ * Get the index of a declaration's value
+ *
+ * @param {import('postcss').Declaration} decl
+ * @returns {number}
+ */
+function declarationValueIndex(decl) {
+  const raws = decl.raws;
+  const prop = raws.prop;
+
+  return [
+    validateTypes.isObject(prop) && "prefix" in prop && prop.prefix,
+    (validateTypes.isObject(prop) && "raw" in prop && prop.raw) || decl.prop,
+    validateTypes.isObject(prop) && "suffix" in prop && prop.suffix,
+    raws.between || ":",
+    raws.value && "prefix" in raws.value && raws.value.prefix
+  ].reduce((/** @type {number} */ count, str) => {
+    if (validateTypes.isString(str)) {
+      return count + str.length;
+    }
+
+    return count;
+  }, 0);
+}
+
+module.exports = declarationValueIndex;

src/utils/validateTypes.js

@@ -0,0 +1,19 @@
+/** * Checks if the value is an object.
+ * @param {unknown} value
+ * @returns {value is object}
+ */
+function isObject(value) {
+  return value !== null && typeof value === "object";
+}
+
+/**
+ * Checks if the value is a string or a String object.
+ * @param {unknown} value
+ * @returns {value is string}
+ */
+function isString(value) {
+  return typeof value === "string" || value instanceof String;
+}
+
+exports.isObject = isObject;
+exports.isString = isString;