@asterxsk/kiln 0.3.0 → 0.4.2

package/agent/extensions/skillsconf/package.json

@@ -1,18 +1,18 @@
-{
-  "name": "skillsconf",
-  "private": true,
-  "type": "module",
-  "version": "0.1.0",
-  "description": "Per-skill visibility manager with packages (groups) for mass enable/disable",
-  "scripts": {
-    "check": "tsc --noEmit"
-  },
-  "devDependencies": {
-    "@types/node": "^26.4.1",
-    "typescript": "^5.6.0"
-  },
-  "allowScripts": {},
-  "dependencies": {
-    "@earendil-works/pi-coding-agent": "^0.84.4"
-  }
-}
+{
+  "name": "skillsconf",
+  "private": true,
+  "type": "module",
+  "version": "0.1.0",
+  "description": "Per-skill visibility manager with packages (groups) for mass enable/disable",
+  "scripts": {
+    "check": "tsc --noEmit"
+  },
+  "devDependencies": {
+    "@types/node": "^26.4.1",
+    "typescript": "^5.6.0"
+  },
+  "allowScripts": {},
+  "dependencies": {
+    "@earendil-works/pi-coding-agent": "^0.84.4"
+  }
+}

package/agent/extensions/skillsconf/src/delete-skill.test.ts

