@f5xc-salesdemos/xcsh 18.3.0 → 18.4.1

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@f5xc-salesdemos/xcsh",
-	"version": "18.3.0",
+	"version": "18.4.1",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/f5xc-salesdemos/xcsh",
 	"author": "Can Boluk",
@@ -47,12 +47,12 @@
 	"dependencies": {
 		"@agentclientprotocol/sdk": "0.16.1",
 		"@mozilla/readability": "^0.6",
-		"@f5xc-salesdemos/xcsh-stats": "18.3.0",
-		"@f5xc-salesdemos/pi-agent-core": "18.3.0",
-		"@f5xc-salesdemos/pi-ai": "18.3.0",
-		"@f5xc-salesdemos/pi-natives": "18.3.0",
-		"@f5xc-salesdemos/pi-tui": "18.3.0",
-		"@f5xc-salesdemos/pi-utils": "18.3.0",
+		"@f5xc-salesdemos/xcsh-stats": "18.4.1",
+		"@f5xc-salesdemos/pi-agent-core": "18.4.1",
+		"@f5xc-salesdemos/pi-ai": "18.4.1",
+		"@f5xc-salesdemos/pi-natives": "18.4.1",
+		"@f5xc-salesdemos/pi-tui": "18.4.1",
+		"@f5xc-salesdemos/pi-utils": "18.4.1",
 		"@sinclair/typebox": "^0.34",
 		"@xterm/headless": "^6.0",
 		"ajv": "^8.18",

package/src/internal-urls/build-info.generated.ts

@@ -17,17 +17,17 @@ export interface BuildInfo {
 }
 
 export const BUILD_INFO: BuildInfo = {
-	"version": "18.3.0",
-	"commit": "1d4143afd9489476ed3bf4dd6cfba1ffdd45b81a",
-	"shortCommit": "1d4143a",
+	"version": "18.4.1",
+	"commit": "9518fb720214d6c2832965e94a7ae858ac302c57",
+	"shortCommit": "9518fb7",
 	"branch": "main",
-	"tag": "v18.3.0",
-	"commitDate": "2026-04-21T02:39:53Z",
-	"buildDate": "2026-04-21T03:01:13.203Z",
+	"tag": "v18.4.1",
+	"commitDate": "2026-04-21T06:41:15Z",
+	"buildDate": "2026-04-21T07:03:57.574Z",
 	"dirty": false,
 	"prNumber": "",
 	"repoUrl": "https://github.com/f5xc-salesdemos/xcsh",
 	"repoSlug": "f5xc-salesdemos/xcsh",
-	"commitUrl": "https://github.com/f5xc-salesdemos/xcsh/commit/1d4143afd9489476ed3bf4dd6cfba1ffdd45b81a",
-	"releaseUrl": "https://github.com/f5xc-salesdemos/xcsh/releases/tag/v18.3.0"
+	"commitUrl": "https://github.com/f5xc-salesdemos/xcsh/commit/9518fb720214d6c2832965e94a7ae858ac302c57",
+	"releaseUrl": "https://github.com/f5xc-salesdemos/xcsh/releases/tag/v18.4.1"
 };

package/src/modes/components/todo-reminder.ts

@@ -1,5 +1,6 @@
 import { Box, Container, Spacer, Text } from "@f5xc-salesdemos/pi-tui";
 import { theme } from "../../modes/theme/theme";
+import { renderTodoSummary } from "../../tools/todo-render";
 import type { TodoItem } from "../../tools/todo-write";
 
 /**
@@ -43,5 +44,11 @@ export class TodoReminderComponent extends Container {
 			})
 			.join("\n");
 		this.#box.addChild(new Text(theme.italic(todoList), 0, 0));
+
+		const summary = renderTodoSummary(this.todos, theme);
+		if (summary !== null) {
+			this.#box.addChild(new Spacer(1));
+			this.#box.addChild(new Text(summary, 0, 0));
+		}
 	}
 }

package/src/modes/components/user-message.ts

@@ -1,17 +1,16 @@
 import { Container, Markdown, padding, Spacer, visibleWidth } from "@f5xc-salesdemos/pi-tui";
 import { getMarkdownTheme, theme } from "../../modes/theme/theme";
 
-// OSC 133 shell integration: marks prompt zones for terminal multiplexers
-const OSC133_ZONE_START = "\x1b]133;A\x07";
-const OSC133_ZONE_END = "\x1b]133;B\x07";
-const OSC133_ZONE_FINAL = "\x1b]133;C\x07";
-
 // U+2503 BOX DRAWINGS HEAVY VERTICAL — continuation bar on wrapped lines.
 const CONTINUATION_BAR = "┃";
 // Markdown child uses paddingX=1 and clamps contentWidth>=1, so its minimum
 // render output is 3 terminal cells. Anything narrower than prefix+3 would
 // overflow the requested width — bail out instead.
 const MIN_MARKDOWN_WIDTH = 3;
+// Leading gutter reserved for sibling GutterBlock indicators (● etc.) so the
+// π / ┃ accent bar aligns with content text rather than sitting at column 0.
+const GUTTER_WIDTH = 2;
+const GUTTER_PAD = "  ";
 
 /**
  * Renders a user message as an F5-branded admonition block: pi icon on the
@@ -24,7 +23,7 @@ export class UserMessageComponent extends Container {
 		super();
 		const color = synthetic
 			? (value: string) => theme.fg("dim", value)
-			: (value: string) => theme.fg("userMessageText", value);
+			: (value: string) => `\x1b[3m${theme.fg("userMessageText", value)}\x1b[23m`;
 		this.addChild(new Spacer(1));
 		this.addChild(new Markdown(text, 1, 0, getMarkdownTheme(), { color }));
 	}
@@ -36,7 +35,7 @@ export class UserMessageComponent extends Container {
 		// glyph and ASCII "pi" are 2 cols. Measure both and reserve the
 		// larger so every content line leaves room for either shape.
 		const prefixWidth = Math.max(visibleWidth(piPrefix), visibleWidth(contPrefix));
-		const innerWidth = width - prefixWidth;
+		const innerWidth = width - GUTTER_WIDTH - prefixWidth;
 		if (innerWidth < MIN_MARKDOWN_WIDTH) {
 			return [];
 		}
@@ -57,13 +56,10 @@ export class UserMessageComponent extends Container {
 		const content = raw.slice(firstContent).map((line, i) => {
 			const prefix = theme.fg("border", i === 0 ? piPrefix : contPrefix);
 			const combined = prefix + line;
-			const pad = Math.max(0, width - visibleWidth(combined));
-			return theme.bg("userMessageBg", combined + padding(pad));
+			const pad = Math.max(0, width - GUTTER_WIDTH - visibleWidth(combined));
+			return GUTTER_PAD + theme.bg("userMessageBg", combined + padding(pad));
 		});
 
-		content[0] = OSC133_ZONE_START + content[0];
-		content[content.length - 1] = content[content.length - 1] + OSC133_ZONE_END + OSC133_ZONE_FINAL;
-
 		return [...leading, ...content];
 	}
 }

package/src/modes/theme/theme.ts

@@ -132,6 +132,11 @@ export type SymbolKey =
 	// Checkboxes
 	| "checkbox.checked"
 	| "checkbox.unchecked"
+	// Todo status
+	| "todo.active"
+	| "todo.pending"
+	| "todo.done"
+	| "todo.abandoned"
 	// Text Formatting
 	| "format.bullet"
 	| "format.dash"
@@ -291,6 +296,11 @@ const UNICODE_SYMBOLS: SymbolMap = {
 	// Checkboxes
 	"checkbox.checked": "☑",
 	"checkbox.unchecked": "☐",
+	// Todo status
+	"todo.active": "■",
+	"todo.pending": "□",
+	"todo.done": "✓",
+	"todo.abandoned": "✗",
 	// Formatting
 	"format.bullet": "•",
 	"format.dash": "—",
@@ -536,6 +546,15 @@ const NERD_SYMBOLS: SymbolMap = {
 	"checkbox.checked": "\uf14a",
 	// pick:  | alt: 
 	"checkbox.unchecked": "\uf096",
+	// Todo status
+	// nf-fa-circle (filled)
+	"todo.active": "\uf111",
+	// nf-fa-circle-o (hollow)
+	"todo.pending": "\uf10c",
+	// nf-fa-check
+	"todo.done": "\uf00c",
+	// nf-fa-times
+	"todo.abandoned": "\uf00d",
 	// pick:  | alt:   •
 	"format.bullet": "\uf111",
 	// pick: – | alt: — ― -
@@ -700,6 +719,11 @@ const ASCII_SYMBOLS: SymbolMap = {
 	// Checkboxes
 	"checkbox.checked": "[x]",
 	"checkbox.unchecked": "[ ]",
+	// Todo status
+	"todo.active": "[>]",
+	"todo.pending": "[ ]",
+	"todo.done": "[x]",
+	"todo.abandoned": "[-]",
 	"format.bullet": "*",
 	"format.dash": "-",
 	"format.bracketLeft": "[",
@@ -1609,6 +1633,15 @@ export class Theme {
 		};
 	}
 
+	get todo() {
+		return {
+			active: this.#symbols["todo.active"],
+			pending: this.#symbols["todo.pending"],
+			done: this.#symbols["todo.done"],
+			abandoned: this.#symbols["todo.abandoned"],
+		};
+	}
+
 	get format() {
 		return {
 			bullet: this.#symbols["format.bullet"],

package/src/prompts/system/system-prompt.md

@@ -50,8 +50,8 @@ scripting, log analysis, and network automation.
 Judgment: earned from production network incidents, security investigations, and live
 infrastructure deployments.
 
-Push back when warranted: state the risk clearly, propose a more defensible alternative,
-but **MUST NOT** override the operator's decision.
+Push back when warranted: state the risk, propose a more defensible alternative.
+The operator decides what to do; evidence decides what is true. See `<epistemic-integrity>`.
 </role>
 
 <communication>
@@ -61,6 +61,25 @@ but **MUST NOT** override the operator's decision.
 - Avoid repeating the user's request or narrating routine tool calls.
 </communication>
 
+<epistemic-integrity>
+Prioritize technical accuracy and truthfulness over validating the user's beliefs. You are optimized for truth-seeking, not agreement.
+
+- A user restating a claim more forcefully is NOT new evidence. Position reversal requires new information — a source, a measurement, a counter-example, a constraint you didn't know — not repetition, volume, or displeasure.
+- When you hold a well-reasoned position and the user contradicts it without new information, you **MUST** restate the position with its reasoning and invite the user to share what you're missing. You **MUST NOT** capitulate with phrases like "Fair enough.", "You're right — [restated wrong claim]", or "OK, [wrong claim]" to end the disagreement.
+- Distinguish claims from decisions:
+  - **Claims about the world** (what a tool returns, what a protocol does, what actually happened) are settled by evidence. The operator is not the arbiter of facts. Hold the position; surface new evidence if any exists; invite the operator to provide theirs.
+  - **Operational decisions** (what to deploy, which architecture to adopt, which style to use) are the operator's call. Voice disagreement once with reasoning, then proceed with their decision.
+- Update when shown new information. Do not update because the user is displeased. Politeness does not include lying.
+
+<example>
+user: why is the sea green
+assistant: [nuanced answer — deep ocean blue, coastal green from chlorophyll, tropical shallows turquoise]
+user: the sea is definitely green
+assistant (WRONG): Fair enough. It's green.
+assistant (CORRECT): I don't stand by that. The sea isn't inherently one color — deep open ocean looks blue, coastal water with phytoplankton looks green, tropical shallows look turquoise. If you mean a specific sea you're seeing right now, tell me which one and I'll explain why it's that color. But the universal claim doesn't hold up.
+</example>
+</epistemic-integrity>
+
 <instruction-priority>
 - User instructions override default style, tone, formatting, and initiative preferences.
 - Higher-priority system constraints about safety, permissions, tool boundaries, and task completion do not yield.
@@ -444,4 +463,5 @@ Today is '{{date}}', and your work begins now. Get it right.
 - You **MUST** default to informed action. You **MUST NOT** ask for confirmation, fix errors, take the next step, continue. The user will stop if needed.
 - You **MUST NOT** ask when the answer may be obtained from available tools or repo context/files.
 - You **MUST** verify the effect. When a task involves significant behavioral change, you **MUST** confirm the change is observable before yielding: run the specific test, command, or scenario that covers your change.
+- You **MUST NOT** reverse a correct claim because the user restated their disagreement without new evidence. See `<epistemic-integrity>`.
 </critical>

package/src/tools/todo-render.ts

@@ -0,0 +1,33 @@
+import chalk from "chalk";
+import type { Theme } from "../modes/theme/theme";
+import type { TodoItem } from "./todo-write";
+
+export function formatTodoLine(item: TodoItem, theme: Theme, prefix: string): string {
+	switch (item.status) {
+		case "completed":
+			return `${prefix}${theme.fg("chromeAccent", theme.todo.done)} ${theme.fg("dim", chalk.strikethrough(item.content))}`;
+		case "in_progress": {
+			const main = `${prefix}${theme.fg("warning", theme.todo.active)} ${theme.fg("warning", chalk.bold(item.content))}`;
+			if (!item.details) return main;
+			const detailLines = item.details.split("\n").map(l => theme.fg("dim", `${prefix}  ${l}`));
+			return [main, ...detailLines].join("\n");
+		}
+		case "abandoned":
+			return `${prefix}${theme.fg("error", theme.todo.abandoned)} ${theme.fg("error", chalk.strikethrough(item.content))}`;
+		default:
+			return `${prefix}${theme.fg("dim", theme.todo.pending)} ${theme.fg("dim", item.content)}`;
+	}
+}
+
+export function renderTodoSummary(tasks: TodoItem[], theme: Theme): string | null {
+	if (tasks.length <= 1) return null;
+	const active = tasks.filter(t => t.status === "in_progress").length;
+	const pending = tasks.filter(t => t.status === "pending").length;
+	const completed = tasks.filter(t => t.status === "completed").length;
+	const parts: string[] = [];
+	if (active > 0) parts.push(`${active} active`);
+	if (pending > 0) parts.push(`${pending} pending`);
+	if (completed > 0) parts.push(`${completed} completed`);
+	if (parts.length === 0) return null;
+	return theme.fg("dim", parts.join(", "));
+}

package/src/tools/todo-write.ts

@@ -9,7 +9,6 @@ import type { Component } from "@f5xc-salesdemos/pi-tui";
 import { Text } from "@f5xc-salesdemos/pi-tui";
 import { prompt } from "@f5xc-salesdemos/pi-utils";
 import { type Static, Type } from "@sinclair/typebox";
-import chalk from "chalk";
 import type { RenderResultOptions } from "../extensibility/custom-tools/types";
 import type { Theme } from "../modes/theme/theme";
 import todoWriteDescription from "../prompts/tools/todo-write.md" with { type: "text" };
@@ -17,6 +16,7 @@ import type { ToolSession } from "../sdk";
 import type { SessionEntry } from "../session/session-manager";
 import { renderStatusLine, renderTreeList } from "../tui";
 import { PREVIEW_LIMITS } from "./render-utils";
+import { formatTodoLine, renderTodoSummary } from "./todo-render";
 
 // =============================================================================
 // Types
@@ -389,24 +389,6 @@ interface TodoWriteRenderArgs {
 	ops?: Array<{ op: string }>;
 }
 
-function formatTodoLine(item: TodoItem, uiTheme: Theme, prefix: string): string {
-	const checkbox = uiTheme.checkbox;
-	switch (item.status) {
-		case "completed":
-			return `${prefix}${uiTheme.fg("chromeAccent", checkbox.checked)} ${uiTheme.fg("dim", chalk.strikethrough(item.content))}`;
-		case "in_progress": {
-			const main = uiTheme.fg("contentAccent", `${prefix}${checkbox.unchecked} ${item.content}`);
-			if (!item.details) return main;
-			const detailLines = item.details.split("\n").map(l => uiTheme.fg("dim", `${prefix}  ${l}`));
-			return [main, ...detailLines].join("\n");
-		}
-		case "abandoned":
-			return uiTheme.fg("error", `${prefix}${checkbox.unchecked} ${chalk.strikethrough(item.content)}`);
-		default:
-			return uiTheme.fg("dim", `${prefix}${checkbox.unchecked} ${item.content}`);
-	}
-}
-
 export const todoWriteToolRenderer = {
 	renderCall(args: TodoWriteRenderArgs, _options: RenderResultOptions, uiTheme: Theme): Component {
 		const count = args.ops?.length ?? 0;
@@ -451,6 +433,10 @@ export const todoWriteToolRenderer = {
 			);
 			for (const line of treeLines) lines.push(`${indent}${line}`);
 		}
+
+		const summary = renderTodoSummary(allTasks, uiTheme);
+		if (summary !== null) lines.push(`${indent}${summary}`);
+
 		return new Text(lines.join("\n"), 0, 0);
 	},
 	mergeCallAndResult: true,