Remove unused AssistantThreadFeedback event (#27021)

Joseph T. Lyons created

It looks like:

- https://github.com/zed-industries/zed/pull/26780

accidentally added a new event type, `AssistantThreadFeedback`, using
the old event system, that it didn't end up actually using, as the code
actually relies on using the newer (preferred) `telemetry::event!()`.

Release Notes:

- N/A

Change summary

crates/collab/src/api/events.rs                 |  4 ---
crates/telemetry_events/src/telemetry_events.rs | 21 ------------------
2 files changed, 25 deletions(-)

Detailed changes

crates/collab/src/api/events.rs 🔗

@@ -660,10 +660,6 @@ fn for_snowflake(
                 e.event_type.clone(),
                 serde_json::to_value(&e.event_properties).unwrap(),
             ),
-            Event::AssistantThreadFeedback(e) => (
-                "Assistant Feedback".to_string(),
-                serde_json::to_value(&e).unwrap(),
-            ),
         };
 
         if let serde_json::Value::Object(ref mut map) = event_properties {

crates/telemetry_events/src/telemetry_events.rs 🔗

@@ -97,7 +97,6 @@ pub enum Event {
     InlineCompletionRating(InlineCompletionRatingEvent),
     Call(CallEvent),
     Assistant(AssistantEvent),
-    AssistantThreadFeedback(AssistantThreadFeedbackEvent),
     Cpu(CpuEvent),
     Memory(MemoryEvent),
     App(AppEvent),
@@ -231,26 +230,6 @@ pub struct ReplEvent {
     pub repl_session_id: String,
 }
 
-#[derive(Clone, Debug, PartialEq, Serialize, Deserialize)]
-pub enum ThreadFeedbackRating {
-    Positive,
-    Negative,
-}
-
-#[derive(Clone, Debug, PartialEq, Serialize, Deserialize)]
-pub struct AssistantThreadFeedbackEvent {
-    /// Unique identifier for the thread
-    pub thread_id: String,
-    /// The feedback rating (thumbs up or thumbs down)
-    pub rating: ThreadFeedbackRating,
-    /// The serialized thread data containing messages, tool calls, etc.
-    pub thread_data: serde_json::Value,
-    /// The initial project snapshot taken when the thread was created
-    pub initial_project_snapshot: serde_json::Value,
-    /// The final project snapshot taken when the thread was first saved
-    pub final_project_snapshot: serde_json::Value,
-}
-
 #[derive(Clone, Debug, PartialEq, Serialize, Deserialize)]
 pub struct BacktraceFrame {
     pub ip: usize,