@oh-my-pi/pi-coding-agent 12.7.1 → 12.7.3

package/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 ## [Unreleased]
 
+## [12.7.3] - 2026-02-16
+
+### Fixed
+- Suppressed hashline output for subagents without the edit tool (e.g. explore agents)
+
 ## [12.7.1] - 2026-02-16
 ### Changed
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "12.7.1",
+	"version": "12.7.3",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"bin": {
@@ -84,12 +84,12 @@
 	},
 	"dependencies": {
 		"@mozilla/readability": "0.6.0",
-		"@oh-my-pi/omp-stats": "12.7.1",
-		"@oh-my-pi/pi-agent-core": "12.7.1",
-		"@oh-my-pi/pi-ai": "12.7.1",
-		"@oh-my-pi/pi-natives": "12.7.1",
-		"@oh-my-pi/pi-tui": "12.7.1",
-		"@oh-my-pi/pi-utils": "12.7.1",
+		"@oh-my-pi/omp-stats": "12.7.3",
+		"@oh-my-pi/pi-agent-core": "12.7.3",
+		"@oh-my-pi/pi-ai": "12.7.3",
+		"@oh-my-pi/pi-natives": "12.7.3",
+		"@oh-my-pi/pi-tui": "12.7.3",
+		"@oh-my-pi/pi-utils": "12.7.3",
 		"@sinclair/typebox": "^0.34.48",
 		"@xterm/headless": "^6.0.0",
 		"ajv": "^8.18.0",

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

@@ -110,26 +110,17 @@ Don't open a file hoping. Hope is not a strategy.
 
 <procedure>
 ## Execution
-
-**Assess scope first.** 
+**Assess scope first.**
 {{#if skills.length}}- If a skill matches the domain, read it before starting.{{/if}}
 {{#if rules.length}}- If an applicable rule exists, read it before starting.{{/if}}
 {{#has tools "task"}}- Consider if the task is parallelizable via Task tool? Make a conflict-free plan to delegate to subagents if possible.{{/has}}
 - If the task is multi-file or ambiguous, write a 3–7 bullet plan.
-
 **Do the work.**
-Every turn must include at least one tool call that advances the deliverable (edit, write, run, delegate, etc.). Planning and tracking do not count.
-
-**After each tool call**: 
-- Interpret result
-- Decide next action
-- Execute. No echoing output.
-
-**If blocked**: 
+Every turn must advance towards the deliverable, edit, write, run, delegate.
+**If blocked**:
 - Exhaust tools/context/files first.
 - Only then ask — minimum viable question.
-
-**If requested change includes refactor**: 
+**If requested change includes refactor**:
 Cleanup dead code and unused elements, do not yield before the codebase is pristine.
 
 {{#has tools "todo_write"}}
@@ -285,13 +276,13 @@ User works in a high-reliability industry—defense, finance, healthcare, infras
 </stakes>
 
 <prime_directive>
-**GET THE WORK DONE.** 
-Everything else is subordinate to producing the requested output. If you find yourself emitting metadata (todos, plans, status updates) without having made a single edit or produced a single artifact, you have failed.
+**GET THE WORK DONE.**
+Everything else is subordinate to producing the requested output. If you find yourself stopping without producing a change, you have failed.
 </prime_directive>
 
 <critical>
 Keep going until finished.
-- Every turn must advance the deliverable. Metadata-only turns are failures.
+- Every turn must advance the deliverable.
 - Quote only what's needed; rest is noise.
 - Don't claim unverified correctness.
 - Do not ask when it may be obtained from available tools or repo context/files.

package/src/sdk.ts

@@ -701,6 +701,9 @@ export async function createAgentSession(options: CreateAgentSessionOptions = {}
 		cwd,
 		hasUI: options.hasUI ?? false,
 		enableLsp,
+		get hasEditTool() {
+			return !options.toolNames || options.toolNames.includes("edit");
+		},
 		skipPythonPreflight: options.skipPythonPreflight,
 		contextFiles,
 		skills,

package/src/session/agent-session.ts

@@ -1072,6 +1072,11 @@ export class AgentSession {
 		return this.agent.state.tools.map(t => t.name);
 	}
 
+	/** Whether the edit tool is registered in this session. */
+	get hasEditTool(): boolean {
+		return this.#toolRegistry.has("edit");
+	}
+
 	/**
 	 * Get a tool by name from the registry.
 	 */
@@ -1518,7 +1523,7 @@ export class AgentSession {
 			if (fileMentions.length > 0) {
 				const fileMentionMessages = await generateFileMentionMessages(fileMentions, this.sessionManager.getCwd(), {
 					autoResizeImages: this.settings.get("images.autoResize"),
-					useHashLines: resolveFileDisplayMode(this.settings).hashLines,
+					useHashLines: resolveFileDisplayMode(this).hashLines,
 				});
 				messages.push(...fileMentionMessages);
 			}

package/src/tools/grep.ts

@@ -62,7 +62,7 @@ export class GrepTool implements AgentTool<typeof grepSchema, GrepToolDetails> {
 	readonly parameters = grepSchema;
 
 	constructor(private readonly session: ToolSession) {
-		const displayMode = resolveFileDisplayMode(session.settings);
+		const displayMode = resolveFileDisplayMode(session);
 		this.description = renderPromptTemplate(grepDescription, {
 			IS_HASHLINE_MODE: displayMode.hashLines,
 			IS_LINE_NUMBER_MODE: !displayMode.hashLines && displayMode.lineNumbers,
@@ -103,7 +103,7 @@ export class GrepTool implements AgentTool<typeof grepSchema, GrepToolDetails> {
 			const patternHasNewline = normalizedPattern.includes("\n") || normalizedPattern.includes("\\n");
 			const effectiveMultiline = multiline ?? patternHasNewline;
 
-			const useHashLines = resolveFileDisplayMode(this.session.settings).hashLines;
+			const useHashLines = resolveFileDisplayMode(this.session).hashLines;
 			const searchPath = resolveToCwd(searchDir || ".", this.session.cwd);
 			const scopePath = (() => {
 				const relative = path.relative(this.session.cwd, searchPath).replace(/\\/g, "/");

package/src/tools/index.ts

@@ -119,6 +119,8 @@ export interface ToolSession {
 	promptTemplates?: PromptTemplate[];
 	/** Whether LSP integrations are enabled */
 	enableLsp?: boolean;
+	/** Whether the edit tool is available in this session (controls hashline output) */
+	hasEditTool?: boolean;
 	/** Event bus for tool/extension communication */
 	eventBus?: EventBus;
 	/** Output schema for structured completion (subagents) */

package/src/tools/read.ts

@@ -544,7 +544,7 @@ export class ReadTool implements AgentTool<typeof readSchema, ReadToolDetails> {
 	readonly #autoResizeImages: boolean;
 
 	constructor(private readonly session: ToolSession) {
-		const displayMode = resolveFileDisplayMode(session.settings);
+		const displayMode = resolveFileDisplayMode(session);
 		this.#autoResizeImages = session.settings.get("images.autoResize");
 		this.description = renderPromptTemplate(readDescription, {
 			DEFAULT_MAX_LINES: String(DEFAULT_MAX_LINES),
@@ -562,7 +562,7 @@ export class ReadTool implements AgentTool<typeof readSchema, ReadToolDetails> {
 	): Promise<AgentToolResult<ReadToolDetails>> {
 		const { path: readPath, offset, limit } = params;
 
-		const displayMode = resolveFileDisplayMode(this.session.settings);
+		const displayMode = resolveFileDisplayMode(this.session);
 
 		// Handle internal URLs (agent://, skill://)
 		const internalRouter = this.session.internalRouter;
@@ -848,7 +848,7 @@ export class ReadTool implements AgentTool<typeof readSchema, ReadToolDetails> {
 	async #handleInternalUrl(url: string, offset?: number, limit?: number): Promise<AgentToolResult<ReadToolDetails>> {
 		const internalRouter = this.session.internalRouter!;
 
-		const displayMode = resolveFileDisplayMode(this.session.settings);
+		const displayMode = resolveFileDisplayMode(this.session);
 
 		// Check if URL has query extraction (agent:// only)
 		let parsed: URL;

package/src/utils/file-display-mode.ts

@@ -1,25 +1,34 @@
 /**
  * Resolve line-display mode for file-like outputs (read, grep, @file mentions).
  */
-export interface FileDisplayModeSettings {
-	get(key: "readLineNumbers" | "readHashLines" | "edit.mode"): unknown;
-}
 
 export interface FileDisplayMode {
 	lineNumbers: boolean;
 	hashLines: boolean;
 }
 
+/** Session-like object providing settings and tool availability for display mode resolution. */
+export interface FileDisplayModeSession {
+	/** Whether the edit tool is available. Hashlines are suppressed without it. */
+	hasEditTool?: boolean;
+	settings: {
+		get(key: "readLineNumbers" | "readHashLines" | "edit.mode"): unknown;
+	};
+}
+
 /**
- * Computes effective line display mode from settings/env.
+ * Computes effective line display mode from session settings/env.
  * Hashline mode takes precedence and implies line-addressed output everywhere.
+ * Hashlines are suppressed when the edit tool is not available (e.g. explore agents).
  */
-export function resolveFileDisplayMode(settings: FileDisplayModeSettings): FileDisplayMode {
+export function resolveFileDisplayMode(session: FileDisplayModeSession): FileDisplayMode {
+	const { settings } = session;
+	const hasEditTool = session.hasEditTool ?? true;
 	const hashLines =
-		settings.get("readHashLines") === true ||
-		settings.get("edit.mode") === "hashline" ||
-		Bun.env.PI_EDIT_VARIANT === "hashline";
-
+		hasEditTool &&
+		(settings.get("readHashLines") === true ||
+			settings.get("edit.mode") === "hashline" ||
+			Bun.env.PI_EDIT_VARIANT === "hashline");
 	return {
 		hashLines,
 		lineNumbers: hashLines || settings.get("readLineNumbers") === true,