Fix multiple line endings in text

syntax-tree · Sep 2, 2020 · 15a21f1 · 15a21f1
1 parent 914a6b7
commit 15a21f1
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/lib/handlers/text.js b/lib/handlers/text.js
@@ -3,8 +3,10 @@
 module.exports = text
 
 var u = require('unist-builder')
-var trimLines = require('trim-lines')
 
 function text(h, node) {
-  return h.augment(node, u('text', trimLines(node.value)))
+  return h.augment(
+    node,
+    u('text', String(node.value).replace(/[ \t]*(\r?\n|\r)[ \t]*/g, '$1'))
+  )
 }
diff --git a/package.json b/package.json
@@ -35,7 +35,6 @@
     "@types/unist": "^2.0.3",
     "mdast-util-definitions": "^3.0.0",
     "mdurl": "^1.0.0",
-    "trim-lines": "^1.0.0",
     "unist-builder": "^2.0.0",
     "unist-util-generated": "^1.0.0",
     "unist-util-position": "^3.0.0",

diff --git a/test/text.js b/test/text.js
@@ -7,5 +7,11 @@ var to = require('..')
 test('Nodes', function (t) {
   t.deepEqual(to(u('text', 'alpha')), u('text', 'alpha'), 'should map `text`s')
 
+  t.deepEqual(
+    to(u('text', 'alpha \n \n bravo')),
+    u('text', 'alpha\n\nbravo'),
+    'should trim spaces and tabs around eols'
+  )
+
   t.end()
 })