Skip to content

Attachment references accept any attachment id - #47

Merged
jph00 merged 1 commit into
mainfrom
pr/attachment-references-accept-any-attachment-id
Sep 2, 2026
Merged

Attachment references accept any attachment id#47
jph00 merged 1 commit into
mainfrom
pr/attachment-references-accept-any-attachment-id

Conversation

@jph00

@jph00 jph00 commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

_media_atts only matched attachment: references whose id had the generated UUID shape, so an attachment given its own id was never sent to the model. References are now matched against the message's attachments by id, whatever the id looks like. See _media_atts in the History notebook.

@jph00 jph00 added the bug Something isn't working label Sep 2, 2026
@jph00
jph00 merged commit ebb1d28 into main Sep 2, 2026
1 check passed
@jph00
jph00 deleted the pr/attachment-references-accept-any-attachment-id branch September 2, 2026 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant