From c2f38aba5a44b7b7b532560fdc55f45c5280b1f0 Mon Sep 17 00:00:00 2001 From: Relayflow Lead Date: Sun, 4 Oct 2026 12:49:45 +0200 Subject: [PATCH 1/2] ci: run the CLI packaging gate when its own inputs change #608 added scripts/cli-package-gate.mjs (run by packages/sdk/tests/cli-package-gate.test.ts) to catch a runtime dependency missing from the published CLI. The workflows that run that test only trigger on pull requests touching packages/sdk/** (and their own paths), so a PR that changes only the gate script or packages/relayflows/** (the `relayflows` wrapper package, whose dependency on @relayflows/sdk the gate installs) skipped it. Add both paths to the pull_request filters of cloud-runtime-artifact.yml and to both filters of surface-package.yml. Co-Authored-By: Claude Opus 5.5 (1M context) --- .github/workflows/cloud-runtime-artifact.yml | 2 ++ .github/workflows/surface-package.yml | 4 ++++ 2 files changed, 6 insertions(+) diff --git a/.github/workflows/cloud-runtime-artifact.yml b/.github/workflows/cloud-runtime-artifact.yml index 64c2a1392..75ee5dc0b 100644 --- a/.github/workflows/cloud-runtime-artifact.yml +++ b/.github/workflows/cloud-runtime-artifact.yml @@ -24,6 +24,8 @@ on: - "scripts/cloud-artifact.mjs" - "scripts/build-standalone-cli.mjs" - "scripts/cloud-artifact.test.mjs" + - "scripts/cli-package-gate.mjs" + - "packages/relayflows/**" - "testdata/**" permissions: diff --git a/.github/workflows/surface-package.yml b/.github/workflows/surface-package.yml index 000f3dbbb..8b143bdba 100644 --- a/.github/workflows/surface-package.yml +++ b/.github/workflows/surface-package.yml @@ -9,6 +9,8 @@ on: - "packages/surface/**" - "regressions/**" - "packages/sdk/**" + - "scripts/cli-package-gate.mjs" + - "packages/relayflows/**" push: branches: - main @@ -18,6 +20,8 @@ on: - "packages/surface/**" - "regressions/**" - "packages/sdk/**" + - "scripts/cli-package-gate.mjs" + - "packages/relayflows/**" permissions: contents: read From 5335ab242e263eb3ce60c25872383002bd815b26 Mon Sep 17 00:00:00 2001 From: Relayflow Lead Date: Sun, 4 Oct 2026 12:53:58 +0200 Subject: [PATCH 2/2] ci: list every CLI packaging-gate input in the artifact workflow's filter Review on #610 (Devin, Codex): the gate also runs scripts/pack-release.mjs, packs packages/surface/**, and checks examples/dependency-upgrade-bot/dependency-upgrade-bot.flow.ts, so a PR touching only one of those still skipped it. And the gate only runs in cloud-runtime-artifact.yml's full vitest run; surface-package.yml runs tests/authored-flow.test.ts, so adding the paths there gated nothing. Put all gate inputs in cloud-runtime-artifact.yml's pull_request filter, with a comment saying why, and drop the surface-package.yml change. Co-Authored-By: Claude Opus 5.5 (1M context) --- .github/workflows/cloud-runtime-artifact.yml | 7 +++++++ .github/workflows/surface-package.yml | 4 ---- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/.github/workflows/cloud-runtime-artifact.yml b/.github/workflows/cloud-runtime-artifact.yml index 75ee5dc0b..2235d25db 100644 --- a/.github/workflows/cloud-runtime-artifact.yml +++ b/.github/workflows/cloud-runtime-artifact.yml @@ -24,8 +24,15 @@ on: - "scripts/cloud-artifact.mjs" - "scripts/build-standalone-cli.mjs" - "scripts/cloud-artifact.test.mjs" + # Every input of the CLI packaging gate (scripts/cli-package-gate.mjs, + # run by packages/sdk/tests/cli-package-gate.test.ts in the vitest run + # below): the script, the packer it calls, the three packages it packs + # and installs (packages/sdk/** is above), and the fixture it checks. - "scripts/cli-package-gate.mjs" + - "scripts/pack-release.mjs" - "packages/relayflows/**" + - "packages/surface/**" + - "examples/dependency-upgrade-bot/dependency-upgrade-bot.flow.ts" - "testdata/**" permissions: diff --git a/.github/workflows/surface-package.yml b/.github/workflows/surface-package.yml index 8b143bdba..000f3dbbb 100644 --- a/.github/workflows/surface-package.yml +++ b/.github/workflows/surface-package.yml @@ -9,8 +9,6 @@ on: - "packages/surface/**" - "regressions/**" - "packages/sdk/**" - - "scripts/cli-package-gate.mjs" - - "packages/relayflows/**" push: branches: - main @@ -20,8 +18,6 @@ on: - "packages/surface/**" - "regressions/**" - "packages/sdk/**" - - "scripts/cli-package-gate.mjs" - - "packages/relayflows/**" permissions: contents: read