@oh-my-pi/pi-coding-agent 14.4.0 → 14.4.1

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

@@ -10,6 +10,7 @@ import {
 	truncateToWidth,
 	visibleWidth,
 } from "@oh-my-pi/pi-tui";
+import { formatBytes } from "@oh-my-pi/pi-utils";
 import { theme } from "../../modes/theme/theme";
 import { matchesAppInterrupt } from "../../modes/utils/keybinding-matchers";
 import type { SessionInfo } from "../../session/session-manager";
@@ -157,10 +158,9 @@ class SessionList implements Component {
 				lines.push(messageLine);
 			}
 
-			// Metadata line: date + message count
+			// Metadata line: date + file size
 			const modified = formatDate(session.modified);
-			const msgCount = `${session.messageCount} message${session.messageCount !== 1 ? "s" : ""}`;
-			const metadata = `  ${modified} ${theme.sep.dot} ${msgCount}`;
+			const metadata = `  ${modified} ${theme.sep.dot} ${formatBytes(session.size)}`;
 			const metadataLine = theme.fg("dim", truncateToWidth(metadata, width));
 
 			lines.push(metadataLine);

package/src/prompts/tools/ast-edit.md

@@ -14,7 +14,7 @@ Performs structural AST-aware rewrites via native ast-grep.
 </instruction>
 
 <output>
-- Replacement summary, per-file replacement counts, and change diffs
+- Replacement summary, per-file replacement counts, and change diffs as `-LINE+ID|before` / `+LINE+ID|after` lines
 - Parse issues when files cannot be processed
 </output>
 

package/src/prompts/tools/ast-grep.md

@@ -18,6 +18,7 @@ Performs structural code search using AST matching via native ast-grep.
 
 <output>
 - Grouped matches with file path, byte range, line/column ranges, metavariable captures
+- Match lines are anchor-prefixed: `LINE+ID>content` for the matched line and `LINE+ID:content` for surrounding context
 - Summary counts (`totalMatches`, `filesWithMatches`, `filesSearched`) and parse issues when present
 </output>
 

package/src/prompts/tools/atom.md

@@ -15,6 +15,10 @@ Verbs:
 - `set: ["…"]` — replace the anchor line
 - `pre: ["…"]` — insert before the anchor line (or at BOF when `loc:"^"`)
 - `post: ["…"]` — insert after the anchor line (or at EOF when `loc:"$"`)
+- `sed: "s/foo/bar/"` — sed-style substitution applied to the anchor line. **Prefer this over `set` for token-level changes**
+Flags: `g` (all occurrences), `i` (case-insensitive), `F` (literal/fixed-string, no regex).
+Delimiter is whatever character follows `s`.
+You **MUST** keep the pattern as short as possible.
 
 Combination rules:
 - On a single-anchor `loc`, you may combine `pre`, `set`, and `post` in the same entry.
@@ -26,60 +30,45 @@ Combination rules:
 All examples below reference the same file:
 
 ```ts title="a.ts"
-{{hline  1 "// @ts-ignore"}}
-{{hline  2 "const timeout = 5000;"}}
-{{hline  3 "const tag = \"DO NOT SHIP\";"}}
-{{hline  4 "const fallback = group.targetFramework || 'All Frameworks';"}}
-{{hline  5 "function alpha() {"}}
-{{hline  6 "\tlog();"}}
-{{hline  7 "}"}}
-{{hline  8 ""}}
-{{hline  9 "function beta(x) {"}}
-{{hline 10 "\tif (x) {"}}
-{{hline 11 "\t\treturn parse(data);"}}
-{{hline 12 "\t}"}}
-{{hline 13 "\treturn null;"}}
-{{hline 14 "}"}}
+{{hline 1 "const tag = \"BAD\";"}}
+{{hline 2 ""}}
+{{hline 3 "function beta(x) {"}}
+{{hline 4 "\tif (x) {"}}
+{{hline 5 "\t\treturn parse(data) || fallback;"}}
+{{hline 6 "\t}"}}
+{{hline 7 "\treturn null;"}}
+{{hline 8 "}"}}
 ```
 
