Repository navigation
agent, cli, docs: monid() managed tool catalog and platform secret scope - #858
Open
devin-ai-integration[bot] wants to merge 2 commits into
Open
devin-ai-integration[bot] wants to merge 2 commits into
devin-ai-integration[bot] wants to merge 2 commits into
Conversation
Templates gain tools without asking the installer for keys: calling
monid() (or naming "monid" as an MCP server) attaches the managed Monid
catalog — discover/inspect/run meta-tools over hundreds of providers —
with the credential supplied by the platform at the egress edge.
- @opencomputer/agent: monid() helper selecting the managed catalog;
useSecret() gains a "platform" scope for connection headers
- @opencomputer/cli: emit the managed mcp.monid.ai connection + server on
monid() or useMcpServer("monid"); a project's own monid definitions
win a string selection, while an explicit monid() call is a conflict
- docs: managed catalogs section on the MCP page
Co-Authored-By: Mohamed <mo@digger.dev>
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Automations to automatically generate PRs for you. |
Contributor
Author
|
I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".
|
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
Contributor
Author
There was a problem hiding this comment.
👀 2 findings need your review
Devin fixed 2 of 4 findings on c08c79c.
For your review (2)
- Custom connection leaves catalog undefined
- Platform credential reaches arbitrary origins
Fixed by Devin (2)
- Managed catalog requires an installer secret
- Re-exported catalog lacks a deployment binding
View all findings in Devin Review
- buildTemplateProject skips platform-scoped headers: they are supplied
by the platform at the egress edge, never an install-time secret the
installer is asked to enter
- monid() detection resolves re-export chains through the project's own
modules (export { monid } from "@opencomputer/agent" in a barrel
keeps the helper recognized at its call site), including namespace
imports and export *
- isTypeOnly lives on the import clause, not the declaration
Co-Authored-By: Mohamed <mo@digger.dev>
This branch was successfully deployed
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
1 participant
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.
Summary
Templates gain a full external tool catalog without asking the installer for a single key.
monid()— or naming"monid"as an MCP server — attaches the managed Monid catalog (https://mcp.monid.ai/v1: discover/inspect/run meta-tools over ~1700 tools across 55+ providers) and the compiler emits aplatform-scopedMONID_API_KEYsecret header, which the platform's egress edge substitutes from its own credential (companion blue PR adds that scope).@opencomputer/agent:monid()helper returning a catalog markeruseMcpServer()/useFetch()accept;useSecret()gainsscope: "platform"for connection headers.@opencomputer/cli(compileAgent): onmonid()oruseMcpServer("monid")emitsMANAGED_MONID_CONNECTION+MANAGED_MONID_MCP_SERVERinto the deployment manifest. A project that defines its ownmonidconnection or server wins a bare"monid"selection; an explicitmonid()call alongside one is a compile error (the id is reserved for the managed catalog).agents/mcp.mdx.Merge order: blue (platform scope) → this PR → template PRs that adopt
monid().Link to Devin session: https://app.devin.ai/sessions/f74b8cc046c94dc583e998a6153ee14a
Open in Devin Desktop: https://app.devin.ai/desktop/session/f74b8cc046c94dc583e998a6153ee14a?variant=devin
Requested by: @motatoes
Polylane reviews this pull request when you ask: