fix: browser field should point to cjs (#20)

> a top-level "browser" field should point to CJS files that, when bundled by a non-broken module bundler, work in a browser

browserify/browser-resolve#101 (comment)

I've changed the `browser` field to point to the `cjs` version of this module in
line with the above comment because this module is currently broken when used with
browserify.

I've also added the [util](https://www.npmjs.com/package/util) module as a dep since it's used
in the code. It would be ignored in most environments so the only cost is a slightly larger
bundle but it's likely to be included in any non-trivial bundle somewhere anyway so there's
likely to be no real-world impact.

Also:

- Adds typescript dep to tests that test typescript, otherwise tests fail with
  'cannot determine executable to run' - unless you have tsc installed globally I guess?
- Fixes a typo in a comment
- Simplifies .gitignore

Fixes #19

Author: achingbrain

.gitignore

@@ -1,3 +1,2 @@
-node_modules/**
-test/**/package-lock.json
-**/node_modules
+node_modules
+package-lock.json

package.json

@@ -10,7 +10,7 @@
   "react-native": "./src/lib.react-native.js",
   "main": "./src/lib.cjs",
   "module": "./src/lib.js",
-  "browser": "./src/lib.js",
+  "browser": "./src/lib.cjs",
   "types": "./src/lib.d.ts",
   "exports": {
     ".": {
@@ -33,6 +33,9 @@
   "license": "MIT",
   "author": "Irakli Gozalishvili <[email protected]>",
   "homepage": "https://github.com/gozala/web-encoding",
+  "dependencies": {
+    "util": "^0.12.3"
+  },
   "devDependencies": {
     "assert": "^2.0.0",
     "jest": "^26.6.3",

src/lib.js

@@ -1,5 +1,5 @@
 // In node `export { TextEncoder }` throws:
-// "Export 'TextEncoder' is not definedin module"
+// "Export 'TextEncoder' is not defined in module"
 // To workaround we first define constants and then export with as.
 const Encoder = TextEncoder
 const Decoder = TextDecoder

test/cjs-use/package.json

@@ -5,6 +5,9 @@
     "web-encoding": "file:../.."
   },
   "scripts": {
-    "test": "npm install && npx typescript --noEmit --allowJs --checkJs src/main.js"
+    "test": "npm install && tsc --noEmit --allowJs --checkJs src/main.js"
+  },
+  "devDependencies": {
+    "typescript": "^4.2.3"
   }
 }

test/esm-use/package.json

@@ -5,6 +5,9 @@
     "web-encoding": "file:../.."
   },
   "scripts": {
-    "test": "npm install && npx typescript --noEmit --allowJs --checkJs src/main.js"
+    "test": "npm install && tsc --noEmit --allowJs --checkJs src/main.js"
+  },
+  "devDependencies": {
+    "typescript": "^4.2.3"
   }
 }

test/ts-use/package.json

@@ -5,6 +5,9 @@
     "web-encoding": "file:../.."
   },
   "scripts": {
-    "test": "npm install && npx typescript --noEmit"
+    "test": "npm install && tsc --noEmit"
+  },
+  "devDependencies": {
+    "typescript": "^4.2.3"
   }
 }