refactor: change the return type of OnEvent::on_event to `crate::Re…

…sult<()>`

src/messages/events/client/clear_activity.rs

@@ -8,5 +8,7 @@ impl ClearActivityEvent {
 }
 
 impl OnEvent for ClearActivityEvent {
-    fn on_event(self, _ctx: &EventContext) {}
+    fn on_event(self, _ctx: &EventContext) -> crate::Result<()> {
+        Ok(())
+    }
 }

src/messages/events/client/connect.rs

@@ -8,5 +8,7 @@ impl ConnectEvent {
 }
 
 impl OnEvent for ConnectEvent {
-    fn on_event(self, _ctx: &EventContext) {}
+    fn on_event(self, _ctx: &EventContext) -> crate::Result<()> {
+        Ok(())
+    }
 }

src/messages/events/client/disconnect.rs

@@ -8,5 +8,7 @@ impl DisconnectEvent {
 }
 
 impl OnEvent for DisconnectEvent {
-    fn on_event(self, _ctx: &EventContext) {}
+    fn on_event(self, _ctx: &EventContext) -> crate::Result<()> {
+        Ok(())
+    }
 }

src/messages/events/client/initialize.rs

@@ -7,7 +7,9 @@ pub struct InitializeEvent {
 }
 
 impl OnEvent for InitializeEvent {
-    fn on_event(self, _ctx: &EventContext) {}
+    fn on_event(self, _ctx: &EventContext) -> crate::Result<()> {
+        Ok(())
+    }
 }
 
 impl InitializeEvent {

src/messages/events/client/mod.rs

@@ -75,7 +75,7 @@ impl ClientEvent {
 }
 
 impl OnEvent for ClientEvent {
-    fn on_event(self, ctx: &EventContext) {
+    fn on_event(self, ctx: &EventContext) -> crate::Result<()> {
         match self {
             Self::Initialize(e) => e.on_event(ctx),
             Self::Connect(e) => e.on_event(ctx),

src/messages/events/client/reset_timestamp.rs

@@ -4,5 +4,7 @@ use crate::messages::events::event::{EventContext, OnEvent};
 pub struct ResetTimestampEvent;
 
 impl OnEvent for ResetTimestampEvent {
-    fn on_event(self, _ctx: &EventContext) {}
+    fn on_event(self, _ctx: &EventContext) -> crate::Result<()> {
+        Ok(())
+    }
 }

src/messages/events/client/update_activity.rs

@@ -13,5 +13,7 @@ impl UpdateActivityEvent {
 }
 
 impl OnEvent for UpdateActivityEvent {
-    fn on_event(self, _ctx: &EventContext) {}
+    fn on_event(self, _ctx: &EventContext) -> crate::Result<()> {
+        Ok(())
+    }
 }

src/messages/events/client/update_workspace.rs

@@ -12,5 +12,7 @@ impl UpdateWorkspaceEvent {
 }
 
 impl OnEvent for UpdateWorkspaceEvent {
-    fn on_event(self, _ctx: &EventContext) {}
+    fn on_event(self, _ctx: &EventContext) -> crate::Result<()> {
+        Ok(())
+    }
 }

src/messages/events/event.rs

@@ -17,11 +17,11 @@ pub struct EventContext<'a> {
 }
 
 pub trait OnEvent {
-    fn on_event(self, ctx: &EventContext);
+    fn on_event(self, ctx: &EventContext) -> crate::Result<()>;
 }
 
 impl OnEvent for Event {
-    fn on_event(self, ctx: &EventContext) {
+    fn on_event(self, ctx: &EventContext) -> crate::Result<()> {
         match self {
             Event::Client(e) => e.on_event(ctx),
             Event::Local(e) => e.on_event(ctx),

src/messages/events/local/client_disconnected.rs

@@ -4,5 +4,7 @@ use crate::messages::events::event::{EventContext, OnEvent};
 pub struct ClientDisconnectedEvent;
 
 impl OnEvent for ClientDisconnectedEvent {
-    fn on_event(self, _ctx: &EventContext) {}
+    fn on_event(self, _ctx: &EventContext) -> crate::Result<()> {
+        Ok(())
+    }
 }

src/messages/events/local/error.rs

@@ -14,5 +14,7 @@ impl ErrorEvent {
 }
 
 impl OnEvent for ErrorEvent {
-    fn on_event(self, _ctx: &EventContext) {}
+    fn on_event(self, _ctx: &EventContext) -> crate::Result<()> {
+        Ok(())
+    }
 }

src/messages/events/local/mod.rs

@@ -13,7 +13,7 @@ pub enum LocalEvent {
 }
 
 impl OnEvent for LocalEvent {
-    fn on_event(self, ctx: &EventContext) {
+    fn on_event(self, ctx: &EventContext) -> crate::Result<()> {
         match self {
             Self::ClientDisconnected(e) => e.on_event(ctx),
             Self::Error(e) => e.on_event(ctx),

src/messages/events/server/log.rs

@@ -6,8 +6,9 @@ pub struct LogEvent {
 use crate::messages::events::event::{EventContext, OnEvent};
 
 impl OnEvent for LogEvent {
-    fn on_event(self, _ctx: &EventContext) {
+    fn on_event(self, _ctx: &EventContext) -> crate::Result<()> {
         println!("{}", self.message);
+        Ok(())
     }
 }
 

src/messages/events/server/mod.rs

@@ -13,7 +13,7 @@ pub enum ServerEvent {
 }
 
 impl OnEvent for ServerEvent {
-    fn on_event(self, ctx: &EventContext) {
+    fn on_event(self, ctx: &EventContext) -> crate::Result<()> {
         match self {
             Self::Ready(e) => e.on_event(ctx),
             Self::Log(e) => e.on_event(ctx),

src/messages/events/server/ready.rs

@@ -12,10 +12,10 @@ use crate::{
 pub struct ReadyEvent;
 
 impl OnEvent for ReadyEvent {
-    fn on_event(self, ctx: &EventContext) {
-        if let Ok(json) = Json::serialize(&self) {
-            let _ = ctx.pipe.broadcast(json.as_bytes());
-        }
+    fn on_event(self, ctx: &EventContext) -> crate::Result<()> {
+        ctx.pipe.broadcast(Json::serialize(&self)?.as_bytes())?;
+
+        Ok(())
     }
 }