@sayknow-cli/coding-agent 0.6.4 → 0.6.6

package/src/slash-commands/builtin-registry.ts

@@ -51,6 +51,12 @@ import {
 } from "../setup/provider-onboarding";
 import { parseThinkingLevel } from "../thinking";
 import { getDisplayChangelogEntries } from "../utils/changelog";
+import {
+	beginSessionTitleGeneration,
+	invalidateSessionTitleGeneration,
+	isSessionTitleGenerationCurrent,
+} from "../utils/session-title-generation";
+import { buildConversationTitleInput, generateSessionTitle } from "../utils/title-generator";
 import { buildContextReportText } from "./helpers/context-report";
 import { buildFastStatusReport } from "./helpers/fast-status-report";
 import { formatDuration } from "./helpers/format";
@@ -113,6 +119,45 @@ function fastStatusRoleTargets(): Array<{ id: SkcModelAssignmentTargetId; label:
 	}));
 }
 
+async function regenerateSessionTitle(runtime: SlashCommandRuntime): Promise<SlashCommandResult> {
+	try {
+		const generation = beginSessionTitleGeneration(runtime.sessionManager);
+		const input = buildConversationTitleInput(runtime.session.messages);
+		if (!input) {
+			return usage("Nothing to summarize yet — pass a title: /rename <title>", runtime);
+		}
+
+		const generated = await generateSessionTitle(
+			input,
+			runtime.session.modelRegistry,
+			runtime.settings,
+			runtime.session.sessionId,
+			runtime.session.model,
+			provider => runtime.session.agent.metadataForProvider(provider),
+		);
+		// A newer prompt, message, or explicit rename landed while the model ran: drop this result.
+		if (
+			!isSessionTitleGenerationCurrent(runtime.sessionManager, generation) ||
+			buildConversationTitleInput(runtime.session.messages) !== input
+		)
+			return commandConsumed();
+		if (!generated) {
+			return usage("Could not generate a session title — pass one: /rename <title>", runtime);
+		}
+
+		const stored = await runtime.sessionManager.setSessionName(generated, "user");
+		if (!stored) {
+			await runtime.output("Session name not changed (a user-set name takes precedence).");
+			return commandConsumed();
+		}
+		await runtime.notifyTitleChanged?.();
+		await runtime.output(`Session renamed to ${generated}.`);
+		return commandConsumed();
+	} catch (err) {
+		return usage(`Rename failed: ${errorMessage(err)}`, runtime);
+	}
+}
+
 function toSlashCommandRuntime(runtime: TuiSlashCommandRuntime): SlashCommandRuntime {
 	const ctx = runtime.ctx;
 	return {
@@ -139,6 +184,29 @@ function toSlashCommandRuntime(runtime: TuiSlashCommandRuntime): SlashCommandRun
 	};
 }
 
+async function updateSessionStar(
+	starred: boolean,
+	command: ParsedSlashCommand,
+	runtime: SlashCommandRuntime,
+): Promise<SlashCommandResult> {
+	if (command.args) return usage(`Usage: /${starred ? "star" : "unstar"}`, runtime);
+	if (!runtime.sessionManager.isPersisted()) {
+		await runtime.output("Session stars require a persisted session.");
+		return commandConsumed();
+	}
+	const changed = await runtime.sessionManager.setSessionStarred(starred);
+	await runtime.output(
+		changed
+			? starred
+				? "Session starred."
+				: "Session unstarred."
+			: starred
+				? "Session is already starred."
+				: "Session is not starred.",
+	);
+	return commandConsumed();
+}
+
 function parseProviderSetupSlashArgs(args: string): {
 	preset?: string;
 	compat?: string;
@@ -1619,6 +1687,24 @@ const BUILTIN_SLASH_COMMAND_REGISTRY: ReadonlyArray<SlashCommandSpec> = [
 			await runtime.ctx.handleContributionPrepCommand(command.args || undefined);
 		},
 	},
