Refactor code-style

Author: wooorm

.prettierignore

@@ -0,0 +1,3 @@
+coverage/
+retext-syntax-urls.js
+retext-syntax-urls.min.js

index.js

@@ -1,130 +1,130 @@
-'use strict';
+'use strict'
 
-var position = require('unist-util-position');
-var modifyChildren = require('unist-util-modify-children');
-var toString = require('nlcst-to-string');
+var position = require('unist-util-position')
+var modifyChildren = require('unist-util-modify-children')
+var toString = require('nlcst-to-string')
 
-module.exports = urls;
+module.exports = urls
 
-var slashes = /^\/{1,3}$/;
+var slashes = /^\/{1,3}$/
 
 function urls() {
-  this.Parser.prototype.useFirst('tokenizeSentence', modifyChildren(mergeLinks));
+  this.Parser.prototype.useFirst('tokenizeSentence', modifyChildren(mergeLinks))
 }
 
 /* eslint-disable complexity */
 function mergeLinks(child, index, parent) {
-  var siblings = parent.children;
-  var nodes = [child];
-  var start = index;
-  var end = index;
-  var currentIndex = index;
-  var value;
-  var type;
-  var initial;
-  var final;
+  var siblings = parent.children
+  var nodes = [child]
+  var start = index
+  var end = index
+  var currentIndex = index
+  var value
+  var type
+  var initial
+  var final
 
   if (!puncOrSymbol(child.type) || toString(child) !== '.') {
-    return;
+    return
   }
 
-  /* Find preceding word/punctuation. Stop before slashes, break after `www`. */
+  // Find preceding word/punctuation. Stop before slashes, break after `www`.
   while (siblings[start - 1]) {
-    type = siblings[start - 1].type;
+    type = siblings[start - 1].type
 
     if (!applicable(type)) {
-      break;
+      break
     }
 
     if (puncOrSymbol(type) && slashes.test(toString(siblings[start - 1]))) {
-      break;
+      break
     }
 
-    start--;
+    start--
 
-    nodes.unshift(siblings[start]);
+    nodes.unshift(siblings[start])
 
     if (type === 'WordNode' && toString(siblings[start]) === 'www') {
-      break;
+      break
     }
   }
 
-  /* Find following word/punctuation. */
+  // Find following word/punctuation.
   while (siblings[end + 1]) {
-    type = siblings[end + 1].type;
+    type = siblings[end + 1].type
 
     if (!applicable(type)) {
-      break;
+      break
     }
 
-    end++;
-    nodes.push(siblings[end]);
+    end++
+    nodes.push(siblings[end])
   }
 
-  /* This full stop doesnt look like a link:  it’s either not followed,
-   * or not preceded, by words or punctuation. */
+  // This full stop doesnt look like a link:  it’s either not followed, or not
+  // preceded, by words or punctuation.
   if (currentIndex === start || currentIndex === end) {
-    return;
+    return
   }
 
-  /* 1-3 slashes. */
+  // 1-3 slashes.
   if (
     start > 0 &&
     puncOrSymbol(siblings[start - 1].type) &&
     slashes.test(toString(siblings[start - 1]))
   ) {
-    start--;
-    nodes.unshift(siblings[start]);
+    start--
+    nodes.unshift(siblings[start])
   }
 
-  /* URL protocol and colon. */
+  // URL protocol and colon.
   if (
     start > 2 &&
     puncOrSymbol(siblings[start - 1].type) &&
     toString(siblings[start - 1]) === ':' &&
     siblings[start - 2].type === 'WordNode'
   ) {
-    nodes.unshift(siblings[start - 2], siblings[start - 1]);
-    start -= 2;
+    nodes.unshift(siblings[start - 2], siblings[start - 1])
+    start -= 2
   }
 
-  value = null;
+  value = null
 
-  /* Remove the last node if it's punctuation, unless it's `/` or `)`. */
+  // Remove the last node if it's punctuation, unless it's `/` or `)`.
   if (puncOrSymbol(siblings[end].type)) {
-    value = toString(siblings[end]);
+    value = toString(siblings[end])
 
     if (value !== '/' && value !== ')') {
-      end--;
-      nodes.pop();
+      end--
+      nodes.pop()
     }
   }
 
-  child = {type: 'SourceNode', value: toString(nodes)};
-  initial = position.start(nodes[0]);
-  final = position.end(nodes[nodes.length - 1]);
+  child = {type: 'SourceNode', value: toString(nodes)}
+  initial = position.start(nodes[0])
+  final = position.end(nodes[nodes.length - 1])
 
   if (initial.line && final.line) {
-    child.position = {start: initial, end: final};
+    child.position = {start: initial, end: final}
   }
 
-  /* Remove the nodes and insert a SourceNode. */
-  siblings.splice(start, end - start + 1, child);
+  // Remove the nodes and insert a SourceNode.
+  siblings.splice(start, end - start + 1, child)
 
-  index++;
+  index++
 
-  /* Ignore the following full-stop: it's not part of a link. */
+  // Ignore the following full-stop: it's not part of a link.
   if (value === '.') {
-    index++;
+    index++
   }
 
-  return index;
+  return index
 }
 
 function applicable(type) {
-  return type === 'WordNode' || puncOrSymbol(type);
+  return type === 'WordNode' || puncOrSymbol(type)
 }
 
 function puncOrSymbol(type) {
-  return type === 'PunctuationNode' || type === 'SymbolNode';
+  return type === 'PunctuationNode' || type === 'SymbolNode'
 }

package.json

@@ -31,6 +31,7 @@
     "is-hidden": "^1.1.1",
     "not": "^0.1.0",
     "nyc": "^13.0.0",
+    "prettier": "^1.14.3",
     "remark-cli": "^6.0.0",
     "remark-preset-wooorm": "^4.0.0",
     "retext": "^5.0.0",
@@ -40,23 +41,30 @@
     "xo": "^0.23.0"
   },
   "scripts": {
-    "build-md": "remark . -qfo",
+    "format": "remark . -qfo && prettier --write \"**/*.js\" && xo --fix",
     "build-bundle": "browserify index.js --bare -s retextSyntaxUrls > retext-syntax-urls.js",
     "build-mangle": "esmangle retext-syntax-urls.js > retext-syntax-urls.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/index.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,
     "ignores": [
       "retext-syntax-urls.js"

readme.md

@@ -16,21 +16,24 @@ npm install retext-syntax-urls
 Without `syntax-urls`:
 
 ```javascript
-var dictionary = require('dictionary-en-gb');
-var unified = require('unified');
-var english = require('retext-english');
-var stringify = require('retext-stringify');
-var spell = require('retext-spell');
-var urls = require('retext-syntax-urls');
-var report = require('vfile-reporter');
+var dictionary = require('dictionary-en-gb')
+var unified = require('unified')
+var english = require('retext-english')
+var stringify = require('retext-stringify')
+var spell = require('retext-spell')
+var urls = require('retext-syntax-urls')
+var report = require('vfile-reporter')
 
 unified()
   .use(english)
   .use(spell, dictionary)
   .use(stringify)
-  .process('Have you read readme.md? Check it out: www.example.com/readme.md', function (err, file) {
-    console.log(report(err || file));
-  });
+  .process(
+    'Have you read readme.md? Check it out: www.example.com/readme.md',
+    function(err, file) {
+      console.log(report(err || file))
+    }
+  )
 ```
 
 Yields: