@asterxsk/kiln 0.4.0 → 0.4.2

package/agent/extensions/modelconf/package.json

@@ -1,13 +1,13 @@
-{
-  "name": "modelconf",
-  "private": true,
-  "type": "module",
-  "version": "0.1.0",
-  "description": "Per-provider model visibility manager with fuzzy search and bulk glob",
-  "scripts": {
-    "check": "tsc --noEmit"
-  },
-  "devDependencies": { "typescript": "^5.6.0" },
-  "allowScripts": {},
-  "dependencies": {}
-}
+{
+  "name": "modelconf",
+  "private": true,
+  "type": "module",
+  "version": "0.1.0",
+  "description": "Per-provider model visibility manager with fuzzy search and bulk glob",
+  "scripts": {
+    "check": "tsc --noEmit"
+  },
+  "devDependencies": { "typescript": "^5.6.0" },
+  "allowScripts": {},
+  "dependencies": {}
+}

package/agent/extensions/modelconf/src/fuzzy.ts

@@ -1,26 +1,26 @@
-export function fuzzyScore(query: string, target: string): number | null {
-  if (!query) return 0;
-  const q = query.toLowerCase();
-  const t = target.toLowerCase();
-  let qi = 0, ti = 0, score = 0, last = -2;
-  for (; qi < q.length && ti < t.length; ti++) {
-    if (q[qi] === t[ti]) {
-      // bonus for contiguous + start-of-word
-      score += (ti === last + 1 ? 2 : 1);
-      if (ti === 0 || t[ti-1] === "/" || t[ti-1] === "-" || t[ti-1] === "_") score += 2;
-      last = ti; qi++;
-    }
-  }
-  if (qi !== q.length) return null;
-  // penalize longer targets slightly
-  score -= Math.floor(t.length / 20);
-  return score;
-}
-
-export function fuzzyFilter<T>(query: string, items: T[], key: (t:T)=>string): T[] {
-  if (!query.trim()) return items;
-  const scored = items.map(i => ({ i, s: fuzzyScore(query, key(i)) }))
-    .filter(x => x.s !== null) as Array<{i:T; s:number}>;
-  scored.sort((a,b)=> b.s - a.s);
-  return scored.map(x=>x.i);
-}
+export function fuzzyScore(query: string, target: string): number | null {
+  if (!query) return 0;
+  const q = query.toLowerCase();
+  const t = target.toLowerCase();
+  let qi = 0, ti = 0, score = 0, last = -2;
+  for (; qi < q.length && ti < t.length; ti++) {
+    if (q[qi] === t[ti]) {
+      // bonus for contiguous + start-of-word
+      score += (ti === last + 1 ? 2 : 1);
+      if (ti === 0 || t[ti-1] === "/" || t[ti-1] === "-" || t[ti-1] === "_") score += 2;
+      last = ti; qi++;
+    }
+  }
+  if (qi !== q.length) return null;
+  // penalize longer targets slightly
+  score -= Math.floor(t.length / 20);
+  return score;
+}
+
+export function fuzzyFilter<T>(query: string, items: T[], key: (t:T)=>string): T[] {
+  if (!query.trim()) return items;
+  const scored = items.map(i => ({ i, s: fuzzyScore(query, key(i)) }))
+    .filter(x => x.s !== null) as Array<{i:T; s:number}>;
+  scored.sort((a,b)=> b.s - a.s);
+  return scored.map(x=>x.i);
+}

package/agent/extensions/modelconf/src/glob.ts