+	{
+		name: "star",
+		priority: 91,
+		description: "Star the current session for easier discovery",
+		// Accept args at dispatch so the handler can consume them as a usage error
+		// instead of silently forwarding `/star ...` to the model.
+		allowArgs: true,
+		handle: (command, runtime) => updateSessionStar(true, command, runtime),
+	},
+	{
+		name: "unstar",
+		priority: 91,
+		description: "Remove the star from the current session",
+		// Accept args at dispatch so the handler can consume them as a usage error
+		// instead of silently forwarding `/unstar ...` to the model.
+		allowArgs: true,
+		handle: (command, runtime) => updateSessionStar(false, command, runtime),
+	},
 	{
 		name: "resume",
 		priority: 92,
@@ -1742,13 +1828,27 @@ const BUILTIN_SLASH_COMMAND_REGISTRY: ReadonlyArray<SlashCommandSpec> = [
 			await runtime.ctx.handleMemoryCommand(command.text);
 		},
 	},
+	{
+		name: "fork",
+		description: "Choose an earlier prompt to continue in a new session",
+		allowArgs: true,
+		handleTui: async (command, runtime) => {
+			runtime.ctx.editor.setText("");
+			if (command.args.trim()) {
+				runtime.ctx.showError("Usage: /fork");
+				return;
+			}
+			runtime.ctx.showUserMessageSelector();
+		},
+	},
 	{
 		name: "rename",
-		description: "Rename the current session",
-		inlineHint: "<title>",
+		description: "Rename the current session, or regenerate the title from the conversation",
+		inlineHint: "[title]",
 		allowArgs: true,
 		handle: async (command, runtime) => {
-			if (!command.args) return usage("Usage: /rename <title>", runtime);
+			if (!command.args) return regenerateSessionTitle(runtime);
+			invalidateSessionTitleGeneration(runtime.sessionManager);
 			const ok = await runtime.sessionManager.setSessionName(command.args, "user");
 			if (!ok) {
 				await runtime.output("Session name not changed (a user-set name takes precedence).");
@@ -1761,8 +1861,8 @@ const BUILTIN_SLASH_COMMAND_REGISTRY: ReadonlyArray<SlashCommandSpec> = [
 		handleTui: async (command, runtime) => {
 			const title = command.args.trim();
 			if (!title) {
-				runtime.ctx.showError("Usage: /rename <title>");
 				runtime.ctx.editor.setText("");
+				await runtime.ctx.handleRenameCommand();
 				return;
 			}
 			runtime.ctx.editor.setText("");

package/src/tools/debug.ts

@@ -593,7 +593,6 @@ export const debugToolRenderer = {
 							{ label: theme.fg("toolTitle", "Output"), lines: displayedLines },
 						],
 						width,
-						applyBg: false,
 					},
 					theme,
 				);

package/src/tools/fetch.ts

@@ -1546,7 +1546,6 @@ export function renderReadUrlResult(
 						{ label: uiTheme.fg("toolTitle", "Content Preview"), lines: contentPreviewLines },
 					],
 					width,
-					applyBg: false,
 				},
 				uiTheme,
 			);

package/src/tui/output-block.ts

@@ -1,5 +1,5 @@
 /**
- * Bordered output container with optional header and sections.
+ * Railed output container with optional header and sections.
  */
 import {
 	ImageProtocol,
@@ -13,7 +13,7 @@ import type { Theme } from "../modes/theme/theme";
 import { containsSixelSequence, getSixelLineMask } from "../utils/sixel";
 import type { State } from "./types";
 import type { RenderCache } from "./utils";
-import { getStateBgColor, Hasher, padToWidth, truncateToWidth } from "./utils";
+import { Hasher, padToWidth, truncateToWidth } from "./utils";
 
 export interface OutputBlockOptions {
 	header?: string;
@@ -21,17 +21,25 @@ export interface OutputBlockOptions {
 	state?: State;
 	sections?: Array<{ label?: string; lines: string[] }>;
 	width: number;
-	applyBg?: boolean;
 }
 
+/**
+ * Render a tool output block as an open rail: a header line, then content hung
+ * off a single left rule in the state color. There is no enclosing box and no
+ * filled background in any state; the rail color alone carries running/success/error.
+ *
+ * ```
+ * ✓ Bash · 0.4s
+ * │ $ bun test
+ * ├ Output
+ * │ 12 pass
+ * ```
+ */
 export function renderOutputBlock(options: OutputBlockOptions, theme: Theme): string[] {
-	const { header, headerMeta, state, sections = [], width, applyBg = true } = options;
-	const h = theme.boxSharp.horizontal;
-	const v = theme.boxSharp.vertical;
-	const cap = h.repeat(3);
+	const { header, headerMeta, state, sections = [], width } = options;
 	const lineWidth = Math.max(0, width);
-	// Border colors: running/pending use accent, success uses dim (gray), error/warning keep their colors
-	const borderColor: "error" | "warning" | "accent" | "dim" =
+	// Rail colors: running/pending use accent, success recedes to dim, error/warning keep their colors.
+	const railColor: "error" | "warning" | "accent" | "dim" =
 		state === "error"
 			? "error"
 			: state === "warning"
@@ -39,53 +47,19 @@ export function renderOutputBlock(options: OutputBlockOptions, theme: Theme): st
 				: state === "running" || state === "pending"
 					? "accent"
 					: "dim";
-	const border = (text: string) => theme.fg(borderColor, text);
-	const bgFn = (() => {
-		if (!state || !applyBg) return undefined;
-		const bgAnsi = theme.getBgAnsi(getStateBgColor(state));
-		// Keep block background stable even if inner content contains SGR resets (e.g. "\x1b[0m"),
-		// which would otherwise clear the outer background mid-line.
-		return (text: string) => {
-			const stabilized = text
-				.replace(/\x1b\[(?:0)?m/g, m => `${m}${bgAnsi}`)
-				.replace(/\x1b\[49m/g, m => `${m}${bgAnsi}`);
-			return `${bgAnsi}${stabilized}\x1b[49m`;
-		};
-	})();
-
-	const buildBarLine = (leftChar: string, rightChar: string, label?: string, meta?: string): string => {
-		const left = border(`${leftChar}${cap}`);
-		const right = border(rightChar);
-		if (lineWidth <= 0) return left + right;
-		const labelText = [label, meta].filter(Boolean).join(theme.sep.dot);
-		const rawLabel = labelText ? ` ${labelText} ` : " ";
-		const leftWidth = visibleWidth(left);
-		const rightWidth = visibleWidth(right);
-		const maxLabelWidth = Math.max(0, lineWidth - leftWidth - rightWidth);
-		const trimmedLabel = truncateToWidth(rawLabel, maxLabelWidth);
-		const labelWidth = visibleWidth(trimmedLabel);
-		const fillCount = Math.max(0, lineWidth - leftWidth - labelWidth - rightWidth);
-		return `${left}${trimmedLabel}${border(h.repeat(fillCount))}${right}`;
-	};
-
-	const contentPrefix = border(`${v} `);
-	const contentSuffix = border(v);
-	const contentWidth = Math.max(0, lineWidth - visibleWidth(contentPrefix) - visibleWidth(contentSuffix));
+	const rail = (text: string) => theme.fg(railColor, text);
 	const lines: string[] = [];
+	const labelText = [header, headerMeta].filter(Boolean).join(theme.sep.dot);
+	if (labelText) lines.push(padToWidth(truncateToWidth(labelText, lineWidth), lineWidth));
 
-	lines.push(
-		padToWidth(buildBarLine(theme.boxSharp.topLeft, theme.boxSharp.topRight, header, headerMeta), lineWidth, bgFn),
-	);
-
-	const hasSections = sections.length > 0;
-	const normalizedSections = hasSections ? sections : [{ lines: [] }];
+	const contentPrefix = rail(`${theme.boxSharp.vertical} `);
+	const contentWidth = Math.max(0, lineWidth - visibleWidth(contentPrefix));
 
-	for (let i = 0; i < normalizedSections.length; i++) {
-		const section = normalizedSections[i];
+	for (const section of sections) {
 		if (section.label) {
-			lines.push(
-				padToWidth(buildBarLine(theme.boxSharp.teeRight, theme.boxSharp.teeLeft, section.label), lineWidth, bgFn),
-			);
+			const tee = rail(`${theme.boxSharp.teeRight} `);
+			const label = truncateToWidth(theme.fg("dim", section.label), contentWidth);
+			lines.push(padToWidth(`${tee}${label}`, lineWidth));
 		}
 		const allLines = section.lines.flatMap(l => l.split("\n"));
 		const fallbackActive = isTerminalGraphicsFallbackActive();
@@ -102,18 +76,13 @@ export function renderOutputBlock(options: OutputBlockOptions, theme: Theme): st
 			const wrappedLines = wrapTextWithAnsi(line.trimEnd(), contentWidth);
 			for (const wrappedLine of wrappedLines) {
 				const innerPadding = padding(Math.max(0, contentWidth - visibleWidth(wrappedLine)));
-				const fullLine = `${contentPrefix}${wrappedLine}${innerPadding}${contentSuffix}`;
-				lines.push(padToWidth(fullLine, lineWidth, bgFn));
+				lines.push(padToWidth(`${contentPrefix}${wrappedLine}${innerPadding}`, lineWidth));
 			}
 		}
 	}
 
-	const bottomLeft = border(`${theme.boxSharp.bottomLeft}${cap}`);
-	const bottomRight = border(theme.boxSharp.bottomRight);
-	const bottomFillCount = Math.max(0, lineWidth - visibleWidth(bottomLeft) - visibleWidth(bottomRight));
-	const bottomLine = `${bottomLeft}${border(h.repeat(bottomFillCount))}${bottomRight}`;
-	lines.push(padToWidth(bottomLine, lineWidth, bgFn));
-
+	// A block with neither header nor content still marks its place with a bare rail row.
+	if (lines.length === 0) lines.push(padToWidth(rail(theme.boxSharp.vertical), lineWidth));
 	return lines;
 }
 
@@ -147,7 +116,6 @@ export class CachedOutputBlock {
 		h.optional(options.header);
 		h.optional(options.headerMeta);
 		h.optional(options.state);
-		h.bool(options.applyBg ?? true);
 		h.bool(isTerminalGraphicsFallbackActive());
 		if (options.sections) {
 			for (const s of options.sections) {

package/src/tui/utils.ts

@@ -2,8 +2,7 @@
  * Shared helpers for tool-rendered UI components.
  */
 import { padding, visibleWidth } from "@sayknow-cli/tui";
-import type { Theme, ThemeBg } from "../modes/theme/theme";
-import type { State } from "./types";
+import type { Theme } from "../modes/theme/theme";
 
 export { Ellipsis, truncateToWidth } from "@sayknow-cli/tui";
 
@@ -95,9 +94,3 @@ export function padToWidth(text: string, width: number, bgFn?: (s: string) => st
 	const padded = paddingNeeded > 0 ? text + padding(paddingNeeded) : text;
 	return bgFn ? bgFn(padded) : padded;
 }
-
-export function getStateBgColor(state: State): ThemeBg {
-	if (state === "success") return "toolSuccessBg";
-	if (state === "error") return "toolErrorBg";
-	return "toolPendingBg";
-}

package/src/utils/session-title-generation.ts

@@ -0,0 +1,29 @@
+/**
+ * In-memory generation lease for asynchronous session-title regeneration.
+ *
+ * The lease is keyed by the session manager instance so TUI and ACP command
+ * dispatchers share the same ordering state without adding protocol fields to
+ * the persisted session format.
+ */
+const generations = new WeakMap<object, number>();
+
+function currentGeneration(owner: object): number {
+	return generations.get(owner) ?? 0;
+}
+
+/** Start a new regeneration and invalidate every older completion. */
+export function beginSessionTitleGeneration(owner: object): number {
+	const generation = currentGeneration(owner) + 1;
+	generations.set(owner, generation);
+	return generation;
+}
+
+/** Invalidate pending regeneration after a newer session mutation is accepted. */
+export function invalidateSessionTitleGeneration(owner: object): void {
+	generations.set(owner, currentGeneration(owner) + 1);
+}
+
+/** Return whether a regeneration still owns the latest generation lease. */
+export function isSessionTitleGenerationCurrent(owner: object, generation: number): boolean {
+	return currentGeneration(owner) === generation;
+}

package/src/utils/title-generator.ts

@@ -3,6 +3,7 @@
  */
 import * as path from "node:path";
 
+import type { AgentMessage } from "@sayknow-cli/agent-core";
 import { type Api, type AssistantMessage, completeSimple, type Model, type Tool } from "@sayknow-cli/ai";
 import { logger, prompt } from "@sayknow-cli/utils";
 import type { ModelRegistry } from "../config/model-registry";
@@ -16,6 +17,9 @@ const TITLE_SYSTEM_PROMPT = prompt.render(titleSystemPrompt);
 const DEFAULT_TERMINAL_TITLE = "SKC";
 const TERMINAL_TITLE_CONTROL_CHARS = /[\u0000-\u001f\u007f-\u009f]/g;
 
+const MAX_TITLE_CONVERSATION_MESSAGES = 6;
+const MAX_TITLE_MESSAGE_CHARS = 400;
+
 const MAX_INPUT_CHARS = 2000;
 const TITLE_MAX_TOKENS = 30;
 const REASONING_SAFE_MAX_TOKENS = 1024;
@@ -28,6 +32,44 @@ const SET_TITLE_TOOL_NAME = "set_title";
 const MAX_TITLE_CHARS = 80;
 const MAX_TITLE_WORDS = 12;
 
+/**
+ * Build a bounded conversation digest for title regeneration. Regeneration
+ * reads several messages so a throwaway first prompt can be corrected, while
+ * the automatic first-message path intentionally uses only its initial input.
+ */
+export function buildConversationTitleInput(messages: readonly AgentMessage[]): string | undefined {
+	const userMessages: string[] = [];
+	for (const message of messages) {
+		if (message.role !== "user") continue;
+		const content = message.content;
+		const text =
+			typeof content === "string"
+				? content
+				: content
+						.filter(block => block.type === "text")
+						.map(block => block.text)
+						.join("\n");
+		const trimmed = text.trim();
+		if (trimmed) userMessages.push(trimmed);
+	}
+
+	if (userMessages.length === 0) return undefined;
+
+	const keptMessages =
+		userMessages.length > MAX_TITLE_CONVERSATION_MESSAGES
+			? [userMessages[0]!, ...userMessages.slice(-(MAX_TITLE_CONVERSATION_MESSAGES - 1))]
+			: userMessages;
+	// Reserve separator and ellipsis room so the digest is already within the
+	// generator's MAX_INPUT_CHARS bound; its existing truncation is a no-op here.
+	const maxMessageChars = Math.min(
+		MAX_TITLE_MESSAGE_CHARS,
+		Math.floor((MAX_INPUT_CHARS - (keptMessages.length - 1) - keptMessages.length) / keptMessages.length),
+	);
+	return keptMessages
+		.map(message => (message.length > maxMessageChars ? `${message.slice(0, maxMessageChars)}…` : message))
+		.join("\n");
+}
+
 const setTitleTool: Tool = {
 	name: SET_TITLE_TOOL_NAME,
 	description: "Set the generated session title.",
@@ -57,9 +99,11 @@ function getTitleModel(registry: ModelRegistry, settings: Settings, currentModel
 }
 
 /**
- * Generate a title for a session based on the first user message.
+ * Generate a title for a session based on the provided user-message input.
+ * Automatic titles pass the first user message; regeneration passes a bounded
+ * conversation digest.
  *
- * @param firstMessage The first user message
+ * @param firstMessage The first user message or bounded conversation digest
  * @param registry Model registry
  * @param settings Settings used to resolve the smol role
  * @param sessionId Optional session id for sticky API key selection