Fix false positives for Less variables and mixins in at-rule-*

lib/rules/at-rule-semicolon-newline-after/__tests__/index.js

@@ -125,3 +125,25 @@ testRule(rule, {
     }
   ]
 });
+
+testRule(rule, {
+  ruleName,
+  syntax: "less",
+  config: ["always"],
+  accept: [
+    {
+      code: `
+        .someMixin() { margin: 0; }
+        span { .someMixin(); }
+      `,
+      description: "ignore Less mixin"
+    },
+    {
+      code: `
+        @myVariable: #f7f8f9;
+        span { background-color: @myVariable; }
+      `,
+      description: "ignore Less variable"
+    }
+  ]
+});

lib/rules/at-rule-semicolon-newline-after/index.js

@@ -1,6 +1,7 @@
 "use strict";
 
 const hasBlock = require("../../utils/hasBlock");
+const isStandardSyntaxAtRule = require("../../utils/isStandardSyntaxAtRule");
 const nextNonCommentNode = require("../../utils/nextNonCommentNode");
 const rawNodeString = require("../../utils/rawNodeString");
 const report = require("../../utils/report");
@@ -38,6 +39,10 @@ const rule = function(actual, secondary, context) {
         return;
       }
 
+      if (!isStandardSyntaxAtRule(atRule)) {
+        return;
+      }
+
       // Allow an end-of-line comment
       const nodeToCheck = nextNonCommentNode(nextNode);
 

lib/rules/at-rule-semicolon-space-before/__tests__/index.js

@@ -151,3 +151,27 @@ testRule(rule, {
     }
   ]
 });
+
+testRule(rule, {
+  ruleName,
+  syntax: "less",
+  config: ["always"],
+  skipBasicChecks: true,
+
+  accept: [
+    {
+      code: `
+        .someMixin() { margin: 0; }
+        span { .someMixin(); }
+      `,
+      description: "ignore Less mixin"
+    },
+    {
+      code: `
+        @myVariable: #f7f8f9;
+        span { background-color: @myVariable; }
+      `,
+      description: "ignore Less variable"
+    }
+  ]
+});

lib/rules/at-rule-semicolon-space-before/index.js

@@ -1,6 +1,7 @@
 "use strict";
 
 const hasBlock = require("../../utils/hasBlock");
+const isStandardSyntaxAtRule = require("../../utils/isStandardSyntaxAtRule");
 const rawNodeString = require("../../utils/rawNodeString");
 const report = require("../../utils/report");
 const ruleMessages = require("../../utils/ruleMessages");
@@ -32,6 +33,10 @@ const rule = function(expectation) {
         return;
       }
 
+      if (!isStandardSyntaxAtRule(atRule)) {
+        return;
+      }
+
       const nodeString = rawNodeString(atRule);
 
       checker.before({

lib/rules/at-rule-whitelist/__tests__/index.js

@@ -152,3 +152,21 @@ testRule(rule, {
     }
   ]
 });
+
+testRule(rule, {
+  ruleName,
+  syntax: "less",
+  config: ["keyframes"],
+  skipBasicChecks: true,
+
+  accept: [
+    {
+      code: `
+        .mixin() { margin: 0; }
+
+        span { .mixin(); }
+      `,
+      description: "ignore Less mixin which are treated as at-rule"
+    }
+  ]
+});

lib/rules/at-rule-whitelist/index.js

@@ -1,6 +1,7 @@
 "use strict";
 
 const _ = require("lodash");
+const isStandardSyntaxAtRule = require("../../utils/isStandardSyntaxAtRule");
 const postcss = require("postcss");
 const report = require("../../utils/report");
 const ruleMessages = require("../../utils/ruleMessages");
@@ -29,6 +30,10 @@ const rule = function(whitelistInput) {
     root.walkAtRules(atRule => {
       const name = atRule.name;
 
+      if (!isStandardSyntaxAtRule(atRule)) {
+        return;
+      }
+
       if (
         whitelist.indexOf(postcss.vendor.unprefixed(name).toLowerCase()) !== -1
       ) {