@danypops/papyrus 0.34.2 → 0.35.0

package/extension/src/markdown.ts

@@ -1,60 +0,0 @@
-import { getMarkdownTheme, type Theme } from "@earendil-works/pi-coding-agent";
-import { Markdown, type MarkdownTheme } from "@earendil-works/pi-tui";
-
-export type ActiveTheme = () => Theme;
-export type ActiveMarkdownTheme = () => Pick<MarkdownTheme, "highlightCode">;
-
-function activePiMarkdownTheme(): Pick<MarkdownTheme, "highlightCode"> {
-	try {
-		return getMarkdownTheme();
-	} catch {
-		return {};
-	}
-}
-
-export function createPapyrusMarkdownTheme(
-	activeTheme: ActiveTheme,
-	activeMarkdownTheme: ActiveMarkdownTheme = activePiMarkdownTheme,
-): MarkdownTheme {
-	return {
-		heading: (text) => activeTheme().fg("mdHeading", text),
-		link: (text) => activeTheme().fg("mdLink", text),
-		linkUrl: (text) => activeTheme().fg("mdLinkUrl", text),
-		code: (text) => activeTheme().fg("mdCode", text),
-		codeBlock: (text) => activeTheme().fg("mdCodeBlock", text),
-		codeBlockBorder: (text) => activeTheme().fg("mdCodeBlockBorder", text),
-		quote: (text) => activeTheme().fg("mdQuote", text),
-		quoteBorder: (text) => activeTheme().fg("mdQuoteBorder", text),
-		hr: (text) => activeTheme().fg("mdHr", text),
-		listBullet: (text) => activeTheme().fg("mdListBullet", text),
-		bold: (text) => activeTheme().bold(text),
-		italic: (text) => activeTheme().italic(text),
-		strikethrough: (text) => activeTheme().strikethrough(text),
-		underline: (text) => activeTheme().underline(text),
-		highlightCode: (code, language) => {
-			try {
-				const highlighted = activeMarkdownTheme().highlightCode?.(code, language);
-				if (highlighted) return highlighted;
-			} catch {
-				// The host theme may be unavailable in isolated rendering tests.
-			}
-			return code.split("\n").map((line) => activeTheme().fg("mdCodeBlock", line));
-		},
-	};
-}
-
-export function renderMarkdownBody(
-	body: string,
-	width: number,
-	activeTheme: ActiveTheme,
-	activeMarkdownTheme: ActiveMarkdownTheme = activePiMarkdownTheme,
-): string[] {
-	const markdown = new Markdown(
-		body || "(no body)",
-		0,
-		0,
-		createPapyrusMarkdownTheme(activeTheme, activeMarkdownTheme),
-		{ color: (text) => activeTheme().fg("text", text) },
-	);
-	return markdown.render(Math.max(1, width));
-}

package/extension/src/note-widget.ts

@@ -1,8 +0,0 @@
-import { truncateToWidth } from "@earendil-works/pi-tui";
-import type { Theme } from "@earendil-works/pi-coding-agent";
-
-/** Hidden at 0, matching TaskOverlay's own "nothing open" hiding rule. */
-export function renderNoteWidgetLines(theme: Theme, openCount: number, width: number): string[] {
-	if (openCount === 0) return [];
-	return [truncateToWidth(`${theme.fg("muted", "Notes")} ${theme.fg("accent", String(openCount))}`, width, "…")];
-}

package/extension/src/notes.ts

