fixes test for oauth ser deser

Author: dingfeli

crates/chat-cli/src/auth/builder_id.rs

@@ -58,14 +58,34 @@ use crate::database::secret_store::{
     SecretStore,
 };
 
-#[derive(Debug, Copy, Clone, PartialEq, Eq, serde::Serialize, serde::Deserialize)]
+#[derive(Debug, Copy, Clone, PartialEq, Eq, serde::Deserialize)]
 pub enum OAuthFlow {
     DeviceCode,
     // This must remain backwards compatible
     #[serde(alias = "PKCE")]
     Pkce,
 }
 
+// Implement Serialize manually to ensure proper serialization
+impl serde::Serialize for OAuthFlow {
+    fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error>
+    where
+        S: serde::Serializer,
+    {
+        match *self {
+            OAuthFlow::DeviceCode => serializer.serialize_str("DeviceCode"),
+            OAuthFlow::Pkce => serialize_pkce(serializer),
+        }
+    }
+}
+
+fn serialize_pkce<S>(serializer: S) -> Result<S::Ok, S::Error>
+where
+    S: serde::Serializer,
+{
+    serializer.serialize_str("PKCE")
+}
+
 impl std::fmt::Display for OAuthFlow {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
         match *self {

crates/chat-cli/src/cli/chat/conversation_state.rs

@@ -895,7 +895,6 @@ mod tests {
     };
     use crate::cli::chat::tool_manager::ToolManager;
     use crate::database::Database;
-    use crate::platform::Env;
 
     fn assert_conversation_state_invariants(state: FigConversationState, assertion_iteration: usize) {
         if let Some(Some(msg)) = state.history.as_ref().map(|h| h.first()) {