[ben-reitz]

The exec tool starts every run without metadata, and the AI SDK and TanStack adapters pass only the abort signal on, so an app using the tool sets has no way to tell host modules who a run is for.

createExecTool now takes metadata and every tool set takes execMetadata: a record of strings, or a function that picks one from the tool call's id, which every agent library passes on. A throw from the function fails the call before anything runs.

const tools = createAITools({
  workspace,
  execMetadata: ({ toolCallId }) => ({ conversation: this.name, toolCall: toolCallId }),
});

A pi caller runs each call itself, so it can also pass metadata with the call: execute(call, { metadata }). When both are given, metadata passed with the call wins outright and the function isn't called, since the caller that passed it knows that call best. defineExec accepts either on each execute.

The function gets only { toolCallId }, not each library's whole call. The known callers need a value per call (pi passes it) or per agent (closed over), and the call id covers linking runs to tool calls. Handing over the AI SDK's or TanStack's own call object can be added later without breaking anyone.

Tests cover each adapter, a static record, the precedence rule, and a picker that throws. docs/09_tool_interface.md has a new section on run metadata.

[changeset-bot]

🦋 Changeset detected

Latest commit: a4e28b0

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages

Name	Type
@cloudflare/computer	Minor
@cloudflare/dofs	Minor
@cloudflare/computer-rpc	Minor
@cloudflare/computerd	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/@cloudflare/computer@292

commit: a4e28b0

[devin-ai-integration]

Note

Newer findings are available below. Devin Review posted a newer report on this PR, in addition to the findings presented here.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

[aron-cf]

I think I'd rather this was exec: { metadata: ... }

[ben-reitz]

@aron-cf exec is just used for the backend definitions right now.

createAITools({
  workspace,
  exec: { javascript: {}, container: { description: "…" } }, // backends, keyed by id
  execOutput: { maxBytes, maxLines },
  execMetadata: ({ toolCallId }) => ({ … }),                  // added in this pr
});

we could change the api to something like this instead which is a bit nicer than what we have right now?

createAITools({
  workspace,
  exec: { 
    backends: { javascript: {}, container: { description: "…" } }, // backends, keyed by id
    output: { maxBytes, maxLines },
    metadata: ({ toolCallId }) => ({ … }),                  // added in this pr
  }
});

[ben-reitz]

#300 i've opened this in case we want to make this change.