Remove version ID from SSE events

Author: Robert

docs/src/usage/change-notifications.md

@@ -11,7 +11,7 @@ When `AddVersion` accepts a new version for that client, the stream emits a
 
 ```text
 event: version
-data: {"clientId":"...","versionId":"..."}
+data: {"clientId":"..."}
 ```
 
 This endpoint is only an invalidation signal. Clients should perform a normal

server/src/api/add_version.rs

@@ -74,7 +74,7 @@ pub(crate) async fn service(
                         rb.append_header((SNAPSHOT_REQUEST_HEADER, "urgency=high"));
                     }
                 };
-                server_state.changes.notify(client_id, version_id);
+                server_state.changes.notify(client_id);
                 Ok(rb.finish())
             }
             Ok((AddVersionResult::ExpectedParentVersion(parent_version_id), _)) => {
@@ -148,11 +148,9 @@ mod test {
         // the passed parent version ID, at least
         let new_version_id = resp.headers().get("X-Version-Id").unwrap();
         assert!(new_version_id != &version_id.to_string());
-        let new_version_id = Uuid::parse_str(new_version_id.to_str().unwrap()).unwrap();
 
         let event = changes.next().await.unwrap();
         assert_eq!(event.client_id, client_id);
-        assert_eq!(event.version_id, new_version_id);
 
         // Shapshot should be requested, since there is no existing snapshot
         let snapshot_request = resp.headers().get("X-Snapshot-Request").unwrap();

server/src/api/events.rs

@@ -9,13 +9,12 @@ use std::{
     collections::HashMap,
     sync::{Arc, Mutex},
 };
-use taskchampion_sync_server_core::{ClientId, VersionId};
+use taskchampion_sync_server_core::ClientId;
 
 #[derive(Clone, Debug, Serialize)]
 #[serde(rename_all = "camelCase")]
 pub(crate) struct ChangeEvent {
     pub(crate) client_id: ClientId,
-    pub(crate) version_id: VersionId,
 }
 
 #[derive(Clone, Default)]
@@ -35,11 +34,8 @@ impl ChangeNotifier {
         rx
     }
 
-    pub(crate) fn notify(&self, client_id: ClientId, version_id: VersionId) {
-        let event = ChangeEvent {
-            client_id,
-            version_id,
-        };
+    pub(crate) fn notify(&self, client_id: ClientId) {
+        let event = ChangeEvent { client_id };
         let mut subscribers = self
             .subscribers
             .lock()
@@ -90,13 +86,11 @@ mod test {
     async fn notifier_delivers_events_for_matching_client() {
         let notifier = ChangeNotifier::default();
         let client_id = Uuid::new_v4();
-        let version_id = Uuid::new_v4();
         let mut rx = notifier.subscribe(client_id);
 
-        notifier.notify(client_id, version_id);
+        notifier.notify(client_id);
         let event = rx.next().await.unwrap();
         assert_eq!(event.client_id, client_id);
-        assert_eq!(event.version_id, version_id);
     }
 
     #[actix_rt::test]