Skip to content

Reject streaming agent methods from MCP export - #3807

Merged
vigoo merged 1 commit into
mainfrom
danielvigovszky/gol-94-mcp-export-streaming-method-support
Sep 3, 2026
Merged

Reject streaming agent methods from MCP export#3807
vigoo merged 1 commit into
mainfrom
danielvigovszky/gol-94-mcp-export-streaming-method-support

Conversation

@vigoo

@vigoo vigoo commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Overview

  • reject stream-bearing agent methods from MCP capability and prompt export
  • expose parameterless methods as resources or resource templates only when they are read-only; export other non-streaming methods as tools
  • add unit and end-to-end MCP listing coverage, including auto-injected parameters and recursive stream schemas

This is aligned with #3789 and does not add an MCP protocol extension or buffer streams into arrays.

Verification

  • cargo test -p golem-worker-service --lib -- agent_mcp_capability::tests --report-time
  • cargo clippy -p golem-worker-service --lib --tests -- -D warnings
  • cargo check -p golem-worker-service --all-targets
  • cargo clippy -p integration-tests --test integration --no-deps -- -D warnings
  • SQLite MCP integration tests for listing tools, resources, resource templates, and prompts
  • rebuilt the agent-mcp test component
  • package-scoped formatting and git diff --check

Resolves GOL-94

@vigoo
vigoo requested a review from a team September 1, 2026 12:33
@netlify

netlify Bot commented Sep 1, 2026

Copy link
Copy Markdown

Deploy Preview for golemcloud canceled.

Name Link
🔨 Latest commit f1e0473
🔍 Latest deploy log https://app.netlify.com/projects/golemcloud/deploys/6a96c61076d95a0008c26e11

@vigoo
vigoo merged commit 01a6e59 into main Sep 3, 2026
57 checks passed
@vigoo
vigoo deleted the danielvigovszky/gol-94-mcp-export-streaming-method-support branch September 3, 2026 09:09
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 3, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant