Skip to content

Fix README examples for NDJSON output#60

Merged
turlockmike merged 1 commit intomasterfrom
docs/condense-readme
Feb 21, 2026
Merged

Fix README examples for NDJSON output#60
turlockmike merged 1 commit intomasterfrom
docs/condense-readme

Conversation

@turlockmike
Copy link
Owner

Summary

  • Quick Start uses verified examples against real public servers with jq piping
  • Fixed jq usage: NDJSON is line-delimited objects, not arrays (jq -r '.name' not jq '.[0].name')
  • README and Show HN examples are now consistent

Generated with Claude Code

- Quick Start now uses verified examples against public MCP servers
- All examples include jq piping to show Unix pipeline friendliness
- Fixed jq usage: NDJSON uses .name not .[0].name

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings February 21, 2026 00:09
@turlockmike turlockmike merged commit 5dc8f02 into master Feb 21, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes incorrect jq usage examples in the README to properly handle NDJSON (newline-delimited JSON) output. The previous examples incorrectly treated NDJSON as a JSON array using jq '.[0].name', but NDJSON consists of individual JSON objects on each line, requiring jq -r '.name' instead. The PR also updates examples to use verified public servers and adds clarifying comments about the NDJSON output format.

Changes:

  • Fixed jq syntax from array indexing (.[0].name) to line-by-line processing (.name) with -r flag for raw output
  • Updated Quick Start examples to use verified public servers with complete command pipelines
  • Reordered public demo servers to prioritize mcp.deepwiki.com over remote.mcpservers.org

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants