@oh-my-pi/pi-coding-agent 14.5.11 → 14.5.12

package/src/tools/plan-mode-guard.ts

@@ -1,3 +1,4 @@
+import * as path from "node:path";
 import { resolveLocalUrlToPath } from "../internal-urls";
 import type { ToolSession } from ".";
 import { normalizeLocalScheme, resolveToCwd } from "./path-utils";
@@ -5,7 +6,7 @@ import { ToolError } from "./tool-errors";
 
 const LOCAL_SCHEME_PREFIX = "local:";
 
-export function resolvePlanPath(session: ToolSession, targetPath: string): string {
+function resolveRawPath(session: ToolSession, targetPath: string): string {
 	const normalized = normalizeLocalScheme(targetPath);
 	if (normalized.startsWith(LOCAL_SCHEME_PREFIX)) {
 		return resolveLocalUrlToPath(normalized, {
@@ -17,6 +18,31 @@ export function resolvePlanPath(session: ToolSession, targetPath: string): strin
 	return resolveToCwd(normalized, session.cwd);
 }
 
+/**
+ * Resolve a write/edit target to its absolute filesystem path.
+ *
+ * In plan mode, transparently redirects targets whose basename matches the
+ * plan file's basename (e.g. a bare `PLAN.md` or `./PLAN.md`) to the canonical
+ * plan file location at `state.planFilePath`. This lets `write` and `edit`
+ * accept the unqualified plan filename and have the change land at the
+ * session-scoped `local://PLAN.md` artifact instead of a stray cwd-relative
+ * file the plan-mode guard would otherwise reject.
+ *
+ * Outside plan mode (or when the basename does not match) this is a no-op.
+ */
+export function resolvePlanPath(session: ToolSession, targetPath: string): string {
+	const resolved = resolveRawPath(session, targetPath);
+
+	const state = session.getPlanModeState?.();
+	if (!state?.enabled) return resolved;
+
+	const planResolved = resolveRawPath(session, state.planFilePath);
+	if (resolved === planResolved) return resolved;
+	if (path.basename(resolved) !== path.basename(planResolved)) return resolved;
+
+	return planResolved;
+}
+
 export function enforcePlanModeWrite(
 	session: ToolSession,
 	targetPath: string,

package/src/tools/renderers.ts

@@ -14,6 +14,7 @@ import { askToolRenderer } from "./ask";
 import { astEditToolRenderer } from "./ast-edit";
 import { astGrepToolRenderer } from "./ast-grep";
 import { bashToolRenderer } from "./bash";
+import { browserToolRenderer } from "./browser/render";
 import { calculatorToolRenderer } from "./calculator";
 import { debugToolRenderer } from "./debug";
 import { findToolRenderer } from "./find";
@@ -49,6 +50,7 @@ export const toolRenderers: Record<string, ToolRenderer> = {
 	ast_grep: astGrepToolRenderer as ToolRenderer,
 	ast_edit: astEditToolRenderer as ToolRenderer,
 	bash: bashToolRenderer as ToolRenderer,
+	browser: browserToolRenderer as ToolRenderer,
 	recipe: recipeToolRenderer as ToolRenderer,
 	debug: debugToolRenderer as ToolRenderer,
 	python: pythonToolRenderer as ToolRenderer,