@gavin-hu/my-pi-agent 0.7.0 → 0.8.0

package/extensions/plan/steps.ts

@@ -166,28 +166,40 @@ export function extractPlanSteps(plan: string): PlanStep[] {
 	return topLevelItems(plan).map((item) => ({ content: elide(item.step.content), status: item.step.status }));
 }
 
+/** One checkbox change: a 1-based step number and the state to set. */
+export interface StepUpdate {
+	step: number;
+	status: PlanStep["status"];
+}
+
 /**
- * Set the checkbox of the `step`th top-level step (1-based, matching
- * {@link extractPlanSteps}) and return the rewritten plan with the updated step,
- * or undefined when the step number is out of range. Only the marker changes: a
- * plain `1.`/`-` item gains a checkbox, an existing one is set, and the step
- * text is left exactly as written.
+ * Apply a batch of checkbox changes to the given top-level steps (1-based,
+ * matching {@link extractPlanSteps}) and return the rewritten plan, or
+ * undefined when any step number is out of range. All-or-nothing: every step is
+ * validated before any edit, so one bad entry changes nothing. Only the marker
+ * changes: a plain `1.`/`-` item gains a checkbox, an existing one is set, a
+ * `[DONE:n]` marker is dropped when the step is reopened, and the step text is
+ * otherwise left exactly as written. When a step appears twice, the last entry
+ * wins.
  */
