Refactor code-style

Author: wooorm

.prettierignore

@@ -0,0 +1,3 @@
+coverage/
+hast-to-hyperscript.js
+hast-to-hyperscript.min.js

index.js

@@ -1,43 +1,45 @@
-'use strict';
+'use strict'
 
-var trim = require('trim');
-var paramCase = require('kebab-case');
-var information = require('property-information');
-var spaces = require('space-separated-tokens');
-var commas = require('comma-separated-tokens');
-var nan = require('is-nan');
-var is = require('unist-util-is');
+var trim = require('trim')
+var paramCase = require('kebab-case')
+var information = require('property-information')
+var spaces = require('space-separated-tokens')
+var commas = require('comma-separated-tokens')
+var nan = require('is-nan')
+var is = require('unist-util-is')
 
-module.exports = wrapper;
+module.exports = wrapper
 
 function wrapper(h, node, prefix) {
-  var r;
-  var v;
+  var r
+  var v
 
   if (typeof h !== 'function') {
-    throw new Error('h is not a function');
+    throw new Error('h is not a function')
   }
 
-  r = react(h);
-  v = vdom(h);
+  r = react(h)
+  v = vdom(h)
 
   if (prefix === null || prefix === undefined) {
-    prefix = r === true || v === true ? 'h-' : false;
+    prefix = r === true || v === true ? 'h-' : false
   }
 
   if (is('root', node)) {
     if (node.children.length === 1 && is('element', node.children[0])) {
-      node = node.children[0];
+      node = node.children[0]
     } else {
       node = {
         type: 'element',
         tagName: 'div',
         properties: {},
         children: node.children
-      };
+      }
     }
   } else if (!is('element', node)) {
-    throw new Error('Expected root or element, not `' + ((node && node.type) || node) + '`');
+    throw new Error(
+      'Expected root or element, not `' + ((node && node.type) || node) + '`'
+    )
   }
 
   return toH(h, node, {
@@ -46,41 +48,41 @@ function wrapper(h, node, prefix) {
     react: r,
     vdom: v,
     hyperscript: hyperscript(h)
-  });
+  })
 }
 
 /* Transform a HAST node through a hyperscript interface
  * to *anything*! */
 function toH(h, node, ctx) {
-  var selector = node.tagName;
-  var properties;
-  var attributes;
-  var children;
-  var property;
-  var elements;
-  var length;
-  var index;
-  var value;
-
-  properties = node.properties;
-  attributes = {};
+  var selector = node.tagName
+  var properties
+  var attributes
+  var children
+  var property
+  var elements
+  var length
+  var index
+  var value
+
+  properties = node.properties
+  attributes = {}
 
   for (property in properties) {
-    addAttribute(attributes, property, properties[property], ctx);
+    addAttribute(attributes, property, properties[property], ctx)
   }
 
   if (ctx.vdom === true) {
-    selector = selector.toUpperCase();
+    selector = selector.toUpperCase()
   }
 
   if (ctx.hyperscript === true && attributes.id) {
-    selector += '#' + attributes.id;
-    delete attributes.id;
+    selector += '#' + attributes.id
+    delete attributes.id
   }
 
   if ((ctx.hyperscript === true || ctx.vdom === true) && attributes.className) {
-    selector += '.' + spaces.parse(attributes.className).join('.');
-    delete attributes.className;
+    selector += '.' + spaces.parse(attributes.className).join('.')
+    delete attributes.className
   }
 
   if (typeof attributes.style === 'string') {
@@ -89,46 +91,48 @@ function toH(h, node, ctx) {
      * docs/vnode.md#propertiesstyle-vs-propertiesattributesstyle */
     if (ctx.vdom === true) {
       if (!attributes.attributes) {
-        attributes.attributes = {};
+        attributes.attributes = {}
       }
 
-      attributes.attributes.style = attributes.style;
-      delete attributes.style;
-    /* React only accepts `style` as object. */
+      attributes.attributes.style = attributes.style
+      delete attributes.style
+      /* React only accepts `style` as object. */
     } else if (ctx.react === true) {
-      attributes.style = parseStyle(attributes.style);
+      attributes.style = parseStyle(attributes.style)
     }
   }
 
   if (ctx.prefix) {
-    ctx.key++;
-    attributes.key = ctx.prefix + ctx.key;
+    ctx.key++
+    attributes.key = ctx.prefix + ctx.key
   }
 
-  elements = [];
-  children = node.children || [];
-  length = children.length;
-  index = -1;
+  elements = []
+  children = node.children || []
+  length = children.length
+  index = -1
 
   while (++index < length) {
-    value = children[index];
+    value = children[index]
 
     if (is('element', value)) {
-      elements.push(toH(h, value, ctx));
+      elements.push(toH(h, value, ctx))
     } else if (is('text', value)) {
-      elements.push(value.value);
+      elements.push(value.value)
     }
   }
 
   /* Ensure no React warnings are triggered for
    * void elements having children passed in. */
-  return elements.length === 0 ? h(selector, attributes) : h(selector, attributes, elements);
+  return elements.length === 0
+    ? h(selector, attributes)
+    : h(selector, attributes, elements)
 }
 
 /* Add `name` and its `value` to `props`. */
 function addAttribute(props, name, value, ctx) {
-  var info = information(name) || {};
-  var subprop;
+  var info = information(name) || {}
+  var subprop
 
   /* Ignore nully, `false`, `NaN`, and falsey known
    * booleans. */
@@ -139,109 +143,112 @@ function addAttribute(props, name, value, ctx) {
     nan(value) ||
     (info.boolean && !value)
   ) {
-    return;
+    return
   }
 
   if (info.name) {
-    name = info.name;
+    name = info.name
   } else if (ctx.react && !paramCasedReactProp(name)) {
-    name = camelCase(name);
+    name = camelCase(name)
   } else {
-    name = paramCase(name);
+    name = paramCase(name)
   }
 
   if (value !== null && typeof value === 'object' && 'length' in value) {
     /* Accept `array`.  Most props are space-separater. */
-    value = (info.commaSeparated ? commas : spaces).stringify(value);
+    value = (info.commaSeparated ? commas : spaces).stringify(value)
   }
 
   /* Treat `true` and truthy known booleans. */
   if (info.boolean && ctx.hyperscript === true) {
-    value = '';
+    value = ''
   }
 
   if (info.name !== 'class' && (info.mustUseAttribute || !info.name)) {
     if (ctx.vdom === true) {
-      subprop = 'attributes';
+      subprop = 'attributes'
     } else if (ctx.hyperscript === true) {
-      subprop = 'attrs';
+      subprop = 'attrs'
     }
 
     if (subprop) {
       if (props[subprop] === undefined) {
-        props[subprop] = {};
+        props[subprop] = {}
       }
 
-      props[subprop][name] = value;
+      props[subprop][name] = value
 
-      return;
+      return
     }
   }
 
-  props[info.propertyName || name] = value;
+  props[info.propertyName || name] = value
 }
 
 /* Check if `h` is `react.createElement`.  It doesn’t accept
  * `class` as an attribute, it must be added through the
  * `selector`. */
 function react(h) {
-  var node = h && h('div');
-  return Boolean(node && ('_owner' in node || '_store' in node) && node.key === null);
+  var node = h && h('div')
+  return Boolean(
+    node && ('_owner' in node || '_store' in node) && node.key === null
+  )
 }
 
 /* Check if `h` is `hyperscript`.  It doesn’t accept
  * `class` as an attribute, it must be added through the
  * `selector`. */
 function hyperscript(h) {
-  return Boolean(h && h.context && h.cleanup);
+  return Boolean(h && h.context && h.cleanup)
 }
 
-/**
- * Check if `h` is `virtual-dom/h`.  It’s the only
+/* Check if `h` is `virtual-dom/h`.  It’s the only
  * hyperscript “compatible” interface needing `attributes`. */
 function vdom(h) {
   try {
-    return h('div').type === 'VirtualNode';
-  } catch (err) { /* Empty */ }
+    return h('div').type === 'VirtualNode'
+  } catch (err) {
+    /* Empty */
+  }
 
   /* istanbul ignore next */
-  return false;
+  return false
 }
 
 function parseStyle(value) {
-  var result = {};
-  var declarations = value.split(';');
-  var length = declarations.length;
-  var index = -1;
-  var declaration;
-  var prop;
-  var pos;
+  var result = {}
+  var declarations = value.split(';')
+  var length = declarations.length
+  var index = -1
+  var declaration
+  var prop
+  var pos
 
   while (++index < length) {
-    declaration = declarations[index];
-    pos = declaration.indexOf(':');
+    declaration = declarations[index]
+    pos = declaration.indexOf(':')
     if (pos !== -1) {
-      prop = camelCase(trim(declaration.slice(0, pos)));
-      result[prop] = trim(declaration.slice(pos + 1));
+      prop = camelCase(trim(declaration.slice(0, pos)))
+      result[prop] = trim(declaration.slice(pos + 1))
     }
   }
 
-  return result;
+  return result
 }
 
 function paramCasedReactProp(name) {
-  var head = name.slice(0, 4);
-  return (head === 'data' || head === 'aria') && name.length > 4;
+  var head = name.slice(0, 4)
+  return (head === 'data' || head === 'aria') && name.length > 4
 }
 
 function camelCase(val) {
   if (val.slice(0, 4) === '-ms-') {
-    val = 'ms-' + val.slice(4);
+    val = 'ms-' + val.slice(4)
   }
 
-  return val.replace(/-([a-z])/g, replace);
+  return val.replace(/-([a-z])/g, replace)
 }
 
 function replace($0, $1) {
-  return $1.toUpperCase();
+  return $1.toUpperCase()
 }

