claudeup 6.2.1 → 6.3.0

package/src/__tests__/alias-store.test.ts

@@ -1,163 +1,163 @@
-import { describe, it, expect } from "bun:test";
+import { describe, expect, it } from "bun:test";
+import { ALIAS_FLAGS } from "../data/alias-flags";
 import {
-  defaultAliasConfig,
-  defaultValueFor,
-  validateAliasName,
-  derivedChannelValues,
-  withoutDerivedChannel,
-  listValues,
-  CHANNELS_FLAG_ID,
-  DEVLOAD_FLAG_ID,
-  DEFAULT_ALIAS_NAME,
-  type FlagValue,
+	CHANNELS_FLAG_ID,
+	DEFAULT_ALIAS_NAME,
+	DEVLOAD_FLAG_ID,
+	type FlagValue,
+	defaultAliasConfig,
+	defaultValueFor,
+	derivedChannelValues,
+	listValues,
+	validateAliasName,
+	withoutDerivedChannel,
 } from "../services/alias-store";
-import { ALIAS_FLAGS } from "../data/alias-flags";
 
 function textList(values: string[], enabled = true): FlagValue {
-  return { kind: "text-list", enabled, values };
+	return { kind: "text-list", enabled, values };
 }
 
 describe("defaultAliasConfig", () => {
-  it("seeds every known flag with a sane default and the default alias name", () => {
-    const c = defaultAliasConfig();
-    expect(c.aliasName).toBe(DEFAULT_ALIAS_NAME);
-    expect(c.aliasName).toBe("c");
-    expect(c.flags["dangerously-skip-permissions"]).toEqual({
-      kind: "boolean",
-      enabled: false,
-    });
-    expect(c.flags["chrome"]).toEqual({ kind: "tri-state", state: "unset" });
-    expect(c.flags["effort"]).toEqual({
-      kind: "select",
-      enabled: false,
-      value: "low",
-    });
-    expect(c.flags["dangerously-load-development-channels"]).toEqual({
-      kind: "text-list",
-      enabled: false,
-      values: ["plugin:claudish@magus"],
-    });
-    expect(c.flags["debug"]).toEqual({
-      kind: "multi-with-custom",
-      enabled: false,
-      picked: [],
-      custom: [],
-    });
-  });
-
-  it("covers every flag in the catalog", () => {
-    const c = defaultAliasConfig();
-    for (const flag of ALIAS_FLAGS) {
-      expect(c.flags[flag.id]).toBeDefined();
-    }
-  });
+	it("seeds every known flag with a sane default and the default alias name", () => {
+		const c = defaultAliasConfig();
+		expect(c.aliasName).toBe(DEFAULT_ALIAS_NAME);
+		expect(c.aliasName).toBe("c");
+		expect(c.flags["dangerously-skip-permissions"]).toEqual({
+			kind: "boolean",
+			enabled: false,
+		});
+		expect(c.flags.chrome).toEqual({ kind: "tri-state", state: "unset" });
+		expect(c.flags.effort).toEqual({
+			kind: "select",
+			enabled: false,
+			value: "low",
+		});
+		expect(c.flags["dangerously-load-development-channels"]).toEqual({
+			kind: "text-list",
+			enabled: false,
+			values: ["plugin:claudish@magus"],
+		});
+		expect(c.flags.debug).toEqual({
+			kind: "multi-with-custom",
+			enabled: false,
+			picked: [],
+			custom: [],
+		});
+	});
+
+	it("covers every flag in the catalog", () => {
+		const c = defaultAliasConfig();
+		for (const flag of ALIAS_FLAGS) {
+			expect(c.flags[flag.id]).toBeDefined();
+		}
+	});
 });
 
 describe("defaultValueFor", () => {
-  it("returns the right default for each kind", () => {
-    const ide = ALIAS_FLAGS.find((f) => f.id === "ide")!;
-    expect(defaultValueFor(ide)).toEqual({ kind: "boolean", enabled: false });
-
-    const chrome = ALIAS_FLAGS.find((f) => f.id === "chrome")!;
-    expect(defaultValueFor(chrome)).toEqual({
-      kind: "tri-state",
-      state: "unset",
-    });
-
-    const debug = ALIAS_FLAGS.find((f) => f.id === "debug")!;
-    expect(defaultValueFor(debug)).toEqual({
-      kind: "multi-with-custom",
-      enabled: false,
-      picked: [],
-      custom: [],
-    });
-  });
+	it("returns the right default for each kind", () => {
+		const ide = ALIAS_FLAGS.find((f) => f.id === "ide")!;
+		expect(defaultValueFor(ide)).toEqual({ kind: "boolean", enabled: false });
+
+		const chrome = ALIAS_FLAGS.find((f) => f.id === "chrome")!;
+		expect(defaultValueFor(chrome)).toEqual({
+			kind: "tri-state",
+			state: "unset",
+		});
+
+		const debug = ALIAS_FLAGS.find((f) => f.id === "debug")!;
+		expect(defaultValueFor(debug)).toEqual({
+			kind: "multi-with-custom",
+			enabled: false,
+			picked: [],
+			custom: [],
+		});
+	});
 });
 
 describe("derivedChannelValues (derive-on-display model)", () => {
-  it("returns dev-load values when the dev-load flag is enabled", () => {
-    const flags: Record<string, FlagValue> = {
-      [DEVLOAD_FLAG_ID]: textList(["plugin:claudish@magus"], true),
-      [CHANNELS_FLAG_ID]: textList([], false),
-    };
-    expect(derivedChannelValues(flags)).toEqual(["plugin:claudish@magus"]);
-  });
-
-  it("returns nothing when the dev-load flag is DISABLED", () => {
-    const flags: Record<string, FlagValue> = {
-      [DEVLOAD_FLAG_ID]: textList(["plugin:claudish@magus"], false),
-      [CHANNELS_FLAG_ID]: textList([], false),
-    };
-    expect(derivedChannelValues(flags)).toEqual([]);
-  });
-
-  it("returns nothing when the dev-load flag is absent", () => {
-    expect(derivedChannelValues({})).toEqual([]);
-  });
-
-  it("does NOT read from channels.values (provenance stays separate)", () => {
-    // channels holds OWN values; dev-load is the only derived source.
-    const flags: Record<string, FlagValue> = {
-      [CHANNELS_FLAG_ID]: textList(["plugin:own@m"], true),
-      [DEVLOAD_FLAG_ID]: textList(["plugin:dev@m"], true),
-    };
-    expect(derivedChannelValues(flags)).toEqual(["plugin:dev@m"]);
-    // channels.values is untouched — own only.
-    expect(listValues(flags[CHANNELS_FLAG_ID])).toEqual(["plugin:own@m"]);
-  });
+	it("returns dev-load values when the dev-load flag is enabled", () => {
+		const flags: Record<string, FlagValue> = {
+			[DEVLOAD_FLAG_ID]: textList(["plugin:claudish@magus"], true),
+			[CHANNELS_FLAG_ID]: textList([], false),
+		};
+		expect(derivedChannelValues(flags)).toEqual(["plugin:claudish@magus"]);
+	});
+
+	it("returns nothing when the dev-load flag is DISABLED", () => {
+		const flags: Record<string, FlagValue> = {
+			[DEVLOAD_FLAG_ID]: textList(["plugin:claudish@magus"], false),
+			[CHANNELS_FLAG_ID]: textList([], false),
+		};
+		expect(derivedChannelValues(flags)).toEqual([]);
+	});
+
+	it("returns nothing when the dev-load flag is absent", () => {
+		expect(derivedChannelValues({})).toEqual([]);
+	});
+
+	it("does NOT read from channels.values (provenance stays separate)", () => {
+		// channels holds OWN values; dev-load is the only derived source.
+		const flags: Record<string, FlagValue> = {
+			[CHANNELS_FLAG_ID]: textList(["plugin:own@m"], true),
+			[DEVLOAD_FLAG_ID]: textList(["plugin:dev@m"], true),
+		};
+		expect(derivedChannelValues(flags)).toEqual(["plugin:dev@m"]);
+		// channels.values is untouched — own only.
+		expect(listValues(flags[CHANNELS_FLAG_ID])).toEqual(["plugin:own@m"]);
+	});
 });
 
 describe("withoutDerivedChannel (delete-through to dev-load)", () => {
-  it("removes the value from dev-load, NOT from channels.values", () => {
-    const flags: Record<string, FlagValue> = {
-      [CHANNELS_FLAG_ID]: textList(["plugin:own@m"], true),
-      [DEVLOAD_FLAG_ID]: textList(["plugin:dev@m", "plugin:keep@m"], true),
-    };
-    const next = withoutDerivedChannel(flags, "plugin:dev@m");
-    expect(listValues(next[DEVLOAD_FLAG_ID])).toEqual(["plugin:keep@m"]);
-    // channels.values untouched — the deletion landed on the source flag.
-    expect(listValues(next[CHANNELS_FLAG_ID])).toEqual(["plugin:own@m"]);
-  });
-
-  it("disables dev-load when its last value is removed", () => {
-    const flags: Record<string, FlagValue> = {
-      [DEVLOAD_FLAG_ID]: textList(["plugin:only@m"], true),
-    };
-    const next = withoutDerivedChannel(flags, "plugin:only@m");
-    expect(next[DEVLOAD_FLAG_ID]).toEqual({
-      kind: "text-list",
-      enabled: false,
-      values: [],
-    });
-  });
-
-  it("is a no-op when the value isn't present", () => {
-    const flags: Record<string, FlagValue> = {
-      [DEVLOAD_FLAG_ID]: textList(["plugin:a@m"], true),
-    };
-    expect(listValues(withoutDerivedChannel(flags, "nope")[DEVLOAD_FLAG_ID])).toEqual([
-      "plugin:a@m",
-    ]);
-  });
+	it("removes the value from dev-load, NOT from channels.values", () => {
+		const flags: Record<string, FlagValue> = {
+			[CHANNELS_FLAG_ID]: textList(["plugin:own@m"], true),
+			[DEVLOAD_FLAG_ID]: textList(["plugin:dev@m", "plugin:keep@m"], true),
+		};
+		const next = withoutDerivedChannel(flags, "plugin:dev@m");
+		expect(listValues(next[DEVLOAD_FLAG_ID])).toEqual(["plugin:keep@m"]);
+		// channels.values untouched — the deletion landed on the source flag.
+		expect(listValues(next[CHANNELS_FLAG_ID])).toEqual(["plugin:own@m"]);
+	});
+
+	it("disables dev-load when its last value is removed", () => {
+		const flags: Record<string, FlagValue> = {
+			[DEVLOAD_FLAG_ID]: textList(["plugin:only@m"], true),
+		};
+		const next = withoutDerivedChannel(flags, "plugin:only@m");
+		expect(next[DEVLOAD_FLAG_ID]).toEqual({
+			kind: "text-list",
+			enabled: false,
+			values: [],
+		});
+	});
+
+	it("is a no-op when the value isn't present", () => {
+		const flags: Record<string, FlagValue> = {
+			[DEVLOAD_FLAG_ID]: textList(["plugin:a@m"], true),
+		};
+		expect(
+			listValues(withoutDerivedChannel(flags, "nope")[DEVLOAD_FLAG_ID]),
+		).toEqual(["plugin:a@m"]);
+	});
 });
 
 describe("validateAliasName", () => {
-  it("accepts safe identifiers", () => {
-    expect(validateAliasName("c")).toBeNull();
-    expect(validateAliasName("claude")).toBeNull();
-    expect(validateAliasName("cc-yolo_2")).toBeNull();
-    expect(validateAliasName("_underscore")).toBeNull();
-  });
-
-  it("rejects empty, leading-digit, and special-char names", () => {
-    expect(validateAliasName("")).toMatch(/empty/);
-    expect(validateAliasName("9live")).toMatch(/letter/);
-    expect(validateAliasName("c laude")).toMatch(/letters/);
-    expect(validateAliasName("c'")).toMatch(/letters/);
-    expect(validateAliasName("c=d")).toMatch(/letters/);
-  });
-
-  it("rejects pathologically long names", () => {
-    expect(validateAliasName("a".repeat(64))).toMatch(/long/);
-  });
+	it("accepts safe identifiers", () => {
+		expect(validateAliasName("c")).toBeNull();
+		expect(validateAliasName("claude")).toBeNull();
+		expect(validateAliasName("cc-yolo_2")).toBeNull();
+		expect(validateAliasName("_underscore")).toBeNull();
+	});
+
+	it("rejects empty, leading-digit, and special-char names", () => {
+		expect(validateAliasName("")).toMatch(/empty/);
+		expect(validateAliasName("9live")).toMatch(/letter/);
+		expect(validateAliasName("c laude")).toMatch(/letters/);
+		expect(validateAliasName("c'")).toMatch(/letters/);
+		expect(validateAliasName("c=d")).toMatch(/letters/);
+	});
+
+	it("rejects pathologically long names", () => {
+		expect(validateAliasName("a".repeat(64))).toMatch(/long/);
+	});
 });