Refactor code-style

Author: wooorm

Diff for: index.js

@@ -1,16 +1,27 @@
-'use strict';
+'use strict'
 
-var assign = require('object-assign');
+var assign = require('object-assign')
 
+module.exports = u
 
-module.exports = function u (type, props, value) {
-  if (value == null && (typeof props != 'object' || Array.isArray(props))) {
-    value = props;
-    props = {};
+function u(type, props, value) {
+  var node
+
+  if (
+    (value === null || value === undefined) &&
+    (typeof props !== 'object' || Array.isArray(props))
+  ) {
+    value = props
+    props = {}
+  }
+
+  node = assign({type: String(type)}, props)
+
+  if (Array.isArray(value)) {
+    node.children = value
+  } else if (value !== null && value !== undefined) {
+    node.value = String(value)
   }
 
-  return assign({}, props, { type: String(type) },
-                value != null && (Array.isArray(value)
-                                  ? { children: value }
-                                  : { value: String(value) }));
-};
+  return node
+}

Diff for: package.json

@@ -29,9 +29,24 @@
     "object-assign": "^4.1.0"
   },
   "devDependencies": {
-    "tape": "^4.2.0"
+    "prettier": "^1.14.2",
+    "tape": "^4.2.0",
+    "xo": "^0.22.0"
   },
   "scripts": {
+    "format": "prettier --write '**/*.js' && xo --fix",
     "test": "tape test/*.js"
+  },
+  "prettier": {
+    "tabWidth": 2,
+    "useTabs": false,
+    "singleQuote": true,
+    "bracketSpacing": false,
+    "semi": false,
+    "trailingComma": "none"
+  },
+  "xo": {
+    "prettier": true,
+    "esnext": false
   }
 }

Diff for: readme.md

@@ -17,52 +17,38 @@ Helper for creating [unist][] trees with [hyperscript][]-like syntax.
 ## Example
 
 ```js
-var u = require('unist-builder');
-
-u('root', [
-  u('subtree', { id: 1 }),
-  u('subtree', { id: 2 }, [
-    u('node', [
-      u('leaf', 'leaf-1'),
-      u('leaf', 'leaf-2')
-    ]),
-    u('leaf', { id: 3 }, 'leaf-3')
+var u = require('unist-builder')
+
+var tree = u('root', [
+  u('subtree', {id: 1}),
+  u('subtree', {id: 2}, [
+    u('node', [u('leaf', 'leaf-1'), u('leaf', 'leaf-2')]),
+    u('leaf', {id: 3}, 'leaf-3')
   ])
 ])
+
+console.dir(tree, {depth: null})
 ```
 
 results in the following tree:
 
-```json
+```js
 {
-  "type": "root",
-  "children": [
+  type: 'root',
+  children: [
+    {type: 'subtree', id: 1},
     {
-      "type": "subtree",
-      "id": 1
-    },
-    {
-      "type": "subtree",
-      "id": 2,
-      "children": [
+      type: 'subtree',
+      id: 2,
+      children: [
         {
-          "type": "node",
-          "children": [
-            {
-              "type": "leaf",
-              "value": "leaf-1"
-            },
-            {
-              "type": "leaf",
-              "value": "leaf-2"
-            }
+          type: 'node',
+          children: [
+            {type: 'leaf', value: 'leaf-1'},
+            {type: 'leaf', value: 'leaf-2'}
           ]
         },
-        {
-          "type": "leaf",
-          "id": 3,
-          "value": "leaf-3"
-        }
+        {type: 'leaf', id: 3, value: 'leaf-3'}
       ]
     }
   ]

Diff for: test/test.js

@@ -1,50 +1,40 @@
-'use strict';
+'use strict'
 
-var u = require('..');
+var u = require('..')
 
-var test = require('tape');
+var test = require('tape')
 
+test(function(t) {
+  t.deepEqual(
+    u('root', [
+      u('subtree', {id: 1}),
+      u('subtree', {id: 2}, [
+        u('node', [u('leaf', 'leaf-1'), u('leaf', 'leaf-2'), u('leaf', '')]),
+        u('leaf', {id: 3}, 'leaf-3')
+      ])
+    ]),
+    {
+      type: 'root',
+      children: [
+        {type: 'subtree', id: 1},
+        {
+          type: 'subtree',
+          id: 2,
+          children: [
+            {
+              type: 'node',
+              children: [
+                {type: 'leaf', value: 'leaf-1'},
+                {type: 'leaf', value: 'leaf-2'},
+                {type: 'leaf', value: ''}
+              ]
+            },
+            {type: 'leaf', id: 3, value: 'leaf-3'}
+          ]
+        }
+      ]
+    }
+  )
 
-test(function (t) {
-  var ast = u('root', [
-    u('subtree', { id: 1 }),
-    u('subtree', { id: 2 }, [
-      u('node', [
-        u('leaf', 'leaf-1'),
-        u('leaf', 'leaf-2'),
-        u('leaf', '')
-      ]),
-      u('leaf', { id: 3 }, 'leaf-3')
-    ])
-  ]);
-
-  t.deepEqual(ast, {
-    type: 'root',
-    children: [{
-      type: 'subtree',
-      id: 1
-    }, {
-      type: 'subtree',
-      id: 2,
-      children: [{
-        type: 'node',
-        children: [{
-          type: 'leaf',
-          value: 'leaf-1'
-        }, {
-          type: 'leaf',
-          value: 'leaf-2'
-        }, {
-          type: 'leaf',
-          value: ''
-        }]
-      }, {
-        type: 'leaf',
-        id: 3,
-        value: 'leaf-3'
-      }]
-    }]
-  });
-
-  t.end();
-});
+  t.end()
+})