chore(modules-sdk, medusa): Cleanup migration helpers (#7342)

**What**
Cleanup the migrations scripts

packages/core/modules-sdk/src/medusa-app.ts

@@ -389,15 +389,15 @@ async function MedusaApp_({
         }
       }
 
-      if (!revert) {
+      if (revert) {
-        await MedusaModule.migrateUp(
+        await MedusaModule.migrateDown(
           moduleResolution.definition.key,
           moduleResolution.resolutionPath as string,
           moduleResolution.options,
           moduleResolution.moduleExports
         )
       } else {
-        await MedusaModule.migrateDown(
+        await MedusaModule.migrateUp(
           moduleResolution.definition.key,
           moduleResolution.resolutionPath as string,
           moduleResolution.options,
@@ -411,15 +411,15 @@ async function MedusaApp_({
       ...(sharedResourcesConfig?.database ?? {}),
     }
 
-    if (!revert) {
+    if (revert) {
-      linkModuleMigration &&
+      revertLinkModuleMigration &&
-        (await linkModuleMigration({
+        (await revertLinkModuleMigration({
           options: linkModuleOpt,
           injectedDependencies,
         }))
     } else {
-      revertLinkModuleMigration &&
+      linkModuleMigration &&
-        (await revertLinkModuleMigration({
+        (await linkModuleMigration({
           options: linkModuleOpt,
           injectedDependencies,
         }))

packages/medusa/src/commands/migrate.js

@@ -8,9 +8,12 @@ import {
 import configModuleLoader from "../loaders/config"
 import featureFlagLoader from "../loaders/feature-flags"
 import Logger from "../loaders/logger"
-import { loadMedusaApp, migrateMedusaApp } from "../loaders/medusa-app"
+import {
+  loadMedusaApp,
+  migrateMedusaApp,
+  revertMedusaApp,
+} from "../loaders/medusa-app"
 import pgConnectionLoader from "../loaders/pg-connection"
-import { MedusaAppMigrateDown } from "@medusajs/modules-sdk"
 
 const runLinkMigrations = async (directory) => {
   const configModule = configModuleLoader(directory)
@@ -57,18 +60,12 @@ const main = async function ({ directory }) {
   })
 
   if (args[0] === "run") {
-    await migrateMedusaApp(
+    await migrateMedusaApp({ configModule, container })
-      { configModule, container },
-      { registerInContainer: false }
-    )
 
     Logger.info("Migrations completed.")
     process.exit()
   } else if (args[0] === "revert") {
-    await MedusaAppMigrateDown(
+    await revertMedusaApp({ configModule, container })
-      { configModule, container },
-      { registerInContainer: false }
-    )
     await revertIsolatedModulesMigration(configModule)
     Logger.info("Migrations reverted.")
   } else if (args[0] === "show") {

packages/medusa/src/loaders/medusa-app.ts

@@ -1,9 +1,10 @@
 import {
-  MODULE_PACKAGE_NAMES,
   MedusaApp,
+  MedusaAppMigrateDown,
   MedusaAppMigrateUp,
   MedusaAppOutput,
   MedusaModule,
+  MODULE_PACKAGE_NAMES,
   Modules,
   ModulesDefinition,
 } from "@medusajs/modules-sdk"
@@ -58,19 +59,18 @@ export function mergeDefaultModules(
   return configModules
 }
 
-export async function migrateMedusaApp(
+async function runMedusaAppMigrations({
-  {
+  configModule,
-    configModule,
+  container,
-    container,
+  revert = false,
-  }: {
+}: {
-    configModule: {
+  configModule: {
-      modules?: CommonTypes.ConfigModule["modules"]
+    modules?: CommonTypes.ConfigModule["modules"]
-      projectConfig: CommonTypes.ConfigModule["projectConfig"]
+    projectConfig: CommonTypes.ConfigModule["projectConfig"]
-    }
+  }
-    container: MedusaContainer
+  container: MedusaContainer
-  },
+  revert?: boolean
-  config = { registerInContainer: true }
+}): Promise<void> {
-): Promise<void> {
   const injectedDependencies = {
     [ContainerRegistrationKeys.PG_CONNECTION]: container.resolve(
       ContainerRegistrationKeys.PG_CONNECTION
@@ -92,12 +92,55 @@ export async function migrateMedusaApp(
   }
   const configModules = mergeDefaultModules(configModule.modules)
 
-  await MedusaAppMigrateUp({
+  if (revert) {
-    modulesConfig: configModules,
+    await MedusaAppMigrateDown({
-    remoteFetchData: remoteQueryFetchData(container),
+      modulesConfig: configModules,
-    sharedContainer: container,
+      remoteFetchData: remoteQueryFetchData(container),
-    sharedResourcesConfig,
+      sharedContainer: container,
-    injectedDependencies,
+      sharedResourcesConfig,
+      injectedDependencies,
+    })
+  } else {
+    await MedusaAppMigrateUp({
+      modulesConfig: configModules,
+      remoteFetchData: remoteQueryFetchData(container),
+      sharedContainer: container,
+      sharedResourcesConfig,
+      injectedDependencies,
+    })
+  }
+}
+
+export async function migrateMedusaApp({
+  configModule,
+  container,
+}: {
+  configModule: {
+    modules?: CommonTypes.ConfigModule["modules"]
+    projectConfig: CommonTypes.ConfigModule["projectConfig"]
+  }
+  container: MedusaContainer
+}): Promise<void> {
+  await runMedusaAppMigrations({
+    configModule,
+    container,
+  })
+}
+
+export async function revertMedusaApp({
+  configModule,
+  container,
+}: {
+  configModule: {
+    modules?: CommonTypes.ConfigModule["modules"]
+    projectConfig: CommonTypes.ConfigModule["projectConfig"]
+  }
+  container: MedusaContainer
+}): Promise<void> {
+  await runMedusaAppMigrations({
+    configModule,
+    container,
+    revert: true,
   })
 }