@@ -1,102 +0,0 @@
-import type { ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
-import { NOTE_LIST_MAX_LIMIT } from "../../src/constants.ts";
-import { NOTE_DISPOSITIONS } from "../../src/note-service.ts";
-import type { Artifact } from "../../src/domain/artifact.ts";
-import { showArtifactBrowser, showArtifactDetails } from "./artifact-browser.ts";
-import { NOTE_STATUS_PRESENTATION } from "./artifact-status-presentation.ts";
-import { callService } from "./service-client.ts";
-
-export function noteRowMeta(note: Artifact): string {
-	const history = Array.isArray(note.extra["noteHistory"]) ? note.extra["noteHistory"].length : 0;
-	return `${history} event${history === 1 ? "" : "s"}`;
-}
-
-export function noteCaptureInput(request: string, projectRoot: string): Record<string, unknown> | null {
-	const body = request.trim();
-	if (!body) return null;
-	return { body, project_root: projectRoot, actor: "human", source: "note-command" };
-}
-
-/**
- * The generic artifact browser (extension/src/artifact-browser.ts) requests a fixed 500-row
- * page by default, but notes.list enforces its own tighter NOTE_LIST_MAX_LIMIT (200) — an
- * unqualified /notes call exceeded that bound and the browser surfaced the daemon's rejection
- * as an opaque extension error instead of ever rendering. Passing an explicit limit here that
- * respects the Notes-specific bound is the fix; the generic browser's default stays as-is
- * since no other kind's list operation has a bound below 500.
- */
-export function noteListInput(projectRoot: string): Record<string, unknown> {
-	return { project_root: projectRoot, limit: NOTE_LIST_MAX_LIMIT };
-}
-
-export async function captureNote(request: string, ctx: ExtensionCommandContext): Promise<Artifact | null> {
-	const input = noteCaptureInput(request, ctx.cwd);
-	if (!input) {
-		ctx.ui.notify("Usage: /note <request for later>", "warning");
-		return null;
-	}
-	try {
-		const note = await callService<Record<string, unknown>, Artifact>("notes.capture", input);
-		ctx.ui.notify(`Captured note: ${note.title}`, "info");
-		return note;
-	} catch (error) {
-		ctx.ui.notify(`Note capture failed: ${error instanceof Error ? error.message : error}`, "error");
-		return null;
-	}
-}
-
-export async function showNotes(ctx: ExtensionCommandContext): Promise<void> {
-	await showArtifactBrowser(ctx, {
-		kind: "note",
-		title: "Notes inbox",
-		listOperation: "notes.list",
-		listInput: noteListInput(ctx.cwd),
-		statusOrder: ["draft", "active", "archived"],
-		presentation: NOTE_STATUS_PRESENTATION,
-		rowMeta: noteRowMeta,
-		actions: (note) => [
-			"Show details",
-			...(note.status === "draft" ? ["Consume"] : []),
-			"Promote",
-			"Archive",
-		],
-		handleAction: async (choice, note, commandCtx) => {
-			if (choice === "Show details") {
-				await showArtifactDetails(commandCtx, note.id, "notes.show", { project_root: commandCtx.cwd });
-				return;
-			}
-			if (choice === "Consume") {
-				await callService("notes.consume", { id: note.id, project_root: commandCtx.cwd, actor: "human", source: "notes-tui" });
-				commandCtx.ui.notify(`Consumed ${note.title}`, "info");
-				return;
-			}
-			if (choice === "Promote") {
-				const targetId = await commandCtx.ui.input("Resulting artifact id:", "");
-				if (!targetId) return;
-				const reason = await commandCtx.ui.input("Disposition note (optional):", "");
-				await callService("notes.promote", {
-					id: note.id,
-					target_id: targetId,
-					project_root: commandCtx.cwd,
-					actor: "human",
-					source: "notes-tui",
-					...(reason ? { reason } : {}),
-				});
-				commandCtx.ui.notify(`Promoted ${note.title} → ${targetId}`, "info");
-				return;
-			}
-			const disposition = await commandCtx.ui.select("Archive disposition", [...NOTE_DISPOSITIONS]);
-			if (!disposition) return;
-			const reason = await commandCtx.ui.input("Reason (optional):", "");
-			await callService("notes.archive", {
-				id: note.id,
-				disposition,
-				project_root: commandCtx.cwd,
-				actor: "human",
-				source: "notes-tui",
-				...(reason ? { reason } : {}),
-			});
-			commandCtx.ui.notify(`Archived ${note.title} · ${disposition}`, "info");
-		},
-	});
-}

package/extension/src/playbook-bridge.ts

@@ -1,91 +0,0 @@
-/**
- * playbook-bridge.ts — materializes active Papyrus Playbooks as their own /playbook:name slash
- * commands, one per playbook, the same one-entry-per-item autocomplete experience Pi's own
- * /skill:name gives real skills.
- *
- * /skill:name itself is a hardcoded core mechanism (Pi's interactive mode builds it directly
- * from its own skill loader) -- not something an extension can retarget to a different prefix.
- * pi.registerCommand(name, ...) accepts any string, colons included, so "playbook:<slug>"
- * registers and invokes as literally /playbook:<slug> -- a real, supported extension API, no
- * core touched.
- *
- * Real limitation: ExtensionAPI has no unregisterCommand. A disabled or renamed playbook's old
- * /playbook:<slug> command lingers until a full Pi restart -- registerCommand can only add or
- * overwrite, never remove. Mitigated two ways: registrations refresh on every resources_discover
- * (session start and /reload), so a renamed playbook's NEW slug appears promptly even though the
- * old one lingers; and each command's handler re-fetches the live playbook by id at invocation
- * time rather than baking in stale content, so even a lingering stale name fails cleanly with a
- * real error instead of running deleted content.
- */
-import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-import type { Artifact } from "../../src/domain/artifact.ts";
-import { callService } from "./service-client.ts";
-
-export const PLAYBOOK_BRIDGE_MAX_PLAYBOOKS = 100;
-
-function slugify(title: string): string {
-	const slug = title.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-+|-+$/g, "").slice(0, 64);
-	return slug.length > 0 ? slug : "playbook";
-}
-
-async function activePlaybooks(): Promise<Artifact[]> {
-	return callService<Record<string, unknown>, Artifact[]>("playbooks.list", { status: "active", limit: PLAYBOOK_BRIDGE_MAX_PLAYBOOKS });
-}
-
-/** Exported for direct testing without a real ExtensionAPI. */
-export function playbookCommandName(title: string): string {
-	return `playbook:${slugify(title)}`;
-}
-
-/**
- * One line per active Playbook for context injection -- the passive, every-turn surfacing that
- * gives the model the same "this exists and might match my task" awareness Pi's own Skill catalog
- * gives real Skills, without a file-based bridge. See buildContextInjection (rules and open tasks
- * already work this way).
- */
-export function playbookInjectionPreview(playbook: Pick<Artifact, "title" | "extra">): string {
-	const trigger = typeof playbook.extra["trigger"] === "string" ? playbook.extra["trigger"] : "manual invocation";
-	return `• ${playbook.title} (when: ${trigger})`;
-}
-
-/** Exported for direct testing without a real ExtensionAPI: what would be registered right now. */
-export async function planPlaybookCommandRegistrations(): Promise<Array<{ name: string; id: string; title: string; trigger: string }>> {
-	const playbooks = await activePlaybooks();
-	const usedNames = new Set<string>();
-	return playbooks.map((playbook) => {
-		let name = playbookCommandName(playbook.title);
-		if (usedNames.has(name)) name = `${name}-${playbook.id.slice(0, 8)}`; // a real title collision, not the common case
-		usedNames.add(name);
-		const trigger = typeof playbook.extra["trigger"] === "string" ? playbook.extra["trigger"] : "manual invocation";
-		return { name, id: playbook.id, title: playbook.title, trigger };
-	});
-}
-
-export function registerPlaybookBridge(pi: ExtensionAPI): void {
-	const refresh = async () => {
-		try {
-			const registrations = await planPlaybookCommandRegistrations();
-			for (const { name, id, title, trigger } of registrations) {
-				pi.registerCommand(name, {
-					description: trigger,
-					handler: async (_args, ctx) => {
-						try {
-							// Re-fetched live, not captured at registration time: a lingering stale
-							// command (renamed or disabled since, since registerCommand can't be
-							// unregistered) must fail cleanly, never run deleted/stale content.
-							const invocation = await callService<Record<string, unknown>, string>("playbooks.invoke", { id });
-							ctx.ui.setEditorText(invocation);
-							ctx.ui.notify(`"${title}" invocation placed in the editor`, "info");
-						} catch (error) {
-							ctx.ui.notify(error instanceof Error ? error.message : String(error), "error");
-						}
-					},
-				});
-			}
-		} catch {
-			// A Papyrus daemon hiccup must never break Pi's own resource discovery -- degrades to
-			// "no new/updated playbook commands this cycle", not a broken session start.
-		}
-	};
-	pi.on("resources_discover", async () => { await refresh(); return {}; });
-}

