temporary scripts for branch maintenance by borinquenkid · Pull Request #15488 · apache/grails-core

borinquenkid · 2026-03-09T01:43:09Z

Two tasks one to protect the release branches and another to delete stale branches

Copilot

Pull request overview

Adds Gradle-invokable maintenance automation for GitHub branch hygiene in apache/grails-core (protect release branches, delete stale branches) via two Groovy scripts executed from the root build.gradle.

Changes:

Registers two new Gradle maintenance tasks: deleteBranches and protectBranches.
Introduces ProtectBranches.groovy to apply GitHub branch protection to a curated list of release branches.
Introduces DeleteBranches.groovy to delete a curated list of merged/closed branches via the GitHub API.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 8 comments.

File	Description
build.gradle	Adds Gradle tasks that execute the maintenance Groovy scripts via `GroovyShell`.
ProtectBranches.groovy	New script that calls GitHub’s branch protection API for listed branches.
DeleteBranches.groovy	New script that calls GitHub’s refs API to delete listed branches.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

ProtectBranches.groovy

DeleteBranches.groovy

ProtectBranches.groovy

DeleteBranches.groovy

ProtectBranches.groovy

DeleteBranches.groovy

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated 3 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

ProtectBranches.groovy

DeleteBranches.groovy

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated 7 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

DeleteBranches.groovy

ProtectBranches.groovy

DeleteBranches.groovy

ProtectBranches.groovy

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

ProtectBranches.groovy

DeleteBranches.groovy

Copilot

Pull request overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-09T17:57:26Z

ProtectBranches.groovy

+ */
+import groovy.json.JsonBuilder
+
+// --- Configuration ---


Codebase style guidance discourages section-separator/grouping comments like // --- ... ---. Please remove this header comment to align with the repository conventions (see AGENTS.md guidance).

Copilot · 2026-03-09T17:57:27Z

DeleteBranches.groovy

