@sayknow-cli/coding-agent 0.3.1 → 0.3.4

package/src/modes/controllers/extension-ui-controller.ts

@@ -26,8 +26,6 @@ import type { InteractiveModeContext } from "../../modes/types";
 import { setSessionTerminalTitle, setTerminalTitle } from "../../utils/title-generator";
 
 const MAX_WIDGET_LINES = 10;
-const HOOK_SELECTOR_MOUSE_REPORTING_ENABLE = "\x1b[?1006h\x1b[?1000h";
-const HOOK_SELECTOR_MOUSE_REPORTING_DISABLE = "\x1b[?1000l\x1b[?1006l";
 const HOOK_SELECTOR_CHROME_ROWS = 7;
 const HOOK_SELECTOR_OUTLINE_ROWS = 2;
 const HOOK_SELECTOR_INLINE_INPUT_ROWS = 2;
@@ -36,7 +34,6 @@ export class ExtensionUiController {
 	#extensionTerminalInputUnsubscribers = new Set<() => void>();
 	#hookWidgetsAbove = new Map<string, ExtensionUiComponent>();
 	#hookWidgetsBelow = new Map<string, ExtensionUiComponent>();
-	#hookSelectorMouseReportingEnabled = false;
 	#activeHookCustomComponent?: Component & { dispose?(): void };
 	#activeHookCustomOverlay?: OverlayHandle;
 
@@ -625,9 +622,6 @@ export class ExtensionUiController {
 			this.ctx.ui.terminal.rows - scrollOptionRows - listChromeRows - inlineInputRows - HOOK_SELECTOR_CHROME_ROWS;
 		const scrollTitleRows =
 			requestedTitleRows === undefined ? undefined : Math.max(1, Math.min(requestedTitleRows, availableTitleRows));
-		if (scrollTitleRows !== undefined) {
-			this.#enableHookSelectorMouseReporting();
-		}
 
 		this.ctx.hookSelector = new HookSelectorComponent(
 			title,
@@ -692,31 +686,10 @@ export class ExtensionUiController {
 		return promise;
 	}
 
-	#enableHookSelectorMouseReporting(): void {
-		if (this.#hookSelectorMouseReportingEnabled) return;
-		this.#hookSelectorMouseReportingEnabled = true;
-		this.#writeTerminalControl(HOOK_SELECTOR_MOUSE_REPORTING_ENABLE);
-	}
-
-	#disableHookSelectorMouseReporting(): void {
-		if (!this.#hookSelectorMouseReportingEnabled) return;
-		this.#hookSelectorMouseReportingEnabled = false;
-		this.#writeTerminalControl(HOOK_SELECTOR_MOUSE_REPORTING_DISABLE);
-	}
-
-	#writeTerminalControl(sequence: string): void {
-		try {
-			this.ctx.ui.terminal.write(sequence);
-		} catch {
-			// Terminal teardown can race selector cleanup; normal shutdown restores modes.
-		}
-	}
-
 	/**
 	 * Hide the hook selector.
 	 */
 	hideHookSelector(): void {
-		this.#disableHookSelectorMouseReporting();
 		this.ctx.hookSelector?.dispose();
 		this.ctx.editorContainer.clear();
 		this.ctx.editorContainer.addChild(this.ctx.editor);

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

@@ -4,8 +4,10 @@ import type { OAuthProvider } from "@sayknow-cli/ai/utils/oauth/types";
 import type { Component, OverlayHandle } from "@sayknow-cli/tui";
 import { Input, Loader, Spacer, Text } from "@sayknow-cli/tui";
 import { getAgentDbPath, getProjectDir } from "@sayknow-cli/utils";
-import { activateModelProfile } from "../../config/model-profile-activation";
+import { activateModelProfile, materializeActiveModelProfileAssignment } from "../../config/model-profile-activation";
 import { recommendModelProfileForProvider } from "../../config/model-profiles";
+import { SKC_MODEL_ASSIGNMENT_TARGETS } from "../../config/model-registry";
+import { formatModelSelectorValue } from "../../config/model-resolver";
 import { settings } from "../../config/settings";
 import { DebugSelectorComponent } from "../../debug";
 import { disableProvider, enableProvider } from "../../discovery";
@@ -49,6 +51,7 @@ import {
 	setPreferredImageProvider,
 	setPreferredSearchProvider,
 	setSearchFallbackProviders,
+	setSearchHardTimeoutMs,
 } from "../../tools";
 import { setSessionTerminalTitle } from "../../utils/title-generator";
 import { AgentDashboard } from "../components/agent-dashboard";
@@ -643,6 +646,11 @@ export class SelectorController {
 					);
 				}
 				break;
+			case "web_search.timeout":
+				if (typeof value === "number" && Number.isFinite(value) && value > 0) {
+					setSearchHardTimeoutMs(value * 1000);
+				}
+				break;
 			case "providers.image":
 				if (
 					value === "auto" ||
@@ -667,7 +675,8 @@ export class SelectorController {
 
 	showModelSelector(options?: { temporaryOnly?: boolean }): void {
 		this.showSelector(done => {
-			const selector = new ModelSelectorComponent(
+			let modelSelector: ModelSelectorComponent;
+			modelSelector = new ModelSelectorComponent(
 				this.ctx.ui,
 				this.ctx.session.model,
 				this.ctx.settings,
@@ -717,38 +726,73 @@ export class SelectorController {
 							this.ctx.ui.requestRender();
 							return;
 						}
-						const { model, role, thinkingLevel, selector } = selection;
+						const { model, role, thinkingLevel, selector: selectedSelector } = selection;
 						if (role === null) {
 							// Temporary: update agent state but don't persist to settings
 							await this.ctx.session.setModelTemporary(model, thinkingLevel);
 							this.ctx.statusLine.invalidate();
 							this.ctx.updateEditorBorderColor();
-							this.ctx.showStatus(`Temporary model: ${selector ?? model.id}`);
+							this.ctx.showStatus(`Temporary model: ${selectedSelector ?? model.id}`);
 							done();
 							this.ctx.ui.requestRender();
 						} else if (role === "default") {
 							// Default: update agent state and persist as the active default model.
 							await this.ctx.session.setModel(model, role, {
-								selector,
+								selector: selectedSelector,
+								thinkingLevel,
+							});
+							const value = formatModelSelectorValue(
+								selectedSelector ?? `${model.provider}/${model.id}`,
 								thinkingLevel,
+							);
+							materializeActiveModelProfileAssignment({
+								session: this.ctx.session,
+								settings: this.ctx.settings,
+								role,
+								selector: value,
 							});
 							if (thinkingLevel && thinkingLevel !== ThinkingLevel.Inherit) {
 								this.ctx.session.setThinkingLevel(thinkingLevel);
 							}
+							modelSelector.refreshRoleAssignments({
+								currentModel: this.ctx.session.model,
+								currentThinkingLevel: this.ctx.session.thinkingLevel,
+								activeModelProfile:
+									this.ctx.session.getActiveModelProfile?.() ?? this.ctx.settings.get("modelProfile.default"),
+							});
 							this.ctx.statusLine.invalidate();
 							this.ctx.updateEditorBorderColor();
-							this.ctx.showStatus(`Default model: ${selector ?? model.id}`);
+							this.ctx.showStatus(`Default model: ${selectedSelector ?? model.id}`);
 							done();
 							this.ctx.ui.requestRender();
 						} else {
-							// Role-agent assignments configure Task dispatch and must not switch the active chat model.
 							const apiKey = await this.ctx.session.modelRegistry.getApiKey(model, this.ctx.session.sessionId);
 							if (!apiKey) {
 								throw new Error(`No API key for ${model.provider}/${model.id}`);
 							}
-							const overrides = this.ctx.settings.get("task.agentModelOverrides");
-							const value = selector ?? `${model.provider}/${model.id}`;
-							this.ctx.settings.set("task.agentModelOverrides", { ...overrides, [role]: value });
+							const value =
+								selectedSelector ?? formatModelSelectorValue(`${model.provider}/${model.id}`, thinkingLevel);
+							const materializedProfile = materializeActiveModelProfileAssignment({
+								session: this.ctx.session,
+								settings: this.ctx.settings,
+								role,
+								selector: value,
+							});
+							if (!materializedProfile) {
+								const target = SKC_MODEL_ASSIGNMENT_TARGETS[role];
+								if (target.settingsPath === "modelRoles") {
+									this.ctx.settings.setModelRole(role, value);
+								} else {
+									const overrides = this.ctx.settings.get("task.agentModelOverrides");
+									this.ctx.settings.set("task.agentModelOverrides", { ...overrides, [role]: value });
+								}
+							}
+							modelSelector.refreshRoleAssignments({
+								currentModel: this.ctx.session.model,
+								currentThinkingLevel: this.ctx.session.thinkingLevel,
+								activeModelProfile:
+									this.ctx.session.getActiveModelProfile?.() ?? this.ctx.settings.get("modelProfile.default"),
+							});
 							this.ctx.settings.getStorage()?.recordModelUsage(`${model.provider}/${model.id}`);
 							this.ctx.showStatus(`${role} agent model: ${value}`);
 							done();
@@ -765,11 +809,15 @@ export class SelectorController {
 				{
 					...options,
 					sessionId: this.ctx.session.sessionId,
+					currentThinkingLevel: this.ctx.session.thinkingLevel,
+					activeModelProfile:
+						this.ctx.session.getActiveModelProfile?.() ?? this.ctx.settings.get("modelProfile.default"),
 					isFastForProvider: provider => this.ctx.session.isFastForProvider(provider),
 					isFastForSubagentProvider: provider => this.ctx.session.isFastForSubagentProvider(provider),
+					isCurrentModelFastModeActive: () => this.ctx.session.isFastModeActive(),
 				},
 			);
-			return { component: selector, focus: selector };
+			return { component: modelSelector, focus: modelSelector };
 		});
 	}
 

package/src/modes/interactive-mode.ts

@@ -156,7 +156,7 @@ function getShellInputPrefix(isNoContext: boolean): string {
 
 function configureDefaultComposerChrome(editor: CustomEditor): void {
 	editor.setBorderVisible(true);
-	editor.setBorderStyle("sharp");
+	editor.setBorderStyle("round");
 	editor.setClosedBorderBox(true);
 	editor.setPromptGutter(undefined);
 	editor.setInputPrefix(getDefaultInputPrefix());
@@ -432,6 +432,8 @@ export class InteractiveMode implements InteractiveModeContext {
 		this.#resizeHandler = () => {
 			this.#syncEditorMaxHeight();
 			this.updateEditorChrome();
+			this.editor.invalidate();
+			this.ui.requestRender(true, "resize");
 		};
 		process.stdout.on("resize", this.#resizeHandler);
 		try {
@@ -569,6 +571,7 @@ export class InteractiveMode implements InteractiveModeContext {
 		this.ui.addChild(this.hookWidgetContainerAbove);
 		this.ui.addChild(this.editorContainer);
 		this.ui.addChild(this.hookWidgetContainerBelow);
+		this.ui.setBottomPinnedComponent(this.statusLine);
 		this.ui.setFocus(this.editor);
 
 		this.#inputController.setupKeyHandlers();

package/src/modes/shared/agent-wire/scopes.ts

@@ -73,7 +73,7 @@ const RPC_COMMAND_SCOPE_REGISTRY: Record<RpcCommandType, BridgeCommandScope> = {
 	get_login_providers: "admin",
 	login: "admin",
 	negotiate_unattended: "control",
-	workflow_gate_response: "prompt",
+	workflow_gate_response: "control",
 };
 
 export const RPC_COMMAND_TYPES: readonly RpcCommandType[] = Object.keys(RPC_COMMAND_SCOPE_REGISTRY) as RpcCommandType[];

package/src/modes/theme/defaults/gruvbox-dark.json

@@ -0,0 +1,99 @@
+{
+	"$schema": "https://raw.githubusercontent.com/jaybeyond/Sayknow_CLI/main/packages/coding-agent/theme-schema.json",
+	"name": "gruvbox-dark",
+	"vars": {
+		"bgHard": "#1d2021",
+		"surface": "#32302f",
+		"surfaceBright": "#504945",
+		"borderNeutral": "#504945",
+		"borderSubtle": "#3c3836",
+		"fg": "#ebdbb2",
+		"muted": "#a89984",
+		"dim": "#7c6f64",
+		"gray": "#928374",
+		"red": "#fb4934",
+		"green": "#b8bb26",
+		"yellow": "#fabd2f",
+		"blue": "#83a598",
+		"purple": "#d3869b",
+		"aqua": "#8ec07c",
+		"orange": "#fe8019",
+		"diffRemovalRed": "#cc241d"
+	},
+	"colors": {
+		"accent": "orange",
+		"border": "borderNeutral",
+		"borderAccent": "orange",
+		"borderMuted": "borderSubtle",
+		"success": "green",
+		"error": "red",
+		"warning": "yellow",
+		"muted": "muted",
+		"dim": "dim",
+		"text": "fg",
+		"thinkingText": "muted",
+		"selectedBg": "surfaceBright",
+		"userMessageBg": "surface",
+		"userMessageText": "fg",
+		"customMessageBg": "surface",
+		"customMessageText": "fg",
+		"customMessageLabel": "orange",
+		"toolPendingBg": "surface",
+		"toolSuccessBg": "#283626",
+		"toolErrorBg": "#3c2323",
+		"toolTitle": "fg",
+		"toolOutput": "muted",
+		"mdHeading": "yellow",
+		"mdLink": "blue",
+		"mdLinkUrl": "muted",
+		"mdCode": "aqua",
+		"mdCodeBlock": "fg",
+		"mdCodeBlockBorder": "borderNeutral",
+		"mdQuote": "muted",
+		"mdQuoteBorder": "borderNeutral",
+		"mdHr": "dim",
+		"mdListBullet": "orange",
+		"toolDiffAdded": "green",
+		"toolDiffRemoved": "diffRemovalRed",
+		"toolDiffContext": "muted",
+		"syntaxComment": "gray",
+		"syntaxKeyword": "red",
+		"syntaxFunction": "green",
+		"syntaxVariable": "blue",
+		"syntaxString": "green",
+		"syntaxNumber": "purple",
+		"syntaxType": "yellow",
+		"syntaxOperator": "aqua",
+		"syntaxPunctuation": "muted",
+		"thinkingOff": "dim",
+		"thinkingMinimal": "muted",
+		"thinkingLow": "aqua",
+		"thinkingMedium": "yellow",
+		"thinkingHigh": "orange",
+		"thinkingXhigh": "red",
+		"bashMode": "green",
+		"pythonMode": "yellow",
+		"statusLineBg": "bgHard",
+		"statusLineSep": "dim",
+		"statusLineModel": "orange",
+		"statusLinePath": "blue",
+		"statusLineGitClean": "green",
+		"statusLineGitDirty": "yellow",
+		"statusLineContext": "aqua",
+		"statusLineSpend": "yellow",
+		"statusLineStaged": "green",
+		"statusLineDirty": "yellow",
+		"statusLineUntracked": "diffRemovalRed",
+		"statusLineOutput": "fg",
+		"statusLineCost": "orange",
+		"statusLineSubagents": "purple"
+	},
+	"export": {
+		"pageBg": "#1d2021",
+		"cardBg": "#282828",
+		"infoBg": "#32302f"
+	},
+	"symbols": {
+		"preset": "unicode"
+	}
+}

package/src/modes/theme/defaults/index.ts

@@ -1,6 +1,7 @@
 import blue_octopus from "./blue-octopus.json" with { type: "json" };
 import claude_code from "./claude-code.json" with { type: "json" };
 import codex from "./codex.json" with { type: "json" };
+import gruvbox_dark from "./gruvbox-dark.json" with { type: "json" };
 import opencode from "./opencode.json" with { type: "json" };
 import red_octopus from "./red-octopus.json" with { type: "json" };
 
@@ -8,6 +9,7 @@ export const defaultThemes = {
 	"blue-octopus": blue_octopus,
 	"claude-code": claude_code,
 	codex,
+	"gruvbox-dark": gruvbox_dark,
 	opencode,
 	"red-octopus": red_octopus,
 };

package/src/modes/theme/theme.ts

@@ -183,7 +183,6 @@ export type SymbolKey =
 	| "tab.appearance"
 	| "tab.model"
 	| "tab.interaction"
-	| "tab.notifications"
 	| "tab.context"
 	| "tab.editing"
 	| "tab.tools"
@@ -348,7 +347,6 @@ const UNICODE_SYMBOLS: SymbolMap = {
 	"tab.appearance": "🎨",
 	"tab.model": "🤖",
 	"tab.interaction": "⌨",
-	"tab.notifications": "🔔",
 	"tab.context": "📋",
 	"tab.editing": "💻",
 	"tab.tools": "🔧",
@@ -608,7 +606,6 @@ const NERD_SYMBOLS: SymbolMap = {
 	"tab.appearance": "󰃣",
 	"tab.model": "󰚩",
 	"tab.interaction": "󰌌",
-	"tab.notifications": "󰂚",
 	"tab.context": "󰘸",
 	"tab.editing": "",
 	"tab.tools": "󰠭",
@@ -771,7 +768,6 @@ const ASCII_SYMBOLS: SymbolMap = {
 	"tab.appearance": "[A]",
 	"tab.model": "[M]",
 	"tab.interaction": "[I]",
-	"tab.notifications": "[N]",
 	"tab.context": "[X]",
 	"tab.editing": "[E]",
 	"tab.tools": "[T]",

package/src/modes/utils/hotkeys-markdown.ts

@@ -23,7 +23,7 @@ export function buildHotkeysMarkdown(bindings: HotkeysMarkdownBindings): string
 		"|-----|--------|",
 		"| `Enter` | Send / queue while busy |",
 		`| \`${appKey(bindings, "app.message.queue")}\` | Queue message for next turn |`,
-		"| `Shift+Enter` | New line |",
+		"| `Shift+Enter` / `Ctrl+J` | New line |",
 		"| `Ctrl+W` / `Option+Backspace` | Delete word backwards |",
 		"| `Ctrl+U` | Delete to start of line |",
 		"| `Ctrl+K` | Delete to end of line |",

package/src/notifications/html-format.ts

@@ -349,16 +349,54 @@ export function buttonLabel(label: string, index: number): string {
 	return `${index + 1}. ${stripped}`;
 }
 
+/** Numbered, escaped option list for the Telegram message body. */
+export function numberedOptionList(labels: string[]): string {
+	return labels.map((label, i) => `${i + 1}. ${escapeHtml(label.replace(/^\s*\d+[.)]\s+/, ""))}`).join("\n");
+}
+
+/** Compact numeric button label; full option text belongs in the message body. */
+export function choiceButtonLabel(index: number): string {
+	return String(index + 1);
+}
+
 export interface InlineButton {
 	text: string;
 	callback_data: string;
 }
 
+const COMPACT_BUTTONS_PER_ROW = 5;
+
 /** A prefixed button label is "long" when it is wide or contains a newline. */
 function isLongLabel(label: string): boolean {
 	return label.length > 18 || /[\r\n]/.test(label);
 }
 
+/**
+ * Lay out option callbacks as compact numeric buttons. Telegram mobile clients
+ * ellipsize long inline-keyboard labels and tall keyboards can be obscured by
+ * the composer, so the full choice text is rendered in the message body while
+ * the keyboard keeps only stable one-based tap targets.
+ */
+export function buildCompactChoiceGrid(
+	labels: string[],
+	callbackForIndex: (index: number) => string,
+): InlineButton[][] {
+	const rows: InlineButton[][] = [];
+	let run: InlineButton[] = [];
+	const flush = () => {
+		if (run.length) {
+			rows.push(run);
+			run = [];
+		}
+	};
+	labels.forEach((_label, i) => {
+		run.push({ text: choiceButtonLabel(i), callback_data: callbackForIndex(i) });
+		if (run.length === COMPACT_BUTTONS_PER_ROW) flush();
+	});
+	flush();
+	return rows;
+}
+
 /**
  * Lay out option labels as a numbered button grid. Long buttons take a
  * full-width row; runs of short buttons are packed into rows of up to 3. The