Skip to content

fix(app): omit empty prompt text parts - #37577

Merged
Hona merged 3 commits into
devfrom
mute-comment-error
Jul 20, 2026
Merged

fix(app): omit empty prompt text parts#37577
Hona merged 3 commits into
devfrom
mute-comment-error

Conversation

@opencode-agent

@opencode-agentopencode-agentBot commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • omit empty text parts from comment-only prompt requests
  • keep the synthetic review comment and attached file unchanged
  • avoid the backend error that was triggering the error sound

Check

  • bun typecheck

Requested by: @jayair (Jay via Slack)

@opencode-agentopencode-agentBot changed the title fix(app): mute comment-only prompt errorsfix(app): omit empty prompt text partsJul 18, 2026
@Hona
Hona merged commit a19b52e into devJul 20, 2026
11 checks passed
@Hona
Hona deleted the mute-comment-error branch July 20, 2026 07:09
Bkm016 pushed a commit to Bkm016/opencode that referenced this pull request Jul 20, 2026
Co-authored-by: Jay <53023+jayair@users.noreply.github.com>
avion23 pushed a commit to avion23/opencode that referenced this pull request Jul 20, 2026
Co-authored-by: Jay <53023+jayair@users.noreply.github.com>
tobocop2 pushed a commit to tobocop2/opencode that referenced this pull request Jul 29, 2026
Co-authored-by: Jay <53023+jayair@users.noreply.github.com>
3kaiu pushed a commit to 3kaiu/opencode-x that referenced this pull request Jul 31, 2026
Co-authored-by: Jay <53023+jayair@users.noreply.github.com>
MarsQiu007 referenced this pull request in MarsQiu007/openNovel Jul 31, 2026
Co-authored-by: Jay <53023+jayair@users.noreply.github.com>
banlanzs pushed a commit to banlanzs/opencode that referenced this pull request Aug 6, 2026
Co-authored-by: Jay <53023+jayair@users.noreply.github.com>
charlesverge pushed a commit to charlesverge/opencode that referenced this pull request Aug 11, 2026
Co-authored-by: Jay <53023+jayair@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@jayair@Hona