-# Swap an operator by replacing the line
-Original line 4: `const fallback = group.targetFramework || 'All Frameworks';`
-`{path:"a.ts",edits:[{loc:{{href 4 "const fallback = group.targetFramework || 'All Frameworks';"}},set:["const fallback = group.targetFramework ?? 'All Frameworks';"]}]}`
-
-# Flip a literal by replacing the line
-Original line 2: `const timeout = 5000;`
-`{path:"a.ts",edits:[{loc:{{href 2 "const timeout = 5000;"}},set:["const timeout = 30_000;"]}]}`
-
-# Negate a condition by replacing the line
-Original line 10: `\tif (x) {`
-`{path:"a.ts",edits:[{loc:{{href 10 "\tif (x) {"}},set:["\tif (!x) {"]}]}`
+# Replace a line with `set`
+`{path:"a.ts",edits:[{loc:{{href 1 "const tag = \"BAD\";"}},set:["const tag = \"OK\";"]}]}`
 
 # Combine `pre` + `set` + `post` in one entry
-`{path:"a.ts",edits:[{loc:{{href 6 "\tlog();"}},pre:["\tvalidate();"],set:["\tlog();"],post:["\tcleanup();"]}]}`
-
-# Replace one whole line with `set`
-Use `set` to replace the full anchored line, preserving any unchanged surrounding lines yourself.
-`{path:"a.ts",edits:[{loc:{{href 3 "const tag = \"DO NOT SHIP\";"}},set:["const tag = \"OK\";"]}]}`
-
-# Replace multiple non-adjacent lines
-`{path:"a.ts",edits:[{loc:{{href 11 "\t\treturn parse(data);"}},set:["\t\treturn parse(data) ?? fallback;"]},{loc:{{href 13 "\treturn null;"}},set:["\treturn fallback;"]}]}`
+`{path:"a.ts",edits:[{loc:{{href 4 "\tif (x) {"}},pre:["\tvalidate();"],set:["\tif (!x) {"],post:["\t\tlog();"]}]}`
 
 # Delete a line with `set: []`
-`{path:"a.ts",edits:[{loc:{{href 11 "\t\treturn parse(data);"}},set:[]}]}`
+`{path:"a.ts",edits:[{loc:{{href 7 "\treturn null;"}},set:[]}]}`
 
 # Preserve a blank line with `set:[""]`
-`{path:"a.ts",edits:[{loc:{{href 8 ""}},set:[""]}]}`
+`{path:"a.ts",edits:[{loc:{{href 2 ""}},set:[""]}]}`
 
 # Insert before / after a line
-`{path:"a.ts",edits:[{loc:{{href 9 "function beta(x) {"}},pre:["function gamma() {","\tvalidate();","}",""]}]}`
-`{path:"a.ts",edits:[{loc:{{href 6 "\tlog();"}},post:["\tvalidate();"]}]}`
+`{path:"a.ts",edits:[{loc:{{href 3 "function beta(x) {"}},pre:["function gamma() {","\tvalidate();","}",""]}]}`
+
+# Substitute one token with `sed` (regex) — preferred for token-level edits
+Use the smallest pattern that uniquely identifies the change.
+`{path:"a.ts",edits:[{loc:{{href 5 "\t\treturn parse(data) || fallback;"}},sed:"s/\\|\\|/??/"}]}`
+
+# Substitute every occurrence with `sed` (literal/fixed-string)
+Use the `F` flag to disable regex; the delimiter can be any non-alphanumeric char.
+`{path:"a.ts",edits:[{loc:{{href 5 "\t\treturn parse(data) || fallback;"}},sed:"s|data|input|gF"}]}`
 
 # Prepend / append at file edges
 `{path:"a.ts",edits:[{loc:"^",pre:["// Copyright (c) 2026",""]}]}`
 `{path:"a.ts",edits:[{loc:"$",post:["","export const VERSION = \"1.0.0\";"]}]}`
 
 # Cross-file override inside `loc`
