Now has PhantomJS alternative (slightly faster, though without JS execution)

Author: spookd

lib/domino-chains/extract.coffee

@@ -16,7 +16,9 @@ getElementAttributes = (el, ignore...) ->
 
 module.exports = exports =
   extractStylesheets: (window, document, options, next) ->
-    options.cout "Extracting stylesheets"
+    options.cout "Extracting stylesheets from page"
+    
+    media  = _.union(["", "all", "screen"], options.cssMedia)
     links  = document.querySelectorAll("link[rel='stylesheet']")
     sheets = []
 
@@ -36,6 +38,8 @@ module.exports = exports =
 
     urlParsed = url.parse(options.url)
 
+    console.log sheets
+
     sheets = sheets.filter (sheet) ->
       _.every options.ignoreSheets, (ignore) ->
         if ignore instanceof RegExp and ignore.test(sheet.href)
@@ -44,12 +48,11 @@ module.exports = exports =
 
         return sheet.href isnt ignore
 
-      if options.ignoreExternalSheets
-        if urlParsed.hostname isnt url.parse(sheet.href).hostname
-          options.cout "|> Ignoring stylesheet (#{sheet.href}) because external stylesheets are ignored"
-          return no
+      if options.ignoreExternalSheets and urlParsed.hostname isnt url.parse(sheet.href).hostname
+        options.cout "|> Ignoring stylesheet (#{sheet.href}) because external stylesheets are ignored"
+        return no
 
-        return true
+      return yes
 
     sheets = sheets.filter((sheet) -> return media.indexOf(sheet.media) isnt -1)
 

lib/domino-chains/return.coffee

@@ -3,9 +3,9 @@ module.exports = exports =
     return next(null) if not options.cssExpose or typeof options.cssExpose isnt "string"
 
     name = options.cssExpose
-    
+
     script = document.createElement("script")
-    script.innerHTML = "#{name} = #{JSON.stringify(stylesheets)};"
+    script.innerHTML = "#{name} = #{JSON.stringify(sheets)};"
     script.innerHTML = "var " + script.innerHTML if name.indexOf(".") < 0
     document.getElementsByTagName("body")[0].appendChild(script)
 

lib/inliner.coffee

@@ -1,35 +1,49 @@
 async    = require("async")
-_        = require("underscore")
+_        = require("lodash")
 Phantom  = require("./phantom")
 domino   = require("domino")
 request  = require("request")
 
 defaultOptions =
   # Core
-  log:             no
+  log:                  no
 
   # CSS
-  cssMedia:        ""
-  cssMinify:       no
-  cssOnly:         no
-  cssId:           no
-  cssExpose:       no
+  cssMedia:             ""
+  cssMinify:            no
+  cssOnly:              no
+  cssId:                no
+  cssExpose:            no
 
-  # Experimental
-  useDomino:       no
+
+  # Ignore
+  ignoreSheets:         []
+  ignoreSelectors:      []
+  ignoreExternalSheets: no
 
   # Miscellaneous
-  ignoreSheets:    []
-  ignoreSelectors: []
-  # ...
-  userAgent:       "CSS Inliner for node.js by Nicolai Persson"
+  userAgent:            "CSS Inliner for node.js by Nicolai Persson"
+  # !! Experimental
+  useDomino:            no
 
 module.exports = exports = (options = {}, cb) ->
   options = _.extend(defaultOptions, options)
 
+  # Validate options
   if not options?.url?
     cb("The URL option is required both for 'faking' (i.e. for providing the HTML) and loading an actual website")
-    return
+    return false
+
+  for attribute of options
+    switch attribute
+      when "log", "cssOnly", "ignoreExternalSheets"
+        if not _.isBoolean(options[attribute])
+          cb('Options type error: "log", "cssOnly", "ignoreExternalSheets" must be booleans.')   
+          return false
+      when "ignoreSheets", "ignoreSelectors"
+        if not _.isArray(options[attribute])
+          cb('Options type error: "ignoreSheets", "ignoreSelectors" must be arrays.')
+          return false
 
   if options.log
     console.log "Inlining #{options.url}", if options.html then "(fake URL with content)" else "" 
@@ -63,7 +77,7 @@ module.exports = exports = (options = {}, cb) ->
           return cb(error) if error
 
           if options.cssOnly