package/extension/src/playbooks.ts

@@ -1,97 +0,0 @@
-import type { AutocompleteItem } from "@earendil-works/pi-tui";
-import type { ExtensionCommandContext, Theme } from "@earendil-works/pi-coding-agent";
-import type { Artifact } from "../../src/domain/artifact.ts";
-import { showArtifactBrowser, showArtifactDetails } from "./artifact-browser.ts";
-import { PLAYBOOK_STATUS_PRESENTATION } from "./artifact-status-presentation.ts";
-import { matchArtifactByName } from "./domain-tools.ts";
-import { callService } from "./service-client.ts";
-
-const PLAYBOOK_COMPLETION_MAX_CANDIDATES = 100;
-
-async function activePlaybooks(): Promise<Artifact[]> {
-	return callService<Record<string, unknown>, Artifact[]>("playbooks.list", { status: "active", limit: PLAYBOOK_COMPLETION_MAX_CANDIDATES });
-}
-
-/** `/playbook <tab>` completions -- title-prefix match, since that's what a human actually types, not a full-text search of body content. */
-export async function playbookArgumentCompletions(argumentPrefix: string): Promise<AutocompleteItem[] | null> {
-	try {
-		const needle = argumentPrefix.trim().toLowerCase();
-		const rows = await activePlaybooks();
-		return rows
-			.filter((row) => row.title.toLowerCase().startsWith(needle))
-			.sort((a, b) => a.title.localeCompare(b.title))
-			.map((row) => ({ value: row.title, label: row.title, description: typeof row.extra["trigger"] === "string" ? row.extra["trigger"] : undefined }));
-	} catch {
-		return null; // a Papyrus daemon hiccup degrades to "no suggestions", never breaks the command line
-	}
-}
-
-/** `/playbook <name>` (no args opens the full browser instead): resolves by exact title, then places its invocation directly in the editor -- one step, not browse-then-select-then-invoke. */
-export async function openPlaybookByName(name: string, ctx: ExtensionCommandContext): Promise<void> {
-	if (!name.trim()) { await showPlaybooks(ctx); return; }
-	try {
-		const id = matchArtifactByName(await activePlaybooks(), name);
-		const invocation = await callService<Record<string, unknown>, string>("playbooks.invoke", { id });
-		ctx.ui.setEditorText(invocation);
-		ctx.ui.notify(`"${name.trim()}" invocation placed in the editor`, "info");
-	} catch (error) {
-		ctx.ui.notify(error instanceof Error ? error.message : String(error), "error");
-	}
-}
-
-const PLAYBOOK_RELATIONS = ["references", "documents", "relates_to", "contains", "part_of"];
-
-function strings(value: unknown): string[] {
-	return Array.isArray(value) ? value.filter((item): item is string => typeof item === "string") : [];
-}
-
-export function playbookRowMeta(playbook: Artifact): string {
-	const trigger = typeof playbook.extra["trigger"] === "string" ? `when ${playbook.extra["trigger"]}` : "manual invocation";
-	const tools = strings(playbook.extra["tools"]);
-	return [trigger, tools.join(", ")].filter(Boolean).join(" \u00b7 ");
-}
-
-export async function showPlaybooks(ctx: ExtensionCommandContext): Promise<void> {
-	await showArtifactBrowser(ctx, {
-		kind: "playbook",
-		title: "Playbooks",
-		listOperation: "playbooks.list",
-		statusOrder: ["active", "deprecated"],
-		presentation: PLAYBOOK_STATUS_PRESENTATION,
-		rowMeta: playbookRowMeta,
-		actions: (playbook) => ["Show details", "Edit", "Invoke", "Link artifact", playbook.status === "active" ? "Disable" : "Enable"],
-		handleAction: async (choice, playbook, commandCtx) => {
-			if (choice === "Show details") {
-				await showArtifactDetails(commandCtx, playbook.id, "playbooks.show");
-				return;
-			}
-			if (choice === "Edit") {
-				const title = await commandCtx.ui.input("Title:", playbook.title);
-				if (title === undefined) return; // canceled
-				const body = await commandCtx.ui.input("Body:", playbook.body);
-				if (body === undefined) return; // canceled
-				const updated = await callService<Record<string, unknown>, Artifact>("playbooks.update", { id: playbook.id, title, body });
-				commandCtx.ui.notify(`Updated "${updated.title}"`, "info");
-				return;
-			}
-			if (choice === "Invoke") {
-				const invocation = await callService<Record<string, unknown>, string>("playbooks.invoke", { id: playbook.id });
-				commandCtx.ui.setEditorText(invocation);
-				commandCtx.ui.notify("Invocation placed in the editor", "info");
-				return;
-			}
-			if (choice === "Link artifact") {
-				const targetId = await commandCtx.ui.input("Target artifact id:", "");
-				if (!targetId) return;
-				const relation = await commandCtx.ui.select("Relation", PLAYBOOK_RELATIONS);
-				if (!relation) return;
-				await callService("graph.link", { from: playbook.id, relation, to: targetId });
-				commandCtx.ui.notify(`Linked "${playbook.title}" via ${relation}`, "info");
-				return;
-			}
-			const operation = choice === "Disable" ? "playbooks.disable" : "playbooks.enable";
-			const updated = await callService<Record<string, unknown>, Artifact>(operation, { id: playbook.id });
-			commandCtx.ui.notify(`${updated.title} \u2192 [${updated.status}]`, "info");
-		},
-	});
-}

package/extension/src/rules.ts

@@ -1,51 +0,0 @@
-import type { ExtensionCommandContext, Theme } from "@earendil-works/pi-coding-agent";
-import type { Artifact } from "../../src/domain/artifact.ts";
-import { showArtifactBrowser, showArtifactDetails } from "./artifact-browser.ts";
-import { RULE_STATUS_PRESENTATION, severityColor } from "./artifact-status-presentation.ts";
-import { callService } from "./service-client.ts";
-
-export function ruleRowMeta(rule: Artifact, theme: Theme): string {
-	const severity = typeof rule.extra["severity"] === "string" ? rule.extra["severity"] : "info";
-	const severityText = theme.fg(severityColor(severity), severity.toUpperCase());
-	const condition = typeof rule.extra["condition"] === "string" ? `when ${rule.extra["condition"]}` : "always";
-	return `${severityText} · ${condition}`;
-}
-
-export function ruleInjectionPreview(rule: Pick<Artifact, "title" | "body" | "extra">): string {
-	const condition = typeof rule.extra["condition"] === "string" ? ` (when: ${rule.extra["condition"]})` : "";
-	const action = rule.body || (typeof rule.extra["action"] === "string" ? rule.extra["action"] : "");
-	return `• ${rule.title}${condition}\n  ${action}`;
-}
-
-export async function showRules(ctx: ExtensionCommandContext): Promise<void> {
-	await showArtifactBrowser(ctx, {
-		kind: "rule",
-		title: "Rules",
-		listOperation: "rules.list",
-		statusOrder: ["active", "deprecated"],
-		presentation: RULE_STATUS_PRESENTATION,
-		rowMeta: ruleRowMeta,
-		actions: (rule) => ["Show details", "Edit", "Preview injection", "Link gated task", rule.status === "active" ? "Disable" : "Enable"],
-		handleAction: async (choice, rule, commandCtx) => {
-			if (choice === "Show details") await showArtifactDetails(commandCtx, rule.id, "rules.show");
-			else if (choice === "Edit") {
-				const title = await commandCtx.ui.input("Title:", rule.title);
-				if (title === undefined) return; // canceled
-				const body = await commandCtx.ui.input("Body:", rule.body);
-				if (body === undefined) return; // canceled
-				const updated = await callService<Record<string, unknown>, Artifact>("rules.update", { id: rule.id, title, body });
-				commandCtx.ui.notify(`Updated "${updated.title}"`, "info");
-			} else if (choice === "Preview injection") {
-				const preview = await callService<Record<string, unknown>, string>("rules.preview", { id: rule.id });
-				commandCtx.ui.notify(preview, "info");
-			} else if (choice === "Link gated task") {
-				const taskId = await commandCtx.ui.input("Task artifact id:", "");
-				if (taskId) await callService("rules.gate", { id: rule.id, task_id: taskId });
-			} else {
-				const operation = choice === "Disable" ? "rules.disable" : "rules.enable";
-				const updated = await callService<Record<string, unknown>, Artifact>(operation, { id: rule.id });
-				commandCtx.ui.notify(`${updated.title} → [${updated.status}]`, "info");
-			}
-		},
-	});
-}

package/extension/src/service-client.ts

@@ -1,29 +0,0 @@
-import { createRetryingClient, type RetryingClient } from "@danypops/daemon-kit/pi-client";
-import { connectPapyrusClient, type PapyrusClient } from "../../src/client.ts";
-import type { OperationName } from "../../src/service.ts";
-
-type ClientConnector = () => Promise<PapyrusClient>;
-
-let connector: ClientConnector = () => connectPapyrusClient();
-const client: RetryingClient<PapyrusClient> = createRetryingClient<PapyrusClient>(() => connector(), { label: "Papyrus" });
-
-export async function papyrusClient(): Promise<PapyrusClient> {
-	return client.call(async (resolved) => resolved);
-}
-
-export async function callService<Input extends Record<string, unknown>, Output>(
-	operation: OperationName,
-	input: Input,
-): Promise<Output> {
-	return client.call((resolved) => resolved.call<Input, Output>(operation, input));
-}
-
-export function setPapyrusClientConnectorForTests(value: ClientConnector): void {
-	connector = value;
-	client.reset();
-}
-
-export function resetPapyrusClientForTests(): void {
-	connector = () => connectPapyrusClient();
-	client.reset();
-}

package/extension/src/session-identity.ts

@@ -1,22 +0,0 @@
-/**
- * Client-side cache for the extension's own session_secret, registered with the daemon at
- * session_start and released at session_shutdown (see index.ts). Keyed by sessionId (not a
- * single "current" variable) defensively -- multiple call sites reference an explicit
- * sessionId already, and a Map costs nothing extra for real correctness. See
- * src/domain/session-identity.ts (daemon side) for the full design rationale.
- */
-const secretsBySessionId = new Map<string, string>();
-
-export function cacheSessionSecret(sessionId: string, secret: string): void {
-	secretsBySessionId.set(sessionId, secret);
-}
-
-export function forgetSessionSecret(sessionId: string): void {
-	secretsBySessionId.delete(sessionId);
-}
-
-/** Spread into any Focus-mutating request body alongside session_id -- empty object when no secret is cached for this sessionId (unregistered, or registration hasn't completed yet), matching the daemon's opt-in-armor default. */
-export function sessionSecretField(sessionId: string | undefined): { session_secret?: string } {
-	const secret = sessionId ? secretsBySessionId.get(sessionId) : undefined;
-	return secret ? { session_secret: secret } : {};
-}

package/extension/src/skill-catalog-footprint.ts

@@ -1,183 +0,0 @@
-import { existsSync, readFileSync, readdirSync, statSync } from "node:fs";
-import { dirname, join } from "node:path";
-import { CONTEXT_ESTIMATE_CHARACTERS_PER_TOKEN } from "../../src/constants.ts";
-
-/**
- * Pi-native skills (SKILL.md) carry a real, permanent context tax independent of Papyrus:
- * per Pi's own docs, every discovered skill's name+description is injected into the system
- * prompt unconditionally at startup (the Agent Skills spec's "catalog" tier, ~50-100 tokens
- * per skill). This module measures that tax by replicating Pi's own documented discovery
- * rules (docs/skills.md "Locations" section) directly against the filesystem, rather than
- * trying to parse it back out of the assembled system prompt -- Pi does not document (and
- * this repo must not depend on) the exact wire format it uses to inject the catalog, so
- * re-deriving the same inputs Pi itself reads is the robust approach, not a fragile one.
- * Package-declared skills (pi.skills in package.json / packages' own skills/ directories)
- * are deliberately out of scope: enumerating every installed package for skill declarations
- * is a materially larger, slower scan than reading a handful of known directories, and this
- * tool is a budget estimate, not an exhaustive audit.
- */
-export interface SkillCatalogEntry {
-	name: string;
-	description: string;
-	location: string;
-	characters: number;
-	estimatedTokens: number;
-}
-
-export interface SkillCatalogFootprint {
-	entries: SkillCatalogEntry[];
-	totalCharacters: number;
-	totalEstimatedTokens: number;
-	scannedDirectories: string[];
-}
-
-export const SKILL_SCAN_MAX_DEPTH = 6;
-export const SKILL_SCAN_MAX_DIRECTORIES = 2000;
-export const SKILL_SCAN_MAX_SKILLS = 500;
-
-function unquote(value: string): string {
-	if (value.length >= 2 && ((value.startsWith('"') && value.endsWith('"')) || (value.startsWith("'") && value.endsWith("'")))) {
-		return value.slice(1, -1);
-	}
-	return value;
-}
-
-/**
- * Extracts `name` and `description` from a SKILL.md's YAML frontmatter, tolerating the
- * folded (`>`) and literal (`|`) block-scalar forms real-world skills commonly use for
- * multi-line descriptions. Deliberately not a general YAML parser -- only the two fields
- * the Agent Skills spec requires are extracted; anything else in the frontmatter is ignored.
- */
-export function parseSkillFrontmatter(content: string): { name: string; description: string } | null {
-	const match = content.match(/^---\r?\n([\s\S]*?)\r?\n---/);
-	if (!match) return null;
-	const lines = match[1]!.split(/\r?\n/);
-	let name = "";
-	let description = "";
-	for (let index = 0; index < lines.length; index++) {
-		const line = lines[index]!;
-		const nameMatch = line.match(/^name:\s*(.*)$/);
-		if (nameMatch) {
-			name = unquote(nameMatch[1]!.trim());
-			continue;
-		}
-		const descriptionMatch = line.match(/^description:\s*(.*)$/);
-		if (!descriptionMatch) continue;
-		const rest = descriptionMatch[1]!.trim();
-		if (rest === ">" || rest === ">-" || rest === "|" || rest === "|-") {
-			const collected: string[] = [];
-			let cursor = index + 1;
-			while (cursor < lines.length && (lines[cursor] === "" || /^\s+/.test(lines[cursor]!))) {
-				collected.push(lines[cursor]!.trim());
-				cursor++;
-			}
-			description = collected.join(rest.startsWith("|") ? "\n" : " ").trim();
-			index = cursor - 1;
-		} else {
-			description = unquote(rest);
-		}
-	}
-	if (!name || !description) return null;
-	return { name, description };
-}
-
-/** True at the filesystem root on POSIX (`/`) and Windows (`C:\`, `D:\`, ...). */
-function isFilesystemRoot(path: string): boolean {
-	return dirname(path) === path;
-}
-
-/**
- * Global and project skill directories per Pi's own documented discovery rules, plus any
- * explicit paths configured in settings.json's `skills` array. Project directories are
- * collected walking from `cwd` up to the git repository root (or filesystem root when not
- * in a repo), matching "up to git repo root, or filesystem root when not in a repo" exactly.
- */
-export function discoverSkillDirectories(homeDirectory: string, cwd: string, settingsSkills: readonly string[] = []): string[] {
-	const directories = [join(homeDirectory, ".pi", "agent", "skills"), join(homeDirectory, ".agents", "skills")];
-	let current = cwd;
-	for (let depth = 0; depth < SKILL_SCAN_MAX_DIRECTORIES; depth++) {
-		directories.push(join(current, ".pi", "skills"), join(current, ".agents", "skills"));
-		if (existsSync(join(current, ".git")) || isFilesystemRoot(current)) break;
-		current = dirname(current);
-	}
-	directories.push(...settingsSkills);
-	return [...new Set(directories)];
-}
-
-interface ScanContext {
-	entries: SkillCatalogEntry[];
-	seenLocations: Set<string>;
-	directoriesVisited: number;
-}
-
-/** Root-level .md files count as individual skills only in these two locations, per Pi's docs. */
-function allowsRootMarkdownFiles(directory: string): boolean {
-	return directory.endsWith(join(".pi", "agent", "skills")) || directory.endsWith(join(".pi", "skills"));
-}
-
-function recordSkillFile(context: ScanContext, path: string): void {
-	if (context.seenLocations.has(path) || context.entries.length >= SKILL_SCAN_MAX_SKILLS) return;
-	let content: string;
-	try {
-		content = readFileSync(path, "utf8");
-	} catch {
-		return;
-	}
-	const parsed = parseSkillFrontmatter(content);
-	if (!parsed) return;
-	context.seenLocations.add(path);
-	const characters = parsed.name.length + parsed.description.length;
-	context.entries.push({
-		name: parsed.name,
-		description: parsed.description,
-		location: path,
-		characters,
-		estimatedTokens: Math.ceil(characters / CONTEXT_ESTIMATE_CHARACTERS_PER_TOKEN),
-	});
-}
-
-function walk(context: ScanContext, directory: string, depth: number, allowRootMarkdown: boolean): void {
-	if (depth > SKILL_SCAN_MAX_DEPTH || context.directoriesVisited >= SKILL_SCAN_MAX_DIRECTORIES) return;
-	context.directoriesVisited++;
-	let names: string[];
-	try {
-		names = readdirSync(directory);
-	} catch {
-		return;
-	}
-	for (const name of names) {
-		if (name === "node_modules" || name === ".git") continue;
-		const path = join(directory, name);
-		let stat: ReturnType<typeof statSync>;
-		try {
-			stat = statSync(path);
-		} catch {
-			continue;
-		}
-		if (stat.isDirectory()) {
-			const skillFile = join(path, "SKILL.md");
-			if (existsSync(skillFile)) recordSkillFile(context, skillFile);
-			else walk(context, path, depth + 1, false);
-		} else if (allowRootMarkdown && depth === 0 && name.toLowerCase().endsWith(".md")) {
-			recordSkillFile(context, path);
-		}
-	}
-}
-
-/** Bounded, best-effort scan: a missing or unreadable directory is silently skipped, not an error. */
-export function scanSkillCatalogFootprint(directories: readonly string[]): SkillCatalogFootprint {
-	const context: ScanContext = { entries: [], seenLocations: new Set(), directoriesVisited: 0 };
-	const scanned: string[] = [];
-	for (const directory of directories) {
-		if (!existsSync(directory) || !statSync(directory).isDirectory()) continue;
-		scanned.push(directory);
-		walk(context, directory, 0, allowsRootMarkdownFiles(directory));
-	}
-	const entries = context.entries.sort((a, b) => b.characters - a.characters);
-	return {
-		entries,
-		totalCharacters: entries.reduce((sum, entry) => sum + entry.characters, 0),
-		totalEstimatedTokens: entries.reduce((sum, entry) => sum + entry.estimatedTokens, 0),
-		scannedDirectories: scanned,
-	};
-}