Skip to content

docs(git,time): fix invalid JSON in Zed uvx config examples#3989

Open
abhicris wants to merge 1 commit intomodelcontextprotocol:mainfrom
abhicris:contrib-bot/2026-04-21-mcp-servers-zed-json
Open

docs(git,time): fix invalid JSON in Zed uvx config examples#3989
abhicris wants to merge 1 commit intomodelcontextprotocol:mainfrom
abhicris:contrib-bot/2026-04-21-mcp-servers-zed-json

Conversation

@abhicris
Copy link
Copy Markdown

@abhicris abhicris commented Apr 21, 2026

The ""Using uvx"" Zed configuration snippets in src/git/README.md and src/time/README.md wrap an object-style key-value entry in square brackets:

"context_servers": [
  "mcp-server-time": {
    ...
  }
],

That is not valid JSON — the key-with-colon form only parses inside an object. The adjacent ""Using pip"" snippets in the same files already use the correct { ... } form, so this change just makes the uvx variants consistent with them.

Why

Users copy-pasting the uvx example straight into Zed's settings.json will hit a JSON parse error. Switching the two brackets to braces makes the example parse and matches the corrected pattern from PR #114 (which fixed the same issue elsewhere in the repo but missed these two uvx blocks).

Scope

  • Docs-only, no code touched
  • 8 lines changed across 2 files (well under the 50 LOC limit)
  • No new dependencies, no breaking changes
  • One concern: invalid JSON in Zed uvx examples

Types of changes

  • Documentation update

Part of open-source blockchain work from kcolbchain.com — maintained by Abhishek Krishna. PR opened via kcolbchain contrib-bot.

The "Using uvx" Zed configuration snippets in src/git/README.md and
src/time/README.md wrapped an object-style key-value entry in square
brackets, producing invalid JSON. Switch to curly braces so the example
parses and matches the adjacent "Using pip" snippets in the same files.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant