Skip to content

fix(react): update exporters and guard browser runtime - #47

Merged
ivoIturrieta merged 1 commit into
mainfrom
fix/exporters-browser-runtime
Jul 15, 2026
Merged

fix(react): update exporters and guard browser runtime#47
ivoIturrieta merged 1 commit into
mainfrom
fix/exporters-browser-runtime

Conversation

@ivoIturrieta

@ivoIturrieta ivoIturrieta commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

What does this PR do? Keep it brief.

Changes

Test Plan

  • pnpm build passes
  • pnpm test passes
  • Tested in Storybook (if UI change)
  • Added/updated tests for new behavior

Notes

Any additional context for reviewers.


📖 Storybook Preview: https://unlayer.github.io/elements/pr/47/

Copilot AI review requested due to automatic review settings July 15, 2026 17:11

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 the @unlayer/react-elements dependency pins and adds a CI/runtime probe to ensure the shipped React package can be bundled and executed in a real browser environment without relying on Node globals (notably process).

Changes:

  • Bump @unlayer/exporters and @unlayer/types catalog pins to 1.455.0.
  • Add a Playwright + tsup “browser runtime safety gate” that bundles the package entry and runs renderToHtml() in Chromium with no process global.
  • Wire the new runtime gate into @unlayer/react-elements scripts and the main CI workflow.

Reviewed changes

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

Show a summary per file
File Description
pnpm-workspace.yaml Updates catalog pins for Unlayer packages.
pnpm-lock.yaml Updates lockfile resolutions for the new catalog versions.
packages/react/scripts/browser-runtime-probe.mjs New Node script that bundles + executes runtime probe in Chromium and enforces “no process global” behavior.
packages/react/scripts/browser-runtime-entry.mjs New browser entry used by the probe to call renderToHtml() and publish results to globalThis.
packages/react/package.json Adds test:browser-runtime script to run the new probe.
.github/workflows/test.yml Adds CI step to run the browser runtime safety gate.
Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/react/scripts/browser-runtime-probe.mjs
@ivoIturrieta
ivoIturrieta merged commit 19664a0 into main Jul 15, 2026
3 checks passed
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.

3 participants