kleneway
commented
7 months ago @cpirich A few things here to note
- I made the "uploadSnapshot" api more generic so it's just an "upload any image" endpoint.
- I added the "imageUrls" to the Figma -> create issue api. Any image urls are added as markdown to the issue.
- I updated the "getImages" to use the "static assets" folder from settings (defaulted to /public/images for backwards compat). I also noticed a bug there where it was only getting images from the root folder, not any subfolders, so I fixed that.
- I added a new "saveImages" function. This will extract the saved images from the issue, check to see if that image already exists in that branch, and if not it will download and save to the filesystem. The new images are added to the "existing images" string that is sent to the LLM.
Overall, any thoughts about this approach? I think the "get existing images -> check for dups -> save images -> update image prompt" flow should handle any cases where we are going through these steps multiple times. And I think using the signed urls that expire in an hour is a "good enough" tradeoff between security and simplicity. But open to other ideas here too.
There are a few accompanying updates to the Figma plugin that I'll push up shortly.
The latest updates on your projects. Learn more about Vercel for Git ↗︎