fix: AgentPrimitive TUI try/catch, standardize uses safeParse

Author: Hweinstock

src/cli/primitives/AgentPrimitive.tsx

@@ -354,24 +354,29 @@ export class AgentPrimitive extends BasePrimitive<AddAgentOptions, RemovableReso
             };
           });
         } else {
-          // TUI fallback — dynamic imports to avoid pulling ink (async) into registry
-          requireTTY();
-          const [{ render }, { default: React }, { AddFlow }] = await Promise.all([
-            import('ink'),
-            import('react'),
-            import('../tui/screens/add/AddFlow'),
-          ]);
-          const { clear, unmount } = render(
-            React.createElement(AddFlow, {
-              isInteractive: false,
-              initialResource: 'agent',
-              onExit: () => {
-                clear();
-                unmount();
-                process.exit(0);
-              },
-            })
-          );
+          try {
+            // TUI fallback — dynamic imports to avoid pulling ink (async) into registry
+            requireTTY();
+            const [{ render }, { default: React }, { AddFlow }] = await Promise.all([
+              import('ink'),
+              import('react'),
+              import('../tui/screens/add/AddFlow'),
+            ]);
+            const { clear, unmount } = render(
+              React.createElement(AddFlow, {
+                isInteractive: false,
+                initialResource: 'agent',
+                onExit: () => {
+                  clear();
+                  unmount();
+                  process.exit(0);
+                },
+              })
+            );
+          } catch (error) {
+            console.error(getErrorMessage(error));
+            process.exit(1);
+          }
         }
       });
 

src/cli/telemetry/schemas/common-shapes.ts

@@ -11,7 +11,11 @@ export function safeSchema<T extends Record<string, SafeField>>(shape: T) {
 /** Lowercase a CLI value and parse it through a Zod enum, returning the narrowed type. */
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 export function standardize<T extends z.ZodEnum<any>>(schema: T, value: string): z.infer<T> {
-  return schema.parse(value.toLowerCase()) as z.infer<T>;
+  const lower = value.toLowerCase();
+  const result = schema.safeParse(lower);
+  // If the value doesn't match the enum, return the lowercased value anyway —
+  // recordCommandRun's try/catch will silently drop the invalid metric.
+  return (result.success ? result.data : lower) as z.infer<T>;
 }
 
 // Primitive types