Skip to content

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

Merged
TheMarex merged 1 commit intoProject-OSRM:masterfrom
afarber:7347-fix-npm-run-docs
Feb 6, 2026
Merged

Fix documentation build ESM/CommonJS conflict and tar vulnerability#7348
TheMarex merged 1 commit intoProject-OSRM:masterfrom
afarber:7347-fix-npm-run-docs

Conversation

@afarber
Copy link
Copy Markdown
Contributor

@afarber afarber commented Jan 29, 2026

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

Comment thread package.json Outdated
@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
@TheMarex
Copy link
Copy Markdown
Member

TheMarex commented Feb 6, 2026

Sorry forgot about this. Looks good now.

@TheMarex TheMarex merged commit d8ff021 into Project-OSRM:master Feb 6, 2026
19 checks passed
@afarber afarber deleted the 7347-fix-npm-run-docs branch February 6, 2026 10:36
MarcelloPerathoner pushed a commit to MarcelloPerathoner/osrm-backend that referenced this pull request Feb 24, 2026
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.

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

3 participants