Refactor: Use serde instead of TryFrom<Vec<u8>> traits in IPC (#316)

## 🎟️ Tracking

<!-- Paste the link to the Jira or GitHub issue or otherwise describe /
point to where this change is coming from. -->

## 📔 Objective

Refactoring the IPC to use serde instead. Putting this in a separate PR
to make reviewing easier

## ⏰ Reminders before review

- Contributor guidelines followed
- All formatters and local linters executed and passed
- Written new unit and / or integration tests where applicable
- Protected functional changes with optionality (feature flags)
- Used internationalization (i18n) for all UI strings
- CI builds passed
- Communicated to DevOps any deployment requirements
- Updated any necessary documentation (Confluence, contributing docs) or
informed the documentation
  team

## 🦮 Reviewer guidelines

<!-- Suggested interactions but feel free to use (or not) as you desire!
-->

- 👍 (`:+1:`) or similar for great changes
- 📝 (`:memo:`) or ℹ️ (`:information_source:`) for notes or general info
- ❓ (`:question:`) for questions
- 🤔 (`:thinking:`) or 💭 (`:thought_balloon:`) for more open inquiry
that's not quite a confirmed
  issue and could potentially benefit from discussion
- 🎨 (`:art:`) for suggestions / improvements
- ❌ (`:x:`) or ⚠️ (`:warning:`) for more significant problems or
concerns needing attention
- 🌱 (`:seedling:`) or ♻️ (`:recycle:`) for future improvements or
indications of technical debt
- ⛏ (`:pick:`) for minor or nitpick changes

Author: coroiu

Cargo.lock

@@ -23,6 +23,7 @@ wasm = [
 async-trait = { workspace = true }
 bitwarden-error = { workspace = true }
 bitwarden-threading = { workspace = true }
+erased-serde = "0.4.6"
 js-sys = { workspace = true, optional = true }
 log = { workspace = true }
 serde = { workspace = true }

crates/bitwarden-ipc/Cargo.toml

@@ -1,5 +1,3 @@
-use std::convert::Infallible;
-
 use serde::{Deserialize, Serialize};
 #[cfg(feature = "wasm")]
 use {tsify_next::Tsify, wasm_bindgen::prelude::*};
@@ -23,38 +21,6 @@ impl RpcRequest for DiscoverRequest {
     }
 }
 
-impl TryFrom<DiscoverRequest> for Vec<u8> {
-    type Error = Infallible;
-
-    fn try_from(_value: DiscoverRequest) -> Result<Self, Self::Error> {
-        Ok(vec![])
-    }
-}
-
-impl TryFrom<Vec<u8>> for DiscoverRequest {
-    type Error = Infallible;
-
-    fn try_from(_value: Vec<u8>) -> Result<Self, Self::Error> {
-        Ok(DiscoverRequest)
-    }
-}
-
-impl TryFrom<DiscoverResponse> for Vec<u8> {
-    type Error = serde_json::Error;
-
-    fn try_from(value: DiscoverResponse) -> Result<Self, Self::Error> {
-        serde_json::to_vec(&value)
-    }
-}
-
-impl TryFrom<Vec<u8>> for DiscoverResponse {
-    type Error = serde_json::Error;
-
-    fn try_from(value: Vec<u8>) -> Result<Self, Self::Error> {
-        serde_json::from_slice(&value)
-    }
-}
-
 pub struct DiscoverHandler {
     response: DiscoverResponse,
 }