Use wiggle "trappable error" to implement wasi-common

crates/wasi-common/cap-std-sync/src/sched/unix.rs

@@ -47,7 +47,7 @@ pub async fn poll_oneoff<'a>(poll: &mut Poll<'a>) -> Result<(), Error> {
         match rustix::io::poll(&mut pollfds, poll_timeout) {
             Ok(ready) => break ready,
             Err(rustix::io::Errno::INTR) => continue,
-            Err(err) => return Err(err.into()),
+            Err(err) => return Err(std::io::Error::from(err).into()),
         }
     };
     if ready > 0 {

crates/wasi-common/cap-std-sync/src/stdio.rs

@@ -128,7 +128,9 @@ macro_rules! wasi_file_write_impl {
             }
             async fn write_vectored<'a>(&mut self, bufs: &[io::IoSlice<'a>]) -> Result<u64, Error> {
                 let n = (&*self.0.as_filelike_view::<File>()).write_vectored(bufs)?;
-                Ok(n.try_into().map_err(|c| Error::range().context(c))?)
+                Ok(n.try_into().map_err(|_| {
+                    Error::range().context("converting write_vectored total length")
+                })?)
             }
             async fn write_vectored_at<'a>(
                 &mut self,

crates/wasi-common/src/error.rs

@@ -1,145 +1,15 @@
-//! `wasi_common::Error` is now `anyhow::Error`.
+//! wasi-common uses an [`Error`] type which represents either a preview 1 [`Errno`] enum, on
+//! [`anyhow::Error`] for trapping execution.
 //!
-//! Snapshots (right now only `wasi_common::snapshots::preview_1`) contains
-//! all of the logic for transforming an `Error` into the snapshot's own
-//! `Errno`. They may do so by downcasting the error into any of:
-//! * `std::io::Error` - these are thrown by `std`, `cap_std`, etc for most of
-//! the operations WASI is concerned with.
-//! * `wasi_common::ErrorKind` - these are a subset of the Errnos, and are
-//! constructed directly by wasi-common or an impl rather than coming from the
-//! OS or some library which doesn't know about WASI.
-//! * `wiggle::GuestError`
-//! * `std::num::TryFromIntError`
-//! * `std::str::Utf8Error`
-//! and then applying specialized logic to translate each of those into
-//! `Errno`s.
-//!
-//! The `wasi_common::ErrorExt` trait provides human-friendly constructors for
-//! the `wasi_common::ErrorKind` variants .
-//!
-//! If you throw an error that does not downcast to one of those, it will turn
-//! into a `wiggle::Trap` and terminate execution.
-//!
-//! The real value of using `anyhow::Error` here is being able to use
-//! `anyhow::Result::context` to aid in debugging of errors.
+//! The user can construct an [`Error`] out of an [`Errno`] using the `From`/`Into` traits.
+//! They may also use [`Error::trap`] to construct an error that traps execution. The contents
+//! can be inspected with [`Error::downcast`] and [`Error::downcast_ref`]. Additional context
+//! can be provided with the [`Error::context`] method. This context is only observable with the
+//! `Display` and `Debug` impls of the error.
 
-pub use anyhow::{Context, Error};
+pub use crate::snapshots::preview_1::error::{Errno, Error, ErrorExt};
 use std::fmt;
 
