@sayknow-cli/coding-agent 0.5.1 → 0.5.6

package/src/lsp/client.ts

@@ -89,12 +89,10 @@ function deleteClientLock(key: string, clientPromise: Promise<LspClient>): void
 	}
 }
 
-function evictDeadCachedClient(key: string, client: LspClient): void {
+function evictDeadCachedClient(client: LspClient): void {
 	if (client.proc.exitCode === null && !client.proc.killed && !client.owner?.disposed && !killedClients.has(client))
 		return;
-	deleteCachedClient(key, client);
-	client.resolveProjectLoaded();
-	rejectPendingRequests(client, new Error("LSP server exited"));
+	terminalizeTransport(client, new Error("LSP server exited"));
 }
 
 function terminalizeTransport(client: LspClient, cause: unknown): Error {
@@ -275,7 +273,9 @@ async function writeMessage(
 	if (terminalError) throw terminalError;
 
 	try {
-		client.proc.stdin.write(`Content-Length: ${Buffer.byteLength(content, "utf-8")}\r\n\r\n${content}`);
+		await Promise.resolve(
+			client.proc.stdin.write(`Content-Length: ${Buffer.byteLength(content, "utf-8")}\r\n\r\n${content}`),
+		);
 		await client.proc.stdin.flush();
 	} catch (error) {
 		if (isKnownSinkPeerClosedError(error)) {
@@ -302,7 +302,7 @@ function queueWriteMessage(
  * Start background message reader for a client.
  * Routes responses to pending requests and handles notifications.
  */
-async function startMessageReader(client: LspClient): Promise<void> {
+export async function startMessageReader(client: LspClient): Promise<void> {
 	if (client.isReading) return;
 	client.isReading = true;
 
@@ -311,7 +311,10 @@ async function startMessageReader(client: LspClient): Promise<void> {
 	try {
 		while (true) {
 			const { done, value } = await reader.read();
-			if (done) break;
+			if (done) {
+				terminalizeTransport(client, new Error("LSP stdout closed"));
+				break;
+			}
 
 			// Atomically update buffer before processing
 			const currentBuffer: Buffer = Buffer.concat([client.messageBuffer, value]);
@@ -368,11 +371,7 @@ async function startMessageReader(client: LspClient): Promise<void> {
 			client.messageBuffer = workingBuffer;
 		}
 	} catch (err) {
-		// Connection closed or error - reject all pending requests
-		for (const pending of Array.from(client.pendingRequests.values())) {
-			pending.reject(new Error(`LSP connection closed: ${err}`));
-		}
-		client.pendingRequests.clear();
+		terminalizeTransport(client, err);
 	} finally {
 		reader.releaseLock();
 		client.isReading = false;
@@ -488,7 +487,7 @@ export async function getOrCreateClient(config: ServerConfig, cwd: string, initT
 	// Check if client already exists
 	const existingClient = clients.get(key);
 	if (existingClient) {
-		evictDeadCachedClient(key, existingClient);
+		evictDeadCachedClient(existingClient);
 		if (clients.has(key)) {
 			existingClient.lastActivity = Date.now();
 			return existingClient;
@@ -558,13 +557,10 @@ export async function getOrCreateClient(config: ServerConfig, cwd: string, initT
 		};
 		clients.set(key, client);
 
-		// Register crash recovery - remove client on process exit
+		// Register crash recovery and durably terminalize retained references on exit.
 		proc.exited.then(async () => {
-			deleteCachedClient(key, client);
 			deleteClientLock(key, clientPromise);
-			client.resolveProjectLoaded();
-
-			// Reject any pending requests — the server is gone, they will never complete.
+			let error = new Error(`LSP server exited unexpectedly (code ${proc.exitCode})`);
 			if (client.pendingRequests.size > 0) {
 				// Strip informational log lines (e.g. marksman's [INF]/[DBG] prefix)
 				// — they are startup noise, not actionable errors.
@@ -588,16 +584,13 @@ export async function getOrCreateClient(config: ServerConfig, cwd: string, initT
 				);
 				const diagnosticSuffix = crashNotice ? `\n${crashNotice}` : "";
 				const code = proc.exitCode;
-				const err = new Error(
+				error = new Error(
 					stderr
 						? `LSP server exited (code ${code}): ${stderr}${diagnosticSuffix}`
 						: `LSP server exited unexpectedly (code ${code})${diagnosticSuffix}`,
 				);
-				for (const pending of client.pendingRequests.values()) {
-					pending.reject(err);
-				}
-				client.pendingRequests.clear();
 			}
+			terminalizeTransport(client, error);
 		});
 
 		// Start background message reader
@@ -980,12 +973,7 @@ export async function sendNotification(client: LspClient, method: string, params
 	};
 
 	client.lastActivity = Date.now();
-	try {
-		await queueWriteMessage(client, notification);
-	} catch (error) {
-		if (transportClosedErrors.get(client) === error) return;
-		throw error;
-	}
+	await queueWriteMessage(client, notification);
 }
 
 /**

package/src/main.ts

@@ -1331,7 +1331,7 @@ export async function runRootCommand(
 	// Initialize discovery system with settings for provider persistence
 	logger.time("initializeWithSettings", initializeWithSettings, settingsInstance);
 	// Auto-start Telegram Remote gateway if enabled in settings
-	if (!parsedArgs.print && !autoPrint) await maybeAutostartTelegramRemote();
+	if (!parsedArgs.print && !autoPrint) await maybeAutostartTelegramRemote(settingsInstance);
 
 	// Apply model role overrides from CLI args or env vars (ephemeral, not persisted).
 	// Precedence per role: CLI flag > documented SKC_*_MODEL > legacy PI_*_MODEL.

package/src/modes/action-registry.ts

@@ -39,6 +39,7 @@ export const APP_ACTION_METADATA: readonly ActionMetadata[] = [
 	action("app.model.select", "Select model", "Model", ["composer"]),
 	action("app.model.selectTemporary", "Select temporary model", "Model", ["composer"]),
 	action("app.tools.expand", "Toggle tool expansion", "Tools", ["composer"]),
+	action("app.todo.toggle", "Toggle todo expansion", "Tasks", ["composer"]),
 	action("app.tool.backgroundFold", "Fold foreground tool", "Tools", ["composer"]),
 	action("app.editor.external", "Open external editor", "Editor", ["composer"]),
 	action("app.message.followUp", "Send follow-up", "Messages", ["composer"]),
@@ -47,6 +48,7 @@ export const APP_ACTION_METADATA: readonly ActionMetadata[] = [
 	action("app.clipboard.pasteImage", "Paste image", "Clipboard", ["composer"]),
 	action("app.clipboard.copyLine", "Copy line", "Clipboard", ["composer"]),
 	action("app.clipboard.copyPrompt", "Copy prompt", "Clipboard", ["composer"]),
+	action("app.oauth.copyUrl", "Copy OAuth authorization URL", "OAuth", ["composer"]),
 	action("app.session.new", "New session", "Session", ["composer"]),
 	action("app.session.tree", "Session tree", "Session", ["composer"]),
 	action("app.session.fork", "Branch from message", "Session", ["composer"]),

package/src/modes/components/pet-capability.ts

@@ -1,10 +1,12 @@
 import {
+	getOverlayImageProtocol,
 	ImageProtocol,
 	isUnderTerminalMultiplexer,
+	isUnderTmux,
 	onImageProtocolChanged,
 	type SelectItem,
+	shouldProbeKittyPassthrough,
 	shouldProbeSixelCapability,
-	TERMINAL,
 } from "@sayknow-cli/tui";
 
 export type PetPixelProtocol = "sixel" | "kitty";
@@ -14,16 +16,22 @@ export const PET_SAVED_UNAVAILABLE_DESCRIPTION =
 	"Saved, unavailable — requires compatible Kitty or Sixel overlay rendering";
 export const PET_UNAVAILABLE_WARNING =
 	"⚠ Pets aren’t available in this terminal. Its image support isn’t compatible with Sayknow Pet’s overlay rendering yet. Try Kitty, Ghostty, WezTerm, or a terminal with compatible Sixel support.";
+const PET_TMUX_UNAVAILABLE_WARNING =
+	"⚠ Sayknow Pet graphics are unavailable: the terminal attached to this tmux client answered neither the Kitty nor the Sixel capability query forwarded through DCS passthrough. Attach from Ghostty, Kitty, WezTerm, or a Sixel-capable terminal, and leave tmux `allow-passthrough` enabled.";
 const PET_MULTIPLEXER_UNAVAILABLE_WARNING =
-	"⚠ Sayknow Pet graphics are unavailable inside tmux, screen, or zellij because image escapes are not forwarded end to end. Run skc outside the multiplexer, or set PI_FORCE_IMAGE_PROTOCOL=sixel only when the full terminal chain supports Sixel.";
+	"⚠ Sayknow Pet graphics are unavailable inside screen and zellij: neither forwards image escapes to the outer terminal. Run skc under tmux (which has DCS passthrough) or outside the multiplexer.";
 
 export function getPetUnavailableWarning(env: NodeJS.ProcessEnv = Bun.env): string {
+	if (isUnderTmux(env)) return PET_TMUX_UNAVAILABLE_WARNING;
 	return isUnderTerminalMultiplexer(env) ? PET_MULTIPLEXER_UNAVAILABLE_WARNING : PET_UNAVAILABLE_WARNING;
 }
 
 export function getPetPixelProtocol(): PetPixelProtocol | null {
-	if (TERMINAL.imageProtocol === ImageProtocol.Kitty) return "kitty";
-	if (TERMINAL.imageProtocol === ImageProtocol.Sixel) return "sixel";
+	// Overlay channel: inline graphics when the terminal renders them directly,
+	// otherwise the tmux passthrough protocol proven by the startup probe.
+	const protocol = getOverlayImageProtocol();
+	if (protocol === ImageProtocol.Kitty) return "kitty";
+	if (protocol === ImageProtocol.Sixel) return "sixel";
 	return null;
 }
 
@@ -55,24 +63,25 @@ export function createPetSelectItems(
 
 /**
  * Grace period before declaring the terminal pet-incapable at startup. The
- * asynchronous Sixel capability probe starts inside `TUI.start()` and answers
- * within its own 250 ms deadline; this margin covers probe scheduling so a
- * supported terminal is never told it is incompatible while the probe is
- * still in flight.
+ * asynchronous capability probes start inside `TUI.start()` and answer within
+ * their own deadline (250 ms directly, 600 ms through tmux passthrough); this
+ * margin covers probe scheduling so a supported terminal is never told it is
+ * incompatible while a probe is still in flight.
  */
-export const PET_CAPABILITY_SETTLE_MS = 1_000;
+export const PET_CAPABILITY_SETTLE_MS = 1_500;
 
 /**
- * Whether the asynchronous startup Sixel capability probe may still enable
- * graphics for this session, meaning current unavailability is not final.
+ * Whether an asynchronous startup capability probe (Sixel, or Kitty through
+ * tmux passthrough) may still enable graphics for this session, meaning current
+ * unavailability is not final.
  */
 export function isPetCapabilityProbePending(
 	env: NodeJS.ProcessEnv = Bun.env,
 	platform: NodeJS.Platform = process.platform,
 ): boolean {
-	if (TERMINAL.imageProtocol !== null) return false;
+	if (getPetPixelProtocol() !== null) return false;
 	if (!process.stdin.isTTY || !process.stdout.isTTY) return false;
-	return shouldProbeSixelCapability(env, platform);
+	return shouldProbeSixelCapability(env, platform) || shouldProbeKittyPassthrough(env);
 }
 
 /**

package/src/modes/components/pet-selector.ts

@@ -13,7 +13,7 @@ const PET_OPTIONS: { value: PetMode; label: string; description: string }[] = [
 ];
 
 /**
- * Theme-style picker for the sayknow pet skin (Off / RedSayknow / BlueSayknow). Preview
+ * Theme-style picker for the sayknow pet skin (Off / RedOctopus / BlueOctopus). Preview
  * fires as the selection moves; select commits, cancel restores.
  */
 export class PetSelectorComponent extends Container {

package/src/modes/components/sayknow-pet-widget.ts

@@ -4,6 +4,7 @@ import {
 	type Component,
 	type Container,
 	getCellDimensions,
+	getTmuxOverlayImageProtocol,
 	isUnderTmux,
 	PARA_PARA_STEPS,
 	PET_SKINS,
@@ -14,8 +15,10 @@ import {
 	registerAnimationCallback,
 	type SayknowPixelFrameName,
 	type SayknowPixelFrames,
+	TERMINAL,
 	type TUI,
 	tmuxOwnsSixel,
+	tmuxPaneOffset,
 	wrapTmuxPassthrough,
 } from "@sayknow-cli/tui";
 import type { CustomEditor } from "./custom-editor";
@@ -216,7 +219,7 @@ export class SayknowPetWidget {
 
 	/** Live preview during a selector: change the sprite without re-mounting the
 	 *  composer editor (that would tear down the open overlay). After a short idle
-	 *  eye-roll it fires the signature burst once (RedSayknow flex, BlueSayknow para-para
+	 *  eye-roll it fires the signature burst once (RedOctopus flex, BlueOctopus para-para
 	 *  then sob) so the selector demos the animation instead of waiting the random gap. */
 	previewMode(mode: PetMode): void {
 		this.#applyMode(mode, false);
@@ -341,7 +344,7 @@ export class SayknowPetWidget {
 		const mode = this.#mode;
 		if (mode === "off") return "base";
 		// Random idle burst → the skin's own animation, driven by its burst descriptor
-		// (RedSayknow holds a flex; BlueSayknow dances the para-para then sobs).
+		// (RedOctopus holds a flex; BlueOctopus dances the para-para then sobs).
 		if (now < this.#flexUntil) {
 			const burst = PET_SKINS[mode].burst;
 			const elapsed = now - (this.#flexUntil - petBurstDurationMs(burst));
@@ -384,7 +387,7 @@ export class SayknowPetWidget {
 			}
 		}
 		// Random show-off, both while idle and while working. Each skin's burst runs for
-		// its own length (RedSayknow a brief flex; BlueSayknow a para-para cycle plus sob).
+		// its own length (RedOctopus a brief flex; BlueOctopus a para-para cycle plus sob).
 		if (this.#autoFlexGapMs && now >= this.#flexUntil) {
 			if (this.#nextAutoFlexAt === 0) {
 				this.#scheduleAutoFlex(now);
@@ -429,11 +432,35 @@ export class SayknowPetWidget {
 		return out;
 	}
 
+	/**
+	 * Whether pet art must be smuggled to the outer terminal through tmux's DCS
+	 * passthrough envelope. True exactly when the multiplexer suppressed inline
+	 * graphics and the startup probe proved the outer terminal can draw them.
+	 */
+	#usesTmuxPassthrough(): boolean {
+		return TERMINAL.imageProtocol === null && getTmuxOverlayImageProtocol() !== null;
+	}
+
+	/**
+	 * Cursor-position escape for the pet's top-left cell.
+	 *
+	 * Passthrough payloads address the OUTER terminal's screen, so they carry the
+	 * pane origin; tmux-level payloads stay pane-relative.
+	 */
+	#cursorTo(x: number, y: number, outer: boolean): string {
+		const offset = outer ? tmuxPaneOffset() : { top: 0, left: 0 };
+		return `\x1b[${y + offset.top + 1};${x + offset.left + 1}H`;
+	}
+
 	/** Pending on-screen image cleanup. Pure: authority is consumed separately, on delivery. */
 	#imageCleanupPayload(): string {
 		let out = "";
 		if (this.#kittyCleanupPending && this.#kittyImageId !== undefined) {
-			out += `\x1b_Ga=d,d=I,i=${this.#kittyImageId},q=2\x1b\\`;
+			// Deleting by image id needs no cursor addressing, but it still has to
+			// reach the terminal that holds the placement — the outer one when the
+			// art was delivered through passthrough.
+			const del = `\x1b_Ga=d,d=I,i=${this.#kittyImageId},q=2\x1b\\`;
+			out += this.#usesTmuxPassthrough() ? wrapTmuxPassthrough(del) : del;
 		}
 		if (this.#lastSixelFootprint) {
 			out += this.#clearSixelFootprint(this.#lastSixelFootprint);
@@ -491,8 +518,10 @@ export class SayknowPetWidget {
 		// A full frame supersedes any cleanup-only frame still awaiting ack.
 		this.#frameCleanupAwaitingAck = false;
 		const { x, y } = pos;
+		const frame = pixel.frames[this.#frame];
 		let out = "";
-		const positioned = `\x1b[${y + 1};${x + 1}H${pixel.frames[this.#frame]}`;
+		const positioned = `${this.#cursorTo(x, y, false)}${frame}`;
+		const positionedOuter = `${this.#cursorTo(x, y, true)}${frame}`;
 
 		if (pixel.protocol === "sixel") {
 			const footprint = { x, y, columns: pixel.columns, rows: pixel.rasterRows };
@@ -518,7 +547,7 @@ export class SayknowPetWidget {
 			out += tmuxOwnsSixel()
 				? positioned
 				: isUnderTmux()
-					? wrapTmuxPassthrough(`\x1b7${positioned}\x1b8`)
+					? wrapTmuxPassthrough(`\x1b7${positionedOuter}\x1b8`)
 					: positioned;
 			return out;
 		}
@@ -526,7 +555,12 @@ export class SayknowPetWidget {
 		// A kitty frame (re)places the image cursor-neutrally, so cleanup is pending
 		// again even if a narrow-terminal pass consumed it earlier.
 		this.#kittyCleanupPending = true;
-		out += positioned;
+		// Under tmux the placement is cursor-neutral for the OUTER terminal but its
+		// addressing is not: tmux forwards passthrough bytes without first syncing
+		// the physical cursor, so the frame carries its own outer-screen position,
+		// and the save/restore pair sits inside the envelope so the outer cursor is
+		// exactly where tmux believes it left it.
+		out += this.#usesTmuxPassthrough() ? wrapTmuxPassthrough(`\x1b7${positionedOuter}\x1b8`) : positioned;
 		return out;
 	}
 }

package/src/modes/components/welcome.ts

@@ -25,6 +25,7 @@ export interface WelcomeComponentOptions {
 	collapseChangelog?: boolean;
 	buildLabel?: string;
 	keyDisplayContext?: KeyDisplayContext;
+	skipLogoAnimation?: boolean;
 }
 
 const WELCOME_STATIC_RIGHT_ROWS_EXCLUDING_DYNAMIC_SECTIONS = 15;
@@ -75,6 +76,10 @@ export class WelcomeComponent implements Component {
 	 */
 	playIntro(requestRender: () => void): void {
 		this.#stopAnimation();
+		if (this.options.skipLogoAnimation) {
+			requestRender();
+			return;
+		}
 		this.#animStart = performance.now();
 		requestRender();
 		this.#animTimer = setInterval(() => {

package/src/modes/controllers/event-controller.ts

@@ -869,7 +869,7 @@ export class EventController {
 				(content: { type: string; text?: string }) => content.type === "text",
 			)?.text;
 			this.ctx.showWarning(
-				`Todo update failed${textContent ? `: ${textContent}` : ". Progress may be stale until todo_write succeeds."}`,
+				`Todo update failed${textContent ? `: ${textContent}` : ""}. Work continues without visible todo progress for this turn.`,
 			);
 		}
 		if (event.toolName === "resolve" && !event.isError) {

package/src/modes/controllers/goal-mode-controller.ts

@@ -264,12 +264,15 @@ export class GoalModeController {
 			if (this.#enabled || this.#paused) this.ctx.modeGate.enter("goal");
 			if (restored?.goal) {
 				this.#previousTools = this.ctx.session.getActiveToolNames();
-				await this.ctx.session.setActiveToolsByName([...new Set([...this.#previousTools, "goal"])]);
+				if (!this.ctx.session.hasExplicitlyDisabledTools()) {
+					await this.ctx.session.setActiveToolsByName([...new Set([...this.#previousTools, "goal"])]);
+				}
 			}
 			this.#updateStatus();
 			return true;
 		}
 		if (recoveryHydration) return false;
+		if (this.ctx.session.hasExplicitlyDisabledTools()) return false;
 		const pendingGoal = goalEnabled
 			? await consumePendingGoalModeRequest(this.ctx.sessionManager.getCwd(), this.ctx.sessionManager.getSessionId())
 			: null;
@@ -321,7 +324,9 @@ export class GoalModeController {
 					objective: options.objective ?? "",
 					provenance: options.provenance,
 				});
-		await this.ctx.session.setActiveToolsByName([...new Set([...this.#previousTools, "goal"])]);
+		if (!this.ctx.session.hasExplicitlyDisabledTools()) {
+			await this.ctx.session.setActiveToolsByName([...new Set([...this.#previousTools, "goal"])]);
+		}
 		this.ctx.session.setGoalModeState(state);
 		this.#enabled = true;
 		this.#resetContinuationSuppression();

package/src/modes/controllers/input-controller.ts

@@ -91,6 +91,7 @@ export class InputController {
 			"app.model.select": () => this.ctx.showModelSelector(),
 			"app.model.selectTemporary": () => this.ctx.showModelSelector({ temporaryOnly: true }),
 			"app.tools.expand": () => this.toggleToolOutputExpansion(),
+			"app.todo.toggle": () => this.ctx.toggleTodoExpansion(),
 			"app.tool.backgroundFold": () => {
 				this.handleForegroundToolBackgroundFold();
 			},
@@ -103,6 +104,7 @@ export class InputController {
 			},
 			"app.clipboard.copyLine": () => this.handleCopyCurrentLine(),
 			"app.clipboard.copyPrompt": () => this.handleCopyPrompt(),
+			"app.oauth.copyUrl": () => this.ctx.copyOAuthAuthorizationUrl(),
 			"app.session.new": async () => {
 				await this.ctx.handleClearCommand();
 			},
@@ -153,6 +155,8 @@ export class InputController {
 				return this.ctx.session.getRoleModelCycleCandidateCount() > 1;
 			case "app.tools.expand":
 				return this.ctx.chatContainer.children.some(isExpandable);
+			case "app.todo.toggle":
+				return this.ctx.todoPhases.some(phase => phase.tasks.length > 0);
 			case "app.tool.backgroundFold":
 				return Boolean(this.ctx.session.hasForegroundBashBackgroundRequestHandler?.());
 			case "app.editor.external":
@@ -178,6 +182,8 @@ export class InputController {
 				);
 			case "app.clipboard.copyPrompt":
 				return this.ctx.editor.getText().length > 0;
+			case "app.oauth.copyUrl":
+				return this.ctx.hasOAuthAuthorizationUrl();
 			case "app.session.tree":
 			case "app.session.fork":
 				return this.ctx.session.messages.length > 0;
@@ -258,14 +264,20 @@ export class InputController {
 	 *  so abort cleanup going idle cannot turn the second Esc into an idle action. */
 	#steerConsumePending = false;
 	#commandPaletteActions = new Map<AppKeybinding, CommandPaletteAction>();
+	readonly #availabilityGatedPaletteActions = new Set<AppKeybinding>();
 	#paletteCommandInFlight = false;
 
-	#registerCommandPaletteAction(action: AppKeybinding, handler: () => void | Promise<void>): void {
+	#registerCommandPaletteAction(
+		action: AppKeybinding,
+		handler: () => void | Promise<void>,
+		availabilityGated = false,
+	): void {
 		this.#commandPaletteActions.set(action, {
 			id: action,
 			label: KEYBINDINGS[action].description,
 			handler,
 		});
+		if (availabilityGated) this.#availabilityGatedPaletteActions.add(action);
 	}
 
 	#globalInterruptUnsubscribe: (() => void) | undefined;
@@ -623,14 +635,12 @@ export class InputController {
 		this.ctx.editor.setActionKeys("app.tools.expand", this.ctx.keybindings.getKeys("app.tools.expand"));
 		this.ctx.editor.onExpandTools = () => this.#executeAction("app.tools.expand");
 		this.#registerCommandPaletteAction("app.tools.expand", expandTools);
-		const dequeue = () => this.handleDequeue();
 		this.ctx.editor.setActionKeys("app.message.dequeue", this.ctx.keybindings.getKeys("app.message.dequeue"));
 		this.ctx.editor.onDequeue = () => this.#executeAction("app.message.dequeue");
-		this.#registerCommandPaletteAction("app.message.dequeue", dequeue);
-		const queue = () => this.handleQueueSubmit();
+		this.#registerCommandPaletteAction("app.message.dequeue", () => this.#executeAction("app.message.dequeue"), true);
 		this.ctx.editor.setActionKeys("app.message.queue", this.ctx.keybindings.getKeys("app.message.queue"));
 		this.ctx.editor.onQueue = () => this.#executeAction("app.message.queue");
-		this.#registerCommandPaletteAction("app.message.queue", queue);
+		this.#registerCommandPaletteAction("app.message.queue", () => this.#executeAction("app.message.queue"), true);
 
 		this.ctx.editor.onViewportPageScroll = direction => this.ctx.ui.scrollViewportPages(direction);
 		this.ctx.editor.onViewportFollowLive = () => {
@@ -699,6 +709,11 @@ export class InputController {
 				return true;
 			});
 		}
+		this.#registerCommandPaletteAction(
+			"app.message.followUp",
+			() => this.#executeAction("app.message.followUp"),
+			true,
+		);
 		for (const key of this.ctx.keybindings.getKeys("app.message.followUp")) {
 			this.ctx.editor.setCustomKeyHandler(key, () => {
 				if (!this.actionRegistry.isAvailable("app.message.followUp")) return false;
@@ -706,6 +721,46 @@ export class InputController {
 				return true;
 			});
 		}
+		this.#registerCommandPaletteAction(
+			"app.transcript.prevTurn",
+			() => this.#executeAction("app.transcript.prevTurn"),
+			true,
+		);
+		for (const key of this.ctx.keybindings.getKeys("app.transcript.prevTurn")) {
+			this.ctx.editor.setCustomKeyHandler(key, () => {
+				if (!this.actionRegistry.isAvailable("app.transcript.prevTurn")) return false;
+				this.#executeAction("app.transcript.prevTurn");
+				return true;
+			});
+		}
+		this.#registerCommandPaletteAction(
+			"app.transcript.nextTurn",
+			() => this.#executeAction("app.transcript.nextTurn"),
+			true,
+		);
+		for (const key of this.ctx.keybindings.getKeys("app.transcript.nextTurn")) {
+			this.ctx.editor.setCustomKeyHandler(key, () => {
+				if (!this.actionRegistry.isAvailable("app.transcript.nextTurn")) return false;
+				this.#executeAction("app.transcript.nextTurn");
+				return true;
+			});
+		}
+		this.#registerCommandPaletteAction("app.message.sendNow", () => this.#executeAction("app.message.sendNow"), true);
+		for (const key of this.ctx.keybindings.getKeys("app.message.sendNow")) {
+			this.ctx.editor.setCustomKeyHandler(key, () => {
+				if (!this.actionRegistry.isAvailable("app.message.sendNow")) return false;
+				this.#executeAction("app.message.sendNow");
+				return true;
+			});
+		}
+		this.#registerCommandPaletteAction("app.todo.toggle", () => this.#executeAction("app.todo.toggle"), true);
+		for (const key of this.ctx.keybindings.getKeys("app.todo.toggle")) {
+			this.ctx.editor.setCustomKeyHandler(key, () => {
+				if (!this.actionRegistry.isAvailable("app.todo.toggle")) return false;
+				this.#executeAction("app.todo.toggle");
+				return true;
+			});
+		}
 		for (const key of this.ctx.keybindings.getKeys("app.stt.toggle")) {
 			this.ctx.editor.setCustomKeyHandler(key, () => {
 				this.#executeAction("app.stt.toggle");
@@ -717,6 +772,14 @@ export class InputController {
 				this.#executeAction("app.clipboard.copyLine");
 			});
 		}
+		this.#registerCommandPaletteAction("app.oauth.copyUrl", () => this.#executeAction("app.oauth.copyUrl"), true);
+		for (const key of this.ctx.keybindings.getKeys("app.oauth.copyUrl")) {
+			this.ctx.editor.setCustomKeyHandler(key, () => {
+				if (!this.actionRegistry.isAvailable("app.oauth.copyUrl")) return false;
+				this.#executeAction("app.oauth.copyUrl");
+				return true;
+			});
+		}
 		for (const key of this.ctx.keybindings.getKeys("app.session.observe")) {
 			this.ctx.editor.setCustomKeyHandler(key, () => {
 				this.#executeAction("app.session.observe");
@@ -1900,7 +1963,9 @@ export class InputController {
 			return;
 		}
 
-		const actions = [...this.#commandPaletteActions.values()];
+		const actions = [...this.#commandPaletteActions.entries()]
+			.filter(([id]) => !this.#availabilityGatedPaletteActions.has(id) || this.actionRegistry.isAvailable(id))
+			.map(([, action]) => action);
 		const slashCommands = [...(this.ctx.getSlashCommands?.() ?? this.#slashCommands)];
 		if (!this.ctx.showCommandPalette) {
 			let overlayHandle: ReturnType<typeof this.ctx.ui.showOverlay> | undefined;

package/src/modes/controllers/plan-mode-controller.ts

@@ -103,6 +103,11 @@ export class PlanModeController {
 				this.ctx.sessionManager.appendModeChange("none");
 			return;
 		}
+		if (this.ctx.session.hasExplicitlyDisabledTools()) {
+			if (sessionContext.mode === "plan" || sessionContext.mode === "plan_paused")
+				this.ctx.sessionManager.appendModeChange("none");
+			return;
+		}
 		if (sessionContext.mode === "plan") {
 			await this.enter({ planFilePath: sessionContext.modeData?.planFilePath as string | undefined });
 		} else if (sessionContext.mode === "plan_paused") {
@@ -115,6 +120,8 @@ export class PlanModeController {
 
 	async enter(options?: { planFilePath?: string; workflow?: "parallel" | "iterative" }): Promise<void> {
 		if (this.#enabled) return;
+		if (this.ctx.session.hasExplicitlyDisabledTools())
+			return this.ctx.showWarning("Plan mode requires the resolve tool.");
 		if (!this.ctx.modeGate.enter("plan")) return this.ctx.showWarning("Exit goal mode first.");
 		this.#paused = false;
 		const planFilePath = options?.planFilePath ?? "local://PLAN.md";
@@ -143,7 +150,7 @@ export class PlanModeController {
 	async exit(options?: { silent?: boolean; paused?: boolean }): Promise<void> {
 		if (!this.#enabled) return;
 		await this.ctx.session.abort({ timeoutMs: ABORT_TIMEOUT_MS });
-		if (this.#previousTools?.length) await this.ctx.session.setActiveToolsByName(this.#previousTools);
+		if (this.#previousTools !== undefined) await this.ctx.session.setActiveToolsByName(this.#previousTools);
 		if (this.#providerSessionScope && !this.ctx.session.isStreaming) {
 			if (this.ctx.session.restoreTemporaryProviderSessionScope(this.#providerSessionScope))
 				this.#providerSessionScope = undefined;

package/src/modes/controllers/runtime-mcp-command-controller.ts

@@ -540,6 +540,7 @@ export class MCPCommandController {
 
 		const resolvedClientId = clientId.trim() || parsedAuthUrl.searchParams.get("client_id") || undefined;
 		const resolvedClientSecret = clientSecret.trim() || undefined;
+		let authorizationUrlLease: number | undefined;
 
 		try {
 			// Create OAuth flow
@@ -556,6 +557,7 @@ export class MCPCommandController {
 				},
 				{
 					onAuth: (info: { url: string; instructions?: string }) => {
+						authorizationUrlLease = this.ctx.setOAuthAuthorizationUrl(info.url);
 						// Show auth URL prominently in chat
 						this.ctx.chatContainer.addChild(new Spacer(1));
 						this.ctx.chatContainer.addChild(
@@ -565,6 +567,16 @@ export class MCPCommandController {
 						this.ctx.chatContainer.addChild(
 							new Text(theme.fg("muted", "Preparing browser authorization..."), 1, 0),
 						);
+						this.ctx.chatContainer.addChild(
+							new Text(
+								theme.fg(
+									"muted",
+									`Press ${this.ctx.keybindings.getDisplayString("app.oauth.copyUrl")} to copy the authorization URL.`,
+								),
+								1,
+								0,
+							),
+						);
 						this.ctx.chatContainer.addChild(new Spacer(1));
 						this.ctx.chatContainer.addChild(
 							new Text(
@@ -661,6 +673,8 @@ export class MCPCommandController {
 			} else {
 				throw new Error(`OAuth authentication failed: ${errorMsg}`);
 			}
+		} finally {
+			if (authorizationUrlLease !== undefined) this.ctx.clearOAuthAuthorizationUrl(authorizationUrlLease);
 		}
 	}