claudeup 6.2.1 → 6.3.0

package/src/services/alias-store.ts

@@ -19,18 +19,18 @@ import { ALIAS_FLAGS, type AliasFlag } from "../data/alias-flags.js";
  * Per-flag value shapes. Discriminator is `kind`, mirroring `AliasFlag.kind`.
  */
 export type FlagValue =
-  | { kind: "boolean"; enabled: boolean }
-  | { kind: "tri-state"; state: "unset" | "on" | "off" }
-  | { kind: "select"; enabled: boolean; value: string }
-  | { kind: "text"; enabled: boolean; value: string }
-  | { kind: "optional-text"; enabled: boolean; value: string }
-  | { kind: "text-list"; enabled: boolean; values: string[] }
-  | {
-      kind: "multi-with-custom";
-      enabled: boolean;
-      picked: string[];
-      custom: string[];
-    };
+	| { kind: "boolean"; enabled: boolean }
+	| { kind: "tri-state"; state: "unset" | "on" | "off" }
+	| { kind: "select"; enabled: boolean; value: string }
+	| { kind: "text"; enabled: boolean; value: string }
+	| { kind: "optional-text"; enabled: boolean; value: string }
+	| { kind: "text-list"; enabled: boolean; values: string[] }
+	| {
+			kind: "multi-with-custom";
+			enabled: boolean;
+			picked: string[];
+			custom: string[];
+	  };
 
 /**
  * In-memory alias configuration. No `version` field since this isn't
@@ -38,13 +38,13 @@ export type FlagValue =
  * and flag values come from the rc file.
  */
 export interface AliasConfig {
-  /**
-   * Name of the shell alias. Default `"c"` so users can keep typing the
-   * vanilla `claude` command alongside the wrapped one.
-   */
-  aliasName: string;
-  /** Per-flag state, keyed by `AliasFlag.id`. */
-  flags: Record<string, FlagValue>;
+	/**
+	 * Name of the shell alias. Default `"c"` so users can keep typing the
+	 * vanilla `claude` command alongside the wrapped one.
+	 */
+	aliasName: string;
+	/** Per-flag state, keyed by `AliasFlag.id`. */
+	flags: Record<string, FlagValue>;
 }
 
 /** Default alias name. POSIX/fish-safe, short, and not in conflict with `claude`. */
@@ -59,12 +59,12 @@ export const DEFAULT_ALIAS_NAME = "c";
  * set, and avoids the need to quote the name on either side of the `=`.
  */
 export function validateAliasName(name: string): string | null {
-  if (name.length === 0) return "Alias name cannot be empty.";
-  if (name.length > 32) return "Alias name is unreasonably long (>32 chars).";
-  if (!/^[A-Za-z_][A-Za-z0-9_-]*$/.test(name)) {
-    return "Alias name must start with a letter or underscore and contain only letters, digits, _ or -.";
-  }
-  return null;
+	if (name.length === 0) return "Alias name cannot be empty.";
+	if (name.length > 32) return "Alias name is unreasonably long (>32 chars).";
+	if (!/^[A-Za-z_][A-Za-z0-9_-]*$/.test(name)) {
+		return "Alias name must start with a letter or underscore and contain only letters, digits, _ or -.";
+	}
+	return null;
 }
 
 /**
@@ -72,11 +72,11 @@ export function validateAliasName(name: string): string | null {
  * the rc file (every flag disabled, default alias name).
  */
 export function defaultAliasConfig(): AliasConfig {
-  const flags: Record<string, FlagValue> = {};
-  for (const flag of ALIAS_FLAGS) {
-    flags[flag.id] = defaultValueFor(flag);
-  }
-  return { aliasName: DEFAULT_ALIAS_NAME, flags };
+	const flags: Record<string, FlagValue> = {};
+	for (const flag of ALIAS_FLAGS) {
+		flags[flag.id] = defaultValueFor(flag);
+	}
+	return { aliasName: DEFAULT_ALIAS_NAME, flags };
 }
 
 // ─── Channels ⊇ dev-load (derive-on-display) ───────────────────────────
