Skip to content

fix(cli): update error message to reference --stdin flag#1408

Merged
stack72 merged 1 commit into
mainfrom
fix/stdin-error-message
May 19, 2026
Merged

fix(cli): update error message to reference --stdin flag#1408
stack72 merged 1 commit into
mainfrom
fix/stdin-error-message

Conversation

@stack72
Copy link
Copy Markdown
Contributor

@stack72 stack72 commented May 19, 2026

Summary

Test plan

  • swamp model method run foo bar --stdin --input-file x.yaml shows the updated error message

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CLI UX Review

Blocking

None.

Suggestions

None.

Verdict

PASS — The two error messages now correctly reference --stdin (the explicit flag introduced in #1407) instead of the generic phrase "piped stdin". Both the updated message and the --input-file flag description already say "cannot combine with --stdin", so the wording is now consistent across both directions of the mutual-exclusion check. No other user-visible surface is changed.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

Blocking Issues

None.

Suggestions

None.

Clean two-line fix — error messages now correctly reference --stdin to match the explicit flag introduced in #1407. Both call sites (model_method_run.ts:207, workflow_run.ts:183) are consistent and the surrounding code already uses options.stdin, so the message accurately describes the user-facing flag.

@stack72 stack72 merged commit 4f9d3e7 into main May 19, 2026
11 checks passed
@stack72 stack72 deleted the fix/stdin-error-message branch May 19, 2026 22:52
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