Fix false positives for parametric mixins in selector-class-pattern

lib/utils/__tests__/isStandardSyntaxRule.test.js

@@ -38,8 +38,8 @@ describe('isStandardSyntaxRule', () => {
 	it('Scss nested properties', () => {
 		expect(isStandardSyntaxRule(node('foo: {};'))).toBeFalsy();
 	});
-	it('called Less class parametric mixin', () => {
-		expect(isStandardSyntaxRule(lessNode('.mixin-name(@var);'))).toBeFalsy();
+	it('Less class parametric mixin', () => {
+		expect(isStandardSyntaxRule(lessNode('.mixin-name(@var) {}'))).toBeFalsy();
 	});
 	it('non-outputting parametric Less class mixin definition', () => {
 		expect(isStandardSyntaxRule(lessNode('.mixin-name() {}'))).toBeFalsy();
@@ -53,32 +53,23 @@ describe('isStandardSyntaxRule', () => {
 	it('non-outputting Less ID mixin definition', () => {
 		expect(isStandardSyntaxRule(lessNode('#mixin-name() {}'))).toBeFalsy();
 	});
-	it('called Less ID mixin', () => {
-		expect(isStandardSyntaxRule(lessNode('#mixin-name;'))).toBeFalsy();
-	});
-	it('called namespaced Less mixin (child)', () => {
-		expect(isStandardSyntaxRule(lessNode('#namespace > .mixin-name;'))).toBeFalsy();
-	});
-	it('called namespaced Less mixin (descendant)', () => {
-		expect(isStandardSyntaxRule(lessNode('#namespace .mixin-name;'))).toBeFalsy();
-	});
-	it('called namespaced Less mixin (compound)', () => {
-		expect(isStandardSyntaxRule(lessNode('#namespace.mixin-name;'))).toBeFalsy();
-	});
 	it('less mixin', () => {
 		expect(
 			isStandardSyntaxRule(lessNode('.box-shadow(@style, @c) when (iscolor(@c)) {}')),
 		).toBeFalsy();
 	});
 	it('less extend', () => {
-		expect(isStandardSyntaxRule(lessNode('&:extend(.inline);'))).toBeFalsy();
+		expect(isStandardSyntaxRule(lessNode('&:extend(.inline) {}'))).toBeFalsy();
 	});
 	it('less detached rulesets', () => {
 		expect(isStandardSyntaxRule(lessNode('@foo: {};'))).toBeFalsy();
 	});
 	it('less guarded namespaces', () => {
 		expect(isStandardSyntaxRule(lessNode('#namespace when (@mode=huge) {}'))).toBeFalsy();
 	});
+	it('less parametric mixins', () => {
+		expect(isStandardSyntaxRule(lessNode('.mixin (@variable: 5) {}'))).toBeFalsy();
+	});
 	it('mixin guards', () => {
 		expect(
 			isStandardSyntaxRule(lessNode('.mixin (@variable) when (@variable = 10px) {}')),

lib/utils/isStandardSyntaxRule.js

@@ -22,36 +22,13 @@ module.exports = function (rule) {
 		return false;
 	}
 
-	// Called Less mixin (e.g. a { .mixin() })
-	// @ts-expect-error -- TODO TYPES support LESS and SASS types somehow
-	if (rule.mixin) {
-		return false;
-	}
-
-	// Less detached rulesets
-	if (selector.startsWith('@') && selector.endsWith(':')) {
-		return false;
-	}
-
-	// Ignore Less &:extend rule
-	if ('extend' in rule && rule.extend) {
-		return false;
-	}
-
-	// Ignore mixin or &:extend rule
-	// https://github.com/shellscape/postcss-less/blob/master/lib/less-parser.js#L52
-	// @ts-expect-error -- TODO TYPES support LESS and SASS types somehow
-	if (rule.params && rule.params[0]) {
-		return false;
-	}
-
 	// Non-outputting Less mixin definition (e.g. .mixin() {})
 	if (selector.endsWith(')') && !selector.includes(':')) {
 		return false;
 	}
 
-	// Less guards
-	if (/when\s+(not\s+)*\(/.test(selector)) {
+	// Less Parametric mixins (e.g. .mixin(@variable: x) {})
+	if (/\(@.*\)$/.test(selector)) {
 		return false;
 	}