Skip to content

Fix unsupported JSON help for mcp and completions - #2797

Merged
Widthdom merged 6 commits into
mainfrom
fix-issue1424
May 31, 2026
Merged

Fix unsupported JSON help for mcp and completions#2797
Widthdom merged 6 commits into
mainfrom
fix-issue1424

Conversation

@Widthdom

@Widthdom Widthdom commented May 31, 2026

Copy link
Copy Markdown
Owner

Summary

  • Document that mcp and completions do not support --json in subcommand help notes.
  • Make direct --json errors for both commands explain the command-specific output format.
  • Add focused CLI coverage for help and error paths.

Validation

  • dotnet format CodeIndex.sln --verify-no-changes --no-restore --verbosity minimal
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --framework net8.0 --filter "FullyQualifiedName~ProgramCliTests"
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --framework net9.0 --filter "FullyQualifiedName~ProgramCliTests"
  • CLI smoke checks for mcp --help, completions --help, mcp --json, and completions --json
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json

Documentation / Changelog

  • Added changelog.d/unreleased/1424.fixed.md.
  • No AGENT.md, CLAUDE.md, or AGENT_GUIDE.md changes.

Review

  • Required Codex adversarial review completed with no blocking/actionable issues found.

Follow-up Candidates

  • None for this PR.

Fixes #1424

@Widthdom
Widthdom marked this pull request as ready for review May 31, 2026 18:35
@Widthdom
Widthdom merged commit 2dfc300 into main May 31, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue1424 branch May 31, 2026 18:58
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.

mcp and completions reject --json with a generic 'not supported' error

1 participant