bumps telemetry finish timeout to 1 second and surface errors other than timeout

Author: dingfeli

crates/chat-cli/src/cli/chat/tools/custom_tool.rs

@@ -107,7 +107,6 @@ impl CustomToolClient {
     }
 
     pub fn assign_messenger(&mut self, messenger: Box<dyn Messenger>) {
-        tracing::error!("## background: assigned {} with messenger", self.get_server_name());
         match self {
             CustomToolClient::Stdio { client, .. } => {
                 client.messenger = Some(messenger);

crates/chat-cli/src/telemetry/mod.rs

@@ -162,7 +162,9 @@ impl TelemetryThread {
     pub async fn finish(self) -> Result<(), TelemetryError> {
         drop(self.tx);
         if let Some(handle) = self.handle {
-            let _ = tokio::time::timeout(std::time::Duration::from_millis(100), handle).await;
+            if let Err(e) = tokio::time::timeout(std::time::Duration::from_millis(1000), handle).await {
+                return Err(TelemetryError::Timeout(e));
+            }
         }
 
         Ok(())