Fix: wrap list JSON output in array for parseable output#80
Conversation
Fix ExplanationBug: Root cause: Solution: Modified
This likely fixes all list endpoints that use the same |
|
👋 Hi! Just checking in — this PR has been open 14+ days with no review. Happy to make any adjustments if needed! 🙏 |
|
👋 Hi! Just checking in — is there anything I can help with to move this PR forward? Happy to address any feedback! 🙏 |
|
👍 Looks good to merge — clean fix, thanks for contributing! |
|
Hi @ALL, gentle reminder that this PR is waiting for review. Happy to make any changes — just let me know! 🙏 |
|
Thanks for this contribution! Bumping for visibility. |
|
Thank you for this contribution! Bumping for visibility. 🙏 |
|
Since there has been no response from the maintainer for a while, I would like to kindly follow up on this PR. I believe the fix is valuable and would love to contribute. Please let me know if any changes are needed, or if it can be merged. Thank you! |
|
Bump — ready to merge 🎯 |
|
Thanks for the fix! Gentle ping to see if this is ready for review. 👍 |
|
bump |
|
➿ bump (R337) |
|
🙌 bump |
1 similar comment
|
🙌 bump |
Summary
Fixes mercury-cli issue #70.
When running commands like
mercury accounts list --format json, multiple pretty-printed JSON objects were emitted back-to-back without an enclosing array or commas, making output unparseable by standard JSON parsers:{"id": "...", "name": "..."} {"id": "...", "name": "..."}This change wraps JSON/pretty formatted list output in a proper JSON array with commas between items:
[\n, appends,\n]\nto collected output[\n, emits items with,\nseparators, writes\n]\nBefore:
{...}{...}(unparseable)After:
[{...},{...}](valid JSON array)Closes #70