Hi,

I tried your script but it doesn't work properly.
Doesn't work:

var cave = require('cave');
var fs = require('fs');
cave('deploy/css/style.css', fs.readFileSync('deploy/css/critical.css', { encoding: 'utf8' }), function() {
     console.log('OUTPUT');
 })

Error: Warning: Maximum call stack size exceeded Use --force to continue.

It seems to the css library has some problems to parse the css-file.

Works:
cat deploy/css/style.css | ./node_modules/cave/bin/cave deploy/css/critical.css > deploy/css/slim.css

How can i fix it? Can you help me?

Kind regards
Stephan

I'm keen to use this programatically and pair it with critical in a gulp workflow, but i have no idea where to start. Any chance you can add a snippet to the documentation?

I've also noticed that when passing 2 minified css files into cave, the output is an un-minified file - which is a little fiddly as it requires further processing.

cat .temp/production/assets/css/site.css | npm run cave .temp/production/assets/css/critical.css > main.css

(main.css is un-minified.)

Was the done callback removed? just curious

var diff = cave(pathToFile('style.css'), {
            css: criticalCss
});
console.log(diff) // outputs the css

but adding a function after the css object never gets called

var diff = cave(pathToFile('style.css'), {
            css: criticalCss
}, function(){
   console.log('done'); // never gets called
});

Do not remove the @font-face rule?