@@ -1,13 +1,13 @@
-export function globToRegExp(pattern: string): RegExp {
-  let p = pattern;
-  if (!p.includes("*") && !p.includes("?")) p = `*${p}*`;
-  const esc = p.replace(/[.+^${}()|[\]\\]/g, "\\$&").replace(/\*/g, ".*").replace(/\?/g, ".");
-  return new RegExp(`^${esc}$`, "i");
-}
-export function matchesGlob(input: string, pattern: string): boolean {
-  return globToRegExp(pattern).test(input);
-}
-export function filterByGlob<T>(pattern: string, items: T[], key:(t:T)=>string): T[] {
-  const rx = globToRegExp(pattern);
-  return items.filter(i => rx.test(key(i)));
-}
+export function globToRegExp(pattern: string): RegExp {
+  let p = pattern;
+  if (!p.includes("*") && !p.includes("?")) p = `*${p}*`;
+  const esc = p.replace(/[.+^${}()|[\]\\]/g, "\\$&").replace(/\*/g, ".*").replace(/\?/g, ".");
+  return new RegExp(`^${esc}$`, "i");
+}
+export function matchesGlob(input: string, pattern: string): boolean {
+  return globToRegExp(pattern).test(input);
+}
+export function filterByGlob<T>(pattern: string, items: T[], key:(t:T)=>string): T[] {
+  const rx = globToRegExp(pattern);
+  return items.filter(i => rx.test(key(i)));
+}

package/agent/extensions/modelconf/src/persistence.test.ts

