@narumitw/pi-btw 0.42.0 → 0.43.0

package/README.md

@@ -8,13 +8,14 @@ Use it when you want to ask a temporary question, inspect context, or get a shor
 
 ## ✨ Features
 
-- Adds a `/btw` side-thread command to Pi, with an optional initial question.
+- Adds a `/btw` menu for starting a side thread or changing pi-btw settings.
+- Keeps `/btw <question>` as a direct fast path.
 - Answers side questions in a temporary, scrollable UI.
 - Supports follow-up questions in the same ephemeral side thread.
 - Optionally brings the latest answer, a question-to-end suffix, an exact line range, or the entire side thread into the main editor.
 - Uses the current session branch as context.
 - Uses Pi's current model or an independent model selected in `pi-btw.json`.
-- Inherits Pi's current thinking level or uses a fixed level from `pi-btw.json`.
+- Uses a pi-btw thinking level that can be changed with Pi's configured thinking shortcut and remembered for next time.
 - Does not append the side question or answer to the main conversation.
 - Works as an independently installable npm Pi extension package.
 
@@ -38,7 +39,7 @@ pi -e ./extensions/pi-btw
 
 ## 🚀 Usage
 
-Start an empty side thread or provide its first question immediately:
+Open the pi-btw menu or provide the first question immediately:
 
 ```text
 /btw
@@ -54,16 +55,23 @@ Examples:
 /btw is this API name idiomatic?
 ```
 
-Running `/btw` alone opens an empty ephemeral side thread with its editor ready. When an
-initial question is provided, its answer opens above the same editor. A compact
-`btw · side thread` header stays fixed above the content so the ephemeral workspace remains
-recognizable while scrolling. Messages use Pi's normal user and assistant presentation without
-numbered turns or role labels. Type each question and press `Enter`; no follow-up shortcut is
-required.
+Running `/btw` alone opens a two-row menu. **Start side thread** is selected first, so pressing
+`Enter` opens an empty ephemeral side thread; **Settings** changes the starting thinking level
+and whether shortcut changes are remembered. `/btw <question>` bypasses this menu, and its answer
+opens above the side-thread editor. A compact `btw · side thread` header stays fixed above the
+content so the ephemeral workspace remains recognizable while scrolling. Messages use Pi's normal
+user and assistant presentation without numbered turns or role labels. Type each question and press
+`Enter`; no follow-up shortcut is required.
 Previous side questions and answers remain available to the model and visible for that
-invocation. While a response is running, the transcript stays visible above a compact
-`Answering…` status. The footer shows `PgUp`/`PgDn` only when history can scroll; press
-`Ctrl+C` to cancel an in-progress answer or leave the side thread.
+invocation. The side-thread header shows its current thinking level. Press Pi's configured
+`app.thinking.cycle` shortcut (`Shift+Tab` by default) in the composer to cycle the levels
+supported by the side-thread model; every later question uses the displayed level until it is
+changed again. By default, each shortcut change is also written to `pi-btw.json` for the next
+invocation. Turn **Remember thinking level changes** off in Settings to keep changes local to the
+current side thread. Neither path changes the main session's thinking level. While a response is
+running, the transcript stays visible above a compact `Answering…` status.
+The footer shows `PgUp`/`PgDn` only when history can scroll; press `Ctrl+C` to cancel an
+in-progress answer or leave the side thread.
 
 After at least one successful answer, press `Ctrl+R` to bring selected context to the main
 editor. The scope menu shows the size of the latest question and answer and the entire side
@@ -106,7 +114,8 @@ setting; pi-btw does not add any environment variables.
 ```json
 {
   "model": "anthropic/claude-sonnet-4-5",
-  "thinkingLevel": "low"
+  "thinkingLevel": "low",
+  "rememberThinkingLevelChanges": true
 }
 ```
 
@@ -117,17 +126,26 @@ cannot be found or authenticated, pi-btw warns and falls back to the current ses
 If neither model is available, `/btw` reports an error and stops. This selection affects only
 `/btw`; it does not change the main session model.
 
-Pi calls its reasoning setting the **thinking level**. By default, `/btw` inherits the
-current runtime level, including changes made through `/settings` or `Shift+Tab`. It does
-not read or change `defaultThinkingLevel` directly. Supported fixed values are `off`,
-`minimal`, `low`, `medium`, `high`, and `xhigh`. The selected value applies to the model
-actually used by `/btw` and does not change the main session. Pi's provider layer may clamp
-a requested level when that model does not support it.
-
-The settings file is optional and is never created automatically. A missing file, `{}`, or
-omitted fields silently inherit the current Pi model and thinking level. The file is read for
-each `/btw` invocation, so edits apply to the next side question without `/reload`. Invalid
-or unreadable settings produce a warning and fall back to the current Pi defaults.
+Pi calls its reasoning setting the **thinking level**. `thinkingLevel` sets pi-btw's starting
+level; accepted values are `off`, `minimal`, `low`, `medium`, `high`, `xhigh`, and `max`. When the
+field is absent for backward compatibility, the next invocation starts from the current session
+level. The initial value and shortcut cycle are clamped to the selected side model's capabilities
+using Pi's model rules. Pi-btw does not read, write, or change the main session's
+`defaultThinkingLevel`.
+
+`rememberThinkingLevelChanges` controls only persistence and defaults to `true` when omitted. A
+side-thread shortcut always changes that side thread immediately. When remembering is on, the
+concrete level is written for the next invocation; when off, `pi-btw.json` stays unchanged. If a
+shortcut write fails, the local change remains active and pi-btw warns that it was not remembered.
+A failed Settings-screen save instead restores the previous displayed value.
+
+A missing settings file is a side-effect-free read: pi-btw creates it only after a Settings change
+or a remembered shortcut change. Saves are ordered within the Pi process and published atomically
+with a same-directory temporary file and rename. They preserve `model` and unknown fields; malformed
+or invalid files block saves and remain unchanged. Settings must be valid UTF-8 and no larger than
+64 KiB, so unexpectedly large or invalidly encoded files are rejected without being rewritten.
+Separate Pi processes and external editors are outside this in-process ordering boundary. The file
+is read for each `/btw` invocation, so edits apply without `/reload`.
 
 ## 🧠 Why use pi-btw?
 
@@ -141,7 +159,10 @@ extensions/pi-btw/
 │   ├── index.ts
 │   ├── btw.ts
 │   ├── bring-to-main.ts
+│   ├── menu.ts
+│   ├── settings.ts
 │   ├── side-thread.ts
+│   ├── text.ts
 │   └── transcript-pager.ts
 ├── README.md
 ├── LICENSE

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@narumitw/pi-btw",
-	"version": "0.42.0",
+	"version": "0.43.0",
 	"description": "Pi extension that adds a /btw side-question command.",
 	"type": "module",
 	"license": "MIT",
@@ -38,5 +38,13 @@
 		"type": "git",
 		"url": "https://github.com/narumiruna/pi-extensions",
 		"directory": "extensions/pi-btw"
+	},
+	"dependencies": {
+		"@narumitw/pi-tui-kit": "^0.42.0"
+	},
+	"peerDependencies": {
+		"@earendil-works/pi-ai": "*",
+		"@earendil-works/pi-coding-agent": "*",
+		"@earendil-works/pi-tui": "*"
 	}
 }

package/src/bring-to-main.ts

@@ -47,11 +47,6 @@ export type BtwQuickBringToMainScope =
 	| { kind: "from"; answeredTurnIndex: number }
 	| { kind: "entire" };
 
-export type BtwMenuSelectorAction =
-	| { kind: "select"; value: string }
-	| { kind: "back" }
-	| { kind: "close" };
-
 export type BtwTextRangeSelectorAction =
 	| { kind: "confirm"; segments: BtwBringToMainSegment[] }
 	| { kind: "back" }
@@ -175,210 +170,6 @@ export function formatBtwBringToMain(segments: readonly BtwBringToMainSegment[])
 	].join("\n");
 }
 
