Skip to content

feat(mcp): expose running HTTP listeners through status - #1732

Open
mariusvniekerk wants to merge 1 commit into
mainfrom
feat/mcp-listener-status
Open

feat(mcp): expose running HTTP listeners through status#1732
mariusvniekerk wants to merge 1 commit into
mainfrom
feat/mcp-listener-status

Conversation

@mariusvniekerk

Copy link
Copy Markdown
Collaborator

agentsview mcp status --json reports running HTTP MCP listeners, including their actual bound ports, backend targets, and private token-file paths. Clients can discover an existing listener instead of guessing its port. Status does not start a daemon or print bearer tokens.

Listeners publish after binding and remove their records on orderly shutdown. Status omits dead processes; stdio sessions do not appear in the list.

MCP clients need the actual listening port and backend target to connect to an existing HTTP server. Fixed port guesses do not work when several servers run or when a listener binds port zero.

Publish local listener records after binding and expose them through mcp status. Keep bearer values in private files so status output can locate credentials without printing them.
@roborev-ci

roborev-ci Bot commented Sep 11, 2026

Copy link
Copy Markdown

roborev: Combined Review (db8d5ac)

Verdict: Changes require fixes for 1 finding.

Medium

  • internal/mcpdiscovery/discovery.go:34: Wildcard binds publish URLs such as http://0.0.0.0:/mcp or http://[::]:/mcp, which are not reliable client-connectable addresses, especially with port zero. Generate the client URL separately from the bind address, mapping wildcard hosts to loopback for local status output or exposing a usable configured host; construct it with net.JoinHostPort.

    Reported by: codex


Reviewers: 2 done | Synthesis: codex, 9s | Total: 6m55s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant