Skip to content

Stream formatted NDJSON responses incrementally#615

Merged
ryanfowler merged 1 commit into
mainfrom
stream-formatted-ndjson-stdout
May 27, 2026
Merged

Stream formatted NDJSON responses incrementally#615
ryanfowler merged 1 commit into
mainfrom
stream-formatted-ndjson-stdout

Conversation

@ryanfowler
Copy link
Copy Markdown
Owner

Summary

  • Added a dedicated NDJSON stdout streaming path so formatted NDJSON no longer waits for EOF before rendering.
  • Split decoded bytes on newlines, format each record with the JSON-line formatter, and flush after each record.
  • Added a unit guard for the NDJSON streaming decision and a PTY integration test that verifies output appears before the stream closes.
  • Updated the repo notes in AGENTS.md to reflect the new behavior.

Testing

  • cargo fmt
  • cargo clippy --locked --all-targets --all-features -- -D warnings
  • cargo test --all-features
  • cargo test --all-features --test integration -- --test-threads=1

@ryanfowler ryanfowler enabled auto-merge May 27, 2026 05:14
@ryanfowler ryanfowler merged commit d9c1b2d into main May 27, 2026
4 checks passed
@ryanfowler ryanfowler deleted the stream-formatted-ndjson-stdout branch May 27, 2026 05:17
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