zidane-ui 9.0.4 → 9.1.3

package/src/settings/headless/fixture-performance.ts

@@ -1,350 +0,0 @@
-/**
- * A performance member that runs in memory, for docs, galleries and tests:
- * a believable desktop app (main, a window, the GPU, a language server, the
- * daemon, a shared MCP server, and two sessions: one's terminal running a dev
- * server, the other's test watcher and own MCP server) whose figures drift on
- * a timer while watched, and that stops ticking when nothing
- * watches, like a real host.
- */
-import type {
-  PerfOsThread,
-  PerfProcess,
-  PerfProfileOutcome,
-  PerfProfileTarget,
-  PerfSample,
-  PerfTotals,
-  PerfWatchError,
-  SettingsPerformanceHost,
-} from './performance';
-
-/** Options for {@link createFixturePerformanceHost}. */
-export interface FixturePerformanceOptions {
-  /** Samples already held. Default: 30, ending now. */
-  history?: number;
-  /** Milliseconds between samples while watched; `0` never ticks (tests drive `control.tick`). Default 2000. */
-  intervalMs?: number;
-  /** Profiles answer `busy`. Default: false. */
-  profileBusy?: boolean;
-  /** The history held, instead of the generated one. */
-  samples?: PerfSample[];
-  /** What `threads(pid)` answers per pid, instead of a generic list. */
-  threads?: Record<number, PerfOsThread[]>;
-  /** Where `exportReport` says it wrote. */
-  reportPath?: string;
-  /** What `profile` answers, instead of the generated profile. */
-  profileOutcome?: PerfProfileOutcome;
-  /** Every watch fails with this (a server too old, a token without admin): no samples follow. */
-  watchError?: PerfWatchError;
-}
-
-/** Drives the fixture from outside. */
-export interface FixturePerformanceControl {
-  /** Emits the next generated sample to every watcher. */
-  tick(): PerfSample;
-  /** Emits `sample` to every watcher. */
-  emit(sample: PerfSample): void;
-  /** How many watchers are open. */
-  watchers(): number;
-  /** Counter resets taken. */
-  resets(): number;
-  /** Profiles recorded, in order. */
-  profiles(): { target: PerfProfileTarget; seconds: number }[];
-}
-
-/** The fixture member and its control. */
-export type FixturePerformanceHost = SettingsPerformanceHost & { control: FixturePerformanceControl };
-
-const MB = 1024 * 1024;
-
-/** The fixture's two sessions. */
-const LOGIN = { id: 'fixture-login', title: 'Fix the login redirect' };
-const VITE = { id: 'fixture-vite', title: 'Upgrade to Vite 7' };
-
-const sum = (processes: readonly PerfProcess[]): PerfTotals => ({
-  cpu: processes.reduce((total, p) => total + p.cpu, 0),
-  memory: processes.reduce((total, p) => total + p.memory, 0),
-});
-
-/**
- * Sample `n`, deterministic: a smooth wave per process, a spike every 20 samples.
- *
- * @param n - The sample's index.
- * @param at - Its time.
- */
-export function fixturePerfSample(n: number, at: number): PerfSample {
-  const wave = (period: number, low: number, high: number) =>
-    low + ((Math.sin((n / period) * Math.PI * 2) + 1) / 2) * (high - low);
-  const spike = n % 20 === 0 ? 1 : 0;
-  const processes: PerfSample['processes'] = [
-    {
-      pid: 4100,
-      parent: null,
-      kind: 'main',
-      label: 'Main process',
-      origin: 'app',
-      cpu: wave(9, 2, 9) + spike * 30,
-      memory: 310 * MB,
-    },
-    {
-      pid: 4120,
-      parent: 4100,
-      kind: 'window',
-      origin: 'app',
-      label: 'Window: Zidane',
-      detail: 'file:///Applications/Zidane.app/index.html',
-      cpu: wave(7, 4, 22),
-      memory: 540 * MB,
-    },
-    {
-      pid: 4111,
-      parent: 4100,
-      kind: 'gpu',
-      label: 'GPU process',
-      origin: 'app',
-      cpu: wave(5, 3, 11),
-      memory: 190 * MB,
-    },
-    {
-      pid: 4112,
-      parent: 4100,
-      kind: 'utility',
-      origin: 'app',
-      label: 'Network Service',
-      detail: 'network.mojom.NetworkService',
-      cpu: 0.4,
-      memory: 38 * MB,
-    },
-    {
-      pid: 4300,
-      parent: 4100,
-      kind: 'child',
-      origin: 'app',
-      label: 'typescript-language-server (node)',
-      detail: 'node /usr/local/lib/node_modules/typescript-language-server/lib/cli.mjs --stdio',
-      cpu: wave(13, 0, 14),
-      memory: 420 * MB,
-    },
-    {
-      pid: 5200,
-      parent: null,
-      kind: 'daemon',
-      origin: 'app',
-      label: 'Daemon',
-      detail: 'zidane daemon',
-      cpu: wave(11, 1, 6),
-      memory: 160 * MB,
-    },
-    {
-      pid: 5300,
-      parent: 5200,
-      kind: 'child',
-      origin: 'app',
-      label: 'mcp-server-git (python)',
-      detail: '/home/ada/.local/share/uv/tools/mcp-server-git/bin/python /home/ada/.local/bin/mcp-server-git',
-      cpu: wave(17, 0, 2),
-      memory: 64 * MB,
-      mcp: { name: 'git', shared: true, sessions: 2 },
-    },
-    {
-      pid: 5310,
-      parent: 5200,
-      kind: 'child',
-      origin: 'launched',
-      label: 'zsh',
-      detail: '-zsh',
-      cpu: 0,
-      memory: 6 * MB,
-      session: LOGIN,
-    },
-    {
-      pid: 5311,
-      parent: 5310,
-      kind: 'child',
-      origin: 'launched',
-      label: 'vite (node)',
-      detail: 'node /home/ada/app/node_modules/.bin/vite --port 5173',
-      cpu: wave(6, 90, 160),
-      memory: 380 * MB,
-      session: LOGIN,
-    },
-    {
-      pid: 5320,
-      parent: 5200,
-      kind: 'child',
-      origin: 'launched',
-      label: 'bun',
-      detail: 'bun test --watch',
-      cpu: wave(10, 2, 40),
-      memory: 120 * MB,
-      session: VITE,
-    },
-    {
-      pid: 5330,
-      parent: 5200,
-      kind: 'child',
-      origin: 'app',
-      label: 'playwright-mcp (node)',
-      detail: 'node /home/ada/.npm/_npx/9a1/node_modules/.bin/playwright-mcp',
-      cpu: wave(15, 0, 3),
-      memory: 96 * MB,
-      mcp: { name: 'playwright', shared: false, sessions: 1 },
-      session: VITE,
-    },
-  ];
-  const mainBusy = wave(9, 0.04, 0.2) + spike * 0.4;
-  return {
-    at,
-    intervalMs: 2000,
-    cores: 10,
-    processes,
-    threads: [
-      { id: 'main', label: 'Main process event loop', pid: 4100, kind: 'main', busy: mainBusy, heap: 84 * MB },
-      { id: 'worker:1', label: 'History pager', pid: 4100, kind: 'worker', busy: wave(6, 0, 0.1), heap: 12 * MB },
-      { id: 'worker:2', label: 'Session search', pid: 4100, kind: 'worker', busy: 0, heap: 9 * MB },
-      { id: 'renderer:1', label: 'Zidane', pid: 4120, kind: 'renderer', busy: wave(7, 0, 0.18), heap: 210 * MB },
-      {
-        id: 'renderer:1:Code highlighting',
-        label: 'Code highlighting (Zidane)',
-        pid: 4120,
-        kind: 'renderer-worker',
-        busy: wave(4, 0, 0.3),
-      },
-    ],
-    channels: [
-      {
-        channel: 'sessions:loadEvents',
-        direction: 'invoke',
-        count: 40 + n,
-        totalMs: 1800 + n * 12,
-        maxMs: 240,
-        p95Ms: 90,
-      },
-      { channel: 'chat:stream', direction: 'push', count: 3000 + n * 40, totalMs: 410 + n * 3, maxMs: 6, p95Ms: 0.4 },
-      { channel: 'sessions:list', direction: 'invoke', count: 120 + n, totalMs: 260 + n, maxMs: 31, p95Ms: 4 },
-    ],
-    frames: [
-      {
-        source: '/ui/src/chat/transcript/TranscriptRow.tsx',
-        fn: 'renderRow',
-        window: 'Zidane',
-        count: 14 + Math.floor(n / 5),
-        totalMs: 1260 + n * 9,
-        maxMs: 140,
-      },
-      {
-        source: '(rendering)',
-        fn: 'style, layout and paint',
-        window: 'Zidane',
-        count: 9 + Math.floor(n / 8),
-        totalMs: 520 + n * 4,
-        maxMs: 70,
-      },
-    ],
-    countersSince: at - n * 2000,
-    lag: { p50: 0.6, p99: 4 + spike * 80, max: 9 + spike * 140 },
-    gpu: { device: wave(8, 8, 34), renderer: wave(8, 6, 30), tiler: wave(8, 3, 12), memory: 1400 * MB },
-    totals: sum(processes.filter((p) => p.origin === 'app')),
-    launched: sum(processes.filter((p) => p.origin === 'launched')),
-    self: { cpu: 0.15, commandMs: 18 },
-  };
-}
-
-const THREADS: PerfOsThread[] = [
-  { id: '1', label: '', cpu: 6.2 },
-  { id: '7', label: '', cpu: 1.1 },
-  { id: '3', label: '', cpu: 0.2 },
-];
-
-/**
- * Creates the in-memory member.
- *
- * @param options - History, tick rate, whether profiles are busy.
- */
-export function createFixturePerformanceHost(options: FixturePerformanceOptions = {}): FixturePerformanceHost {
-  const intervalMs = options.intervalMs ?? 2000;
-  // Samples stay 2 s apart even when the timer is off (tests tick by hand).
-  const spacing = intervalMs || 2000;
-  const start = Date.now() - (options.history ?? 30) * spacing;
-  const history: PerfSample[] = [];
-  let n = 0;
-  if (options.samples) {
-    history.push(...options.samples);
-    n = options.samples.length;
-  } else for (; n < (options.history ?? 30); n++) history.push(fixturePerfSample(n, start + n * spacing));
-  const listeners = new Set<(sample: PerfSample) => void>();
-  let timer: ReturnType<typeof setInterval> | null = null;
-  let resets = 0;
-  const profiles: { target: PerfProfileTarget; seconds: number }[] = [];
-
-  const emit = (sample: PerfSample) => {
-    history.push(sample);
-    if (history.length > 150) history.shift();
-    for (const listener of [...listeners]) listener(sample);
-  };
-  const tick = () => {
-    const sample = fixturePerfSample(n, start + n * spacing);
-    n += 1;
-    emit(sample);
-    return sample;
-  };
-
-  return {
-    history: async () => [...history],
-    watch(onSample, onError) {
-      const refusal = options.watchError;
-      if (refusal) {
-        let live = true;
-        queueMicrotask(() => live && onError?.(refusal));
-        return () => {
-          live = false;
-        };
-      }
-      listeners.add(onSample);
-      if (!timer && intervalMs > 0) timer = setInterval(tick, intervalMs);
-      return () => {
-        listeners.delete(onSample);
-        if (listeners.size === 0 && timer) {
-          clearInterval(timer);
-          timer = null;
-        }
-      };
-    },
-    threads: async (pid) => (options.threads ? (options.threads[pid] ?? []) : THREADS),
-    resetCounters: async () => {
-      resets += 1;
-    },
-    exportReport: async () =>
-      options.reportPath ?? '/home/ada/Library/Application Support/Zidane/perf/report-2026-10-22T10-00-00-000Z.json',
-    profile: async (target, seconds): Promise<PerfProfileOutcome> => {
-      profiles.push({ target, seconds });
-      if (options.profileOutcome) return options.profileOutcome;
-      if (options.profileBusy) return { ok: false, error: { kind: 'busy', detail: 'DevTools is attached' } };
-      return {
-        ok: true,
-        profile: {
-          path: `/home/ada/Library/Application Support/Zidane/perf/${target}-2026-10-22T10-00-00-000Z.cpuprofile`,
-          target,
-          durationMs: seconds * 1000,
-          top: [
-            { fn: 'renderRow', url: '/ui/src/chat/transcript/TranscriptRow.tsx', line: 88, selfMs: 412.5, share: 0.31 },
-            { fn: '(garbage collector)', url: '', line: 1, selfMs: 160.2, share: 0.12 },
-            {
-              fn: 'collapseEvents',
-              url: '/ui/src/chat/headless/transcript/collapse.ts',
-              line: 41,
-              selfMs: 98.1,
-              share: 0.07,
-            },
-          ],
-        },
-      };
-    },
-    control: {
-      tick,
-      emit,
-      watchers: () => listeners.size,
-      resets: () => resets,
-      profiles: () => profiles,
-    },
-  };
-}

