Refactor to improve bundle size

Author: wooorm

index.js

@@ -1,12 +1,12 @@
 'use strict'
 
-var html = require('property-information/html')
-var svg = require('property-information/svg')
-var find = require('property-information/find')
-var ns = require('web-namespaces')
 var s = require('hastscript/svg')
 var h = require('hastscript')
+var find = require('property-information/find')
+var html = require('property-information/html')
+var svg = require('property-information/svg')
 var vfileLocation = require('vfile-location')
+var ns = require('web-namespaces')
 
 module.exports = wrapper
 
@@ -45,7 +45,7 @@ function transform(ast, config) {
   var schema = config.schema
   var fn = own.call(map, ast.nodeName) ? map[ast.nodeName] : element
   var children
-  var node
+  var result
   var position
 
   if (fn === element) {
@@ -56,20 +56,20 @@ function transform(ast, config) {
     children = nodes(ast.childNodes, config)
   }
 
-  node = fn(ast, children, config)
+  result = fn(ast, children, config)
 
   if (ast.sourceCodeLocation && config.file) {
-    position = location(node, ast.sourceCodeLocation, config)
+    position = location(result, ast.sourceCodeLocation, config)
 
     if (position) {
       config.location = true
-      node.position = position
+      result.position = position
     }
   }
 
   config.schema = schema
 
-  return node
+  return result
 }
 
 // Transform children.
@@ -87,7 +87,7 @@ function nodes(children, config) {
 // Transform a document.
 // Stores `ast.quirksMode` in `node.data.quirksMode`.
 function root(ast, children, config) {
-  var node = {
+  var result = {
     type: 'root',
     children: children,
     data: {quirksMode: ast.mode === 'quirks' || ast.mode === 'limited-quirks'}
@@ -98,13 +98,13 @@ function root(ast, children, config) {
   if (config.file && config.location) {
     doc = String(config.file)
     location = vfileLocation(doc)
-    node.position = {
+    result.position = {
       start: location.toPoint(0),
       end: location.toPoint(doc.length)
     }
   }
 
-  return node
+  return result
 }
 
 // Transform a doctype.
@@ -130,73 +130,58 @@ function comment(ast) {
 // Transform an element.
 function element(ast, children, config) {
   var fn = config.schema.space === 'svg' ? s : h
-  var name = ast.tagName
-  var attributes = ast.attrs
-  var length = attributes.length
   var props = {}
   var index = -1
+  var result
   var attribute
-  var prop
-  var node
   var pos
   var start
   var end
 
-  while (++index < length) {
-    attribute = attributes[index]
-    prop = (attribute.prefix ? attribute.prefix + ':' : '') + attribute.name
-    props[prop] = attribute.value
+  while (++index < ast.attrs.length) {
+    attribute = ast.attrs[index]
+    props[(attribute.prefix ? attribute.prefix + ':' : '') + attribute.name] =
+      attribute.value
   }
 
-  node = fn(name, props, children)
+  result = fn(ast.tagName, props, children)
 
-  if (name === 'template' && 'content' in ast) {
+  if (result.tagName === 'template' && 'content' in ast) {
     pos = ast.sourceCodeLocation
     start = pos && pos.startTag && position(pos.startTag).end
     end = pos && pos.endTag && position(pos.endTag).start
 
-    node.content = transform(ast.content, config)
+    result.content = transform(ast.content, config)
 
     if ((start || end) && config.file) {
-      node.content.position = {start: start, end: end}
+      result.content.position = {start: start, end: end}
     }
   }
 
-  return node
+  return result
 }
 
 // Create clean positional information.
 function location(node, location, config) {
-  var schema = config.schema
-  var verbose = config.verbose
-  var pos = position(location)
-  var reference
-  var attributes
-  var attribute
+  var result = position(location)
+  var tail
+  var key
   var props
-  var prop
 
   if (node.type === 'element') {
-    reference = node.children[node.children.length - 1]
+    tail = node.children[node.children.length - 1]
 
     // Bug for unclosed with children.
     // See: <https://github.com/inikulin/parse5/issues/109>.
-    if (
-      !location.endTag &&
-      reference &&
-      reference.position &&
-      reference.position.end
-    ) {
-      pos.end = Object.assign({}, reference.position.end)
+    if (!location.endTag && tail && tail.position && tail.position.end) {
+      result.end = Object.assign({}, tail.position.end)
     }
 
-    if (verbose) {
-      attributes = location.attrs
+    if (config.verbose) {
       props = {}
 
-      for (attribute in attributes) {
-        prop = find(schema, attribute).property
-        props[prop] = position(attributes[attribute])
+      for (key in location.attrs) {
+        props[find(config.schema, key).property] = position(location.attrs[key])
       }
 
       node.data = {
@@ -209,7 +194,7 @@ function location(node, location, config) {
     }
   }
 
-  return pos
+  return result
 }
 
 function position(loc) {

package.json

@@ -53,7 +53,7 @@
     "xo": "^0.34.0"
   },
   "scripts": {
-    "format": "remark . -qfo && prettier --write . && xo --fix",
+    "format": "remark . -qfo && prettier . -w --loglevel warn && xo --fix",
     "build-bundle": "browserify index.js -s hastUtilFromParse5 > hast-util-from-parse5.js",
     "build-mangle": "browserify index.js -p tinyify -s hastUtilFromParse5 > hast-util-from-parse5.min.js",
     "build": "npm run build-bundle && npm run build-mangle",