Merge pull request #32 from ByteVeda/fix/wasm-bytes-serialization

fix(wasm): serialize doc_bytes as Uint8Array instead of Array<number>

Author: pratyush618

crates/paperjam-wasm/Cargo.toml

@@ -12,6 +12,7 @@ crate-type = ["cdylib"]
 paperjam-core = { path = "../paperjam-core", default-features = false, features = ["signatures", "validation"] }
 wasm-bindgen = "0.2"
 serde = { version = "1.0", features = ["derive"] }
+serde_bytes = "0.11"
 serde-wasm-bindgen = "0.6"
 js-sys = "0.3"
 getrandom_v02 = { package = "getrandom", version = "0.2", features = ["js"] }

crates/paperjam-wasm/src/lib.rs

@@ -354,16 +354,16 @@ impl WasmDocument {
         let ranges: Vec<(u32, u32)> = serde_wasm_bindgen::from_value(ranges)
             .map_err(|e| JsValue::from_str(&e.to_string()))?;
         let docs = paperjam_core::manipulation::split(&self.inner, &ranges).map_err(to_js_err)?;
-        let mut byte_arrays: Vec<Vec<u8>> = Vec::new();
+        let result = js_sys::Array::new();
         for doc in docs {
             let mut inner = doc.into_inner();
             let mut buf = Vec::new();
             inner
                 .save_to(&mut buf)
                 .map_err(|e| JsValue::from_str(&e.to_string()))?;
-            byte_arrays.push(buf);
+            result.push(&js_sys::Uint8Array::from(buf.as_slice()));
         }
-        serde_wasm_bindgen::to_value(&byte_arrays).map_err(|e| JsValue::from_str(&e.to_string()))
+        Ok(result.into())
     }
 
     /// Sanitize the document by removing potentially dangerous content.
@@ -396,6 +396,7 @@ impl WasmDocument {
         };
         #[derive(Serialize)]
         struct SanitizeResponse {
+            #[serde(with = "serde_bytes")]
             doc_bytes: Vec<u8>,
             result: SanitizeResultJs,
         }
@@ -432,6 +433,7 @@ impl WasmDocument {
         };
         #[derive(Serialize)]
         struct RedactResponse {
+            #[serde(with = "serde_bytes")]
             doc_bytes: Vec<u8>,
             result: RedactResultJs,
         }