Skip to content

Conversation

@lefarcen
Copy link
Contributor

Summary

Remove file type restrictions for workflow variables and fix file download issues.

File Type Restrictions Removed:

  • Removed resource type selector from ResourceTypeForm, defaulting to all types
  • Allow any file type upload without extension restrictions
  • File-to-variable conversion now accepts all file types with default resourceTypes: ['document', 'image', 'audio', 'video']

Download Fix:

  • Fixed empty file download in unsupported file preview by using useDownloadFile hook
  • Added loading state for download button

Import Modal:

  • Disabled import button while files are uploading

Impact Areas

  • Free-form Canvas Interface
  • Workflow Variables
  • Resource Library

Checklist

  • I understand that this PR may be closed in case there was no previous discussion or issues.
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods

…ile download loading states, and configure Serena project settings.
@coderabbitai
Copy link

coderabbitai bot commented Dec 18, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/mashu/remove-file-type

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@lefarcen lefarcen closed this Dec 18, 2025
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