-/// Internal error type for the `wasi-common` crate.
-///
-/// This Contains variants of the WASI `$errno` type that are used internally
-/// by the crate, and which aren't one-to-one with a `std::io::ErrorKind`
-/// error.
-///
-/// When the Rust [io_error_more] feature is stabilized, that will enable
-/// us to replace several more of these codes with `std::io::ErrorKind` codes.
-///
-/// [io_error_more]: https://doc.rust-lang.org/beta/unstable-book/library-features/io-error-more.html
-#[derive(Copy, Clone, Debug, PartialEq, Eq, thiserror::Error)]
-#[non_exhaustive]
-pub enum ErrorKind {
-    /// Errno::TooBig: Argument list too long
-    #[error("TooBig: Argument list too long")]
-    TooBig,
-    /// Errno::Badf: Bad file descriptor
-    #[error("Badf: Bad file descriptor")]
-    Badf,
-    /// Errno::Ilseq: Illegal byte sequence
-    #[error("Ilseq: Illegal byte sequence")]
-    Ilseq,
-    /// Errno::Io: I/O error
-    #[error("Io: I/O error")]
-    Io,
-    /// Errno::Nametoolong: Filename too long
-    #[error("Nametoolong: Filename too long")]
-    Nametoolong,
-    /// Errno::Notdir: Not a directory or a symbolic link to a directory.
-    #[error("Notdir: Not a directory or a symbolic link to a directory")]
-    Notdir,
-    /// Errno::Notsup: Not supported, or operation not supported on socket.
-    #[error("Notsup: Not supported, or operation not supported on socket")]
-    Notsup,
-    /// Errno::Overflow: Value too large to be stored in data type.
-    #[error("Overflow: Value too large to be stored in data type")]
-    Overflow,
-    /// Errno::Range: Result too large
-    #[error("Range: Result too large")]
-    Range,
-    /// Errno::Spipe: Invalid seek
-    #[error("Spipe: Invalid seek")]
-    Spipe,
-    /// Errno::Perm: Permission denied
-    #[error("Permission denied")]
-    Perm,
-}
-
-pub trait ErrorExt {
-    fn trap(msg: impl Into<String>) -> Self;
-    fn not_found() -> Self;
-    fn too_big() -> Self;
-    fn badf() -> Self;
-    fn exist() -> Self;
-    fn illegal_byte_sequence() -> Self;
-    fn invalid_argument() -> Self;
-    fn io() -> Self;
-    fn name_too_long() -> Self;
-    fn not_dir() -> Self;
-    fn not_supported() -> Self;
-    fn overflow() -> Self;
-    fn range() -> Self;
-    fn seek_pipe() -> Self;
-    fn perm() -> Self;
-}
-
-impl ErrorExt for Error {
-    fn trap(msg: impl Into<String>) -> Self {
-        anyhow::anyhow!(msg.into())
-    }
-    fn not_found() -> Self {
-        std::io::Error::from(std::io::ErrorKind::NotFound).into()
-    }
-    fn too_big() -> Self {
-        ErrorKind::TooBig.into()
-    }
-    fn badf() -> Self {
-        ErrorKind::Badf.into()
-    }
-    fn exist() -> Self {
-        std::io::Error::from(std::io::ErrorKind::AlreadyExists).into()
-    }
-    fn illegal_byte_sequence() -> Self {
-        ErrorKind::Ilseq.into()
-    }
-    fn invalid_argument() -> Self {
-        std::io::Error::from(std::io::ErrorKind::InvalidInput).into()
-    }
-    fn io() -> Self {
-        ErrorKind::Io.into()
-    }
-    fn name_too_long() -> Self {
-        ErrorKind::Nametoolong.into()
-    }
-    fn not_dir() -> Self {
-        ErrorKind::Notdir.into()
-    }
-    fn not_supported() -> Self {
-        ErrorKind::Notsup.into()
-    }
-    fn overflow() -> Self {
-        ErrorKind::Overflow.into()
-    }
-    fn range() -> Self {
-        ErrorKind::Range.into()
-    }
-    fn seek_pipe() -> Self {
-        ErrorKind::Spipe.into()
-    }
-    fn perm() -> Self {
-        ErrorKind::Perm.into()
-    }
-}
-
 /// An error returned from the `proc_exit` host syscall.
 ///
 /// Embedders can test if an error returned from wasm is this error, in which

crates/wasi-common/src/lib.rs

@@ -66,7 +66,7 @@ pub use cap_rand::RngCore;
 pub use clocks::{SystemTimeSpec, WasiClocks, WasiMonotonicClock, WasiSystemClock};
 pub use ctx::WasiCtx;
 pub use dir::WasiDir;
-pub use error::{Context, Error, ErrorExt, ErrorKind, I32Exit};
+pub use error::{Error, ErrorExt, I32Exit};
 pub use file::WasiFile;
 pub use sched::{Poll, WasiSched};
 pub use string_array::StringArrayError;