@@ -99,7 +99,7 @@ export const DEVLOAD_FLAG_ID = "dangerously-load-development-channels";
 
 /** Read the value list off a flag value, or `[]` for non-list kinds. */
 export function listValues(v: FlagValue | undefined): string[] {
-  return v && v.kind === "text-list" ? v.values : [];
+	return v && v.kind === "text-list" ? v.values : [];
 }
 
 /**
@@ -108,11 +108,11 @@ export function listValues(v: FlagValue | undefined): string[] {
  * flag is enabled (a disabled dev-load contributes nothing to the alias).
  */
 export function derivedChannelValues(
-  flags: Record<string, FlagValue>,
+	flags: Record<string, FlagValue>,
 ): string[] {
-  const devload = flags[DEVLOAD_FLAG_ID];
-  if (!devload || devload.kind !== "text-list" || !devload.enabled) return [];
-  return devload.values;
+	const devload = flags[DEVLOAD_FLAG_ID];
+	if (!devload || devload.kind !== "text-list" || !devload.enabled) return [];
+	return devload.values;
 }
 
 /**
@@ -123,50 +123,50 @@ export function derivedChannelValues(
  * it stops emitting. Pure — leaves `channels.values` untouched.
  */
 export function withoutDerivedChannel(
-  flags: Record<string, FlagValue>,
-  value: string,
+	flags: Record<string, FlagValue>,
+	value: string,
 ): Record<string, FlagValue> {
-  const cur = flags[DEVLOAD_FLAG_ID];
-  if (!cur || cur.kind !== "text-list") return flags;
-  const values = cur.values.filter((v) => v !== value);
-  return {
-    ...flags,
-    [DEVLOAD_FLAG_ID]: {
-      kind: "text-list",
-      enabled: values.length > 0 && cur.enabled,
-      values,
-    },
-  };
+	const cur = flags[DEVLOAD_FLAG_ID];
+	if (!cur || cur.kind !== "text-list") return flags;
+	const values = cur.values.filter((v) => v !== value);
+	return {
+		...flags,
+		[DEVLOAD_FLAG_ID]: {
+			kind: "text-list",
+			enabled: values.length > 0 && cur.enabled,
+			values,
+		},
+	};
 }
 
 export function defaultValueFor(flag: AliasFlag): FlagValue {
-  switch (flag.kind) {
-    case "boolean":
-      return { kind: "boolean", enabled: false };
-    case "tri-state":
-      return { kind: "tri-state", state: "unset" };
-    case "select":
-      return {
-        kind: "select",
-        enabled: false,
-        value: flag.options?.[0]?.value ?? "",
-      };
-    case "text":
-      return { kind: "text", enabled: false, value: "" };
-    case "optional-text":
-      return { kind: "optional-text", enabled: false, value: "" };
-    case "text-list":
-      return {
-        kind: "text-list",
-        enabled: false,
-        values: [...(flag.defaultValues ?? [])],
-      };
-    case "multi-with-custom":
-      return {
-        kind: "multi-with-custom",
-        enabled: false,
-        picked: [],
-        custom: [...(flag.defaultValues ?? [])],
-      };
-  }
+	switch (flag.kind) {
+		case "boolean":
+			return { kind: "boolean", enabled: false };
+		case "tri-state":
+			return { kind: "tri-state", state: "unset" };
+		case "select":
+			return {
+				kind: "select",
+				enabled: false,
+				value: flag.options?.[0]?.value ?? "",
+			};
+		case "text":
+			return { kind: "text", enabled: false, value: "" };
+		case "optional-text":
+			return { kind: "optional-text", enabled: false, value: "" };
+		case "text-list":
+			return {
+				kind: "text-list",
+				enabled: false,
+				values: [...(flag.defaultValues ?? [])],
+			};
+		case "multi-with-custom":
+			return {
+				kind: "multi-with-custom",
+				enabled: false,
+				picked: [],
+				custom: [...(flag.defaultValues ?? [])],
+			};
+	}
 }

