remove empty paragraphs throughout the whole tree

Author: eush77

index.js

@@ -1,12 +1,16 @@
 'use strict';
 
+var remove = require('unist-util-remove');
 
-module.exports = function (ast) {
-  ast.children = ast.children.filter(function (node) {
-    return node.type != 'paragraph' || node.children.some(function (node) {
-      return node.type != 'text' || !/^\s*$/.test(node.value);
-    });
-  });
 
-  return ast;
+module.exports = function (ast) {
+  return remove(ast, { cascade: false }, isEmptyParagraph);
 };
+
+
+// Whether paragraph is empty or composed only of whitespace.
+function isEmptyParagraph (node) {
+  return node.type == 'paragraph' && node.children.every(function (node) {
+    return node.type == 'text' && /^\s*$/.test(node.value);
+  });
+}

package.json

@@ -28,7 +28,9 @@
     "util",
     "clean"
   ],
-  "dependencies": {},
+  "dependencies": {
+    "unist-util-remove": "^0.2.0"
+  },
   "devDependencies": {
     "tape": "^4.0.0",
     "unist-builder": "^1.0.0"

test/data/input.js

@@ -19,6 +19,21 @@ module.exports = function () {
       u('text', ' '),
       u('text', 'value')
     ]),
-    u('paragraph', [])
+    u('paragraph', []),
+    u('list', {
+      ordered: false,
+      start: null,
+      loose: false
+    }, [
+      u('listItem', { loose: false }, [
+        u('paragraph', [])
+      ]),
+      u('listItem', { loose: false }, [
+        u('paragraph', [
+          u('text', ' '),
+          u('text', ' ')
+        ])
+      ])
+    ])
   ]);
 };

test/data/output.js

@@ -12,6 +12,14 @@ module.exports = function () {
       u('text', 'second'),
       u('text', ' '),
       u('text', 'value')
+    ]),
+    u('list', {
+      ordered: false,
+      start: null,
+      loose: false
+    }, [
+      u('listItem', { loose: false }, []),
+      u('listItem', { loose: false }, [])
     ])
   ]);
 };