@d-zero/stylelint-rules 5.0.0-alpha.13 → 5.0.0-alpha.15

package/dist/rules/declaration-value-type-disallowed-list/index.js

@@ -34,6 +34,9 @@ export default createRule({
             }
             root.walkDecls((decl) => {
                 const nodes = getValueType(decl);
+                if (nodes === null) {
+                    return;
+                }
                 for (const node of nodes) {
                     if (!node.valueType) {
                         continue;

package/dist/utils/get-value-type.js

@@ -1,6 +1,9 @@
 import CSSTree from '@d-zero/csstree-scss-syntax';
 import postcssValueParser from 'postcss-value-parser';
 export function getValueType(decl) {
+    if (decl.prop.startsWith('$')) {
+        return null;
+    }
     return _getValueType(decl.prop, decl.value);
 }
 function _getValueType(prop, value) {
@@ -14,33 +17,40 @@ function _getValueType(prop, value) {
     const values = valueAst.nodes.filter((node) => node.type === 'string' || node.type === 'function' || node.type === 'word');
     const props = cssTreeDecl.matched;
     if (props === null) {
-        throw cssTreeDecl.error;
+        return null;
     }
-    const valueTypes = props.match.map((node) => getValueNode(node).syntax.name);
+    const valueTypes = props.match
+        .flatMap((node) => getValueNode(node))
+        .map((node) => node.syntax.name);
     return values.map((value, i) => {
         const valueType = valueTypes[i] ?? null;
-        if (valueType === null) {
-            if (value.type === 'word' && value.value.startsWith('$')) {
-                return {
-                    value: value,
-                    valueType: '$SASS_VARIABLE',
-                };
-            }
-            throw new Error('Value type not found');
+        if (valueType === null && value.type === 'word' && value.value.startsWith('$')) {
+            return {
+                value: value,
+                valueType: '$SASS_VARIABLE',
+            };
         }
         return {
             value,
-            valueType,
+            valueType: valueType ?? 'unknown',
         };
     });
 }
 function getValueNode(node) {
-    if (isProperty(node)) {
-        const firstChild = node.match.at(0);
-        return getValueNode(firstChild);
+    if (isProperty(node) || isType(node)) {
+        if (node.match[0].syntax === null) {
+            return isProperty(node) ? [] : [node];
+        }
+        if (isType(node) && node.match.length === 1) {
+            return [node];
+        }
+        return node.match.flatMap((child) => child.syntax === null ? [] : getValueNode(child));
     }
-    return node;
+    return [node];
 }
 function isProperty(node) {
-    return node.syntax.type === 'Property';
+    return node.syntax?.type === 'Property';
+}
+function isType(node) {
+    return node.syntax?.type === 'Type';
 }

package/dist/utils/get-value-type.spec.js

@@ -6,7 +6,7 @@ function p(css) {
     const rule = root.first;
     const decl = rule.first;
     const nodeWithType = getValueType(decl);
-    return nodeWithType.map((node) => node.valueType);
+    return nodeWithType?.map((node) => node.valueType) ?? null;
 }
 describe('getValueType', () => {
     test('flex', () => {
@@ -22,7 +22,19 @@ describe('getValueType', () => {
     test('height vw', () => {
         expect(p('a { height: 100vw }')).toEqual(['length']);
     });
+    test('background', () => {
+        expect(p('a { background: url(../../img/edit.png) no-repeat scroll 0 0 }')).toEqual([
+            'bg-image',
+            'repeat-style',
+            'attachment',
+            'length-percentage',
+            'length-percentage',
+        ]);
+    });
     test('SASS Variable', () => {
         expect(p('a { flex: 1 2 $basis }')).toEqual(['number', 'number', '$SASS_VARIABLE']);
     });
+    test('SASS Variable Definition', () => {
+        expect(p('a { $var: value }')).toEqual(null);
+    });
 });

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@d-zero/stylelint-rules",
-	"version": "5.0.0-alpha.13",
+	"version": "5.0.0-alpha.15",
 	"description": "Rules of Stylelint for D-ZERO",
 	"repository": "https://github.com/d-zero-dev/linters.git",
 	"author": "D-ZERO Co., Ltd.",
@@ -20,10 +20,10 @@
 		"build": "tsc"
 	},
 	"dependencies": {
-		"@d-zero/csstree-scss-syntax": "5.0.0-alpha.13",
+		"@d-zero/csstree-scss-syntax": "5.0.0-alpha.15",
 		"css-tree": "2.3.1",
 		"postcss-value-parser": "4.2.0",
 		"stylelint": "16.2.1"
 	},
-	"gitHead": "a68c1dad34f200e966c4003f53b7b072bbea7900"
+	"gitHead": "0a6a1f910f3ca2802a5e7cfee9bc00a67889d24e"
 }