Skip to content

Expose completed MCP result files as text links - #642

Merged
kvz merged 3 commits into
mainfrom
mcp-result-links
Nov 5, 2026
Merged

kvz merged 3 commits into
mainfrom
mcp-result-links

Conversation

@kvz

@kvz kvz commented Nov 4, 2026

Copy link
Copy Markdown
Member

Gemini completed the “Video covers” Builtin but displayed no thumbnails or links because it does not use the result widget. Completed Assembly tools now return a readable file list in MCP text so clients can present the outputs directly.

  • Share the formatter across Builtin shortcuts, transloadit_create_assembly, transloadit_get_assembly_status and transloadit_wait_for_assembly.
  • List each file’s Step, name and download URL. Prefer signed URLs for private outputs, then ssl_url or url.
  • Keep the first 20 files and their metadata in text, with an omitted-file count. Preserve every result in structuredContent and keep the existing widget metadata and behavior.
  • Add a patch changeset for @transloadit/mcp-server.

Images use ordinary URL lines rather than Markdown image syntax. This keeps the download address visible in clients with different rendering support; ChatGPT continues to get its previews from the existing widget.

Validation: corepack yarn verify and corepack yarn workspace @transloadit/mcp-server check. The 20 new protocol tests cover completed/running responses on all four tool paths, multiple Steps, the cap and its boundary, metadata-only outputs, signed and fallback URLs, empty results and file labels.

Council review ran twice. Its metadata-loss and private-URL findings were reproduced with failing tests and fixed. The final suggestion to preserve an unbounded JSON fallback was intentionally declined to honor the requested 20-file text cap; an inline comment explains this choice. A live Gemini confirmation remains for the hosted deployment.

🤖 Claimed by kvz on framework1/node-sdk since 2026-11-04 · resume: see delegate run

@kvz kvz self-assigned this Nov 4, 2026
kvz and others added 2 commits November 4, 2026 22:18
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@kvz
kvz merged commit fdaae24 into main Nov 5, 2026
17 checks passed
@kvz
kvz deleted the mcp-result-links branch November 5, 2026 07:40
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