fix(telemetry): emit dev command telemetry before blocking (#1375)

* feat(telemetry): allow dynamic attribute merging

* refactor: avoid killing the process on validation errors

* fix(telemetry): capture dev validation failures by wrapping at top-level

* fix: avoid defaulting to http protocol on telemetry failures

* fix: remove redundant property check

* feat(telemetry): support partial attribute overrides

* refactor: leverage mutable recorder in favor of immutable returns

Author: Hweinstock

integ-tests/dev-server.test.ts

@@ -88,7 +88,7 @@ describe('integration: dev server', () => {
   });
 
   it.skipIf(!hasNpm || !hasGit || !hasUv)(
-    'starts dev server and responds to health check',
+    'starts dev server, responds to health check, and emits telemetry',
     async () => {
       expect(projectPath, 'Project should have been created').toBeTruthy();
 
@@ -98,12 +98,21 @@ describe('integration: dev server', () => {
       devProcess = spawn('node', [cliPath, 'dev', '--port', String(port), '--logs'], {
         cwd: projectPath,
         stdio: 'pipe',
-        env: { ...process.env, INIT_CWD: undefined },
+        env: { ...process.env, INIT_CWD: undefined, ...telemetry.env },
       });
 
       const serverReady = await waitForServer(port, 20000);
       expect(serverReady, 'Dev server should respond to ping within 20s').toBeTruthy();
 
+      // Verify telemetry was emitted for the server startup (before blocking)
+      telemetry.assertMetricEmitted({
+        command: 'dev',
+        dev_action: 'server',
+        ui_mode: 'terminal',
+        exit_reason: 'success',
+      });
+      telemetry.clearEntries();
+
       // Invoke the running server and verify telemetry
       const invokeResult = await runCLI(['dev', 'hello', '--port', String(port)], projectPath, {
         env: telemetry.env,
@@ -135,4 +144,27 @@ describe('integration: dev server', () => {
     },
     30000
   );
+
+  it.skipIf(!hasNpm || !hasGit || !hasUv)(
+    'exits with error when runtime not found and emits failure telemetry',
+    async () => {
+      expect(projectPath, 'Project should have been created').toBeTruthy();
+
+      telemetry.clearEntries();
+      const result = await runCLI(['dev', '--logs', '--runtime', 'nonexistent-agent'], projectPath, {
+        env: telemetry.env,
+      });
+
+      expect(result.exitCode).toBe(1);
+      expect(result.stderr).toContain('nonexistent-agent');
+      expect(result.stderr).toContain('not found');
+
+      telemetry.assertMetricEmitted({
+        command: 'dev',
+        dev_action: 'server',
+        exit_reason: 'failure',
+      });
+    },
+    15000
+  );
 });