package.json

@@ -38,6 +38,7 @@
     "esmangle": "^1.0.0",
     "hyperscript": "^2.0.2",
     "nyc": "^12.0.0",
+    "prettier": "^1.13.5",
     "react": "^16.1.1",
     "react-dom": "^16.1.1",
     "rehype": "^5.0.0",
@@ -50,26 +51,34 @@
     "xo": "^0.21.0"
   },
   "scripts": {
-    "build-md": "remark . -qfo",
+    "format": "remark . -qfo && prettier --write '**/*.js' && xo --fix",
     "build-bundle": "browserify index.js --bare -s hast-to-hyperscript > hast-to-hyperscript.js",
     "build-mangle": "esmangle hast-to-hyperscript.js > hast-to-hyperscript.min.js",
-    "build": "npm run build-md && npm run build-bundle && npm run build-mangle",
-    "lint": "xo",
+    "build": "npm run build-bundle && npm run build-mangle",
     "test-api": "node test",
     "test-coverage": "nyc --reporter lcov tape test.js",
-    "test": "npm run build && npm run lint && npm run test-coverage"
+    "test": "npm run format && npm run build && npm run test-coverage"
   },
   "nyc": {
     "check-coverage": true,
     "lines": 100,
     "functions": 100,
     "branches": 100
   },
+  "prettier": {
+    "tabWidth": 2,
+    "useTabs": false,
+    "singleQuote": true,
+    "bracketSpacing": false,
+    "semi": false,
+    "trailingComma": "none"
+  },
   "xo": {
-    "space": true,
+    "prettier": true,
     "esnext": false,
     "rules": {
       "unicorn/prefer-type-error": "off",
+      "complexity": "off",
       "guard-for-in": "off"
     }
   },