fix: remove cyclic dependency between tasks

Author: hurali97

gradle-plugins/react/brownfield/src/main/kotlin/com/callstack/react/brownfield/plugin/RNSourceSets.kt

@@ -96,7 +96,6 @@ object RNSourceSets {
     }
 
     private fun configureTasks() {
-        val projectName = project.name
         val appProjectName = appProject.name
 
         project.tasks.register("copyAutolinkingSources", Copy::class.java) {
@@ -108,32 +107,13 @@ object RNSourceSets {
             patchRNEntryPoint(it, path)
         }
 
-        androidExtension.buildTypes.forEach { buildType ->
-            val capitalisedBuildType = buildType.name.replaceFirstChar { it.titlecase() }
-            val codegenTaskName = "generateCodegenSchemaFromJavaScript"
-            val strippedNativeLibsPath = "$appBuildDir/intermediates/stripped_native_libs"
-            val strippedDebugSymbolsPath = "strip${capitalisedBuildType}DebugSymbols/out/lib"
-
-            val copyLibTask =
-                project.tasks.register("copy${capitalisedBuildType}LibSources", Copy::class.java) {
-                    it.dependsOn(":$appProjectName:$codegenTaskName")
-                    it.dependsOn(":$appProjectName:strip${capitalisedBuildType}DebugSymbols")
-                    it.dependsOn(":$projectName:$codegenTaskName")
-
-                    it.from(
-                        "$strippedNativeLibsPath/${buildType.name}/$strippedDebugSymbolsPath",
-                    )
-                    it.into(project.rootProject.file("$projectName/libs$capitalisedBuildType"))
-                    it.include("**/libappmodules.so", "**/libreact_codegen_*.so")
-                }
-
-            project.tasks.named("preBuild").configure {
-                it.dependsOn("copyAutolinkingSources")
-                it.dependsOn(copyLibTask)
-                if (capitalisedBuildType == "Release") {
-                    it.dependsOn(":${appProject.name}:createBundleReleaseJsAndAssets")
-                }
-            }
+        project.tasks.named("preBuild").configure {
+            /**
+             * As part of the build task, we need to make sure the auto-gen files are present.
+             * This makes sure that AutoLinking and ReactNativeEntryPoint are generated.
+             */
+            it.dependsOn("copyAutolinkingSources")
+            it.dependsOn(":${appProject.name}:createBundleReleaseJsAndAssets")
         }
     }
 }

gradle-plugins/react/brownfield/src/main/kotlin/com/callstack/react/brownfield/processors/JNILibsProcessor.kt

@@ -17,6 +17,7 @@ import com.callstack.react.brownfield.shared.BaseProject
 import com.callstack.react.brownfield.shared.Logging
 import com.callstack.react.brownfield.utils.AndroidArchiveLibrary
 import org.gradle.api.Task
+import org.gradle.api.tasks.Copy
 import java.io.File
 
 class JNILibsProcessor : BaseProject() {
@@ -34,8 +35,28 @@ class JNILibsProcessor : BaseProject() {
         }
 
         val androidExtension = project.extensions.getByName("android") as LibraryExtension
+
+        val appBuild = project.project(":app").layout.buildDirectory.get()
+        val strippedNativeLibsPath = "$appBuild/intermediates/stripped_native_libs"
+        val strippedDebugSymbolsPath = "strip${upperCaseVariantName}DebugSymbols/out/lib"
+        val codegenTaskName = "generateCodegenSchemaFromJavaScript"
+
+        val copyTaskName = "copy${upperCaseVariantName}LibSources"
+        project.tasks.register(copyTaskName, Copy::class.java) { copyTask ->
+            copyTask.dependsOn(":app:strip${upperCaseVariantName}DebugSymbols")
+            copyTask.dependsOn(":${project.name}:$codegenTaskName")
+
+            copyTask.from(
+                "$strippedNativeLibsPath/${variant.name}/$strippedDebugSymbolsPath",
+            )
+
+            copyTask.into(project.rootProject.file("${project.name}/libs$upperCaseVariantName"))
+            copyTask.include("**/libappmodules.so", "**/libreact_codegen_*.so")
+        }
+
         mergeJniLibsTask.configure {
             it.dependsOn(explodeTasks)
+            it.dependsOn(copyTaskName)
 
             it.doFirst {
                 for (archiveLibrary in aarLibraries) {