Make registry configuration optional with fallback support

- Remove intermediate 'publishing' block, move 'registry' to top level
- Always create release tasks regardless of configuration
- Add fallback support for npr.apiUrl/NPR_API_URL and npr.apiKey/NPR_API_KEY
- Add comprehensive test coverage for all configuration scenarios
- Update README with detailed configuration options and examples
- Delete unused PluginReleaseConfig class

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>

Author: pditommaso

README.md

@@ -36,19 +36,56 @@ nextflowPlugin {
         'com.example.ExampleFunctions'
     ]
 
-    publishing {
-        registry {
-            url = 'https://registry.nextflow.io/api'
-            authToken = project.findProperty('registry_access_token')
-        }
+    registry {
+        url = 'https://registry.nextflow.io/api'
+        authToken = project.findProperty('registry_access_token')
     }
 }
 ```
 
+### Registry Configuration
+
+The `registry` block is optional and supports several configuration methods:
+
+**Option 1: Explicit configuration**
+```gradle
+registry {
+    url = 'https://registry.nextflow.io/api'
+    authToken = 'your-auth-token'
+}
+```
+
+**Option 2: Using project properties**
+```gradle
+registry {
+    // Uses npr.apiUrl and npr.apiKey project properties as fallbacks
+}
+```
+
+**Option 3: Using environment variables**
+```gradle
+registry {
+    // Uses NPR_API_URL and NPR_API_KEY environment variables as fallbacks
+}
+```
+
+**Option 4: No registry block (uses fallbacks)**
+```gradle
+nextflowPlugin {
+    // ... other configuration ...
+    // No registry block - will use npr.apiUrl/NPR_API_URL and npr.apiKey/NPR_API_KEY
+}
+```
+
+The configuration precedence is: explicit values → project properties → environment variables → defaults.
+
+### Available Tasks
+
 This will add some useful tasks to your Gradle build:
 * `assemble` - Compile the Nextflow plugin code and assemble it into a zip file
 * `installPlugin` - Copy the assembled plugin into your local Nextflow plugins dir
-* `releasePlugin` - Release the assembled plugin to the plugin registry
+* `releasePlugin` - Release the assembled plugin to the plugin registry (always available)
+* `releasePluginToRegistry` - Release the plugin to the configured registry (always available)
 
 You should also ensure that your project's `settings.gradle` declares the plugin name, eg:
 ```gradle

src/main/groovy/io/nextflow/gradle/NextflowPlugin.groovy

@@ -123,31 +123,16 @@ class NextflowPlugin implements Plugin<Project> {
         project.tasks.test.dependsOn << project.tasks.assemble
 
         project.afterEvaluate {
-            if (config.publishing) {
-                // track the release tasks
-                def releaseTasks = []
-
-                // add registry release task, if configured
-                if (config.publishing.registry) {
-                    // releasePluginToRegistry - releases plugin to a plugin registry
-                    project.tasks.register('releasePluginToRegistry', RegistryReleaseTask)
-                    project.tasks.releasePluginToRegistry.dependsOn << project.tasks.packagePlugin
-                    releaseTasks << project.tasks.releasePluginToRegistry
-                }
-
-
-                // finally, configure the destination-agnostic 'release' task
-                if (!releaseTasks.isEmpty()) {
-                    // releasePlugin - all the release actions
-                    project.tasks.register('releasePlugin', {
-                        group = 'Nextflow Plugin'
-                        description = 'Release plugin to configured destination'
-                    })
-                    for (task in releaseTasks) {
-                        project.tasks.releasePlugin.dependsOn << task
-                    }
-                }
-            }
+            // Always create registry release task - it will use fallback configuration if needed
+            project.tasks.register('releasePluginToRegistry', RegistryReleaseTask)
+            project.tasks.releasePluginToRegistry.dependsOn << project.tasks.packagePlugin
+
+            // Always create the main release task
+            project.tasks.register('releasePlugin', {
+                group = 'Nextflow Plugin'
+                description = 'Release plugin to configured destination'
+            })
+            project.tasks.releasePlugin.dependsOn << project.tasks.releasePluginToRegistry
         }
     }
 

