Skip to content

Support catalogue preview images in caption creation - #41

Merged
sarthakagrawal927 merged 1 commit into
mainfrom
agent/meme-lab-caption-preview-coverage-20261005
Oct 5, 2026
Merged

sarthakagrawal927 merged 1 commit into
mainfrom
agent/meme-lab-caption-preview-coverage-20261005

Conversation

@sarthakagrawal927

Copy link
Copy Markdown
Member

The live caption smoke found that most catalogue templates use Imgflip's standard /4/<id>.<raster> preview path. The image allowlist accepted only root-level assets, leaving 1,460 static references without the existing vision-caption workflow.

Accept that exact additional path format on the same catalogue-only Imgflip host. Keep other directories, nested paths, SVGs, arbitrary caller URLs, redirects and oversize/non-raster responses rejected. This raises eligible static sources from 221 to 1,681 of 1,805; eligibility does not guarantee a readable layout or humour quality. Baked Memegen references and GIFs retain original-reference behavior.

Validation: focused composition and Workerd checks pass, including the 1,200-token vision request grounded to the actual catalogue preview and adversarial path rejection. The Anna template module is rebuilt; no caption geometry, prompt, provider routing, grants, dependencies or production configuration changed. The frozen Anna 1.3.0 client obtains its plan from the updated server and does not use this URL predicate locally.

@sarthakagrawal927
sarthakagrawal927 merged commit a555da7 into main Oct 5, 2026
1 of 2 checks passed
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.

1 participant