Merge pull request #74 from gynzy/hidden-sourcemaps

feat: allow none as mapCommentType

README.md

@@ -49,6 +49,8 @@ Type: `string`
 
 If `'line'` is specified, `sourceMappingURL` will be written in a single-line comment (`//`).
 
+If `'none'` is specified, no reference to the source map is included in the source file.
+
 If anything else truthy is specified, `sourceMappingURL` will be written in a block comment (`/* */`).
 
 If no value is given, or a falsey value is given, will default to `'line'`.

lib/source-map.js

@@ -442,10 +442,15 @@ class SourceMap {
           sourceMappingUrl = `${dataUrlPrelude}${base64Content}`;
         }
 
-        if (sourceMap.mapCommentType === 'line') {
-          stream.write('//# sourceMappingURL=' + sourceMappingUrl + '\n');
-        } else {
-          stream.write('/*# sourceMappingURL=' + sourceMappingUrl + ' */\n');
+        switch (sourceMap.mapCommentType) {
+          case 'line':
+            stream.write('//# sourceMappingURL=' + sourceMappingUrl + '\n');
+            break;
+          case 'none':
+            break;
+          default:
+            stream.write('/*# sourceMappingURL=' + sourceMappingUrl + ' */\n');
+            break;
         }
 
         if (sourceMap.mapFile) {

test/test.js

@@ -223,6 +223,16 @@ describe('fast sourcemap concat', function() {
     });
   });
 
+  it("outputs no comments when 'mapCommentType' is 'none'", function() {
+    let FILE = 'tmp/mapcommenttype.css';
+    let s = new SourceMap({outputFile: FILE, mapCommentType: 'none'});
+    return s.end().then(function() {
+      let result = fs.readFileSync(FILE, 'utf-8');
+      let expected = "";
+      assert.equal(result, expected);
+    });
+  });
+
   it("should warn but tolerate broken sourcemap URL", function() {
     let s = new SourceMap({outputFile: 'tmp/with-broken-input-map.js', baseDir: path.join(__dirname, 'fixtures')});
     s._warn = sinon.spy();