Skip to content

feat(tools): route and redact fetch tool results - #6

Merged
konglyyy merged 1 commit into
mainfrom
feat/simple-fetch-tool
Jul 24, 2026
Merged

feat(tools): route and redact fetch tool results#6
konglyyy merged 1 commit into
mainfrom
feat/simple-fetch-tool

Conversation

@konglyyy

@konglyyy konglyyy commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • dispatch acornops_fetch through the existing platform-function client
  • give Fetch reads a bounded client timeout that exceeds the control-plane executor timeout
  • redact Fetch query values and response data from durable run events
  • add contract and regression coverage for routing, timeout, and redaction

Validation

  • task validate: 193 tests passed
  • Ruff, contract, and harness checks passed
  • workspace platform-contract check passed

Related PRs and merge order

  1. control-plane — feat(agents): add secure fetch native tool control-plane#17
  2. execution-engine (this PR)feat(tools): route and redact fetch tool results #6
  3. management-console — feat(agents): configure fetch native tool management-console#19
  4. docs-website — docs(tools): document fetch native tool docs-website#12

Documentation

Updates the execution-engine contract manifest. User-facing documentation is delivered in the related docs PR.

Residual risk

No new transport is introduced; Fetch continues through the existing platform-function boundary. The timeout margin assumes the control-plane keeps its documented 15-second Fetch deadline.

@konglyyy
konglyyy marked this pull request as ready for review July 24, 2026 10:40
@konglyyy
konglyyy merged commit 9f30eb4 into main Jul 24, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant