Skip to content

Handle image processing errors gracefully with automatic retry#7

Merged
rossnelson merged 1 commit intomainfrom
claude/1771083390-448109
Feb 14, 2026
Merged

Handle image processing errors gracefully with automatic retry#7
rossnelson merged 1 commit intomainfrom
claude/1771083390-448109

Conversation

@rossnelson
Copy link
Copy Markdown
Member

Summary

  • Adds graceful recovery when the API cannot process an attached image: strips image blocks from the prompt and re-enqueues the request so the user's text is still fulfilled
  • Introduces IMAGE_ERROR failure type and retried_without_images recovery action for explicit image error classification
  • Defers Slack error notifications to after failure classification, so users see a contextual warning (image dropped) instead of a raw error message

Test plan

  • Send a request with a corrupt/unsupported image and verify the bot posts a warning then retries with text only
  • Send an image-only request (no text) with a bad image and verify the bot posts an appropriate "no text content" error
  • Send a normal text request and verify behavior is unchanged
  • Trigger a non-image error (e.g. transient network failure) and verify the original error notification still appears

When the API cannot process an attached image, strip the image blocks
from the prompt and re-enqueue the request instead of showing a raw
error. Users see a warning that the image was dropped and their text
request is still fulfilled. Also defers the error notification to after
failure classification so non-recoverable errors are reported correctly.
@rossnelson rossnelson merged commit 81e8748 into main Feb 14, 2026
rossnelson pushed a commit that referenced this pull request Feb 14, 2026
Replace hardcoded PR titles with AI-generated content by calling
claude --print with the branch diff and commit log, parsing the
JSON response into a descriptive title and markdown body.
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.

2 participants