Skip to content

Keep earlier images from blocking new messages - #1421

Merged
fazxes merged 2 commits into
mainfrom
fazxes/withhold-earlier-images
Oct 23, 2026
Merged

fazxes merged 2 commits into
mainfrom
fazxes/withhold-earlier-images

Conversation

@fazxes

@fazxes fazxes commented Oct 22, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Send a new message in a session with an earlier image even when fx can't confirm the model accepts images, instead of failing it with "Unable to verify image support for this model".
  • Leave earlier images out of that request with a note for the model, and send them inline again once the model catalog lists the model with vision.
  • Show "Earlier images aren't reaching the model right now" once per message when an unreachable model catalog keeps earlier images out.
  • Send a message after an earlier image to a model without vision and no vision tool, with the image left out and a note, instead of failing it.
  • Keep failing a message whose own attached image can't be confirmed, as before.

When fx could not confirm that a model accepts images, because the model catalog was unreachable or did not list the model, every later message in a session that had attached an image failed with "Unable to verify image support for this model". Only an image attached to the current message can fail it now. Earlier images go inline to a model the catalog lists with vision, and through the vision tool's image references to a model without vision. Otherwise they stay in the session and leave the request as a note for the model, and an unreachable catalog tells the user once per message.

- The image route core separates images attached this turn from earlier ones, which follow the tool image rule.
- `bun run verify` in `tests/tla/image_route` gains a stage that drives the built binary through `fx ask` sessions and has TLC check every turn against the model.
- The `image_route` and `earlier_image_notes` fuzz targets check the core's laws on generated event sequences and the notes left for earlier images.
@fazxes fazxes added the type: bug Fixes incorrect behavior label Oct 22, 2026
@fazxes
fazxes marked this pull request as ready for review October 22, 2026 22:21
@fazxes
fazxes merged commit 7a64a79 into main Oct 23, 2026
72 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug Fixes incorrect behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant