Remove pure mode in favour of linter

Author: TrySound

src/index.js

@@ -116,13 +116,12 @@ function localizeNode(node, context) {
   return node;
 }
 
-const localizeSelectors = (selectors, context) => {
+const localizeSelectors = (selectors, mode) => {
   const node = Tokenizer.parse(selectors);
   var resultingGlobal;
-  context.hasPureGlobals = false;
   node.nodes = node.nodes.map(n => {
     var nContext = {
-      global: context.global,
+      global: mode === "global",
       lastWasSpacing: true,
       hasLocals: false,
       explicit: false
@@ -136,46 +135,26 @@ const localizeSelectors = (selectors, context) => {
           ` (multiple selectors must result in the same mode for the rule)`
       );
     }
-    if (!nContext.hasLocals) {
-      context.hasPureGlobals = true;
-    }
     return n;
   });
-  context.global = resultingGlobal;
   return Tokenizer.stringify(node);
 };
 
-module.exports = postcss.plugin(plugin, (options = {}) => css => {
-  if (
-    options.mode &&
-    options.mode !== "global" &&
-    options.mode !== "local" &&
-    options.mode !== "pure"
-  ) {
-    throw Error(
-      'options.mode must be either "global", "local" or "pure" (default "local")'
-    );
-  }
-  var pureMode = options.mode === "pure";
-  var globalMode = options.mode === "global";
+const walkRules = (css, callback) => {
   css.walkRules(rule => {
-    if (rule.parent.type === "atrule" && /keyframes$/.test(rule.parent.name)) {
-      // ignore keyframe rules
-      return;
+    if (rule.parent.type !== "atrule" || !/keyframes$/.test(rule.parent.name)) {
+      callback(rule);
     }
-    var context = {
-      options: options,
-      global: globalMode,
-      hasPureGlobals: false
-    };
+  });
+};
+
+module.exports = postcss.plugin(plugin, (options = {}) => css => {
+  walkRules(css, rule => {
     try {
-      const newSelector = localizeSelectors(rule.selector, context);
-      if (pureMode && context.hasPureGlobals) {
-        throw Error(
-          `Selector "${rule.selector}" is not pure (pure selectors must contain at least one local class or id)`
-        );
-      }
-      rule.selector = newSelector;
+      rule.selector = localizeSelectors(
+        rule.selector,
+        options.mode === "global" ? "global" : "local"
+      );
     } catch (e) {
       throw rule.error(e.message);
     }

test/test.js

@@ -299,14 +299,6 @@ test("compile explict global attribute", () => {
   });
 });
 
-test("throw on invalid mode", () => {
-  return runError({
-    fixture: "",
-    options: { mode: "???" },
-    error: /"global", "local" or "pure"/
-  });
-});
-
 test("throw on inconsistent selector result", () => {
   return runError({
     fixture: ":global .foo, .bar {}",
@@ -356,46 +348,6 @@ test("throw on incorrect spacing with broad :local", () => {
   });
 });
 
-test("throw on not pure selector (global class)", () => {
-  return runError({
-    fixture: ":global(.foo) {}",
-    options: { mode: "pure" },
-    error: /":global\(\.foo\)" is not pure/
-  });
-});
-
-test("throw on not pure selector (with multiple 1)", () => {
-  return runError({
-    fixture: ".foo, :global(.bar) {}",
-    options: { mode: "pure" },
-    error: /".foo, :global\(\.bar\)" is not pure/
-  });
-});
-
-test("throw on not pure selector (with multiple 2)", () => {
-  return runError({
-    fixture: ":global(.bar), .foo {}",
-    options: { mode: "pure" },
-    error: /":global\(\.bar\), .foo" is not pure/
-  });
-});
-
-test("throw on not pure selector (element)", () => {
-  return runError({
-    fixture: "input {}",
-    options: { mode: "pure" },
-    error: /"input" is not pure/
-  });
-});
-
-test("throw on not pure selector (attribute)", () => {
-  return runError({
-    fixture: '[type="radio"] {}',
-    options: { mode: "pure" },
-    error: /"\[type="radio"\]" is not pure/
-  });
-});
-
 test("pass through global element", () => {
   return runCSS({
     fixture: "input {}",