replacing md5 with simple hash; misc cleanup

Author: chrisdothtml

package.json

@@ -17,9 +17,6 @@
   "peerDependencies": {
     "dustjs-linkedin": "^2.7.5"
   },
-  "dependencies": {
-    "md5": "^2.2.1"
-  },
   "devDependencies": {
     "dustjs-linkedin": "^2.7.5",
     "rollup": "^0.41.6",

rollup.config.js

@@ -4,7 +4,6 @@ export default {
   format: 'cjs',
   external: [
     'dustjs-linkedin',
-    'md5',
     'path'
   ]
 }

src/index.js

@@ -1,27 +1,18 @@
-import md5 from 'md5'
 import dust, { compile } from 'dustjs-linkedin'
-import { basename, dirname, extname, resolve } from 'path'
-
-const { floor, random } = Math
-const token = floor(random() * 10000000)
-
-function getFullPath (from, to) {
-  return resolve(dirname(from), to)
-}
-
-function generateName (path) {
-  const name = basename(path, '.dust')
-  const hash = md5(token + path)
-
-  return `${name}-${hash}`
-}
+import { dirname, extname, resolve } from 'path'
+import { generateName, hash } from './utils.js'
 
+/**
+ * Finds all valid partials and replaces paths with respective
+ * unique template names
+ */
 function resolvePartials (source, parentPath) {
   const partials = []
+  // matches {>"./file.dust" /}
   const partialRegex = /\{>\s*"([^"]+)"[^\n]*\/\}/g
   const newSource = source.replace(partialRegex, (tag, relPath) => {
     if (extname(relPath) === '.dust') {
-      const fullPath = getFullPath(parentPath, relPath)
+      const fullPath = resolve(dirname(parentPath), relPath)
       const name = generateName(fullPath)
 
       partials.push(relPath)

src/utils.js

@@ -0,0 +1,32 @@
+import { basename } from 'path'
+
+const { abs, floor, random } = Math
+const token = floor(random() * 10000000)
+
+/*
+ * http://werxltd.com/wp/2010/05/13/javascript-implementation-of-javas-string-hashcode-method/
+ */
+export function hash (input) {
+  let hash = 0
+
+  if (input.length) {
+    for (let i = 0; i < input.length; i++) {
+      const char = input.charCodeAt(i)
+
+      hash = ((hash << 5) - hash) + char
+      hash = hash & hash
+    }
+  }
+
+  return abs(hash)
+}
+
+/**
+ * Generates unique template name for the provided path
+ */
+export function generateName (path) {
+  const name = basename(path, '.dust')
+  const pathHash = hash(token + path)
+
+  return `${name}-${pathHash}`
+}

yarn.lock

@@ -130,10 +130,6 @@ caseless@~0.12.0:
   version "0.12.0"
   resolved "https://registry.yarnpkg.com/caseless/-/caseless-0.12.0.tgz#1b681c21ff84033c826543090689420d187151dc"
 
-charenc@~0.0.1:
-  version "0.0.2"
-  resolved "https://registry.yarnpkg.com/charenc/-/charenc-0.0.2.tgz#c0a1d2f3a7092e03774bfa83f14c0fc5790a8667"
-
 chokidar@~1.5.1:
   version "1.5.2"
   resolved "https://registry.yarnpkg.com/chokidar/-/chokidar-1.5.2.tgz#293e728640cc93dd8277424334b3c6d4ad3a348a"
@@ -182,10 +178,6 @@ core-util-is@~1.0.0:
   version "1.0.2"
   resolved "https://registry.yarnpkg.com/core-util-is/-/core-util-is-1.0.2.tgz#b5fd54220aa2bc5ab57aab7140c940754503c1a7"
 
-crypt@~0.0.1:
-  version "0.0.2"
-  resolved "https://registry.yarnpkg.com/crypt/-/crypt-0.0.2.tgz#88d7ff7ec0dfb86f713dc87bbb42d044d3e6c41b"
-
 cryptiles@2.x.x:
   version "2.0.5"
   resolved "https://registry.yarnpkg.com/cryptiles/-/cryptiles-2.0.5.tgz#3bdfecdc608147c1c67202fa291e7dca59eaa3b8"
@@ -427,7 +419,7 @@ is-binary-path@^1.0.0:
   dependencies:
     binary-extensions "^1.0.0"
 
-is-buffer@^1.0.2, is-buffer@~1.1.1:
+is-buffer@^1.0.2:
   version "1.1.5"
   resolved "https://registry.yarnpkg.com/is-buffer/-/is-buffer-1.1.5.tgz#1f3b26ef613b214b88cbca23cc6c01d87961eecc"
 
@@ -536,14 +528,6 @@ kind-of@^3.0.2:
   dependencies:
     is-buffer "^1.0.2"
 
-md5@^2.2.1:
-  version "2.2.1"
-  resolved "https://registry.yarnpkg.com/md5/-/md5-2.2.1.tgz#53ab38d5fe3c8891ba465329ea23fac0540126f9"
-  dependencies:
-    charenc "~0.0.1"
-    crypt "~0.0.1"
-    is-buffer "~1.1.1"
-
 micromatch@^2.1.5:
   version "2.3.11"
   resolved "https://registry.yarnpkg.com/micromatch/-/micromatch-2.3.11.tgz#86677c97d1720b363431d04d0d15293bd38c1565"