ember-simple-auth-oidc 4.1.0 breaks application that do not use apollo client

Upgrading to 4.1.0 breaks build of my applications:

ember-simple-auth-oidc tried to import "@apollo/client" in "ember-simple-auth-oidc/apollo.js" but the package was not resolvable from /storage/workspaces/leaf/leaf-sign/node_modules/ember-simple-auth-oidc

Is it intended? is ember-simple-auth-oidc going to require apollo?

Mar 23 '22 08:03 wmlele

Thank you for creating an issue for this @wmlele !

This is not intended. @apollo/client should be an optional peer dependency as specified here: https://github.com/adfinis/ember-simple-auth-oidc/blob/3c69b18222cbe4e296655f398221208b456867e8/package.json#L83-L90

I'll take a look at it to figure out why this is causing an error..

Oct 19 '22 07:10 anehx

This might be a bug of embroider, I created an issue: https://github.com/embroider-build/embroider/issues/1266

For now, we might have to make @apollo/client a dependency to resolve this issue..

Oct 20 '22 09:10 anehx

This is temporarly fixed in v5.0.1 with #685. However, I'm gonna leave this issue open since we need to revert that PR as soon as embroider fixes its issue.

Oct 20 '22 11:10 anehx

FWIW I still need to add graphql as deps to get the build to work with v5.0.3.

Apr 17 '23 19:04 bjornharrtell