@bytecodealliance/preview2-shim 0.17.9 → 0.18.1

package/lib/browser/filesystem.js

@@ -43,6 +43,22 @@ const timeZero = {
   nanoseconds: 0,
 };
 
+/** Coerce the given object to a safe integer */
+function coerceToSafeIntegerNumber(obj) {
+  let n;
+  if (typeof obj === "number") {
+    n = obj;
+  } else if (typeof obj == "bigint") {
+    n = Number(obj);
+  } else {
+    throw new TypeError(`unexpected non-numeric type: ${obj}`);
+  }
+  if (n > Number.MAX_SAFE_INTEGER) {
+    throw new TypeError(`excessively large number: ${n}`);
+  }
+  return n;
+}
+
 function getChildEntry(parentEntry, subpath, openFlags) {
   if (subpath === "." && _rootPreopen && descriptorGetEntry(_rootPreopen[0]) === parentEntry) {
     subpath = _getCwd();
@@ -188,7 +204,9 @@ class Descriptor {
 
   read(length, offset) {
     const source = getSource(this.#entry);
-    return [source.slice(offset, offset + length), offset + length >= source.byteLength];
+    const off = coerceToSafeIntegerNumber(offset);
+    const len = coerceToSafeIntegerNumber(length);
+    return [source.slice(off, off + len), off + len >= source.byteLength];
   }
 
   write(buffer, offset) {

package/lib/browser/http.js

@@ -46,6 +46,19 @@ class Fields {
     return tableEntries.map(([, v]) => v);
   }
 
+  /**
+   * WIT spec (https://github.com/WebAssembly/WASI/blob/91bb44c3c3a9b1e09187db23c85fa844d1fd6b15/proposals/http/wit/types.wit#L215-L223):
+   *
+   * > Set all of the values for a name. Clears any existing values for that
+   * > name, if they have been set.
+   * >
+   * > Fails with `header-error.immutable` if the `fields` are immutable.
+   * >
+   * > Fails with `header-error.invalid-syntax` if the `field-name` or any of
+   * > the `field-value`s are syntactically invalid.
+   *
+   * The existing-branch splice/reuse is an allocation optimization — values are cleared, not retained.
+   */
   set(name, values) {
     if (this.#immutable) {
       throw { tag: "immutable" };

package/lib/io/worker-sockets.js

@@ -87,7 +87,7 @@ export function socketResolveAddress(name) {
   return lookup(name, dnsLookupOptions).then(
     (addresses) => {
       return addresses.map(({ address, family }) => {
-        [
+        return [
           {
             tag: "ipv" + family,
             val: (family === 4 ? ipv4ToTuple : ipv6ToTuple)(address),

package/lib/nodejs/http.js

@@ -499,6 +499,19 @@ class Fields {
     }
     return tableEntries.map(([, v]) => v);
   }
+  /**
+   * WIT spec (https://github.com/WebAssembly/WASI/blob/91bb44c3c3a9b1e09187db23c85fa844d1fd6b15/proposals/http/wit/types.wit#L215-L223):
+   *
+   * > Set all of the values for a name. Clears any existing values for that
+   * > name, if they have been set.
+   * >
+   * > Fails with `header-error.immutable` if the `fields` are immutable.
+   * >
+   * > Fails with `header-error.invalid-syntax` if the `field-name` or any of
+   * > the `field-value`s are syntactically invalid.
+   *
+   * The existing-branch splice/reuse is an allocation optimization — values are cleared, not retained.
+   */
   set(name, values) {
     if (this.#immutable) {
       throw { tag: "immutable" };
@@ -514,17 +527,19 @@ class Fields {
       } catch {
         throw { tag: "invalid-syntax" };
       }
-      throw { tag: "invalid-syntax" };
     }
     const lowercased = name.toLowerCase();
     if (_forbiddenHeaders.has(lowercased)) {
       throw { tag: "forbidden" };
     }
-    const tableEntries = this.#table.get(lowercased);
+    let tableEntries = this.#table.get(lowercased);
     if (tableEntries) {
       this.#entries = this.#entries.filter((entry) => !tableEntries.includes(entry));
+      tableEntries.splice(0, tableEntries.length);
+    } else {
+      this.#table.set(lowercased, []);
+      tableEntries = this.#table.get(lowercased);
     }
-    tableEntries.splice(0, tableEntries.length);
     for (const value of values) {
       const entry = [name, value];
       this.#entries.push(entry);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bytecodealliance/preview2-shim",
-  "version": "0.17.9",
+  "version": "0.18.1",
   "description": "WASI Preview2 shim for JS environments",
   "homepage": "https://github.com/bytecodealliance/jco#readme",
   "bugs": {
@@ -48,18 +48,18 @@
       "default": "./lib/common/instantiation.js"
     }
   },
+  "devDependencies": {
+    "@bytecodealliance/componentize-js": "0.21.0",
+    "@bytecodealliance/jco": "1.15.2",
+    "mime": "^4.0.7",
+    "puppeteer": "^25.1.0"
+  },
   "scripts": {
     "types:check": "tsc --noEmit",
     "fmt": "oxfmt",
     "fmt:check": "oxfmt --check",
     "lint": "oxlint",
-    "lint:fix": "npm run lint -- --fix",
+    "lint:fix": "oxlint --fix",
     "test": "vitest run -c test/vitest.ts"
-  },
-  "devDependencies": {
-    "@bytecodealliance/componentize-js": "0.19.3",
-    "@bytecodealliance/jco": "1.15.2",
-    "mime": "^4.0.7",
-    "puppeteer": "^24.16.2"
   }
-}
+}