Enable pnpm symlinks without hoisting

[karlhorky]

Wonder if this new @rnx-kit/metro-resolver-symlinks could be used also with an Expo project... 🤔 (to avoid the node-linker=hoist in pnpm)

facebook/metro#1 (comment)

Looks like this kitchen-sink Expo app is using it 👀

https://github.com/tamagui/tamagui/blob/fd7105ebf1ce6feb362dcad5da64f205e6b71b69/apps/kitchen-sink/metro.config.js#L2-L9

cc @ProchaLu

[karlhorky]

Reant Native 0.72 with symlink support was just released:

https://reactnative.dev/blog/2023/06/21/0.72-metro-package-exports-symlinks

As soon as Expo upgrades and supports this version, we can try it out

[karlhorky]

Expo SDK 50 is out, which uses React Native 0.73 internally.

React 0.73 has made symlink support stable.

If it doesn't work out of the box (without node-linker=hoisted or any other workarounds), maybe the details in this issue would be helpful:

• Experimental symlink support not working for PNPM Monorepo facebook/metro#1042

Specifically, maybe this workaround is helpful (although unstable_enableSymlinks no longer needed for RN 0.73):

const path = require('path');

const config = {
  watchFolders: [path.join(__dirname, '..', '..')],
  resolver: {
    unstable_enableSymlinks: true,
    unstable_enablePackageExports: true,
  },
};

[karlhorky]

Seems like Expo SDK 50 with RN 0.73 is still a problem for pnpm without hoisting config, at least for now:

• Pnpm not working with Expo expo/expo#22413 (comment)

[karlhorky]

Also asked Expo and pnpm maintainers over here in the open pnpm issue about whether it would be possible to get symlinks support out of the box:

• [bug] - Pnpm not working with expo pnpm/pnpm#4286 (comment)