Mark plugin manager impl as EagerInitialization so plugins will load …

…on startup

recaf-core/src/main/java/software/coley/recaf/services/plugin/BasicPluginManager.java

@@ -7,8 +7,10 @@
 import jakarta.inject.Inject;
 import org.slf4j.Logger;
 import software.coley.recaf.analytics.logging.Logging;
+import software.coley.recaf.cdi.EagerInitialization;
 import software.coley.recaf.plugin.*;
 import software.coley.recaf.services.file.RecafDirectoriesConfig;
+import software.coley.recaf.util.TestEnvironment;
 import software.coley.recaf.util.io.ByteSource;
 import software.coley.recaf.util.io.ByteSources;
 
@@ -24,6 +26,7 @@
  * @author xDark
  */
 @ApplicationScoped
+@EagerInitialization
 public class BasicPluginManager implements PluginManager {
 	private static final Logger logger = Logging.get(BasicPluginManager.class);
 	private final List<PluginLoader> loaders = new ArrayList<>();
@@ -45,7 +48,11 @@ public BasicPluginManager(PluginManagerConfig config, CdiClassAllocator classAll
 	@PostConstruct
 	@SuppressWarnings("unused")
 	private void setup(RecafDirectoriesConfig directoriesConfig) {
-		if (config.isAllowLocalScan())
+		// Do not load plugins in a test environment
+		if (TestEnvironment.isTestEnv())
+			return;
+
+		if (config.doScanOnStartup())
 			scanPlugins(directoriesConfig.getPluginDirectory());
 	}
 
@@ -135,14 +142,20 @@ public void removeLoader(@Nonnull PluginLoader loader) {
 		loaders.remove(loader);
 	}
 
+	@Override
+	public boolean isPluginLoaded(@Nonnull String name) {
+		return nameMap.get(name) != null;
+	}
+
 	@Nonnull
 	@Override
 	public <T extends Plugin> PluginContainer<T> loadPlugin(@Nonnull PluginContainer<T> container) throws PluginLoadException {
 		String name = container.getInformation().getName();
 		if (nameMap.putIfAbsent(name.toLowerCase(Locale.ROOT), container) != null) {
-			// Plugin already exists, we do not allow
-			// multiple plugins with the same name.
+			// Plugin already exists, we do not allow multiple plugins with the same name.
+			// The passed in plugin container will be disabled since it shouldn't be used.
 			try {
+				logger.warn("Attempted to load duplicate instance of plugin '{}'", name);
 				container.getLoader().disablePlugin(container);
 			} catch (Exception ignored) {
 			}

recaf-core/src/main/java/software/coley/recaf/services/plugin/PluginManager.java

@@ -129,6 +129,8 @@ default <T extends Plugin> PluginContainer<T> loadPlugin(@Nonnull ByteSource sou
 
 				// Load and record plugin container
 				PluginContainer<T> container = loader.load(getAllocator(), source);
+				PluginContainer<Plugin> existingPlugin = getPlugin(container.getInformation().getName());
+
 				PluginContainer<T> loadedContainer = loadPlugin(container);
 				if (shouldEnablePluginOnLoad(loadedContainer))
 					loader.enablePlugin(loadedContainer);
@@ -140,6 +142,16 @@ default <T extends Plugin> PluginContainer<T> loadPlugin(@Nonnull ByteSource sou
 		throw new PluginLoadException("Plugin manager was unable to locate suitable loader for the source.");
 	}
 
+	/**
+	 * Checks if a plugin is loaded.
+	 *
+	 * @param name
+	 * 		Name of plugin to check for.
+	 *
+	 * @return {@code true} if the plugin has been registered/loaded by this manager.
+	 */
+	boolean isPluginLoaded(@Nonnull String name);
+
 	/**
 	 * Loads and registers a plugin.
 	 *

recaf-core/src/main/java/software/coley/recaf/services/plugin/PluginManagerConfig.java

@@ -1,8 +1,11 @@
 package software.coley.recaf.services.plugin;
 
+import jakarta.annotation.Nonnull;
 import jakarta.enterprise.context.ApplicationScoped;
 import jakarta.inject.Inject;
+import software.coley.observables.ObservableBoolean;
 import software.coley.recaf.config.BasicConfigContainer;
+import software.coley.recaf.config.BasicConfigValue;
 import software.coley.recaf.config.ConfigGroups;
 import software.coley.recaf.services.ServiceConfig;
 
@@ -13,27 +16,19 @@
  */
 @ApplicationScoped
 public class PluginManagerConfig extends BasicConfigContainer implements ServiceConfig {
-	private boolean allowLocalScan = true;
+	private final ObservableBoolean scanOnStartup = new ObservableBoolean(true);
 
 	@Inject
 	public PluginManagerConfig() {
 		super(ConfigGroups.SERVICE_PLUGIN, PluginManager.SERVICE_ID + CONFIG_SUFFIX);
+		addValue(new BasicConfigValue<>("scan-on-start", boolean.class, scanOnStartup));
 	}
 
 	/**
 	 * @return {@code true} when local plugins should be scanned when the plugin manager implementation initializes.
 	 * {@code false} to disable local automatic plugin loading.
 	 */
-	public boolean isAllowLocalScan() {
-		return allowLocalScan;
-	}
-
-	/**
-	 * @param allowLocalScan
-	 *        {@code true} when local plugins should be scanned when the plugin manager implementation initializes.
-	 *        {@code false} to disable local automatic plugin loading.
-	 */
-	public void setAllowLocalScan(boolean allowLocalScan) {
-		this.allowLocalScan = allowLocalScan;
+	public boolean doScanOnStartup() {
+		return scanOnStartup.getValue();
 	}
 }

recaf-core/src/testFixtures/java/software/coley/recaf/test/TestBase.java

@@ -27,10 +27,6 @@ public static void setupWorkspaceManager() {
 		// We'll use this a lot so may as well grab it
 		workspaceManager = recaf.get(WorkspaceManager.class);
 		workspaceManager.setCurrent(null);
-
-		// Disable plugin scanning in tests
-		PluginManagerConfig pluginConfig = recaf.get(PluginManagerConfig.class);
-		pluginConfig.setAllowLocalScan(false);
 	}
 
 	/**