feat: enrich telemetry with arch, locale, project_types, execution_ms, success, terminal; disable GeoIP

Author: SoftDryzz

src/main/java/pm/ProjectManager.java

@@ -113,6 +113,8 @@ public static void main(String[] args) {
         }
 
         String command = args[0].toLowerCase();
+        long startTime = System.currentTimeMillis();
+        boolean success = true;
 
         try {
             switch (command) {
@@ -146,10 +148,12 @@ public static void main(String[] args) {
                 case "version", "-v", "--version" -> printVersion();
                 default -> handleGenericCommand(command, args);
             }
-            Telemetry.trackCommand(command);
         } catch (Exception e) {
+            success = false;
             handleFatalError(e);
         } finally {
+            long elapsedMs = System.currentTimeMillis() - startTime;
+            Telemetry.trackCommand(command, success, elapsedMs);
             Telemetry.flush();
         }
     }

src/main/java/pm/telemetry/Telemetry.java

@@ -5,6 +5,8 @@
 import pm.util.Constants;
 
 import java.nio.file.Files;
+import java.util.HashMap;
+import java.util.List;
 import java.util.Map;
 import java.util.UUID;
 
@@ -48,21 +50,30 @@ public static void init() {
     /**
      * Tracks a command execution event (fire-and-forget).
      *
-     * @param command The command name (e.g., "build", "test")
+     * @param command     The command name (e.g., "build", "test")
+     * @param success     Whether the command completed without exception
+     * @param elapsedMs   Execution time in milliseconds
      */
-    public static void trackCommand(String command) {
+    public static void trackCommand(String command, boolean success, long elapsedMs) {
         if (config == null || !config.isTelemetryEnabled()) {
             return;
         }
         try {
-            TelemetryEvent event = new TelemetryEvent("command_executed", Map.of(
-                    "command", command,
-                    "version", Constants.VERSION,
-                    "os", System.getProperty("os.name"),
-                    "os_version", System.getProperty("os.version"),
-                    "java_version", System.getProperty("java.version"),
-                    "project_count", getProjectCount()
-            ));
+            Map<String, Object> props = new HashMap<>();
+            props.put("command", command);
+            props.put("version", Constants.VERSION);
+            props.put("os", System.getProperty("os.name"));
+            props.put("os_version", System.getProperty("os.version"));
+            props.put("java_version", System.getProperty("java.version"));
+            props.put("arch", System.getProperty("os.arch"));
+            props.put("locale", java.util.Locale.getDefault().toString());
+            props.put("terminal", System.getenv("TERM") != null ? System.getenv("TERM") : "unknown");
+            props.put("project_count", getProjectCount());
+            props.put("project_types", getProjectTypes());
+            props.put("success", success);
+            props.put("execution_ms", elapsedMs);
+
+            TelemetryEvent event = new TelemetryEvent("command_executed", props);
             TelemetryClient.send(event, Constants.POSTHOG_KEY, config.getDistinctId());
         } catch (Exception ignored) {
             // Telemetry tracking failure is non-critical
@@ -134,16 +145,41 @@ private static void promptConsent() {
      * Reads projects.json directly and counts top-level keys.
      */
     private static int getProjectCount() {
+        try {
+            return loadProjectsMap().size();
+        } catch (Exception e) {
+            return 0;
+        }
+    }
+
+    /**
+     * Extracts unique project types (e.g., ["Maven", "Rust", "Flutter"]).
+     * No project names or paths — only the type field.
+     */
+    private static List<String> getProjectTypes() {
+        try {
+            Map<String, Map<String, Object>> projects = loadProjectsMap();
+            return projects.values().stream()
+                    .map(p -> p.getOrDefault("type", "UNKNOWN").toString())
+                    .distinct()
+                    .sorted()
+                    .toList();
+        } catch (Exception e) {
+            return List.of();
+        }
+    }
+
+    private static Map<String, Map<String, Object>> loadProjectsMap() {
         try {
             if (!Files.exists(Constants.PROJECTS_FILE)) {
-                return 0;
+                return Map.of();
             }
             String json = Files.readString(Constants.PROJECTS_FILE);
-            Map<String, Object> projects = new Gson().fromJson(json,
-                    new TypeToken<Map<String, Object>>() {}.getType());
-            return projects != null ? projects.size() : 0;
+            Map<String, Map<String, Object>> projects = new Gson().fromJson(json,
+                    new TypeToken<Map<String, Map<String, Object>>>() {}.getType());
+            return projects != null ? projects : Map.of();
         } catch (Exception e) {
-            return 0;
+            return Map.of();
         }
     }
 

src/main/java/pm/telemetry/TelemetryEvent.java

@@ -32,6 +32,7 @@ public Map<String, Object> toPostHogPayload(String apiKey, String distinctId) {
 
         Map<String, Object> props = new HashMap<>(properties);
         props.put("$lib", "projectmanager");
+        props.put("$ip", null);
         payload.put("properties", props);
 
         return payload;

src/test/java/pm/telemetry/TelemetryTest.java

@@ -177,7 +177,7 @@ class TrackingTests {
         void trackCommandWithNullConfig() {
             Telemetry.reset();
             // Should not throw
-            assertDoesNotThrow(() -> Telemetry.trackCommand("build"));
+            assertDoesNotThrow(() -> Telemetry.trackCommand("build", true, 100));
         }
 
         @Test