@oh-my-pi/pi-coding-agent 14.0.2 → 14.0.4

package/src/modes/components/session-observer-overlay.ts

@@ -43,6 +43,8 @@ export class SessionObserverOverlayComponent extends Container {
 	#observeKeys: KeyId[];
 	/** Cached parsed transcript per session file to avoid reparsing on every refresh */
 	#transcriptCache?: { path: string; bytesRead: number; entries: SessionMessageEntry[] };
+	/** Live stats text component, placed after transcript to avoid above-viewport diffs */
+	#statsText?: Text;
 
 	constructor(registry: SessionObserverRegistry, onDone: () => void, observeKeys: KeyId[]) {
 		super();
@@ -87,11 +89,13 @@ export class SessionObserverOverlayComponent extends Container {
 		this.#mode = "viewer";
 		this.children = [];
 		this.#viewerContainer = new Container();
+		this.#statsText = new Text("", 1, 0);
 		this.#refreshViewer();
 
 		this.addChild(new DynamicBorder());
 		this.addChild(this.#viewerContainer);
 		this.addChild(new Spacer(1));
+		this.addChild(this.#statsText);
 		this.addChild(new Text(theme.fg("dim", "Esc: back to picker  |  Ctrl+S: back to picker"), 1, 0));
 		this.addChild(new DynamicBorder());
 	}
@@ -133,16 +137,17 @@ export class SessionObserverOverlayComponent extends Container {
 		const session = sessions.find(s => s.id === this.#selectedSessionId);
 		if (!session) {
 			this.#viewerContainer.addChild(new Text(theme.fg("dim", "Session no longer available."), 1, 0));
+			this.#updateStats(undefined);
 			return;
 		}
 
 		this.#renderSessionHeader(session);
 		this.#renderSessionTranscript(session);
+		this.#updateStats(session);
 	}
 
 	#renderSessionHeader(session: ObservableSession): void {
 		const c = this.#viewerContainer;
-		const progress = session.progress;
 
 		// Header: label + status + [agent]
 		const statusColor = session.status === "active" ? "success" : session.status === "failed" ? "error" : "dim";
@@ -154,17 +159,6 @@ export class SessionObserverOverlayComponent extends Container {
 			c.addChild(new Text(theme.fg("muted", session.description), 1, 0));
 		}
 
-		// Stats from progress
-		if (progress) {
-			const stats: string[] = [];
-			if (progress.toolCount > 0) stats.push(`${formatNumber(progress.toolCount)} tools`);
-			if (progress.tokens > 0) stats.push(`${formatNumber(progress.tokens)} tokens`);
-			if (progress.durationMs > 0) stats.push(formatDuration(progress.durationMs));
-			if (stats.length > 0) {
-				c.addChild(new Text(theme.fg("dim", stats.join(theme.sep.dot)), 1, 0));
-			}
-		}
-
 		if (session.sessionFile) {
 			c.addChild(new Text(theme.fg("dim", `Session: ${shortenPath(session.sessionFile)}`), 1, 0));
 		}
@@ -172,6 +166,21 @@ export class SessionObserverOverlayComponent extends Container {
 		c.addChild(new DynamicBorder());
 	}
 
+	/** Update live stats in-place (below transcript, within viewport). */
+	#updateStats(session: ObservableSession | undefined): void {
+		if (!this.#statsText) return;
+		const progress = session?.progress;
+		if (!progress) {
+			this.#statsText.setText("");
+			return;
+		}
+		const stats: string[] = [];
+		if (progress.toolCount > 0) stats.push(`${formatNumber(progress.toolCount)} tools`);
+		if (progress.tokens > 0) stats.push(`${formatNumber(progress.tokens)} tokens`);
+		if (progress.durationMs > 0) stats.push(formatDuration(progress.durationMs));
+		this.#statsText.setText(stats.length > 0 ? theme.fg("dim", stats.join(theme.sep.dot)) : "");
+	}
+
 	/** Incrementally read and parse the session JSONL, caching already-parsed entries. */
 	#loadTranscript(sessionFile: string): SessionMessageEntry[] | null {
 		// Invalidate cache if session file changed (e.g. switched to different subagent)

package/src/modes/theme/theme.ts

@@ -20,6 +20,8 @@ import { defaultThemes } from "./defaults";
 import lightThemeJson from "./light.json" with { type: "json" };
 import { getMermaidAscii } from "./mermaid-cache";
 
+export { getLanguageFromPath } from "../../utils/lang-from-path";
+
 // ============================================================================
 // Symbol Presets
 // ============================================================================
@@ -2305,167 +2307,6 @@ export function highlightCode(code: string, lang?: string): string[] {
 	}
 }
 
-/**
- * Get language identifier from file path extension.
- */
-export function getLanguageFromPath(filePath: string): string | undefined {
-	const baseName = path.basename(filePath).toLowerCase();
-	if (baseName === ".env" || baseName.startsWith(".env.")) return "env";
-	if (
-		baseName === ".gitignore" ||
-		baseName === ".gitattributes" ||
-		baseName === ".gitmodules" ||
-		baseName === ".editorconfig" ||
-		baseName === ".npmrc" ||
-		baseName === ".prettierrc" ||
-		baseName === ".eslintrc"
-	) {
-		return "conf";
-	}
-	if (baseName === "dockerfile" || baseName.startsWith("dockerfile.") || baseName === "containerfile") {
-		return "dockerfile";
-	}
-	if (baseName === "justfile") return "just";
-	if (baseName === "cmakelists.txt") return "cmake";
-
-	const ext = filePath.split(".").pop()?.toLowerCase();
-	if (!ext) return undefined;
-
-	const extToLang: Record<string, string> = {
-		ts: "typescript",
-		cts: "typescript",
-		mts: "typescript",
-		tsx: "tsx",
-		js: "javascript",
-		jsx: "javascript",
-		mjs: "javascript",
-		cjs: "javascript",
-		py: "python",
-		pyi: "python",
-		rb: "ruby",
-		rbw: "ruby",
-		gemspec: "ruby",
-		rs: "rust",
-		go: "go",
-		java: "java",
-		kt: "kotlin",
-		ktm: "kotlin",
-		kts: "kotlin",
-		swift: "swift",
-		c: "c",
-		h: "c",
-		cpp: "cpp",
-		cc: "cpp",
-		cxx: "cpp",
-		hh: "cpp",
-		hpp: "cpp",
-		cu: "cpp",
-		ino: "cpp",
-		cs: "csharp",
-		clj: "clojure",
-		cljc: "clojure",
-		cljs: "clojure",
-		edn: "clojure",
-		php: "php",
-		sh: "bash",
-		bash: "bash",
-		zsh: "bash",
-		ksh: "bash",
-		bats: "bash",
-		tmux: "bash",
-		cgi: "bash",
-		fcgi: "bash",
-		command: "bash",
-		tool: "bash",
-		fish: "fish",
-		ps1: "powershell",
-		psm1: "powershell",
-		sql: "sql",
-		html: "html",
-		htm: "html",
-		xhtml: "html",
-		astro: "astro",
-		vue: "vue",
-		svelte: "svelte",
-		css: "css",
-		scss: "scss",
-		sass: "sass",
-		less: "less",
-		json: "json",
-		ipynb: "ipynb",
-		hbs: "handlebars",
-		hsb: "handlebars",
-		handlebars: "handlebars",
-		yaml: "yaml",
-		yml: "yaml",
-		toml: "toml",
-		xml: "xml",
-		xsl: "xml",
-		xslt: "xml",
-		svg: "xml",
-		plist: "xml",
-		md: "markdown",
-		markdown: "markdown",
-		mdx: "markdown",
-		diff: "diff",
-		patch: "diff",
-		dockerfile: "dockerfile",
-		containerfile: "dockerfile",
-		makefile: "make",
-		justfile: "just",
-		mk: "make",
-		mak: "make",
-		cmake: "cmake",
-		lua: "lua",
-		jl: "julia",
-		pl: "perl",
-		pm: "perl",
-		perl: "perl",
-		r: "r",
-		scala: "scala",
-		sc: "scala",
-		sbt: "scala",
-		ex: "elixir",
-		exs: "elixir",
-		erl: "erlang",
-		hs: "haskell",
-		nix: "nix",
-		odin: "odin",
-		zig: "zig",
-		star: "starlark",
-		bzl: "starlark",
-		sol: "solidity",
-		v: "verilog",
-		sv: "verilog",
-		svh: "verilog",
-		vh: "verilog",
-		m: "objc",
-		mm: "objc",
-		ml: "ocaml",
-		vim: "vim",
-		graphql: "graphql",
-		proto: "protobuf",
-		tf: "hcl",
-		hcl: "hcl",
-		tfvars: "hcl",
-		txt: "text",
-		text: "text",
-		log: "log",
-		csv: "csv",
-		tsv: "tsv",
-		ini: "ini",
-		cfg: "conf",
-		conf: "conf",
-		config: "conf",
-		properties: "conf",
-		tla: "tlaplus",
-		tlaplus: "tlaplus",
-		env: "env",
-	};
-
-	return extToLang[ext];
-}
-
 export function getSymbolTheme(): SymbolTheme {
 	const preset = theme.getSymbolPreset();
 

package/src/prompts/review-request.md

@@ -62,3 +62,9 @@ _Full diff too large ({{len files}} files). Showing first ~{{linesPerFile}} line
 {{rawDiff}}
 </diff>
 {{/if}}
+
+{{#if additionalInstructions}}
+### Additional Instructions
+
+{{additionalInstructions}}
+{{/if}}

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

@@ -52,9 +52,29 @@ Push back when warranted: state the downside, propose an alternative, but **MUST
 <communication>
 - No emojis, filler, or ceremony.
 - (1) Correctness first, (2) Brevity second, (3) Politeness third.
-- User-supplied content **MUST** override any other guidelines.
+- Prefer concise, information-dense writing.
+- Avoid repeating the user's request or narrating routine tool calls.
 </communication>
 
+<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.
+- If a newer user instruction conflicts with an earlier user instruction, follow the newer one.
+- Preserve earlier instructions that do not conflict.
+</instruction-priority>
+
+<output-contract>
+- Brief preambles are allowed when they improve orientation, but they **MUST** stay short and **MUST NOT** be treated as completion.
+- Claims about code, tools, tests, docs, or external sources **MUST** be grounded in what you actually observed. If a statement is an inference, say so.
+- Apply brevity to prose, not to evidence, verification, or blocking details.
+</output-contract>
+
+<default-follow-through>
+- If the user's intent is clear and the next step is reversible and low-risk, proceed without asking.
+- Ask only when the next step is irreversible, has external side effects, or requires a missing choice that would materially change the outcome.
+- If you proceed, state what you did, what you verified, and what remains optional.
+</default-follow-through>
+
 <behavior>
 You **MUST** guard against the completion reflex — the urge to ship something that compiles before you've understood the problem:
 - Compiling ≠ Correctness. "It works" ≠ "Works in all cases".
@@ -248,6 +268,15 @@ Don't open a file hoping. Hope is not a strategy.
 {{#has tools "task"}}- `task` for investigate+edit in one pass — prefer this over a separate explore→task chain{{/has}}
 {{/ifAny}}
 
+<tool-persistence>
+- Use tools whenever they materially improve correctness, completeness, or grounding.
+- Do not stop at the first plausible answer if another tool call would materially reduce uncertainty, verify a dependency, or improve coverage.
+- Before taking an action, check whether prerequisite discovery, lookup, or memory retrieval is required. Resolve prerequisites first.
+- If a lookup is empty, partial, or suspiciously narrow, retry with a different strategy before concluding nothing exists.
+- When multiple retrieval steps are independent, parallelize them. When one result determines the next step, keep the workflow sequential.
+- After parallel retrieval, pause to synthesize before making more calls.
+</tool-persistence>
+
 {{#if (includes tools "inspect_image")}}
 ### Image inspection
 - For image understanding tasks: **MUST** use `inspect_image` over `read` to avoid overloading main session context.
@@ -262,7 +291,14 @@ These are inviolable. Violation is system failure.
 - You **MUST NOT** suppress tests to make code pass. You **MUST NOT** fabricate outputs not observed.
 - You **MUST NOT** solve the wished-for problem instead of the actual problem.
 - You **MUST NOT** ask for information obtainable from tools, repo context, or files.
-- You **MUST** always design a clean solution. You **MUST NOT** introduce unnecessary backwards compatibiltity layers, no shims, no gradual migration, no bridges to old code unless user explicitly asks for it. Let the errors guide you on what to include in the refactoring. **ALWAYS default to performing full CUTOVER!**
+- You **MUST** always design a clean solution. You **MUST NOT** introduce unnecessary backwards compatibility layers, no shims, no gradual migration, no bridges to old code unless user explicitly asks for it. Let the errors guide you on what to include in the refactoring. **ALWAYS default to performing full CUTOVER!**
+
+<completeness-contract>
+- Treat the task as incomplete until every requested deliverable is done or explicitly marked [blocked].
+- Keep an internal checklist of requested outcomes, implied cleanup, affected callsites, tests, docs, and follow-on edits.
+- For lists, batches, paginated results, or multi-file migrations, determine expected scope when possible and confirm coverage before yielding.
+- If something is blocked, label it [blocked], say exactly what is missing, and distinguish it from work that is complete.
+</completeness-contract>
 
 # Design Integrity
 
@@ -280,6 +316,7 @@ Design integrity means the code tells the truth about what the system currently
 {{#has tools "task"}}- You **MUST** determine if the task is parallelizable via `task` tool.{{/has}}
 - If multi-file or imprecisely scoped, you **MUST** write out a step-by-step plan, phased if it warrants, before touching any file.
 - For new work, you **MUST**: (1) think about architecture, (2) search official docs/papers on best practices, (3) review existing codebase, (4) compare research with codebase, (5) implement the best fit or surface tradeoffs.
+- If required context is missing, do **NOT** guess. Prefer tool-based retrieval first, ask a minimal question only when the answer cannot be recovered from tools, repo context, or files.
 ## 2. Before You Edit
 - Read the relevant section of any file before editing. Don't edit from a grep snippet alone — context above and below the match changes what the correct edit is.
 - You **MUST** grep for existing examples before implementing any pattern, utility, or abstraction. If the codebase already solves it, you **MUST** use that. Inventing a parallel convention is **PROHIBITED**.
@@ -313,6 +350,7 @@ When a tool call fails, read the full error before doing anything else. When a f
 - Test everything rigorously → Future contributor cannot break behavior without failure. Prefer unit/e2e.
 - You **MUST NOT** rely on mocks — they invent behaviors that never happen in production and hide real bugs.
 - You **SHOULD** run only tests you added/modified unless asked otherwise.
+- Before yielding, verify: (1) every requirement is satisfied, (2) claims match files/tool output/source material, (3) the output format matches the ask, and (4) any high-impact action was either verified or explicitly held for permission.
 - You **MUST NOT** yield without proof when non-trivial work, self-assessment is deceptive: tests, linters, type checks, repro steps… exhaust all external verification.
 
 {{#if secretsEnabled}}

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

@@ -7,11 +7,20 @@ Edits files via syntax-aware chunks. Run `read(path="file.ts")` first. The edit
   - replacements: `chunk#CRC` or `chunk#CRC@region`
 - Without a `@region` it defaults to the entire chunk including leading trivia. Valid regions: `head`, `body`, `tail`, `decl`.
 - If the exact chunk path is unclear, run `read(path="file", sel="?")` and copy a selector from that listing.
-- Use a single leading space per indent level in `content`. Write content at indent-level 0 — the tool re-indents it to match the chunk's position in the file. For example, to replace `@body` of a method, write the body starting at column 0:
+{{#if chunkAutoIndent}}
+- Use `\t` for indentation in `content`. Write content at indent-level 0 — the tool re-indents it to match the chunk's position in the file. For example, to replace `@body` of a method, write the body starting at column 0:
   ```
-  content: "if (x) {\n return true;\n}"
+  content: "if (x) {\n\treturn true;\n}"
   ```
   The tool adds the correct base indent automatically. Never manually pad with the chunk's own indentation.
+{{else}}
+- Match the file's literal tabs/spaces in `content`. Do not convert indentation to canonical `\t`.
+- Write content at indent-level 0 relative to the target region. For example, to replace `@body` of a method, write:
+  ```
+  content: "if (x) {\n  return true;\n}"
+  ```
+  The tool adds the correct base indent automatically, then preserves the tabs/spaces you used inside the snippet. Never manually pad with the chunk's own indentation.
+{{/if}}
 - `@region` only works on container chunks (classes, functions, impl blocks, sections). Do **not** use `@head`/`@body`/`@tail` on leaf chunks (enum variants, fields, single statements) — use the whole chunk instead.
 - `replace` requires the current CRC. Insertions do not.
 - **CRCs change after every edit.** Always use the selectors/CRCs from the most recent `read` or edit response. Never reuse a CRC from a previous edit.
@@ -19,6 +28,10 @@ Edits files via syntax-aware chunks. Run `read(path="file.ts")` first. The edit
 
 <critical>
 You **MUST** use the narrowest region that covers your change. Replacing without a region replaces the **entire chunk including leading comments, decorators, and attributes** — omitting them from `content` deletes them.
+
+**`replace` is total, not surgical.** The `content` you supply becomes the *complete* new content for the targeted region. Everything in the original region that you omit from `content` is deleted. Before replacing `@body` on any chunk, verify the chunk does not contain children you intend to keep. If a chunk spans hundreds of lines and your change touches only a few, target a specific child chunk — not the parent.
+
+**Group chunks (`stmts_*`, `imports_*`, `decls_*`) are containers.** They hold many sibling items (test functions, import statements, declarations). Replacing `@body` on a group chunk replaces **all** of its children. To edit one item inside a group, target that item's own chunk path. If no child chunk exists, use the specific child's chunk selector from `read` output — do not replace the parent group.
 </critical>
 
 <regions>
@@ -108,9 +121,13 @@ Given this `read` output for `example.ts`:
 ```
 
 **Replace a whole chunk** (rename a function):
-```
-{ "sel": "fn_createCounter#PQQY", "op": "replace", "content": "function makeCounter(start: number): Counter {\n const c = new Counter();\n c.value = start;\n return c;\n}\n" }
-```
+~~~json
+{{#if chunkAutoIndent}}
+{ "sel": "fn_createCounter#PQQY", "op": "replace", "content": "function makeCounter(start: number): Counter {\n\tconst c = new Counter();\n\tc.value = start;\n\treturn c;\n}\n" }
+{{else}}
+{ "sel": "fn_createCounter#PQQY", "op": "replace", "content": "function makeCounter(start: number): Counter {\n  const c = new Counter();\n  c.value = start;\n  return c;\n}\n" }
+{{/if}}
+~~~
 Result — the entire chunk is rewritten:
 ```
 function makeCounter(start: number): Counter {
@@ -158,9 +175,13 @@ function createCounter(initial: number): Counter {
 ```
 
 **Insert after a chunk** (`after`):
-```
-{ "sel": "enum_Status", "op": "after", "content": "\nfunction isActive(s: Status): boolean {\n return s === Status.Active;\n}\n" }
-```
+~~~json
+{{#if chunkAutoIndent}}
+{ "sel": "enum_Status", "op": "after", "content": "\nfunction isActive(s: Status): boolean {\n\treturn s === Status.Active;\n}\n" }
+{{else}}
+{ "sel": "enum_Status", "op": "after", "content": "\nfunction isActive(s: Status): boolean {\n  return s === Status.Active;\n}\n" }
+{{/if}}
+~~~
 Result — a new function appears after the enum:
 ```
 enum Status {
@@ -189,9 +210,13 @@ class Counter {
 ```
 
 **Append inside a container** (`@body` + `append`):
-```
-{ "sel": "class_Counter@body", "op": "append", "content": "\nreset(): void {\n this.value = 0;\n}\n" }
-```
+~~~json
+{{#if chunkAutoIndent}}
+{ "sel": "class_Counter@body", "op": "append", "content": "\nreset(): void {\n\tthis.value = 0;\n}\n" }
+{{else}}
+{ "sel": "class_Counter@body", "op": "append", "content": "\nreset(): void {\n  this.value = 0;\n}\n" }
+{{/if}}
+~~~
 Result — a new method is added at the end of the class body, before the closing `}`:
 ```
   toString(): string {
@@ -210,10 +235,18 @@ Result — a new method is added at the end of the class body, before the closin
 ```
 Result — the method is removed from the class.
 - Indentation rules (important):
-  - Use one leading space for each indent level in canonical chunk-edit content. The tool expands those levels to the file's actual style (2-space, 4-space, tabs, etc.).
+{{#if chunkAutoIndent}}
+  - Use `\t` for each indent level. The tool converts tabs to the file's actual style (2-space, 4-space, etc.).
+{{else}}
+  - Match the file's real indentation characters in your snippet. The tool preserves your literal tabs/spaces after adding the target region's base indent.
+{{/if}}
   - Do NOT include the chunk's base indentation — only indent relative to the region's opening level.
   - For `@body` of a function: write at column 0, e.g. `"return x;\n"`. The tool adds the correct base indent.
   - For `@head`: write at the chunk's own depth. A class member's head uses `"/** doc */\nstart(): void {"`.
-  - For a top-level item: start at zero indent. Write `"function foo() {\n return 1;\n}\n"`.
+{{#if chunkAutoIndent}}
+  - For a top-level item: start at zero indent. Write `"function foo() {\n\treturn 1;\n}\n"`.
+{{else}}
+  - For a top-level item: start at zero indent. Write `"function foo() {\n  return 1;\n}\n"`.
+{{/if}}
   - The tool strips common leading indentation from your content as a safety net, so accidental over-indentation is corrected.
 </examples>

package/src/prompts/tools/read-chunk.md

@@ -9,6 +9,12 @@ Each opening anchor `[< full.chunk.path#CCCC ]` in the default output identifies
 If you need a canonical target list, run `read(path="file", sel="?")`. That listing shows chunk paths with CRCs.
 Line numbers in the gutter are absolute file line numbers.
 
+{{#if chunkAutoIndent}}
+Chunk reads normalize leading indentation so copied content round-trips cleanly into chunk edits.
+{{else}}
+Chunk reads preserve literal leading tabs/spaces from the file. When editing, keep the same whitespace characters you see here.
+{{/if}}
+
 Chunk trees: JS, TS, TSX, Python, Rust, Go. Others use blank-line fallback.
 </instruction>
 

package/src/sdk.ts

@@ -15,6 +15,7 @@ import { SearchDb } from "@oh-my-pi/pi-natives";
 import type { Component } from "@oh-my-pi/pi-tui";
 import {
 	$env,
+	$flag,
 	getAgentDbPath,
 	getAgentDir,
 	getProjectDir,
@@ -1262,7 +1263,7 @@ export async function createAgentSession(options: CreateAgentSessionOptions = {}
 
 	const repeatToolDescriptions = settings.get("repeatToolDescriptions");
 	const eagerTasks = settings.get("task.eager");
-	const intentField = settings.get("tools.intentTracing") || $env.PI_INTENT_TRACING === "1" ? INTENT_FIELD : undefined;
+	const intentField = settings.get("tools.intentTracing") || $flag("PI_INTENT_TRACING") ? INTENT_FIELD : undefined;
 	const rebuildSystemPrompt = async (toolNames: string[], tools: Map<string, AgentTool>): Promise<string> => {
 		toolContextStore.setToolNames(toolNames);
 		const discoverableMCPTools = mcpDiscoveryEnabled ? collectDiscoverableMCPTools(tools.values()) : [];

package/src/tools/ast-edit.ts

@@ -3,7 +3,7 @@ import type { AgentTool, AgentToolContext, AgentToolResult, AgentToolUpdateCallb
 import { type AstReplaceChange, astEdit } from "@oh-my-pi/pi-natives";
 import type { Component } from "@oh-my-pi/pi-tui";
 import { Text } from "@oh-my-pi/pi-tui";
-import { prompt, untilAborted } from "@oh-my-pi/pi-utils";
+import { $envpos, prompt, untilAborted } from "@oh-my-pi/pi-utils";
 import { type Static, Type } from "@sinclair/typebox";
 import { computeLineHash } from "../edit/line-hash";
 import type { RenderResultOptions } from "../extensibility/custom-tools/types";
@@ -103,7 +103,7 @@ export class AstEditTool implements AgentTool<typeof astEditSchema, AstEditToolD
 			if (maxReplacements !== undefined && (!Number.isFinite(maxReplacements) || maxReplacements < 1)) {
 				throw new ToolError("limit must be a positive number");
 			}
-			const maxFiles = parseInt(process.env.PI_MAX_AST_FILES ?? "", 10) || 1000;
+			const maxFiles = $envpos("PI_MAX_AST_FILES", 1000);
 
 			const formatScopePath = (targetPath: string): string => {
 				const relative = path.relative(this.session.cwd, targetPath).replace(/\\/g, "/");

package/src/tools/index.ts

@@ -1,7 +1,7 @@
 import type { AgentTool } from "@oh-my-pi/pi-agent-core";
 import type { ToolChoice } from "@oh-my-pi/pi-ai";
 import type { SearchDb } from "@oh-my-pi/pi-natives";
-import { $env, logger } from "@oh-my-pi/pi-utils";
+import { $env, $flag, isBunTestRuntime, logger } from "@oh-my-pi/pi-utils";
 import type { AsyncJobManager } from "../async";
 import type { PromptTemplate } from "../config/prompt-templates";
 import type { Settings } from "../config/settings";
@@ -297,8 +297,7 @@ export async function createTools(session: ToolSession, toolNames?: string[]): P
 		!skipPythonPreflight &&
 		pythonMode !== "bash-only" &&
 		(requestedTools === undefined || requestedTools.includes("python"));
-	const isTestEnv = Bun.env.BUN_ENV === "test" || Bun.env.NODE_ENV === "test";
-	const skipPythonWarm = isTestEnv || $env.PI_PYTHON_SKIP_CHECK === "1";
+	const skipPythonWarm = isBunTestRuntime() || $flag("PI_PYTHON_SKIP_CHECK");
 	if (shouldCheckPython) {
 		const availability = await logger.time("createTools:pythonCheck", checkPythonKernelAvailability, session.cwd);
 		pythonAvailable = availability.ok;

package/src/tools/read.ts

@@ -14,6 +14,7 @@ import {
 	parseChunkReadPath,
 	parseChunkSelector,
 	resolveAnchorStyle,
+	resolveChunkAutoIndent,
 } from "../edit/modes/chunk";
 import type { RenderResultOptions } from "../extensibility/custom-tools/types";
 import { parseInternalUrl } from "../internal-urls/parse";
@@ -449,6 +450,7 @@ export class ReadTool implements AgentTool<typeof readSchema, ReadToolDetails> {
 			resolveEditMode(session) === "chunk"
 				? prompt.render(readChunkDescription, {
 						anchorStyle: resolveAnchorStyle(session.settings),
+						chunkAutoIndent: resolveChunkAutoIndent(),
 					})
 				: prompt.render(readDescription, {
 						DEFAULT_LIMIT: String(this.#defaultLimit),

package/src/tools/report-tool-issue.ts

@@ -8,7 +8,7 @@
 import { Database } from "bun:sqlite";
 import path from "node:path";
 import type { AgentTool } from "@oh-my-pi/pi-agent-core";
-import { $env, getAgentDir, logger, VERSION } from "@oh-my-pi/pi-utils";
+import { $flag, getAgentDir, logger, VERSION } from "@oh-my-pi/pi-utils";
 import { Type } from "@sinclair/typebox";
 import type { Settings } from "..";
 import type { ToolSession } from "./index";
@@ -19,7 +19,7 @@ const ReportToolIssueParams = Type.Object({
 });
 
 export function isAutoQaEnabled(settings?: Settings): boolean {
-	return $env.PI_AUTO_QA === "1" || !!settings?.get("dev.autoqa");
+	return $flag("PI_AUTO_QA") || !!settings?.get("dev.autoqa");
 }
 
 export function getAutoQaDbPath(): string {

package/src/utils/edit-mode.ts

@@ -1,4 +1,4 @@
-import { $env } from "@oh-my-pi/pi-utils";
+import { $env, $flag } from "@oh-my-pi/pi-utils";
 
 export type EditMode = "replace" | "patch" | "hashline" | "chunk";
 
@@ -36,7 +36,7 @@ export function resolveEditMode(session: EditModeSessionLike): EditMode {
 	const envMode = normalizeEditMode($env.PI_EDIT_VARIANT);
 	if (envMode) return envMode;
 
-	if ($env.PI_STRICT_EDIT_MODE === "1") {
+	if (!$flag("PI_STRICT_EDIT_MODE")) {
 		if (activeModel?.includes("spark")) return "replace";
 		if (activeModel?.includes("nano")) return "replace";
 		if (activeModel?.includes("mini")) return "replace";