@@ -1,46 +1,46 @@
-import { isVisible, toModelId, getVisibilityMap, toggleInEnabled, applyBulk } from "./persistence.js";
-import type { ModelId } from "./persistence.js";
-import assert from "node:assert";
-
-const all = ["opencode/a","opencode/b","9router/c"] as unknown as ModelId[];
-
-// isVisible (denylist: the list holds HIDDEN models)
-assert.equal(isVisible("opencode/a" as ModelId, undefined, [...all]), true);
-assert.equal(isVisible("opencode/a" as ModelId, [], [...all]), true);
-assert.equal(isVisible("opencode/a" as ModelId, ["opencode/a"], [...all]), false);
-assert.equal(isVisible("opencode/b" as ModelId, ["opencode/a"], [...all]), true);
-
-// toModelId
-assert.equal(toModelId({ provider: "opencode", id: "a" }), "opencode/a");
-
-// getVisibilityMap
-const vis = getVisibilityMap([{provider:"opencode",id:"a"},{provider:"opencode",id:"b"}], ["opencode/a"]);
-assert.equal(vis.get("opencode/a" as ModelId), false);
-assert.equal(vis.get("opencode/b" as ModelId), true);
-const visAll = getVisibilityMap([{provider:"opencode",id:"a"},{provider:"opencode",id:"b"}], undefined);
-assert.equal(visAll.get("opencode/a" as ModelId), true);
-assert.equal(visAll.get("opencode/b" as ModelId), true);
-
-// toggleInEnabled (denylist: toggling a visible model HIDES it and vice versa)
-// undefined => hide just this one
-assert.deepEqual(toggleInEnabled("opencode/a" as ModelId, undefined, [...all]), ["opencode/a"]);
-// toggle visible => append exact id
-assert.deepEqual(toggleInEnabled("9router/c" as ModelId, ["opencode/a"], [...all]), ["opencode/a","9router/c"]);
-// toggle hidden => remove exact id
-assert.deepEqual(toggleInEnabled("opencode/a" as ModelId, ["opencode/a","opencode/b"], [...all]), ["opencode/b"]);
-// removing the last hidden id collapses to undefined (all visible)
-assert.equal(toggleInEnabled("opencode/a" as ModelId, ["opencode/a"], [...all]), undefined);
-// hidden via glob: showing removes the matching pattern so the model becomes visible
-assert.equal(toggleInEnabled("opencode/a" as ModelId, ["*a"], [...all]), undefined);
-
-// applyBulk exclude = hide (union into denylist, sorted)
-assert.deepEqual(applyBulk(undefined, [...all], ["opencode/a" as ModelId], "exclude"), ["opencode/a"]);
-assert.deepEqual(applyBulk(["opencode/a","opencode/b","9router/c"] as unknown as ModelId[], [...all], ["opencode/a" as ModelId], "exclude"), ["9router/c","opencode/a","opencode/b"]);
-
-// applyBulk include = show (remove exact ids from denylist)
-assert.deepEqual(applyBulk(["opencode/a","opencode/b"] as unknown as ModelId[], [...all], ["opencode/b" as ModelId], "include"), ["opencode/a"]);
-assert.equal(applyBulk(undefined, [...all], [...all], "include"), undefined); // all visible => undefined
-assert.deepEqual(applyBulk(["opencode/a","opencode/b","9router/c"] as unknown as ModelId[], [...all], [] as ModelId[], "include"), ["opencode/a","opencode/b","9router/c"]); // empty selection changes nothing
-assert.deepEqual(applyBulk(["*luna*","opencode/a"] as unknown as ModelId[], [...all], ["opencode/a" as ModelId], "include"), ["*luna*"]); // exact-only removal keeps unrelated globs
-
-console.log("persistence test pass");
+import { isVisible, toModelId, getVisibilityMap, toggleInEnabled, applyBulk } from "./persistence.js";
+import type { ModelId } from "./persistence.js";
+import assert from "node:assert";
+
+const all = ["opencode/a","opencode/b","9router/c"] as unknown as ModelId[];
+
+// isVisible (denylist: the list holds HIDDEN models)
+assert.equal(isVisible("opencode/a" as ModelId, undefined, [...all]), true);
+assert.equal(isVisible("opencode/a" as ModelId, [], [...all]), true);
+assert.equal(isVisible("opencode/a" as ModelId, ["opencode/a"], [...all]), false);
+assert.equal(isVisible("opencode/b" as ModelId, ["opencode/a"], [...all]), true);
+
+// toModelId
+assert.equal(toModelId({ provider: "opencode", id: "a" }), "opencode/a");
+
+// getVisibilityMap
+const vis = getVisibilityMap([{provider:"opencode",id:"a"},{provider:"opencode",id:"b"}], ["opencode/a"]);
+assert.equal(vis.get("opencode/a" as ModelId), false);
+assert.equal(vis.get("opencode/b" as ModelId), true);
+const visAll = getVisibilityMap([{provider:"opencode",id:"a"},{provider:"opencode",id:"b"}], undefined);
+assert.equal(visAll.get("opencode/a" as ModelId), true);
+assert.equal(visAll.get("opencode/b" as ModelId), true);
+
+// toggleInEnabled (denylist: toggling a visible model HIDES it and vice versa)
+// undefined => hide just this one
+assert.deepEqual(toggleInEnabled("opencode/a" as ModelId, undefined, [...all]), ["opencode/a"]);
+// toggle visible => append exact id
+assert.deepEqual(toggleInEnabled("9router/c" as ModelId, ["opencode/a"], [...all]), ["opencode/a","9router/c"]);
+// toggle hidden => remove exact id
+assert.deepEqual(toggleInEnabled("opencode/a" as ModelId, ["opencode/a","opencode/b"], [...all]), ["opencode/b"]);
+// removing the last hidden id collapses to undefined (all visible)
+assert.equal(toggleInEnabled("opencode/a" as ModelId, ["opencode/a"], [...all]), undefined);
+// hidden via glob: showing removes the matching pattern so the model becomes visible
+assert.equal(toggleInEnabled("opencode/a" as ModelId, ["*a"], [...all]), undefined);
+
+// applyBulk exclude = hide (union into denylist, sorted)
+assert.deepEqual(applyBulk(undefined, [...all], ["opencode/a" as ModelId], "exclude"), ["opencode/a"]);
+assert.deepEqual(applyBulk(["opencode/a","opencode/b","9router/c"] as unknown as ModelId[], [...all], ["opencode/a" as ModelId], "exclude"), ["9router/c","opencode/a","opencode/b"]);
+
+// applyBulk include = show (remove exact ids from denylist)
+assert.deepEqual(applyBulk(["opencode/a","opencode/b"] as unknown as ModelId[], [...all], ["opencode/b" as ModelId], "include"), ["opencode/a"]);
+assert.equal(applyBulk(undefined, [...all], [...all], "include"), undefined); // all visible => undefined
+assert.deepEqual(applyBulk(["opencode/a","opencode/b","9router/c"] as unknown as ModelId[], [...all], [] as ModelId[], "include"), ["opencode/a","opencode/b","9router/c"]); // empty selection changes nothing
+assert.deepEqual(applyBulk(["*luna*","opencode/a"] as unknown as ModelId[], [...all], ["opencode/a" as ModelId], "include"), ["*luna*"]); // exact-only removal keeps unrelated globs
+
+console.log("persistence test pass");

