@skydiveai/pi-extensions 0.1.0-beta.2642 → 0.1.0-beta.2654

package/dist/index.d.mts

@@ -1,7 +1,7 @@
 import { A2AOptions, Middleware, ProtocolsOptions } from "@skydiveai/pi-server";
+import { ExtensionFactory } from "@earendil-works/pi-coding-agent";
 import { Type } from "typebox";
 import { Client } from "@modelcontextprotocol/sdk/client/index.js";
-import { ExtensionFactory } from "@earendil-works/pi-coding-agent";
 import { Logger } from "pino";
 
 //#region src/harness.d.ts

package/dist/index.mjs

@@ -5,6 +5,7 @@ import { buildAgentCard, chainMiddleware, composeHandlers, createAgentExecutor,
 import { mkdir, open, readFile, readdir, stat, unlink, writeFile } from "node:fs/promises";
 import { basename, dirname, join, relative, resolve } from "node:path";
 import { pathToFileURL } from "node:url";
+import { createEditToolDefinition, createLocalBashOperations } from "@earendil-works/pi-coding-agent";
 import { z } from "zod";
 import { CallToolResultSchema } from "@modelcontextprotocol/sdk/types.js";
 import { Type } from "typebox";
@@ -29,7 +30,6 @@ import { quote } from "shell-quote";
 import { createHash } from "node:crypto";
 import { createWriteStream } from "node:fs";
 import { finished } from "node:stream/promises";
-import { createLocalBashOperations } from "@earendil-works/pi-coding-agent";
 //#region src/platform-env-middleware.ts
 const ENVD_TOKEN_HEADER = "x-e2b-envd-token";
 const ENVD_URL = "http://localhost:49983/envs";
@@ -459,6 +459,13 @@ function withoutInjectedSummary(args) {
 *
 * Fails open, like the rest of this module: if the strip throws, the original
 * arguments are used rather than failing the call.
+*
+* The type constraint names only the fields this wrapper reads, rather than
+* `ToolDefinition` itself: a concretely-typed definition (e.g. the built-in
+* edit tool's `ToolDefinition<typeof editSchema, EditToolDetails, ...>`) is
+* not assignable to the default `ToolDefinition<TSchema, unknown, any>`
+* because of the render hooks' parameter variance, and this wrapper never
+* touches those hooks anyway.
 */
 function withSummaryStrippedBeforeValidation(tool) {
 	if (schemaDeclaresSummary(tool.parameters)) return tool;
@@ -526,6 +533,19 @@ const toolCallSummaryExtension = (pi) => {
 	pi.on("tool_call", (event) => {
 		stripInjectedSummary(pi, event.toolName, event.input);
 	});
+	let editOverrideRegistered = false;
+	pi.on("session_start", (_event, ctx) => {
+		if (editOverrideRegistered) return;
+		try {
+			pi.registerTool(withSummaryStrippedBeforeValidation(createEditToolDefinition(ctx.cwd)));
+			editOverrideRegistered = true;
+		} catch (err) {
+			log$15.error({
+				err,
+				event: "tool_call_summary_edit_override_failed"
+			}, "failed to register summary-tolerant edit tool; built-in remains active");
+		}
+	});
 };
 //#endregion
 //#region src/extensions/local-tools.ts
@@ -4219,7 +4239,7 @@ const TaskItem = Type.Object({
 	persona: Type.Optional(Type.String({ description: "Optional extra system prompt / role for this task, applied ON TOP of the child run's own default persona (your full identity and soul are still there underneath). Omit to run with just your default persona." })),
 	model: Type.Optional(Type.String({ description: "Optional model id to run this subagent on. PREFER A LOWER-COST, FASTER MODEL when the task is well-scoped and does not need your full reasoning depth — most delegated subtasks (searching, summarizing, mechanical edits, gathering or reformatting data, running a check) run just as well on a lighter model and cost far less. Reserve a top-tier model for subtasks that genuinely need deep reasoning or careful judgment. Must be a real catalogued model id. Omit to inherit your own model. If you are locked to a Google-compliant model, only compliant models are accepted." })),
 	timeoutMinutes: Type.Optional(Type.Integer({
-		description: "Optional wall-clock timeout for this subagent, in minutes. If the run is still going after this long it is ended and you are rewoken with a timeout result, so a hung subagent can never strand you. Omit for the default (30 minutes). Raise it for genuinely long work (a big migration, a large audit); lower it for a quick lookup. Range 1-360.",
+		description: "Optional wall-clock timeout for this subagent, in minutes. If the run is still going after this long it is ended and you are rewoken with a timeout result, so a hung subagent can never strand you. Omit for the default (30 minutes). Set a shorter timeout only when the task has a strict runtime bound; open-ended work should use the default or longer. Range 1-360.",
 		minimum: 1,
 		maximum: 360
 	}))

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skydiveai/pi-extensions",
-  "version": "0.1.0-beta.2642",
+  "version": "0.1.0-beta.2654",
   "homepage": "https://skydive.com",
   "license": "MIT",
   "author": "Create, Inc.",