Repository navigation
Conversation
…ust, Copilot via the marketplace and one .mcp.json
/docs/clients opened with a 253-word paragraph on protocol eras and roots
before any setup. It now opens with one line ("Run it from the project, or
pass --root"), a box with the one .mcp.json that Claude Code, VS Code,
GitHub Copilot CLI and Grok Build read, and moves how the server finds the
project to a closing section.
- Gemini CLI: a gemini mcp add -s user one-liner, and the trusted-folder rule
(an untrusted folder lists the server as Disabled, user-level ones too).
- Antigravity (CLI and IDE): its own mcp_config.json paths, and a README badge.
- Copilot CLI: install the plugin through the marketplace; the direct install
from a repository path prints a deprecation warning.
- The hook runs in Claude Code and Codex CLI, on apply_patch edits in Codex.
OS2-28
This branch has not been 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
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.
Roadmap item: OS2-28.
What was wrong
/docs/clients, the setup page for everyone not on the Claude Code plugin, opened with a 253-word paragraph on protocol eras, roots deprecation and project discovery before any setup.gemini mcp list(Gemini CLI 0.64.0) printed "Warning: MCP servers are configured but disabled because this folder is untrusted." and○ onsystem: ... - Disabled. There was also nogemini mcp addone-liner.mcp_config.json. The docs did not cover it.copilot plugin install dgesteves/onsystem:plugins/onsystem-agent) prints "Direct plugin installs (repos, URLs, local paths) are deprecated." (Copilot CLI 1.0.98)..mcp.jsonthat several clients read was only shown under "Claude Code without the plugin".What changed
docs/clients.md:--root..."..mcp.jsonto commit. It names the clients that read it (Claude Code, VS Code, GitHub Copilot CLI, Grok Build), says it needs no--root, and says Claude Code teams should use it instead of the plugin, not as well.gemini mcp add -s user ...command, what each scope writes, and the trusted-folder rule. That covers/permissionsand, for headless runs,--skip-trustorGEMINI_CLI_TRUST_WORKSPACE=true.~/.gemini/config/mcp_config.jsonand.agents/mcp_config.jsonpaths and the same server entry.copilot plugin marketplace add dgesteves/onsystem, thencopilot plugin install onsystem@dgesteves). The Copilot CLI config block notes that it also reads a committed.mcp.jsonin a trusted folder.apply_patchadds or updates (fix(hook): check the files a Codex apply_patch changes; init offers an AGENTS.md section #182)..mcp.jsonbox and no longer repeats the JSON.README.md: an Antigravity badge under "Also works with", and Antigravity in the list of client configs.site/app/docs/clients/page.tsxis unchanged. The page rendersdocs/clients.mdafter the two install buttons, so the new order shows on the site as it is.No changeset: docs and site only.
How it was tested
New tests, both failing on
mainfor the reasons above (7 of 7 and 3 of 3) and passing now:test/clients-docs.test.tschecks the following:--rootline.##section is "How the server finds your project", and it holds the protocol text, which the intro no longer does..mcp.jsonand names the four clients.site/test/clients-page.test.tsrenders/docs/clientsand checks the following:.mcp.jsonbox sits above the first section.Gate, all passing:
pnpm install --frozen-lockfile --prefer-offline,pnpm format:check,pnpm lint,pnpm typecheck,pnpm build,pnpm test(82 files passed, 1 skipped; 1049 tests passed, 3 skipped). Site:pnpm --filter site generate,test(17 files, 272 tests),typecheck,lint,build. I then served the built site locally and checked/docs/clientsin a browser:#how-the-server-finds-your-project./docs/rollout#give-every-agent-the-design-system.Setup commands: which were run, which were checked against docs
Run for real, each CLI installed with
--ignore-scriptsinto its own folder and run underenv -iwith a throwawayHOME:gemini mcp add -s user onsystem npx -y onsystem --root <app>wrote theargsexactly as shown, with no--needed. Without-s, it wrote.gemini/settings.jsonin the project.gemini mcp listshowed onsystem as Disabled in an untrusted folder. It showed✓ onsystem ... - Connectedonce the folder was intrustedFolders.json, and again withGEMINI_CLI_TRUST_WORKSPACE=true.--skip-trustis fromgemini --help.gemini trustcommand, but 0.64.0 has no such subcommand: it reads "trust" as a prompt. So the docs point to the trust prompt and/permissions.plugin listshowedonsystem@dgesteves (v0.8.0), andmcp listshowedPlugin servers: onsystem (local)..mcp.jsonwas skipped in an untrusted folder ("No MCP servers configured."). Once the folder was trusted, it was listed under "Workspace servers", withSource: Workspace (.../.mcp.json).copilot mcp addstill writes~/.copilot/mcp-config.jsonas the docs show.codex plugin marketplace add dgesteves/onsystemandcodex plugin add onsystem@dgestevesinstalled plugin 0.8.0.codex mcp addwrote the documentedconfig.tomlentry.claude mcp add --scope project onsystem -- npx -y onsystemwrote.mcp.jsonwith the same command and args, plus"type": "stdio"and"env": {}.Checked against current docs only (no CLI to run here):
mcp_config.jsonpaths, the IDE menu path (MCP Servers, Manage MCP Servers, View raw config) and/mcpin the CLI..mcp.jsonwithmcpServers, under Workspace Trust..mcp.jsonfiles./hooks.Follow-ups (outside this item's files)
plugins/onsystem-agent/README.md:25still gives the deprecatedcopilot plugin install dgesteves/onsystem:plugins/onsystem-agent.site/lib/faq.ts:45says the edit-time hook "ships for Claude Code only so far". Codex runs it too since fix(hook): check the files a Codex apply_patch changes; init offers an AGENTS.md section #182.site/lib/docs.ts(the/docs/clientsdescription) anddocs/quickstart.md:42don't name Antigravity yet.docs/tools.mdlinks "how the server finds the project" toclients.md. It can now point toclients.md#how-the-server-finds-your-project.docs/clients.md) builds on this rewrite.