docs(integrations): name the MCP access surface and the two Setup entries - #1367
Merged
Merged
Conversation
…ries (#909) The What-ships-today list omitted /api/v1/mcp, a default-on access surface, and no page said what the Connect an Agent and Datasources entries under Setup -> Integrations actually are. Added in all three locales, after confirming in a browser that both entries really render. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WMzCeNC4SZcPNBpE2zCVCg
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
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.
Fixes #909
The integrations guide's What ships today list named the data API, export, import, outbound email, in-app notification and your own code — and never mentioned
/api/v1/mcp, an access surface that is on by default. It also left two entries an administrator can see under Setup → Integrations completely unexplained. Both gaps are closed here, in all three locales.The browser gate came first
Everything on the card was read statically out of
dist/, and registration is not rendering. Before a word of prose was written, the app was booted and driven as a real administrator:objectstack dev --ui --seed-adminon a private port with its own SQLite file, waited for/api/v1/healthto return 200. Boot banner listed 38 plugins includingMCPServerPlugin,DatasourceAdminServicePluginandDatasourceAdminRoutes, and advertisedMCP: http://localhost:PORT/api/v1/mcp.admin@objectos.ai, opened the Setup app.Integrationsis a sidebar group, expanded by default — no click needed — withDatasourcesandConnect an Agentunder it, each with a non-zero bounding box. Screenshots taken.Connect an Agentrenders the endpoint URL, per-client setup for Claude / Claude Code / Cursor / VS Code / Copilot / Codex CLI, a headless API-keys route and aSKILL.mddownload.Datasourcesrenders a real roster — the deployment's own store as Default withTestandSync objectsactions.Gating, from the server's own metadata rather than from the DOM: the
group_integrationsgroup declaresrequiredPermissions: ["manage_platform_settings"].nav_datasourcesdeclares the same permission and nothing further;nav_connect_agentdeclares none of its own and inherits the group's. Neither entry is gated behind anything beyondmanage_platform_settings.The gate also corrected a static reading, which is the point of having it
Reading
dist/said both labels were hardcoded English literals with no i18n key —grepfortranslationKey/labelKeyreturns 0 in both plugins. That conclusion was wrong. The server translates these labels at metadata-serve time. Measured live against the running app:enzh-Hanszh-HantSo the zh-Hans page names these entries by the labels a zh-CN reader actually sees. The zh-Hant page keeps this repo's existing ruling that Traditional pages spell platform navigation in English.
What changed
Three
.mdxfiles, nothing else:webhooksis deliberately still absent — unlike the other two it genuinely is not loaded, so the page is right to omit it.Verification
pnpm verifygreen on9e95480, the final commit — run through this container's shared verify lock, whose own verdict line readscommand-exit 0 · held the lock 189s. Each step's own verdict:The navigation guard was proven to read these new lines, not merely to pass. Its green is otherwise unfalsifiable, so the new zh-Hans citation was mutated on disk to a first segment the platform ships nowhere (
集成to集成中心); the mutation was confirmed landed by counting both the injected and the removed text, anddocs-setup-navigation-nameswent red naming this PR's own file and citation:Restored with
git checkout HEAD -- ABSOLUTE_PATH; restoration proven by state, not by exit code —git diff HEADempty, worktree blob3d8ae76identical to the HEAD blob, mutant marker count back to 0 — and the guard returned to30 passed (30). No anchored links were added; every link on the touched lines is a plain page link, so #867's anchor guard has nothing new to resolve.Changeset carries empty frontmatter — verified against the diff rather than assumed: the change is three
.mdxfiles and publishes nothing.Generated by Claude Code
Generated by Claude Code