Updated with requested changes for error message and IArray

docs/02-guides/goodies.md

@@ -7,7 +7,7 @@ Tyrian comes with a number of handy functions built-in that you can make use of
 These nuggets of functionality are used as commands.
 
 - `Dom` - A few methods such as `focus` and `blur` to manipulate the DOM. Inspired by the Elm [Browser.Dom](https://package.elm-lang.org/packages/elm/browser/latest/Browser.Dom) package.
-- `FileReader` - Given the id of a file input field that has had a file selected, this Cmd will read either raw bytes, an image or text file to return a `Vector[Byte]` or an `HTMLImageElement` or `String` respectively.
+- `FileReader` - Given the id of a file input field that has had a file selected, this Cmd will read either raw bytes, an image or text file to return a `IArray[Byte]` or an `HTMLImageElement` or `String` respectively.
 - `Http` - Make HTTP requests that return their responses as a message.
 - `ImageLoader` - Given a path, this cmd will load an image and return an `HTMLImageElement` for you to make use of.
 - `LocalStorage` - Allows you to save and load to/from your browsers local storage.

sandbox/src/main/scala/example/Sandbox.scala

@@ -345,7 +345,7 @@ object Sandbox extends TyrianIOApp[Msg, Model]:
 
     case Msg.ReadBytesFile(file) =>
       val cmd: Cmd[IO, Msg] =
-        FileReader.readBytes(file)((r: FileReader.Result[Vector[Byte]]) =>
+        FileReader.readBytes(file)((r: FileReader.Result[IArray[Byte]]) =>
           r match {
             case FileReader.Result.File(_, _, d) =>
               Msg.FileBytesRead(d)
@@ -642,7 +642,7 @@ object Sandbox extends TyrianIOApp[Msg, Model]:
                   .map(data => p(data))
                   .toList ++
                 model.loadedBytes
-                  .map(data => p(s"Read ${data.length} bytes"))
+                  .map(data => p(s"Read ${data.length} bytes in an IArray"))
                   .toList
             )
           )
@@ -757,7 +757,7 @@ enum Msg:
   case FileTextRead(fileData: String)
   case SelectBytesFile
   case ReadBytesFile(file: dom.File)
-  case FileBytesRead(fileData: Vector[Byte])
+  case FileBytesRead(fileData: IArray[Byte])
   case NoOp
 
 enum Status:
@@ -809,7 +809,7 @@ final case class Model(
     fruitInput: String,
     loadedImage: Option[String],
     loadedText: Option[String],
-    loadedBytes: Option[Vector[Byte]]
+    loadedBytes: Option[IArray[Byte]]
 )
 
 final case class Fruit(name: String, available: Boolean)

tyrian/js/src/main/scala/tyrian/cmds/FileReader.scala

@@ -12,7 +12,7 @@ import scala.scalajs.js
 import scala.scalajs.js.typedarray
 
 /** Given the id of a file input field that has had a file selected, this Cmd will read either raw bytes, an image or
-  * text file to return a `Vector[Byte]` or an `HTMLImageElement` or `String` respectively.
+  * text file to return a `IArray[Byte]` or an `HTMLImageElement` or `String` respectively.
   */
 object FileReader:
 
@@ -57,22 +57,22 @@ object FileReader:
     readFile(file, ReadType.AsText)(cast andThen resultToMessage)
 
   /** Reads an input file from an input field as bytes */
-  def readBytes[F[_]: Async, Msg](inputFieldId: String)(resultToMessage: Result[Vector[Byte]] => Msg): Cmd[F, Msg] =
-    val cast: Result[js.Any] => Result[Vector[Byte]] =
+  def readBytes[F[_]: Async, Msg](inputFieldId: String)(resultToMessage: Result[IArray[Byte]] => Msg): Cmd[F, Msg] =
+    val cast: Result[js.Any] => Result[IArray[Byte]] =
       case Result.Error(msg) => Result.Error(msg)
       case Result.File(n, p, d) =>
-        try Result.File(n, p, new typedarray.Int8Array(d.asInstanceOf[typedarray.ArrayBuffer]).toVector)
-        catch case _ => Result.Error("File is not bytes")
+        try Result.File(n, p, IArray.from(new typedarray.Int8Array(d.asInstanceOf[typedarray.ArrayBuffer])))
+        catch case _ => Result.Error("Could not cast loaded file data to byte array")
 
     readFromInputField(inputFieldId, ReadType.AsArrayBuffer)(cast andThen resultToMessage)
 
   /** Reads an input file as bytes */
-  def readBytes[F[_]: Async, Msg](file: dom.File)(resultToMessage: Result[Vector[Byte]] => Msg): Cmd[F, Msg] =
-    val cast: Result[js.Any] => Result[Vector[Byte]] =
+  def readBytes[F[_]: Async, Msg](file: dom.File)(resultToMessage: Result[IArray[Byte]] => Msg): Cmd[F, Msg] =
+    val cast: Result[js.Any] => Result[IArray[Byte]] =
       case Result.Error(msg) => Result.Error(msg)
       case Result.File(n, p, d) =>
-        try Result.File(n, p, new typedarray.Int8Array(d.asInstanceOf[typedarray.ArrayBuffer]).toVector)
-        catch case _ => Result.Error("File is not bytes")
+        try Result.File(n, p, IArray.from(new typedarray.Int8Array(d.asInstanceOf[typedarray.ArrayBuffer])))
+        catch case _ => Result.Error("Could not cast loaded file data to byte array")
 
     readFile(file, ReadType.AsArrayBuffer)(cast andThen resultToMessage)