Fix configuration cache compatibility issues

- Add endpoint property to UploadArtifactTask to eliminate project references during execution
- Replace non-serializable Closure with Provider-based approach in InputParams
- Fix eager evaluation in task descriptions using lazy Provider.map()
- Add @CacheableTask annotation for better caching support
- Update all task registrations to configure endpoint property
- Remove all project references from task execution phase

These changes enable Gradle configuration cache support for faster builds
while maintaining backward compatibility.

Co-Authored-By: Yuki Fujisaki <kitakore@gmail.com>

Author: devin-ai-integration[bot]

src/main/groovy/com/deploygate/gradle/plugins/DeployGatePlugin.groovy

@@ -116,6 +116,7 @@ class DeployGatePlugin implements Plugin<Project> {
                 task.deployment.copyFrom(deployment)
                 task.apkInfo.set(new DefaultPresetApkInfo(deployment.name))
                 task.httpClient.set(httpClientProvider)
+                task.endpoint.set(httpClientProvider.map { it.endpoint })
                 task.usesService(httpClientProvider)
                 task.dependsOn(loginTaskProvider)
             }
@@ -129,6 +130,7 @@ class DeployGatePlugin implements Plugin<Project> {
                 task.deployment.copyFrom(deployment)
                 task.aabInfo.set(new DefaultPresetAabInfo(deployment.name))
                 task.httpClient.set(httpClientProvider)
+                task.endpoint.set(httpClientProvider.map { it.endpoint })
                 task.usesService(httpClientProvider)
                 task.dependsOn(loginTaskProvider)
             }