@@ -1,64 +1,64 @@
-import assert from "node:assert";
-import fs from "node:fs";
-import os from "node:os";
-import path from "node:path";
-import { deleteSkillFiles, resolveSkillDeleteTarget } from "./delete-skill.js";
-
-// resolve: SKILL.md -> parent dir
-assert.equal(
-  resolveSkillDeleteTarget(path.join("C:", "s", "foo", "SKILL.md")),
-  path.join("C:", "s", "foo"),
-);
-// resolve: lone file -> itself
-assert.equal(
-  resolveSkillDeleteTarget(path.join("C:", "s", "solo.md")),
-  path.join("C:", "s", "solo.md"),
-);
-// resolve: empty / relative -> null
-assert.equal(resolveSkillDeleteTarget(""), null);
-assert.equal(resolveSkillDeleteTarget("relative/SKILL.md"), null);
-
-// delete: skill dir with SKILL.md (+ extra files) goes away entirely
-{
-  const dir = fs.mkdtempSync(path.join(os.tmpdir(), "skillsconf-del-"));
-  const skillDir = path.join(dir, "my-skill");
-  fs.mkdirSync(path.join(skillDir, "scripts"), { recursive: true });
-  fs.writeFileSync(path.join(skillDir, "SKILL.md"), "# test");
-  fs.writeFileSync(path.join(skillDir, "scripts", "x.py"), "x");
-  const removed = await deleteSkillFiles(path.join(skillDir, "SKILL.md"));
-  assert.equal(removed, skillDir);
-  assert.equal(fs.existsSync(skillDir), false);
-  fs.rmSync(dir, { recursive: true, force: true });
-}
-
-// delete: lone .md file
-{
-  const dir = fs.mkdtempSync(path.join(os.tmpdir(), "skillsconf-del-"));
-  const file = path.join(dir, "solo.md");
-  fs.writeFileSync(file, "# test");
-  assert.equal(await deleteSkillFiles(file), file);
-  assert.equal(fs.existsSync(file), false);
-  fs.rmSync(dir, { recursive: true, force: true });
-}
-
-// delete: missing path throws, touches nothing
-{
-  const dir = fs.mkdtempSync(path.join(os.tmpdir(), "skillsconf-del-"));
-  await assert.rejects(deleteSkillFiles(path.join(dir, "nope", "SKILL.md")), /not found/);
-  fs.rmSync(dir, { recursive: true, force: true });
-}
-
-// delete: relative path refused
-await assert.rejects(deleteSkillFiles("some/SKILL.md"), /non-absolute/);
-
-// delete: type mismatch refused (non-SKILL.md path pointing at a directory)
-{
-  const dir = fs.mkdtempSync(path.join(os.tmpdir(), "skillsconf-del-"));
-  const sub = path.join(dir, "notmd");
-  fs.mkdirSync(sub);
-  await assert.rejects(deleteSkillFiles(sub), /expected a file/);
-  assert.equal(fs.existsSync(sub), true, "refused delete leaves dir alone");
-  fs.rmSync(dir, { recursive: true, force: true });
-}
-
-console.log("delete-skill.test.ts: all assertions passed");
+import assert from "node:assert";
+import fs from "node:fs";
+import os from "node:os";
+import path from "node:path";
+import { deleteSkillFiles, resolveSkillDeleteTarget } from "./delete-skill.js";
+
+// resolve: SKILL.md -> parent dir
+assert.equal(
+  resolveSkillDeleteTarget(path.join("C:", "s", "foo", "SKILL.md")),
+  path.join("C:", "s", "foo"),
+);
+// resolve: lone file -> itself
+assert.equal(
+  resolveSkillDeleteTarget(path.join("C:", "s", "solo.md")),
+  path.join("C:", "s", "solo.md"),
+);
+// resolve: empty / relative -> null
+assert.equal(resolveSkillDeleteTarget(""), null);
+assert.equal(resolveSkillDeleteTarget("relative/SKILL.md"), null);
+
+// delete: skill dir with SKILL.md (+ extra files) goes away entirely
+{
+  const dir = fs.mkdtempSync(path.join(os.tmpdir(), "skillsconf-del-"));
+  const skillDir = path.join(dir, "my-skill");
+  fs.mkdirSync(path.join(skillDir, "scripts"), { recursive: true });
+  fs.writeFileSync(path.join(skillDir, "SKILL.md"), "# test");
+  fs.writeFileSync(path.join(skillDir, "scripts", "x.py"), "x");
+  const removed = await deleteSkillFiles(path.join(skillDir, "SKILL.md"));
+  assert.equal(removed, skillDir);
+  assert.equal(fs.existsSync(skillDir), false);
+  fs.rmSync(dir, { recursive: true, force: true });
+}
+
+// delete: lone .md file
+{
+  const dir = fs.mkdtempSync(path.join(os.tmpdir(), "skillsconf-del-"));
+  const file = path.join(dir, "solo.md");
+  fs.writeFileSync(file, "# test");
+  assert.equal(await deleteSkillFiles(file), file);
+  assert.equal(fs.existsSync(file), false);
+  fs.rmSync(dir, { recursive: true, force: true });
+}
+
+// delete: missing path throws, touches nothing
+{
+  const dir = fs.mkdtempSync(path.join(os.tmpdir(), "skillsconf-del-"));
+  await assert.rejects(deleteSkillFiles(path.join(dir, "nope", "SKILL.md")), /not found/);
+  fs.rmSync(dir, { recursive: true, force: true });
+}
+
+// delete: relative path refused
+await assert.rejects(deleteSkillFiles("some/SKILL.md"), /non-absolute/);
+
+// delete: type mismatch refused (non-SKILL.md path pointing at a directory)
+{
+  const dir = fs.mkdtempSync(path.join(os.tmpdir(), "skillsconf-del-"));
+  const sub = path.join(dir, "notmd");
+  fs.mkdirSync(sub);
+  await assert.rejects(deleteSkillFiles(sub), /expected a file/);
+  assert.equal(fs.existsSync(sub), true, "refused delete leaves dir alone");
+  fs.rmSync(dir, { recursive: true, force: true });
+}
+
+console.log("delete-skill.test.ts: all assertions passed");

package/agent/extensions/skillsconf/src/delete-skill.ts