package/src/settings/headless/fixture-sections.ts

@@ -1,323 +0,0 @@
-/**
- * The in-memory catalog and diagnostics members of the fixture settings host
- * (`createFixtureSettingsHost({ sections })`): skills, extensions, plugins,
- * a session's instructions, environments, logs and a host's catalogs, each
- * over the arrays given, every action landing at once. A group whose data is
- * left out is left out of the host.
- */
-import type { SkillConfig, WireExtensionConfigSchema } from 'zidane';
-import type { ContextSegmentsReport, EnvironmentDescriptor, EnvironmentSettingsChange } from 'zidane/chat';
-import type { LogRecord, SkillSourceListing } from 'zidane/chat/pure';
-import type { HostCatalogExtension, HostCatalogMcp, HostCatalogPlugin, HostCatalogSkill } from 'zidane/ipc';
-
-import { createSettingsResource, keyedSettingsResources } from './resource';
-import type {
-  SettingsCatalogKind,
-  SettingsExtension,
-  SettingsExtensionAction,
-  SettingsExtensionActionResult,
-  SettingsExtensionActionStatus,
-  SettingsFile,
-  SettingsPlugin,
-  SettingsHostCatalog,
-  SettingsEnvironmentsHost,
-  SettingsExtensionsHost,
-  SettingsInstructionsHost,
-  SettingsLogsHost,
-  SettingsPluginsHost,
-  SettingsSkillsHost,
-} from './sections-host';
-
-/** A host's catalogs, as `catalog.*` reports them. */
-export interface FixtureHostCatalogs {
-  /** The host's skills. */
-  skills: HostCatalogSkill[];
-  /** The host's MCP servers. */
-  mcps: HostCatalogMcp[];
-  /** The host's extensions. */
-  extensions: HostCatalogExtension[];
-  /** The host's plugins. */
-  plugins: HostCatalogPlugin[];
-}
-
-/** The extension options of the fixture: the schema, the action rows, their probes and what a run does. */
-export interface FixtureExtensionOptions {
-  /** The options declared. */
-  schema?: WireExtensionConfigSchema;
-  /** The action rows declared. */
-  actions?: SettingsExtensionAction[];
-  /** The probes, keyed `extension:key`. */
-  statuses?: Record<string, SettingsExtensionActionStatus>;
-  /** What running each action (keyed `extension:key`) answers; unlisted: `done`. Offers `runAction` with `actions`. */
-  runs?: Record<string, SettingsExtensionActionResult>;
-}
-
-/** What the fixture's catalog and diagnostics members hold. Each key offers its group. */
-export interface FixtureSettingsSectionsData {
-  /** The skills (for every folder); their `instructions` are their SKILL.md bodies. */
-  skills?: SkillConfig[];
-  /** What a source spec lists, for the skills browser. */
-  skillSources?: Record<string, SkillSourceListing>;
-  /** The extensions (for every folder). */
-  extensions?: SettingsExtension[];
-  /** The options and actions the enabled extensions declare (with `extensions`); each key offers its member. */
-  extensionOptions?: FixtureExtensionOptions;
-  /** The plugins (for every folder). */
-  plugins?: SettingsPlugin[];
-  /** Each session's prompt (`null`: no live agent). */
-  instructions?: Record<string, ContextSegmentsReport | null>;
-  /** The environment kinds. */
-  environments?: EnvironmentDescriptor[];
-  /** The log records held. */
-  logs?: LogRecord[];
-  /** A host's catalogs (`null`: the host can't list them). */
-  hostCatalog?: FixtureHostCatalogs | null;
-  /** The host takes installs from this connection. Default: true. */
-  hostInstalls?: boolean;
-}
-
-/** Changes the fixture's catalogs from outside and reads what actions did. */
-export interface FixtureSettingsSectionsControl {
-  /** What the fixture holds now. */
-  data(): FixtureSettingsSectionsData;
-  /** A new log record, told to followers. */
-  log(record: LogRecord): void;
-  /** Environment edits applied, in order. */
-  environmentChanges(): readonly EnvironmentSettingsChange[];
-  /** System overrides created, by session. */
-  overrides(): readonly string[];
-  /** A host catalog changed there: `hostCatalog.onChanged` listeners hear it. */
-  catalogChanged(kind: SettingsCatalogKind): void;
-}
-
-/** The members and their control. */
-export interface FixtureSettingsSections {
-  /** Over `skills` (and `skillSources`). */
-  skills?: SettingsSkillsHost;
-  /** Over `extensions`. */
-  extensions?: SettingsExtensionsHost;
-  /** Over `plugins`. */
-  plugins?: SettingsPluginsHost;
-  /** Over `instructions`. */
-  instructions?: SettingsInstructionsHost;
-  /** Over `environments`. */
-  environments?: SettingsEnvironmentsHost;
-  /** Over `logs`. */
-  logs?: SettingsLogsHost;
-  /** Over `hostCatalog`. */
-  hostCatalog?: SettingsHostCatalog;
-  /** Reads and changes what they hold. */
-  control: FixtureSettingsSectionsControl;
-}
-
-/**
- * Creates the in-memory catalog and diagnostics members.
- *
- * @param data - What they hold. A key left out leaves its group out.
- */
-export function createFixtureSettingsSections(data: FixtureSettingsSectionsData = {}): FixtureSettingsSections {
-  const held: FixtureSettingsSectionsData = structuredClone(data);
-  const changes: EnvironmentSettingsChange[] = [];
-  const overrides: string[] = [];
-  const followers = new Set<(record: LogRecord) => void>();
-  const files = new Map<string, SettingsFile>();
-  const catalogListeners = new Set<(kind: SettingsCatalogKind) => void>();
-  const changed = (kind: SettingsCatalogKind) => {
-    for (const listener of [...catalogListeners]) listener(kind);
-  };
-  const out: FixtureSettingsSections = {
-    control: {
-      data: () => held,
-      log: (record) => {
-        held.logs = [...(held.logs ?? []), record];
-        for (const follower of followers) follower(record);
-      },
-      environmentChanges: () => changes,
-      overrides: () => overrides,
-      catalogChanged: changed,
-    },
-  };
-
-  if (held.skills) {
-    const skillsOf = () => held.skills ?? [];
-    const catalogs = keyedSettingsResources((_cwd: string) => createSettingsResource(async () => [...skillsOf()]));
-    const fileOf = (name: string): SettingsFile | null => {
-      const skill = skillsOf().find((s) => s.name === name);
-      if (!skill?.location) return null;
-      return files.get(name) ?? { path: skill.location, contents: skill.instructions ?? '', mtimeMs: 1 };
-    };
-    const fileResources = keyedSettingsResources(([, name]: [string, string]) =>
-      createSettingsResource(async () => fileOf(name)),
-    );
-    const sources = held.skillSources ?? {};
-    out.skills = {
-      catalog: (cwd) => catalogs(cwd),
-      browse: async (input) => {
-        const listing = sources[input];
-        if (!listing) throw new Error(`Nothing to install at ${input}`);
-        return listing;
-      },
-      install: async (_cwd, request, onProgress) => {
-        const picked = Object.values(sources)
-          .flatMap((listing) => listing.skills)
-          .filter((s) => s.install.source === request.source && request.skills.includes(s.install.rel));
-        if (!picked?.length) return 'unmatched';
-        onProgress?.(`installing ${picked.length} from ${request.source}`);
-        held.skills = [
-          ...skillsOf(),
-          ...picked.map((s): SkillConfig => ({
-            name: s.name,
-            description: s.description,
-            instructions: '',
-            source: request.scope,
-          })),
-        ];
-        return 'installed';
-      },
-      remove: async (_cwd, skill) => {
-        held.skills = skillsOf().filter((s) => s.name !== skill.name);
-      },
-      file: (cwd, name) => fileResources([cwd, name]),
-      write: async (_cwd, name, contents, expectedMtimeMs) => {
-        const current = fileOf(name);
-        if (!current) return { ok: false, reason: 'missing' };
-        if (current.mtimeMs !== expectedMtimeMs) return { ok: false, reason: 'changed' };
-        const mtimeMs = current.mtimeMs + 1;
-        files.set(name, { ...current, contents, mtimeMs });
-        return { ok: true, mtimeMs };
-      },
-    };
-  }
-
-  if (held.extensions) {
-    out.extensions = {
-      list: async () => [...(held.extensions ?? [])],
-      install: async (_cwd, request, onProgress) => {
-        const name = request.extension ?? request.source.split('/').pop() ?? request.source;
-        onProgress?.(`installing ${name}`);
-        held.extensions = [
-          ...(held.extensions ?? []),
-          { name, source: request.scope, enabled: true, dir: `~/.zidane/extensions/${name}` },
-        ];
-        return { name };
-      },
-      update: async (_cwd, extension) => `${extension.name} is up to date`,
-      remove: async (_cwd, extension) => {
-        held.extensions = (held.extensions ?? []).filter((e) => e.name !== extension.name);
-      },
-      create: async (request) => {
-        const dir =
-          request.scope === 'project'
-            ? `${request.cwd}/.zidane/extensions/${request.name}`
-            : `~/.zidane/extensions/${request.name}`;
-        held.extensions = [
-          ...(held.extensions ?? []),
-          { name: request.name, source: request.scope, enabled: true, dir },
-        ];
-        return { dir, name: request.name, ...(request.scope === 'project' ? { needsTrust: true } : {}) };
-      },
-    };
-    const options = held.extensionOptions;
-    if (options?.schema) {
-      const schema = options.schema;
-      out.extensions.configSchema = keyedSettingsResources((_cwd: string) =>
-        createSettingsResource(async () => schema),
-      );
-    }
-    if (options?.actions) {
-      const actions = options.actions;
-      out.extensions.actions = keyedSettingsResources((_cwd: string) => createSettingsResource(async () => actions));
-      out.extensions.runAction = async (_cwd, extension, key) =>
-        options.runs?.[`${extension}:${key}`] ?? { message: 'done' };
-    }
-    if (options?.statuses) {
-      const statuses = options.statuses;
-      out.extensions.actionStatuses = async () => statuses;
-    }
-  }
-
-  if (held.plugins) {
-    const plugins = keyedSettingsResources((_cwd: string) =>
-      createSettingsResource(async () => [...(held.plugins ?? [])]),
-    );
-    out.plugins = { catalog: (cwd) => plugins(cwd) };
-  }
-
-  if (held.instructions) {
-    out.instructions = {
-      segments: async (sessionId) => held.instructions?.[sessionId] ?? null,
-      createSystemOverride: async (sessionId) => {
-        overrides.push(sessionId);
-      },
-    };
-  }
-
-  if (held.environments) {
-    out.environments = {
-      list: async () => [...(held.environments ?? [])],
-      update: async (change) => {
-        changes.push(change);
-        held.environments = (held.environments ?? []).map((d) =>
-          d.kind === change.kind
-            ? {
-                ...d,
-                ...(change.enabled !== undefined ? { enabled: change.enabled } : {}),
-                ...(change.makeDefault ? { isDefault: true } : {}),
-              }
-            : change.makeDefault
-              ? { ...d, isDefault: false }
-              : d,
-        );
-      },
-      verify: async (kind) => ({ ok: true, detail: `${kind} answered.` }),
-    };
-  }
-
-  if (held.logs) {
-    out.logs = {
-      list: async () => [...(held.logs ?? [])],
-      follow: (onRecord) => {
-        followers.add(onRecord);
-        return () => followers.delete(onRecord);
-      },
-    };
-  }
-
-  if (held.hostCatalog !== undefined) {
-    const installs = held.hostInstalls ?? true;
-    const catalog = () => held.hostCatalog;
-    out.hostCatalog = {
-      list: async (kind) => {
-        const all = catalog();
-        return all ? ([...all[kind]] as never) : null;
-      },
-      onChanged: (listener) => {
-        catalogListeners.add(listener);
-        return () => catalogListeners.delete(listener);
-      },
-      installs: async () => installs,
-      install: async (kind, spec) => {
-        const all = catalog();
-        if (!all) return;
-        const name = spec.split('/').pop() ?? spec;
-        if (kind === 'skill') {
-          all.skills.push({ name, description: '', enabled: true, toggleable: true, origin: 'disk', source: 'user' });
-          changed('skills');
-        } else {
-          all.extensions.push({ name, source: 'user', enabled: true, toggleable: true });
-          changed('extensions');
-        }
-      },
-      remove: async (kind, name) => {
-        const all = catalog();
-        if (!all) return;
-        if (kind === 'skill') all.skills = all.skills.filter((s) => s.name !== name);
-        else all.extensions = all.extensions.filter((e) => e.name !== name);
-        changed(kind === 'skill' ? 'skills' : 'extensions');
-      },
-      update: async () => undefined,
-    };
-  }
-
-  return out;
-}