Skip to content

Fix documentation build ESM/CommonJS conflict and tar vulnerability #7348

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.

Sign up for GitHub

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

Jump to bottom
Open
afarber wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix documentation build ESM/CommonJS conflict and tar vulnerability #7348

afarber wants to merge 1 commit into from
+7 −5

Conversation

@afarber
Copy link
Contributor

@afarber afarber commented Jan 29, 2026 •
edited
Loading

Issue

Fixes #7347

  • Fix docs build by using Babel's --out-file-extension .cjs flag so transpiled output is treated as CommonJS
  • Additionally, run "npm audit --fix" to fix GHSA-34x7-hfp2-rc4v vulnerability

Tasklist

Requirements / Relations

TheMarex
TheMarex reviewed Jan 29, 2026
View reviewed changes
@afarber afarber force-pushed the 7347-fix-npm-run-docs branch from e03e122 to c95b6b4 Compare February 1, 2026 19:46
@afarber afarber requested a review from TheMarex February 1, 2026 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DennisOSRM DennisOSRM DennisOSRM approved these changes

@TheMarex TheMarex Awaiting requested review from TheMarex

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Documentation build fails: "require is not defined in ES module scope"

3 participants

@afarber @TheMarex @DennisOSRM