Skip to content

fix: reject empty message content - #6580

Open
jaezinpark wants to merge 1 commit into
block:mainfrom
jaezinpark:fix/buzz-reject-empty-message
Open

fix: reject empty message content#6580
jaezinpark wants to merge 1 commit into
block:mainfrom
jaezinpark:fix/buzz-reject-empty-message

Conversation

@jaezinpark

Copy link
Copy Markdown

Summary

  • reject messages whose trimmed content and attachment list are both empty
  • fail before network publication with a user error that points to missing stdin
  • preserve normal text messages and attachment-only posts

Motivation

Two agent callbacks invoked messages send with --content - but delivered EOF. The CLI accepted and published blank events, so accepted:true masked the caller failure.

Verification

  • cargo fmt --all -- --check
  • cargo test -p buzz-cli (366 passed)
  • cargo clippy -p buzz-cli -- -D warnings
  • empty stdin smoke: exit 1, stdout 0 bytes, retryable false
  • whitespace-only stdin smoke: exit 1, stdout 0 bytes, retryable false

Signed-off-by: jaezinpark <jaezinpark@gmail.com>
@jaezinpark
jaezinpark requested a review from a team as a code owner August 22, 2026 21:11
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