Rename exception to be more generic

Author: yissachar

core/src/main/kotlin/cash/grammar/kotlindsl/parse/BuildScriptParseException.kt

@@ -0,0 +1,12 @@
+package cash.grammar.kotlindsl.parse
+
+public class KotlinParseException private constructor(msg: String) : RuntimeException(msg) {
+
+  public companion object {
+    public fun withErrors(messages: List<String>): KotlinParseException {
+      var i = 1
+      val msg = messages.joinToString { "${i++}: $it" }
+      return KotlinParseException(msg)
+    }
+  }
+}

core/src/main/kotlin/cash/grammar/kotlindsl/parse/KotlinParseException.kt

@@ -1,7 +1,7 @@
 package cash.grammar.kotlindsl.utils
 
 import cash.grammar.kotlindsl.model.RemovableBlock
-import cash.grammar.kotlindsl.parse.BuildScriptParseException
+import cash.grammar.kotlindsl.parse.KotlinParseException
 import cash.grammar.kotlindsl.parse.Parser
 import cash.grammar.kotlindsl.parse.Rewriter
 import cash.grammar.kotlindsl.utils.Whitespace.trimGently
@@ -28,10 +28,10 @@ public class BlockRemover private constructor(
   private val rewriter = Rewriter(tokens)
   private var terminalNewlines = 0
 
-  @Throws(BuildScriptParseException::class)
+  @Throws(KotlinParseException::class)
   public fun rewritten(): String {
     errorListener.getErrorMessages().ifNotEmpty {
-      throw BuildScriptParseException.withErrors(it)
+      throw KotlinParseException.withErrors(it)
     }
 
     return rewriter.text.trimGently(terminalNewlines)

core/src/main/kotlin/cash/grammar/kotlindsl/utils/BlockRemover.kt

@@ -1,7 +1,7 @@
 package cash.recipes.plugins
 
 import cash.grammar.kotlindsl.model.Plugin
-import cash.grammar.kotlindsl.parse.BuildScriptParseException
+import cash.grammar.kotlindsl.parse.KotlinParseException
 import cash.grammar.kotlindsl.parse.Parser
 import cash.grammar.kotlindsl.parse.Rewriter
 import cash.grammar.kotlindsl.utils.Blocks.isPlugins
@@ -37,7 +37,7 @@ import java.nio.file.Path
  *
  * @throws NonNormalizedScriptException if the script has not been normalized.
  * @throws IllegalArgumentException if conflicting plugins are found in both the pluginsToAdd and pluginsToRemove sets.
- * @throws BuildScriptParseException if the script cannot be parsed.
+ * @throws KotlinParseException if the script cannot be parsed.
  */
 public class PluginMutator private constructor(
   private val tokens: CommonTokenStream,
@@ -57,10 +57,10 @@ public class PluginMutator private constructor(
 
   public fun getPluginIds(): Set<String> = pluginIds
 
-  @Throws(BuildScriptParseException::class, NonNormalizedScriptException::class)
+  @Throws(KotlinParseException::class, NonNormalizedScriptException::class)
   public fun rewritten(): String {
     errorListener.getErrorMessages().ifNotEmpty {
-      throw BuildScriptParseException.withErrors(it)
+      throw KotlinParseException.withErrors(it)
     }
 
     return rewriter.text.trimGently(terminalNewlines)

recipes/plugins/src/main/kotlin/cash/recipes/plugins/PluginMutator.kt

@@ -2,7 +2,7 @@ package cash.recipes.plugins
 
 import cash.grammar.kotlindsl.model.Plugin
 import cash.grammar.kotlindsl.model.Plugin.Type
-import cash.grammar.kotlindsl.parse.BuildScriptParseException
+import cash.grammar.kotlindsl.parse.KotlinParseException
 import cash.grammar.kotlindsl.parse.Parser
 import cash.grammar.kotlindsl.parse.Rewriter
 import cash.grammar.kotlindsl.utils.Blocks.isBuildscript
@@ -72,10 +72,10 @@ public class PluginNormalizer private constructor(
   // Is this an empty script?
   private var empty = true
 
-  @Throws(BuildScriptParseException::class)
+  @Throws(KotlinParseException::class)
   public fun rewritten(): String {
     errorListener.getErrorMessages().ifNotEmpty {
-      throw BuildScriptParseException.withErrors(it)
+      throw KotlinParseException.withErrors(it)
     }
 
     return rewriter.text.trimGently(terminalNewlines)

recipes/plugins/src/main/kotlin/cash/recipes/plugins/PluginNormalizer.kt

@@ -1,6 +1,6 @@
 package cash.recipes.repos
 
-import cash.grammar.kotlindsl.parse.BuildScriptParseException
+import cash.grammar.kotlindsl.parse.KotlinParseException
 import cash.grammar.kotlindsl.parse.Parser
 import cash.grammar.kotlindsl.utils.Blocks.isDependencyResolutionManagement
 import cash.grammar.kotlindsl.utils.Blocks.isPlugins
@@ -53,10 +53,10 @@ public class DependencyResolutionManagementAdder private constructor(
 
   private val rewriter = TokenStreamRewriter(tokens)
 
-  @Throws(BuildScriptParseException::class, AlreadyHasBlockException::class)
+  @Throws(KotlinParseException::class, AlreadyHasBlockException::class)
   public fun rewritten(): String {
     errorListener.getErrorMessages().ifNotEmpty {
-      throw BuildScriptParseException.withErrors(it)
+      throw KotlinParseException.withErrors(it)
     }
 
     if (alreadyHasBlock) {

recipes/repos/src/main/kotlin/cash/recipes/repos/DependencyResolutionManagementAdder.kt

@@ -1,6 +1,6 @@
 package cash.recipes.repos
 
-import cash.grammar.kotlindsl.parse.BuildScriptParseException
+import cash.grammar.kotlindsl.parse.KotlinParseException
 import cash.grammar.kotlindsl.parse.Parser
 import cash.grammar.kotlindsl.parse.Rewriter
 import cash.grammar.kotlindsl.utils.Blocks
@@ -66,10 +66,10 @@ public class RepositoriesDeleter private constructor(
   private var terminalNewlines = 0
   private val blockStack = ArrayDeque<NamedBlockContext>()
 
-  @Throws(BuildScriptParseException::class)
+  @Throws(KotlinParseException::class)
   public fun rewritten(): String {
     errorListener.getErrorMessages().ifNotEmpty {
-      throw BuildScriptParseException.withErrors(it)
+      throw KotlinParseException.withErrors(it)
     }
 
     return rewriter.text.trimGently(terminalNewlines)