package/agent/extensions/modelconf/src/persistence.ts

@@ -1,164 +1,164 @@
-import fs from "node:fs/promises";
-import os from "node:os";
-import path from "node:path";
-import { globToRegExp } from "./glob.js";
-
-export type ModelId = `${string}/${string}`;
-
-export function toModelId(m: { provider: string; id: string }): ModelId {
-  return `${m.provider}/${m.id}` as ModelId;
-}
-
-function settingsPath(): string {
-  return path.join(os.homedir(), ".pi", "agent", "settings.json");
-}
-
-const VALID_THINKING = new Set(["off", "minimal", "low", "medium", "high", "xhigh", "max"]);
-function stripThinkingLevel(pattern: string): string {
-  const idx = pattern.lastIndexOf(":");
-  if (idx !== -1) {
-    const suffix = pattern.slice(idx + 1);
-    if (VALID_THINKING.has(suffix)) return pattern.slice(0, idx);
-  }
-  return pattern;
-}
-
-function matchesPattern(modelRef: string, pattern: string): boolean {
-  const pat = stripThinkingLevel(pattern);
-  const lowerRef = modelRef.toLowerCase();
-  const lowerPat = pat.toLowerCase();
-  const bareId = modelRef.split("/").slice(1).join("/")?.toLowerCase() ?? lowerRef;
-  const hasGlob = /[*?\[\]]/.test(pat);
-  if (hasGlob) {
-    const rx = globToRegExp(pat);
-    if (rx.test(modelRef)) return true;
-    if (rx.test(bareId)) return true;
-    return false;
-  } else {
-    return lowerRef === lowerPat || bareId === lowerPat;
-  }
-}
-
-function modelMatchesAnyPattern(modelRef: string, patterns: string[]): boolean {
-  for (const p of patterns) if (matchesPattern(modelRef, p)) return true;
-  return false;
-}
-
-// ---------- hiddenModels (denylist) — hides from full picker list ----------
-export async function loadHiddenModels(): Promise<string[] | undefined> {
-  try {
-    const raw = await fs.readFile(settingsPath(), "utf8");
-    const j = JSON.parse(raw) as { hiddenModels?: string[] };
-    return j.hiddenModels;
-  } catch { return undefined; }
-}
-
-// Backward compat alias — old code used enabledModels
-export async function loadEnabledModels(): Promise<string[] | undefined> {
-  return loadHiddenModels();
-}
-
-export async function saveHiddenModels(ids: string[] | undefined): Promise<void> {
-  const p = settingsPath();
-  let j: Record<string, unknown> = {};
-  try {
-    const raw = await fs.readFile(p, "utf8");
-    j = JSON.parse(raw) as Record<string, unknown>;
-  } catch (e: any) {
-    if ((e as any)?.code !== "ENOENT") {
-      j = {};
-      try { const raw = await fs.readFile(p, "utf8"); JSON.parse(raw); } catch {}
-    }
-  }
-  if (ids === undefined || ids.length === 0) delete (j as any).hiddenModels;
-  else (j as any).hiddenModels = ids;
-  // Do not touch enabledModels — this extension no longer manages it.
-  await fs.writeFile(p, JSON.stringify(j, null, 2) + "\n", "utf8");
-}
-
-export async function saveEnabledModels(ids: string[] | undefined): Promise<void> {
-  return saveHiddenModels(ids);
-}
-
-export function isHidden(modelId: ModelId, hidden: string[] | undefined): boolean {
-  if (!hidden || hidden.length === 0) return false;
-  return modelMatchesAnyPattern(modelId, hidden) || modelMatchesAnyPattern(modelId.split("/").slice(1).join("/") as ModelId, hidden);
-}
-
-export function isVisible(modelId: ModelId, hidden: string[] | undefined, _allIds: ModelId[]): boolean {
-  if (!hidden || hidden.length === 0) return true;
-  return !isHidden(modelId, hidden);
-}
-
-export function getVisibilityMap(all: Array<{provider:string;id:string}>, hidden: string[]|undefined): Map<ModelId, boolean> {
-  const map = new Map<ModelId, boolean>();
-  const allIds = all.map(toModelId);
-  for (const m of all) map.set(toModelId(m), isVisible(toModelId(m), hidden, allIds));
-  return map;
-}
-
-// Hide/show operates on the FULL model list (allIds = full catalogue).
-// hidden is a denylist: adding = hide, removing = show.
-export function toggleHidden(modelId: ModelId, hidden: string[] | undefined, _allIds: ModelId[]): string[] | undefined {
-  if (!hidden || hidden.length === 0) {
-    return [modelId];
-  }
-  // Exact match toggle. If hidden contains globs that already hide this model, we still toggle exact id.
-  // If model is hidden via pattern, toggling will add exact? Instead remove matching patterns? Simplify: exact toggle.
-  const hasExact = hidden.includes(modelId);
-  // Also consider pattern hiding: if model matches a pattern, showing should remove? But we can't know which pattern.
-  // For now, if hidden via pattern and not exact, toggling to "show" should add an exception? Simpler: if currently hidden (pattern or exact), remove exact and also remove any pattern that matches it? That's ambiguous.
-  // We treat visibility via pattern: if isHidden true but not exact, toggling "show" means we need to keep patterns but add exception — not supported.
-  // Simplest: if currently hidden, remove exact if present, otherwise add exact and keep patterns — model stays hidden. To actually show, we'd need to remove matching patterns.
-  // So if hidden via pattern, show action should remove matching patterns.
-  const currentlyHidden = isHidden(modelId, hidden);
-  if (currentlyHidden) {
-    // Remove exact if present
-    let next = hidden.filter(id => id !== modelId);
-    // Also remove any glob pattern that matches this model (so it becomes visible)
-    next = next.filter(p => !matchesPattern(modelId, p));
-    if (next.length === 0) return undefined;
-    return next;
-  } else {
-    // Currently visible — hide by adding exact id
-    if (hasExact) return hidden.filter(id => id !== modelId);
-    return [...hidden, modelId];
-  }
-}
-
-// Aliases for old names
-export function toggleInEnabled(modelId: ModelId, hidden: string[] | undefined, allIds: ModelId[]): string[] | undefined {
-  return toggleHidden(modelId, hidden, allIds);
-}
-
-export function applyBulk(hidden: string[]|undefined, allIds: ModelId[], matching: ModelId[], action: "include"|"exclude"): string[]|undefined {
-  // For denylist: "exclude" = hide (add to hidden), "include" = show (remove from hidden)
-  if (action === "exclude") {
-    const base = hidden === undefined ? [] : [...hidden];
-    const set = new Set(base);
-    for (const id of matching) set.add(id);
-    const arr = [...set].sort();
-    if (arr.length === 0) return undefined;
-    return arr;
-  } else {
-    // include = show
-    if (!hidden || hidden.length === 0) return undefined;
-    // Remove exact ids and any pattern that matches them? For now remove exact and patterns that match
-    let next = [...hidden];
-    for (const id of matching) {
-      next = next.filter(p => p !== id && !matchesPattern(id, p));
-      // If p is exact id matching model, remove; if p is glob that would match id, also remove? But glob may hide other models — removing it would unhide others unintentionally.
-      // Better only remove exact matches for bulk include, keep globs.
-      // Rebuild: only remove exact ids
-    }
-    // Recompute strictly exact removal to avoid over-removing globs
-    const exactSet = new Set(matching as string[]);
-    next = (hidden as string[]).filter(p => {
-      // If p is exact id in matching, remove
-      if (exactSet.has(p)) return false;
-      return true;
-    });
-    if (next.length === 0) return undefined;
-    return next;
-  }
-}
+import fs from "node:fs/promises";
+import os from "node:os";
+import path from "node:path";
+import { globToRegExp } from "./glob.js";
+
+export type ModelId = `${string}/${string}`;
+
+export function toModelId(m: { provider: string; id: string }): ModelId {
+  return `${m.provider}/${m.id}` as ModelId;
+}
+
+function settingsPath(): string {
+  return path.join(os.homedir(), ".pi", "agent", "settings.json");
+}
+
+const VALID_THINKING = new Set(["off", "minimal", "low", "medium", "high", "xhigh", "max"]);
+function stripThinkingLevel(pattern: string): string {
+  const idx = pattern.lastIndexOf(":");
+  if (idx !== -1) {
+    const suffix = pattern.slice(idx + 1);
+    if (VALID_THINKING.has(suffix)) return pattern.slice(0, idx);
+  }
+  return pattern;
+}
+
+function matchesPattern(modelRef: string, pattern: string): boolean {
+  const pat = stripThinkingLevel(pattern);
+  const lowerRef = modelRef.toLowerCase();
+  const lowerPat = pat.toLowerCase();
+  const bareId = modelRef.split("/").slice(1).join("/")?.toLowerCase() ?? lowerRef;
+  const hasGlob = /[*?\[\]]/.test(pat);
+  if (hasGlob) {
+    const rx = globToRegExp(pat);
+    if (rx.test(modelRef)) return true;
+    if (rx.test(bareId)) return true;
+    return false;
+  } else {
+    return lowerRef === lowerPat || bareId === lowerPat;
+  }
+}
+
+function modelMatchesAnyPattern(modelRef: string, patterns: string[]): boolean {
+  for (const p of patterns) if (matchesPattern(modelRef, p)) return true;
+  return false;
+}
+
+// ---------- hiddenModels (denylist) — hides from full picker list ----------
+export async function loadHiddenModels(): Promise<string[] | undefined> {
+  try {
+    const raw = await fs.readFile(settingsPath(), "utf8");
+    const j = JSON.parse(raw) as { hiddenModels?: string[] };
+    return j.hiddenModels;
+  } catch { return undefined; }
+}
+
+// Backward compat alias — old code used enabledModels
+export async function loadEnabledModels(): Promise<string[] | undefined> {
+  return loadHiddenModels();
+}
+
+export async function saveHiddenModels(ids: string[] | undefined): Promise<void> {
+  const p = settingsPath();
+  let j: Record<string, unknown> = {};
+  try {
+    const raw = await fs.readFile(p, "utf8");
+    j = JSON.parse(raw) as Record<string, unknown>;
+  } catch (e: any) {
+    if ((e as any)?.code !== "ENOENT") {
+      j = {};
+      try { const raw = await fs.readFile(p, "utf8"); JSON.parse(raw); } catch {}
+    }
+  }
+  if (ids === undefined || ids.length === 0) delete (j as any).hiddenModels;
+  else (j as any).hiddenModels = ids;
+  // Do not touch enabledModels — this extension no longer manages it.
+  await fs.writeFile(p, JSON.stringify(j, null, 2) + "\n", "utf8");
+}
+
+export async function saveEnabledModels(ids: string[] | undefined): Promise<void> {
+  return saveHiddenModels(ids);
+}
+
+export function isHidden(modelId: ModelId, hidden: string[] | undefined): boolean {
+  if (!hidden || hidden.length === 0) return false;
+  return modelMatchesAnyPattern(modelId, hidden) || modelMatchesAnyPattern(modelId.split("/").slice(1).join("/") as ModelId, hidden);
+}
+
+export function isVisible(modelId: ModelId, hidden: string[] | undefined, _allIds: ModelId[]): boolean {
+  if (!hidden || hidden.length === 0) return true;
+  return !isHidden(modelId, hidden);
+}
+
+export function getVisibilityMap(all: Array<{provider:string;id:string}>, hidden: string[]|undefined): Map<ModelId, boolean> {
+  const map = new Map<ModelId, boolean>();
+  const allIds = all.map(toModelId);
+  for (const m of all) map.set(toModelId(m), isVisible(toModelId(m), hidden, allIds));
+  return map;
+}
+
+// Hide/show operates on the FULL model list (allIds = full catalogue).
+// hidden is a denylist: adding = hide, removing = show.
+export function toggleHidden(modelId: ModelId, hidden: string[] | undefined, _allIds: ModelId[]): string[] | undefined {
+  if (!hidden || hidden.length === 0) {
+    return [modelId];
+  }
+  // Exact match toggle. If hidden contains globs that already hide this model, we still toggle exact id.
+  // If model is hidden via pattern, toggling will add exact? Instead remove matching patterns? Simplify: exact toggle.
+  const hasExact = hidden.includes(modelId);
+  // Also consider pattern hiding: if model matches a pattern, showing should remove? But we can't know which pattern.
+  // For now, if hidden via pattern and not exact, toggling to "show" should add an exception? Simpler: if currently hidden (pattern or exact), remove exact and also remove any pattern that matches it? That's ambiguous.
+  // We treat visibility via pattern: if isHidden true but not exact, toggling "show" means we need to keep patterns but add exception — not supported.
+  // Simplest: if currently hidden, remove exact if present, otherwise add exact and keep patterns — model stays hidden. To actually show, we'd need to remove matching patterns.
+  // So if hidden via pattern, show action should remove matching patterns.
+  const currentlyHidden = isHidden(modelId, hidden);
+  if (currentlyHidden) {
+    // Remove exact if present
+    let next = hidden.filter(id => id !== modelId);
+    // Also remove any glob pattern that matches this model (so it becomes visible)
+    next = next.filter(p => !matchesPattern(modelId, p));
+    if (next.length === 0) return undefined;
+    return next;
+  } else {
+    // Currently visible — hide by adding exact id
+    if (hasExact) return hidden.filter(id => id !== modelId);
+    return [...hidden, modelId];
+  }
+}
+
+// Aliases for old names
+export function toggleInEnabled(modelId: ModelId, hidden: string[] | undefined, allIds: ModelId[]): string[] | undefined {
+  return toggleHidden(modelId, hidden, allIds);
+}
+
+export function applyBulk(hidden: string[]|undefined, allIds: ModelId[], matching: ModelId[], action: "include"|"exclude"): string[]|undefined {
+  // For denylist: "exclude" = hide (add to hidden), "include" = show (remove from hidden)
+  if (action === "exclude") {
+    const base = hidden === undefined ? [] : [...hidden];
+    const set = new Set(base);
+    for (const id of matching) set.add(id);
+    const arr = [...set].sort();
+    if (arr.length === 0) return undefined;
+    return arr;
+  } else {
+    // include = show
+    if (!hidden || hidden.length === 0) return undefined;
+    // Remove exact ids and any pattern that matches them? For now remove exact and patterns that match
+    let next = [...hidden];
+    for (const id of matching) {
+      next = next.filter(p => p !== id && !matchesPattern(id, p));
+      // If p is exact id matching model, remove; if p is glob that would match id, also remove? But glob may hide other models — removing it would unhide others unintentionally.
+      // Better only remove exact matches for bulk include, keep globs.
+      // Rebuild: only remove exact ids
+    }
+    // Recompute strictly exact removal to avoid over-removing globs
+    const exactSet = new Set(matching as string[]);
+    next = (hidden as string[]).filter(p => {
+      // If p is exact id in matching, remove
+      if (exactSet.has(p)) return false;
+      return true;
+    });
+    if (next.length === 0) return undefined;
+    return next;
+  }
+}