-`{path:"a.ts",edits:[{loc:"b.ts:{{href 2 "const timeout = 5000;"}}",set:["const timeout = 30_000;"]}]}`
+`{path:"a.ts",edits:[{loc:"b.ts:{{href 1 "const tag = \"BAD\";"}}",set:["const tag = \"OK\";"]}]}`
 </examples>
 
 <critical>
@@ -91,6 +80,8 @@ Use `set` to replace the full anchored line, preserving any unchanged surroundin
 - `set: []` deletes the anchored line. `set:[""]` preserves a blank line.
 - Within a single request you may submit edits in any order — the runtime applies them bottom-up so they don't shift each other. After any request that mutates a file, anchors below the mutation are stale on disk; re-read before issuing more edits to that file.
 - `set` operations target the current file content only. Do not try to reference old line text after the file has changed.
+- For token-level edits, prefer `sed` over `set`. The `loc` anchor already pins the line — repeating the entire line in a `set` array invites hallucinated content. Use the smallest `sed` pattern that uniquely identifies the change on that line; do not pad it with surrounding text just to feel safe.
+- When you do use `set`, re-read the anchored line first and copy it verbatim, changing only the required token(s). Anchor identity does not verify line content, so a hallucinated replacement will silently corrupt the file.
 - Text content must be literal file content with matching indentation. If the file uses tabs, use real tabs.
 - You **MUST NOT** use this tool to reformat or clean up unrelated code.
 </critical>

package/src/prompts/tools/grep.md

