feat(mcp): facade-only run mode — adapter-less [mcp] config is valid#1453
Merged
Conversation
…alid Closes #1451. An adapter-less config with [mcp] present now runs just the OAB MCP Facade listener in the foreground (one entrypoint, config-driven) instead of bailing with 'no adapter configured'. The bail message now mentions the [mcp] option. Removes the openab-agent mcp-facade subcommand — serving surfaces do not belong on the coding-agent binary (nothing shipped depends on it; #1449 established the openab-side home).
thepagent
approved these changes
Jul 25, 2026
pahud
pushed a commit
to brettchien/openab
that referenced
this pull request
Jul 25, 2026
…browser tunnel wiring alongside the [mcp] facade block; Cargo.lock regenerated
6 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #1451, per the #1449 review discussion: the standalone facade entrypoint existed only because
openab runrefused to start without a chat adapter. One entrypoint, config-driven.What changes
openab runwith no chat adapter but[mcp]present now serves just the OAB MCP Facade listener in the foreground (the facade is the workload; bind failure/server exit = process exit, fail fast). Log line announcesfacade-only mode.no adapter configuredbail message now mentions the[mcp]option.openab-agent mcp-facade— serving surfaces don't belong on the coding-agent binary; shipped only hours ago in feat(mcp): OAB MCP Facade MVP — shared openab-mcp crate + broker loopback HTTP server #1448, nothing depends on it.config.toml.exampledocuments the mode.Who needs facade-only
Coding-CLI-only hosts (no bot anywhere), provider/dev iteration loops (validated tonight — the #1449 e2e had to detour to the agent binary), and CI/agent hosts where a chat token would be absurd. Normal bot deployments are unchanged:
[discord]+[mcp]in one process remains the production path.Validation
cargo clippy --workspace --all-features -- -D warningsclean.openab run -cwith a two-line[mcp]-only config →facade-only mode+listeninglogged, MCPinitializeon the configured port returns HTTP 200; process killed cleanly.Review Contract
Goal
Make an adapter-less config with
[mcp]a validopenab rundeployment (facade-only), and remove the misplacedopenab-agent mcp-facadeserving surface.Non-goals
[discord]+[mcp]in-process facade).mcpCLI (list|status|doctor|connect|login) is untouched.Accepted Residual Risks
openab-agent mcp-facadein the hours since feat(mcp): OAB MCP Facade MVP — shared openab-mcp crate + broker loopback HTTP server #1448 merged must switch toopenab runwith an[mcp]-only config (judged: nobody).Acceptance Criteria
openab runwith[mcp]-only config serves/mcp(initialize succeeds) and logs facade-only mode; with neither adapter nor[mcp], the bail fires with the updated message.openab-agent mcp-facadeno longer parses;openab-agent mcp …client commands unaffected.Follow-ups
gmail-nativealongside the facade) — future PR per Relocate standalone mcp-facade entrypoint from openab-agent to the broker binary #1451.