Skip to content

Remove unused stream completion stdout helper - #2085

Merged
gold-silver-copper merged 1 commit into
mainfrom
issue-2073-delete-stream-completion-helper
Jul 9, 2026
Merged

Remove unused stream completion stdout helper#2085
gold-silver-copper merged 1 commit into
mainfrom
issue-2073-delete-stream-completion-helper

Conversation

@gold-silver-copper

Copy link
Copy Markdown
Contributor

Summary

  • Remove the unused public low-level streaming::stream_completion_to_stdout helper.
  • Keep the high-level agent::stream_to_stdout helper, re-export, examples, and tests intact.
  • Document the breaking removal in the root and rig-core changelogs.

Closes #2073.

Validation

  • cargo fmt
  • cargo fmt --check
  • cargo check -p rig-core
  • cargo check -p rig-core --all-features
  • cargo check -p rig-core --all-features --examples --tests
  • cargo check -p rig-core --all-targets --all-features
  • cargo test -p rig-core --all-features streaming -- --nocapture
  • cargo test -p rig-core --lib streaming::
  • git diff --check
  • exact searches confirmed stream_completion_to_stdout has no references outside changelogs and agent::stream_to_stdout remains.

Not run

  • Full workspace cargo test
  • Full workspace cargo clippy --all-targets --all-features
  • cargo doc
  • Live provider/network tests

@gold-silver-copper
gold-silver-copper added this pull request to the merge queue Jul 9, 2026
Merged via the queue into main with commit aebf757 Jul 9, 2026
6 checks passed
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.

chore(core)!: delete unused streaming::stream_completion_to_stdout helper

1 participant