feat: Move userpass default definition to defineConfig instead of a hard-coded value (#8557)

packages/core/utils/src/common/__tests__/define-config.spec.ts

@@ -11,7 +11,17 @@ describe("defineConfig", function () {
         "featureFlags": {},
         "modules": {
           "apiKey": true,
-          "auth": true,
+          "auth": {
+            "options": {
+              "providers": [
+                {
+                  "id": "emailpass",
+                  "resolve": "@medusajs/auth-emailpass",
+                },
+              ],
+            },
+            "resolve": "@medusajs/auth",
+          },
           "cacheService": true,
           "cart": true,
           "currency": true,
@@ -107,7 +117,17 @@ describe("defineConfig", function () {
         "featureFlags": {},
         "modules": {
           "apiKey": true,
-          "auth": true,
+          "auth": {
+            "options": {
+              "providers": [
+                {
+                  "id": "emailpass",
+                  "resolve": "@medusajs/auth-emailpass",
+                },
+              ],
+            },
+            "resolve": "@medusajs/auth",
+          },
           "cacheService": true,
           "cart": true,
           "currency": true,
@@ -206,7 +226,17 @@ describe("defineConfig", function () {
         "featureFlags": {},
         "modules": {
           "apiKey": true,
-          "auth": true,
+          "auth": {
+            "options": {
+              "providers": [
+                {
+                  "id": "emailpass",
+                  "resolve": "@medusajs/auth-emailpass",
+                },
+              ],
+            },
+            "resolve": "@medusajs/auth",
+          },
           "cacheService": true,
           "cart": true,
           "currency": true,
@@ -305,7 +335,17 @@ describe("defineConfig", function () {
         "featureFlags": {},
         "modules": {
           "apiKey": true,
-          "auth": true,
+          "auth": {
+            "options": {
+              "providers": [
+                {
+                  "id": "emailpass",
+                  "resolve": "@medusajs/auth-emailpass",
+                },
+              ],
+            },
+            "resolve": "@medusajs/auth",
+          },
           "cacheService": true,
           "currency": true,
           "customer": true,

packages/core/utils/src/common/define-config.ts

@@ -61,7 +61,6 @@ export function defineConfig(config: Partial<ConfigModule> = {}): ConfigModule {
   const modules: ConfigModule["modules"] = {
     [Modules.CACHE]: true,
     [Modules.EVENT_BUS]: true,
-    [Modules.AUTH]: true,
     [Modules.WORKFLOW_ENGINE]: true,
     [Modules.STOCK_LOCATION]: true,
     [Modules.INVENTORY]: true,
@@ -78,6 +77,18 @@ export function defineConfig(config: Partial<ConfigModule> = {}): ConfigModule {
     [Modules.CURRENCY]: true,
     [Modules.PAYMENT]: true,
     [Modules.ORDER]: true,
+    [Modules.AUTH]: {
+      resolve: "@medusajs/auth",
+      options: {
+        providers: [
+          {
+            resolve: "@medusajs/auth-emailpass",
+            id: "emailpass",
+          },
+        ],
+      },
+    },
+
     [Modules.USER]: {
       resolve: "@medusajs/user",
       options: {

packages/modules/auth/package.json

@@ -44,7 +44,6 @@
     "typescript": "^5.1.6"
   },
   "dependencies": {
-    "@medusajs/auth-emailpass": "0.0.1",
     "@medusajs/modules-sdk": "^1.12.9",
     "@medusajs/utils": "^1.11.7",
     "@mikro-orm/core": "5.9.7",

packages/modules/auth/src/loaders/providers.ts

@@ -1,5 +1,3 @@
-import EmailPassProvider from "@medusajs/auth-emailpass"
-
 import { LoaderOptions, ModulesSdkTypes, ModuleProvider } from "@medusajs/types"
 import { Lifetime, asFunction, asValue } from "awilix"
 import { moduleProviderLoader } from "@medusajs/modules-sdk"
@@ -33,18 +31,9 @@ export default async ({
     | ModulesSdkTypes.ModuleServiceInitializeCustomDataLayerOptions
   ) & { providers: ModuleProvider[] }
 >): Promise<void> => {
-  // Note: For now we want to inject some providers out of the box
-  const providerConfig = [
-    {
-      resolve: EmailPassProvider,
-      id: "emailpass",
-    },
-    ...(options?.providers ?? []),
-  ]
-
   await moduleProviderLoader({
     container,
-    providers: providerConfig,
+    providers: options?.providers || [],
     registerServiceFn: registrationFn,
   })
 }