@tduyng/pi-edit-path 0.0.1 → 0.0.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tduyng/pi-edit-path",
-  "version": "0.0.1",
+  "version": "0.0.2",
   "description": "Two halves of one edit path: fuse the validation command into the edit that needs it, and name the region on disk a failed edit meant so the retry lands without a re-read.",
   "keywords": [
     "pi",

package/src/action-fusion.ts

@@ -62,7 +62,7 @@ export function coerceThenRun(value: ThenRun | string | undefined): ThenRun | un
 function commandFrom(literal: string): ThenRun {
     const trimmed = literal.trim()
     if (trimmed === '') throw new Error(NO_COMMAND)
-    if (!/^[{[]?\s*"?[A-Za-z_]\w*"?\s*:/.test(trimmed)) return { command: trimmed }
+    if (!/^[{[]/.test(trimmed)) return { command: trimmed }
     const command = literalCommand(trimmed)
     if (command === undefined) throw new Error(NO_COMMAND)
     const ran: ThenRun = { command }
@@ -79,11 +79,13 @@ function literalCommand(literal: string): string | undefined {
         parsed = undefined
     }
     if (isCommandObject(parsed)) return parsed.command.trim()
-    const match = /"?command"?\s*:\s*(?:"([^"]*)"|'([^']*)'|([^,:}]+))/.exec(literal)
+    const match = /"?command"?\s*:\s*(?:"([^"]*)"|'([^']*)'|([^,}]*))/.exec(literal)
     const command = (match?.[1] ?? match?.[2] ?? match?.[3] ?? '').trim()
     return command === '' ? undefined : command
 }
 
+const MAX_OUTPUT_CHARS = 12_000
+
 type JsonValue = null | boolean | number | string | JsonValue[] | JsonObject
 
 interface JsonObject {
@@ -124,7 +126,11 @@ export function resultText(result: AgentToolResult<unknown>): string {
 }
 
 export function fuseResults<T>(mutation: AgentToolResult<T>, ran: AgentToolResult<unknown>): AgentToolResult<T> {
-    const output = resultText(ran)
+    const raw = resultText(ran)
+    const output =
+        raw.length > MAX_OUTPUT_CHARS
+            ? `${raw.slice(-MAX_OUTPUT_CHARS)}\n[then_run output truncated to the last ${MAX_OUTPUT_CHARS} characters]`
+            : raw
     return {
         ...mutation,
         content: [...mutation.content, { type: 'text' as const, text: output ? `${SUCCEEDED}\n${output}` : SUCCEEDED }]
@@ -183,7 +189,10 @@ export default function actionFusionExtension(pi: ExtensionAPI): void {
             edits: t.array(t.object({ oldText: t.string(), newText: t.string() })),
             then_run: thenRunSchema
         }),
-        prepareArguments: (input) => (isJsonObject(input) ? normalizeEditInput(input) : input) as EditArgs,
+        prepareArguments: (input) => {
+            const prepared = editBuiltin.prepareArguments?.(input) ?? input
+            return (isJsonObject(prepared) ? normalizeEditInput(prepared) : prepared) as EditArgs
+        },
         async execute(toolCallId, input: EditArgs, signal, onUpdate, ctx) {
             const { then_run, ...mutation } = input as typeof input & { then_run?: ThenRun }
             const result = await editFor(ctx.cwd).execute(toolCallId, mutation, signal, onUpdate, ctx)