Added CLI arguments for password (not recommended) and startup plugin runs. Fixed a bug in the plugin framework crashing.

Author: sebinside

src/main/scala/org/codeoverflow/chatoverflow/Launcher.scala

@@ -19,14 +19,36 @@ object Launcher extends WithLogger {
   def main(args: Array[String]): Unit = {
     parse(args) { config =>
 
+      // Set globally available plugin data path
       this.pluginDataPath = config.pluginDataPath
 
       // The complete project visible trough one single instance
       val chatOverflow = new ChatOverflow(config.pluginFolderPath,
         config.configFolderPath, config.requirementPackage, config.requirePasswordOnStartup, config.pluginLogOutputOnConsole)
 
+      // Initialize chat overflow
       chatOverflow.init()
 
+      // Login if a password is specified
+      if (config.loginPassword.nonEmpty && !chatOverflow.isLoaded) {
+        chatOverflow.credentialsService.setPassword(config.loginPassword)
+        chatOverflow.load()
+      }
+
+      // Start plugins if specified
+      // TODO: Move this down after server start when the REPL is history
+      if (chatOverflow.isLoaded && config.startupPlugins.nonEmpty) {
+        for (instanceName <- config.startupPlugins) {
+          val instance = chatOverflow.pluginInstanceRegistry.getPluginInstance(instanceName)
+
+          if (instance.isEmpty) {
+            println(s"No plugin instance named '$instanceName' was registered.")
+          } else {
+            instance.get.start()
+          }
+        }
+      }
+
       // Launch UI
       config.ui match {
         case UI.GUI =>

src/main/scala/org/codeoverflow/chatoverflow/framework/PluginFramework.scala

@@ -91,7 +91,9 @@ class PluginFramework(pluginDirectoryPath: String) extends WithLogger {
               logger info s"Successfully tested instantiation of plugin '${plugin.getName}'"
               pluginTypes += plugin
             } catch {
-              case _: Exception => logger warn s"Error while test init of plugin '${plugin.getName}'."
+              // Note that we catch not only exceptions, but also errors like NoSuchMethodError. Deep stuff
+              case _: Error => logger warn s"Error while test init of plugin '${plugin.getName}'."
+              case _: Exception => logger warn s"Exception while test init of plugin '${plugin.getName}'."
             }
           }
         }

src/main/scala/org/codeoverflow/chatoverflow/ui/CLI.scala

@@ -30,10 +30,16 @@ object CLI {
     opt[String]('r', "requirementPackage").action((x, c) =>
       c.copy(requirementPackage = x)).text("path to the package where all requirements are defined")
 
-    // Subject of change. After GUI will be -l (for requireLogin)
+    // Subject of change. After GUI will be -l (for login)
     opt[Unit]('n', "noPassword").action((_, c) =>
       c.copy(requirePasswordOnStartup = false)).text("set this flag to disable password checking on framework startup")
 
+    opt[String]('l', "login").action((x, c) =>
+      c.copy(loginPassword = x.toCharArray)).text("the password to login to chat overflow (not recommended, has to be combined with -n)")
+
+    opt[Seq[String]]('s', "start").action((x, c) =>
+      c.copy(startupPlugins = x)).text("a comma-separated list of plugin instances to start after login (has to be combined with -n and -l)")
+
     opt[String]('d', "pluginDataFolder").action((x, c) =>
       c.copy(pluginDataPath = x)).text("path to the data folder, accessible from within plugins")
 
@@ -73,7 +79,9 @@ object CLI {
                     requirePasswordOnStartup: Boolean = true,
                     pluginDataPath: String = "data",
                     webServerPort: Int = 2400,
-                    pluginLogOutputOnConsole: Boolean = true)
+                    pluginLogOutputOnConsole: Boolean = true,
+                    loginPassword: Array[Char] = Array[Char](),
+                    startupPlugins: Seq[String] = Seq[String]())
 
   object UI extends Enumeration {
     type UI = Value