package/src/services/claude-settings.ts

@@ -1,19 +1,19 @@
-import fs from "fs-extra";
-import path from "node:path";
 import os from "node:os";
+import path from "node:path";
+import fs from "fs-extra";
 import semver from "semver";
 import type {
-	ClaudeSettings,
 	ClaudeLocalSettings,
-	McpServerConfig,
-	MarketplaceSource,
+	ClaudeSettings,
 	DiscoveredMarketplace,
-	InstalledPluginsRegistry,
 	InstalledPluginEntry,
+	InstalledPluginsRegistry,
+	MarketplaceSource,
+	McpServerConfig,
 } from "../types/index.js";
 import { parsePluginId } from "../utils/string-utils.js";
-import { inheritablePaths } from "./git-worktree.js";
 import { repoHeadSha } from "./content-drift.js";
+import { inheritablePaths } from "./git-worktree.js";
 
 const CLAUDE_DIR = ".claude";
 const SETTINGS_FILE = "settings.json";
@@ -143,6 +143,7 @@ export async function addMcpServer(
 	// Extract env vars from config - they go to settings.local.json, not .mcp.json
 	const envVars = config.env || {};
 	const configWithoutEnv: McpServerConfig = { ...config };
+	// biome-ignore lint/performance/noDelete: this object is serialised into .mcp.json; the env key must be absent, not present as undefined
 	delete configWithoutEnv.env;
 
 	// Add to .mcp.json (without env vars)
@@ -437,13 +438,18 @@ export async function removeLocalInstalledPluginVersion(
 // or { type: "command", command: "..." }. Writing a bare string causes a validation error
 // that skips the entire settings file.
 
-function wrapStatusLine(template: string): { type: string; template: string } | undefined {
+function wrapStatusLine(
+	template: string,
+): { type: string; template: string } | undefined {
 	if (!template) return undefined;
 	return { type: "template", template };
 }
 
 function unwrapStatusLine(
-	value: string | { type: string; template?: string; command?: string } | undefined,
+	value:
+		| string
+		| { type: string; template?: string; command?: string }
+		| undefined,
 ): string | undefined {
 	if (!value) return undefined;
 	if (typeof value === "string") return value; // legacy format
@@ -459,6 +465,7 @@ export async function setStatusLine(
 	if (wrapped) {
 		settings.statusLine = wrapped;
 	} else {
+		// biome-ignore lint/performance/noDelete: settings is written back to settings.json; the key must be removed from the file, and readers check `Object.keys`/`in`
 		delete settings.statusLine;
 	}
 	await writeSettings(settings, projectPath);
@@ -478,6 +485,7 @@ export async function setGlobalStatusLine(template: string): Promise<void> {
 	if (wrapped) {
 		settings.statusLine = wrapped;
 	} else {
+		// biome-ignore lint/performance/noDelete: settings is written back to settings.json; the key must be removed from the file, and readers check `Object.keys`/`in`
 		delete settings.statusLine;
 	}
 	await writeGlobalSettings(settings);
@@ -678,7 +686,9 @@ export async function saveGlobalInstalledPluginVersion(
  */
 export async function gapFillInstalledPluginVersions(
 	projectPath?: string,
-): Promise<Array<{ pluginId: string; oldVersion: string; newVersion: string }>> {
+): Promise<
+	Array<{ pluginId: string; oldVersion: string; newVersion: string }>
+> {
 	const globalVersions = await getGlobalInstalledPluginVersions();
 
 	// Collect versions from project and local scopes
@@ -749,8 +759,7 @@ export async function gapFillInstalledPluginVersions(
 	// Apply bumps in a single write
 	if (bumped.length > 0) {
 		const settings = await readGlobalSettings();
-		settings.installedPluginVersions =
-			settings.installedPluginVersions || {};
+		settings.installedPluginVersions = settings.installedPluginVersions || {};
 		for (const { pluginId, newVersion } of bumped) {
 			settings.installedPluginVersions[pluginId] = newVersion;
 		}
@@ -1048,10 +1057,10 @@ async function decodeProjectDirName(encoded: string): Promise<string | null> {
 
 	// Build path greedily: at each step, try joining the next segment with a dash first
 	// (preserving directory names like "circl-infra"), fall back to slash (new path segment)
-	let current = "/" + segments[0];
+	let current = `/${segments[0]}`;
 	for (let i = 1; i < segments.length; i++) {
-		const withDash = current + "-" + segments[i];
-		const withSlash = current + "/" + segments[i];
+		const withDash = `${current}-${segments[i]}`;
+		const withSlash = `${current}/${segments[i]}`;
 		// Prefer dash (keeps compound names intact) if that path prefix exists
 		if (await fs.pathExists(withDash)) {
 			current = withDash;
@@ -1388,6 +1397,7 @@ export async function cleanupExtraKnownMarketplaces(
 				changed &&
 				Object.keys(globalSettings.extraKnownMarketplaces).length === 0
 			) {
+				// biome-ignore lint/performance/noDelete: the empty block is removed from settings.json entirely; an undefined value would survive Object.keys and `in` checks
 				delete globalSettings.extraKnownMarketplaces;
 			}
 			if (changed) {
@@ -1417,6 +1427,7 @@ export async function cleanupExtraKnownMarketplaces(
 					changed &&
 					Object.keys(projectSettings.extraKnownMarketplaces).length === 0
 				) {
+					// biome-ignore lint/performance/noDelete: the empty block is removed from settings.json entirely; an undefined value would survive Object.keys and `in` checks
 					delete projectSettings.extraKnownMarketplaces;
 				}
 				if (changed) {
@@ -1452,9 +1463,7 @@ async function readKnownMarketplaces(): Promise<KnownMarketplaces> {
  * enough: an entry with a missing directory means the clone was lost; a
  * directory without an entry means Claude Code doesn't know about it.
  */
-export async function isMarketplaceRegistered(
-	name: string,
-): Promise<boolean> {
+export async function isMarketplaceRegistered(name: string): Promise<boolean> {
 	const known = await readKnownMarketplaces();
 	if (!known[name]) return false;
 	const marketplacesDir = path.join(

package/src/services/content-drift.ts

@@ -50,9 +50,15 @@ export function clearContentDriftCache(): void {
 	cache.clear();
 }
 
-function git(cwd: string, args: string[]): Promise<{ code: number; out: string }> {
+function git(
+	cwd: string,
+	args: string[],
+): Promise<{ code: number; out: string }> {
 	return new Promise((resolve) => {
-		const child = spawn("git", args, { cwd, stdio: ["ignore", "pipe", "ignore"] });
+		const child = spawn("git", args, {
+			cwd,
+			stdio: ["ignore", "pipe", "ignore"],
+		});
 		let out = "";
 		child.stdout.on("data", (d) => {
 			out += String(d);
@@ -154,7 +160,11 @@ export async function hasContentDrift(q: DriftQuery): Promise<boolean> {
 	const rel = (q.sourcePath ?? `plugins/${q.pluginName}`).replace(/^\.\//, "");
 
 	// The recorded commit must exist locally, or the diff is meaningless.
-	const known = await git(repo, ["cat-file", "-e", `${q.installedSha}^{commit}`]);
+	const known = await git(repo, [
+		"cat-file",
+		"-e",
+		`${q.installedSha}^{commit}`,
+	]);
 	if (known.code !== 0) {
 		cache.set(key, false);
 		return false;