@bytecodealliance/preview2-shim 0.17.0 → 0.17.1

package/lib/browser/cli.js

@@ -29,16 +29,19 @@ export const environment = {
 };
 
 class ComponentExit extends Error {
-  constructor(ok) {
-    super(`Component exited ${ok ? 'successfully' : 'with error'}`);
+  constructor(code) {
+    super(`Component exited ${code === 0 ? 'successfully' : 'with error'}`);
     this.exitError = true;
-    this.ok = ok;
+    this.code = code;
   }
 }
 
 export const exit = {
   exit (status) {
-    throw new ComponentExit(status.tag === 'err' ? true : false);
+    throw new ComponentExit(status.tag === 'err' ? 1 : 0);
+  },
+  exitWithCode (code) {
+    throw new ComponentExit(code);
   }
 };
 

package/lib/io/worker-socket-tcp.js

@@ -31,7 +31,7 @@ import {
 } from "./worker-sockets.js";
 import { Socket, Server } from "node:net";
 
-const winOrMac = process.platform === 'win32' || process.platform === 'darwin';
+const win = process.platform === 'win32';
 
 /**
  * @typedef {import("../../types/interfaces/wasi-sockets-network.js").IpSocketAddress} IpSocketAddress
@@ -268,7 +268,7 @@ export function socketTcpGetRemoteAddress(id) {
 export function socketTcpShutdown(id, _shutdownType) {
   const socket = tcpSockets.get(id);
   if (socket.state !== SOCKET_STATE_CONNECTION) throw "invalid-state";
-  if (winOrMac && socket.tcpSocket.destroySoon)
+  if (win && socket.tcpSocket.destroySoon)
     socket.tcpSocket.destroySoon();
   else
     socket.tcpSocket.destroy();

package/lib/io/worker-thread.js

@@ -393,8 +393,9 @@ function handle(call, id, payload) {
       return;
     }
     case HTTP_OUTGOING_BODY_DISPOSE:
-      if (!streams.delete(id))
-        throw new Error("wasi-io trap: stream not found to dispose");
+      if (debug && !streams.has(id))
+        console.warn(`wasi-io: stream ${id} not found to dispose`);
+      streams.delete(id);  
       return;
     case HTTP_SERVER_START:
       return startHttpServer(id, payload);

package/lib/nodejs/cli.js

@@ -47,6 +47,9 @@ export const exit = {
   exit(status) {
     process.exit(status.tag === "err" ? 1 : 0);
   },
+  exitWithCode(code) {
+    process.exit(code);
+  }
 };
 
 // Stdin is created as a FILE descriptor

package/lib/nodejs/filesystem.js

@@ -368,19 +368,16 @@ class Descriptor {
       }
     }
     try {
-      const fd = openSync(fullPath, fsOpenFlags);
+      const fd = openSync(fullPath.endsWith('/') ? fullPath.slice(0, -1) : fullPath, fsOpenFlags);
       const descriptor = descriptorCreate(
         fd,
         descriptorFlags,
         fullPath,
         preopenEntries
       );
-      if (fullPath.endsWith("/") && isWindows) {
-        // check if its a directory
-        if (descriptor.getType() !== "directory") {
-          descriptor[symbolDispose]();
-          throw "not-directory";
-        }
+      if (fullPath.endsWith('/') && descriptor.getType() !== 'directory') {
+        descriptor[symbolDispose]();
+        throw "not-directory";
       }
       return descriptor;
     } catch (e) {
@@ -671,6 +668,7 @@ function convertFsError(e) {
     case "ENOSPC":
       return "insufficient-space";
     case "ENOTDIR":
+    case 'ERR_FS_EISDIR':
       return "not-directory";
     case "ENOTEMPTY":
       return "not-empty";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bytecodealliance/preview2-shim",
-  "version": "0.17.0",
+  "version": "0.17.1",
   "description": "WASI Preview2 shim for JS environments",
   "author": "Guy Bedford, Eduardo Rodrigues<16357187+eduardomourar@users.noreply.github.com>",
   "type": "module",

package/types/interfaces/wasi-filesystem-types.d.ts

@@ -569,12 +569,6 @@ export class Descriptor {
   /**
   * Open a file or directory.
   * 
-  * The returned descriptor is not guaranteed to be the lowest-numbered
-  * descriptor not currently open/ it is randomized to prevent applications
-  * from depending on making assumptions about indexes, since this is
-  * error-prone in multi-threaded contexts. The returned descriptor is
-  * guaranteed to be less than 2**31.
-  * 
   * If `flags` contains `descriptor-flags::mutate-directory`, and the base
   * descriptor doesn't have `descriptor-flags::mutate-directory` set,
   * `open-at` fails with `error-code::read-only`.

package/types/interfaces/wasi-http-types.d.ts

@@ -255,7 +255,7 @@ export interface ErrorCodeInternalError {
  */
 export type HeaderError = HeaderErrorInvalidSyntax | HeaderErrorForbidden | HeaderErrorImmutable;
 /**
- * This error indicates that a `field-key` or `field-value` was
+ * This error indicates that a `field-name` or `field-value` was
  * syntactically invalid when used with an operation that sets headers in a
  * `fields`.
  */
@@ -263,7 +263,7 @@ export interface HeaderErrorInvalidSyntax {
   tag: 'invalid-syntax',
 }
 /**
- * This error indicates that a forbidden `field-key` was used when trying
+ * This error indicates that a forbidden `field-name` was used when trying
  * to set a header in a `fields`.
  */
 export interface HeaderErrorForbidden {
@@ -278,8 +278,22 @@ export interface HeaderErrorImmutable {
 }
 /**
  * Field keys are always strings.
+ * 
+ * Field keys should always be treated as case insensitive by the `fields`
+ * resource for the purposes of equality checking.
+ * 
+ * # Deprecation
+ * 
+ * This type has been deprecated in favor of the `field-name` type.
  */
 export type FieldKey = string;
+/**
+ * Field names are always strings.
+ * 
+ * Field names should always be treated as case insensitive by the `fields`
+ * resource for the purposes of equality checking.
+ */
+export type FieldName = FieldKey;
 /**
  * Field values should always be ASCII strings. However, in
  * reality, HTTP implementations often have to interpret malformed values,
@@ -312,68 +326,71 @@ export class Fields {
   * 
   * The resulting `fields` is mutable.
   * 
-  * The list represents each key-value pair in the Fields. Keys
+  * The list represents each name-value pair in the Fields. Names
   * which have multiple values are represented by multiple entries in this
-  * list with the same key.
+  * list with the same name.
   * 
-  * The tuple is a pair of the field key, represented as a string, and
+  * The tuple is a pair of the field name, represented as a string, and
   * Value, represented as a list of bytes.
   * 
-  * An error result will be returned if any `field-key` or `field-value` is
+  * An error result will be returned if any `field-name` or `field-value` is
   * syntactically invalid, or if a field is forbidden.
   */
-  static fromList(entries: Array<[FieldKey, FieldValue]>): Fields;
+  static fromList(entries: Array<[FieldName, FieldValue]>): Fields;
   /**
-  * Get all of the values corresponding to a key. If the key is not present
+  * Get all of the values corresponding to a name. If the name is not present
   * in this `fields` or is syntactically invalid, an empty list is returned.
-  * However, if the key is present but empty, this is represented by a list
+  * However, if the name is present but empty, this is represented by a list
   * with one or more empty field-values present.
   */
-  get(name: FieldKey): Array<FieldValue>;
+  get(name: FieldName): Array<FieldValue>;
   /**
-  * Returns `true` when the key is present in this `fields`. If the key is
+  * Returns `true` when the name is present in this `fields`. If the name is
   * syntactically invalid, `false` is returned.
   */
-  has(name: FieldKey): boolean;
+  has(name: FieldName): boolean;
   /**
-  * Set all of the values for a key. Clears any existing values for that
-  * key, if they have been set.
+  * 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-key` or any of
+  * Fails with `header-error.invalid-syntax` if the `field-name` or any of
   * the `field-value`s are syntactically invalid.
   */
-  set(name: FieldKey, value: Array<FieldValue>): void;
+  set(name: FieldName, value: Array<FieldValue>): void;
   /**
-  * Delete all values for a key. Does nothing if no values for the key
+  * Delete all values for a name. Does nothing if no values for the name
   * exist.
   * 
   * Fails with `header-error.immutable` if the `fields` are immutable.
   * 
-  * Fails with `header-error.invalid-syntax` if the `field-key` is
+  * Fails with `header-error.invalid-syntax` if the `field-name` is
   * syntactically invalid.
   */
-  'delete'(name: FieldKey): void;
+  'delete'(name: FieldName): void;
   /**
-  * Append a value for a key. Does not change or delete any existing
-  * values for that key.
+  * Append a value for a name. Does not change or delete any existing
+  * values for that name.
   * 
   * Fails with `header-error.immutable` if the `fields` are immutable.
   * 
-  * Fails with `header-error.invalid-syntax` if the `field-key` or
+  * Fails with `header-error.invalid-syntax` if the `field-name` or
   * `field-value` are syntactically invalid.
   */
-  append(name: FieldKey, value: FieldValue): void;
+  append(name: FieldName, value: FieldValue): void;
   /**
-  * Retrieve the full set of keys and values in the Fields. Like the
-  * constructor, the list represents each key-value pair.
+  * Retrieve the full set of names and values in the Fields. Like the
+  * constructor, the list represents each name-value pair.
   * 
-  * The outer list represents each key-value pair in the Fields. Keys
+  * The outer list represents each name-value pair in the Fields. Names
   * which have multiple values are represented by multiple entries in this
-  * list with the same key.
+  * list with the same name.
+  * 
+  * The names and values are always returned in the original casing and in
+  * the order in which they will be serialized for transport.
   */
-  entries(): Array<[FieldKey, FieldValue]>;
+  entries(): Array<[FieldName, FieldValue]>;
   /**
   * Make a deep copy of the Fields. Equivalent in behavior to calling the
   * `fields` constructor on the return value of `entries`. The resulting

package/types/interfaces/wasi-io-streams.d.ts

@@ -14,6 +14,9 @@ export type StreamError = StreamErrorLastOperationFailed | StreamErrorClosed;
  * The last operation (a write or flush) failed before completion.
  * 
  * More information is available in the `error` payload.
+ * 
+ * After this, the stream will be closed. All future operations return
+ * `stream-error::closed`.
  */
 export interface StreamErrorLastOperationFailed {
   tag: 'last-operation-failed',

package/types/interfaces/wasi-sockets-ip-name-lookup.d.ts

@@ -53,7 +53,7 @@ export class ResolveAddressStream {
   /**
   * Create a `pollable` which will resolve once the stream is ready for I/O.
   * 
-  * Note: this function is here for WASI Preview2 only.
+  * Note: this function is here for WASI 0.2 only.
   * It's planned to be removed when `future` is natively supported in Preview3.
   */
   subscribe(): Pollable;

package/types/interfaces/wasi-sockets-tcp.d.ts

@@ -332,7 +332,7 @@ export class TcpSocket {
   * See <https://github.com/WebAssembly/wasi-sockets/blob/main/TcpSocketOperationalSemantics.md#pollable-readiness>
   * for more information.
   * 
-  * Note: this function is here for WASI Preview2 only.
+  * Note: this function is here for WASI 0.2 only.
   * It's planned to be removed when `future` is natively supported in Preview3.
   */
   subscribe(): Pollable;

package/types/interfaces/wasi-sockets-udp.d.ts

@@ -82,7 +82,7 @@ export class IncomingDatagramStream {
   /**
   * Create a `pollable` which will resolve once the stream is ready to receive again.
   * 
-  * Note: this function is here for WASI Preview2 only.
+  * Note: this function is here for WASI 0.2 only.
   * It's planned to be removed when `future` is natively supported in Preview3.
   */
   subscribe(): Pollable;
@@ -143,7 +143,7 @@ export class OutgoingDatagramStream {
   /**
   * Create a `pollable` which will resolve once the stream is ready to send again.
   * 
-  * Note: this function is here for WASI Preview2 only.
+  * Note: this function is here for WASI 0.2 only.
   * It's planned to be removed when `future` is natively supported in Preview3.
   */
   subscribe(): Pollable;
@@ -290,7 +290,7 @@ export class UdpSocket {
       /**
       * Create a `pollable` which will resolve once the socket is ready for I/O.
       * 
-      * Note: this function is here for WASI Preview2 only.
+      * Note: this function is here for WASI 0.2 only.
       * It's planned to be removed when `future` is natively supported in Preview3.
       */
       subscribe(): Pollable;