Skip to content

feat: include images in tasks #328

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

lequocbinh04
Copy link

feat: include images in tasks, and change the prompt to make AI model to read the image and understand it.

Summary

this PR implements comprehensive image attachment functionality for tasks, allowing users to upload, view, and manage images that AI models can analyze during task execution

Key Features

  • Multiple upload methods: Drag-and-drop, file selection, and clipboard paste
  • AI integration: All executors now instruct AI models to read and analyze attached images
  • Storage: Images stored securely with UUID filenames, metadata tracked in database
  • UI components: New ImageUpload widget and TaskAttachments grid with lightbox preview
  • API endpoints: Upload, download, and delete attachments with proper validation

Technical Changes

  • Added attachments table with foreign key to tasks
  • Created /api/attachments/* endpoints with 10MB file size limit
  • Updated all AI executors to include attachment paths in prompts
  • Enhanced task forms with image upload capability
  • Implemented proper blob URL cleanup to prevent memory leaks

Some Images:

CleanShot 2025-07-23 at 17 37 17@2x CleanShot 2025-07-23 at 17 37 39@2x

@ggordonhall ggordonhall self-requested a review July 23, 2025 11:04
@nguyenvanduocit
Copy link

Very useful

@nguyenvanduocit
Copy link

nguyenvanduocit commented Jul 23, 2025

I think the attachment section should be collapsible. style should be match with task details:

image

@lequocbinh04
Copy link
Author

attachments section can be collapsible now
CleanShot 2025-07-24 at 10 01 09@2x

CleanShot 2025-07-24 at 10 01 17@2x

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