Skip to content

fix require error#2076

Merged
giurgiur99 merged 1 commit intomainfrom
fix/esm-require-fs
Apr 17, 2026
Merged

fix require error#2076
giurgiur99 merged 1 commit intomainfrom
fix/esm-require-fs

Conversation

@giurgiur99
Copy link
Copy Markdown
Contributor

@giurgiur99 giurgiur99 commented Apr 17, 2026

Fixes # .

Changes proposed in this PR:

  • ESM can't use require, require('fs') in ESM module → ReferenceError
  • Swapped runtime require for static import
  • Added browser: { fs: false } to keep browser compat

@giurgiur99 giurgiur99 merged commit 30ee76a into main Apr 17, 2026
12 checks passed
@giurgiur99 giurgiur99 deleted the fix/esm-require-fs branch April 17, 2026 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ndrpp ndrpp ndrpp approved these changes

@mihaisc mihaisc Awaiting requested review from mihaisc mihaisc is a code owner

@alexcos20 alexcos20 Awaiting requested review from alexcos20 alexcos20 is a code owner

@bogdanfazakas bogdanfazakas Awaiting requested review from bogdanfazakas bogdanfazakas is a code owner

@andreip136 andreip136 Awaiting requested review from andreip136 andreip136 is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@giurgiur99 @ndrpp