+if (!githubToken) {
+    throw new IllegalStateException(
+        "GitHub token is required. Set the GITHUB_TOKEN environment variable."
+    )


The exception text says only GITHUB_TOKEN is supported, but the script also reads System.getProperty('github.token') above. Update the message to mention -Dgithub.token as an alternative so users aren't misled when running locally/CI.

Copilot

Pull request overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated 5 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-09T18:21:16Z

ProtectBranches.groovy

+        if (conn.responseCode in [200, 201, 204]) {
+            println "SUCCESS: ${conn.responseCode}"
+        } else {
+            def errorText = conn.errorStream?.text ?: "No error stream available"
+            throw new RuntimeException("HTTP ${conn.responseCode} - ${errorText}")
+        }


sendRequest currently treats a 404 as a hard failure. For branch protection maintenance, it’s common for some branches in the list to no longer exist; consider treating 404 as a skip (log and continue) so one missing branch doesn’t make the whole run fail.

Copilot · 2026-03-09T18:21:17Z

DeleteBranches.groovy

+def githubToken = System.getenv('GITHUB_TOKEN') ?: System.getProperty('github.token')
+def repoOwner   = "apache"
+def repoName    = "grails-core"
+def baseApiUrl  = "https://api.github.com/repos/${repoOwner}/${repoName}"


The target repository is hardcoded to apache/grails-core. That makes it easy to accidentally run this from a fork/clone and still delete branches on the upstream repo if the token has access. Make repoOwner/repoName configurable (env/Gradle properties) and/or require an explicit confirmation that prints the resolved target before proceeding.

Copilot · 2026-03-09T18:21:17Z

DeleteBranches.groovy

+        if (conn.responseCode in [200, 201, 204]) {
+            println "SUCCESS: ${conn.responseCode}"
+        } else {
+            def errorText = conn.errorStream?.text ?: "No error stream available"
+            throw new RuntimeException("HTTP ${conn.responseCode} - ${errorText}")
+        }


sendRequest currently treats a 404 as a hard failure. For a branch-cleanup script, DELETE is typically intended to be idempotent (branch may already be gone), so consider treating 404 as a success/skip and continuing; otherwise the script can fail at the end due to already-deleted branches.

Copilot · 2026-03-09T18:21:17Z

ProtectBranches.groovy

+def githubToken = System.getenv('GITHUB_TOKEN') ?: System.getProperty('github.token')
+def repoOwner   = "apache"
+def repoName    = "grails-core"
+def baseApiUrl  = "https://api.github.com/repos/${repoOwner}/${repoName}"


The target repository is hardcoded to apache/grails-core. That makes it easy to accidentally run this from a fork/clone and still change branch protection on the upstream repo if the token has access. Make repoOwner/repoName configurable (env/Gradle properties) and/or require an explicit confirmation that prints the resolved target before proceeding.

Copilot · 2026-03-09T18:21:17Z

ProtectBranches.groovy

+        conn.setRequestProperty("Authorization", "token ${token}")
+        conn.setRequestProperty("Accept", "application/vnd.github.v3+json")
+        if (body) {


GitHub’s REST API requires a User-Agent header on requests; DeleteBranches.groovy sets one but this script doesn’t. Add a User-Agent request header here to avoid 403/400 responses from the API.

temporary scripts for branch maintenance

b56d014

borinquenkid requested review from Copilot and jdaugherty March 9, 2026 01:43

github-project-automation bot added this to Apache Grails Mar 9, 2026

Copilot started reviewing on behalf of borinquenkid March 9, 2026 01:43 View session

Copilot AI reviewed Mar 9, 2026

View reviewed changes

borinquenkid and others added 7 commits March 8, 2026 20:51

ASL

53ebfc2

Update ProtectBranches.groovy

6f429c4

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update DeleteBranches.groovy

090a103

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update ProtectBranches.groovy

b43da02

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update DeleteBranches.groovy

2b807a4

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update ProtectBranches.groovy

96b6f7a

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

fix PR comments

1ffc7bb

borinquenkid requested a review from Copilot March 9, 2026 01:58

Copilot started reviewing on behalf of borinquenkid March 9, 2026 01:59 View session

Copilot AI reviewed Mar 9, 2026

View reviewed changes

ProtectBranches.groovy Outdated Show resolved Hide resolved

DeleteBranches.groovy Outdated Show resolved Hide resolved

DeleteBranches.groovy Show resolved Hide resolved

borinquenkid and others added 3 commits March 8, 2026 21:03

Update ProtectBranches.groovy

c82b35a

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update DeleteBranches.groovy

3f73539

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

more fix PR comments

81d9d19

borinquenkid requested a review from Copilot March 9, 2026 02:07

Copilot started reviewing on behalf of borinquenkid March 9, 2026 02:08 View session

Copilot AI reviewed Mar 9, 2026

View reviewed changes

borinquenkid and others added 7 commits March 9, 2026 07:38

Update DeleteBranches.groovy

1614fd7

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update ProtectBranches.groovy

21ff471

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update ProtectBranches.groovy

9fd7b84

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update ProtectBranches.groovy

dba1910

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update DeleteBranches.groovy

8bad7a6

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update ProtectBranches.groovy

44d8a6e

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

more fix PR comments

a593ce8

borinquenkid requested a review from Copilot March 9, 2026 12:44

Copilot AI reviewed Mar 9, 2026

View reviewed changes

ProtectBranches.groovy Outdated Show resolved Hide resolved

DeleteBranches.groovy Show resolved Hide resolved

Copilot started reviewing on behalf of borinquenkid March 9, 2026 12:56 View session

more PR comments

37535a2

borinquenkid requested a review from Copilot March 9, 2026 17:51

Copilot started reviewing on behalf of borinquenkid March 9, 2026 17:53 View session

Copilot AI reviewed Mar 9, 2026

View reviewed changes

more PR comments

268419b

borinquenkid requested a review from Copilot March 9, 2026 18:16

Copilot started reviewing on behalf of borinquenkid March 9, 2026 18:17 View session

Copilot AI reviewed Mar 9, 2026

View reviewed changes

Uh oh!

Conversation

borinquenkid commented Mar 9, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants