Fixes #5 - handles non-boolean compatibility options.

Why:

* There was no way in CLI to set them as API allows doing it via a hash;
* ideally this should be moved into API as a way of setting
  compatibility options in a similar manner as optimization levels.

Author: jakubpawlowicz

History.md

@@ -4,6 +4,7 @@
 * Bumps clean-css dependency to 4.1.x.
 * Fixed issue [#1](https://github.com/jakubpawlowicz/clean-css-cli/issues/1) - option to remove inlined files.
 * Fixed issue [#2](https://github.com/jakubpawlowicz/clean-css-cli/issues/2) - glob matching source paths.
+* Fixed issue [#5](https://github.com/jakubpawlowicz/clean-css-cli/issues/5) - non-boolean compatibility options.
 * Fixed issue [#7](https://github.com/jakubpawlowicz/clean-css-cli/issues/7) - using CLI as a module.
 
 [4.0.12 / 2017-04-12](https://github.com/jakubpawlowicz/clean-css-cli/compare/v4.0.11...v4.0.12)

index.js

@@ -5,6 +5,8 @@ var CleanCSS = require('clean-css');
 var commands = require('commander');
 var glob = require('glob');
 
+var COMPATIBILITY_PATTERN = /([\w\.]+)=(\w+)/g;
+
 function cli(process, beforeMinifyCallback) {
   var packageConfig = fs.readFileSync(path.join(__dirname, 'package.json'));
   var buildVersion = JSON.parse(packageConfig).version;
@@ -211,6 +213,7 @@ function expandGlobs(paths) {
 function minify(process, beforeMinifyCallback, options, debugMode, removeInlinedFiles, data) {
   var cleanCss = new CleanCSS(options);
 
+  applyNonBooleanCompatibilityFlags(cleanCss, options.compatibility);
   beforeMinifyCallback(cleanCss);
   cleanCss.minify(data, function (errors, minified) {
     var mapFilename;
@@ -250,6 +253,33 @@ function minify(process, beforeMinifyCallback, options, debugMode, removeInlined
   });
 }
 
+function applyNonBooleanCompatibilityFlags(cleanCss, compatibility) {
+  var match;
+  var scope;
+  var parts;
+  var i, l;
+
+  if (!compatibility) {
+    return;
+  }
+
+  patternLoop:
+  while ((match = COMPATIBILITY_PATTERN.exec(compatibility)) !== null) {
+    scope = cleanCss.options.compatibility;
+    parts = match[1].split('.');
+
+    for (i = 0, l = parts.length - 1; i < l; i++) {
+      scope = scope[parts[i]];
+
+      if (!scope) {
+        continue patternLoop;
+      }
+    }
+
+    scope[parts.pop()] = match[2];
+  }
+}
+
 function outputFeedback(messages, isError) {
   var prefix = isError ? '\x1B[31mERROR\x1B[39m:' : 'WARNING:';
 

test/binary-test.js

@@ -394,6 +394,13 @@ vows.describe('cleancss')
       }
     })
   })
+  .addBatch({
+    'custom compatibility non-boolean options': pipedContext('.block-1{color:red}.block-2{color:red}', '--compatibility "selectors.mergeLimit=1,unknown.option=all" -O2', {
+      'keeps source intact': function (error, stdout) {
+        assert.equal(stdout, '.block-1{color:red}.block-2{color:red}');
+      }
+    })
+  })
   .addBatch({
     'rounding precision': {
       'default': pipedContext('div{width:0.10051px}', '', {