You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Add built-in MCP entries for time and context7 without requiring user-defined server configuration.
Enable the bundled time MCP by default; keep context7 disabled until the user manually enables it.
Keep explicit same-name user or session MCP configuration authoritative and update the frontend mock data used by local browser UI.
Behavior
time uses a bundled Reasonix stdio MCP (builtin-mcp time), so it does not require uvx, Python packages, Node, or npm packages on the user's machine.
context7 uses a JavaScript runner selected progressively: npx, then pnpm dlx, then bunx; if none are available, the default npx spec is retained so diagnostics stay clear.
Enabled built-ins use the lazy tier, so startup does not spawn subprocesses or wait on package/network work.
Verification
go test ./internal/config ./internal/builtinmcp ./internal/boot ./internal/control ./internal/cli with isolated user config directories
Reviewed end-to-end and approved. The architecture is clean: Entries() as the catalogue, EnabledNames() driving injection, and AppendEnabled suppressing built-ins against both configured [[plugins]] and session ExtraPlugins — so user/host config stays authoritative. The bundled builtin-mcp time server is a correct dependency-free MCP stdio implementation (newline-framed JSON-RPC, initialize/tools-list/tools-call, notifications ignored, EOF-clean), which is the right answer to the "no uvx/npm package required" goal. All three Codex P2s are genuinely fixed (configured-wins precedence, ExtraPlugins de-dup with regression coverage, and the real bundled command). time on-by-default is safe (lazy tier, dependency-free); Context7 stays opt-in. Merging.
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
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
timeandcontext7without requiring user-defined server configuration.timeMCP by default; keepcontext7disabled until the user manually enables it.Behavior
timeuses a bundled Reasonix stdio MCP (builtin-mcp time), so it does not requireuvx, Python packages, Node, or npm packages on the user's machine.context7uses a JavaScript runner selected progressively:npx, thenpnpm dlx, thenbunx; if none are available, the defaultnpxspec is retained so diagnostics stay clear.lazytier, so startup does not spawn subprocesses or wait on package/network work.Verification
go test ./internal/config ./internal/builtinmcp ./internal/boot ./internal/control ./internal/cliwith isolated user config directoriesgo test .indesktopnpm exec -- tsc --noEmitindesktop/frontendgit diff --check