feat(feedback): address feedback, add index.web.js, clean up test implementation

Author: jonluca

README.md

@@ -97,9 +97,7 @@ and you can call it in your code like this:
 import init, { decompress } from "brotli-wasm/pkg.web/brotli_wasm";
 
 const initPromise = init("brotli_wasm_bg.wasm");
-export const brotliDecompress = zlib.brotliDecompress
-    ? promisify(zlib.brotliDecompress)
-    : async (buffer: Uint8Array): Promise<Uint8Array | undefined> => {
+export const brotliDecompress = async (buffer: Uint8Array): Promise<Uint8Array | undefined> => {
           try {
               await initPromise;
               const output = decompress(buffer);

index.web.d.ts

@@ -0,0 +1,5 @@
+import { InitInput, InitOutput } from './pkg.web/brotli_wasm';
+import * as BrotliTypes from './pkg.web/brotli_wasm';
+export default function init(
+    module_or_path?: InitInput | Promise<InitInput>
+): Promise<typeof BrotliTypes>;

index.web.js

@@ -0,0 +1,5 @@
+// The web version needs to be explicitly initiated, but we want the API to be the same
+// as the bundler version, so we write this small wrapper to make it work.
+import init, * as brotliWasm from "./pkg.web/brotli_wasm";
+
+export default initOpts => init(initOpts).then(() => brotliWasm);

package.json

@@ -10,7 +10,9 @@
     "pkg.node",
     "pkg.bundler",
     "pkg.web",
-    "index.browser.js"
+    "index.browser.js",
+    "index.web.d.ts",
+    "index.web.js"
   ],
   "scripts": {
     "build": "node ./build.js",

test/brotli.spec.ts

@@ -8,29 +8,27 @@ if (typeof global.TextEncoder === 'undefined') {
 
 import { expect } from 'chai';
 import * as brotliPromise from '..';
-import init from '../pkg.web/brotli_wasm';
-import * as brotliWeb from '../pkg.web/brotli_wasm';
+import init from '../index.web.js';
 import * as fs from 'fs';
 import * as path from 'path';
 
 const webWasmPath = path.join(__dirname, '../pkg.web/brotli_wasm_bg.wasm');
 
+const bundlerName = 'bundler';
+const webName = 'web (vite compatible)';
+const canReadFile = typeof fs.readFileSync !== 'undefined';
+const initOpts = canReadFile ? fs.readFileSync(webWasmPath) : undefined;
 describe('Brotli-wasm', () => {
     let brotli!: typeof import('..');
-    for (let i = 0; i < 2; i++) {
-        let bundlerToUse = i === 0 ? 'bundler' : 'web (vite compatible)';
+
+    for (const bundlerToUse of [bundlerName, webName]) {
         let bundlerTestName = `Brotli ${bundlerToUse}`;
         describe(bundlerTestName, function () {
             beforeEach(async () => {
-                if (i === 0) {
+                if (bundlerToUse === bundlerName) {
                     brotli = await brotliPromise;
                 } else {
-                    brotli = brotliWeb;
-                }
-                if (typeof fs.readFileSync === 'undefined') {
-                    await init();
-                } else {
-                    await init(fs.readFileSync(webWasmPath));
+                    brotli = await init(initOpts);
                 }
             });