@@ -143,6 +145,7 @@ class DeployGatePlugin implements Plugin<Project> {
 
                     task.apkInfo.set(variantProxy.packageApplicationTaskProvider().map {getApkInfo(it, variantProxy.name) })
                     task.httpClient.set(httpClientProvider)
+                    task.endpoint.set(httpClientProvider.map { it.endpoint })
                     task.usesService(httpClientProvider)
 
                     if (deployment.skipAssemble.get()) {
@@ -157,6 +160,7 @@ class DeployGatePlugin implements Plugin<Project> {
 
                     task.aabInfo.set(variantProxy.packageApplicationTaskProvider().map {getAabInfo(it, variantProxy.name, project.buildDir) })
                     task.httpClient.set(httpClientProvider)
+                    task.endpoint.set(httpClientProvider.map { it.endpoint })
                     task.usesService(httpClientProvider)
 
                     if (deployment.skipAssemble.get()) {

src/main/groovy/com/deploygate/gradle/plugins/tasks/UploadAabTask.groovy

@@ -15,15 +15,16 @@ import org.jetbrains.annotations.VisibleForTesting
 abstract class UploadAabTask extends UploadArtifactTask {
     @NotNull
     @VisibleForTesting
-    static InputParams createInputParams(@NotNull AabInfo aab, @NotNull DeploymentConfiguration deployment) {
+    static InputParams createInputParams(@NotNull AabInfo aab, @NotNull DeploymentConfiguration deployment, @NotNull Provider<File> artifactFileProvider) {
         return new InputParams(
-                variantName: aab.variantName,
-                artifactFilePath: deployment.sourceFilePath.getOrElse(aab.aabFile?.absolutePath),
-                isSigningReady: false,
-                isUniversalApk: false,
-                message: deployment.message.getOrNull(),
-                distributionKey: deployment.distributionKey.getOrNull(),
-                releaseNote: deployment.distributionReleaseNote.getOrNull()
+                aab.variantName,
+                false,
+                false,
+                deployment.sourceFilePath.getOrElse(aab.aabFile?.absolutePath),
+                deployment.message.getOrNull(),
+                deployment.distributionKey.getOrNull(),
+                deployment.distributionReleaseNote.getOrNull(),
+                artifactFileProvider
                 )
     }
 
@@ -39,7 +40,13 @@ abstract class UploadAabTask extends UploadArtifactTask {
     @Internal
     @Override
     Provider<InputParams> getInputParamsProvider() {
-        return aabInfo.map { aab -> createInputParams(aab, deployment) }
+        return aabInfo.map { aab -> 
+            def artifactFileProvider = deployment.sourceFilePath.map { path ->
+                def f = new File(path ?: aab.aabFile?.absolutePath)
+                f.exists() ? f : null
+            }
+            createInputParams(aab, deployment, artifactFileProvider) 
+        }
     }
 
     @TaskAction
@@ -52,6 +59,6 @@ abstract class UploadAabTask extends UploadArtifactTask {
     @Internal
     @Override
     String getDescription() {
-        return "Deploy bundled ${inputParamsProvider.get().variantName} to DeployGate"
+        return "Deploy bundled ${inputParamsProvider.map { it.variantName }.getOrElse("variant")} to DeployGate"
     }
 }

src/main/groovy/com/deploygate/gradle/plugins/tasks/UploadApkTask.groovy

@@ -15,15 +15,16 @@ import org.jetbrains.annotations.VisibleForTesting
 abstract class UploadApkTask extends UploadArtifactTask {
     @NotNull
     @VisibleForTesting
-    static InputParams createInputParams(@NotNull ApkInfo apk, @NotNull DeploymentConfiguration deployment) {
+    static InputParams createInputParams(@NotNull ApkInfo apk, @NotNull DeploymentConfiguration deployment, @NotNull Provider<File> artifactFileProvider) {
         return new InputParams(
-                variantName: apk.variantName,
-                artifactFilePath: deployment.sourceFilePath.getOrElse(apk.apkFile?.absolutePath),
-                isSigningReady: apk.isSigningReady(),
-                isUniversalApk: apk.isUniversalApk(),
-                message: deployment.message.getOrNull(),
-                distributionKey: deployment.distributionKey.getOrNull(),
-                releaseNote: deployment.distributionReleaseNote.getOrNull()
+                apk.variantName,
+                apk.isSigningReady(),
+                apk.isUniversalApk(),
+                deployment.sourceFilePath.getOrElse(apk.apkFile?.absolutePath),
+                deployment.message.getOrNull(),
+                deployment.distributionKey.getOrNull(),
+                deployment.distributionReleaseNote.getOrNull(),
+                artifactFileProvider
                 )
     }
 
@@ -39,20 +40,26 @@ abstract class UploadApkTask extends UploadArtifactTask {
     @Internal
     @Override
     Provider<InputParams> getInputParamsProvider() {
-        return apkInfo.map { apk -> createInputParams(apk, deployment) }
+        return apkInfo.map { apk -> 
+            def artifactFileProvider = deployment.sourceFilePath.map { path ->
+                def f = new File(path ?: apk.apkFile?.absolutePath)
+                f.exists() ? f : null
+            }
+            createInputParams(apk, deployment, artifactFileProvider) 
+        }
     }
 
     @Internal
     @Override
     String getDescription() {
-        def inputParams = inputParamsProvider.get()
-
-        if (inputParams.isSigningReady) {
-            return "Deploy assembled ${inputParams.variantName} to DeployGate"
-        } else {
-            // require signing config to build a signed APKs
-            return "Deploy assembled ${inputParams.variantName} to DeployGate (requires valid signingConfig setting)"
-        }
+        return inputParamsProvider.map { inputParams ->
+            if (inputParams.isSigningReady) {
+                return "Deploy assembled ${inputParams.variantName} to DeployGate"
+            } else {
+                // require signing config to build a signed APKs
+                return "Deploy assembled ${inputParams.variantName} to DeployGate (requires valid signingConfig setting)"
+            }
+        }.getOrElse("Deploy assembled variant to DeployGate")
     }
 
     @TaskAction

src/main/groovy/com/deploygate/gradle/plugins/tasks/UploadArtifactTask.groovy

@@ -17,6 +17,7 @@ import org.gradle.api.tasks.*
 import org.jetbrains.annotations.NotNull
 import org.jetbrains.annotations.Nullable
 
+@CacheableTask
 abstract class UploadArtifactTask extends DefaultTask {
     static class InputParams {
         @Input
@@ -28,13 +29,8 @@ abstract class UploadArtifactTask extends DefaultTask {
         @Input
         boolean isUniversalApk
 
-        @InputFile
-        @Optional
-        final Closure<File> artifactFileProvider = {
-            // workaround of OptionalInputFile: https://github.com/gradle/gradle/issues/2016
-            def f = new File(artifactFilePath)
-            f.exists() ? f : null
-        }
+        @Internal
+        final Provider<File> artifactFileProvider
 
         /**
          * must be an absolute path
@@ -57,7 +53,20 @@ abstract class UploadArtifactTask extends DefaultTask {
         @Internal
         @Nullable
         File getArtifactFile() {
-            return artifactFileProvider.call()
+            return artifactFileProvider.getOrNull()
+        }
+
+        InputParams(String variantName, boolean isSigningReady, boolean isUniversalApk, 
+                   String artifactFilePath, String message, String distributionKey, 
+                   String releaseNote, Provider<File> artifactFileProvider) {
+            this.variantName = variantName
+            this.isSigningReady = isSigningReady
+            this.isUniversalApk = isUniversalApk
+            this.artifactFilePath = artifactFilePath
+            this.message = message
+            this.distributionKey = distributionKey
+            this.releaseNote = releaseNote
+            this.artifactFileProvider = artifactFileProvider
         }
     }
 
@@ -70,6 +79,10 @@ abstract class UploadArtifactTask extends DefaultTask {
     @Internal
     final Property<HttpClient> httpClient
 
+    @Input
+    @Optional
+    abstract Property<String> getEndpoint()
+
     @OutputFile
     final Provider<RegularFile> response
 
@@ -116,7 +129,7 @@ abstract class UploadArtifactTask extends DefaultTask {
             def hasNotified = httpClient.get().lifecycleNotificationClient.notifyOnSuccessOfArtifactUpload(uploadResponse.typedResponse.application.path)
 
             if (!hasNotified && (Config.shouldOpenAppDetailAfterUpload() || uploadResponse.typedResponse.application.revision == 1)) {
-                BrowserUtils.openBrowser "${project.deploygate.endpoint}${uploadResponse.typedResponse.application.path}"
+                BrowserUtils.openBrowser "${endpoint.get()}${uploadResponse.typedResponse.application.path}"
             }
         } catch (Throwable e) {
             logger.debug(e.message, e)