From b7aa5ab33f40cefa957af3ec722ef3a710104c8d Mon Sep 17 00:00:00 2001
From: Jason Webster <jason@metalabdesign.com>
Date: Sat, 25 Apr 2015 22:56:59 -0700
Subject: [PATCH] Strip comments from config file before parsing

It's nice to allow teams to explain their rules or whatever in the config file.
This allows comments to happily exist in the config file.
---
 package.json            | 3 ++-
 src/commandline.coffee  | 9 +++++++--
 src/configfinder.coffee | 3 ++-
 3 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/package.json b/package.json
index 70c52d0d..ec6acecd 100644
--- a/package.json
+++ b/package.json
@@ -28,7 +28,8 @@
     "glob": "^4.0.0",
     "ignore": "^2.2.15",
     "optimist": "^0.6.1",
-    "resolve": "^0.6.3"
+    "resolve": "^0.6.3",
+    "strip-json-comments": "^1.0.2"
   },
   "devDependencies": {
     "vows": ">=0.6.0",
diff --git a/src/commandline.coffee b/src/commandline.coffee
index c163d227..0dce5e78 100644
--- a/src/commandline.coffee
+++ b/src/commandline.coffee
@@ -13,6 +13,7 @@ os   = require("os")
 glob = require("glob")
 optimist = require("optimist")
 ignore = require('ignore')
+stripComments = require('strip-json-comments')
 thisdir = path.dirname(fs.realpathSync(__filename))
 coffeelint = require(path.join(thisdir, "coffeelint"))
 configfinder = require(path.join(thisdir, "configfinder"))
@@ -59,6 +60,10 @@ lintSource = (source, config, literate = false) ->
     errorReport.lint("stdin", source, config, literate)
     return errorReport
 
+# Load a config file given a path/filename
+loadConfig = (path) ->
+    JSON.parse(stripComments(read(path)))
+
 # Get fallback configuration. With the -F flag found configs in standard places
 # will be used for each file being linted. Standard places are package.json or
 # coffeelint.json in a project's root folder or the user's home folder.
@@ -188,7 +193,7 @@ else
     config = null
     unless options.argv.noconfig
         if options.argv.f
-            config = JSON.parse read options.argv.f
+            config = loadConfig(options.argv.f)
 
             # If -f was specifying a package.json, extract the config
             if config.coffeelintConfig
@@ -196,7 +201,7 @@ else
 
         else if (process.env.COFFEELINT_CONFIG and
         fs.existsSync(process.env.COFFEELINT_CONFIG))
-            config = JSON.parse(read(process.env.COFFEELINT_CONFIG))
+            config = loadConfig(process.env.COFFEELINT_CONFIG)
 
     ruleLoader.loadRule(coffeelint, options.argv.rules) if options.argv.rules
 
diff --git a/src/configfinder.coffee b/src/configfinder.coffee
index 5ab353fa..1cb8228f 100644
--- a/src/configfinder.coffee
+++ b/src/configfinder.coffee
@@ -5,6 +5,7 @@ JSHint does.
 
 fs = require 'fs'
 path = require 'path'
+stripComments = require 'strip-json-comments'
 
 # Cache for findFile
 findFileResults = {}
@@ -31,7 +32,7 @@ loadNpmConfig = (dir) ->
 # Parse a JSON file gracefully.
 loadJSON = (filename) ->
     try
-        JSON.parse(fs.readFileSync(filename).toString())
+        JSON.parse(stripComments(fs.readFileSync(filename).toString()))
     catch e
         process.stderr.write "Could not load JSON file '#{filename}': #{e}"
         null