-
Notifications
You must be signed in to change notification settings - Fork 0
The operator console over stdio, and the registry it is now listed in #51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -41,9 +41,14 @@ answers from these pages rather than from memory. | |
|
|
||
| ## Discovery: the registries | ||
|
|
||
| The gateway is what you run to put CTRLRun in front of an MCP server, and it is listed in the | ||
| MCP registries as such once the listing is live. Until then this section carries no link rather | ||
| than a placeholder; the registry entry is a launch-day item. | ||
| The operator server is the MCP server this project publishes, and it is listed in the | ||
| [official MCP registry](https://registry.modelcontextprotocol.io/v0/servers/io.github.CTRLRun%2Fctrlrun-mcp-operator/versions) | ||
| as `io.github.CTRLRun/ctrlrun-mcp-operator`, and on [Glama](https://glama.ai/mcp/servers/CTRLRun/ctrlrun). | ||
| A client that installs from a registry gets `ctrlrun mcp-operator --stdio`, which is the | ||
| configuration in [Approve from your assistant](/docs/mcp/approve-from-your-assistant). | ||
|
|
||
| The gateway is not listed, and the reason is what it is: a proxy you run in front of an MCP | ||
| server you already have, not a server a registry can point a client at. | ||
|
|
||
| ## Answering: the operator server | ||
|
|
||
|
|
@@ -66,8 +71,9 @@ approver's assistant ──▶ ctrlrun mcp-operator ──▶ the same appro | |
| binding, single use and expiry. | ||
| - Nothing here can make an agent act. There is no tool that proposes or executes an action, and | ||
| no auto-approve. | ||
| - It binds loopback and has no flag that changes that, because its read tools answer without a | ||
| credential. | ||
| - Over HTTP it binds loopback and has no flag that changes that, because its read tools answer | ||
| without a credential. With `--stdio` it binds nothing: the client that launched it is its only | ||
| client, and the approver is that client's OS login. | ||
|
Comment on lines
+74
to
+76
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win Distinguish HTTP and stdio write identities. The preceding write-tool bullet says every write requires a credential from a configured identity provider. These lines state that stdio writes use the process account OS login. Update the bullet to describe an attributable human identity, with an identity provider for HTTP and OS-login attribution for stdio. 🤖 Prompt for AI Agents |
||
|
|
||
| [Approve from your assistant](/docs/mcp/approve-from-your-assistant) is the page with the | ||
| configuration and a transcript. | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
State the registry availability version.
This section does not state that the registry listing became available in version
0.12.2. Add that version boundary so users can determine whether their installed release supports registry installation.As per PR objectives: the registry listing must state that it became available in version 0.12.2.
🤖 Prompt for AI Agents