pi-soly 2.4.2 → 2.5.1

package/mcp/mcp-panel.ts

@@ -1,5 +1,7 @@
 import { matchesKey, truncateToWidth, visibleWidth } from "@earendil-works/pi-tui";
 import { createPanelKeys, type PanelKeybindings, type PanelKeys } from "../visual/panel-keys.ts";
+import { fuzzyScore } from "../visual/fuzzy.ts";
+import { borderTop, borderBottom, borderDivider, borderRow, borderEmpty, type BorderStyler } from "../visual/border.ts";
 import { isToolExcluded } from "./types.ts";
 import type { McpConfig, McpPanelCallbacks, McpPanelResult, ServerProvenance } from "./types.ts";
 import { resourceNameToToolName } from "./resource-tools.ts";
@@ -55,24 +57,7 @@ function rainbowProgress(filled: number, total: number): string {
   return dots.join(" ");
 }
 
-function fuzzyScore(query: string, text: string): number {
-  const lq = query.toLowerCase();
-  const lt = text.toLowerCase();
-  if (lt.includes(lq)) return 100 + (lq.length / lt.length) * 50;
-  let score = 0;
-  let qi = 0;
-  let consecutive = 0;
-  for (let i = 0; i < lt.length && qi < lq.length; i++) {
-    if (lt[i] === lq[qi]) {
-      score += 10 + consecutive;
-      consecutive += 5;
-      qi++;
-    } else {
-      consecutive = 0;
-    }
-  }
-  return qi === lq.length ? score : 0;
-}
+// fuzzyScore moved to ../visual/fuzzy.ts — imported above.
 
 function estimateTokens(tool: CachedTool): number {
   const schemaLen = JSON.stringify(tool.inputSchema ?? {}).length;
@@ -599,18 +584,15 @@ class McpPanel {
     const italic = (s: string) => `\x1b[3m${s}\x1b[23m`;
     const inverse = (s: string) => `\x1b[7m${s}\x1b[27m`;
 
-    const row = (content: string) =>
-      fg(t.border, "│") + truncateToWidth(" " + content, innerW, "…", true) + fg(t.border, "│");
-    const emptyRow = () => fg(t.border, "│") + " ".repeat(innerW) + fg(t.border, "│");
-    const divider = () => fg(t.border, "├" + "─".repeat(innerW) + "┤");
+    // Shared border styler — colors border chars via the panel theme.
+    const borderS: BorderStyler = (s: string) => fg(t.border, s);
+    const titleS: BorderStyler = (s: string) => fg(t.title, s);
+    const row = (content: string) => borderRow(content, innerW, borderS);
+    const emptyRow = () => borderEmpty(innerW, borderS);
+    const divider = () => borderDivider(innerW, borderS);
 
-    const titleText = this.authOnly ? " MCP OAuth " : " MCP Servers ";
-    const borderLen = innerW - visibleWidth(titleText);
-    const leftB = Math.floor(borderLen / 2);
-    const rightB = borderLen - leftB;
-    lines.push(fg(t.border, "╭" + "─".repeat(leftB)) + fg(t.title, titleText) + fg(t.border, "─".repeat(rightB) + "╮"));
-
-    lines.push(emptyRow());
+    const titleText = this.authOnly ? "MCP OAuth" : "MCP Servers";
+    lines.push(borderTop(titleText, innerW, borderS, titleS));
 
     const cursor = fg(t.selected, "│");
     const searchIcon = fg(t.border, "◎");
@@ -622,27 +604,21 @@ class McpPanel {
       lines.push(row(`${searchIcon}  ${fg(t.placeholder, italic("search..."))}`));
     }
 
-    lines.push(emptyRow());
     if (this.noticeLines.length > 0) {
       for (const notice of this.noticeLines) {
         lines.push(row(fg(t.hint, italic(notice))));
       }
-      lines.push(emptyRow());
     }
     lines.push(divider());
 
     if (this.servers.length === 0) {
-      lines.push(emptyRow());
       lines.push(row(fg(t.hint, italic(this.authOnly ? "No OAuth-capable MCP servers configured." : "No MCP servers configured."))));
-      lines.push(emptyRow());
     } else {
       const maxVis = McpPanel.MAX_VISIBLE;
       const total = this.visibleItems.length;
       const startIdx = Math.max(0, Math.min(this.cursorIndex - Math.floor(maxVis / 2), total - maxVis));
       const endIdx = Math.min(startIdx + maxVis, total);
 
-      lines.push(emptyRow());
-
       for (let i = startIdx; i < endIdx; i++) {
         const item = this.visibleItems[i];
         const isCursor = i === this.cursorIndex;
@@ -655,26 +631,20 @@ class McpPanel {
         }
       }
 
-      lines.push(emptyRow());
-
       if (total > maxVis) {
         const prog = Math.round(((this.cursorIndex + 1) / total) * 10);
         lines.push(row(`${rainbowProgress(prog, 10)}  ${fg(t.hint, `${this.cursorIndex + 1}/${total}`)}`));
-        lines.push(emptyRow());
       }
 
       if (this.importNotice) {
         lines.push(row(fg(t.needsAuth, italic(this.importNotice))));
-        lines.push(emptyRow());
       }
       if (this.authNotice) {
         lines.push(row(fg(t.needsAuth, italic(this.authNotice))));
-        lines.push(emptyRow());
       }
     }
 
     lines.push(divider());
-    lines.push(emptyRow());
 
     if (this.confirmingDiscard) {
       const discardBtn = this.discardSelected === 0
@@ -699,7 +669,6 @@ class McpPanel {
       }
     }
 
-    lines.push(emptyRow());
     const hints = this.authOnly
       ? [
           italic("↑↓") + " navigate",
@@ -738,8 +707,7 @@ class McpPanel {
     }
     if (curLine) lines.push(row(fg(t.hint, curLine)));
 
-    lines.push(fg(t.border, "╰" + "─".repeat(innerW) + "╯"));
-
+    lines.push(borderBottom(innerW, borderS));
     return lines;
   }
 

package/mcp/mcp-setup-panel.ts

@@ -1,5 +1,6 @@
 import { matchesKey, truncateToWidth, visibleWidth } from "@earendil-works/pi-tui";
 import { createPanelKeys, type PanelKeybindings, type PanelKeys } from "../visual/panel-keys.ts";
+import { borderTop, borderBottom, borderDivider, borderRow, borderEmpty, type BorderStyler } from "../visual/border.ts";
 import type { ImportKind } from "./types.ts";
 import type { ConfigWritePreview, McpDiscoverySummary } from "./config.ts";
 import type { McpOnboardingState } from "./onboarding-state.ts";
@@ -345,22 +346,24 @@ export class McpSetupPanel {
   render(width: number): string[] {
     const innerW = Math.max(40, width - 2);
     const lines: string[] = [];
-    const border = fg(this.t.border, "─".repeat(innerW));
-    lines.push(`┌${border}┐`);
-    lines.push(this.padLine(fg(this.t.title, "MCP setup"), innerW));
-    lines.push(this.padLine(this.discoverySummaryLine(), innerW));
-    lines.push(this.padLine(fg(this.t.muted, this.secondarySummaryLine()), innerW));
-    lines.push(this.padLine("", innerW));
+    const borderS: BorderStyler = (s: string) => fg(this.t.border, s);
+    const row = (content: string) => borderRow(content, innerW, borderS);
+
+    lines.push(borderTop("", innerW, borderS));
+    lines.push(row(fg(this.t.title, "MCP setup")));
+    lines.push(row(this.discoverySummaryLine()));
+    lines.push(row(fg(this.t.muted, this.secondarySummaryLine())));
+    lines.push(borderEmpty(innerW, borderS));
 
     if (this.notice) {
       const tone = this.notice.tone === "success" ? this.t.success : this.notice.tone === "warning" ? this.t.warning : this.t.hint;
       for (const line of wrapText(this.notice.text, innerW - 6)) {
-        lines.push(this.padLine(fg(tone, line), innerW));
+        lines.push(row(fg(tone, line)));
       }
-      lines.push(this.padLine("", innerW));
+      lines.push(borderEmpty(innerW, borderS));
     }
 
-    lines.push(`├${border}┤`);
+    lines.push(borderDivider(innerW, borderS));
 
     if (this.screen === "imports") {
       lines.push(...this.renderImports(innerW));
@@ -370,7 +373,7 @@ export class McpSetupPanel {
       lines.push(...this.renderActions(innerW));
     }
 
-    lines.push(`└${border}┘`);
+    lines.push(borderBottom(innerW, borderS));
     return lines;
   }
 
@@ -554,12 +557,7 @@ export class McpSetupPanel {
   }
 
   private padLine(text: string, innerW: number): string {
-    const inset = 2;
-    const contentW = Math.max(0, innerW - inset * 2);
-    const fitted = truncateToWidth(text, contentW, "…", true);
-    const plainWidth = visibleWidth(fitted);
-    const padding = Math.max(0, contentW - plainWidth);
-    return `│${" ".repeat(inset)}${fitted}${" ".repeat(padding)}${" ".repeat(inset)}│`;
+    return borderRow(text, innerW, (s) => fg(this.t.border, s));
   }
 
   invalidate(): void {}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-soly",
-  "version": "2.4.2",
+  "version": "2.5.1",
   "description": "Workflow + project management for pi-coding-agent. Plans, state, MANDATORY rules, self-review, multi-question picker. One npm install, zero config. LLM drives the workflow inline via the soly_workflow tool — no external subagent plugin.",
   "type": "module",
   "main": "index.ts",

package/skills/agent-coach/SKILL.md

@@ -0,0 +1,250 @@
+---
+name: agent-coach
+description: Use when the user complains about how the agent wrote code or behaves — phrases like "переделай X по-другому", "мне не нравится что agent делает Y", "почему опять Z", "не делай так", "agent keeps doing X", "I don't like the agent doing X", "заставь agent-а не делать X", "опять этот god switch". Analyzes the complaint, identifies the missing rule that would have prevented it, and proposes a soly rule draft (.agents/rules/*.md) for the user to confirm via ask_pro. Language-agnostic — works for any codebase. NOT for C#/.NET analyzer rules (use analyzer-coach for that).
+priority: high
+---
+
+# agent-coach
+
+Convert a user complaint about agent behavior into a **soly rule** — a
+markdown instruction file in `.agents/rules/` that goes into the system
+prompt and prevents the agent from repeating the mistake.
+
+The feedback loop: user corrects → rule persists → agent doesn't repeat.
+Each complaint is a signal that a rule is **missing** (or exists but the
+agent ignored it — investigate which).
+
+## When to use me
+
+| Symptom in user's message | Use this skill? |
+|---|---|
+| "переделай X по-другому" / "redo X differently" | **Yes** |
+| "мне не нравится что agent делает Y" / "I don't like the agent doing Y" | **Yes** |
+| "почему опять Z" / "why does it keep doing Z" | **Yes** |
+| "не делай так" / "don't do that" | **Yes** |
+| "agent keeps doing X" / "заставь agent-а не делать X" | **Yes** |
+| Complaint about code style, naming, structure, approach | **Yes** |
+| C#/.NET style complaint (Roslynator/Meziantou/CA-*) | **No** — use `analyzer-coach` |
+| Formatting (indent, EOL, trailing whitespace) | **No** — `.editorconfig` |
+| Code style a linter can check (eslint/biome/prettier) | **No** — linter config |
+| Logic bug in the code | **No** — tests |
+| "this rule fires too much" | **Yes, but inverted** — the rule should go, not be added |
+
+## Mental model
+
+A complaint means the agent did X, and the user wanted Y. The rule's job is
+to make the agent do Y next time **without the user having to say it**.
+
+**Good rule:** "Use `cancellationToken` not `ct` for C# cancellation tokens.
+The short form is ambiguous in logs and stack traces." → Prevents the mistake.
+
+**Bad rule:** "Don't write bad code." → Too vague, not actionable, ignored.
+
+The rule must be **specific enough to act on** and **general enough to
+recurring**. One complaint → one rule about the pattern, not about the
+specific instance.
+
+## Workflow (always follow these steps in order)
+
+### Step 1 — Read the complaint + the code
+
+Before analyzing, **read**:
+- The user's complaint (what specifically bothers them — extract the
+  concrete behavior, not the emotion)
+- The code the agent wrote (the file(s) that triggered the complaint) —
+  use `read` / `soly_snippet`
+- The conversation context (what was the agent trying to do?)
+
+**Extract the pattern.** "переделай god switch по-другому" is about a
+specific instance, but the underlying pattern might be:
+- "Don't create God classes/switches" (architecture)
+- "Prefer polymorphism over giant switch statements" (design pattern)
+- "Split switches with >5 cases into a dispatch table" (specific threshold)
+
+Ask yourself: **what general principle, if the agent had known it, would
+have prevented this?**
+
+### Step 2 — Check existing rules (dedup)
+
+Before proposing, **read existing rules** to avoid duplication:
+- Use `soly_doc_search` with keywords from the complaint
+- Or `/rules list` to see all rule files
+- Or `soly_snippet` on `.agents/rules/*.md` files
+
+If a rule already covers this but the agent ignored it:
+- The rule may be too vague → propose **strengthening** it (edit, not new)
+- The rule may not apply to the file's glob → propose adding globs
+- The agent may have genuinely missed it → don't create a duplicate
+
+If no rule covers it → proceed to Step 3.
+
+### Step 3 — Categorize
+
+Pick **one** category (auto-detect from the complaint content):
+
+| Category | Path | Complaint is about |
+|---|---|---|
+| `coding` | `.agents/rules/coding/` | Code style, patterns, idioms, language conventions |
+| `architecture` | `.agents/rules/architecture/` | Structure, layering, coupling, dependencies |
+| `process` | `.agents/rules/process/` | Workflow, git hygiene, commit format, review process |
+| `testing` | `.agents/rules/testing/` | Test structure, coverage, naming, fixtures |
+| `naming` | `.agents/rules/naming/` | Identifier names, file names, conventions |
+
+If unsure, default to `coding/` — it's the catch-all for code-level rules.
+
+If the category directory doesn't exist, the `write` tool creates it
+automatically (it creates parent directories).
+
+### Step 4 — Draft the rule
+
+Write the rule in **soly rule format**. Structure:
+
+```markdown
+---
+description: "[one-line summary — what this rule enforces]"
+globs: ["**/*.ts", "**/*.tsx"]  # optional: file patterns this applies to
+priority: high  # high | medium | low (default medium)
+---
+
+# [Rule Name]
+
+> **[One-line imperative.]** [Why it matters — the cost of violating.]
+
+## The rule
+
+[Concrete, actionable instruction. "Do X" not "consider X".
+Reference the specific pattern. Include a threshold if applicable.]
+
+[✓ Good example — code block showing the right way]
+[✗ Bad example — code block showing what to avoid]
+
+## Why
+
+[The reasoning. Why does the user care? What goes wrong without this rule?
+This section is what convinces the LLM to follow it, not just obey it.]
+
+## Self-audit grep  # optional — only for grep-checkable rules
+
+[If the violation is detectable by grep/rg, include a command the agent
+can run to self-check. Skip for behavioral/process rules.]
+```
+
+**Writing principles:**
+- **Imperative mood:** "Use X" not "You should use X" or "X is preferred"
+- **Concrete threshold:** "Functions under 50 lines" not "Keep functions short"
+- **Show both sides:** ✓ good + ✗ bad code blocks make it unambiguous
+- **Explain why:** the "## Why" section is what makes the LLM internalize it
+- **No filler:** every line earns its place; rules eat system prompt budget
+
+### Step 5 — Propose via ask_pro
+
+**Do not write the file yet.** Show the draft to the user via `ask_pro`:
+
+```
+ask_pro({
+  questions: [{
+    header: "New rule",
+    question: "Create this rule to prevent the agent from repeating the issue?",
+    options: [
+      {
+        label: "Create rule",
+        description: "Write to .agents/rules/[category]/[name].md",
+        recommended: true,
+        preview: "<the full draft markdown in a fenced code block>"
+      },
+      {
+        label: "Edit existing",
+        description: "A similar rule exists — strengthen it instead",
+        preview: "<diff showing what to add>"
+      },
+      {
+        label: "Skip",
+        description: "Don't create a rule — just fix the code this time"
+      }
+    ]
+  }]
+})
+```
+
+The `preview` field is key — it shows the user exactly what they're approving
+in a side panel, formatted as code.
+
+### Step 6 — Write (if confirmed)
+
+If the user confirms "Create rule":
+1. Use `write` to create the file at the chosen path
+2. The hot-reload watcher picks it up automatically — no `/rules reload` needed
+3. Emit a confirmation via `emit()` (if in soly context) or just tell the user
+
+**File naming:** `kebab-case.md`, descriptive. `no-god-switches.md` not
+`rule1.md` or `architecture-rule.md`.
+
+## Anti-patterns (don't do these)
+
+- ❌ **Don't create a rule for a one-off.** If the issue is specific to one
+  file and unlikely to recur, just fix the code. Rules are for patterns.
+- ❌ **Don't create vague rules.** "Write clean code" is useless. "Functions
+  under 50 lines, extract helpers" is useful.
+- ❌ **Don't duplicate linters.** If eslint/biome/prettier can enforce it,
+  point the user there instead. Use `/rulewizard` to decide.
+- ❌ **Don't skip the "## Why" section.** Without reasoning, the LLM treats
+  the rule as arbitrary and may argue with it.
+- ❌ **Don't auto-write without confirmation.** The user may want to rephrase,
+  recategorize, or decide it's not rule-worthy. Always ask_pro first.
+- ❌ **Don't create multiple rules from one complaint.** One complaint → one
+  rule about the underlying pattern. Not five rules about symptoms.
+
+## Integration with soly
+
+- Rules created here are loaded by `loadAllRules()` on next session_start
+  (or immediately via hot-reload if soly's watcher is active)
+- They appear in `/rules list` and the system prompt's `## ⚠️ MANDATORY` block
+- Priority: project rules (`.agents/rules/`) = 3, local overrides
+  (`.agents/rules.local/`) = 5. Suggest `.agents/rules/` for team-shared
+  conventions, `.agents/rules.local/` for personal preferences
+- Frontmatter `globs` makes the rule apply only to matching files (saves
+  prompt budget — a C# rule doesn't need to load for .ts files)
+
+## Example session
+
+**User:** "переделай god switch в customer-service.ts, мне не нравится"
+
+**Agent (using this skill):**
+
+1. Reads `customer-service.ts` — finds a 200-line switch on customer type
+2. Searches existing rules — `/rules list` shows no switch-related rule
+3. Categorizes → `architecture/` (structural issue, not just style)
+4. Drafts:
+
+```markdown
+---
+description: "Avoid giant switch statements — use dispatch maps or polymorphism"
+globs: ["**/*.ts"]
+priority: medium
+---
+
+# No God Switches
+
+> **Switches with >5 cases must be refactored to a dispatch table.** Giant
+> switches are unmaintainable and violate the open-closed principle.
+
+## The rule
+
+When a `switch` statement exceeds 5 cases (or a single case exceeds 20
+lines), refactor to:
+- A `Record<Case, Handler>` dispatch map (for simple dispatch)
+- Polymorphism (when cases share behavior — subclasses + virtual method)
+
+[✓ good: dispatch map example]
+[✗ bad: 50-case switch example]
+
+## Why
+
+Giant switches grow linearly with new cases, are hard to test in isolation,
+and make the agent tend to add "just one more case" instead of refactoring.
+The user has to review the entire switch to verify one case is safe.
+```
+
+5. Shows via ask_pro with the full draft in `preview`
+6. User confirms → writes to `.agents/rules/architecture/no-god-switches.md`
+7. Hot-reload picks it up → rule is active for the next turn

package/visual/border.ts

@@ -0,0 +1,64 @@
+// =============================================================================
+// visual/border.ts — shared rounded-border renderer for modal panels
+// =============================================================================
+//
+// All soly modal panels (ListPanel, McpPanel, McpSetupPanel) draw bordered
+// boxes. Before this module, each had its own helpers:
+//   - list-panel:  frame(), headerLine(), footerLine(), ruleLine()  (┌┐└┘)
+//   - mcp-panel:   row(), emptyRow(), divider()                     (╭╮╰╯ + raw ANSI)
+//   - mcp-setup:   padLine()                                         (┌┐└┘)
+//
+// This module unifies them under one API with rounded corners (╭╮╰╯├┤│─).
+// Each function takes a `styler` ((s: string) => string) so it works with
+// any color system — pi's Theme.fg(), mcp-panel's fg(), or identity in tests.
+//
+// All width math uses pi-tui's visibleWidth/truncateToWidth so ANSI color
+// codes inside `content` never break alignment (same approach as list-panel).
+// =============================================================================
+
+import { truncateToWidth, visibleWidth } from "@earendil-works/pi-tui";
+
+/** Colors the border characters. Identity (s => s) in tests. */
+export type BorderStyler = (s: string) => string;
+
+/** Top border with a centered title: `╭── title ──╮`. */
+export function borderTop(title: string, innerW: number, styler: BorderStyler, titleStyler?: BorderStyler): string {
+	const ts = titleStyler ?? styler;
+	const titleW = visibleWidth(title);
+	const dashTotal = Math.max(0, innerW - titleW);
+	const leftDash = Math.floor(dashTotal / 2);
+	const rightDash = dashTotal - leftDash;
+	return styler("╭" + "─".repeat(leftDash)) + ts(title) + styler("─".repeat(rightDash) + "╮");
+}
+
+/** Bottom border: `╰──...──╯`. */
+export function borderBottom(innerW: number, styler: BorderStyler): string {
+	return styler("╰" + "─".repeat(innerW) + "╯");
+}
+
+/** Mid-panel divider: `├──...──┤`. */
+export function borderDivider(innerW: number, styler: BorderStyler): string {
+	return styler("├" + "─".repeat(innerW) + "┤");
+}
+
+/** Labelled divider inside the panel: `├── label ──┤`. */
+export function borderLabelledDivider(label: string, innerW: number, styler: BorderStyler, labelStyler?: BorderStyler): string {
+	const ls = labelStyler ?? styler;
+	const head = `─ ${label} `;
+	const dashN = Math.max(0, innerW - visibleWidth(head) - 1);
+	return styler("├ ") + ls(head) + styler("─".repeat(dashN) + "┤");
+}
+
+/** Content row: `│ content │` (truncated with `…` if too wide, padded if short). */
+export function borderRow(content: string, innerW: number, styler: BorderStyler): string {
+	const truncated = visibleWidth(content) > innerW
+		? truncateToWidth(content, innerW, "…", true)
+		: content;
+	const pad = Math.max(0, innerW - visibleWidth(truncated));
+	return styler("│ ") + truncated + " ".repeat(pad) + styler(" │");
+}
+
+/** Empty row: `│          │`. */
+export function borderEmpty(innerW: number, styler: BorderStyler): string {
+	return styler("│") + " ".repeat(innerW) + styler("│");
+}

package/visual/fuzzy.ts

@@ -0,0 +1,26 @@
+// =============================================================================
+// visual/fuzzy.ts — shared subsequence fuzzy match scorer
+// =============================================================================
+//
+// Extracted from list-panel.ts (where it was duplicated in mcp-panel.ts).
+// Pure function, no deps — trivially testable.
+//
+// Scoring:
+//   - exact substring match: 100 + (queryLen / textLen) — highest
+//   - in-order subsequence:  1 — match but not contiguous
+//   - no match:              0
+// =============================================================================
+
+/** Subsequence fuzzy match: substring scores highest, then in-order chars.
+ *  Returns 0 for no match, >0 for a match. */
+export function fuzzyScore(query: string, text: string): number {
+	const q = query.toLowerCase();
+	const t = text.toLowerCase();
+	if (!q) return 1;
+	if (t.includes(q)) return 100 + q.length / t.length;
+	let qi = 0;
+	for (let i = 0; i < t.length && qi < q.length; i++) {
+		if (t[i] === q[qi]) qi++;
+	}
+	return qi === q.length ? 1 : 0;
+}

package/visual/list-panel.ts

@@ -16,6 +16,7 @@ import { matchesKey, truncateToWidth, visibleWidth, wrapTextWithAnsi } from "@ea
 import type { Component, TUI } from "@earendil-works/pi-tui";
 import type { Theme } from "@earendil-works/pi-coding-agent";
 import { createPanelKeys, type PanelKeybindings, type PanelKeys } from "./panel-keys.ts";
+import { fuzzyScore } from "./fuzzy.ts";
 // (PanelKeybindings re-exported by re-export side below.)
 
 /** One row in the panel. `body` is shown in the preview pane when selected. */
@@ -60,15 +61,7 @@ const MAX_ROWS = 12; // list window height (render() has no viewport height)
 const PREVIEW_LINES = 4;
 
 /** Subsequence fuzzy match: substring scores highest, then in-order chars. */
-function fuzzyScore(query: string, text: string): number {
-	const q = query.toLowerCase();
-	const t = text.toLowerCase();
-	if (!q) return 1;
-	if (t.includes(q)) return 100 + q.length / t.length;
-	let qi = 0;
-	for (let i = 0; i < t.length && qi < q.length; i++) if (t[i] === q[qi]) qi++;
-	return qi === q.length ? 1 : 0;
-}
+// fuzzyScore moved to ./fuzzy.ts — imported above.
 
 /** Index into `flatRows` (mixed item + group header rows). */
 type RowIndex = number;
@@ -253,7 +246,7 @@ export class ListPanel implements Component {
 		const left = ` ${this.p.title} `;
 		const right = this.p.headerRight ? ` ${this.p.headerRight} ` : "";
 		const fillN = Math.max(1, inner + 2 - visibleWidth(left) - visibleWidth(right));
-		return dim("┌") + muted(left) + dim("─".repeat(fillN)) + muted(right) + dim("┐");
+		return dim("╭") + muted(left) + dim("─".repeat(fillN)) + muted(right) + dim("╮");
 	}
 
 	private searchLine(inner: number, dim: (s: string) => string, muted: (s: string) => string): string {
@@ -289,6 +282,6 @@ export class ListPanel implements Component {
 		const open = this.p.onSelect ? "⏎ open · " : "";
 		const hint = ` ↑↓ move · ${open}/ search${acts ? " · " + acts : ""} · esc `;
 		const fillN = Math.max(1, inner + 2 - visibleWidth(hint));
-		return dim("└") + dim(hint) + dim("─".repeat(fillN)) + dim("┘");
+		return dim("╰") + dim(hint) + dim("─".repeat(fillN)) + dim("╯");
 	}
 }