feat(apollo): accept URLs for FileUploadable tool parameters

@abir-taheerchecks n/achecks…abir/file-upload-apollo-integration → master25 files · +1330 −23review: @sohamganatrareview: @kaavee315review: @lingalarahul7review: @anshugarg15updated 3w ago

GitHub

▸Description · 6 noise

Description

Adds Apollo-side integration for the file-upload-relay worker (#10034). When file_urls.enable is set on a v3.1 tool router session, FileUploadable tool parameters accept { url: string, name?: string } instead of { s3key, name, mimetype }. Files are downloaded via the relay worker and rewritten to the s3key format before execution — agents just pass a URL.

New session config (v3.1 only):

{
  "file_urls": { "enable": true }
}

Schema rewriting — three paths intercepted (all gated by useFileUrls check):

COMPOSIO_SEARCH_TOOLS — in search_result_utils.ts, rewrites request_schema in search results
COMPOSIO_GET_TOOL_SCHEMAS — in getToolSchemas.ts, rewrites schemas from Thermos response
MCP tool registration — in mcp/v2/tools.ts, rewrites schemas for MCP clients

Detection via file_uploadable: true marker that Thermos/Mercury inlines directly on FileUploadable schema objects. The schema rewriter walks the entire schema tree recursively, replacing nodes that match { file_uploadable: true, type: "object" } with the URL upload schema.

Argument transformation — in executeForSessionRPC.ts, before executable.execute():

Detects FileUploadable params via recursive file_uploadable: true marker search in request schema
URL format { url: "https://..." } → relay download → { s3key, name, mimetype }
Array format [{ url }, { url }] → concurrent relay downloads → [{ s3key }, { s3key }]
Legacy format { s3key, name, mimetype } → passthrough (backward compat)
Multiple file params downloaded concurrently

Session config wiring (v3.1 only — v3 frozen):

v3.1 CREATE and PATCH accept file_urls: { enable: boolean }
v3.1 GET/PATCH response includes file_urls in config
DB schema (ToolRouterConfigDB) stores file_urls
Session read (byNanoId, fetchSessionFromDB, createToolRouterSession) maps file_urls → fileUrls

New module: apps/apollo/src/lib/toolRouterV2/features/file_uploads/

schema_rewriter.ts — walks tree, replaces file_uploadable: true objects with { url, name } schema (Object.freeze'd, immutable)
relay_client.ts — HTTP client for file-upload-relay worker (uses safeFetch, Zod-validated response, query-param-stripped URL logging)
arg_transformer.ts — detects FileUploadable params via recursive marker search, transforms URL args via relay

How did I test this PR

16 unit tests (schema rewriter + arg transformer) all passing
Type check clean (pnpm check-types — no new errors)
Lint clean (oxlint + oxfmt pass)
Verified session CREATE correctly stores and surfaces file_urls in v3.1 responses
v3 responses do NOT include file_urls (v3.1-only field)
PATCH stores and clears file_urls correctly
All review comments from Copilot, Codex, and Cursor addressed

🤖 Generated with Claude Code

▸ 6 bot/status comments hidden

@vercel[bot]3w ago

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
apollo	Ready	Preview, Comment	May 21, 2026 6:52am

Project	Deployment	Actions	Updated (UTC)
debby	Skipped		May 21, 2026 6:52am

@github-actions[bot]3w ago

🔍 Suggested Reviewers

Based on git blame analysis of 13 file(s):

Contributor	Contribution	Files
Dhawal Upadhyay	52%	7
Abir Taheer

loading diff…

Description

New session config (v3.1 only):

{ "file_urls": { "enable": true } }

Schema rewriting — three paths intercepted (all gated by useFileUrls check):

COMPOSIO_SEARCH_TOOLS — in search_result_utils.ts, rewrites request_schema in search results

COMPOSIO_GET_TOOL_SCHEMAS — in getToolSchemas.ts, rewrites schemas from Thermos response

MCP tool registration — in mcp/v2/tools.ts, rewrites schemas for MCP clients

Argument transformation — in executeForSessionRPC.ts, before executable.execute():

Detects FileUploadable params via recursive file_uploadable: true marker search in request schema

URL format { url: "https://..." } → relay download → { s3key, name, mimetype }

Array format [{ url }, { url }] → concurrent relay downloads → [{ s3key }, { s3key }]

Legacy format { s3key, name, mimetype } → passthrough (backward compat)

Multiple file params downloaded concurrently

Session config wiring (v3.1 only — v3 frozen):

v3.1 CREATE and PATCH accept file_urls: { enable: boolean }

v3.1 GET/PATCH response includes file_urls in config

DB schema (ToolRouterConfigDB) stores file_urls

Session read (byNanoId, fetchSessionFromDB, createToolRouterSession) maps file_urls → fileUrls

New module: apps/apollo/src/lib/toolRouterV2/features/file_uploads/

schema_rewriter.ts — walks tree, replaces file_uploadable: true objects with { url, name } schema (Object.freeze'd, immutable)

relay_client.ts — HTTP client for file-upload-relay worker (uses safeFetch, Zod-validated response, query-param-stripped URL logging)

arg_transformer.ts — detects FileUploadable params via recursive marker search, transforms URL args via relay

How did I test this PR

16 unit tests (schema rewriter + arg transformer) all passing

Type check clean (pnpm check-types — no new errors)

Lint clean (oxlint + oxfmt pass)

Verified session CREATE correctly stores and surfaces file_urls in v3.1 responses

v3 responses do NOT include file_urls (v3.1-only field)

PATCH stores and clears file_urls correctly

All review comments from Copilot, Codex, and Cursor addressed

🤖 Generated with Claude Code

Project

Deployment

Actions

Updated (UTC)

apollo

Ready

Preview, Comment

May 21, 2026 6:52am

Project

Deployment

Actions

Updated (UTC)

debby

Skipped

May 21, 2026 6:52am

Contributor

Contribution

Files

Dhawal Upadhyay

52%

Abir Taheer

Files with missing lines	Patch %	Lines
...toolRouterV2/features/file_uploads/relay_client.ts	11.81%	112 Missing :warning:
...lRouterV2/features/file_uploads/arg_transformer.ts	77.99%	57 Missing :warning:
...outerV2/features/execution/executeForSessionRPC.ts	20.83%	19 Missing :warning:
apps/apollo/src/lib/toolRouterV2/schema/config.ts	0.00%	11 Missing :warning:
...llo/src/lib/toolRouterV2/features/session/patch.ts	11.11%	8 Missing :warning:
...src/lib/composio_actions/actions/getToolSchemas.ts	76.00%	6 Missing :warning:
...api/v3.1/tool_router/session/[session_id]/index.ts	0.00%	3 Missing :warning:
...ib/composio_actions/actions/search_result_utils.ts	83.33%	2 Missing :warning:
...lRouterV2/features/file_uploads/schema_rewriter.ts	95.34%	2 Missing :warning:
...lRouterV2/features/session/util/sessionResponse.ts	71.42%	2 Missing :warning:
... and 3 more

Flag	Coverage Δ
e2e-tests	`6.02% <0.34%> (-0.02%)`	:arrow_down:
self-hosted-tests	`5.56% <0.34%> (-0.02%)`	:arrow_down:
unit-tests	`58.87% <61.00%> (+<0.01%)`	:arrow_up:

Files with missing lines	Coverage Δ
.../src/lib/composio_actions/actions/search_action.ts	`74.48% <100.00%> (+0.26%)`	:arrow_up:
...ib/composio_actions/actions/search_orchestrator.ts	`79.84% <100.00%> (+0.03%)`	:arrow_up:
...rc/lib/toolRouterV2/features/file_uploads/index.ts	`100.00% <100.00%> (ø)`
...2/features/session/dbUtils/getToolRouterSession.ts	`82.68% <100.00%> (+0.22%)`	:arrow_up:
...pollo/src/lib/toolRouterV2/features/session/get.ts	`95.28% <100.00%> (+0.02%)`	:arrow_up:

feat(apollo): accept URLs for FileUploadable tool parameters

Description

How did I test this PR

🔍 Suggested Reviewers

Description

How did I test this PR

🔍 Suggested Reviewers

💡 Recommendation

🧪 Test Results Summary (Self-Hosted Tests)

Codecov Report

🧪 Test Results Summary (Unit Tests)

🧪 Test Results Summary (E2E Tests)