one-code-extension 0.4.0 → 0.5.0

package/extensions/notebook/index.ts

@@ -1,26 +1,26 @@
 /**
- * notebook extension — Claude Code's NotebookEdit, deferred behind tool_search
+ * notebook extension — Claude Code's NotebookEdit (its text: description.ts), deferred behind tool_search
  * (most sessions never touch a notebook, so its schema stays out of the prompt
  * until needed).
  */
 
 import { randomUUID } from "node:crypto";
 import { readFileSync, writeFileSync } from "node:fs";
-import { isAbsolute, resolve } from "node:path";
 import { StringEnum } from "@earendil-works/pi-ai";
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { Type } from "typebox";
 import { DEFER_CHANNEL } from "../lib/deferred.ts";
 import { ccToolRenderers } from "../lib/tui-render.ts";
 import { applyEdit, type EditMode, type Notebook, notebookErrorHint, parseNotebook } from "./notebook.ts";
+import { resolveToolPath } from "../lib/tool-path.ts";
+import { NOTEBOOK_EDIT_DESCRIPTION } from "./description.ts";
 
 export default function notebookExtension(pi: ExtensionAPI) {
 	pi.registerTool({
 		name: "notebook_edit",
 		label: "Notebook Edit",
 		...ccToolRenderers("Notebook Edit"),
-		description:
-			"Edit a Jupyter notebook (.ipynb): replace a cell's source, insert or append a new cell, or delete a cell. Read the notebook first — like the other file tools, this one refuses to edit a file you have not read. Cells are addressed by their `id` or, for a notebook whose cells have none, by position — `cell-0` is the first cell. Use edit_mode `append` to add a cell at the end without naming one. Editing a code cell clears its outputs.",
+		description: NOTEBOOK_EDIT_DESCRIPTION,
 		parameters: Type.Object({
 			path: Type.Optional(Type.String({ description: "Absolute or workspace-relative path to the .ipynb file" })),
 			// Claude Code's NotebookEdit spells the path `notebook_path` and defaults
@@ -52,7 +52,8 @@ export default function notebookExtension(pi: ExtensionAPI) {
 				};
 			}
 			const editMode = (params.edit_mode ?? "replace") as EditMode;
-			const path = isAbsolute(rawPath) ? rawPath : resolve(ctx.cwd, rawPath);
+			// The file tracker guards this path with the same resolution (lib/tool-path.ts).
+			const path = resolveToolPath(rawPath, ctx.cwd);
 			// Held outside the try so a failure raised AFTER the parse can name the
 			// ids the notebook actually has — a bare "cell_id is required" left both
 			// weak tiers patching ids in by hand with bash (review M4).

package/extensions/notebook/notebook.ts

@@ -93,7 +93,7 @@ export function findCellIndex(notebook: Notebook, cellId: string): number {
 
 /**
  * The "fail loud, name the fix" error for an id that resolves to nothing
- * (docs/decisions/tools.md): which ids exist, and the two ways to add a cell
+ * (working-docs/decisions/tools.md): which ids exist, and the two ways to add a cell
  * without naming one.
  */
 export function noSuchCellError(notebook: Notebook, cellId: string): Error {