From b392310ffde6b21b831da07a3e9041aa2158297b Mon Sep 17 00:00:00 2001 From: Ralph Bean Date: Tue, 30 Jun 2026 12:48:52 -0400 Subject: [PATCH 1/2] fix(ci): trigger e2e tests on .github/scripts/ changes MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The e2e workflow was missing .github/scripts/** from its path filter, so changes like the openshell version bump didn't trigger e2e runs. The functional-tests workflow already had this path — this brings e2e into alignment. Updates both the push.paths filter and the SYNC-WITH grep regex. Signed-off-by: Ralph Bean Assisted-by: Claude claude-opus-4-6 Signed-off-by: Ralph Bean --- .github/workflows/e2e.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/e2e.yml b/.github/workflows/e2e.yml index 7e0d13264..17f57fc39 100644 --- a/.github/workflows/e2e.yml +++ b/.github/workflows/e2e.yml @@ -20,6 +20,7 @@ on: - 'internal/dispatch/gcf/mintsrc/**' - 'internal/sentencetoken/english.json' - 'Makefile' + - '.github/scripts/**' - '.github/workflows/e2e.yml' - '.github/actions/check-e2e-authorization/**' - 'scripts/check-e2e-authorization.sh' @@ -113,7 +114,7 @@ jobs: exit 0 } fi - if echo "$FILES" | grep -qE '\.go$|^go\.(mod|sum)$|^e2e/|^internal/scaffold/fullsend-repo/|^internal/security/hooks/|^internal/dispatch/gcf/mintsrc/|^internal/sentencetoken/english\.json$|^Makefile$|^\.github/workflows/e2e\.yml$|^\.github/actions/check-e2e-authorization/|^scripts/check-e2e-authorization\.sh$'; then + if echo "$FILES" | grep -qE '\.go$|^go\.(mod|sum)$|^e2e/|^internal/scaffold/fullsend-repo/|^internal/security/hooks/|^internal/dispatch/gcf/mintsrc/|^internal/sentencetoken/english\.json$|^Makefile$|^\.github/scripts/|^\.github/workflows/e2e\.yml$|^\.github/actions/check-e2e-authorization/|^scripts/check-e2e-authorization\.sh$'; then echo "relevant=true" >> "$GITHUB_OUTPUT" else echo "::notice::No e2e-relevant files changed — skipping tests" From cbedc19c9fabe919aaf52fb315d5fad0ca3e9bf0 Mon Sep 17 00:00:00 2001 From: Ralph Bean Date: Tue, 30 Jun 2026 15:09:00 -0400 Subject: [PATCH 2/2] fix(ci): trigger e2e tests on action.yml changes The composite action (action.yml) is the entry point for all agent runs in CI. Changes like pinning the supervisor image (#2795) affect sandbox creation but were not triggering e2e tests. Add action.yml to both the paths trigger and the relevance grep. Motivated-by: #2792 Assisted-by: Claude claude-opus-4-6 Signed-off-by: Ralph Bean --- .github/workflows/e2e.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/e2e.yml b/.github/workflows/e2e.yml index 17f57fc39..3c316511f 100644 --- a/.github/workflows/e2e.yml +++ b/.github/workflows/e2e.yml @@ -22,6 +22,7 @@ on: - 'Makefile' - '.github/scripts/**' - '.github/workflows/e2e.yml' + - 'action.yml' - '.github/actions/check-e2e-authorization/**' - 'scripts/check-e2e-authorization.sh' pull_request_target: @@ -114,7 +115,7 @@ jobs: exit 0 } fi - if echo "$FILES" | grep -qE '\.go$|^go\.(mod|sum)$|^e2e/|^internal/scaffold/fullsend-repo/|^internal/security/hooks/|^internal/dispatch/gcf/mintsrc/|^internal/sentencetoken/english\.json$|^Makefile$|^\.github/scripts/|^\.github/workflows/e2e\.yml$|^\.github/actions/check-e2e-authorization/|^scripts/check-e2e-authorization\.sh$'; then + if echo "$FILES" | grep -qE '\.go$|^go\.(mod|sum)$|^e2e/|^internal/scaffold/fullsend-repo/|^internal/security/hooks/|^internal/dispatch/gcf/mintsrc/|^internal/sentencetoken/english\.json$|^Makefile$|^\.github/scripts/|^\.github/workflows/e2e\.yml$|^action\.yml$|^\.github/actions/check-e2e-authorization/|^scripts/check-e2e-authorization\.sh$'; then echo "relevant=true" >> "$GITHUB_OUTPUT" else echo "::notice::No e2e-relevant files changed — skipping tests"