src/main/groovy/io/nextflow/gradle/NextflowPluginConfig.groovy

@@ -2,6 +2,7 @@ package io.nextflow.gradle
 
 import com.github.zafarkhaja.semver.Version
 import groovy.transform.CompileStatic
+import io.nextflow.gradle.registry.RegistryReleaseConfig
 import org.gradle.api.Project
 
 /**
@@ -17,6 +18,10 @@ import org.gradle.api.Project
  *     extensionPoints = [
  *         'com.example.ExampleFunctions'
  *     ]
+ *     registry {
+ *         url = 'https://registry.nextflow.io/api'
+ *         authToken = 'my-auth-token'
+ *     }
  * }
  * </pre>
  */
@@ -57,9 +62,9 @@ class NextflowPluginConfig {
     List<String> extensionPoints = []
 
     /**
-     * Configure how the plugin will be released (optional)
+     * Configure registry publishing settings (optional)
      */
-    PluginReleaseConfig publishing
+    RegistryReleaseConfig registry
 
     NextflowPluginConfig(Project project) {
         this.project = project
@@ -87,10 +92,9 @@ class NextflowPluginConfig {
         }
     }
 
-    // initialises the 'publishing' sub-config
-    def publishing(Closure config) {
-        publishing = new PluginReleaseConfig(project)
-        project.configure(publishing, config)
-        publishing.validate()
+    // initialises the 'registry' sub-config
+    def registry(Closure config) {
+        registry = new RegistryReleaseConfig(project)
+        project.configure(registry, config)
     }
 }

src/main/groovy/io/nextflow/gradle/PluginReleaseConfig.groovy

@@ -17,7 +17,7 @@ class RegistryReleaseConfig {
      * The registry API base URL.
      * Defaults to the official Nextflow registry.
      */
-    String url = 'https://registry.nextflow.io/api'
+    String url
 
     /**
      * The authentication token (bearer token) for registry access.
@@ -28,4 +28,32 @@ class RegistryReleaseConfig {
     RegistryReleaseConfig(Project project) {
         this.project = project
     }
+
+    /**
+     * Gets the resolved registry URL with fallback to environment variable.
+     * @return the resolved URL or default if not configured
+     */
+    String getResolvedUrl() {
+        return url ?: 
+               project.findProperty('npr.apiUrl') ?: 
+               System.getenv('NPR_API_URL') ?: 
+               'https://registry.nextflow.io/api'
+    }
+
+    /**
+     * Gets the resolved auth token with fallback to environment variable.
+     * @return the resolved auth token
+     * @throws RuntimeException if no auth token is configured
+     */
+    String getResolvedAuthToken() {
+        def token = authToken ?: 
+                    project.findProperty('npr.apiKey') ?: 
+                    System.getenv('NPR_API_KEY')
+        
+        if (!token) {
+            throw new RuntimeException('Registry authentication token must be configured either via authToken property, npr.apiKey project property, or NPR_API_KEY environment variable')
+        }
+        
+        return token
+    }
 }

src/main/groovy/io/nextflow/gradle/registry/RegistryReleaseConfig.groovy

@@ -46,9 +46,17 @@ class RegistryReleaseTask extends DefaultTask {
     def run() {
         final version = project.version.toString()
         final plugin = project.extensions.getByType(NextflowPluginConfig)
-        final config = plugin.publishing.registry
+        
+        // Get or create registry configuration
+        def registryConfig
+        if (plugin.registry) {
+            registryConfig = plugin.registry
+        } else {
+            // Create default registry config that will use fallback values
+            registryConfig = new RegistryReleaseConfig(project)
+        }
 
-        def client = new RegistryClient(new URI(config.url), config.authToken)
+        def client = new RegistryClient(new URI(registryConfig.resolvedUrl), registryConfig.resolvedAuthToken)
         client.release(project.name, version, project.file(zipFile))
     }
 }

src/main/groovy/io/nextflow/gradle/registry/RegistryReleaseTask.groovy

@@ -27,10 +27,8 @@ class NextflowPluginTest extends Specification {
             className = 'com.example.TestPlugin'
             nextflowVersion = '24.04.0'
             extensionPoints = ['com.example.TestExtension']
-            publishing {
-                registry {
-                    url = 'https://example.com/registry'
-                }
+            registry {
+                url = 'https://example.com/registry'
             }
         }
 
@@ -52,10 +50,8 @@ class NextflowPluginTest extends Specification {
             className = 'com.example.TestPlugin'
             nextflowVersion = '24.04.0'
             extensionPoints = ['com.example.TestExtension']
-            publishing {
-                registry {
-                    url = 'https://example.com/registry'
-                }
+            registry {
+                url = 'https://example.com/registry'
             }
         }
 
@@ -69,7 +65,7 @@ class NextflowPluginTest extends Specification {
     }
 
 
-    def "should not register releasePlugin task when no publishing is configured"() {
+    def "should always register releasePlugin tasks even when no registry is configured"() {
         given:
         project.nextflowPlugin {
             description = 'A test plugin'
@@ -83,7 +79,79 @@ class NextflowPluginTest extends Specification {
         project.evaluate()
 
         then:
-        project.tasks.findByName('releasePlugin') == null
+        project.tasks.findByName('releasePlugin') != null
+        project.tasks.findByName('releasePluginToRegistry') != null
+        project.tasks.releasePlugin.group == 'Nextflow Plugin'
+        project.tasks.releasePlugin.description == 'Release plugin to configured destination'
+    }
+
+    def "should register releasePlugin tasks when no explicit registry configuration exists"() {
+        given:
+        project.nextflowPlugin {
+            description = 'A test plugin'
+            provider = 'Test Author'
+            className = 'com.example.TestPlugin'
+            nextflowVersion = '24.04.0'
+            extensionPoints = ['com.example.TestExtension']
+            // No registry block - should use fallback configuration
+        }
+
+        when:
+        project.evaluate()
+
+        then:
+        project.tasks.findByName('releasePlugin') != null
+        project.tasks.findByName('releasePluginToRegistry') != null
+    }
+
+    def "should create releasePlugin task that works with fallback configuration"() {
+        given:
+        project.nextflowPlugin {
+            description = 'A test plugin'
+            provider = 'Test Author'
+            className = 'com.example.TestPlugin'
+            nextflowVersion = '24.04.0'
+            extensionPoints = ['com.example.TestExtension']
+            // No registry block - should use fallback configuration
+        }
+
+        when:
+        project.evaluate()
+        def task = project.tasks.findByName('releasePlugin')
+
+        then:
+        task != null
+        task.group == 'Nextflow Plugin'
+        
+        and: "Other tasks should also exist"
+        project.tasks.findByName('packagePlugin') != null
+        project.tasks.findByName('installPlugin') != null
+        project.tasks.findByName('releasePluginToRegistry') != null
+    }
+
+    def "should handle empty registry block and use fallback configuration"() {
+        given:
+        project.nextflowPlugin {
+            description = 'A test plugin'
+            provider = 'Test Author'
+            className = 'com.example.TestPlugin'
+            nextflowVersion = '24.04.0'
+            extensionPoints = ['com.example.TestExtension']
+            registry {
+                // completely empty registry block
+            }
+        }
+
+        when:
+        project.evaluate()
+
+        then:
+        project.tasks.findByName('releasePlugin') != null
+        project.tasks.findByName('releasePluginToRegistry') != null
+        
+        and: "Other tasks should also exist"
+        project.tasks.findByName('packagePlugin') != null
+        project.tasks.findByName('installPlugin') != null
     }
 
 }