[Fix] Image-only Fast replies show command help - #1446
Draft
roomote-roomote[bot] wants to merge 4 commits into
Draft
[Fix] Image-only Fast replies show command help#1446roomote-roomote[bot] wants to merge 4 commits into
roomote-roomote[bot] wants to merge 4 commits into
Conversation
Contributor
Member
Member
|
Reopened. The previous close was caused by a PR-number mix-up; no code from this PR was merged. |
…e-attachments-1h04fpjleisoq
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
Image-only Slack follow-ups are treated as real multimodal Fast questions instead of empty commands. Fast also limits each turn to three images before downloading them, with the same limit enforced again before inference.
Why this change was made
The base branch now contains the core Fast screenshot transport, but a screenshot-only follow-up still produced command usage help because its text payload was empty. Unbounded image processing also allowed unnecessary downloads before inference.
Impact
Users can continue a Fast conversation by sending only a screenshot, without receiving command help. Bounded image processing avoids excess Slack downloads and model input while preserving text-only Fast behavior.