-export function setStepStatus(
-	plan: string,
-	step: number,
-	status: "pending" | "completed",
-): { plan: string; step: PlanStep } | undefined {
-	if (!Number.isInteger(step) || step < 1) return undefined;
-	const target = topLevelItems(plan)[step - 1];
-	if (!target) return undefined;
-
-	const existing = target.raw.match(CHECKBOX);
-	const text = existing ? existing[2] : target.raw;
-	const checkbox = status === "completed" ? "[x]" : "[ ]";
+export function setStepStatuses(plan: string, updates: StepUpdate[]): string | undefined {
+	const items = topLevelItems(plan);
+	for (const update of updates) {
+		if (!Number.isInteger(update.step) || update.step < 1 || !items[update.step - 1]) return undefined;
+	}
+
 	const lines = plan.split("\n");
-	lines[target.line] = `${" ".repeat(target.indent)}${target.lead}${checkbox} ${text}`;
-	return { plan: lines.join("\n"), step: parseStep(`${checkbox} ${text}`) };
+	for (const update of updates) {
+		const target = items[update.step - 1];
+		const existing = target.raw.match(CHECKBOX);
+		let text = existing ? existing[2] : target.raw;
+		// A reopened step must lose its marker too: the marker alone reads as
+		// completed, so keeping it would contradict the checkbox.
+		if (update.status === "pending") text = text.replace(DONE_MARKER, "").trimEnd();
+		const checkbox = update.status === "completed" ? "[x]" : "[ ]";
+		lines[target.line] = `${" ".repeat(target.indent)}${target.lead}${checkbox} ${text}`;
+	}
+	return lines.join("\n");
 }
 
 /**

package/extensions/plan/tools.ts

@@ -7,7 +7,7 @@
  * approval (a scrollable review screen in the TUI, a select dialog elsewhere),
  * and on approval seeds the `todo` tool with the plan's steps (through
  * `ctx.executeTool`, so the todo widget and validation run normally).
- * `set_plan_step` then keeps the file current: after approval the model ticks
+ * `set_plan_steps` then keeps the file current: after approval the model ticks
  * each step off in place as it finishes it. The file is the source of truth:
  * the model executes from it.
  *
@@ -26,10 +26,10 @@ import { FULL_SCREEN_OVERLAY, reuseText, toolDetails } from "../../lib/tui.ts";
 import { expandHint } from "../../lib/ui.ts";
 import { READ_ONLY_SUMMARY } from "./policy.ts";
 import { feedbackSummary } from "./feedback.ts";
-import { ENTER_TOOL, EXIT_TOOL, SET_PLAN_STEP_TOOL, WRITE_PLAN_TOOL, type PlanRuntime } from "./runtime.ts";
-import { PLAN_OUTLINE, extractPlanSteps, type PlanStep } from "./steps.ts";
+import { ENTER_TOOL, EXIT_TOOL, SET_PLAN_STEPS_TOOL, WRITE_PLAN_TOOL, type PlanRuntime } from "./runtime.ts";
+import { PLAN_OUTLINE, extractPlanSteps, type PlanStep, type StepUpdate } from "./steps.ts";
 import { PlanViewComponent, sanitizePlanText, type PlanViewAction } from "./tui.ts";
-import type { EnterPlanModeDetails, ExitPlanModeDetails, SetPlanStepDetails, WritePlanDetails } from "./types.ts";
+import type { EnterPlanModeDetails, ExitPlanModeDetails, SetPlanStepsDetails, WritePlanDetails } from "./types.ts";
 import { createPlanWebReview, type PlanWebReview } from "./web.ts";
 
 const ExitPlanModeParams = Type.Object({
@@ -54,10 +54,7 @@ const WritePlanParams = Type.Object({
 	),
 });
 
-const SetPlanStepParams = Type.Object({
-	plan_path: Type.String({
-		description: "Path to the approved plan file, as returned by write_plan or exit_plan_mode.",
-	}),
+const StepUpdateParams = Type.Object({
 	step: Type.Integer({
 		minimum: 1,
 		description: "1-based step number, in the order the plan was approved.",
@@ -67,9 +64,19 @@ const SetPlanStepParams = Type.Object({
 	}),
 });
 
+const SetPlanStepsParams = Type.Object({
+	plan_path: Type.String({
+		description: "Path to the approved plan file, as returned by write_plan or exit_plan_mode.",
+	}),
+	updates: Type.Array(StepUpdateParams, {
+		minItems: 1,
+		description: "The steps to set; each entry sets one step's checkbox, and the last entry wins for a duplicate step.",
+	}),
+});
+
 type ExitPlanModeArgs = Static<typeof ExitPlanModeParams>;
 type WritePlanArgs = Static<typeof WritePlanParams>;
-type SetPlanStepArgs = Static<typeof SetPlanStepParams>;
+type SetPlanStepsArgs = Static<typeof SetPlanStepsParams>;
 
 /** Lines of plan shown in an unexpanded result. */
 const PREVIEW_LINES = 8;
@@ -89,6 +96,21 @@ function preview(plan: string): { body: string; more: number } {
 	return { body: lines.slice(0, PREVIEW_LINES).join("\n"), more: lines.length - PREVIEW_LINES };
 }
 
+/** "Step 2" or "Steps 1, 2" for a set of step numbers. */
+function stepLabel(steps: number[]): string {
+	return `${steps.length === 1 ? "Step" : "Steps"} ${steps.join(", ")}`;
+}
+
+/** "Steps 1, 2 completed; step 3 reopened" for the tool's text result. */
+function describeUpdates(updates: StepUpdate[]): string {
+	const completed = updates.filter((update) => update.status === "completed").map((update) => update.step);
+	const pending = updates.filter((update) => update.status === "pending").map((update) => update.step);
+	const parts: string[] = [];
+	if (completed.length > 0) parts.push(`${stepLabel(completed)} completed`);
+	if (pending.length > 0) parts.push(`${stepLabel(pending)} reopened`);
+	return parts.join("; ");
+}
+
 /** Seed the todo list with the plan's steps; returns what was recorded. */
 async function seedTodos(
 	ctx: ExtensionToolContext,
@@ -464,9 +486,9 @@ export function registerTools(
 					: "";
 			const tail =
 				recorded > 0
-					? ` Recorded ${recorded} step${recorded === 1 ? "" : "s"} in the todo list. As you finish each step, mark it done there and tick it off in the file with ${SET_PLAN_STEP_TOOL}.`
+					? ` Recorded ${recorded} step${recorded === 1 ? "" : "s"} in the todo list. As you finish each step, mark it done there and tick it off in the file with ${SET_PLAN_STEPS_TOOL}.`
 					: steps.length > 0
-						? ` Could not record the steps in the todo list${error ? `: ${error}` : ""}. Keep these steps in mind and tick them off with ${SET_PLAN_STEP_TOOL} as you go.`
+						? ` Could not record the steps in the todo list${error ? `: ${error}` : ""}. Keep these steps in mind and tick them off with ${SET_PLAN_STEPS_TOOL} as you go.`
 						: " If the todo tool is available, record the steps with it before you start.";
 			return {
 				content: [
@@ -525,22 +547,23 @@ export function registerTools(
 	});
 
 	pi.registerTool({
-		name: SET_PLAN_STEP_TOOL,
-		label: "Set plan step",
+		name: SET_PLAN_STEPS_TOOL,
+		label: "Set plan steps",
 		description:
-			"Keep an approved plan file current by setting one top-level step's checkbox in place. The step number matches " +
-			"the order the plan was approved in. Only the checkbox changes; the step text is left as written. Call it as you " +
-			"finish (or reopen) each step so the file mirrors reality.",
-		promptSnippet: "Tick a step off in the approved plan file.",
+			"Keep an approved plan file current by setting the checkboxes of the given top-level steps in place. Each step " +
+			"number matches the order the plan was approved in; a step can be marked completed or reopened. Only the " +
+			"checkboxes change; the step text is left as written. The file is rewritten once, and nothing changes when any " +
+			"step is out of range.",
+		promptSnippet: "Tick steps off in the approved plan file.",
 		promptGuidelines: [
-			"After a plan is approved, call set_plan_step as you complete each step so the plan file stays current.",
+			"After a plan is approved, call set_plan_steps as you complete steps so the plan file stays current.",
 		],
-		parameters: SetPlanStepParams,
+		parameters: SetPlanStepsParams,
 		executionMode: "sequential",
 		annotations: { readOnlyHint: false, destructiveHint: false, idempotentHint: true, openWorldHint: false },
 
 		async execute(_toolCallId, params, _signal, _onUpdate, ctx) {
-			const args = params as SetPlanStepArgs;
+			const args = params as SetPlanStepsArgs;
 			if (runtime.isEnabled()) {
 				return {
 					content: [{ type: "text", text: "Not while planning; refine the plan with write_plan instead." }],
@@ -549,24 +572,23 @@ export function registerTools(
 				};
 			}
 			try {
-				const plan = await runtime.plans.setStep(ctx.cwd, args.plan_path, args.step, args.status);
+				const plan = await runtime.plans.setSteps(ctx.cwd, args.plan_path, args.updates);
 				const steps = extractPlanSteps(plan.content);
 				const done = steps.filter((step) => step.status === "completed").length;
 				return {
 					content: [
 						{
 							type: "text",
-							text: `Step ${args.step} is ${args.status} (${done}/${steps.length} done) in ${plan.relativePath}.`,
+							text: `${describeUpdates(args.updates)} (${done}/${steps.length} done) in ${plan.relativePath}.`,
 						},
 					],
 					details: {
 						path: plan.path,
 						relativePath: plan.relativePath,
-						step: args.step,
-						status: args.status,
+						updates: args.updates,
 						done,
 						total: steps.length,
-					} satisfies SetPlanStepDetails,
+					} satisfies SetPlanStepsDetails,
 				};
 			} catch (error) {
 				const message = error instanceof Error ? error.message : String(error);
@@ -580,24 +602,32 @@ export function registerTools(
 
 		renderCall(args, theme, context) {
 			const text = reuseText(context?.lastComponent);
-			const step = (args as Partial<SetPlanStepArgs>).step;
-			const suffix = theme.fg("muted", step === undefined ? "a step" : `step ${step}`);
-			text.setText(theme.fg("toolTitle", theme.bold(`${SET_PLAN_STEP_TOOL} `)) + suffix);
+			const updates = (args as Partial<SetPlanStepsArgs>).updates;
+			const label =
+				updates && updates.length === 1 && updates[0]?.step !== undefined
+					? `step ${updates[0].step}`
+					: updates && updates.length > 1
+						? `${updates.length} steps`
+						: "steps";
+			text.setText(theme.fg("toolTitle", theme.bold(`${SET_PLAN_STEPS_TOOL} `)) + theme.fg("muted", label));
 			return text;
 		},
 
 		renderResult(result, _options, theme, context) {
 			const text = reuseText(context?.lastComponent);
-			const details = toolDetails<SetPlanStepDetails>(result.details);
+			const details = toolDetails<SetPlanStepsDetails>(result.details);
 			if (!details) {
 				const first = result.content[0];
 				text.setText(first?.type === "text" ? first.text : "");
 				return text;
 			}
+			const completed = details.updates.filter((update) => update.status === "completed").map((u) => u.step);
+			const pending = details.updates.filter((update) => update.status === "pending").map((u) => u.step);
 			const heading =
-				details.status === "completed"
-					? theme.fg("success", `✓ Step ${details.step} done`)
-					: theme.fg("muted", `Step ${details.step} reopened`);
+				completed.length > 0
+					? theme.fg("success", `✓ ${stepLabel(completed)} done`) +
+						(pending.length > 0 ? theme.fg("muted", ` · ${stepLabel(pending)} reopened`) : "")
+					: theme.fg("muted", `${stepLabel(pending)} reopened`);
 			text.setText(heading + theme.fg("dim", ` · ${details.done}/${details.total}`));
 			return text;
 		},

package/extensions/plan/types.ts

@@ -2,6 +2,8 @@
  * Shared types for the plan-mode extension.
  */
 
+import type { StepUpdate } from "./steps.ts";
+
 /** Persisted (custom entry) plan-mode state. */
 export interface PlanModeEntry {
 	enabled: boolean;
@@ -32,16 +34,14 @@ export interface WritePlanDetails {
 	bytes: number;
 }
 
-/** Structured result carried in `set_plan_step`'s `details`. */
-export interface SetPlanStepDetails {
+/** Structured result carried in `set_plan_steps`'s `details`. */
+export interface SetPlanStepsDetails {
 	/** Absolute path of the plan file that was updated. */
 	path: string;
 	/** Path relative to the working directory, for transcript display. */
 	relativePath: string;
-	/** The 1-based step number that was set. */
-	step: number;
-	/** The step's new status. */
-	status: "pending" | "completed";
+	/** The checkbox changes that were applied, in call order. */
+	updates: StepUpdate[];
 	/** Completed steps after the update. */
 	done: number;
 	/** Total top-level steps in the plan. */

package/lib/README.md

@@ -24,7 +24,7 @@ belongs in that extension instead — see `extensions/web-access/http.ts` and
 | [`interaction.ts`](./interaction.ts) | Turn-affinity prompt contract: the channel a remote adapter installs on `ctx.ui` under a string key, the `InteractionChannel` interface (`request` / `post` / `postFile`), and `remoteChannel` / `isRemoteTurn` / `supportsCustomUI` / `askHuman` (with an optional file/text preface) plus the shared “Other” label. |
 | [`git/`](./git/) | Shared git plumbing. [`git/runner.ts`](./git/runner.ts) is the `RunGit` seam (`createExecRunner` with timeout/env handling, `runGitOrThrow`, `GitError`); [`git/read.ts`](./git/read.ts) holds the read helpers (`repoRoot`, `repoRootFor`, `gitDir`, `revParse`, `hasCommits`, `currentBranch`); [`git/index.ts`](./git/index.ts) is the barrel. |
 | [`highlight.ts`](./highlight.ts) | First-party, escape-first syntax tokenizer (`detectLanguage`, `fenceLanguage`, `highlightLines`, `HighlightLanguage`, `TokenClass`) for js/ts, json, css, html, shell, python, java, yaml, and sql; shared by `file-browse` and plan's code blocks. |
-| [`list-cursor.ts`](./list-cursor.ts) | Scrollable-list behavior for the `/rewind`, `/extensions`, `/doctor`, `/todos`, and `/jobs` screens: `fitRows`, `keepVisible`, `clampScroll`, `navIntent`, `navigateList`, `navigateScroll`, `wheelDelta`, `scrollList`, `isCloseKey`, `selectionMarker`, `formatRange`, and the `ListCursor` state. |
+| [`list-cursor.ts`](./list-cursor.ts) | Scrollable-list behavior for the `/rewind`, `/bytefly`, `/todos`, and `/jobs` screens: `fitRows`, `keepVisible`, `clampScroll`, `navIntent`, `navigateList`, `navigateScroll`, `wheelDelta`, `scrollList`, `isCloseKey`, `selectionMarker`, `formatRange`, and the `ListCursor` state. |
 | [`path.ts`](./path.ts) | `isInside` (string containment) plus `realPathOfNearest` / `isInsideReal` (symlink-aware): the containment predicates the file-browse, worktree, and doc guards share. |
 | [`pi-invocation.ts`](./pi-invocation.ts) | `getPiInvocation`: how to launch a nested `pi` process, shared by `subagent` and bytefly's doctor surface. |
 | [`policy.ts`](./policy.ts) | `createReadOnlyPolicy`: default-deny tool classification from `readOnlyHint` + explicit allow/deny. |

package/lib/requirements.ts

@@ -194,9 +194,9 @@ export const REQUIREMENTS: Readonly<Record<string, ExtensionRequirements>> = {
 	bytefly: {
 		configFiles: [SHARED_CONFIG_FILE],
 		notes: [
-			"TUI only for the header, footer, and turn rule; `/extensions` prints a listing and `/doctor` prints its report in RPC, JSON, and print modes.",
-			"The `/extensions` picker writes through Pi's own settings; project scope requires a trusted project.",
-			"`/doctor`'s list screen and the proactive missing-prerequisite warning need an interactive UI.",
+			"TUI only for the header, footer, and turn rule; `/bytefly` (and the `/extensions` and `/doctor` aliases) prints a listing and report in RPC, JSON, and print modes.",
+			"The panel writes extension enablement through Pi's own settings; project scope requires a trusted project.",
+			"`/bytefly`'s list screen and the proactive missing-prerequisite warning need an interactive UI.",
 		],
 	},
 };

package/lib/tui.ts

@@ -3,9 +3,9 @@
  * todo).
  *
  * `FULL_SCREEN_OVERLAY` is for the one surface that must own the whole terminal:
- * the plan read/review screen. Every list screen (`/todos`, `/jobs`,
- * `/rewind`) stays in the dock's editor slot so they match each other and
- * leave the transcript visible.
+ * the plan read/review screen. Every list screen (`/bytefly`, `/todos`, `/jobs`,
+ * `/rewind`) stays in the dock's editor slot so they match each other and leave
+ * the transcript visible.
  *
  * Screens are full-width, terminal-height-aware components. These helpers build
  * the top rule, clamp a screen's body to the terminal, and render the key-hint
@@ -128,11 +128,10 @@ export interface DockListFrame {
  * its blank line, the body, an optional range row, optional detail rows, and
  * the blank-line-padded key-hint footer.
  *
- * This is the convention `/extensions` shares with `/jobs` and `/todos`.
- * `/doctor` (no summary, no trailing blank, range last) and `/rewind` (a blank
- * after the header and no blank before the hint) place their blanks
- * differently, so they stay hand-rolled rather than changing their output to
- * fit this helper.
+ * This is the convention the bytefly panel (`/bytefly`, and the `/extensions`
+ * and `/doctor` aliases) shares with `/jobs` and `/todos`. `/rewind` (a blank
+ * after the header and no blank before the hint) places its blanks differently,
+ * so it stays hand-rolled rather than changing its output to fit this helper.
  */
 export function dockListFrame(frame: DockListFrame): string[] {
 	const width = Math.max(1, frame.width);

package/lib/web/components.ts

@@ -113,6 +113,17 @@ th[aria-sort="descending"] .bf-sort::after{content:" \\25BC";font-size:9px;color
 .bf-md pre{border:1px solid var(--border);border-radius:var(--radius-md);padding:var(--space-3) var(--space-4);overflow:auto;background:var(--canvas)}
 .bf-md pre code{background:none;border:0;padding:0}
 .bf-md blockquote{margin-left:0;padding:0 1em;border-left:3px solid var(--border);color:var(--muted)}
+.bf-md li.task{list-style:none;margin-left:-1.4em}
+.bf-md .md-check{color:var(--success)}
+.bf-md table{margin:0;border-collapse:collapse;table-layout:auto;width:auto;font-size:var(--fs-sm)}
+.bf-md .md-table-wrap{overflow-x:auto;margin:0 0 1em}
+.bf-md th,.bf-md td{border:1px solid var(--border);padding:6px 10px;text-align:left;vertical-align:top}
+.bf-md thead th{background:var(--surface);font-weight:var(--fw-medium)}
+.bf-md .md-left{text-align:left}
+.bf-md .md-center{text-align:center}
+.bf-md .md-right{text-align:right}
+.bf-md img{max-width:100%;height:auto;border-radius:var(--radius-sm)}
+.bf-md hr{border:0;border-top:1px solid var(--border);margin:1.4em 0}
 .bf-comment{padding:var(--space-3) var(--space-4);background:var(--surface);border:1px solid var(--border);border-left:3px solid var(--warn);border-radius:var(--radius-md)}
 .bf-comment__quote{margin-bottom:var(--space-1);color:var(--muted);font-size:var(--fs-xs);overflow-wrap:anywhere}
 .bf-comment__body{white-space:pre-wrap;overflow-wrap:anywhere}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gavin-hu/my-pi-agent",
-  "version": "0.7.0",
+  "version": "0.8.0",
   "description": "Personal Pi customizations, packaged as a Pi package: worktree isolation and rewind snapshots, question, todo, goal, plan, subagent, job, watch, web-access, file-browse, file-open, document, memory, wechat, and bytefly.",
   "keywords": [
     "pi-package",

package/extensions/bytefly/doctor/commands.ts

@@ -1,68 +0,0 @@
-/**
- * The `/doctor` command.
- *
- * One command for the whole surface: `/doctor` reports everything, `/doctor
- * document` narrows it, `/doctor refresh` re-probes. The TUI opens a docked
- * list screen; every other mode prints the same report as text.
- */
-
-import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-import { withRailsSuppressed } from "../../../lib/rails.ts";
-import { REQUIREMENTS, type ExtensionReport } from "../../../lib/requirements.ts";
-import { formatExtension, formatReport, hasProblems } from "./format.ts";
-import type { DoctorRuntime } from "./runtime.ts";
-import { DoctorScreen } from "./tui.ts";
-
-/** The report text for a set of reports, narrowed when one extension was named. */
-function reportText(name: string | undefined, reports: readonly ExtensionReport[]): string {
-	const first = reports[0];
-	return name && first ? formatExtension(first) : formatReport(reports);
-}
-
-/** Register the `/doctor` command. */
-export function registerCommands(pi: ExtensionAPI, runtime: DoctorRuntime): void {
-	pi.registerCommand("doctor", {
-		description: "Check that every extension has what it needs: /doctor [extension] [refresh]",
-		getArgumentCompletions: (prefix: string) => {
-			const options = [...Object.keys(REQUIREMENTS), "refresh"].filter((option) => option.startsWith(prefix));
-			return options.length > 0 ? options.map((value) => ({ value, label: value })) : null;
-		},
-		handler: async (args, ctx) => {
-			const parts = args.trim().split(/\s+/).filter(Boolean);
-			const refresh = parts.includes("refresh");
-			const name = parts.find((part) => part !== "refresh");
-
-			const reports = await runtime.probeAll({ refresh });
-			if (name && !reports.some((report) => report.extension === name)) {
-				ctx.ui.notify(`doctor: no extension named "${name}"`, "warning");
-				return;
-			}
-			const selected = name ? reports.filter((report) => report.extension === name) : reports;
-
-			if (ctx.mode !== "tui") {
-				ctx.ui.notify(reportText(name, selected), hasProblems(selected) ? "warning" : "info");
-				return;
-			}
-
-			// The dock screen owns the editor slot; hide the rails for its lifetime.
-			let current = selected;
-			await withRailsSuppressed(pi, () =>
-				ctx.ui.custom<void>(
-					(tui, theme, _keybindings, done) =>
-						new DoctorScreen({
-							reports: () => current,
-							rows: () => tui.terminal?.rows,
-							theme,
-							requestRender: () => tui.requestRender(),
-							refresh: async () => {
-								const next = await runtime.probeAll({ refresh: true });
-								current = name ? next.filter((report) => report.extension === name) : next;
-								tui.requestRender();
-							},
-							done,
-						}),
-				),
-			);
-		},
-	});
-}

package/extensions/bytefly/doctor/tui.ts

@@ -1,140 +0,0 @@
-/**
- * The `/doctor` TUI screen.
- *
- * One row per extension with a severity glyph, plus a detail pane for the
- * focused row. It stays in the dock's editor slot like `/jobs` and `/rewind`,
- * so the transcript stays visible; `Esc` closes it.
- *
- * The screen holds no themed strings: `render` reads the live report each time
- * and `invalidate` has nothing to rebuild. Report text comes from `format.ts`,
- * which sanitizes everything that originated in a probe.
- */
-
-import type { Theme } from "@earendil-works/pi-coding-agent";
-import {
-	Key,
-	matchesKey,
-	truncateToWidth,
-	type Component,
-	type TuiMouseEvent,
-	type TuiMouseEventResult,
-} from "@earendil-works/pi-tui";
-import { fitRows, formatRange, ListCursor, navigateList, scrollList } from "../../../lib/list-cursor.ts";
-import { screenHeader, screenHint } from "../../../lib/tui.ts";
-import type { ExtensionReport } from "../../../lib/requirements.ts";
-import { countFindings, detailLines, extensionRow } from "./format.ts";
-
-/** Body rows when the terminal height is unknown. */
-const SCREEN_DEFAULT_ROWS = 12;
-/** Rows reserved for the focused extension's detail pane. */
-const DETAIL_ROWS = 4;
-/** Fixed rows around the list: header, blank, detail label, hint. */
-const CHROME = 4;
-
-export interface DoctorScreenOptions {
-	/** Current reports; read on every render so a refresh is picked up. */
-	reports: () => ExtensionReport[];
-	/** Live terminal height, for the row budget. */
-	rows: () => number | undefined;
-	theme: Theme;
-	requestRender: () => void;
-	/** Re-probe with `refresh` and update `reports`. May be async; awaited before the busy state clears. */
-	refresh: () => void | Promise<void>;
-	/** Close the screen. */
-	done: () => void;
-}
-
-/** The docked `/doctor` list screen. */
-export class DoctorScreen implements Component {
-	private readonly cursor: ListCursor;
-	private showAll = false;
-	/** True while a re-probe is in flight, so `r` does not stack probes. */
-	private refreshing = false;
-
-	constructor(private readonly options: DoctorScreenOptions) {
-		this.cursor = new ListCursor(() => this.options.requestRender());
-	}
-
-	invalidate(): void {}
-
-	/** Re-probe once; a second request while one is in flight is ignored. */
-	async refresh(): Promise<void> {
-		if (this.refreshing) return;
-		this.refreshing = true;
-		this.showAll = false;
-		this.options.requestRender();
-		try {
-			await this.options.refresh();
-		} finally {
-			this.refreshing = false;
-			this.options.requestRender();
-		}
-	}
-
-	private count(): number {
-		return this.options.reports().length;
-	}
-
-	handleInput(data: string): void {
-		if (matchesKey(data, Key.escape)) {
-			this.options.done();
-			return;
-		}
-		if (matchesKey(data, "r")) {
-			void this.refresh();
-			return;
-		}
-		if (matchesKey(data, Key.enter)) {
-			this.showAll = !this.showAll;
-			this.options.requestRender();
-			return;
-		}
-
-		navigateList(data, this.cursor, this.count());
-	}
-
-	handleMouse(event: TuiMouseEvent): TuiMouseEventResult | undefined {
-		return scrollList(event, this.cursor, this.count());
-	}
-
-	// Kept off `dockListFrame`: `/doctor` has no summary, prints the range after
-	// the detail, and has no trailing blank, so the shared frame would change its
-	// output. Its blank-line convention is deliberately local.
-	render(width: number): string[] {
-		const w = Math.max(1, width);
-		const reports = this.options.reports();
-		const count = reports.length;
-
-		this.cursor.visible = Math.max(1, fitRows(this.options.rows(), count, CHROME + DETAIL_ROWS, SCREEN_DEFAULT_ROWS));
-		this.cursor.sync(count);
-		const window = this.cursor.window(count);
-
-		const lines: string[] = [screenHeader(this.options.theme, w, "doctor")];
-		for (let index = window.start; index < window.end; index += 1) {
-			const report = reports[index];
-			if (report) lines.push(extensionRow(report, this.options.theme, w, index === this.cursor.selected));
-		}
-		lines.push("");
-
-		const focused = reports[this.cursor.selected];
-		const counts = countFindings(focused ? [focused] : []);
-		const label = focused
-			? `  ${focused.extension} — ${counts.missing} missing, ${counts.degraded} degraded`
-			: "  no extensions to report";
-		lines.push(truncateToWidth(this.options.theme.fg("dim", label), w));
-		lines.push(...detailLines(focused, this.options.theme, w, DETAIL_ROWS, this.showAll));
-
-		if (this.cursor.clipped(count)) {
-			lines.push(formatRange(this.options.theme, w, { start: window.start, end: window.end, total: count }));
-		}
-		lines.push(
-			screenHint(this.options.theme, w, [
-				"Esc close",
-				"↑/↓ select",
-				this.showAll ? "Enter problems" : "Enter all checks",
-				this.refreshing ? "refreshing…" : "r refresh",
-			]),
-		);
-		return lines;
-	}
-}

package/extensions/bytefly/picker/commands.ts

@@ -1,73 +0,0 @@
-/**
- * The `/extensions` command.
- *
- * In a terminal session it opens the dock list, where Space toggles the focused
- * extension and Tab switches between Global and Project settings. Everywhere
- * else it prints the same list as text. After the screen closes with pending
- * changes it offers to reload so the new selection takes effect.
- */
-
-import { type ExtensionAPI, getAgentDir } from "@earendil-works/pi-coding-agent";
-import { withRailsSuppressed } from "../../../lib/rails.ts";
-import { discoverExtensions, type DiscoveryDeps } from "./discovery.ts";
-import { formatListing } from "./resources.ts";
-import { createExtensionsPickerRuntime } from "./runtime.ts";
-import { ExtensionsListComponent } from "./tui.ts";
-import type { Discovery } from "./types.ts";
-
-/** Host seams, injected by tests. */
-export interface CommandDeps {
-	discover?: (deps: DiscoveryDeps) => Promise<Awaited<ReturnType<typeof discoverExtensions>>>;
-}
-
-export function registerExtensionsCommand(pi: ExtensionAPI, deps: CommandDeps = {}): void {
-	const discover = deps.discover ?? discoverExtensions;
-
-	pi.registerCommand("extensions", {
-		description: "List and enable/disable extensions",
-		handler: async (_args, ctx) => {
-			const cwd = ctx.cwd ?? process.cwd();
-			const agentDir = getAgentDir();
-			let discovery: Discovery;
-			try {
-				discovery = await discover({ cwd, agentDir, trusted: ctx.isProjectTrusted() });
-			} catch (error) {
-				ctx.ui.notify(`Could not list extensions: ${error instanceof Error ? error.message : String(error)}`, "error");
-				return;
-			}
-			const runtime = createExtensionsPickerRuntime({
-				discovery,
-				cwd,
-				agentDir,
-				onError: (message) => ctx.ui.notify(message, "error"),
-			});
-
-			if (ctx.mode !== "tui") {
-				ctx.ui.notify(formatListing(runtime.items(), agentDir), "info");
-				return;
-			}
-
-			await withRailsSuppressed(pi, () =>
-				ctx.ui.custom<void>((tui, theme, _keybindings, done) => {
-					const unsubscribe = runtime.onChange(() => tui.requestRender());
-					return new ExtensionsListComponent(
-						runtime,
-						theme,
-						agentDir,
-						() => {
-							unsubscribe();
-							done();
-						},
-						() => tui.requestRender(),
-						() => tui.terminal?.rows,
-						(message) => ctx.ui.notify(message, "warning"),
-					);
-				}),
-			);
-
-			if (!runtime.isDirty()) return;
-			const reload = await ctx.ui.confirm("Reload to apply changes?", "Extension changes take effect after a reload.");
-			if (reload) await ctx.reload();
-		},
-	});
-}