@oh-my-pi/pi-coding-agent 4.3.1 → 4.4.5

package/CHANGELOG.md

@@ -2,6 +2,55 @@
 
 ## [Unreleased]
 
+## [4.4.5] - 2026-01-11
+
+### Changed
+
+- Removed `format: "date-time"` from timestamp type conversion in JTD to JSON Schema transformation
+- Reorganized system prompt to display context, environment, and tools sections before discipline guidelines
+- Updated system prompt to show file paths more clearly in output
+- Improved YAML frontmatter parsing with better error messages including source file information
+
+### Fixed
+
+- Fixed frontmatter parsing to properly report source location when YAML parsing fails
+
+## [4.4.4] - 2026-01-11
+### Added
+
+- Added `todo_write` tool for creating and managing structured task lists during coding sessions
+- Added persistent todo panel above the editor that displays task progress
+- Added `Ctrl+T` keybinding to toggle todo list expansion
+- Added grouped display for consecutive Read tool calls, showing multiple file reads in a compact tree view
+- Added `todo_write` tool and persistent todo panel above the editor
+
+### Changed
+
+- Changed `Ctrl+Enter` to insert a newline when not streaming (previously `Alt+Enter`)
+- Changed `Ctrl+T` from toggling thinking block visibility to toggling todo list expansion
+- Changed system prompt to use more direct, field-oriented language with emphasis on verification and assumptions
+- Changed temporary model selector keybinding from Ctrl+Y to Alt+P
+- Changed expand hint text from "Ctrl+O to expand" to "Ctrl+O for more"
+- Changed Read tool result display to hide content by default, showing only file path and status
+- Changed `Ctrl+T` to toggle todo panel expansion
+
+### Removed
+
+- Removed `yaml` package dependency in favor of Bun's built-in YAML parser
+
+### Fixed
+
+- Fixed Alt+Enter to insert a newline when not streaming, instead of submitting the message
+- Fixed Alt+Enter inserting a new line when not streaming instead of submitting a message
+- Fixed Cursor provider to avoid advertising the Edit tool, relying on full-file Write operations instead
+- Fixed prompt template loading to strip leading HTML comment metadata blocks
+
+## [4.3.2] - 2026-01-11
+### Changed
+
+- Increased default bash output preview from 5 to 10 lines when collapsed
+- Updated expanded bash output view to show full untruncated output when available
+
 ## [4.3.1] - 2026-01-11
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "4.3.1",
+	"version": "4.4.5",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"ompConfig": {
@@ -39,10 +39,10 @@
 		"prepublishOnly": "bun run generate-template && bun run clean && bun run build"
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-ai": "4.3.1",
-		"@oh-my-pi/pi-agent-core": "4.3.1",
-		"@oh-my-pi/pi-git-tool": "4.3.1",
-		"@oh-my-pi/pi-tui": "4.3.1",
+		"@oh-my-pi/pi-ai": "4.4.5",
+		"@oh-my-pi/pi-agent-core": "4.4.5",
+		"@oh-my-pi/pi-git-tool": "4.4.5",
+		"@oh-my-pi/pi-tui": "4.4.5",
 		"@openai/agents": "^0.3.7",
 		"@sinclair/typebox": "^0.34.46",
 		"ajv": "^8.17.1",
@@ -63,7 +63,6 @@
 		"strip-ansi": "^7.1.2",
 		"winston": "^3.17.0",
 		"winston-daily-rotate-file": "^5.0.0",
-		"yaml": "^2.8.2",
 		"zod": "^4.3.5"
 	},
 	"devDependencies": {

package/src/core/frontmatter.ts

@@ -0,0 +1,98 @@
+import { YAML } from "bun";
+import { logger } from "./logger";
+
+function stripHtmlComments(content: string): string {
+	return content.replace(/<!--[\s\S]*?-->/g, "");
+}
+
+function toError(value: unknown): Error {
+	return value instanceof Error ? value : new Error(String(`YAML: ${value}`));
+}
+
+function truncate(content: string, maxLength: number): string {
+	return content.length > maxLength ? `${content.slice(0, maxLength)}...` : content;
+}
+
+export class FrontmatterError extends Error {
+	constructor(
+		error: Error,
+		public readonly source?: unknown,
+	) {
+		super(`Failed to parse YAML frontmatter: ${error.message}`, { cause: error });
+		this.name = "FrontmatterError";
+	}
+
+	toString(): string {
+		// Format the error with stack and detail, including the error message, stack, and source if present
+		const details: string[] = [this.message];
+		if (this.source !== undefined) {
+			details.push(`Source: ${JSON.stringify(this.source)}`);
+		}
+		if (this.cause && typeof this.cause === "object" && "stack" in this.cause && this.cause.stack) {
+			details.push(`Stack:\n${this.cause.stack}`);
+		} else if (this.stack) {
+			details.push(`Stack:\n${this.stack}`);
+		}
+		return details.join("\n\n");
+	}
+}
+
+export interface FrontmatterOptions {
+	/** Source of the content */
+	source?: unknown;
+	/** Fallback frontmatter values */
+	fallback?: Record<string, unknown>;
+	/** Normalize the content */
+	normalize?: boolean;
+	/** Level of error handling */
+	level?: "off" | "warn" | "fatal";
+}
+
+/**
+ * Parse YAML frontmatter from markdown content
+ * Returns { frontmatter, body } where body has frontmatter stripped
+ */
+export function parseFrontmatter(
+	content: string,
+	options?: FrontmatterOptions,
+): { frontmatter: Record<string, unknown>; body: string } {
+	const { source, fallback, normalize = true, level = "warn" } = options ?? {};
+	const frontmatter: Record<string, unknown> = Object.assign({}, fallback);
+
+	const normalized = normalize ? stripHtmlComments(content.replace(/\r\n/g, "\n").replace(/\r/g, "\n")) : content;
+	if (!normalized.startsWith("---")) {
+		return { frontmatter, body: normalized };
+	}
+
+	const endIndex = normalized.indexOf("\n---", 3);
+	if (endIndex === -1) {
+		return { frontmatter, body: normalized };
+	}
+
+	const metadata = normalized.slice(4, endIndex);
+	const body = normalized.slice(endIndex + 4).trim();
+
+	try {
+		// Replace tabs with spaces for YAML compatibility, use failsafe mode for robustness
+		const loaded = YAML.parse(metadata.replaceAll("\t", "  ")) as Record<string, unknown> | null;
+		return { frontmatter: Object.assign(frontmatter, loaded), body: body };
+	} catch (error) {
+		const err = new FrontmatterError(toError(error), source ?? `Inline '${truncate(content, 64)}'`);
+		if (level === "warn" || level === "fatal") {
+			logger.warn("Failed to parse YAML frontmatter", { err: err.toString() });
+		}
+		if (level === "fatal") {
+			throw err;
+		}
+
+		// Simple YAML parsing - just key: value pairs
+		for (const line of metadata.split("\n")) {
+			const match = line.match(/^(\w+):\s*(.*)$/);
+			if (match) {
+				frontmatter[match[1]] = match[2].trim();
+			}
+		}
+
+		return { frontmatter, body: body };
+	}
+}

package/src/core/keybindings.ts

@@ -59,7 +59,7 @@ export const DEFAULT_APP_KEYBINDINGS: Record<AppAction, KeyId | KeyId[]> = {
 	expandTools: "ctrl+o",
 	toggleThinking: "ctrl+t",
 	externalEditor: "ctrl+g",
-	followUp: "alt+enter",
+	followUp: "ctrl+enter",
 	dequeue: "alt+up",
 };
 

package/src/core/prompt-templates.ts

@@ -1,6 +1,7 @@
 import { join, resolve } from "node:path";
 import Handlebars from "handlebars";
 import { CONFIG_DIR_NAME, getPromptsDir } from "../config";
+import { parseFrontmatter } from "./frontmatter";
 import { logger } from "./logger";
 
 /**
@@ -280,36 +281,6 @@ function optimizePromptLayout(input: string): string {
 	return s.trim();
 }
 
-/**
- * Parse YAML frontmatter from markdown content
- * Returns { frontmatter, content } where content has frontmatter stripped
- */
-function parseFrontmatter(content: string): { frontmatter: Record<string, string>; content: string } {
-	const frontmatter: Record<string, string> = {};
-
-	if (!content.startsWith("---")) {
-		return { frontmatter, content };
-	}
-
-	const endIndex = content.indexOf("\n---", 3);
-	if (endIndex === -1) {
-		return { frontmatter, content };
-	}
-
-	const frontmatterBlock = content.slice(4, endIndex);
-	const remainingContent = content.slice(endIndex + 4).trim();
-
-	// Simple YAML parsing - just key: value pairs
-	for (const line of frontmatterBlock.split("\n")) {
-		const match = line.match(/^(\w+):\s*(.*)$/);
-		if (match) {
-			frontmatter[match[1]] = match[2].trim();
-		}
-	}
-
-	return { frontmatter, content: remainingContent };
-}
-
 /**
  * Parse command arguments respecting quoted strings (bash-style)
  * Returns array of arguments
@@ -413,7 +384,7 @@ async function loadTemplatesFromDir(
 
 				if (entry.endsWith(".md")) {
 					const rawContent = await file.text();
-					const { frontmatter, content } = parseFrontmatter(rawContent);
+					const { frontmatter, body } = parseFrontmatter(rawContent, { source: fullPath });
 
 					const name = entry.split("/").pop()!.slice(0, -3); // Remove .md extension
 
@@ -429,9 +400,9 @@ async function loadTemplatesFromDir(
 					}
 
 					// Get description from frontmatter or first non-empty line
-					let description = frontmatter.description || "";
+					let description = String(frontmatter.description || "");
 					if (!description) {
-						const firstLine = content.split("\n").find((line) => line.trim());
+						const firstLine = body.split("\n").find((line) => line.trim());
 						if (firstLine) {
 							// Truncate if too long
 							description = firstLine.slice(0, 60);
@@ -445,7 +416,7 @@ async function loadTemplatesFromDir(
 					templates.push({
 						name,
 						description,
-						content,
+						content: body,
 						source: sourceStr,
 					});
 				}

package/src/core/sdk.ts

@@ -855,6 +855,9 @@ export async function createAgentSession(options: CreateAgentSessionOptions = {}
 			toolRegistry.set(tool.name, wrapToolWithExtensions(tool, extensionRunner));
 		}
 	}
+	if (model?.provider === "cursor") {
+		toolRegistry.delete("edit");
+	}
 	time("combineTools");
 
 	let cursorEventEmitter: ((event: AgentEvent) => void) | undefined;

package/src/core/skills.ts

@@ -6,7 +6,7 @@ import { skillCapability } from "../capability/skill";
 import type { SourceMeta } from "../capability/types";
 import type { Skill as CapabilitySkill, SkillFrontmatter as ImportedSkillFrontmatter } from "../discovery";
 import { loadCapability } from "../discovery";
-import { parseFrontmatter } from "../discovery/helpers";
+import { parseFrontmatter } from "./frontmatter";
 import { logger } from "./logger";
 import type { SkillsSettings } from "./settings-manager";
 
@@ -54,7 +54,7 @@ export function loadSkillsFromDir(options: LoadSkillsFromDirOptions): LoadSkills
 		if (seenPaths.has(skillFile)) return;
 		try {
 			const content = readFileSync(skillFile, "utf-8");
-			const { frontmatter } = parseFrontmatter(content);
+			const { frontmatter } = parseFrontmatter(content, { source: skillFile });
 			const name = (frontmatter.name as string) || dirName;
 			const description = frontmatter.description as string;
 
@@ -118,7 +118,7 @@ function scanDirectoryForSkills(dir: string): LoadSkillsResult {
 		if (seenPaths.has(skillFile)) return;
 		try {
 			const content = readFileSync(skillFile, "utf-8");
-			const { frontmatter } = parseFrontmatter(content);
+			const { frontmatter } = parseFrontmatter(content, { source: skillFile });
 			const name = (frontmatter.name as string) || dirName;
 			const description = frontmatter.description as string;
 

package/src/core/slash-commands.ts

@@ -1,7 +1,7 @@
 import { slashCommandCapability } from "../capability/slash-command";
 import type { SlashCommand } from "../discovery";
 import { loadCapability } from "../discovery";
-import { parseFrontmatter } from "../discovery/helpers";
+import { parseFrontmatter } from "./frontmatter";
 import { renderPromptTemplate } from "./prompt-templates";
 import { EMBEDDED_COMMAND_TEMPLATES } from "./tools/task/commands";
 
@@ -19,8 +19,11 @@ export interface FileSlashCommand {
 
 const EMBEDDED_SLASH_COMMANDS = EMBEDDED_COMMAND_TEMPLATES;
 
-function parseCommandTemplate(content: string): { description: string; body: string } {
-	const { frontmatter, body } = parseFrontmatter(content);
+function parseCommandTemplate(
+	content: string,
+	options: { source: string; level?: "off" | "warn" | "fatal" },
+): { description: string; body: string } {
+	const { frontmatter, body } = parseFrontmatter(content, options);
 	const frontmatterDesc = typeof frontmatter.description === "string" ? frontmatter.description.trim() : "";
 
 	// Get description from frontmatter or first non-empty line
@@ -112,7 +115,10 @@ export async function loadSlashCommands(options: LoadSlashCommandsOptions = {}):
 	const result = await loadCapability<SlashCommand>(slashCommandCapability.id, { cwd: options.cwd });
 
 	const fileCommands: FileSlashCommand[] = result.items.map((cmd) => {
-		const { description, body } = parseCommandTemplate(cmd.content);
+		const { description, body } = parseCommandTemplate(cmd.content, {
+			source: cmd.path ?? `slash-command:${cmd.name}`,
+			level: cmd.level === "native" ? "fatal" : "warn",
+		});
 
 		// Format source label: "via ProviderName Level"
 		const capitalizedLevel = cmd.level.charAt(0).toUpperCase() + cmd.level.slice(1);
@@ -132,7 +138,10 @@ export async function loadSlashCommands(options: LoadSlashCommandsOptions = {}):
 		const name = cmd.name.replace(/\.md$/, "");
 		if (seenNames.has(name)) continue;
 
-		const { description, body } = parseCommandTemplate(cmd.content);
+		const { description, body } = parseCommandTemplate(cmd.content, {
+			source: `embedded:${cmd.name}`,
+			level: "fatal",
+		});
 		fileCommands.push({
 			name,
 			description,

package/src/core/tools/bash.ts

@@ -15,6 +15,8 @@ import { resolveToCwd } from "./path-utils";
 import { createToolUIKit } from "./render-utils";
 import { DEFAULT_MAX_BYTES, formatSize, type TruncationResult, truncateTail } from "./truncate";
 
+export const BASH_DEFAULT_PREVIEW_LINES = 10;
+
 const bashSchema = Type.Object({
 	command: Type.String({ description: "Bash command to execute" }),
 	timeout: Type.Optional(Type.Number({ description: "Timeout in seconds (optional, no default timeout)" })),
@@ -26,6 +28,7 @@ const bashSchema = Type.Object({
 export interface BashToolDetails {
 	truncation?: TruncationResult;
 	fullOutputPath?: string;
+	fullOutput?: string;
 }
 
 /**
@@ -101,9 +104,12 @@ export function createBashTool(session: ToolSession, options?: BashToolOptions):
 						const truncation = truncateTail(currentOutput);
 						onUpdate({
 							content: [{ type: "text", text: truncation.content || "" }],
-							details: {
-								truncation: truncation.truncated ? truncation : undefined,
-							},
+							details: truncation.truncated
+								? {
+										truncation,
+										fullOutput: currentOutput,
+									}
+								: undefined,
 						});
 					}
 				},
@@ -129,6 +135,7 @@ export function createBashTool(session: ToolSession, options?: BashToolOptions):
 				details = {
 					truncation,
 					fullOutputPath: result.fullOutputPath,
+					fullOutput: currentOutput,
 				};
 
 				const startLine = truncation.totalLines - truncation.outputLines + 1;
@@ -173,6 +180,9 @@ interface BashRenderContext {
 	previewLines?: number;
 }
 
+// Preview line limit when not expanded (matches tool-execution behavior)
+export const BASH_PREVIEW_LINES = 10;
+
 export const bashToolRenderer = {
 	renderCall(args: BashRenderArgs, uiTheme: Theme): Component {
 		const ui = createToolUIKit(uiTheme);
@@ -214,21 +224,25 @@ export const bashToolRenderer = {
 		const { renderContext } = options;
 		const details = result.details;
 
+		const expanded = renderContext?.expanded ?? options.expanded;
+		const previewLines = renderContext?.previewLines ?? BASH_DEFAULT_PREVIEW_LINES;
+
 		// Get output from context (preferred) or fall back to result content
 		const output = renderContext?.output ?? (result.content?.find((c) => c.type === "text")?.text ?? "").trim();
-		const expanded = renderContext?.expanded ?? options.expanded;
-		const previewLines = renderContext?.previewLines ?? 5;
+		const fullOutput = details?.fullOutput;
+		const displayOutput = expanded ? (fullOutput ?? output) : output;
+		const showingFullOutput = expanded && fullOutput !== undefined;
 
 		// Build truncation warning lines (static, doesn't depend on width)
 		const truncation = details?.truncation;
 		const fullOutputPath = details?.fullOutputPath;
 		let warningLine: string | undefined;
-		if (truncation?.truncated || fullOutputPath) {
+		if (fullOutputPath || (truncation?.truncated && !showingFullOutput)) {
 			const warnings: string[] = [];
 			if (fullOutputPath) {
 				warnings.push(`Full output: ${fullOutputPath}`);
 			}
-			if (truncation?.truncated) {
+			if (truncation?.truncated && !showingFullOutput) {
 				if (truncation.truncatedBy === "lines") {
 					warnings.push(`Truncated: showing ${truncation.outputLines} of ${truncation.totalLines} lines`);
 				} else {
@@ -237,17 +251,19 @@ export const bashToolRenderer = {
 					);
 				}
 			}
-			warningLine = uiTheme.fg("warning", ui.wrapBrackets(warnings.join(". ")));
+			if (warnings.length > 0) {
+				warningLine = uiTheme.fg("warning", ui.wrapBrackets(warnings.join(". ")));
+			}
 		}
 
-		if (!output) {
+		if (!displayOutput) {
 			// No output - just show warning if any
 			return new Text(warningLine ?? "", 0, 0);
 		}
 
 		if (expanded) {
 			// Show all lines when expanded
-			const styledOutput = output
+			const styledOutput = displayOutput
 				.split("\n")
 				.map((line) => uiTheme.fg("toolOutput", line))
 				.join("\n");
@@ -256,7 +272,7 @@ export const bashToolRenderer = {
 		}
 
 		// Collapsed: use width-aware caching component
-		const styledOutput = output
+		const styledOutput = displayOutput
 			.split("\n")
 			.map((line) => uiTheme.fg("toolOutput", line))
 			.join("\n");

package/src/core/tools/calculator.ts

@@ -478,7 +478,7 @@ export const calculatorToolRenderer = {
 		const hasMore = outputs.length > maxItems;
 		const icon = uiTheme.styledSymbol("status.success", "success");
 		const summary = uiTheme.fg("dim", formatCount("result", outputs.length));
-		const expandHint = formatExpandHint(expanded, hasMore, uiTheme);
+		const expandHint = formatExpandHint(uiTheme, expanded, hasMore);
 		let text = `${icon} ${summary}${expandHint}`;
 
 		// Render each result as a tree branch

package/src/core/tools/edit.ts

@@ -21,7 +21,7 @@ import {
 import type { ToolSession } from "./index";
 import { createLspWritethrough, type FileDiagnosticsResult, writethroughNoop } from "./lsp/index";
 import { resolveToCwd } from "./path-utils";
-import { createToolUIKit, getDiffStats, shortenPath, truncateDiffByHunk } from "./render-utils";
+import { createToolUIKit, formatExpandHint, getDiffStats, shortenPath, truncateDiffByHunk } from "./render-utils";
 
 const editSchema = Type.Object({
 	path: Type.String({ description: "Path to the file to edit (relative or absolute)" }),
@@ -308,7 +308,7 @@ export const editToolRenderer = {
 					if (hiddenLines > 0) remainder.push(`${hiddenLines} more lines`);
 					text += uiTheme.fg(
 						"toolOutput",
-						`\n${uiTheme.format.ellipsis} (${remainder.join(", ")}) ${ui.wrapBrackets("Ctrl+O to expand")}`,
+						`\n${uiTheme.format.ellipsis} (${remainder.join(", ")}) ${formatExpandHint(uiTheme)}`,
 					);
 				}
 			}

package/src/core/tools/exa/render.ts

@@ -13,9 +13,9 @@ import {
 	formatCount,
 	formatExpandHint,
 	formatMoreItems,
+	formatStatusIcon,
 	getDomain,
 	getPreviewLines,
-	getStyledStatusIcon,
 	PREVIEW_LIMITS,
 	TRUNCATE_LENGTHS,
 	truncate,
@@ -32,14 +32,14 @@ const MAX_HIGHLIGHT_LEN = TRUNCATE_LENGTHS.CONTENT;
 function renderErrorMessage(message: string, theme: Theme): Text {
 	const clean = message.replace(/^Error:\s*/, "").trim();
 	return new Text(
-		`${getStyledStatusIcon("error", theme)} ${theme.fg("error", `Error: ${clean || "Unknown error"}`)}`,
+		`${formatStatusIcon("error", theme)} ${theme.fg("error", `Error: ${clean || "Unknown error"}`)}`,
 		0,
 		0,
 	);
 }
 
 function renderEmptyMessage(message: string, theme: Theme): Text {
-	return new Text(`${getStyledStatusIcon("warning", theme)} ${theme.fg("muted", message)}`, 0, 0);
+	return new Text(`${formatStatusIcon("warning", theme)} ${theme.fg("muted", message)}`, 0, 0);
 }
 
 /** Render Exa result with tree-based layout */
@@ -64,9 +64,9 @@ export function renderExaResult(
 			const maxLines = expanded ? rawLines.length : Math.min(rawLines.length, COLLAPSED_PREVIEW_LINES);
 			const displayLines = rawLines.slice(0, maxLines);
 			const remaining = rawLines.length - maxLines;
-			const expandHint = formatExpandHint(expanded, remaining > 0, uiTheme);
+			const expandHint = formatExpandHint(uiTheme, expanded, remaining > 0);
 
-			let text = `${getStyledStatusIcon("info", uiTheme)} ${uiTheme.fg("dim", "Raw response")}${expandHint}`;
+			let text = `${formatStatusIcon("info", uiTheme)} ${uiTheme.fg("dim", "Raw response")}${expandHint}`;
 
 			for (let i = 0; i < displayLines.length; i++) {
 				const isLast = i === displayLines.length - 1 && remaining === 0;
@@ -78,7 +78,10 @@ export function renderExaResult(
 			}
 
 			if (remaining > 0) {
-				text += `\n ${uiTheme.fg("dim", uiTheme.tree.last)} ${uiTheme.fg("muted", formatMoreItems(remaining, "line", uiTheme))}`;
+				text += `\n ${uiTheme.fg("dim", uiTheme.tree.last)} ${uiTheme.fg(
+					"muted",
+					formatMoreItems(remaining, "line", uiTheme),
+				)}`;
 			}
 
 			return new Text(text, 0, 0);
@@ -91,7 +94,7 @@ export function renderExaResult(
 	const cost = response.costDollars?.total;
 	const time = response.searchTime;
 
-	const icon = getStyledStatusIcon(resultCount > 0 ? "success" : "warning", uiTheme);
+	const icon = formatStatusIcon(resultCount > 0 ? "success" : "warning", uiTheme);
 
 	const metaParts = [formatCount("result", resultCount)];
 	if (cost !== undefined) metaParts.push(`cost:$${cost.toFixed(4)}`);
@@ -104,7 +107,7 @@ export function renderExaResult(
 		const totalLines = previewText.split("\n").filter((l) => l.trim()).length;
 		hasMorePreview = totalLines > COLLAPSED_PREVIEW_LINES || resultCount > 1;
 	}
-	const expandHint = formatExpandHint(expanded, hasMorePreview, uiTheme);
+	const expandHint = formatExpandHint(uiTheme, expanded, hasMorePreview);
 
 	let text = `${icon} ${uiTheme.fg("dim", summaryText)}${expandHint}`;
 
@@ -165,11 +168,17 @@ export function renderExaResult(
 		text += `\n ${uiTheme.fg("dim", branch)} ${uiTheme.fg("accent", title)}${domainPart}`;
 
 		if (res.url) {
-			text += `\n ${uiTheme.fg("dim", cont)} ${uiTheme.fg("dim", uiTheme.tree.hook)} ${uiTheme.fg("mdLinkUrl", res.url)}`;
+			text += `\n ${uiTheme.fg("dim", cont)} ${uiTheme.fg("dim", uiTheme.tree.hook)} ${uiTheme.fg(
+				"mdLinkUrl",
+				res.url,
+			)}`;
 		}
 
 		if (res.author) {
-			text += `\n ${uiTheme.fg("dim", cont)} ${uiTheme.fg("dim", uiTheme.tree.hook)} ${uiTheme.fg("muted", `Author: ${res.author}`)}`;
+			text += `\n ${uiTheme.fg("dim", cont)} ${uiTheme.fg("dim", uiTheme.tree.hook)} ${uiTheme.fg(
+				"muted",
+				`Author: ${res.author}`,
+			)}`;
 		}
 
 		if (res.publishedDate) {
@@ -197,7 +206,10 @@ export function renderExaResult(
 		}
 
 		if (res.highlights?.length) {
-			text += `\n ${uiTheme.fg("dim", cont)} ${uiTheme.fg("dim", uiTheme.tree.hook)} ${uiTheme.fg("accent", "Highlights")}`;
+			text += `\n ${uiTheme.fg("dim", cont)} ${uiTheme.fg("dim", uiTheme.tree.hook)} ${uiTheme.fg(
+				"accent",
+				"Highlights",
+			)}`;
 			const maxHighlights = Math.min(res.highlights.length, 3);
 			for (let j = 0; j < maxHighlights; j++) {
 				const h = res.highlights[j];

package/src/core/tools/index.test.ts

@@ -29,6 +29,7 @@ describe("createTools", () => {
 		expect(names).toContain("lsp");
 		expect(names).toContain("notebook");
 		expect(names).toContain("task");
+		expect(names).toContain("todo_write");
 		expect(names).toContain("output");
 		expect(names).toContain("web_fetch");
 		expect(names).toContain("web_search");
@@ -164,6 +165,7 @@ describe("createTools", () => {
 			"output",
 			"read",
 			"task",
+			"todo_write",
 			"web_fetch",
 			"web_search",
 			"write",

package/src/core/tools/index.ts

@@ -28,6 +28,7 @@ export { createReadTool, type ReadToolDetails } from "./read";
 export { reportFindingTool, type SubmitReviewDetails } from "./review";
 export { createSshTool, type SSHToolDetails } from "./ssh";
 export { BUNDLED_AGENTS, createTaskTool, taskTool } from "./task/index";
+export { createTodoWriteTool, type TodoItem, type TodoWriteToolDetails } from "./todo-write";
 export {
 	DEFAULT_MAX_BYTES,
 	DEFAULT_MAX_LINES,
@@ -79,6 +80,7 @@ import { createReadTool } from "./read";
 import { reportFindingTool } from "./review";
 import { createSshTool } from "./ssh";
 import { createTaskTool } from "./task/index";
+import { createTodoWriteTool } from "./todo-write";
 import { createWebFetchTool } from "./web-fetch";
 import { createWebSearchTool } from "./web-search/index";
 import { createWriteTool } from "./write";
@@ -145,6 +147,7 @@ export const BUILTIN_TOOLS: Record<string, ToolFactory> = {
 	output: createOutputTool,
 	read: createReadTool,
 	task: createTaskTool,
+	todo_write: createTodoWriteTool,
 	web_fetch: createWebFetchTool,
 	web_search: createWebSearchTool,
 	write: createWriteTool,

package/src/core/tools/jtd-to-json-schema.ts

@@ -108,12 +108,7 @@ function convertSchema(schema: unknown): unknown {
 		if (!jsonType) {
 			return { type: schema.type };
 		}
-		const result: Record<string, unknown> = { type: jsonType };
-		// Add format for timestamp
-		if (schema.type === "timestamp") {
-			result.format = "date-time";
-		}
-		return result;
+		return { type: jsonType };
 	}
 
 	// Enum form: { enum: ["a", "b"] } → { enum: ["a", "b"] }

package/src/core/tools/ls.ts

@@ -265,7 +265,7 @@ export const lsToolRenderer = {
 		);
 		const maxEntries = expanded ? entries.length : Math.min(entries.length, COLLAPSED_LIST_LIMIT);
 		const hasMoreEntries = entries.length > maxEntries;
-		const expandHint = formatExpandHint(expanded, hasMoreEntries, uiTheme);
+		const expandHint = formatExpandHint(uiTheme, expanded, hasMoreEntries);
 
 		let text = `${icon} ${uiTheme.fg("dim", summaryText)}${formatTruncationSuffix(truncated, uiTheme)}${expandHint}`;
 
@@ -302,7 +302,10 @@ export const lsToolRenderer = {
 		}
 
 		if (hasTruncation) {
-			text += `\n ${uiTheme.fg("dim", uiTheme.tree.last)} ${uiTheme.fg("warning", `truncated: ${truncationReasons.join(", ")}`)}`;
+			text += `\n ${uiTheme.fg("dim", uiTheme.tree.last)} ${uiTheme.fg(
+				"warning",
+				`truncated: ${truncationReasons.join(", ")}`,
+			)}`;
 		}
 
 		return new Text(text, 0, 0);