@@ -1,45 +1,45 @@
-import fs from "node:fs/promises";
-import os from "node:os";
-import path from "node:path";
-
-/**
- * Resolve what `d` on a skill should remove from disk:
- * - a `SKILL.md` file means the skill owns its parent directory -> remove the dir
- * - a lone `.md` file (root-level skill) -> remove just the file
- * Returns null when the location is missing or not absolute (never delete
- * relative to an assumed cwd).
- */
-export function resolveSkillDeleteTarget(filePath: string): string | null {
-  if (!filePath || !path.isAbsolute(filePath)) return null;
-  const normalized = path.normalize(filePath);
-  if (path.basename(normalized) === "SKILL.md") return path.dirname(normalized);
-  return normalized;
-}
-
-/**
- * Delete a skill from disk. Throws on anything unexpected; callers surface
- * the message. Refuses filesystem root, home, and items directly in home.
- * Returns the removed path.
- */
-export async function deleteSkillFiles(filePath: string): Promise<string> {
-  const target = resolveSkillDeleteTarget(filePath);
-  if (!target) throw new Error("unknown or non-absolute skill location");
-  const root = path.parse(target).root;
-  if (target === root) throw new Error(`refusing to delete filesystem root (${target})`);
-  const home = path.normalize(os.homedir());
-  if (target === home) throw new Error("refusing to delete home directory");
-  if (path.dirname(target) === home) {
-    throw new Error("refusing to delete items directly in home directory");
-  }
-  let st: Awaited<ReturnType<typeof fs.stat>>;
-  try {
-    st = await fs.stat(target);
-  } catch {
-    throw new Error(`not found: ${target}`);
-  }
-  const wantsDir = path.basename(path.normalize(filePath)) === "SKILL.md";
-  if (wantsDir && !st.isDirectory()) throw new Error(`expected a directory: ${target}`);
-  if (!wantsDir && !st.isFile()) throw new Error(`expected a file: ${target}`);
-  await fs.rm(target, { recursive: true });
-  return target;
-}
+import fs from "node:fs/promises";
+import os from "node:os";
+import path from "node:path";
+
+/**
+ * Resolve what `d` on a skill should remove from disk:
+ * - a `SKILL.md` file means the skill owns its parent directory -> remove the dir
+ * - a lone `.md` file (root-level skill) -> remove just the file
+ * Returns null when the location is missing or not absolute (never delete
+ * relative to an assumed cwd).
+ */
+export function resolveSkillDeleteTarget(filePath: string): string | null {
+  if (!filePath || !path.isAbsolute(filePath)) return null;
+  const normalized = path.normalize(filePath);
+  if (path.basename(normalized) === "SKILL.md") return path.dirname(normalized);
+  return normalized;
+}
+
+/**
+ * Delete a skill from disk. Throws on anything unexpected; callers surface
+ * the message. Refuses filesystem root, home, and items directly in home.
+ * Returns the removed path.
+ */
+export async function deleteSkillFiles(filePath: string): Promise<string> {
+  const target = resolveSkillDeleteTarget(filePath);
+  if (!target) throw new Error("unknown or non-absolute skill location");
+  const root = path.parse(target).root;
+  if (target === root) throw new Error(`refusing to delete filesystem root (${target})`);
+  const home = path.normalize(os.homedir());
+  if (target === home) throw new Error("refusing to delete home directory");
+  if (path.dirname(target) === home) {
+    throw new Error("refusing to delete items directly in home directory");
+  }
+  let st: Awaited<ReturnType<typeof fs.stat>>;
+  try {
+    st = await fs.stat(target);
+  } catch {
+    throw new Error(`not found: ${target}`);
+  }
+  const wantsDir = path.basename(path.normalize(filePath)) === "SKILL.md";
+  if (wantsDir && !st.isDirectory()) throw new Error(`expected a directory: ${target}`);
+  if (!wantsDir && !st.isFile()) throw new Error(`expected a file: ${target}`);
+  await fs.rm(target, { recursive: true });
+  return target;
+}

package/agent/extensions/skillsconf/src/filter.test.ts

