Started development agreed with dieppa

Removed files and code previously featured in mongock standalone

Added new role in core runner builder, named ModuleConfigurationProvider

Modified RunnerBuilder to incorporate ModuleConfigurationProvider

Added new attribute in MongockConfiguration named mongockModules, and respective getter and setter

Author: DanielSchiavo

mongock-core/mongock-api/src/main/java/io/mongock/api/config/MongockConfiguration.java

@@ -1,6 +1,7 @@
 package io.mongock.api.config;
 
 import io.mongock.api.config.executor.ExecutorConfiguration;
+import io.mongock.api.module.MongockModuleConfig;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -91,6 +92,11 @@ public class MongockConfiguration implements ExecutorConfiguration {
    */
   private List<String> migrationScanPackage = new ArrayList<>();
 
+  /**
+   * List of Mongock modules that provide configuration, such as ChangeLog scan packages. Optional
+   */
+  private List<MongockModuleConfig> mongockModules = new ArrayList<>();
+
   /**
    * System version to start with. Default '0'
    */
@@ -260,6 +266,14 @@ public void setMigrationScanPackage(List<String> migrationScanPackage) {
     this.migrationScanPackage = migrationScanPackage;
   }
 
+  public List<MongockModuleConfig> getMongockModules() {
+    return mongockModules;
+  }
+
+  public void setMongockModules(List<MongockModuleConfig> mongockModules) {
+    this.mongockModules = mongockModules;
+  }
+
   public boolean isIndexCreation() {
     return indexCreation;
   }
@@ -398,6 +412,7 @@ public boolean equals(Object o) {
         Objects.equals(lockRepositoryName, that.lockRepositoryName) &&
         Objects.equals(lockQuitTryingAfterMillis, that.lockQuitTryingAfterMillis) &&
         Objects.equals(migrationScanPackage, that.migrationScanPackage) &&
+        Objects.equals(mongockModules, that.mongockModules) &&
         Objects.equals(startSystemVersion, that.startSystemVersion) &&
         Objects.equals(endSystemVersion, that.endSystemVersion) &&
         Objects.equals(serviceIdentifier, that.serviceIdentifier) &&

mongock-core/mongock-api/src/main/java/io/mongock/api/module/MongockModuleConfig.java

@@ -0,0 +1,17 @@
+package io.mongock.api.module;
+
+import java.util.List;
+
+/**
+ * Represents a module that provides configuration to Mongock,
+ * specifically the list of packages to be scanned for change units.
+ * This allows for a clean and extensible way to add scan packages from different modules.
+ */
+public interface MongockModuleConfig {
+
+  /**
+   * @return A list of package paths to be scanned by Mongock for change units.
+   */
+  List<String> getChangeLogScanPackages();
+
+}

mongock-core/mongock-runner/mongock-runner-core/src/main/java/io/mongock/runner/core/builder/RunnerBuilder.java

@@ -1,16 +1,7 @@
 package io.mongock.runner.core.builder;
 
 import io.mongock.api.config.MongockConfiguration;
-import io.mongock.runner.core.builder.roles.ChangeLogScanner;
-import io.mongock.runner.core.builder.roles.DependencyInjectable;
-import io.mongock.runner.core.builder.roles.DriverConnectable;
-import io.mongock.runner.core.builder.roles.LegacyMigrator;
-import io.mongock.runner.core.builder.roles.MigrationWriter;
-import io.mongock.runner.core.builder.roles.MongockRunnable;
-import io.mongock.runner.core.builder.roles.ServiceIdentificable;
-import io.mongock.runner.core.builder.roles.SystemVersionable;
-import io.mongock.runner.core.builder.roles.TransactionStrategiable;
-import io.mongock.runner.core.builder.roles.Transactioner;
+import io.mongock.runner.core.builder.roles.*;
 import io.mongock.runner.core.event.EventPublisher;
 
 @SuppressWarnings("all")
@@ -27,7 +18,8 @@ public interface RunnerBuilder<
 		ServiceIdentificable<SELF, CONFIG>,
     MongockRunnable<SELF, CONFIG>,
         Transactioner<SELF, CONFIG>,
-		TransactionStrategiable<SELF, CONFIG> {
+		TransactionStrategiable<SELF, CONFIG>,
+    ModuleConfigurationProvider<SELF, CONFIG> {
 
   SELF setEventPublisher(EventPublisher eventPublisher);
 }

mongock-core/mongock-runner/mongock-runner-core/src/main/java/io/mongock/runner/core/builder/roles/ModuleConfigurationProvider.java

@@ -0,0 +1,22 @@
+package io.mongock.runner.core.builder.roles;
+
+import io.mongock.api.config.MongockConfiguration;
+import io.mongock.api.module.MongockModuleConfig;
+
+import java.util.Collection;
+import java.util.Collections;
+
+public interface ModuleConfigurationProvider<SELF extends ModuleConfigurationProvider<SELF, CONFIG>, CONFIG extends MongockConfiguration>
+    extends Configurable<SELF, CONFIG>, SelfInstanstiator<SELF> {
+
+  default SELF addModule(MongockModuleConfig moduleConfig) {
+    return addModules(Collections.singletonList(moduleConfig));
+  }
+
+  default SELF addModules(Collection<MongockModuleConfig> moduleConfigs) {
+    if (moduleConfigs != null) {
+      getConfig().getMongockModules().addAll(moduleConfigs);
+    }
+    return getInstance();
+  }
+}

mongock-core/mongock-runner/standalone/mongock-standalone/src/main/java/io/mongock/runner/standalone/MongockModuleConfig.java

@@ -28,11 +28,6 @@ public RunnerStandaloneBuilderImpl getInstance() {
       return this;
     }
 
-    @Override
-    public RunnerStandaloneBuilder addModuleConfig(MongockModuleConfig mongockModuleConfig) {
-      mongockModuleConfig.configure(new MongockScanPackage(this));
-      return this;
-    }
   }
 
 

mongock-core/mongock-runner/standalone/mongock-standalone/src/main/java/io/mongock/runner/standalone/MongockScanPackage.java

@@ -5,5 +5,4 @@
 
 public interface RunnerStandaloneBuilder extends RunnerStandaloneBuilderBase<RunnerStandaloneBuilder, MongockConfiguration> {
 
-  public RunnerStandaloneBuilder addModuleConfig(MongockModuleConfig mongockModuleConfig);
 }