Skip to content

fix(apollo-react, apollo-wind): bump react-syntax-highlighter to ^16.1.1 (ESM fix)#808

Merged
david-rios-uipath merged 2 commits into
mainfrom
rios/fix-react-syntax-highlighter-esm
Jun 11, 2026
Merged

fix(apollo-react, apollo-wind): bump react-syntax-highlighter to ^16.1.1 (ESM fix)#808
david-rios-uipath merged 2 commits into
mainfrom
rios/fix-react-syntax-highlighter-esm

Conversation

@david-rios-uipath

@david-rios-uipath david-rios-uipath commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Bumps react-syntax-highlighter from ^16.1.0 to ^16.1.1 in apollo-react
  • 16.1.0 has broken ESM exports (extensionless imports in dist/esm/) that fail under Node v24's strict ESM resolution
  • apollo-react's ap-chat/code component imports from react-syntax-highlighter/dist/esm/styles/prism, triggering the issue in consumers (e.g. traceview-ui)

Before

"Cannot find module" errors in testing
image

After

Successful test coverage run
image

Test plan

  • Verify apollo-react builds cleanly
  • Verify consumers (traceview-ui) can drop the react-syntax-highlighter override after this ships

16.1.0 has broken ESM exports (extensionless imports in dist/esm/) that
fail under Node v24 strict ESM resolution. apollo-react's ap-chat code
component imports from the ESM path, triggering the issue in consumers.

16.1.1 adds .js extensions to all ESM imports.
See: react-syntax-highlighter/react-syntax-highlighter#627
Copilot AI review requested due to automatic review settings June 11, 2026 19:55
@github-actions

github-actions Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (PT)
apollo-design 🟢 Ready Preview, Logs Jun 11, 2026, 01:11:20 PM
apollo-docs 🟢 Ready Preview, Logs Jun 11, 2026, 01:11:20 PM
apollo-landing 🟢 Ready Preview, Logs Jun 11, 2026, 01:11:20 PM
apollo-vertex 🟢 Ready Preview, Logs Jun 11, 2026, 01:11:20 PM

@github-actions

Copy link
Copy Markdown
Contributor

Dependency License Review

  • 1922 package(s) scanned
  • ✅ No license issues found
  • ⚠️ 2 package(s) excluded (see details below)
License distribution
License Packages
MIT 1692
ISC 89
Apache-2.0 55
BSD-3-Clause 27
BSD-2-Clause 23
BlueOak-1.0.0 8
MPL-2.0 4
MIT-0 3
CC0-1.0 3
MIT OR Apache-2.0 2
(MIT OR Apache-2.0) 2
Unlicense 2
LGPL-3.0-or-later 1
Python-2.0 1
CC-BY-4.0 1
(MPL-2.0 OR Apache-2.0) 1
Unknown 1
Artistic-2.0 1
(WTFPL OR MIT) 1
(BSD-2-Clause OR MIT OR Apache-2.0) 1
CC-BY-3.0 1
0BSD 1
(MIT OR CC0-1.0) 1
MIT AND ISC 1
Excluded packages
Package Version License Reason
@img/sharp-libvips-linux-x64 1.2.4 LGPL-3.0-or-later LGPL pre-built binary, not linked
khroma 2.1.0 Unknown MIT per GitHub repo, missing license field in package.json

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates apollo-react’s react-syntax-highlighter dependency to pick up upstream’s ESM export fix (adds .js extensions in ESM build) to avoid strict ESM resolution failures in newer Node versions.

Changes:

  • Bump react-syntax-highlighter in packages/apollo-react from ^16.1.0 to ^16.1.1.
  • Update pnpm-lock.yaml to resolve react-syntax-highlighter to 16.1.1.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File Description
pnpm-lock.yaml Locks react-syntax-highlighter to 16.1.1 for the workspace.
packages/apollo-react/package.json Raises the minimum react-syntax-highlighter version to ^16.1.1 in apollo-react.
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Comment thread packages/apollo-react/package.json
@david-rios-uipath david-rios-uipath changed the title fix(apollo-react): bump react-syntax-highlighter to ^16.1.1 (ESM fix) fix(apollo-react, apollo-wind): bump react-syntax-highlighter to ^16.1.1 (ESM fix) Jun 11, 2026
@david-rios-uipath david-rios-uipath added the dev-packages Adds dev package publishing on pushes to this PR label Jun 11, 2026
@github-actions

github-actions Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

📦 Dev Packages

🧹 Dev packages cleaned up after PR close.

Last updated: 2026-06-11 14:26:48 PT

@david-rios-uipath david-rios-uipath marked this pull request as ready for review June 11, 2026 21:15
Copilot AI review requested due to automatic review settings June 11, 2026 21:15

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated no new comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

@david-rios-uipath david-rios-uipath merged commit 8cad41b into main Jun 11, 2026
57 of 58 checks passed
@david-rios-uipath david-rios-uipath deleted the rios/fix-react-syntax-highlighter-esm branch June 11, 2026 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev-packages Adds dev package publishing on pushes to this PR pkg:apollo-react pkg:apollo-wind size:XS 0-9 changed lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants