🐛 Preserve correct line numbers when front matter is stripped

sasstools · Apr 23, 2017 · 7d1d8b7 · 7d1d8b7
1 parent d81cc0e
commit 7d1d8b7
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/lib/groot.js b/lib/groot.js
@@ -15,7 +15,14 @@ module.exports = function (text, syntax, filename) {
 
   // if we're skipping front matter do it here, fall back to just our text in case it fails
   if (fm.test(text)) {
-    text = fm(text).body || text;
+    var withFrontMatter = fm(text);
+    // Replace front matter by empty lines, so that line numbers are preserved
+    var numEmptyLines = 2;
+    if (withFrontMatter.frontmatter !== '') {
+      numEmptyLines += withFrontMatter.frontmatter.split('\r\n|\r|\n').length;
+    }
+    var emptyLines = new Array(numEmptyLines + 1).join('\n');
+    text = emptyLines + withFrontMatter.body || text;
   }
 
   try {

diff --git a/tests/main.js b/tests/main.js
@@ -173,6 +173,13 @@ describe('sass lint', function () {
       });
     });
 
+    it('should preserve line numbers after parsing a file with front matter', function (done) {
+      lintFile('front-matter/front-matter.scss', function (data) {
+        assert.equal(6, data.messages[0].line);
+        done();
+      });
+    });
+
     // ==============================================================================
     //  Parse Errors should return as lint errors
     // ==============================================================================