-export type BtwBringToMainPreviewAction = { kind: "bring" } | { kind: "back" } | { kind: "close" };
-
-export class BtwBringToMainPreview implements Component {
-	private readonly lines: string[];
-	private displayLines: string[];
-	private scrollOffset = 0;
-	private finished = false;
-
-	constructor(
-		private readonly tui: TUI,
-		private readonly theme: Theme,
-		private readonly keybindings: KeybindingsManager,
-		draft: string,
-		private readonly summary: BtwBringToMainSummary,
-		private readonly onAction: (action: BtwBringToMainPreviewAction) => void,
-	) {
-		this.lines = draft.split("\n").map(escapeTerminalControls);
-		this.displayLines = this.lines;
-	}
-
-	render(width: number): string[] {
-		const safeWidth = Math.max(1, width);
-		this.displayLines = this.lines.flatMap((line) => wrapPreviewLine(line, safeWidth));
-		const availableRows = Math.max(1, this.tui.terminal.rows - RESERVED_APP_ROWS);
-		const showFooter = availableRows >= 3;
-		const viewportHeight = Math.max(1, availableRows - 1 - (showFooter ? 1 : 0));
-		this.clampScroll(viewportHeight);
-		const count = this.summary.messages === 1 ? "1 message" : `${this.summary.messages} messages`;
-		const lineCount = this.summary.lines === 1 ? "1 line" : `${this.summary.lines} lines`;
-		const firstVisible = Math.min(this.displayLines.length, this.scrollOffset + 1);
-		const lastVisible = Math.min(this.displayLines.length, this.scrollOffset + viewportHeight);
-		const scrollable = this.displayLines.length > viewportHeight;
-		const position = `${firstVisible}–${lastVisible}/${this.displayLines.length}`;
-		const header = `Preview${scrollable ? ` ${position}` : ""} · ${count} · ${lineCount} · ~${this.summary.tokens} tokens`;
-		const actions = `${confirmKeyLabel(this.keybindings)} bring • ${keybindingLabel(this.keybindings, "tui.select.cancel", ["ctrl+c"])} back • Ctrl+C close`;
-		const scroll = `${position} • ${keybindingLabel(this.keybindings, "tui.select.pageUp")}/${keybindingLabel(this.keybindings, "tui.select.pageDown")} scroll`;
-		const detailedFooter = scrollable ? `${scroll} • ${actions}` : actions;
-		const footer = visibleWidth(detailedFooter) <= safeWidth ? detailedFooter : actions;
-		return fitRows(
-			[
-				truncateToWidth(this.theme.fg("accent", this.theme.bold(header)), safeWidth, ""),
-				...this.displayLines.slice(this.scrollOffset, this.scrollOffset + viewportHeight),
-				...(showFooter ? [truncateToWidth(this.theme.fg("muted", footer), safeWidth, "")] : []),
-			],
-			availableRows,
-		);
-	}
-
-	handleInput(data: string): void {
-		if (this.finished) return;
-		if (matchesKey(data, Key.ctrl("c"))) {
-			this.finish({ kind: "close" });
-			return;
-		}
-		if (this.keybindings.matches(data, "tui.select.cancel")) {
-			this.finish({ kind: "back" });
-			return;
-		}
-		if (matchesConfirm(data, this.keybindings)) {
-			this.finish({ kind: "bring" });
-			return;
-		}
-		if (this.keybindings.matches(data, "tui.select.pageUp")) {
-			this.scrollOffset -= Math.max(1, this.tui.terminal.rows - RESERVED_APP_ROWS - 2);
-			this.clampScroll(Math.max(0, this.tui.terminal.rows - RESERVED_APP_ROWS - 2));
-			this.tui.requestRender();
-			return;
-		}
-		if (this.keybindings.matches(data, "tui.select.pageDown")) {
-			this.scrollOffset += Math.max(1, this.tui.terminal.rows - RESERVED_APP_ROWS - 2);
-			this.clampScroll(Math.max(0, this.tui.terminal.rows - RESERVED_APP_ROWS - 2));
-			this.tui.requestRender();
-		}
-	}
-
-	invalidate(): void {}
-
-	private clampScroll(viewportHeight: number): void {
-		this.scrollOffset = Math.max(
-			0,
-			Math.min(this.scrollOffset, Math.max(0, this.displayLines.length - viewportHeight)),
-		);
-	}
-
-	private finish(action: BtwBringToMainPreviewAction): void {
-		if (this.finished) return;
-		this.finished = true;
-		this.onAction(action);
-	}
-}
-
-export class BtwMenuSelector implements Component {
-	private cursor = 0;
-	private scrollOffset = 0;
-	private finished = false;
-
-	constructor(
-		private readonly tui: TUI,
-		private readonly theme: Theme,
-		private readonly keybindings: KeybindingsManager,
-		private readonly title: string,
-		private readonly options: readonly string[],
-		private readonly onAction: (action: BtwMenuSelectorAction) => void,
-		initialValue?: string,
-	) {
-		const initialIndex = initialValue === undefined ? -1 : options.indexOf(initialValue);
-		if (initialIndex >= 0) this.cursor = initialIndex;
-	}
-
-	render(width: number): string[] {
-		const safeWidth = Math.max(1, width);
-		const availableRows = Math.max(1, this.tui.terminal.rows - RESERVED_APP_ROWS);
-		const showFooter = availableRows >= 3;
-		const viewportHeight = Math.max(1, availableRows - 1 - (showFooter ? 1 : 0));
-		this.keepCursorVisible(viewportHeight);
-		const rows = this.options
-			.slice(this.scrollOffset, this.scrollOffset + viewportHeight)
-			.map((option, visibleIndex) => {
-				const index = this.scrollOffset + visibleIndex;
-				const raw = `${index === this.cursor ? ">" : " "} ${escapeTerminalControls(option)}`;
-				const toned = option.startsWith("⚠") ? this.theme.fg("warning", raw) : raw;
-				const styled =
-					index === this.cursor ? this.theme.bg("selectedBg", this.theme.fg("text", toned)) : toned;
-				return truncateToWidth(styled, safeWidth, "");
-			});
-		return fitRows(
-			[
-				truncateToWidth(
-					this.theme.fg("accent", this.theme.bold(escapeTerminalControls(this.title))),
-					safeWidth,
-					"",
-				),
-				...rows,
-				...(showFooter
-					? [
-							truncateToWidth(
-								this.theme.fg(
-									"muted",
-									`${confirmKeyLabel(this.keybindings)} confirm • ${keybindingLabel(this.keybindings, "tui.select.cancel", ["ctrl+c"])} back • Ctrl+C close`,
-								),
-								safeWidth,
-								"",
-							),
-						]
-					: []),
-			],
-			availableRows,
-		);
-	}
-
-	handleInput(data: string): void {
-		if (this.finished) return;
-		if (matchesKey(data, Key.ctrl("c"))) {
-			this.finish({ kind: "close" });
-			return;
-		}
-		if (this.keybindings.matches(data, "tui.select.cancel")) {
-			this.finish({ kind: "back" });
-			return;
-		}
-		if (matchesConfirm(data, this.keybindings)) {
-			const value = this.options[this.cursor];
-			if (value !== undefined) this.finish({ kind: "select", value });
-			return;
-		}
-		if (this.keybindings.matches(data, "tui.select.up")) {
-			this.cursor = Math.max(0, this.cursor - 1);
-			this.tui.requestRender();
-			return;
-		}
-		if (this.keybindings.matches(data, "tui.select.down")) {
-			this.cursor = Math.min(Math.max(0, this.options.length - 1), this.cursor + 1);
-			this.tui.requestRender();
-			return;
-		}
-		if (this.keybindings.matches(data, "tui.select.pageUp")) {
-			this.cursor = Math.max(0, this.cursor - 10);
-			this.tui.requestRender();
-			return;
-		}
-		if (this.keybindings.matches(data, "tui.select.pageDown")) {
-			this.cursor = Math.min(Math.max(0, this.options.length - 1), this.cursor + 10);
-			this.tui.requestRender();
-			return;
-		}
-	}
-
-	invalidate(): void {}
-
-	private keepCursorVisible(height: number): void {
-		if (height <= 0) return;
-		if (this.cursor < this.scrollOffset) this.scrollOffset = this.cursor;
-		if (this.cursor >= this.scrollOffset + height) {
-			this.scrollOffset = this.cursor - height + 1;
-		}
-	}
-
-	private finish(action: BtwMenuSelectorAction): void {
-		if (this.finished) return;
-		this.finished = true;
-		this.onAction(action);
-	}
-}
-
 export class BtwTextRangeSelector implements Component {
 	private readonly lines: BtwSelectionLine[];
 	private cursor: BtwTextPosition = { line: 0, column: 0 };
@@ -788,35 +579,6 @@ function splitGraphemes(text: string): string[] {
 	return [...GRAPHEME_SEGMENTER.segment(text)].map(({ segment }) => segment);
 }
 
-function wrapPreviewLine(text: string, width: number): string[] {
-	if (!text) return [""];
-	const rows: string[] = [];
-	let row = "";
-	let rowWidth = 0;
-	const append = (value: string, valueWidth: number) => {
-		if (row && rowWidth + valueWidth > width) {
-			rows.push(row);
-			row = "";
-			rowWidth = 0;
-		}
-		row += value;
-		rowWidth += valueWidth;
-	};
-	for (const character of splitGraphemes(text)) {
-		const characterWidth = visibleWidth(character);
-		if (characterWidth <= width) {
-			append(character, characterWidth);
-			continue;
-		}
-		const codePoints = [...character]
-			.map((value) => `\\u{${value.codePointAt(0)?.toString(16) ?? "0"}}`)
-			.join("");
-		for (const value of codePoints) append(value, 1);
-	}
-	rows.push(row);
-	return rows;
-}
-
 function compareTextPositions(first: BtwTextPosition, second: BtwTextPosition): number {
 	return first.line === second.line ? first.column - second.column : first.line - second.line;
 }