Skip to content

Fix/acp agent proxy remote - #10693

Open
Felipenguim wants to merge 3 commits into
marimo-team:mainfrom
Felipenguim:fix/acp-agent-proxy-remote
Open

Fix/acp agent proxy remote#10693
Felipenguim wants to merge 3 commits into
marimo-team:mainfrom
Felipenguim:fix/acp-agent-proxy-remote

Conversation

@Felipenguim

Copy link
Copy Markdown
Contributor

📝 Summary

Closes #8531

The ACP agent panel built its WebSocket URL from a hardcoded per-agent port:

ws://<hostname>:3017/message   # 3017 for claude

That assumes the browser can reach the agent directly. Behind a reverse proxy —
where only marimo's own port is routed — the connection never establishes, so
the agent panel is unusable in any hosted or containerized deployment.

Fixing this needs changes in th frontend and bakcend. The existing community patch (phisignet/marimo#1)
had only the frontend half, and its author noted it does not work on its own.
Both were implemented here.

Before: browser → agent (direct, port 3017)
After: browser → marimo server (/acp/<agent_id>) → agent (127.0.0.1:<port>/message)

The connection is now same-origin and only needs the port marimo is already
served on.

📋 Pre-Review Checklist

  • For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on Discord, or the community discussions (Please provide a link if applicable).
  • Any AI generated code has been reviewed line-by-line by the human PR author, who stands by it.
  • [] Video or media evidence is provided for any visual changes (optional).

✅ Merge Checklist

  • I have read the contributor guidelines.
  • Documentation has been updated where applicable, including docstrings for API changes.
  • Tests have been added for the changes made.

@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
marimo-docs Ready Ready Preview Aug 28, 2026 4:41pm

Request Review

@github-actions

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

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.

Feature: Proxy ACP agent WebSocket through marimo backend for remote/proxied deployments

1 participant