Bug: Unable to resolve "@logto/client/shim" from "node_modules/@logto/rn/lib/index.js"

[alwyntan]

Describe the bug

@logto/client/shim seems to not be exported from @logto/client and errors.

Expected behavior

No crashes

How to reproduce?

Install @logto/rn and import in the root component.

Context

• Logto Cloud
• Self-hosted, Logto version =
  • Container (Docker image)
  • Raw Node.js

[alwyntan]

There might be some linking problems with @logto/client/shim, but I'm quite unfamiliar with how that is bundled. The naive way to fix this (which I have tried locally) is to just patch the node_modules for every file that references @logto/client/shim to @logto/client/lib/shim

[gao-sun]

I'll take a look on it. It seems somehow the resolver doesn't honor the exports filed in package.json.

[gao-sun]

Which RN/Expo version are you using?

[alwyntan]

Using RN0.73.4 and expo 50.0.11

[shibilyt]

any progress in this? I am also facing the same issue.

[Mohamed-kassim]

same issue here expo sdk50 any progress?
expo: 50.0.17 and expo router
@gao-sun

[elvingomezdev]

Same problem, any update?

expo: 50.0.14

[JakimLi]

same issue, is this repo still being maintained? any update about this?

[flavindias]

same problem here, any updates?

[JakimLi]

I did some exploration, and finally solved this issue, this is not working because react native does not support package exports by default, I did below config to make it work:

in metro.config.js , create if not exist.

const { getDefaultConfig } = require('expo/metro-config');  
// because I am using expo, so have to import expo's 
// default metro config, otherwise expo will complain with errors.

const config = getDefaultConfig(__dirname);

config.resolver.unstable_enablePackageExports = true;

module.exports = config;

this is the related react native doc: https://reactnative.dev/blog/2023/06/21/package-exports-support
hope this helps

[Housain-maina]

I did some exploration, and finally solved this issue, this is not working because react native does not support package exports by default, I did below config to make it work:

in metro.config.js , create if not exist.

const { getDefaultConfig } = require('expo/metro-config');  
// because I am using expo, so have to import expo's 
// default metro config, otherwise expo will complain with errors.

const config = getDefaultConfig(__dirname);

config.resolver.unstable_enablePackageExports = true;

module.exports = config;

this is the related react native doc: reactnative.dev/blog/2023/06/21/package-exports-support hope this helps

finally a fix, thanks

[gao-sun]

yes i'm working on the update. however i could not reproduce this issue. can you provide a repo for this issue?

if i can reproduce it, i'll also do a patch to fix this issue

[gao-sun]

@JakimLi thanks for providing the workaround. i'm curious of reproducing this issue because it went just fine on my laptop. even if i change import to require in packages/rn-sample.

[r3dm4n]

Adding this breaks axios for me.

[EDIT]
I have tried adding axios to the sample, and got the same result.

[r3dm4n]

Is there any fix for this ?

[isener]

I did some exploration, and finally solved this issue, this is not working because react native does not support package exports by default, I did below config to make it work:

in metro.config.js , create if not exist.

const { getDefaultConfig } = require('expo/metro-config');  
// because I am using expo, so have to import expo's 
// default metro config, otherwise expo will complain with errors.

const config = getDefaultConfig(__dirname);

config.resolver.unstable_enablePackageExports = true;

module.exports = config;

this is the related react native doc: https://reactnative.dev/blog/2023/06/21/package-exports-support hope this helps

This has sorted the issue!
Thanks a ton!!

EDIT:
This breaks react navigation. Still no workaround.

[simeng-li]

Enabling packageExports in the Metro config is necessary to properly load @logto/client/shim. We've updated the integration documentation to include this information.

@isener, could you please share the detailed error information here?

It seems React Native needs to release a stable version that fully supports npm package exports.

[fvcoder]

I did some exploration, and finally solved this issue, this is not working because react native does not support package exports by default, I did below config to make it work:

in metro.config.js , create if not exist.

const { getDefaultConfig } = require('expo/metro-config');  
// because I am using expo, so have to import expo's 
// default metro config, otherwise expo will complain with errors.

const config = getDefaultConfig(__dirname);

config.resolver.unstable_enablePackageExports = true;

module.exports = config;

this is the related react native doc: https://reactnative.dev/blog/2023/06/21/package-exports-support hope this helps

Works

[dipakparmar]

config.resolver.unstable_enablePackageExports = true;

Turning this on breaks other packages!

[simeng-li]

@dipakparmar can you try the latest 0.4.0 release. We have removed the import of the shim package. unstable_enablePackageExports is no longer needed.

[dipakparmar]

@dipakparmar can you try the latest 0.4.0 release. We have removed the import of the shim package. unstable_enablePackageExports is no longer needed.

Hey @simeng-li Thanks. Did the quick test and it works. Will do full test later this weekend and let you know. But so far its working.

[dipakparmar]

Confirming, this is resolved. Can successfully build the app on iOS.