Skip to content

Make batch a structured read-only query transport - #4661

Merged
Widthdom merged 6 commits into
mainfrom
fix-issue4582
Jul 17, 2026
Merged

Make batch a structured read-only query transport#4661
Widthdom merged 6 commits into
mainfrom
fix-issue4582

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Define the batch allowlist from the command schema and include side-effect-free goto and audit queries.
  • Return typed result/results payloads for JSON and NDJSON while preserving raw stdout for text and failures.
  • Preserve per-command exit/error metadata and bound input lines plus the complete serialized output stream.
  • Document and test the transport contract in both English and Japanese, including command-specific and case-insensitive output formats.

Validation

  • dotnet build CodeIndex.sln -c Release --no-restore
  • dotnet test CodeIndex.sln -c Release --no-build -m:1 (net8.0: 9512 passed, 6 skipped; net9.0: 9076 passed, 384 skipped)
  • Focused batch tests: 20 passed on net8.0 and 20 passed on net9.0
  • dotnet format CodeIndex.sln --verify-no-changes --no-restore
  • Changelog fragment validation
  • git diff --check
  • Codex adversarial review: two rounds completed; all actionable findings addressed

Fixes #4582

@Widthdom
Widthdom marked this pull request as ready for review July 17, 2026 23:42
@Widthdom
Widthdom merged commit d8e1f04 into main Jul 17, 2026
9 checks passed
@Widthdom
Widthdom deleted the fix-issue4582 branch July 17, 2026 23:43
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.

Make batch a complete, structured read-only query transport

1 participant