Skip to content

fix: remove dead --stream CLI flag (streaming is always used)#20

Merged
Coding-Dev-Tools merged 1 commit into
masterfrom
Openclaw/datamorph/remove-dead-stream-flag
May 19, 2026
Merged

fix: remove dead --stream CLI flag (streaming is always used)#20
Coding-Dev-Tools merged 1 commit into
masterfrom
Openclaw/datamorph/remove-dead-stream-flag

Conversation

@Coding-Dev-Tools

Copy link
Copy Markdown
Owner

The --stream\ flag on the \convert\ command and the corresponding \stream\ parameter in the \convert()\ function were dead code — all conversions always stream via
ead_stream()/\write_stream(). Removed both to avoid misleading users into thinking there is a non-streaming mode.

Changes:

  • Removed --stream\ CLI flag from \convert_cmd\
  • Removed unused \stream\ parameter from \convert()\ function signature and docstring
  • Removed test that verified --stream\ flag acceptance

Tests: 84/84 pass, ruff clean

The --stream flag on the convert command and the corresponding stream
parameter in the convert() function were dead code — all conversions
always stream via read_stream()/write_stream(). Removed both to avoid
misleading users into thinking there is a non-streaming mode.

Closes a CLI UX issue where users could pass --stream expecting
different behavior with no effect.
@Coding-Dev-Tools Coding-Dev-Tools merged commit 0aee4e1 into master May 19, 2026
5 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.

1 participant