stylehacks 2.3.1 → 2.3.2

package/CHANGELOG.md

@@ -1,3 +1,7 @@
+# 2.3.2
+
+* Resolves an issue where stylehacks would crash on CSS mixins.
+
 # 2.3.1
 
 * Upgraded postcss-selector-parser to `v2.0.0`.

package/dist/cli.js

@@ -17,12 +17,12 @@ var _minimist = require('minimist');
 
 var _minimist2 = _interopRequireDefault(_minimist);
 
+var _package = require('../package.json');
+
 var _ = require('./');
 
 var _2 = _interopRequireDefault(_);
 
-var _package = require('../package.json');
-
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
 var opts = (0, _minimist2.default)(process.argv.slice(2), {
@@ -38,34 +38,32 @@ var opts = (0, _minimist2.default)(process.argv.slice(2), {
 if (opts.version) {
     console.log(_package.version);
 } else {
-    (function () {
-        var file = opts._[0];
-        var out = opts._[1];
+    var file = opts._[0];
+    var out = opts._[1];
 
-        if (file === 'help' || opts.help) {
-            _fs2.default.createReadStream(__dirname + '/../usage.txt').pipe(process.stdout).on('close', function () {
-                return process.exit(1);
-            });
-        } else {
-            (0, _readFileStdin2.default)(file, function (err, buf) {
-                if (err) {
-                    throw err;
-                }
-                if (file) {
-                    opts.from = file;
+    if (file === 'help' || opts.help) {
+        _fs2.default.createReadStream(__dirname + '/../usage.txt').pipe(process.stdout).on('close', function () {
+            return process.exit(1);
+        });
+    } else {
+        (0, _readFileStdin2.default)(file, function (err, buf) {
+            if (err) {
+                throw err;
+            }
+            if (file) {
+                opts.from = file;
+            }
+            if (out) {
+                opts.to = out;
+            }
+            _2.default.process(String(buf), opts).then(function (result) {
+                if (result.warnings().length) {
+                    process.exit(1);
                 }
-                if (out) {
-                    opts.to = out;
+                if (!opts.lint) {
+                    (0, _writeFileStdout2.default)(result.css);
                 }
-                _2.default.process(String(buf), opts).then(function (result) {
-                    if (result.warnings().length) {
-                        process.exit(1);
-                    }
-                    if (!opts.lint) {
-                        (0, _writeFileStdout2.default)(result.css);
-                    }
-                });
             });
-        }
-    })();
+        });
+    }
 }

package/dist/dictionary/browsers.js

@@ -0,0 +1,9 @@
+'use strict';
+
+exports.__esModule = true;
+var FF_2 = exports.FF_2 = 'firefox 2';
+var IE_5_5 = exports.IE_5_5 = 'ie 5.5';
+var IE_6 = exports.IE_6 = 'ie 6';
+var IE_7 = exports.IE_7 = 'ie 7';
+var IE_8 = exports.IE_8 = 'ie 8';
+var OP_9 = exports.OP_9 = 'opera 9';

package/dist/dictionary/identifiers.js

@@ -0,0 +1,7 @@
+'use strict';
+
+exports.__esModule = true;
+var MEDIA_QUERY = exports.MEDIA_QUERY = 'media query';
+var PROPERTY = exports.PROPERTY = 'property';
+var SELECTOR = exports.SELECTOR = 'selector';
+var VALUE = exports.VALUE = 'value';

package/dist/dictionary/postcss.js

@@ -0,0 +1,6 @@
+'use strict';
+
+exports.__esModule = true;
+var ATRULE = exports.ATRULE = 'atrule';
+var DECL = exports.DECL = 'decl';
+var RULE = exports.RULE = 'rule';

package/dist/dictionary/tags.js

@@ -0,0 +1,5 @@
+'use strict';
+
+exports.__esModule = true;
+var BODY = exports.BODY = 'body';
+var HTML = exports.HTML = 'html';

package/dist/exists.js

@@ -6,4 +6,4 @@ function exists(selector, index, value) {
     var node = selector.at(index);
     return node && node.value === value;
 }
-module.exports = exports['default'];
+module.exports = exports["default"];

package/dist/formatter.js

@@ -2,14 +2,14 @@
 
 exports.__esModule = true;
 
-var _chalk = require('chalk');
-
-var _chalk2 = _interopRequireDefault(_chalk);
-
 var _path = require('path');
 
 var _path2 = _interopRequireDefault(_path);
 
+var _chalk = require('chalk');
+
+var _chalk2 = _interopRequireDefault(_chalk);
+
 var _textTable = require('text-table');
 
 var _textTable2 = _interopRequireDefault(_textTable);

package/dist/index.js

@@ -70,26 +70,34 @@ var plugins = [_bodyEmpty2.default, _htmlCombinatorCommentBody2.default, _htmlFi
 
 
 var stylehacks = _postcss2.default.plugin('stylehacks', function () {
-    var opts = arguments.length <= 0 || arguments[0] === undefined ? {} : arguments[0];
+    var opts = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
 
     var b = opts.browsers;
     var browsers = b instanceof Array ? b : (0, _browserslist2.default)(b);
 
     return function (css, result) {
-        plugins.forEach(function (Plugin) {
-            var hack = new Plugin(css, result);
+        var processors = plugins.reduce(function (list, Plugin) {
+            var hack = new Plugin(result);
             var applied = browsers.some(function (browser) {
                 return hack.targets.some(function (target) {
                     return browser === target;
                 });
             });
             if (applied) {
-                return;
+                return list;
             }
-            if (opts.lint) {
-                return hack.detectAndWarn();
-            }
-            return hack.detectAndResolve();
+            return [].concat(list, [hack]);
+        }, []);
+        css.walk(function (node) {
+            processors.forEach(function (proc) {
+                if (!~proc.nodeTypes.indexOf(node.type)) {
+                    return;
+                }
+                if (opts.lint) {
+                    return proc.detectAndWarn(node);
+                }
+                return proc.detectAndResolve(node);
+            });
         });
     };
 });
@@ -104,7 +112,7 @@ stylehacks.detect = function (node) {
 };
 
 stylehacks.process = function (css) {
-    var opts = arguments.length <= 1 || arguments[1] === undefined ? {} : arguments[1];
+    var opts = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
 
     opts.reporter = {};
     opts.reporter.formatter = _formatter2.default;

package/dist/isMixin.js

@@ -0,0 +1,14 @@
+'use strict';
+
+exports.__esModule = true;
+exports.default = isMixin;
+function isMixin(node) {
+    var selector = node.selector;
+    // If the selector ends with a ':' it is likely a part of a custom mixin.
+
+    if (!selector || selector[selector.length - 1] === ':') {
+        return true;
+    }
+    return false;
+}
+module.exports = exports['default'];

package/dist/plugin.js

@@ -2,53 +2,58 @@
 
 exports.__esModule = true;
 
+var _extends = Object.assign || function (target) { for (var i = 1; i < arguments.length; i++) { var source = arguments[i]; for (var key in source) { if (Object.prototype.hasOwnProperty.call(source, key)) { target[key] = source[key]; } } } return target; };
+
+exports.default = plugin;
+
 function _classCallCheck(instance, Constructor) { if (!(instance instanceof Constructor)) { throw new TypeError("Cannot call a class as a function"); } }
 
-function plugin(targets, nodeTypes, _detect) {
+function plugin(targets, nodeTypes, detect) {
     var Plugin = function () {
-        function Plugin(css, result) {
+        function Plugin(result) {
             _classCallCheck(this, Plugin);
 
             this.nodes = [];
-            this.css = css;
             this.result = result;
             this.targets = targets;
             this.nodeTypes = nodeTypes;
         }
 
         Plugin.prototype.push = function push(node, metadata) {
-            metadata.message = "Bad " + metadata.identifier + ": " + metadata.hack;
-            metadata.browsers = this.targets;
-            node._stylehacks = metadata;
+            node._stylehacks = _extends({}, metadata, {
+                message: "Bad " + metadata.identifier + ": " + metadata.hack,
+                browsers: this.targets
+            });
             this.nodes.push(node);
         };
 
         Plugin.prototype.any = function any(node) {
-            var hasHack = false;
             if (~this.nodeTypes.indexOf(node.type)) {
-                _detect.apply(this, arguments);
-                if (node._stylehacks) {
-                    hasHack = true;
-                }
+                detect.apply(this, arguments);
+                return !!node._stylehacks;
             }
-            return hasHack;
-        };
-
-        Plugin.prototype.detect = function detect() {
-            this.css.walk(function (node) {
-                if (~this.nodeTypes.indexOf(node.type)) {
-                    _detect.apply(this, arguments);
-                }
-            }.bind(this));
+            return false;
         };
 
         Plugin.prototype.detectAndResolve = function detectAndResolve() {
-            this.detect();
+            this.nodes = [];
+
+            for (var _len = arguments.length, args = Array(_len), _key = 0; _key < _len; _key++) {
+                args[_key] = arguments[_key];
+            }
+
+            detect.apply(this, args);
             return this.resolve();
         };
 
         Plugin.prototype.detectAndWarn = function detectAndWarn() {
-            this.detect();
+            this.nodes = [];
+
+            for (var _len2 = arguments.length, args = Array(_len2), _key2 = 0; _key2 < _len2; _key2++) {
+                args[_key2] = arguments[_key2];
+            }
+
+            detect.apply(this, args);
             return this.warn();
         };
 
@@ -62,11 +67,11 @@ function plugin(targets, nodeTypes, _detect) {
             var _this = this;
 
             return this.nodes.forEach(function (node) {
-                var _node$_stylehacks = node._stylehacks;
-                var message = _node$_stylehacks.message;
-                var browsers = _node$_stylehacks.browsers;
-                var identifier = _node$_stylehacks.identifier;
-                var hack = _node$_stylehacks.hack;
+                var _node$_stylehacks = node._stylehacks,
+                    message = _node$_stylehacks.message,
+                    browsers = _node$_stylehacks.browsers,
+                    identifier = _node$_stylehacks.identifier,
+                    hack = _node$_stylehacks.hack;
 
                 return node.warn(_this.result, message, { browsers: browsers, identifier: identifier, hack: hack });
             });
@@ -77,5 +82,4 @@ function plugin(targets, nodeTypes, _detect) {
 
     return Plugin;
 }
-exports.default = plugin;
-module.exports = exports['default'];
+module.exports = exports["default"];

package/dist/plugins/bodyEmpty.js

@@ -2,28 +2,38 @@
 
 exports.__esModule = true;
 
+var _postcssSelectorParser = require('postcss-selector-parser');
+
+var _postcssSelectorParser2 = _interopRequireDefault(_postcssSelectorParser);
+
 var _exists = require('../exists');
 
 var _exists2 = _interopRequireDefault(_exists);
 
+var _isMixin = require('../isMixin');
+
+var _isMixin2 = _interopRequireDefault(_isMixin);
+
 var _plugin = require('../plugin');
 
 var _plugin2 = _interopRequireDefault(_plugin);
 
-var _postcssSelectorParser = require('postcss-selector-parser');
+var _browsers = require('../dictionary/browsers');
 
-var _postcssSelectorParser2 = _interopRequireDefault(_postcssSelectorParser);
+var _identifiers = require('../dictionary/identifiers');
 
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+var _postcss = require('../dictionary/postcss');
 
-var targets = ['firefox 2'];
+var _tags = require('../dictionary/tags');
+
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
 function analyse(ctx, rule) {
     return function (selectors) {
         selectors.each(function (selector) {
-            if ((0, _exists2.default)(selector, 0, 'body') && (0, _exists2.default)(selector, 1, ':empty') && (0, _exists2.default)(selector, 2, ' ') && selector.at(3)) {
+            if ((0, _exists2.default)(selector, 0, _tags.BODY) && (0, _exists2.default)(selector, 1, ':empty') && (0, _exists2.default)(selector, 2, ' ') && selector.at(3)) {
                 ctx.push(rule, {
-                    identifier: 'selector',
+                    identifier: _identifiers.SELECTOR,
                     hack: selector.toString()
                 });
             }
@@ -31,9 +41,10 @@ function analyse(ctx, rule) {
     };
 }
 
-exports.default = (0, _plugin2.default)(targets, ['rule'], function (rule) {
-    if (rule.selector) {
-        (0, _postcssSelectorParser2.default)(analyse(this, rule)).process(rule.selector);
+exports.default = (0, _plugin2.default)([_browsers.FF_2], [_postcss.RULE], function (rule) {
+    if ((0, _isMixin2.default)(rule)) {
+        return;
     }
+    (0, _postcssSelectorParser2.default)(analyse(this, rule)).process(rule.selector);
 });
 module.exports = exports['default'];

package/dist/plugins/htmlCombinatorCommentBody.js

@@ -2,28 +2,38 @@
 
 exports.__esModule = true;
 
+var _postcssSelectorParser = require('postcss-selector-parser');
+
+var _postcssSelectorParser2 = _interopRequireDefault(_postcssSelectorParser);
+
 var _exists = require('../exists');
 
 var _exists2 = _interopRequireDefault(_exists);
 
+var _isMixin = require('../isMixin');
+
+var _isMixin2 = _interopRequireDefault(_isMixin);
+
 var _plugin = require('../plugin');
 
 var _plugin2 = _interopRequireDefault(_plugin);
 
-var _postcssSelectorParser = require('postcss-selector-parser');
+var _browsers = require('../dictionary/browsers');
 
-var _postcssSelectorParser2 = _interopRequireDefault(_postcssSelectorParser);
+var _identifiers = require('../dictionary/identifiers');
 
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+var _postcss = require('../dictionary/postcss');
 
-var targets = ['ie 7', 'ie 6', 'ie 5.5'];
+var _tags = require('../dictionary/tags');
+
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
 function analyse(ctx, rule) {
     return function (selectors) {
         selectors.each(function (selector) {
-            if ((0, _exists2.default)(selector, 0, 'html') && ((0, _exists2.default)(selector, 1, '>') || (0, _exists2.default)(selector, 1, '~')) && selector.at(2) && selector.at(2).type === 'comment' && (0, _exists2.default)(selector, 3, ' ') && (0, _exists2.default)(selector, 4, 'body') && (0, _exists2.default)(selector, 5, ' ') && selector.at(6)) {
+            if ((0, _exists2.default)(selector, 0, _tags.HTML) && ((0, _exists2.default)(selector, 1, '>') || (0, _exists2.default)(selector, 1, '~')) && selector.at(2) && selector.at(2).type === 'comment' && (0, _exists2.default)(selector, 3, ' ') && (0, _exists2.default)(selector, 4, _tags.BODY) && (0, _exists2.default)(selector, 5, ' ') && selector.at(6)) {
                 ctx.push(rule, {
-                    identifier: 'selector',
+                    identifier: _identifiers.SELECTOR,
                     hack: selector.toString()
                 });
             }
@@ -31,8 +41,11 @@ function analyse(ctx, rule) {
     };
 }
 
-exports.default = (0, _plugin2.default)(targets, ['rule'], function (rule) {
-    if (rule.selector && rule.raws.selector && rule.raws.selector.raw) {
+exports.default = (0, _plugin2.default)([_browsers.IE_5_5, _browsers.IE_6, _browsers.IE_7], [_postcss.RULE], function (rule) {
+    if ((0, _isMixin2.default)(rule)) {
+        return;
+    }
+    if (rule.raws.selector && rule.raws.selector.raw) {
         (0, _postcssSelectorParser2.default)(analyse(this, rule)).process(rule.raws.selector.raw);
     }
 });

package/dist/plugins/htmlFirstChild.js

@@ -2,28 +2,38 @@
 
 exports.__esModule = true;
 
+var _postcssSelectorParser = require('postcss-selector-parser');
+
+var _postcssSelectorParser2 = _interopRequireDefault(_postcssSelectorParser);
+
 var _exists = require('../exists');
 
 var _exists2 = _interopRequireDefault(_exists);
 
+var _isMixin = require('../isMixin');
+
+var _isMixin2 = _interopRequireDefault(_isMixin);
+
 var _plugin = require('../plugin');
 
 var _plugin2 = _interopRequireDefault(_plugin);
 
-var _postcssSelectorParser = require('postcss-selector-parser');
+var _browsers = require('../dictionary/browsers');
 
-var _postcssSelectorParser2 = _interopRequireDefault(_postcssSelectorParser);
+var _identifiers = require('../dictionary/identifiers');
 
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+var _postcss = require('../dictionary/postcss');
 
-var targets = ['opera 9'];
+var _tags = require('../dictionary/tags');
+
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
 function analyse(ctx, rule) {
     return function (selectors) {
         selectors.each(function (selector) {
-            if ((0, _exists2.default)(selector, 0, 'html') && (0, _exists2.default)(selector, 1, ':first-child') && (0, _exists2.default)(selector, 2, ' ') && selector.at(3)) {
+            if ((0, _exists2.default)(selector, 0, _tags.HTML) && (0, _exists2.default)(selector, 1, ':first-child') && (0, _exists2.default)(selector, 2, ' ') && selector.at(3)) {
                 ctx.push(rule, {
-                    identifier: 'selector',
+                    identifier: _identifiers.SELECTOR,
                     hack: selector.toString()
                 });
             }
@@ -31,9 +41,10 @@ function analyse(ctx, rule) {
     };
 }
 
-exports.default = (0, _plugin2.default)(targets, ['rule'], function (rule) {
-    if (rule.selector) {
-        (0, _postcssSelectorParser2.default)(analyse(this, rule)).process(rule.selector);
+exports.default = (0, _plugin2.default)([_browsers.OP_9], [_postcss.RULE], function (rule) {
+    if ((0, _isMixin2.default)(rule)) {
+        return;
     }
+    (0, _postcssSelectorParser2.default)(analyse(this, rule)).process(rule.selector);
 });
 module.exports = exports['default'];

package/dist/plugins/important.js

@@ -6,17 +6,19 @@ var _plugin = require('../plugin');
 
 var _plugin2 = _interopRequireDefault(_plugin);
 
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+var _browsers = require('../dictionary/browsers');
+
+var _postcss = require('../dictionary/postcss');
 
-var targets = ['ie 5.5', 'ie 6', 'ie 7'];
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
-exports.default = (0, _plugin2.default)(targets, ['decl'], function (decl) {
+exports.default = (0, _plugin2.default)([_browsers.IE_5_5, _browsers.IE_6, _browsers.IE_7], [_postcss.DECL], function (decl) {
     var match = decl.value.match(/!\w/);
     if (match) {
-        var text = decl.value.substr(match.index, decl.value.length - 1);
+        var hack = decl.value.substr(match.index, decl.value.length - 1);
         this.push(decl, {
             identifier: '!important',
-            hack: text
+            hack: hack
         });
     }
 });

package/dist/plugins/leadingStar.js

@@ -2,60 +2,57 @@
 
 exports.__esModule = true;
 
-var _typeof = typeof Symbol === "function" && typeof Symbol.iterator === "symbol" ? function (obj) { return typeof obj; } : function (obj) { return obj && typeof Symbol === "function" && obj.constructor === Symbol ? "symbol" : typeof obj; };
-
 var _plugin = require('../plugin');
 
 var _plugin2 = _interopRequireDefault(_plugin);
 
+var _browsers = require('../dictionary/browsers');
+
+var _identifiers = require('../dictionary/identifiers');
+
+var _postcss = require('../dictionary/postcss');
+
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
-var targets = ['ie 5.5', 'ie 6', 'ie 7'];
 var hacks = '!_$_&_*_)_=_%_+_,_._/_`_]_#_~_?_:_|'.split('_');
 
-exports.default = (0, _plugin2.default)(targets, ['decl', 'atrule'], function (node) {
+exports.default = (0, _plugin2.default)([_browsers.IE_5_5, _browsers.IE_6, _browsers.IE_7], [_postcss.ATRULE, _postcss.DECL], function (node) {
     var _this = this;
 
-    if (node.type === 'decl') {
-        var _ret = function () {
-            // some values are not picked up by before, so ensure they are
-            // at the beginning of the value
-            hacks.some(function (hack) {
-                if (!node.prop.indexOf(hack)) {
-                    _this.push(node, {
-                        identifier: 'property',
-                        hack: node.prop
-                    });
-                    return true;
-                }
-            });
-            var before = node.raws.before;
-
-            if (!before) {
-                return {
-                    v: void 0
-                };
+    if (node.type === _postcss.DECL) {
+        // some values are not picked up by before, so ensure they are
+        // at the beginning of the value
+        hacks.some(function (hack) {
+            if (!node.prop.indexOf(hack)) {
+                _this.push(node, {
+                    identifier: _identifiers.PROPERTY,
+                    hack: node.prop
+                });
+                return true;
             }
-            hacks.some(function (hack) {
-                if (~before.indexOf(hack)) {
-                    _this.push(node, {
-                        identifier: 'property',
-                        hack: '' + before.trim() + node.prop
-                    });
-                    return true;
-                }
-            });
-        }();
+        });
+        var before = node.raws.before;
 
-        if ((typeof _ret === 'undefined' ? 'undefined' : _typeof(_ret)) === "object") return _ret.v;
-    } else if (node.type === 'atrule') {
+        if (!before) {
+            return;
+        }
+        hacks.some(function (hack) {
+            if (~before.indexOf(hack)) {
+                _this.push(node, {
+                    identifier: _identifiers.PROPERTY,
+                    hack: '' + before.trim() + node.prop
+                });
+                return true;
+            }
+        });
+    } else {
         // test for the @property: value; hack
         var name = node.name;
 
         var len = name.length - 1;
         if (name.lastIndexOf(':') === len) {
             this.push(node, {
-                identifier: 'property',
+                identifier: _identifiers.PROPERTY,
                 hack: '@' + name.substr(0, len)
             });
         }

package/dist/plugins/leadingUnderscore.js

@@ -6,18 +6,23 @@ var _plugin = require('../plugin');
 
 var _plugin2 = _interopRequireDefault(_plugin);
 
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+var _browsers = require('../dictionary/browsers');
+
+var _identifiers = require('../dictionary/identifiers');
 
-var targets = ['ie 6'];
+var _postcss = require('../dictionary/postcss');
 
-exports.default = (0, _plugin2.default)(targets, ['decl'], function (decl) {
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+
+exports.default = (0, _plugin2.default)([_browsers.IE_6], [_postcss.DECL], function (decl) {
     var before = decl.raws.before;
+
     if (!before) {
         return;
     }
     if (~before.indexOf('_') || ~before.indexOf('-')) {
         this.push(decl, {
-            identifier: 'property',
+            identifier: _identifiers.PROPERTY,
             hack: '' + before.trim() + decl.prop
         });
     }

package/dist/plugins/mediaSlash0.js

@@ -6,15 +6,19 @@ var _plugin = require('../plugin');
 
 var _plugin2 = _interopRequireDefault(_plugin);
 
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+var _browsers = require('../dictionary/browsers');
+
+var _identifiers = require('../dictionary/identifiers');
 
-var targets = ['ie 8'];
+var _postcss = require('../dictionary/postcss');
+
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
-exports.default = (0, _plugin2.default)(targets, ['atrule'], function (rule) {
+exports.default = (0, _plugin2.default)([_browsers.IE_8], [_postcss.ATRULE], function (rule) {
     var params = rule.params.trim();
     if (params === '\\0screen') {
         this.push(rule, {
-            identifier: 'media query',
+            identifier: _identifiers.MEDIA_QUERY,
             hack: params
         });
     }

package/dist/plugins/mediaSlash9.js

@@ -6,15 +6,19 @@ var _plugin = require('../plugin');
 
 var _plugin2 = _interopRequireDefault(_plugin);
 
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+var _browsers = require('../dictionary/browsers');
+
+var _identifiers = require('../dictionary/identifiers');
 
-var targets = ['ie 5.5', 'ie 6', 'ie 7'];
+var _postcss = require('../dictionary/postcss');
+
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
-exports.default = (0, _plugin2.default)(targets, ['atrule'], function (rule) {
+exports.default = (0, _plugin2.default)([_browsers.IE_5_5, _browsers.IE_6, _browsers.IE_7], [_postcss.ATRULE], function (rule) {
     var params = rule.params.trim();
     if (params === 'screen\\9') {
         this.push(rule, {
-            identifier: 'media query',
+            identifier: _identifiers.MEDIA_QUERY,
             hack: params
         });
     }

package/dist/plugins/slash9.js

@@ -6,15 +6,19 @@ var _plugin = require('../plugin');
 
 var _plugin2 = _interopRequireDefault(_plugin);
 
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+var _browsers = require('../dictionary/browsers');
+
+var _identifiers = require('../dictionary/identifiers');
 
-var targets = ['ie 8', 'ie 7', 'ie 6'];
+var _postcss = require('../dictionary/postcss');
+
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
-exports.default = (0, _plugin2.default)(targets, ['decl'], function (decl) {
+exports.default = (0, _plugin2.default)([_browsers.IE_6, _browsers.IE_7, _browsers.IE_8], [_postcss.DECL], function (decl) {
     var v = decl.value;
     if (v && v.length > 2 && v.indexOf('\\9') === v.length - 2) {
         this.push(decl, {
-            identifier: 'value',
+            identifier: _identifiers.VALUE,
             hack: v
         });
     }

package/dist/plugins/starHtml.js

@@ -2,28 +2,38 @@
 
 exports.__esModule = true;
 
+var _postcssSelectorParser = require('postcss-selector-parser');
+
+var _postcssSelectorParser2 = _interopRequireDefault(_postcssSelectorParser);
+
 var _exists = require('../exists');
 
 var _exists2 = _interopRequireDefault(_exists);
 
+var _isMixin = require('../isMixin');
+
+var _isMixin2 = _interopRequireDefault(_isMixin);
+
 var _plugin = require('../plugin');
 
 var _plugin2 = _interopRequireDefault(_plugin);
 
-var _postcssSelectorParser = require('postcss-selector-parser');
+var _browsers = require('../dictionary/browsers');
 
-var _postcssSelectorParser2 = _interopRequireDefault(_postcssSelectorParser);
+var _identifiers = require('../dictionary/identifiers');
 
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+var _postcss = require('../dictionary/postcss');
 
-var targets = ['ie 6', 'ie 5.5'];
+var _tags = require('../dictionary/tags');
+
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
 function analyse(ctx, rule) {
     return function (selectors) {
         selectors.each(function (selector) {
-            if ((0, _exists2.default)(selector, 0, '*') && (0, _exists2.default)(selector, 1, ' ') && (0, _exists2.default)(selector, 2, 'html') && (0, _exists2.default)(selector, 3, ' ') && selector.at(4)) {
+            if ((0, _exists2.default)(selector, 0, '*') && (0, _exists2.default)(selector, 1, ' ') && (0, _exists2.default)(selector, 2, _tags.HTML) && (0, _exists2.default)(selector, 3, ' ') && selector.at(4)) {
                 ctx.push(rule, {
-                    identifier: 'selector',
+                    identifier: _identifiers.SELECTOR,
                     hack: selector.toString()
                 });
             }
@@ -31,9 +41,10 @@ function analyse(ctx, rule) {
     };
 }
 
-exports.default = (0, _plugin2.default)(targets, ['rule'], function (rule) {
-    if (rule.selector) {
-        (0, _postcssSelectorParser2.default)(analyse(this, rule)).process(rule.selector);
+exports.default = (0, _plugin2.default)([_browsers.IE_5_5, _browsers.IE_6], [_postcss.RULE], function (rule) {
+    if ((0, _isMixin2.default)(rule)) {
+        return;
     }
+    (0, _postcssSelectorParser2.default)(analyse(this, rule)).process(rule.selector);
 });
 module.exports = exports['default'];

package/dist/plugins/trailingSlashComma.js

@@ -6,21 +6,30 @@ var _plugin = require('../plugin');
 
 var _plugin2 = _interopRequireDefault(_plugin);
 
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+var _isMixin = require('../isMixin');
+
+var _isMixin2 = _interopRequireDefault(_isMixin);
+
+var _browsers = require('../dictionary/browsers');
+
+var _identifiers = require('../dictionary/identifiers');
 
-var targets = ['ie 7', 'ie 6', 'ie 5.5'];
+var _postcss = require('../dictionary/postcss');
 
-exports.default = (0, _plugin2.default)(targets, ['rule'], function (rule) {
-    if (rule.selector) {
-        var selector = rule.selector;
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+
+exports.default = (0, _plugin2.default)([_browsers.IE_5_5, _browsers.IE_6, _browsers.IE_7], [_postcss.RULE], function (rule) {
+    if ((0, _isMixin2.default)(rule)) {
+        return;
+    }
+    var selector = rule.selector;
 
-        var trim = selector.trim();
-        if (trim.lastIndexOf(',') === selector.length - 1 || trim.lastIndexOf('\\') === selector.length - 1) {
-            this.push(rule, {
-                identifier: 'selector',
-                hack: selector
-            });
-        }
+    var trim = selector.trim();
+    if (trim.lastIndexOf(',') === selector.length - 1 || trim.lastIndexOf('\\') === selector.length - 1) {
+        this.push(rule, {
+            identifier: _identifiers.SELECTOR,
+            hack: selector
+        });
     }
 });
 module.exports = exports['default'];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "stylehacks",
-  "version": "2.3.1",
+  "version": "2.3.2",
   "description": "Detect/remove browser hacks from CSS files.",
   "main": "dist/index.js",
   "bin": {
@@ -14,7 +14,9 @@
   "scripts": {
     "pretest": "eslint src",
     "prepublish": "del-cli dist && BABEL_ENV=publish babel src --out-dir dist --ignore /__tests__/",
-    "test": "ava src/__tests__"
+    "report": "nyc report --reporter=html",
+    "test": "nyc ava src/__tests__",
+    "test-012": "nyc ava src/__tests__"
   },
   "keywords": [
     "browsers",
@@ -28,16 +30,20 @@
   ],
   "license": "MIT",
   "devDependencies": {
-    "ava": "^0.14.0",
+    "ava": "^0.17.0",
     "babel-cli": "^6.5.1",
     "babel-core": "^6.5.2",
     "babel-plugin-add-module-exports": "^0.2.0",
     "babel-preset-es2015": "^6.5.0",
     "babel-preset-es2015-loose": "^7.0.0",
     "babel-preset-stage-0": "^6.5.0",
+    "babel-register": "^6.9.0",
     "del-cli": "^0.2.0",
-    "eslint": "^2.2.0",
-    "eslint-config-cssnano": "^2.0.0"
+    "eslint": "^3.0.0",
+    "eslint-config-cssnano": "^3.0.0",
+    "eslint-plugin-babel": "^3.3.0",
+    "eslint-plugin-import": "^2.0.1",
+    "nyc": "^10.0.0"
   },
   "homepage": "https://github.com/ben-eb/stylehacks",
   "author": {
@@ -63,6 +69,13 @@
     "extends": "cssnano"
   },
   "ava": {
-    "require": "babel-core/register"
+    "require": "babel-register"
+  },
+  "nyc": {
+    "exclude": [
+      "dist",
+      "node_modules",
+      "src/__tests__"
+    ]
   }
 }