fix: upgrade posthog-node to v5 to fix missing telemetry events (#28)

posthog-node v4's `captureImmediate` had a bug where the `.then()` chain
didn't return the promise from `captureStatelessImmediate`, causing the
HTTP request to be fire-and-forget. Combined with trpc-cli calling
`process.exit()` after the handler completes, telemetry events were
silently dropped before the request could finish.

This upgrades posthog-node from 4.18.0 to 5.28.2 where the bug is fixed,
and removes the deprecated `captureMode` and `persistence` options that
no longer exist in v5.

Author: AmanVarshney01

bun.lock

@@ -52,7 +52,7 @@
     "execa": "^9.6.1",
     "fs-extra": "^11.3.3",
     "handlebars": "^4.7.8",
-    "posthog-node": "4.18.0",
+    "posthog-node": "^5.28.2",
     "trpc-cli": "^0.12.4",
     "zod": "^4.3.6"
   },

package.json

@@ -117,23 +117,24 @@ export async function trackCliTelemetry(
 
   let client: PostHog | undefined;
   try {
+    const distinctId = await getAnonymousId();
+    const sanitizedProperties = sanitizeProperties({
+      ...getCommonProperties(),
+      ...properties,
+      $process_person_profile: false,
+    });
+
     client = new PostHog(TELEMETRY_API_KEY, {
       host: TELEMETRY_HOST,
-      captureMode: "json",
       disableGeoip: true,
       flushAt: 1,
       flushInterval: 0,
-      persistence: "memory",
     });
 
     await client.captureImmediate({
-      distinctId: await getAnonymousId(),
+      distinctId,
       event,
-      properties: sanitizeProperties({
-        ...getCommonProperties(),
-        ...properties,
-        $process_person_profile: false,
-      }),
+      properties: sanitizedProperties,
       disableGeoip: true,
     });
   } catch {