@@ -1,80 +1,80 @@
-import assert from "node:assert";
-import { filterSkillsPrompt } from "./filter.js";
-
-function samplePrompt(): string {
-  return [
-    "You are a coding agent.",
-    "",
-    "The following skills provide specialized instructions for specific tasks.",
-    "Use the read tool to load a skill's file when the task matches its description.",
-    "When a skill file references a relative path, resolve it against the skill directory (parent of SKILL.md / dirname of the path) and use that absolute path in tool commands.",
-    "",
-    "<available_skills>",
-    "  <skill>",
-    "    <name>alpha</name>",
-    "    <description>First skill</description>",
-    "    <location>/skills/alpha/SKILL.md</location>",
-    "  </skill>",
-    "  <skill>",
-    "    <name>beta</name>",
-    "    <description>Second skill</description>",
-    "    <location>/skills/beta/SKILL.md</location>",
-    "  </skill>",
-    "  <skill>",
-    "    <name>gamma</name>",
-    "    <description>Third skill</description>",
-    "    <location>/skills/gamma/SKILL.md</location>",
-    "  </skill>",
-    "</available_skills>",
-    "",
-    "Trailing instructions.",
-  ].join("\n");
-}
-
-const base = samplePrompt();
-
-// no disabled -> identical
-assert.equal(filterSkillsPrompt(base, []), base);
-assert.equal(filterSkillsPrompt(base, undefined), base);
-
-// unknown name -> identical
-assert.equal(filterSkillsPrompt(base, ["nope"]), base);
-
-// disable one -> only its block removed, rest verbatim
-const one = filterSkillsPrompt(base, ["beta"]);
-assert.ok(!one.includes("<name>beta</name>"), "beta block removed");
-assert.ok(one.includes("<name>alpha</name>"), "alpha kept");
-assert.ok(one.includes("<name>gamma</name>"), "gamma kept");
-assert.ok(one.includes("<available_skills>"), "wrapper kept");
-assert.ok(one.includes("Trailing instructions."), "tail kept");
-assert.ok(one.includes("You are a coding agent."), "head kept");
-
-// disable two
-const two = filterSkillsPrompt(base, ["alpha", "gamma"]);
-assert.ok(!two.includes("<name>alpha</name>"));
-assert.ok(!two.includes("<name>gamma</name>"));
-assert.ok(two.includes("<name>beta</name>"));
-assert.ok(two.includes("<available_skills>"));
-
-// disable all -> whole section gone, no stray blank lines
-const all = filterSkillsPrompt(base, ["alpha", "beta", "gamma"]);
-assert.ok(!all.includes("<available_skills>"), "wrapper removed");
-assert.ok(!all.includes("provide specialized instructions"), "intro removed");
-assert.ok(!all.includes("<name>alpha</name>"));
-assert.equal(all, "You are a coding agent.\n\nTrailing instructions.");
-
-// prompt without a skills section -> identical
-assert.equal(filterSkillsPrompt("plain prompt", ["alpha"]), "plain prompt");
-
-// XML-escaped names match
-const escaped = base.replace("<name>beta</name>", "<name>a&lt;b&gt;c</name>");
-const escFiltered = filterSkillsPrompt(escaped, ["a<b>c"]);
-assert.ok(!escFiltered.includes("a&lt;b&gt;c"), "escaped block removed");
-assert.ok(escFiltered.includes("<name>alpha</name>"));
-
-// Set input works
-const viaSet = filterSkillsPrompt(base, new Set(["gamma"]));
-assert.ok(!viaSet.includes("<name>gamma</name>"));
-assert.ok(viaSet.includes("<name>alpha</name>"));
-
-console.log("filter.test.ts: all assertions passed");
+import assert from "node:assert";
+import { filterSkillsPrompt } from "./filter.js";
+
+function samplePrompt(): string {
+  return [
+    "You are a coding agent.",
+    "",
+    "The following skills provide specialized instructions for specific tasks.",
+    "Use the read tool to load a skill's file when the task matches its description.",
+    "When a skill file references a relative path, resolve it against the skill directory (parent of SKILL.md / dirname of the path) and use that absolute path in tool commands.",
+    "",
+    "<available_skills>",
+    "  <skill>",
+    "    <name>alpha</name>",
+    "    <description>First skill</description>",
+    "    <location>/skills/alpha/SKILL.md</location>",
+    "  </skill>",
+    "  <skill>",
+    "    <name>beta</name>",
+    "    <description>Second skill</description>",
+    "    <location>/skills/beta/SKILL.md</location>",
+    "  </skill>",
+    "  <skill>",
+    "    <name>gamma</name>",
+    "    <description>Third skill</description>",
+    "    <location>/skills/gamma/SKILL.md</location>",
+    "  </skill>",
+    "</available_skills>",
+    "",
+    "Trailing instructions.",
+  ].join("\n");
+}
+
+const base = samplePrompt();
+
+// no disabled -> identical
+assert.equal(filterSkillsPrompt(base, []), base);
+assert.equal(filterSkillsPrompt(base, undefined), base);
+
+// unknown name -> identical
+assert.equal(filterSkillsPrompt(base, ["nope"]), base);
+
+// disable one -> only its block removed, rest verbatim
+const one = filterSkillsPrompt(base, ["beta"]);
+assert.ok(!one.includes("<name>beta</name>"), "beta block removed");
+assert.ok(one.includes("<name>alpha</name>"), "alpha kept");
+assert.ok(one.includes("<name>gamma</name>"), "gamma kept");
+assert.ok(one.includes("<available_skills>"), "wrapper kept");
+assert.ok(one.includes("Trailing instructions."), "tail kept");
+assert.ok(one.includes("You are a coding agent."), "head kept");
+
+// disable two
+const two = filterSkillsPrompt(base, ["alpha", "gamma"]);
+assert.ok(!two.includes("<name>alpha</name>"));
+assert.ok(!two.includes("<name>gamma</name>"));
+assert.ok(two.includes("<name>beta</name>"));
+assert.ok(two.includes("<available_skills>"));
+
+// disable all -> whole section gone, no stray blank lines
+const all = filterSkillsPrompt(base, ["alpha", "beta", "gamma"]);
+assert.ok(!all.includes("<available_skills>"), "wrapper removed");
+assert.ok(!all.includes("provide specialized instructions"), "intro removed");
+assert.ok(!all.includes("<name>alpha</name>"));
+assert.equal(all, "You are a coding agent.\n\nTrailing instructions.");
+
+// prompt without a skills section -> identical
+assert.equal(filterSkillsPrompt("plain prompt", ["alpha"]), "plain prompt");
+
+// XML-escaped names match
+const escaped = base.replace("<name>beta</name>", "<name>a&lt;b&gt;c</name>");
+const escFiltered = filterSkillsPrompt(escaped, ["a<b>c"]);
+assert.ok(!escFiltered.includes("a&lt;b&gt;c"), "escaped block removed");
+assert.ok(escFiltered.includes("<name>alpha</name>"));
+
+// Set input works
+const viaSet = filterSkillsPrompt(base, new Set(["gamma"]));
+assert.ok(!viaSet.includes("<name>gamma</name>"));
+assert.ok(viaSet.includes("<name>alpha</name>"));
+
+console.log("filter.test.ts: all assertions passed");

