bob_karrot
2ec23dc7e1
MCP tools can return non-string results (e.g. structured JSON objects). When this happens, output.output is undefined, causing TypeError crashes in edit-error-recovery and delegate-task-retry hooks that call methods like .toLowerCase() without checking the type first. Add typeof string guard in both hooks, consistent with the existing pattern used in tool-output-truncator.
23 lines
722 B
TypeScript
23 lines
722 B
TypeScript
import type { PluginInput } from "@opencode-ai/plugin"
|
|
|
|
import { buildRetryGuidance } from "./guidance"
|
|
import { detectDelegateTaskError } from "./patterns"
|
|
|
|
export function createDelegateTaskRetryHook(_ctx: PluginInput) {
|
|
return {
|
|
"tool.execute.after": async (
|
|
input: { tool: string; sessionID: string; callID: string },
|
|
output: { title: string; output: string; metadata: unknown }
|
|
) => {
|
|
if (input.tool.toLowerCase() !== "task") return
|
|
if (typeof output.output !== "string") return
|
|
|
|
const errorInfo = detectDelegateTaskError(output.output)
|
|
if (errorInfo) {
|
|
const guidance = buildRetryGuidance(errorInfo)
|
|
output.output += `\n${guidance}`
|
|
}
|
|
},
|
|
}
|
|
}
|