Refactor code-style

Author: wooorm

.prettierignore

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

index.js

@@ -1,30 +1,36 @@
-'use strict';
-
-var xtend = require('xtend');
-var toH = require('hast-to-hyperscript');
-var NS = require('web-namespaces');
-var zwitch = require('zwitch');
-var mapz = require('mapz');
-
-module.exports = transform;
-
-var own = {}.hasOwnProperty;
-var one = zwitch('type');
-var all = mapz(one, {key: 'children', indices: false});
-
-var customProps = ['__location', 'childNodes', 'content', 'parentNode', 'namespaceURI'];
-
-one.handlers.root = root;
-one.handlers.element = element;
-one.handlers.text = text;
-one.handlers.comment = comment;
-one.handlers.doctype = doctype;
+'use strict'
+
+var xtend = require('xtend')
+var toH = require('hast-to-hyperscript')
+var NS = require('web-namespaces')
+var zwitch = require('zwitch')
+var mapz = require('mapz')
+
+module.exports = transform
+
+var own = {}.hasOwnProperty
+var one = zwitch('type')
+var all = mapz(one, {key: 'children', indices: false})
+
+var customProps = [
+  '__location',
+  'childNodes',
+  'content',
+  'parentNode',
+  'namespaceURI'
+]
+
+one.handlers.root = root
+one.handlers.element = element
+one.handlers.text = text
+one.handlers.comment = comment
+one.handlers.doctype = doctype
 
 /* Map of tag-names starting new namespaces. */
 var namespaces = {
   math: NS.mathml,
   svg: NS.svg
-};
+}
 
 /* Map of attributes with namespaces. */
 var attributeSpaces = {
@@ -40,58 +46,62 @@ var attributeSpaces = {
   'xml:space': {prefix: 'xml', name: 'space', namespace: NS.xml},
   xmlns: {prefix: '', name: 'xmlns', namespace: NS.xmlns},
   'xmlns:xlink': {prefix: 'xmlns', name: 'xlink', namespace: NS.xmlns}
-};
+}
 
 /* Transform a tree from HAST to Parse5’s AST. */
 function transform(tree) {
-  return patch(one(tree), null, NS.html);
+  return patch(one(tree), null, NS.html)
 }
 
 function root(node) {
-  var data = node.data || {};
-  var qs = own.call(data, 'quirksMode') ? Boolean(data.quirksMode) : false;
+  var data = node.data || {}
+  var qs = own.call(data, 'quirksMode') ? Boolean(data.quirksMode) : false
 
   return {
     nodeName: '#document',
     mode: qs ? 'quirks' : 'no-quirks',
     childNodes: all(node)
-  };
+  }
 }
 
 function element(node) {
-  var shallow = xtend(node);
+  var shallow = xtend(node)
 
-  shallow.children = [];
+  shallow.children = []
 
-  return toH(function (name, attrs) {
-    var values = [];
-    var content;
-    var value;
-    var key;
+  return toH(function(name, attrs) {
+    var values = []
+    var content
+    var value
+    var key
 
     for (key in attrs) {
-      value = {name: key, value: attrs[key]};
+      value = {name: key, value: attrs[key]}
 
       if (own.call(attributeSpaces, key)) {
-        value = xtend(value, attributeSpaces[key]);
+        value = xtend(value, attributeSpaces[key])
       }
 
-      values.push(value);
+      values.push(value)
     }
 
     if (name === 'template') {
-      content = transform(shallow.content);
-      delete content.mode;
-      content.nodeName = '#document-fragment';
+      content = transform(shallow.content)
+      delete content.mode
+      content.nodeName = '#document-fragment'
     }
 
-    return wrap(node, {
-      nodeName: node.tagName,
-      tagName: node.tagName,
-      attrs: values,
-      childNodes: node.children ? all(node) : []
-    }, content);
-  }, shallow);
+    return wrap(
+      node,
+      {
+        nodeName: node.tagName,
+        tagName: node.tagName,
+        attrs: values,
+        childNodes: node.children ? all(node) : []
+      },
+      content
+    )
+  }, shallow)
 }
 
 function doctype(node) {
@@ -100,21 +110,21 @@ function doctype(node) {
     name: node.name,
     publicId: node.public || null,
     systemId: node.system || null
-  });
+  })
 }
 
 function text(node) {
   return wrap(node, {
     nodeName: '#text',
     value: node.value
-  });
+  })
 }
 
 function comment(node) {
   return wrap(node, {
     nodeName: '#comment',
     data: node.value
-  });
+  })
 }
 
 /* Patch position. */
