Fix function-url-quotes false positives for SCSS with() construct (…

…#6847)

.changeset/nasty-chefs-flow.md

@@ -0,0 +1,5 @@
+---
+"stylelint": patch
+---
+
+Fixed: `function-url-quotes` false positives for SCSS `with()` construct

lib/rules/function-url-quotes/__tests__/index.js

@@ -1,5 +1,7 @@
 'use strict';
 
+const { stripIndent } = require('common-tags');
+
 const naiveCssInJs = require('../../../__tests__/fixtures/postcss-naive-css-in-js');
 
 const { messages, ruleName } = require('..');
@@ -743,12 +745,25 @@ testRule({
 
 	accept: [
 		{
-			code: '$a: (\n  // foo\n)',
+			code: stripIndent`
+				$a: (
+					// foo
+				)`,
 			description: 'SCSS map',
 		},
 		{
-			code: '@function a(\n  // foo\n) {}',
+			code: stripIndent`
+				@function a(
+					// foo
+				) {}`,
 			description: 'SCSS function',
 		},
+		{
+			code: stripIndent`
+				@forward "module" with (
+					// foo
+				)`,
+			description: 'SCSS with',
+		},
 	],
 });

lib/rules/function-url-quotes/index.js

@@ -70,28 +70,21 @@ const rule = (primary, secondaryOptions, context) => {
 			}
 		}
 
-		/**
-		 * @param {import('postcss-value-parser').Node} node
-		 */
-		function containsOnlyComments(node) {
-			return 'nodes' in node && node.nodes.every((child) => child.type === 'comment');
-		}
-
 		/**
 		 * @param {import('postcss').AtRule} atRule
 		 */
 		function checkAtRuleParams(atRule) {
 			const params = getAtRuleParams(atRule);
 			const startIndex = atRuleParamIndex(atRule);
+
+			let hasUrlFunction = false;
+
 			const parsed = functionArgumentsSearch(params, /^url$/i, (args, index, funcNode) => {
+				hasUrlFunction = true;
 				checkArgs(atRule, args, startIndex + index, funcNode);
 			});
 
-			const [funcNode] = parsed.nodes;
-
-			if (funcNode && containsOnlyComments(funcNode)) {
-				return;
-			}
+			if (!hasUrlFunction) return;
 
 			if (context.fix) {
 				atRule.params = parsed.toString();