@oh-my-pi/pi-coding-agent 14.6.6 → 14.7.0

package/CHANGELOG.md

@@ -1,6 +1,47 @@
 # Changelog
 
 ## [Unreleased]
+### Added
+
+- Added `pr_create` operation to the GitHub tool to create pull requests with title/body (or `fill`), base/head branch, draft, reviewer, assignee, and label options and return a summarized result including the new PR URL
+
+## [14.7.0] - 2026-05-04
+### Breaking Changes
+
+- Changed session system-prompt APIs to use ordered string block arrays by requiring `buildSystemPrompt`, `CreateAgentSessionOptions.systemPrompt`, `Session.rebuildSystemPrompt`, and extension `before_agent_start`/`getSystemPrompt` hooks to accept and return `systemPrompt: string[]` instead of a plain system-prompt string or separate `projectPrompt` field
+- Changed `buildSystemPrompt` and session `rebuildSystemPrompt` APIs to return `{ systemPrompt, projectPrompt }`, requiring callers expecting a plain system prompt string to update to the new shape
+- Removed the top-level `sel` parameter from the `read` tool schema, requiring callers to migrate to `path`-embedded selectors (for example `path:50-100`, `path:raw`, or `https://...:L1-L40`)
+
+### Added
+
+- Added a separate `projectPrompt` artifact containing per-session project context (workstation, context files, AGENTS.md rules, workspace tree, and append prompt) so dynamic context is decoupled from the static system prompt
+- Added `Project prompt` token accounting to context-usage breakdowns and charts
+- Added `tools.elideFileMutationInputs` setting to optionally elide large `write`, `edit`, and `apply_patch` payloads in history after successful mutations
+- Added hashline-style return data for elided `write` calls so tools can include the resulting file content without leaking full input text
+- Added `buildDirectoryTree` and `DirectoryTree` exports to generate configurable directory trees with options for depth, entry limits, hidden-file handling, and truncation caps
+- Added `buildWorkspaceTree` and `WorkspaceTree` exports so callers can precompute and pass a workspace context to prompt generation
+- Added `workspaceTree` support to `buildSystemPrompt` options to reuse a prebuilt directory snapshot
+- Added `read.summarize.enabled`, `read.summarize.minBodyLines`, and `read.summarize.minCommentLines` settings to control whether `read` returns structural summaries and how many multiline body/comment lines are collapsed
+- Added `edit.hashlineAutoDropPureInsertDuplicates` setting to opt into dropping 2+ pure-insert hashline payload lines that duplicate adjacent file context; default is `false`.
+
+### Changed
+
+- Updated session dump and HTML export output to serialize ordered system-prompt blocks (including project context) and removed the dedicated project-prompt dump section
+- Renamed context-usage system-prompt accounting from a separate `projectPrompt` bucket to `systemContext` to match the new multi-block prompt structure
+- Changed prompt delivery to inject non-empty `projectPrompt` as a leading `developer` message before conversation messages instead of merging it into the base system prompt
+- Added `projectPrompt` to session dumps to expose the injected per-session project context separately
+- Changed write success output and preview rendering to display hashline-formatted written content from captured file text when mutation inputs are elided
+- Changed `read` directory rendering to return a two-level recency-sorted directory tree (including nested folders) instead of a flat alphabetical entry list, while still applying configurable truncation
+- Changed generated system prompts to include a working-directory tree block after directory context, showing recent files/directories (depth ≤ 3) and truncation notices when entries are elided
+- Changed `read` summary rendering to merge opening- and closing-brace boundaries around elided sections into a single `..` line (including closers like `};` or `})`), reducing those segments to one concise anchored summary line
+- Changed default `read` output for parseable code files without an explicit selector to return a structural summary instead of full verbatim lines, while still supporting full output for `:raw` and explicit ranges
+- Changed truncation/pagination hints in read, archive, and SQLite outputs to use colon syntax (`Use :<offset>`) when continuing reads
+- Changed the read tool UI preview title to include summary elision counts when a summary is returned
+- Changed hashline pure-insert duplicate auto-drop to be opt-in through `edit.hashlineAutoDropPureInsertDuplicates` instead of always enabled.
+
+### Fixed
+
+- Fixed selector parsing for colon-containing paths by only splitting `:<sel>` when the suffix matches a valid line-range or `raw` pattern, preventing paths like `db.sqlite:users:42` from being misread as selectors
 
 ## [14.6.6] - 2026-05-04
 