-            cb(null, usedCss)
+            cb(null, finalCSS)
           else
             returnChain = require("./phantom-chains/return")
 
@@ -86,7 +100,7 @@ module.exports = exports = (options = {}, cb) ->
         return cb(error) if error
 
         if options.cssOnly
-          cb(null, usedCss)
+          cb(null, finalCSS)
         else
           returnChain = require("./domino-chains/return")
 
@@ -100,4 +114,4 @@ module.exports = exports = (options = {}, cb) ->
         return cb(error) if error
         proceed(body)
     else
-      proceed(options.html)
+      proceed(options.html)

lib/phantom-chains/extract.coffee

@@ -1,11 +1,9 @@
-async    = require("async")
-request  = require("request")
-css      = require("css")
-CleanCSS = require("clean-css")
-_        = require("underscore")
-
-
-url = require("url")
+async       = require("async")
+request     = require("request")
+css         = require("css")
+CleanCSS    = require("clean-css")
+_           = require("lodash")
+url         = require("url")
 
 module.exports = exports =
   loadPageContent: (page, options, next) ->
@@ -47,6 +45,8 @@ module.exports = exports =
       stylesheets ||= []
 
       # Loop through ignore list
+      urlParsed = url.parse(options.url)
+
       stylesheets = stylesheets.filter (sheet) ->
         _.every options.ignoreSheets, (ignore) ->
           if ignore instanceof RegExp and ignore.test(sheet.href)
@@ -55,14 +55,11 @@ module.exports = exports =
 
           return sheet.href isnt ignore
 
-        if options.ignoreExternalSheets
-          urlParsed = url.parse(options.url)
-          sheetUrlParsed = url.parse(sheet.href)
-          if urlParsed.hostname isnt sheetUrlParsed.hostname
-            options.cout "|> Ignoring stylesheet (#{sheet.href}) because external stylesheets are ignored"
-            return no
-          
-          return true
+        if options.ignoreExternalSheets and urlParsed.hostname isnt url.parse(sheet.href).hostname
+          options.cout "|> Ignoring stylesheet (#{sheet.href}) because external stylesheets are ignored"
+          return no
+
+        return yes
 
       stylesheets = stylesheets.filter((sheet) -> return media.indexOf(sheet.media) isnt -1)
 
@@ -81,7 +78,7 @@ module.exports = exports =
     mapFn = (sheet, done) ->
       requestOptions =
         headers:
-          "User-Agent": "CSS Inliner for node.js"
+          "User-Agent": options.userAgent
 
       request sheet.href, requestOptions, (error, response, body) ->
         if error
@@ -152,5 +149,5 @@ module.exports = exports =
 
       next(null, stylesheets, finalCSS)
     catch e
-      next(error)
+      next(e.toString())
 

lib/phantom-chains/filter.coffee

@@ -1,6 +1,6 @@
 async   = require("async")
 css     = require("css")
-_       = require("underscore")
+_       = require("lodash")
 
 dePseudify = (->
   ignoredPseudos = [

lib/phantom-chains/return.coffee

@@ -4,14 +4,16 @@ module.exports = exports =
 
     evalFn = (name, stylesheets) ->
       script = document.createElement("script")
-      script.innerHTML = "var #{name} = #{JSON.stringify(stylesheets)};"
+      script.innerHTML = "#{name} = #{JSON.stringify(stylesheets)};"
+      script.innerHTML = "var " + script.innerHTML if name.indexOf(".") < 0
       return document.getElementsByTagName("body")[0].appendChild(script)
 
     resultFn = (error) ->
       next(error)
 
     page.evaluate(evalFn, resultFn, options.cssExpose, stylesheets)
 
+
   removeStylesheetsAndInjectUsedStyles: (page, options, stylesheets, finalCSS, next) ->
     evalFn = (options, stylesheets, finalCSS) ->
       links = document.querySelectorAll("link[rel='stylesheet']")

package.json

@@ -36,9 +36,9 @@
     "css": "~1.6.0",
     "domino": "^1.0.17",
     "generic-pool": "~2.0.4",
+    "lodash": "^2.4.1",
     "node-phantom-simple": "~1.2.0",
     "phantomjs": "~1.9.7-5",
-    "request": "~2.34.0",
-    "underscore": "~1.6.0"
+    "request": "~2.34.0"
   }
 }