Merge pull request #143 from InsForge/feat/posthog-funnel-analytics

feat(posthog): track cli_posthog_invoked on posthog setup

Author: CarmenDou

src/commands/posthog/setup.test.ts

@@ -14,6 +14,12 @@ const configMock = vi.hoisted(() => ({
 }));
 vi.mock('../../lib/config.js', () => configMock);
 
+const analyticsMock = vi.hoisted(() => ({
+  trackPosthog: vi.fn(),
+  shutdownAnalytics: vi.fn(async () => {}),
+}));
+vi.mock('../../lib/analytics.js', () => analyticsMock);
+
 vi.mock('../../lib/prompts.js', () => ({ isInteractive: false }));
 
 // `open` is loaded dynamically inside runConnectFlow; mock so the real browser
@@ -81,6 +87,8 @@ beforeEach(() => {
   outputMock.outputJson.mockReset();
   outputMock.outputSuccess.mockReset();
   clackNoteMock.mockReset();
+  analyticsMock.trackPosthog.mockReset();
+  analyticsMock.shutdownAnalytics.mockClear();
   configMock.getProjectConfig.mockReturnValue({ project_id: 'p1', project_name: 'Test Project' });
   configMock.getAccessToken.mockReturnValue('tok');
 });
@@ -175,4 +183,32 @@ describe('posthog setup', () => {
       expect(payload.wizardCommand).toMatch(/^npx(\.cmd)? -y @posthog\/wizard@latest$/);
     });
   });
+
+  describe('analytics', () => {
+    it('fires cli_posthog_invoked with the project config and flushes on success', async () => {
+      apiMock.startPosthogCliFlow.mockResolvedValue({ type: 'connected' });
+      apiMock.fetchPosthogConnection.mockResolvedValue({
+        kind: 'connected',
+        connection: { apiKey: 'phc_', host: 'h', posthogProjectId: '1' },
+      });
+
+      await runSetup(['--skip-browser']);
+
+      expect(analyticsMock.trackPosthog).toHaveBeenCalledOnce();
+      expect(analyticsMock.trackPosthog).toHaveBeenCalledWith(
+        'setup',
+        expect.objectContaining({ project_id: 'p1' }),
+      );
+      expect(analyticsMock.shutdownAnalytics).toHaveBeenCalledOnce();
+    });
+
+    it('still flushes analytics when setup fails', async () => {
+      apiMock.startPosthogCliFlow.mockResolvedValue({ type: 'connected' });
+      apiMock.fetchPosthogConnection.mockResolvedValue({ kind: 'not-connected' });
+
+      await runSetup(['--skip-browser']);
+
+      expect(analyticsMock.shutdownAnalytics).toHaveBeenCalledOnce();
+    });
+  });
 });

src/commands/posthog/setup.ts

@@ -16,6 +16,7 @@ import {
   startPosthogCliFlow,
 } from '../../lib/api/posthog.js';
 import { outputJson, outputSuccess } from '../../lib/output.js';
+import { trackPosthog, shutdownAnalytics } from '../../lib/analytics.js';
 
 const POLL_INTERVAL_MS = 2000;
 const POLL_TIMEOUT_MS = 15 * 60 * 1000;
@@ -52,6 +53,8 @@ export function registerPosthogSetupCommand(program: Command): void {
         }
       } catch (err) {
         handleError(err, json);
+      } finally {
+        await shutdownAnalytics();
       }
     });
 }
@@ -83,6 +86,8 @@ async function runSetup(opts: RunSetupOpts): Promise<SetupResult> {
     throw new AuthError('Not logged in. Run `insforge login` first.');
   }
 
+  trackPosthog('setup', proj);
+
   if (!opts.json) {
     clack.intro('PostHog setup');
     outputSuccess(`Linked to InsForge project: ${proj.project_name} (${proj.project_id})`);

src/lib/analytics.ts

@@ -61,6 +61,24 @@ export function trackPayments(
   });
 }
 
+// Step 2 of the "dashboard connect → CLI posthog setup" funnel; pair with
+// backend `posthog_connect_started` joined on project_id.
+export function trackPosthog(
+  subcommand: string,
+  config: ProjectConfig,
+  properties?: Record<string, unknown>,
+): void {
+  captureEvent(config.project_id, 'cli_posthog_invoked', {
+    subcommand,
+    project_id: config.project_id,
+    project_name: config.project_name,
+    org_id: config.org_id,
+    region: config.region,
+    oss_mode: config.project_id === FAKE_PROJECT_ID,
+    ...properties,
+  });
+}
+
 // Config commands (apply/plan/export) operate against an OSS backend and may
 // run without a linked cloud project, so the ProjectConfig is optional.
 // Pure-OSS runs fall back to FAKE_PROJECT_ID as the distinct ID — same