Skip to content

feat: HTTP/SSE transport for agentflare's own MCP server - #644

Merged
getappz merged 1 commit into
masterfrom
task/58-idea-daemon-http-transport-for-agentflar
Aug 31, 2026
Merged

feat: HTTP/SSE transport for agentflare's own MCP server#644
getappz merged 1 commit into
masterfrom
task/58-idea-daemon-http-transport-for-agentflar

Conversation

@getappz

@getappz getappz commented Aug 30, 2026

Copy link
Copy Markdown
Owner

Summary

Adds an opt-in Streamable-HTTP transport alongside the existing stdio transport for agentflare's top-level MCP server, so one persistent agentflare process can serve multiple concurrent agent sessions instead of spawning a full process per session.

  • Arc-shared AgentflareMcp across sessions
  • Loopback-only default (no auth yet — documented limitation, not a silent gap)
  • agentflare mcp --http opt-in flag

Context

Investigated in item #58: agentflare's downstream gateway calls already reuse persistent connections (ensure_connected()), but the real bottleneck was one level up — the top-level MCP server itself only ran over stdio, so every connecting agent session cold-started its own full process with its own registries/DB connections.

Known follow-ups (not blocking, tracked separately)

  • Per-process agent identity conflates multi-agent handoff over the shared HTTP transport
  • No dedicated concurrent-request test for run_http yet

Test plan

  • Rebased cleanly onto current master (was 74 commits behind)
  • cargo build --bin agentflare — clean
  • cargo test --bin agentflare mcp_server — 250 passed, 0 failed, 2 ignored (manual benchmarks)

Closes item #58.

…nly default, 244 tests green. Follow-ups: per-process agent identity conflates multi-agent handoff; add run_http concurrent test.

Agentflare-Agent: claude-code_2-1-251_agent
Agentflare-Branch: HEAD
Agentflare-Session: 42b8c2f2-a60f-4b46-9fab-fb3d3370d30f
@getappz

getappz commented Aug 30, 2026

Copy link
Copy Markdown
Owner Author

Tracking this PR for automated review (flared:c997d745ae66).

@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 40 minutes.

View limit details

Limit details: You’ve used the included review currently available. Your 60 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: dd99d1c2-f78a-428c-99b0-b3b36e626d40

📥 Commits

Reviewing files that changed from the base of the PR and between 221e512 and 498ce7a.

📒 Files selected for processing (3)
  • Cargo.toml
  • src/cli/mcp.rs
  • src/mcp_server.rs

Comment @coderabbitai help to get the list of available commands.

@getappz
getappz merged commit 27d63b6 into master Aug 31, 2026
2 of 16 checks passed
@getappz
getappz deleted the task/58-idea-daemon-http-transport-for-agentflar branch August 31, 2026 08:53
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