Skip to content

feat: live-markdown working in both Bundle Mode and Legacy Eval Mode#769

Open
tjzel wants to merge 3 commits into
Expensify:mainfrom
tjzel:@tjzel/both-worklet-modes
Open

feat: live-markdown working in both Bundle Mode and Legacy Eval Mode#769
tjzel wants to merge 3 commits into
Expensify:mainfrom
tjzel:@tjzel/both-worklet-modes

Conversation

@tjzel

@tjzel tjzel commented Jul 21, 2026

Copy link
Copy Markdown

Details

I made changes necessary for the library to be working in both modes.

Related Issues

n/a

Manual Tests

I ran the app in both modes and it works - there's also a mode display text in the example.

Linked PRs

n/a

@war-in war-in left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you for the contribution! I left two small comments and once addressed we can merge it

Comment thread README.md
### Worklets Bundle Mode

If you're using [Worklets Bundle Mode](https://docs.swmansion.com/react-native-worklets/docs/bundleMode) you will have to
configure your [import forwarding](https://docs.swmansion.com/react-native-worklets/docs/worklets-babel-plugin/plugin-options#importforwarding) as follows in your `babel.config.js` file:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Let's also redirect users to the correct section of the Worklets docs about their metro setup (metro.config.js, and the patches). The current note suggests that they only have to configure import forwarding

Comment thread example/package.json
Comment on lines -9 to +10
"start": "react-native start --client-logs"
"start": "react-native start --client-logs",
"bundle-mode-start": "BUNDLE_MODE=1 react-native start --reset-cache --client-logs"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I wonder if we shouldn't make it the default 🤔 Are there any drawbacks to always enabling bundle mode?

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.

2 participants