Skip to content
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

Fix: Replace form-data with formdata-node (Issue #609) #614

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

Conversation

devin-ai-integration[bot]
Copy link

Replaced 'require' usage to support ESM by replacing form-data with formdata-node.

Changes made:

  • Remove form-data package dependency
  • Add formdata-node and form-data-encoder packages
  • Update code to use FormData from formdata-node
  • Update test mocks to use formdata-node
  • Handle ReadableStream attachments properly

Resolves #609

Link to Devin run: https://app.devin.ai/sessions/df9130fe51c44ad18f3ecad9bb834aa2

- Remove form-data package dependency
- Add formdata-node and form-data-encoder packages
- Update code to use FormData from formdata-node
- Update test mocks to use formdata-node
- Handle ReadableStream attachments properly

Fixes #609
Copy link
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

- Remove unused Blob import
- Replace while(true) with proper stream reading loop
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.

ReferenceError: require is not defined
0 participants