fix(generator-common): retry broken contributors

Author: zlataovce

generator/common/src/main/kotlin/me/kcra/takenaka/generator/common/provider/impl/ResolvingMappingProvider.kt

@@ -138,7 +138,8 @@ class ResolvingMappingProvider @Deprecated(
                         if (contributor is OutputContainer<*>) {
                             contributor.forEach { output ->
                                 launch(Dispatchers.Default + CoroutineName("resolve-coro")) {
-                                    output.resolve()
+                                    runCatching { output.resolve() }
+                                        .onFailure { e -> logger.error(e) { "Failed to pre-fetch output $output" } }
                                 }
                             }
                         }
@@ -147,14 +148,14 @@ class ResolvingMappingProvider @Deprecated(
 
                 val treeResult = runCatching {
                     buildMappingTree {
-                        contributor(contributors)
+                        contributor(contributors.map { RetryingContributor(it, 5) })
 
                         interceptors += mappingConfig.interceptors
                     }
                 }
 
-                val tree = treeResult.getOrElse { exc ->
-                    throw ResolveException("Failed to create mapping tree for version ${workspace.version}", exc)
+                val tree = treeResult.getOrElse { e ->
+                    throw ResolveException("Failed to create mapping tree for version ${workspace.version.id}", e)
                 }
                 if (analyzer != null) {
                     val time = measureTimeMillis { analyzer.accept(tree) }

generator/common/src/main/kotlin/me/kcra/takenaka/generator/common/provider/impl/RetryingContributor.kt

@@ -0,0 +1,52 @@
+/*
+ * This file is part of takenaka, licensed under the Apache License, Version 2.0 (the "License").
+ *
+ * Copyright (c) 2023-2024 Matous Kucera
+ *
+ * You may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *    http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package me.kcra.takenaka.generator.common.provider.impl
+
+import io.github.oshai.kotlinlogging.KotlinLogging
+import me.kcra.takenaka.core.mapping.MappingContributor
+import net.fabricmc.mappingio.MappingVisitor
+
+private val logger = KotlinLogging.logger {}
+
+/**
+ * A wrapping mapping contributor that retries for [retries] times, if an exception is thrown.
+ *
+ * @property contributor the wrapped contributor
+ * @property retries the number of retries after which we should give up
+ * @property name the contributor name (used for diagnostics)
+ * @author Matouš Kučera
+ */
+internal class RetryingContributor(
+    val contributor: MappingContributor,
+    val retries: Int,
+    val name: String = contributor.javaClass.simpleName
+) : MappingContributor {
+    override val targetNamespace by contributor::targetNamespace
+
+    override fun accept(visitor: MappingVisitor) {
+        repeat(retries - 1) { i ->
+            try {
+                return contributor.accept(visitor)
+            } catch (e: Throwable) {
+                logger.error(e) { "Mapping contributor '$name' threw exception, try ${i + 1}/$retries" }
+            }
+        }
+
+        return contributor.accept(visitor)
+    }
+}