fix bundle issue, support both node and browser (#4)

* fix bundle issue, support both node and browser

* version bump

* update version as fix

.npmignore

@@ -3,4 +3,5 @@ test/
 .gitignore
 circle.yml
 index.js
+rollup.config.js
 yarn.lock

index.js

@@ -8,7 +8,7 @@ function namehash (inputName) {
     node += '00'
   }
 
-  name = normalize(inputName)
+  var name = normalize(inputName)
 
   if (name) {
     var labels = name.split('.')
@@ -26,5 +26,7 @@ function normalize(name) {
   return name ? uts46.toAscii(name, {useStd3ASCII: true, transitional: false}) : name
 }
 
-exports.hash = namehash
+module.exports = {
-exports.normalize = normalize
+  hash: namehash,
+  normalize
+}

package.json

@@ -1,10 +1,10 @@
 {
   "name": "@ensdomains/eth-ens-namehash",
-  "version": "2.0.13",
+  "version": "2.0.14",
   "description": "A simple module for generating ENS namehashes per spec https://github.com/ethereum/EIPs/issues/137",
   "main": "dist/index.js",
   "scripts": {
-    "bundle": "browserify index.js -o dist/index.js && cp node_modules/idna-uts46-hx/idna-map.js dist/",
+    "bundle": "rollup --config rollup.config.js",
     "test": "node test"
   },
   "repository": {
@@ -22,14 +22,12 @@
   },
   "homepage": "https://github.com/ensdomains/eth-ens-namehash#readme",
   "devDependencies": {
-    "tape": "^4.6.3",
+    "@rollup/plugin-commonjs": "^20.0.0",
-    "babel-preset-es2015": "^6.24.1",
+    "@rollup/plugin-node-resolve": "^13.0.5",
-    "babel-preset-stage-0": "^6.24.1",
-    "browserify": "^14.0.0"
-  },
-  "dependencies": {
     "idna-uts46-hx": "^3.4.0",
-    "js-sha3": "^0.5.7"
+    "js-sha3": "^0.5.7",
+    "rollup": "^2.57.0",
+    "tape": "^4.6.3"
   },
   "directories": {
     "test": "test"

rollup.config.js

@@ -0,0 +1,12 @@
+import commonjs from '@rollup/plugin-commonjs';
+import { nodeResolve } from '@rollup/plugin-node-resolve';
+
+export default {
+  input: 'index.js',
+  output: {
+    dir: 'dist',
+    format: 'umd',
+    name: 'namehash'
+  },
+  plugins: [commonjs(), nodeResolve({ browser: true })]
+};

test/index.js

@@ -1,5 +1,5 @@
 const test = require('tape')
-const namehash = require('../')
+const namehash = require('../dist')
 
 // Test results specified in original ENS Proposal:
 // https://github.com/ethereum/EIPs/issues/137
@@ -52,3 +52,10 @@ test('normalize international domain', (t) => {
   t.equal(output, expected)
 })
 
+test('normalize emoji domain', (t) => {
+  t.plan(1)
+  const input = '🦚.eth'
+  const expected = 'xn--qt9h.eth'
+  const output = namehash.normalize(input)
+  t.equal(output, expected)
+})