@oh-my-pi/pi-coding-agent 14.9.1 → 14.9.3

package/CHANGELOG.md

@@ -2,6 +2,66 @@
 
 ## [Unreleased]
 
+## [14.9.3] - 2026-05-10
+### Breaking Changes
+
+- Changed the `eval` tool input format to canonical `*** Begin <LANG>` ... `*** End <LANG>` cells with `*** Title`, `*** Timeout`, and `*** Reset` directives, so legacy `===== ... =====` eval inputs are no longer accepted for execution
+- Removed the `sectionSeparator` re-export from `config/prompt-templates`, so existing imports from `@oh-my-pi/pi-coding-agent/config/prompt-templates` now need to resolve `sectionSeparator` from its utility package
+
+### Added
+
+- Added support for the `*** Abort` recovery marker in eval and hashline parsing to terminate processing safely when stream corruption is detected
+- Added support for wrapping hashline edits in `*** Begin Patch` and `*** End Patch` markers so patch input with these envelopes is parsed and applied
+- Added support in the HTML export renderer for the new `*** Begin`/`*** End` eval cell format
+- Added a dedicated `[now]` prompt block to `buildSystemPrompt` output containing current date, current working directory, and required end-of-turn continuation/verification guidance
+- Added a new `[project]` prompt block wrapper around workstation and workspace context and ensured it is emitted as a separate system prompt segment
+- Added dedicated HTML rendering for `eval` tool calls, including cell-by-cell parsing of `===== ... =====` blocks with inferred Python/JS/TypeScript highlighting
+- Added dedicated rendering support for `search`, `recipe`, and `irc` tool calls in transcript exports
+- Added a collapsible `Available Tools` section with a tool count and chip-style compact tool names
+- Added macOS power assertion settings `power.preventIdleSleep`, `power.preventSystemSleep`, `power.declareUserActive`, and `power.preventDisplaySleep` so users can control what types of sleep are blocked during sessions
+
+### Changed
+
+- Kept legacy `===== ... =====` eval transcripts renderable in HTML while adding parsing for the new `*** Begin` format for newer transcripts
+- Changed the system prompt’s Bash usage guidance to explicitly forbid specific anti-patterns (`sed`/`awk` line-range reads, stderr redirects, and `| head|tail` pagination) and require using dedicated tools for those operations
+- Changed delegated subagent prompts so shared task context is now rendered only in the system-level `[context]` block, while the user-facing task message now contains only the assignment prompt text
+- Changed system prompt rendering to use block markers such as `[env]`, `[contract]`, `[role]`, `[coop]`, and `[closure]` for more explicit structural instructions
+- Changed the working-directory value in rendered prompts to use `shortenPath` before interpolation
+- Updated subagent prompt assembly to compose prompt blocks and place the `[now]` block after the subagent-specific instructions
+- Changed GitHub (`gh`) tool cards to include operation, PR, branch, and truncated query/title/body details
+- Changed tool-call output to display internal `_i` intent separately and hide it from rendered argument JSON
+- Changed `ast_edit` and `find`/`search` rendering to show resolved path values and option flags such as `limit`, `no-hidden`, and `no-reply`
+- Changed power assertion behavior to take effect only while a prompt is in flight, replacing session-level persistent assertions
+
+### Removed
+
+- Removed the unused `head` and `tail` parameters from the `bash` tool schema, along with the dead `normalizeBashCommand` / `applyHeadTail` post-processing module — output truncation is already handled by the harness's streaming tail buffer and artifact spillover, so the agent should rely on `read` (or the artifact link) instead of inline truncation pipes.
+
+### Fixed
+
+- Fixed eval tool outputs to append a truncation warning and ask users to re-issue remaining work when parsing is aborted by `*** Abort`
+- Fixed hashline parsing and input splitting to stop at `*** Abort` and ignore trailing edits after the marker
+- Fixed subagent task prompt construction so a trailing `[now]` block in the base prompts is preserved and not swallowed when rendering `subagent-system-prompt`
+- Fixed edit rendering so provided `input` text is shown in the export even without a file path
+- Fixed `args.paths` handling in `ast_edit` and `find` so multiple paths are shown as a comma-separated list
+- Fixed power assertion state handling so subsequent prompts are no longer blocked after an aborted or canceled prompt
+- Fixed IRC background exchange poll loop leaking after session disposal: `#scheduleBackgroundExchangeFlush` now stops immediately when `dispose()` is called, preventing stale `setTimeout` callbacks from firing against a torn-down agent
+
+## [14.9.2] - 2026-05-10
+### Added
+
+- Added `agentsMdFiles` to `WorkspaceTree` so AGENTS.md discovery results are returned with the workspace scan output
+
+### Changed
+
+- Changed startup workspace discovery to use one native `listWorkspace` walk for both the rendered tree and AGENTS.md directory-context candidates, removing the layered `git ls-files` orchestration and secondary AGENTS.md glob.
+
+### Fixed
+
+- Fixed AGENTS.md context discovery to include AGENTS.md files that are explicitly gitignored while still excluding AGENTS.md files under ignored directories
+- Fixed task tool renderer spamming `Tool renderer failed: undefined is not an object (evaluating 'args.tasks.length')` warnings while a `task` call was streaming in (the `tasks` array is undefined until the partial JSON parser closes it); the renderer now tolerates an absent `tasks` field and shows `0 agents` until the array arrives ([#985](https://github.com/can1357/oh-my-pi/issues/985)).
+- Fixed MCP HTTP streamable transport spamming `HTTP SSE stream error: ReadableStream already has a controller` after every JSON-RPC request whose response was returned as `text/event-stream`. The transport used to break out of the SSE iterator once the matching response was captured and then re-open `response.body` for a background drain, but the body had already been piped through a `TransformStream` and could not be re-read. The drain now runs from a single iterator that resolves the response promise inline and continues to dispatch piggybacked notifications on the same stream.
+
 ## [14.9.0] - 2026-05-10
 ### Breaking Changes
 

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "14.9.1",
+	"version": "14.9.3",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/can1357/oh-my-pi",
 	"author": "Can Boluk",
@@ -47,12 +47,12 @@
 		"@agentclientprotocol/sdk": "0.21.0",
 		"@babel/parser": "^7.29.3",
 		"@mozilla/readability": "^0.6.0",
-		"@oh-my-pi/omp-stats": "14.9.1",
-		"@oh-my-pi/pi-agent-core": "14.9.1",
-		"@oh-my-pi/pi-ai": "14.9.1",
-		"@oh-my-pi/pi-natives": "14.9.1",
-		"@oh-my-pi/pi-tui": "14.9.1",
-		"@oh-my-pi/pi-utils": "14.9.1",
+		"@oh-my-pi/omp-stats": "14.9.3",
+		"@oh-my-pi/pi-agent-core": "14.9.3",
+		"@oh-my-pi/pi-ai": "14.9.3",
+		"@oh-my-pi/pi-natives": "14.9.3",
+		"@oh-my-pi/pi-tui": "14.9.3",
+		"@oh-my-pi/pi-utils": "14.9.3",
 		"@puppeteer/browsers": "^2.13.0",
 		"@sinclair/typebox": "^0.34.49",
 		"@types/turndown": "5.0.6",

package/scripts/format-prompts.ts

@@ -16,9 +16,9 @@ import { prompt } from "@oh-my-pi/pi-utils";
  */
 import { Glob } from "bun";
 
-const PROMPTS_DIR = new URL("../src/prompts/", import.meta.url).pathname;
-const COMMIT_PROMPTS_DIR = new URL("../src/commit/prompts/", import.meta.url).pathname;
-const AGENTIC_PROMPTS_DIR = new URL("../src/commit/agentic/prompts/", import.meta.url).pathname;
+const PROMPTS_DIR = `${import.meta.dir}/../src/prompts/`;
+const COMMIT_PROMPTS_DIR = `${import.meta.dir}/../src/commit/prompts/`;
+const AGENTIC_PROMPTS_DIR = `${import.meta.dir}/../src/commit/agentic/prompts/`;
 
 const PROMPT_DIRS = [PROMPTS_DIR, COMMIT_PROMPTS_DIR, AGENTIC_PROMPTS_DIR];
 

package/src/config/prompt-templates.ts

@@ -30,11 +30,6 @@ prompt.registerHelper("jtdToTypeScript", (schema: unknown): string => {
 	}
 });
 
-// `sectionSeparator` + SECTION_SEPARATOR helper live in pi-utils/prompt so every
-// template consumer gets them registered without a coupling back to this module.
-// Re-exported here for call sites that already reference the coding-agent path.
-export { sectionSeparator } from "@oh-my-pi/pi-utils/prompt";
-
 function formatHashlineRef(lineNum: unknown, content: unknown): { num: number; text: string; ref: string } {
 	const num = typeof lineNum === "number" ? lineNum : Number.parseInt(String(lineNum), 10);
 	const raw = typeof content === "string" ? content : String(content ?? "");

package/src/config/settings-schema.ts

@@ -243,6 +243,44 @@ export const SETTINGS_SCHEMA = {
 			description: "Automatically resume the most recent session in the current directory",
 		},
 	},
+
+	// macOS power assertions (caffeinate flags). No-op on other platforms.
+	"power.preventIdleSleep": {
+		type: "boolean",
+		default: true,
+		ui: {
+			tab: "interaction",
+			label: "Prevent Idle Sleep (macOS)",
+			description: "caffeinate -i: keep the system awake while a session is open",
+		},
+	},
+	"power.preventSystemSleep": {
+		type: "boolean",
+		default: false,
+		ui: {
+			tab: "interaction",
+			label: "Prevent System Sleep on AC (macOS)",
+			description: "caffeinate -s: block all system sleep while on AC power",
+		},
+	},
+	"power.declareUserActive": {
+		type: "boolean",
+		default: false,
+		ui: {
+			tab: "interaction",
+			label: "Declare User Active (macOS)",
+			description: "caffeinate -u: keep the display lit and treat the user as active",
+		},
+	},
+	"power.preventDisplaySleep": {
+		type: "boolean",
+		default: false,
+		ui: {
+			tab: "interaction",
+			label: "Prevent Display Sleep (macOS)",
+			description: "caffeinate -d: keep the display from idle-sleeping while a session is open",
+		},
+	},
 	shellPath: { type: "string", default: undefined },
 
 	extensions: { type: "array", default: EMPTY_STRING_ARRAY },

package/src/eval/eval.lark

@@ -1,37 +1,16 @@
-%import common.LF
-%import common.WS_INLINE
-
-// Canonical Eval input. Each cell is introduced by a header line:
-//
-//   ===== <info> =====
-//
-// where each side is at least 5 equal signs. The info between the bars is
-// a list of space-separated tokens, all optional, in any order:
-//
-//   py | js | ts                  language for this cell
-//   py:"..." | js:"..." | ts:"..." language plus title shorthand
-//   id:"..."                      cell title (when language unchanged)
-//   t:<digits>(ms|s|m)?           per-cell timeout (default 30s)
-//   rst                           reset this language's kernel before running
-//
-// Everything between one header line and the next (or end of input) is
-// the cell's code, verbatim. The runtime additionally accepts content
-// before the first header as an implicit default-language cell, but that
-// is lenient fallback only and MUST NOT be relied on.
-
 start: cell+
-cell:  header LF code_line*
 
-header: BAR (WS_INLINE attr)+ WS_INLINE BAR
-      | BAR WS_INLINE? BAR
+cell: begin_cell attr* code_line* end_cell
+begin_cell: "*** Begin " LANG LF
+end_cell: "*** End Cell" LF?
 
-attr: LANG_TITLE | LANG | ID_ATTR | T_ATTR | RST_FLAG
+attr: title | timeout | reset
+title:   "*** Title: " /(.+)/ LF
+timeout: "*** Timeout: " /\d+(ms|s|m)?/ LF
+reset:   "*** Reset" LF
 
 code_line: /[^\r\n]*/ LF
 
-BAR:        /={5,}/
-LANG:       "py" | "js" | "ts"
-LANG_TITLE: ("py" | "js" | "ts") ":\"" /[^"\r\n]*/ "\""
-ID_ATTR:    "id:\"" /[^"\r\n]*/ "\""
-T_ATTR:     "t:" /\d+(ms|s|m)?/
-RST_FLAG:   "rst"
+LANG: "JS" | "TS" | "PY"
+
+%import common.LF

package/src/eval/index.ts

@@ -2,4 +2,5 @@ export * from "./backend";
 export { default as jsBackend } from "./js";
 export * from "./parse";
 export { default as pythonBackend } from "./py";
+export * from "./sniff";
 export * from "./types";