@@ -8,14 +8,14 @@ Searches files using powerful regex matching.
 
 <output>
 {{#if IS_HASHLINE_MODE}}
-- Text output is anchor-prefixed: `123th:content` (match) or `123th-content` (context). The 2-letter ID is a content fingerprint.
+- Text output is anchor-prefixed: `123th>content` (match) or `123th:content` (context). The 2-letter ID is a content fingerprint.
 {{else}}
 {{#if IS_LINE_NUMBER_MODE}}
 - Text output is line-number-prefixed
 {{/if}}
 {{/if}}
 {{#if IS_CHUNK_MODE}}
-- Text output is chunk-path-prefixed: `path:sel>123th:content`
+- Text output is chunk-path-prefixed: `path:sel>123|content`
 {{/if}}
 </output>
 

package/src/prompts/tools/read.md

@@ -24,7 +24,7 @@ Max {{DEFAULT_MAX_LINES}} lines per call.
 
 # Filesystem
 {{#if IS_HASHLINE_MODE}}
-- Reading from FS returns lines prefixed with anchors: `41th:def alpha():` (line number, 2-letter ID, colon, then content)
+- Reading from FS returns lines prefixed with anchors: `41th|def alpha():` (line number, 2-letter ID, pipe, then content)
 {{else}}
 {{#if IS_LINE_NUMBER_MODE}}
 - Reading from FS returns lines prefixed with line numbers: `41:def alpha():`

package/src/session/session-manager.ts

@@ -259,6 +259,8 @@ export interface SessionInfo {
 	created: Date;
 	modified: Date;
 	messageCount: number;
+	/** File size in bytes on disk; used for compact list rendering. */
+	size: number;
 	firstMessage: string;
 	allMessagesText: string;
 }
@@ -1264,7 +1266,7 @@ function extractTextFromContent(content: Message["content"]): string {
 		.join(" ");
 }
 
-const SESSION_LIST_PREFIX_BYTES = 1024;
+const SESSION_LIST_PREFIX_BYTES = 4096;
 const SESSION_LIST_PARALLEL_THRESHOLD = 64;
 const SESSION_LIST_MAX_WORKERS = 16;
 const sessionListPrefixDecoder = new TextDecoder("utf-8", { fatal: false });
@@ -1466,6 +1468,7 @@ async function collectSessionFromFile(
 			created: new Date(header.timestamp ?? ""),
 			modified: stats.mtime,
 			messageCount,
+			size: stats.size,
 			firstMessage: firstMessage || "(no messages)",
 			allMessagesText: allMessages.length > 0 ? allMessages.join(" ") : firstMessage,
 		};

package/src/tools/grep.ts

@@ -64,7 +64,7 @@ export interface GrepToolDetails {
 	truncated?: boolean;
 	error?: string;
 	/** Pre-formatted text for the user-visible TUI render. Mirrors the model-facing
-	 * `result.text` lines but uses a `│` gutter and `*` to mark match lines (vs `-` for
+	 * `result.text` lines but uses a `│` gutter and `*` to mark match lines (vs space for
 	 * context). The TUI uses this directly so it never parses model-facing hashline anchors. */
 	displayContent?: string;
 }
@@ -502,7 +502,7 @@ export class GrepTool implements AgentTool<typeof grepSchema, GrepToolDetails> {
 				}
 			}
 			if (hasContextLines && outputLines.length > 0) {
-				outputLines.unshift("[grep] match lines use ':'; context lines use '-'.");
+				outputLines.unshift("[grep] match lines use '>'; context lines use ':'.");
 			}
 			if (matchLimitReached || result.limitReached) {
 				outputLines.push("", limitMessage);

package/src/tools/match-line-format.ts

@@ -1,9 +1,9 @@
-import { computeLineHash, HASHLINE_CONTENT_SEPARATOR } from "../edit/line-hash";
+import { computeLineHash } from "../edit/line-hash";
 
 /**
  * Format a single line of match output for grep/ast-grep style results.
  *
- * Match lines use `:` as the anchor/content separator; context lines use `-`.
+ * Match lines use `>` as the anchor/content separator; context lines use `:`.
  * In hashline mode the anchor is `LINE+ID` (no `#`); in plain mode it is
  * just the line number. Line numbers are never padded.
  */
@@ -13,7 +13,7 @@ export function formatMatchLine(
 	isMatch: boolean,
 	options: { useHashLines: boolean },
 ): string {
-	const separator = isMatch ? HASHLINE_CONTENT_SEPARATOR : "-";
+	const separator = isMatch ? ">" : ":";
 	if (options.useHashLines) {
 		return `${lineNumber}${computeLineHash(lineNumber, line)}${separator}${line}`;
 	}

package/src/tools/read.ts

@@ -92,17 +92,13 @@ function prependLineNumbers(text: string, startNum: number): string {
 	return textLines.map((line, i) => `${startNum + i}|${line}`).join("\n");
 }
 
-function prependHashLines(text: string, startNum: number): string {
-	return formatHashLines(text, startNum);
-}
-
 function formatTextWithMode(
 	text: string,
 	startNum: number,
 	shouldAddHashLines: boolean,
 	shouldAddLineNumbers: boolean,
 ): string {
-	if (shouldAddHashLines) return prependHashLines(text, startNum);
+	if (shouldAddHashLines) return formatHashLines(text, startNum);
 	if (shouldAddLineNumbers) return prependLineNumbers(text, startNum);
 	return text;
 }

package/src/tools/write.ts

@@ -59,7 +59,7 @@ export interface WriteToolDetails {
 /**
  * Strip hashline display prefixes from write content.
  *
- * Only active when hashline edit mode is enabled — the model sees `LINE+ID:`
+ * Only active when hashline edit mode is enabled — the model sees `LINE+ID|`
  * prefixes in read output and sometimes copies them into write content.
  */
 function stripWriteContent(session: ToolSession, content: string): { text: string; stripped: boolean } {
@@ -418,7 +418,7 @@ export class WriteTool implements AgentTool<typeof writeSchema, WriteToolDetails
 		context?: AgentToolContext,
 	): Promise<AgentToolResult<WriteToolDetails>> {
 		return untilAborted(signal, async () => {
-			// Strip hashline display prefixes (LINE+ID:) if the model copied them from read output
+			// Strip hashline display prefixes (LINE+ID|) if the model copied them from read output
 			const { text: cleanContent, stripped } = stripWriteContent(this.session, content);
 			const resolvedArchivePath = await this.#resolveArchiveWritePath(path);
 			if (resolvedArchivePath) {