Remove separator

Author: wooorm

lib/index.js

@@ -13,13 +13,10 @@ const emptyNodes = []
  *
  * @param {Array<Nodes> | Nodes} value
  *   Node or list of nodes to serialize.
- * @param {string | null | undefined} [separator='']
- *   Separator to use (default: `''`).
  * @returns {string}
  *   Result.
  */
-// To do next major: remove `separator`.
-export function toString(value, separator) {
+export function toString(value) {
   let index = -1
 
   if (!value || (!Array.isArray(value) && !value.type)) {
@@ -34,8 +31,8 @@ export function toString(value, separator) {
   const values = []
 
   while (++index < children.length) {
-    values[index] = toString(children[index], separator)
+    values[index] = toString(children[index])
   }
 
-  return values.join(separator || '')
+  return values.join('')
 }

readme.md

@@ -17,7 +17,7 @@
 *   [Install](#install)
 *   [Use](#use)
 *   [API](#api)
-    *   [`toString(value[, separator])`](#tostringvalue-separator)
+    *   [`toString(value)`](#tostringvalue)
 *   [Types](#types)
 *   [Compatibility](#compatibility)
 *   [Contribute](#contribute)
@@ -65,7 +65,7 @@ console.log(
     type: 'WordNode',
     children: [
       {type: 'TextNode', value: 'AT'},
-      {type: 'PunctuationNode', value: '&'},
+      {type: 'SymbolNode', value: '&'},
       {type: 'TextNode', value: 'T'}
     ]
   })
@@ -74,10 +74,10 @@ console.log(
 
 ## API
 
-This package exports the identifier [`toString`][tostring].
+This package exports the identifier [`toString`][api-to-string].
 There is no default export.
 
-### `toString(value[, separator])`
+### `toString(value)`
 
 Get the text content of a node or list of nodes.
 
@@ -86,10 +86,8 @@ if the given value is an array, serialize the nodes in it.
 
 ###### Parameters
 
-*   `node` ([`Node`][node] or `Array<Node>`)
-    — node to serialize.
-*   `separator` (`string`, default: `''`)
-    — value to delimit each item
+*   `node` ([`Array<Node>`][node] or `Node`)
+    — node or list of nodes to serialize
 
 ###### Returns
 
@@ -173,4 +171,4 @@ abide by its terms.
 
 [node]: https://github.com/syntax-tree/nlcst#nodes
 
-[tostring]: #tostringvalue-separator
+[api-to-string]: #tostringvalue

test.js

@@ -14,14 +14,14 @@ test('toString', async function (t) {
     assert.throws(function () {
       // @ts-expect-error: check how the runtime handles no node.
       toString()
-    })
+    }, /Expected node/)
   })
 
   await t.test('should throw when not given a node (#2)', async function () {
     assert.throws(function () {
       // @ts-expect-error: check how the runtime handles no `type`.
       toString({value: 'foo'})
-    })
+    }, /Expected node/)
   })
 
   await t.test('should support texts', async function () {
@@ -61,20 +61,6 @@ test('toString', async function (t) {
     )
   })
 
-  await t.test('should support separators', async function () {
-    assert.equal(
-      toString(
-        u('WordNode', [
-          u('TextNode', 'AT'),
-          u('SymbolNode', '&'),
-          u('TextNode', 'T')
-        ]),
-        ','
-      ),
-      'AT,&,T'
-    )
-  })
-
   await t.test('should support voids', async function () {
     assert.equal(
       toString(