@@ -125,62 +135,62 @@ function wrap(node, ast, content) {
       col: node.position.start.column,
       startOffset: node.position.start.offset,
       endOffset: node.position.end.offset
-    };
+    }
   }
 
   if (content) {
-    ast.content = content;
+    ast.content = content
   }
 
-  return ast;
+  return ast
 }
 
 /* Patch a tree recursively, by adding namespaces
  * and parent references where needed. */
 function patch(node, parent, ns) {
-  var location = node.__location;
-  var children = node.childNodes;
-  var name = node.tagName;
-  var replacement = {};
-  var length;
-  var index;
-  var key;
+  var location = node.__location
+  var children = node.childNodes
+  var name = node.tagName
+  var replacement = {}
+  var length
+  var index
+  var key
 
   for (key in node) {
     if (customProps.indexOf(key) === -1) {
-      replacement[key] = node[key];
+      replacement[key] = node[key]
     }
   }
 
   if (own.call(namespaces, name)) {
-    ns = namespaces[name];
+    ns = namespaces[name]
   }
 
   if (own.call(replacement, 'tagName')) {
-    replacement.namespaceURI = ns;
+    replacement.namespaceURI = ns
   }
 
   if (children) {
-    replacement.childNodes = children;
-    length = children.length;
-    index = -1;
+    replacement.childNodes = children
+    length = children.length
+    index = -1
 
     while (++index < length) {
-      children[index] = patch(children[index], replacement, ns);
+      children[index] = patch(children[index], replacement, ns)
     }
   }
 
   if (name === 'template') {
-    replacement.content = patch(node.content, null, ns);
+    replacement.content = patch(node.content, null, ns)
   }
 
   if (parent) {
-    replacement.parentNode = parent;
+    replacement.parentNode = parent
   }
 
   if (location) {
-    replacement.__location = location;
+    replacement.__location = location
   }
 
-  return replacement;
+  return replacement
 }

package.json

@@ -30,23 +30,31 @@
     "esmangle": "^1.0.1",
     "nyc": "^12.0.0",
     "parse5": "^3.0.0",
+    "prettier": "^1.13.5",
     "remark-cli": "^5.0.0",
     "remark-preset-wooorm": "^4.0.0",
     "tape": "^4.0.0",
     "xo": "^0.21.0"
   },
   "scripts": {
-    "build-md": "remark . --quiet --frail --output",
+    "format": "remark . -qfo && prettier --write '**/*.js' && xo --fix",
     "build-bundle": "browserify index.js --bare -s hastUtilToParse5 > hast-util-to-parse5.js",
     "build-mangle": "esmangle hast-util-to-parse5.js > hast-util-to-parse5.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",
-    "test": "npm run build && npm run lint && npm run test-coverage"
+    "test": "npm run format && npm run build && npm run test-coverage"
+  },
+  "prettier": {
+    "tabWidth": 2,
+    "useTabs": false,
+    "singleQuote": true,
+    "bracketSpacing": false,
+    "semi": false,
+    "trailingComma": "none"
   },
   "xo": {
-    "space": true,
+    "prettier": true,
     "esnext": false,
     "rules": {
       "guard-for-in": "off"

readme.md

@@ -16,16 +16,16 @@ npm install hast-util-to-parse5
 ## Usage
 
 ```javascript
-var toParse5 = require('hast-util-to-parse5');
+var toParse5 = require('hast-util-to-parse5')
 
 var ast = toParse5({
   type: 'element',
   tagName: 'h1',
   properties: {},
   children: [{type: 'text', value: 'World!'}]
-});
+})
 
-console.log(ast);
+console.log(ast)
 ```
 
 Yields:

test/comment.js

@@ -1,14 +1,14 @@
-'use strict';
+'use strict'
 
-var test = require('tape');
-var parse5 = require('parse5');
-var toParse5 = require('..');
+var test = require('tape')
+var parse5 = require('parse5')
+var toParse5 = require('..')
 
-test('comment', function (t) {
-  var node = parse5.parseFragment('<!--Alpha-->');
+test('comment', function(t) {
+  var node = parse5.parseFragment('<!--Alpha-->')
 
-  node = node.childNodes[0];
-  delete node.parentNode;
+  node = node.childNodes[0]
+  delete node.parentNode
 
   t.deepEqual(
     toParse5({
@@ -17,7 +17,7 @@ test('comment', function (t) {
     }),
     node,
     'should transform comments'
-  );
+  )
 
-  t.end();
-});
+  t.end()
+})