package/examples/hooks/handoff.ts

@@ -94,7 +94,7 @@ export default function (pi: HookAPI) {
 
 					const response = await complete(
 						ctx.model!,
-						{ systemPrompt: SYSTEM_PROMPT, messages: [userMessage] },
+						{ systemPrompt: [SYSTEM_PROMPT], messages: [userMessage] },
 						{ apiKey, signal: loader.signal },
 					);
 

package/examples/hooks/qna.ts

@@ -85,7 +85,7 @@ export default function (pi: HookAPI) {
 
 					const response = await complete(
 						ctx.model!,
-						{ systemPrompt: SYSTEM_PROMPT, messages: [userMessage] },
+						{ systemPrompt: [SYSTEM_PROMPT], messages: [userMessage] },
 						{ apiKey, signal: loader.signal },
 					);
 

package/examples/sdk/03-custom-prompt.ts

@@ -7,8 +7,10 @@ import { createAgentSession, SessionManager } from "@oh-my-pi/pi-coding-agent";
 
 // Option 1: Replace prompt entirely
 const { session: session1 } = await createAgentSession({
-	systemPrompt: `You are a helpful assistant that speaks like a pirate.
+	systemPrompt: [
+		`You are a helpful assistant that speaks like a pirate.
 Always end responses with "Arrr!"`,
+	],
 	sessionManager: SessionManager.inMemory(),
 });
 
@@ -24,11 +26,12 @@ console.log("\n");
 
 // Option 2: Modify default prompt (receives default, returns modified)
 const { session: session2 } = await createAgentSession({
-	systemPrompt: defaultPrompt => `${defaultPrompt}
-
-## Additional Instructions
+	systemPrompt: defaultPrompt => [
+		...defaultPrompt,
+		`## Additional Instructions
 - Always be concise
 - Use bullet points when listing things`,
+	],
 	sessionManager: SessionManager.inMemory(),
 });
 

package/examples/sdk/README.md

@@ -87,7 +87,7 @@ const { session } = await createAgentSession({
 	model,
 	authStorage: customAuth,
 	modelRegistry: customRegistry,
-	systemPrompt: "You are helpful.",
+	systemPrompt: ["You are helpful."],
 	toolNames: ["read", "bash"],
 	customTools: [{ tool: myTool }],
 	hooks: [{ factory: myHook }],

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "14.6.6",
+	"version": "14.7.0",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/can1357/oh-my-pi",
 	"author": "Can Boluk",
@@ -46,12 +46,12 @@
 	"dependencies": {
 		"@agentclientprotocol/sdk": "0.20.0",
 		"@mozilla/readability": "^0.6.0",
-		"@oh-my-pi/omp-stats": "14.6.6",
-		"@oh-my-pi/pi-agent-core": "14.6.6",
-		"@oh-my-pi/pi-ai": "14.6.6",
-		"@oh-my-pi/pi-natives": "14.6.6",
-		"@oh-my-pi/pi-tui": "14.6.6",
-		"@oh-my-pi/pi-utils": "14.6.6",
+		"@oh-my-pi/omp-stats": "14.7.0",
+		"@oh-my-pi/pi-agent-core": "14.7.0",
+		"@oh-my-pi/pi-ai": "14.7.0",
+		"@oh-my-pi/pi-natives": "14.7.0",
+		"@oh-my-pi/pi-tui": "14.7.0",
+		"@oh-my-pi/pi-utils": "14.7.0",
 		"@puppeteer/browsers": "^2.13.0",
 		"@sinclair/typebox": "^0.34.49",
 		"@xterm/headless": "^6.0.0",

package/src/autoresearch/index.ts

@@ -356,54 +356,58 @@ export const createAutoresearchExtension: ExtensionFactory = api => {
 				? null
 				: "Heads up: you are not on a dedicated `autoresearch/*` branch. `log_experiment discard` will only revert run-modified files, not reset to baseline — so harness files written before `init_experiment` may not survive a discard. Clean the worktree and re-run `/autoresearch` if you want full revert safety.";
 			return {
-				systemPrompt: prompt.render(setupPromptTemplate, {
-					base_system_prompt: event.systemPrompt,
+				systemPrompt: [
+					prompt.render(setupPromptTemplate, {
+						base_system_prompt: event.systemPrompt.join("\n\n"),
+						has_goal: goal.trim().length > 0,
+						goal,
+						working_dir: ctx.cwd,
+						has_branch: Boolean(currentBranch),
+						branch: currentBranch ?? "",
+						has_baseline_warning: baselineWarning !== null,
+						baseline_warning: baselineWarning ?? "",
+					}),
+				],
+			};
+		}
+		return {
+			systemPrompt: [
+				prompt.render(promptTemplate, {
+					base_system_prompt: event.systemPrompt.join("\n\n"),
 					has_goal: goal.trim().length > 0,
 					goal,
 					working_dir: ctx.cwd,
-					has_branch: Boolean(currentBranch),
-					branch: currentBranch ?? "",
-					has_baseline_warning: baselineWarning !== null,
-					baseline_warning: baselineWarning ?? "",
+					default_metric_name: state.metricName,
+					metric_name: state.metricName,
+					has_branch: Boolean(state.branch),
+					branch: state.branch,
+					has_baseline_commit: Boolean(state.baselineCommit),
+					baseline_commit: state.baselineCommit ? state.baselineCommit.slice(0, 12) : "",
+					has_notes: state.notes.trim().length > 0,
+					notes: state.notes,
+					current_segment: state.currentSegment + 1,
+					current_segment_run_count: currentSegmentResults.length,
+					has_baseline_metric: baselineMetric !== null,
+					baseline_metric_display: formatNum(baselineMetric, state.metricUnit),
+					baseline_run_number: baselineRunNumber,
+					has_best_result: bestResult !== null && bestMetric !== null,
+					best_metric_display: bestMetric !== null ? formatNum(bestMetric, state.metricUnit) : "-",
+					best_run_number: bestResult ? (bestResult.runNumber ?? state.results.indexOf(bestResult) + 1) : null,
+					has_recent_results: recentResults.length > 0,
+					recent_results: recentResults,
+					has_unjustified_runs: unjustifiedRuns.length > 0,
+					unjustified_runs: unjustifiedRuns,
+					has_pending_run: Boolean(pendingRun),
+					pending_run_number: pendingRun?.runNumber,
+					pending_run_command: pendingRun?.command,
+					pending_run_passed: pendingRun?.passed ?? false,
+					has_pending_run_metric: pendingRun?.parsedPrimary !== null && pendingRun?.parsedPrimary !== undefined,
+					pending_run_metric_display:
+						pendingRun?.parsedPrimary !== null && pendingRun?.parsedPrimary !== undefined
+							? formatNum(pendingRun.parsedPrimary, state.metricUnit)
+							: null,
 				}),
-			};
-		}
-		return {
-			systemPrompt: prompt.render(promptTemplate, {
-				base_system_prompt: event.systemPrompt,
-				has_goal: goal.trim().length > 0,
-				goal,
-				working_dir: ctx.cwd,
-				default_metric_name: state.metricName,
-				metric_name: state.metricName,
-				has_branch: Boolean(state.branch),
-				branch: state.branch,
-				has_baseline_commit: Boolean(state.baselineCommit),
-				baseline_commit: state.baselineCommit ? state.baselineCommit.slice(0, 12) : "",
-				has_notes: state.notes.trim().length > 0,
-				notes: state.notes,
-				current_segment: state.currentSegment + 1,
-				current_segment_run_count: currentSegmentResults.length,
-				has_baseline_metric: baselineMetric !== null,
-				baseline_metric_display: formatNum(baselineMetric, state.metricUnit),
-				baseline_run_number: baselineRunNumber,
-				has_best_result: bestResult !== null && bestMetric !== null,
-				best_metric_display: bestMetric !== null ? formatNum(bestMetric, state.metricUnit) : "-",
-				best_run_number: bestResult ? (bestResult.runNumber ?? state.results.indexOf(bestResult) + 1) : null,
-				has_recent_results: recentResults.length > 0,
-				recent_results: recentResults,
-				has_unjustified_runs: unjustifiedRuns.length > 0,
-				unjustified_runs: unjustifiedRuns,
-				has_pending_run: Boolean(pendingRun),
-				pending_run_number: pendingRun?.runNumber,
-				pending_run_command: pendingRun?.command,
-				pending_run_passed: pendingRun?.passed ?? false,
-				has_pending_run_metric: pendingRun?.parsedPrimary !== null && pendingRun?.parsedPrimary !== undefined,
-				pending_run_metric_display:
-					pendingRun?.parsedPrimary !== null && pendingRun?.parsedPrimary !== undefined
-						? formatNum(pendingRun.parsedPrimary, state.metricUnit)
-						: null,
-			}),
+			],
 		};
 	});
 

package/src/cli/read-cli.ts

@@ -0,0 +1,58 @@
+/**
+ * Read CLI command handler.
+ *
+ * Handles `omp read` — invokes the `read` agent tool against a path/URL and
+ * prints the resulting content blocks exactly as the model would receive them
+ * (including truncation/limit notices appended by the meta-notice wrapper).
+ */
+import { getProjectDir } from "@oh-my-pi/pi-utils";
+import chalk from "chalk";
+import { Settings } from "../config/settings";
+import type { ToolSession } from "../tools";
+import { wrapToolWithMetaNotice } from "../tools/output-meta";
+import { ReadTool } from "../tools/read";
+import { renderError } from "../tools/tool-errors";
+
+export interface ReadCommandArgs {
+	path: string;
+	timeout?: number;
+}
+
+export async function runReadCommand(cmd: ReadCommandArgs): Promise<void> {
+	if (!cmd.path) {
+		process.stderr.write(chalk.red("error: path is required\n"));
+		process.exit(1);
+	}
+
+	const cwd = getProjectDir();
+	const settings = await Settings.init({ cwd });
+
+	const session: ToolSession = {
+		cwd,
+		hasUI: false,
+		settings,
+		getSessionFile: () => null,
+		getSessionSpawns: () => "*",
+	};
+
+	const tool = wrapToolWithMetaNotice(new ReadTool(session));
+
+	try {
+		const result = await tool.execute("omp-read", { path: cmd.path, timeout: cmd.timeout });
+
+		for (const block of result.content) {
+			if (block.type === "text") {
+				process.stdout.write(block.text);
+				if (!block.text.endsWith("\n")) process.stdout.write("\n");
+			} else if (block.type === "image") {
+				const decodedBytes = Buffer.from(block.data, "base64").byteLength;
+				process.stdout.write(
+					chalk.dim(`[image content: ${block.mimeType}, ${decodedBytes} bytes base64-decoded]\n`),
+				);
+			}
+		}
+	} catch (err) {
+		process.stderr.write(`${chalk.red(renderError(err))}\n`);
+		process.exit(1);
+	}
+}

package/src/cli.ts

@@ -53,6 +53,7 @@ const commands: CommandEntry[] = [
 	{ name: "plugin", load: () => import("./commands/plugin").then(m => m.default) },
 	{ name: "setup", load: () => import("./commands/setup").then(m => m.default) },
 	{ name: "shell", load: () => import("./commands/shell").then(m => m.default) },
+	{ name: "read", load: () => import("./commands/read").then(m => m.default) },
 	{ name: "ssh", load: () => import("./commands/ssh").then(m => m.default) },
 	{ name: "stats", load: () => import("./commands/stats").then(m => m.default) },
 	{ name: "update", load: () => import("./commands/update").then(m => m.default) },

package/src/commands/read.ts

@@ -0,0 +1,40 @@
+/**
+ * Show what the read tool will return for a given path.
+ */
+import { Args, Command, Flags } from "@oh-my-pi/pi-utils/cli";
+import { type ReadCommandArgs, runReadCommand } from "../cli/read-cli";
+import { initTheme } from "../modes/theme/theme";
+
+export default class Read extends Command {
+	static description = "Show what the read tool will return for a path or URL";
+
+	static args = {
+		path: Args.string({
+			description: "Path or URL to read (append :sel for line ranges or raw mode, e.g. src/foo.ts:50-100)",
+			required: true,
+		}),
+	};
+
+	static flags = {
+		timeout: Flags.integer({ description: "Request timeout in seconds (URLs only)" }),
+	};
+
+	static examples = [
+		"omp read src/foo.ts",
+		"omp read src/foo.ts:50-100",
+		"omp read src/foo.ts:raw",
+		"omp read https://example.com",
+		"omp read path/to/archive.zip:dir/file.ts",
+		"omp read path/to/db.sqlite:users:42",
+	];
+
+	async run(): Promise<void> {
+		const { args, flags } = await this.parse(Read);
+		const cmd: ReadCommandArgs = {
+			path: args.path ?? "",
+			timeout: flags.timeout,
+		};
+		await initTheme();
+		await runReadCommand(cmd);
+	}
+}

package/src/commit/agentic/agent.ts

@@ -60,7 +60,7 @@ export async function runCommitAgentSession(input: CommitAgentInput): Promise<Co
 		settings: input.settings,
 		model: input.model,
 		thinkingLevel: input.thinkingLevel,
-		systemPrompt,
+		systemPrompt: [systemPrompt],
 		customTools: tools,
 		enableLsp: false,
 		enableMCP: false,

package/src/commit/analysis/conventional.ts

@@ -89,7 +89,7 @@ export async function generateConventionalAnalysis({
 	const response = await completeSimple(
 		model,
 		{
-			systemPrompt: prompt.render(analysisSystemPrompt),
+			systemPrompt: [prompt.render(analysisSystemPrompt)],
 			messages: [{ role: "user", content: userContent, timestamp: Date.now() }],
 			tools: [ConventionalAnalysisTool],
 		},

package/src/commit/analysis/summary.ts

@@ -53,7 +53,7 @@ export async function generateSummary({
 	const response = await completeSimple(
 		model,
 		{
-			systemPrompt,
+			systemPrompt: [systemPrompt],
 			messages: [{ role: "user", content: userPrompt, timestamp: Date.now() }],
 			tools: [SummaryTool],
 		},

package/src/commit/changelog/generate.ts

@@ -58,7 +58,7 @@ export async function generateChangelogEntries({
 	const response = await completeSimple(
 		model,
 		{
-			systemPrompt: prompt.render(changelogSystemPrompt),
+			systemPrompt: [prompt.render(changelogSystemPrompt)],
 			messages: [{ role: "user", content: userContent, timestamp: Date.now() }],
 			tools: [changelogTool],
 		},

package/src/commit/map-reduce/map-phase.ts

@@ -62,7 +62,7 @@ export async function runMapPhase({
 			context_header: contextHeader,
 		});
 		const request = {
-			systemPrompt,
+			systemPrompt: [systemPrompt],
 			messages: [{ role: "user", content: userContent, timestamp: Date.now() }] as Message[],
 		};
 

package/src/commit/map-reduce/reduce-phase.ts

@@ -76,7 +76,7 @@ export async function runReducePhase({
 	const response = await completeSimple(
 		model,
 		{
-			systemPrompt: prompt.render(reduceSystemPrompt),
+			systemPrompt: [prompt.render(reduceSystemPrompt)],
 			messages: [{ role: "user", content: userContent, timestamp: Date.now() }],
 			tools: [ReduceTool],
 		},

package/src/config/settings-schema.ts

@@ -1419,6 +1419,15 @@ export const SETTINGS_SCHEMA = {
 		},
 	},
 
+	"edit.hashlineAutoDropPureInsertDuplicates": {
+		type: "boolean",
+		default: false,
+		ui: {
+			tab: "editing",
+			label: "Hashline Duplicate Insert Drop",
+			description: "Drop 2+ pure-insert payload lines that duplicate adjacent file context",
+		},
+	},
 	"edit.blockAutoGenerated": {
 		type: "boolean",
 		default: true,
@@ -1466,6 +1475,36 @@ export const SETTINGS_SCHEMA = {
 		},
 	},
 
+	"read.summarize.enabled": {
+		type: "boolean",
+		default: true,
+		ui: {
+			tab: "editing",
+			label: "Read Summaries",
+			description: "Return structural code summaries when read is called without an explicit selector",
+		},
+	},
+
+	"read.summarize.minBodyLines": {
+		type: "number",
+		default: 4,
+		ui: {
+			tab: "editing",
+			label: "Read Summary Body Lines",
+			description: "Minimum multiline body or literal length before read summaries collapse it",
+		},
+	},
+
+	"read.summarize.minCommentLines": {
+		type: "number",
+		default: 6,
+		ui: {
+			tab: "editing",
+			label: "Read Summary Comment Lines",
+			description: "Minimum multiline block comment length before read summaries collapse it",
+		},
+	},
+
 	"read.toolResultPreview": {
 		type: "boolean",
 		default: false,

package/src/edit/line-hash.ts

@@ -783,18 +783,48 @@ export const HL_BODY_SEP = "|";
 export const HL_BODY_SEP_RE_RAW = regexEscape(HL_BODY_SEP);
 
 const RE_SIGNIFICANT = /[\p{L}\p{N}]/u;
+const RE_STRUCTURAL_STRIP = /[\s{}]/g;
+
+/**
+ * Bigram returned for lines that contain only whitespace and `{`/`}`.
+ * Picks the English ordinal suffix for the line number (`1` → `st`,
+ * `2` → `nd`, `3` → `rd`, `11`/`12`/`13` → `th`, else `th`) so the
+ * line digits + bigram BPE-merge into a single ordinal token (`1st`, `42nd`,
+ * `100th`, …). Brace-only lines therefore cost one token for the whole
+ * `LINE+ID` anchor instead of two.
+ */
+function structuralBigram(line: number): string {
+	const mod100 = line % 100;
+	if (mod100 >= 11 && mod100 <= 13) return "th";
+	switch (line % 10) {
+		case 1:
+			return "st";
+		case 2:
+			return "nd";
+		case 3:
+			return "rd";
+		default:
+			return "th";
+	}
+}
 
 /**
  * Compute a 2-character hash of a single line via xxHash32 mod 647 over
- * {@link HL_BIGRAMS}. Lines with no letter or digit (e.g. bare `}`,
- * bare `{`) mix the line number into the seed so adjacent identical
- * brace-only lines get distinct hashes; lines with significant content stay
- * line-number-independent so a line is identifiable across small shifts.
+ * {@link HL_BIGRAMS}. Lines that contain only whitespace and `{`/`}` collapse
+ * to an ordinal-suffix bigram (see {@link structuralBigram}) so brace-only
+ * structure shares one merged ordinal token (`1st`, `42nd`, `100th`, …).
+ * Other lines with no letter or digit mix the line number into the seed so
+ * adjacent identical punctuation-only lines get distinct hashes; lines with
+ * significant content stay line-number-independent so a line is identifiable
+ * across small shifts.
  *
  * The line input should not include a trailing newline.
  */
 export function computeLineHash(idx: number, line: string): string {
 	line = line.replace(/\r/g, "").trimEnd();
+	if (line.replace(RE_STRUCTURAL_STRIP, "").length === 0) {
+		return structuralBigram(idx);
+	}
 	const seed = RE_SIGNIFICANT.test(line) ? 0 : idx;
 	return HL_BIGRAMS[Bun.hash.xxHash32(line, seed) % HL_BIGRAMS_COUNT];
 }