postcss-discard-unused 1.0.2 → 1.0.3

package/CHANGELOG.md

@@ -1,3 +1,8 @@
+# 1.0.3
+
+* Improved performance by reducing the amount of AST iterations.
+* Converted the codebase to ES6.
+
 # 1.0.2
 
 * Fixes an integration issue where the module would crash on `undefined`

package/dist/index.js

@@ -0,0 +1,89 @@
+'use strict';
+
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { 'default': obj }; }
+
+var _uniqs = require('uniqs');
+
+var _uniqs2 = _interopRequireDefault(_uniqs);
+
+var _postcss = require('postcss');
+
+var _postcss2 = _interopRequireDefault(_postcss);
+
+var _flatten = require('flatten');
+
+var _flatten2 = _interopRequireDefault(_flatten);
+
+var comma = _postcss.list.comma;
+var space = _postcss.list.space;
+
+var filterAtRule = function filterAtRule(css, properties, atrule) {
+    var atRules = [];
+    var values = [];
+    css.eachInside(function (node) {
+        if (node.type === 'decl' && properties.test(node.prop)) {
+            return comma(node.value).forEach(function (val) {
+                return values.push(space(val));
+            });
+        }
+        if (node.type === 'atrule') {
+            if (typeof atrule === 'string' && node.name === atrule) {
+                atRules.push(node);
+            } else if (atrule instanceof RegExp && atrule.test(node.name)) {
+                atRules.push(node);
+            }
+        }
+    });
+    values = (0, _uniqs2['default'])((0, _flatten2['default'])(values));
+    atRules.forEach(function (node) {
+        var hasAtRule = values.some(function (value) {
+            return value === node.params;
+        });
+        if (!hasAtRule) {
+            node.removeSelf();
+        }
+    });
+};
+
+var hasFont = function hasFont(fontFamily, cache) {
+    return comma(fontFamily).some(function (font) {
+        return cache.some(function (c) {
+            return ~c.indexOf(font);
+        });
+    });
+};
+
+module.exports = _postcss2['default'].plugin('postcss-discard-unused', function () {
+    return function (css) {
+        // fonts have slightly different logic
+        var atRules = [];
+        var values = [];
+        css.eachInside(function (node) {
+            if (node.type === 'decl' && node.parent.type === 'rule' && /font(|-family)/.test(node.prop)) {
+                return values.push(comma(node.value));
+            }
+            if (node.type === 'atrule' && node.name === 'font-face' && node.nodes) {
+                atRules.push(node);
+            }
+        });
+        values = (0, _uniqs2['default'])((0, _flatten2['default'])(values));
+        atRules.forEach(function (rule) {
+            var families = rule.nodes.filter(function (node) {
+                return node.prop === 'font-family';
+            });
+            // Discard the @font-face if it has no font-family
+            if (!families.length) {
+                return rule.removeSelf();
+            }
+            families.forEach(function (family) {
+                if (!hasFont(family.value, values)) {
+                    rule.removeSelf();
+                }
+            });
+        });
+
+        // keyframes & counter styles
+        filterAtRule(css, /list-style|system/, 'counter-style');
+        filterAtRule(css, /animation/, /keyframes/);
+    };
+});

package/package.json

@@ -1,20 +1,32 @@
 {
   "name": "postcss-discard-unused",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "Discard unused counter styles, keyframes and fonts.",
-  "main": "index.js",
+  "main": "dist/index.js",
+  "files": [
+    "bin",
+    "LICENSE-MIT",
+    "dist"
+  ],
   "scripts": {
-    "test": "tape test.js | tap-spec"
+    "prepublish": "babel src --out-dir dist --ignore /__tests__/",
+    "test-unformatted": "babel-tape-runner \"src/**/__tests__/*.js\"",
+    "test": "npm run test-unformatted | faucet"
   },
   "keywords": [
     "css",
+    "minify",
+    "optimise",
     "postcss",
-    "postcss-plugin"
+    "postcss-plugin",
+    "unused"
   ],
   "license": "MIT",
   "devDependencies": {
-    "tap-spec": "^2.2.2",
-    "tape": "^3.5.0"
+    "babel": "^5.8.21",
+    "babel-tape-runner": "^1.2.0",
+    "faucet": "0.0.1",
+    "tape": "^4.1.0"
   },
   "homepage": "https://github.com/ben-eb/postcss-discard-unused",
   "author": {
@@ -25,7 +37,7 @@
   "repository": "ben-eb/postcss-discard-unused",
   "dependencies": {
     "flatten": "0.0.1",
-    "postcss": "^4.1.11",
+    "postcss": "^4.1.16",
     "uniqs": "^2.0.0"
   }
 }

package/.npmignore

@@ -1,5 +0,0 @@
-.editorconfig
-.gitignore
-.jshintrc
-.travis.yml
-test.js

package/index.js

@@ -1,62 +0,0 @@
-'use strict';
-
-var uniqs = require('uniqs');
-var postcss = require('postcss');
-var flatten = require('flatten');
-var comma = postcss.list.comma;
-var space = postcss.list.space;
-
-function filterAtRule (css, properties, atrule) {
-    var cache = [];
-    css.eachDecl(properties, function (decl) {
-        comma(decl.value).forEach(function (val) {
-            cache.push(space(val));
-        });
-    });
-    cache = uniqs(flatten(cache));
-    css.eachAtRule(atrule, function (rule) {
-        var hasAtRule = cache.some(function (c) {
-            return c === rule.params;
-        });
-        if (!hasAtRule) {
-            rule.removeSelf();
-        }
-    });
-}
-
-module.exports = postcss.plugin('postcss-discard-unused', function () {
-    return function (css) {
-        // fonts have slightly different logic
-        var cache = [];
-        css.eachRule(function (rule) {
-            rule.eachDecl(/font(|-family)/, function (decl) {
-                cache.push(comma(decl.value));
-            });
-        });
-        cache = uniqs(flatten(cache));
-        css.eachAtRule('font-face', function (rule) {
-            if (!rule.nodes) { return; }
-            var fontFamilies = rule.nodes.filter(function (node) {
-                return node.prop === 'font-family';
-            });
-            // Discard the @font-face if it has no font-family
-            if (!fontFamilies.length) {
-                return rule.removeSelf();
-            }
-            fontFamilies.forEach(function (family) {
-                var hasFont = comma(family.value).some(function (font) {
-                    return cache.some(function (c) {
-                        return ~c.indexOf(font);
-                    });
-                });
-                if (!hasFont) {
-                    rule.removeSelf();
-                }
-            });
-        });
-
-        // keyframes & counter styles
-        filterAtRule(css, /list-style|system/, 'counter-style');
-        filterAtRule(css, /animation/, /keyframes/);
-    };
-});