fix: convert createProfile's raw ZodError throw into ConfigValidationError

createProfile called ConfigProfileSchema.parse() directly on the
--extends list, so a malformed entry (e.g. an empty name produced by
a stray double-comma in --extends "a,,b") crashed with a raw ZodError
stack trace instead of the clean message every other config-write
path in this codebase gives.

createProfile now safeParses and throws ConfigValidationError,
matching readJson/applyPatch/loadConfigFile's existing pattern.

Author: Mearman

src/configProfiles.test.ts

@@ -49,6 +49,10 @@ describe("configProfiles", () => {
       createProfile(paths, "base");
       expect(() => createProfile(paths, "base")).toThrow(ProfileAlreadyExistsError);
     });
+
+    it("throws ConfigValidationError, not a raw ZodError, for an extends list containing an empty name (e.g. from a stray `,,` in --extends)", () => {
+      expect(() => createProfile(paths, "work", ["base", "", "other"])).toThrow(ConfigValidationError);
+    });
   });
 
   describe("readProfile", () => {

src/configProfiles.ts

@@ -16,6 +16,7 @@ import {
 } from "./config/schema";
 import { collectBoolPairs } from "./cli/parsers";
 import { CliError } from "./cliError";
+import { ConfigValidationError } from "./config/load";
 import { realPromptsPort, runProfileWizard } from "./configure";
 import type { LayoutPaths } from "./paths";
 
@@ -64,17 +65,21 @@ export function readProfile(paths: LayoutPaths, name: string): ConfigProfile | u
 }
 
 /**
- * Creates a new, empty configuration profile (optionally extending others). Throws `ProfileAlreadyExistsError` if a profile with this name already has a file.
+ * Creates a new, empty configuration profile (optionally extending others). Throws `ProfileAlreadyExistsError` if a profile with this name already has a file, and `ConfigValidationError` when `extendsList` fails `ConfigProfileSchema` (e.g. contains an empty name from a stray `,,` in `--extends`), rather than letting the underlying `ZodError` escape as an unhandled crash.
  */
 export function createProfile(paths: LayoutPaths, name: string, extendsList?: readonly string[]): ConfigProfile {
   if (profileExists(paths, name)) {
     throw new ProfileAlreadyExistsError(name);
   }
-  const profile = ConfigProfileSchema.parse(
+  const filePath = profileJsonPath(paths, name);
+  const parsed = ConfigProfileSchema.safeParse(
     extendsList !== undefined && extendsList.length > 0 ? { extends: [...extendsList] } : {},
   );
-  writeJsonAtomic(profileJsonPath(paths, name), profile);
-  return profile;
+  if (!parsed.success) {
+    throw new ConfigValidationError(filePath, parsed.error.issues);
+  }
+  writeJsonAtomic(filePath, parsed.data);
+  return parsed.data;
 }
 
 /** One profile as reported by `listProfiles`. */