package/agent/extensions/skillsconf/src/filter.ts

@@ -1,74 +1,74 @@
-// Removes disabled skills from a built system prompt.
-//
-// pi renders loaded skills as a verbatim XML block:
-//
-//   <available_skills>
-//     <skill>
-//       <name>...</name>
-//       <description>...</description>
-//       <location>...</location>
-//     </skill>
-//     ...
-//   </available_skills>
-//
-// Filtering is done by verbatim block surgery (kept blocks are untouched),
-// so prompt output never depends on re-escaping. When every skill is
-// disabled the whole section (intro + wrapper) is removed.
-
-const SECTION_INTRO = "The following skills provide specialized instructions for specific tasks.";
-const OPEN_TAG = "<available_skills>";
-const CLOSE_TAG = "</available_skills>";
-
-function unescapeXml(s: string): string {
-  return s
-    .replace(/&apos;/g, "'")
-    .replace(/&quot;/g, '"')
-    .replace(/&gt;/g, ">")
-    .replace(/&lt;/g, "<")
-    .replace(/&amp;/g, "&");
-}
-
-export function filterSkillsPrompt(
-  systemPrompt: string,
-  disabled: string[] | Set<string> | undefined,
-): string {
-  const disabledSet = disabled instanceof Set ? disabled : new Set(disabled ?? []);
-  if (disabledSet.size === 0) return systemPrompt;
-
-  // Locate the section. The intro is preceded by two newlines in the template;
-  // include them so full removal leaves no stray blank lines.
-  let start = systemPrompt.indexOf("\n\n" + SECTION_INTRO);
-  let introLen = 2 + SECTION_INTRO.length;
-  if (start === -1) {
-    start = systemPrompt.indexOf(SECTION_INTRO);
-    introLen = SECTION_INTRO.length;
-    if (start === -1) return systemPrompt;
-  }
-  const openIdx = systemPrompt.indexOf(OPEN_TAG, start);
-  if (openIdx === -1) return systemPrompt;
-  const closeIdx = systemPrompt.indexOf(CLOSE_TAG, openIdx);
-  if (closeIdx === -1) return systemPrompt;
-
-  const bodyStart = openIdx + OPEN_TAG.length;
-  const closeEnd = closeIdx + CLOSE_TAG.length;
-  const body = systemPrompt.slice(bodyStart, closeIdx);
-
-  const blockRe = /  <skill>\n[\s\S]*?\n  <\/skill>/g;
-  const blocks = body.match(blockRe);
-  if (!blocks) return systemPrompt;
-
-  const kept: string[] = [];
-  let removed = 0;
-  for (const b of blocks) {
-    const m = b.match(/<name>([\s\S]*?)<\/name>/);
-    const name = m ? unescapeXml(m[1]) : "";
-    if (name && disabledSet.has(name)) removed++;
-    else kept.push(b);
-  }
-  if (removed === 0) return systemPrompt;
-  if (kept.length === 0) {
-    // Drop the entire section including its leading blank lines.
-    return systemPrompt.slice(0, start) + systemPrompt.slice(closeEnd);
-  }
-  return systemPrompt.slice(0, bodyStart) + "\n" + kept.join("\n") + "\n" + systemPrompt.slice(closeIdx);
-}
+// Removes disabled skills from a built system prompt.
+//
+// pi renders loaded skills as a verbatim XML block:
+//
+//   <available_skills>
+//     <skill>
+//       <name>...</name>
+//       <description>...</description>
+//       <location>...</location>
+//     </skill>
+//     ...
+//   </available_skills>
+//
+// Filtering is done by verbatim block surgery (kept blocks are untouched),
+// so prompt output never depends on re-escaping. When every skill is
+// disabled the whole section (intro + wrapper) is removed.
+
+const SECTION_INTRO = "The following skills provide specialized instructions for specific tasks.";
+const OPEN_TAG = "<available_skills>";
+const CLOSE_TAG = "</available_skills>";
+
+function unescapeXml(s: string): string {
+  return s
+    .replace(/&apos;/g, "'")
+    .replace(/&quot;/g, '"')
+    .replace(/&gt;/g, ">")
+    .replace(/&lt;/g, "<")
+    .replace(/&amp;/g, "&");
+}
+
+export function filterSkillsPrompt(
+  systemPrompt: string,
+  disabled: string[] | Set<string> | undefined,
+): string {
+  const disabledSet = disabled instanceof Set ? disabled : new Set(disabled ?? []);
+  if (disabledSet.size === 0) return systemPrompt;
+
+  // Locate the section. The intro is preceded by two newlines in the template;
+  // include them so full removal leaves no stray blank lines.
+  let start = systemPrompt.indexOf("\n\n" + SECTION_INTRO);
+  let introLen = 2 + SECTION_INTRO.length;
+  if (start === -1) {
+    start = systemPrompt.indexOf(SECTION_INTRO);
+    introLen = SECTION_INTRO.length;
+    if (start === -1) return systemPrompt;
+  }
+  const openIdx = systemPrompt.indexOf(OPEN_TAG, start);
+  if (openIdx === -1) return systemPrompt;
+  const closeIdx = systemPrompt.indexOf(CLOSE_TAG, openIdx);
+  if (closeIdx === -1) return systemPrompt;
+
+  const bodyStart = openIdx + OPEN_TAG.length;
+  const closeEnd = closeIdx + CLOSE_TAG.length;
+  const body = systemPrompt.slice(bodyStart, closeIdx);
+
+  const blockRe = /  <skill>\n[\s\S]*?\n  <\/skill>/g;
+  const blocks = body.match(blockRe);
+  if (!blocks) return systemPrompt;
+
+  const kept: string[] = [];
+  let removed = 0;
+  for (const b of blocks) {
+    const m = b.match(/<name>([\s\S]*?)<\/name>/);
+    const name = m ? unescapeXml(m[1]) : "";
+    if (name && disabledSet.has(name)) removed++;
+    else kept.push(b);
+  }
+  if (removed === 0) return systemPrompt;
+  if (kept.length === 0) {
+    // Drop the entire section including its leading blank lines.
+    return systemPrompt.slice(0, start) + systemPrompt.slice(closeEnd);
+  }
+  return systemPrompt.slice(0, bodyStart) + "\n" + kept.join("\n") + "\n" + systemPrompt.slice(closeIdx);
+}

package/agent/extensions/skillsconf/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);
+}