From 16c4181fa28ee0add57f9767d40b8be31e0325c2 Mon Sep 17 00:00:00 2001 From: notgitika Date: Wed, 8 Jul 2026 10:57:18 -0400 Subject: [PATCH 1/2] ci(e2e): skip E2E tests on docs-only PRs Adds a paths filter to the pull_request_target trigger so E2E tests only run when a change can actually affect agent behavior: CLI source, e2e or browser tests, packaging manifests, the vitest config, or the e2e workflow files. Docs-only PRs (and other non-code changes) no longer consume a shared CI AWS account slot or ~30 min of runtime. Includes package-lock.json / npm-shrinkwrap.json alongside package.json so dependency-only changes still trigger the suite. --- .github/workflows/e2e-tests.yml | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/.github/workflows/e2e-tests.yml b/.github/workflows/e2e-tests.yml index 1cf5c6430..b6677ee13 100644 --- a/.github/workflows/e2e-tests.yml +++ b/.github/workflows/e2e-tests.yml @@ -10,6 +10,18 @@ on: default: 'main' pull_request_target: branches: [main, feat/**] + # Skip E2E on changes that can't affect agent behavior (e.g. docs-only PRs). + # Only run when CLI source, e2e/browser tests, packaging, or the e2e + # workflows themselves change. + paths: + - 'src/**' + - 'e2e-tests/**' + - 'browser-tests/**' + - 'package.json' + - 'package-lock.json' + - 'npm-shrinkwrap.json' + - 'vitest.config.*' + - '.github/workflows/e2e*' concurrency: group: e2e-${{ github.event.pull_request.number || github.ref }} From e894d04e59661ae2e567ada7d141254d691ba8e9 Mon Sep 17 00:00:00 2001 From: notgitika Date: Wed, 8 Jul 2026 14:52:36 -0400 Subject: [PATCH 2/2] ci(e2e): include build-input paths in E2E filter Address review: the paths filter omitted files that produce the agentcore binary E2E installs. Add tsconfig*.json (build:lib), esbuild.config.* (build:cli), and scripts/** (bundle/copy-assets/generate-schema plus the check-old-cli.mjs postinstall hook). A change to any of these in isolation would otherwise silently skip E2E. --- .github/workflows/e2e-tests.yml | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/.github/workflows/e2e-tests.yml b/.github/workflows/e2e-tests.yml index b6677ee13..80770c1fa 100644 --- a/.github/workflows/e2e-tests.yml +++ b/.github/workflows/e2e-tests.yml @@ -21,6 +21,13 @@ on: - 'package-lock.json' - 'npm-shrinkwrap.json' - 'vitest.config.*' + # Build inputs that produce the `agentcore` binary E2E installs globally: + # TS compiler config, the esbuild bundler config, and scripts/** (which + # includes bundle/copy-assets/generate-schema and the check-old-cli.mjs + # postinstall hook — a bug there breaks `npm install -g` in the E2E job). + - 'tsconfig*.json' + - 'esbuild.config.*' + - 'scripts/**' - '.github/workflows/e2e*' concurrency: