refactor(telemetry): remove package_name and package_version fields (#188)

* fix(docker): recreate wasm stub before final build

* fix(docker): recreate bench stub before final build

* refactor(telemetry): remove package_name and package_version fields

Author: BryanFRD

src/config.rs

@@ -728,7 +728,7 @@ pub fn init(format: Option<ConfigFileFormat>) -> Result<()> {
     println!("Run: ferrflow check");
 
     if config.workspace.anonymous_telemetry {
-        telemetry::send_event(telemetry::EventType::Init, None, None, None, None);
+        telemetry::send_event(telemetry::EventType::Init, None, None);
     }
 
     Ok(())

src/monorepo.rs

@@ -51,7 +51,7 @@ pub fn check(config_path: Option<&Path>, verbose: bool, json: bool) -> Result<()
     let result = run_release_logic(&root, &config, true, verbose, json);
 
     if config.workspace.anonymous_telemetry {
-        telemetry::send_event(telemetry::EventType::Check, None, None, None, None);
+        telemetry::send_event(telemetry::EventType::Check, None, None);
     }
 
     result
@@ -339,8 +339,6 @@ fn run_release_logic(
             if config.workspace.anonymous_telemetry {
                 telemetry::send_event(
                     telemetry::EventType::VersionBump,
-                    Some(&pkg.name),
-                    Some(&new_version),
                     None,
                     Some(commits.len() as i32),
                 );
@@ -532,14 +530,8 @@ fn run_release_logic(
             }
 
             if config.workspace.anonymous_telemetry {
-                for (_, _, _, pkg_name, version, commit_count) in &tags_to_create {
-                    telemetry::send_event(
-                        telemetry::EventType::Release,
-                        Some(pkg_name),
-                        Some(version),
-                        None,
-                        Some(*commit_count),
-                    );
+                for (_, _, _, _, _, commit_count) in &tags_to_create {
+                    telemetry::send_event(telemetry::EventType::Release, None, Some(*commit_count));
                 }
             }
 

src/telemetry.rs

@@ -26,10 +26,6 @@ pub enum EventType {
 struct EventPayload {
     event_type: EventType,
     #[serde(skip_serializing_if = "Option::is_none")]
-    package_name: Option<String>,
-    #[serde(skip_serializing_if = "Option::is_none")]
-    package_version: Option<String>,
-    #[serde(skip_serializing_if = "Option::is_none")]
     metadata: Option<serde_json::Value>,
     #[serde(skip_serializing_if = "Option::is_none")]
     repo_hash: Option<String>,
@@ -75,8 +71,6 @@ fn get_repo_hash() -> Option<String> {
 
 pub fn send_event(
     event_type: EventType,
-    package_name: Option<&str>,
-    package_version: Option<&str>,
     metadata: Option<serde_json::Value>,
     commits_count: Option<i32>,
 ) {
@@ -86,8 +80,6 @@ pub fn send_event(
 
     let payload = EventPayload {
         event_type,
-        package_name: package_name.map(String::from),
-        package_version: package_version.map(String::from),
         metadata,
         repo_hash: get_repo_hash(),
         commits_count,
@@ -154,15 +146,11 @@ mod tests {
     fn payload_skips_none_fields() {
         let payload = EventPayload {
             event_type: EventType::Check,
-            package_name: None,
-            package_version: None,
             metadata: None,
             repo_hash: None,
             commits_count: None,
         };
         let json = serde_json::to_value(&payload).unwrap();
-        assert!(!json.as_object().unwrap().contains_key("package_name"));
-        assert!(!json.as_object().unwrap().contains_key("package_version"));
         assert!(!json.as_object().unwrap().contains_key("metadata"));
         assert!(!json.as_object().unwrap().contains_key("repo_hash"));
         assert!(!json.as_object().unwrap().contains_key("commits_count"));
@@ -172,16 +160,12 @@ mod tests {
     fn payload_includes_present_fields() {
         let payload = EventPayload {
             event_type: EventType::Release,
-            package_name: Some("my-pkg".into()),
-            package_version: Some("1.0.0".into()),
             metadata: None,
             repo_hash: Some("abc123".into()),
             commits_count: Some(42),
         };
         let json = serde_json::to_value(&payload).unwrap();
         assert_eq!(json["event_type"], "release");
-        assert_eq!(json["package_name"], "my-pkg");
-        assert_eq!(json["package_version"], "1.0.0");
         assert_eq!(json["repo_hash"], "abc123");
         assert_eq!(json["commits_count"], 42);
     }