From 49169fbac64b18c2f8b318d0b3d22fa73b2ddb76 Mon Sep 17 00:00:00 2001 From: khaliqgant Date: Fri, 9 Oct 2026 17:34:12 -0700 Subject: [PATCH 1/4] Revert "ci(fleet-e2e): update stale branch note for trunk gating" This reverts commit 6ef6f86c1cb35504db1c353789de4243bc71ed8d. --- .github/workflows/fleet-e2e.yml | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/.github/workflows/fleet-e2e.yml b/.github/workflows/fleet-e2e.yml index f2dfe93c5..54e08bcd6 100644 --- a/.github/workflows/fleet-e2e.yml +++ b/.github/workflows/fleet-e2e.yml @@ -9,9 +9,10 @@ name: Fleet E2E # engine build), which rust-cache amortises on warm runs. The 20-minute job # timeout is a cold-build ceiling, not the matrix runtime. -# NOTE on branches: CI runs only on this repository's `trunk` -> `main` PR and -# on pushes to `main` (see the job `if:`). Feature branches, including -# `feat/fleet-*`, merge into `trunk` without CI. +# NOTE on branches: the fleet stack is developed on `feat/fleet-*` branches that +# merge into each other before `main`, so the trigger must include them — gating +# on `[main]` alone leaves this job inert for the entire feature stack (it would +# only run after something reaches main). Update to `[main]` once the stack lands. on: push: branches: [main] From 0709d984f8b0e18ad67e5c8499b65bdaa8a939d2 Mon Sep 17 00:00:00 2001 From: khaliqgant Date: Fri, 9 Oct 2026 17:34:13 -0700 Subject: [PATCH 2/4] Revert "ci: harden trunk gating" This reverts commit 9024ae34e5f405da530005f8814d041fee7f0077. --- .github/workflows/cancel-on-merge.yml | 4 +++- .github/workflows/codegen-models.yml | 2 +- .github/workflows/e2e-tests.yml | 2 +- .github/workflows/fleet-e2e.yml | 2 +- .github/workflows/large-files.yml | 2 +- .github/workflows/node-compat.yml | 6 +++--- .github/workflows/package-validation.yml | 8 ++++---- .github/workflows/prettier-fmt-fix.yml | 2 +- .github/workflows/relay-evals.yml | 2 +- .github/workflows/relayflow-pr-proof-broker.yml | 2 +- .github/workflows/rust-ci.yml | 14 +++++++------- .github/workflows/rust-fmt-fix.yml | 2 +- .github/workflows/security.yml | 12 ++++++------ .github/workflows/stress-tests.yml | 6 +++--- .../workflows/targeted-feature-verification.yml | 4 ++-- .github/workflows/test-build.yml | 6 +++--- .github/workflows/test-install.yml | 12 ++++++------ .github/workflows/test.yml | 12 ++++++------ .github/workflows/trunk-guard.yml | 15 +++++---------- 19 files changed, 56 insertions(+), 59 deletions(-) diff --git a/.github/workflows/cancel-on-merge.yml b/.github/workflows/cancel-on-merge.yml index a18c65315..592ab6cad 100644 --- a/.github/workflows/cancel-on-merge.yml +++ b/.github/workflows/cancel-on-merge.yml @@ -6,7 +6,9 @@ on: jobs: cancel: - if: github.event.pull_request.merged == true + if: >- + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && + (github.event.pull_request.merged == true) runs-on: ubuntu-latest steps: - name: Cancel running workflows for this PR diff --git a/.github/workflows/codegen-models.yml b/.github/workflows/codegen-models.yml index 72f21434c..4261bb366 100644 --- a/.github/workflows/codegen-models.yml +++ b/.github/workflows/codegen-models.yml @@ -12,7 +12,7 @@ on: jobs: codegen: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' runs-on: ubuntu-latest permissions: contents: write diff --git a/.github/workflows/e2e-tests.yml b/.github/workflows/e2e-tests.yml index 6adfe0825..a6b684258 100644 --- a/.github/workflows/e2e-tests.yml +++ b/.github/workflows/e2e-tests.yml @@ -35,7 +35,7 @@ env: jobs: e2e-test: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: E2E Integration Test runs-on: ${{ matrix.os }} strategy: diff --git a/.github/workflows/fleet-e2e.yml b/.github/workflows/fleet-e2e.yml index 54e08bcd6..df4f591cd 100644 --- a/.github/workflows/fleet-e2e.yml +++ b/.github/workflows/fleet-e2e.yml @@ -54,7 +54,7 @@ env: jobs: fleet-e2e: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Two-node fleet matrix runs-on: ubuntu-latest timeout-minutes: 20 diff --git a/.github/workflows/large-files.yml b/.github/workflows/large-files.yml index 306839141..b4e35abb9 100644 --- a/.github/workflows/large-files.yml +++ b/.github/workflows/large-files.yml @@ -2,7 +2,7 @@ name: Large File Check on: [pull_request] jobs: check: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' runs-on: ubuntu-latest steps: - uses: actions/checkout@v4 diff --git a/.github/workflows/node-compat.yml b/.github/workflows/node-compat.yml index cf365c1f2..b49a24880 100644 --- a/.github/workflows/node-compat.yml +++ b/.github/workflows/node-compat.yml @@ -18,13 +18,13 @@ concurrency: jobs: changes: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' uses: ./.github/workflows/detect-changes.yml install-test: name: Install Test (Node ${{ matrix.node-version }}) needs: changes if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.node_changed == 'true') runs-on: ubuntu-latest strategy: @@ -87,7 +87,7 @@ jobs: name: Fresh Install (Node ${{ matrix.node-version }}) needs: changes if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.node_changed == 'true') runs-on: ubuntu-latest strategy: diff --git a/.github/workflows/package-validation.yml b/.github/workflows/package-validation.yml index 48f241485..1e7a9adf1 100644 --- a/.github/workflows/package-validation.yml +++ b/.github/workflows/package-validation.yml @@ -21,13 +21,13 @@ env: jobs: changes: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' uses: ./.github/workflows/detect-changes.yml validate: name: Build & Validate needs: changes if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.node_changed == 'true') runs-on: ubuntu-latest env: @@ -203,7 +203,7 @@ jobs: name: Publish Fresh Install Build needs: changes if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.node_changed == 'true') runs-on: ubuntu-latest env: @@ -253,7 +253,7 @@ jobs: # provisioning an undeletable workspace; other same-repository PRs and # main pushes must reuse the CI workspace. if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.node_changed == 'true' && github.actor != 'dependabot[bot]' && (github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name == github.repository)) runs-on: macos-latest env: diff --git a/.github/workflows/prettier-fmt-fix.yml b/.github/workflows/prettier-fmt-fix.yml index baa274e8e..763587d18 100644 --- a/.github/workflows/prettier-fmt-fix.yml +++ b/.github/workflows/prettier-fmt-fix.yml @@ -39,7 +39,7 @@ jobs: runs-on: ubuntu-latest # Only run on same-repo PRs where the bot can push back formatting commits. if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository) permissions: contents: write diff --git a/.github/workflows/relay-evals.yml b/.github/workflows/relay-evals.yml index c646a2d13..48066368a 100644 --- a/.github/workflows/relay-evals.yml +++ b/.github/workflows/relay-evals.yml @@ -38,7 +38,7 @@ env: jobs: evals: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Offline evals runs-on: ubuntu-latest diff --git a/.github/workflows/relayflow-pr-proof-broker.yml b/.github/workflows/relayflow-pr-proof-broker.yml index 3b3000b6e..d90146aa9 100644 --- a/.github/workflows/relayflow-pr-proof-broker.yml +++ b/.github/workflows/relayflow-pr-proof-broker.yml @@ -31,7 +31,7 @@ jobs: build: name: ${{ github.event_name == 'pull_request' && 'Validate PR-head Linux broker' || 'Build exact Linux broker' }} if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (github.event_name != 'pull_request_target' || github.event.pull_request.head.repo.full_name == github.repository) runs-on: ubuntu-latest timeout-minutes: 30 diff --git a/.github/workflows/rust-ci.yml b/.github/workflows/rust-ci.yml index 21eb358cf..662b275a7 100644 --- a/.github/workflows/rust-ci.yml +++ b/.github/workflows/rust-ci.yml @@ -17,14 +17,14 @@ env: jobs: changes: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' uses: ./.github/workflows/detect-changes.yml rust-test: name: Rust Tests (${{ matrix.os }}) needs: changes if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.rust_changed == 'true') runs-on: ${{ matrix.os }} strategy: @@ -46,7 +46,7 @@ jobs: name: Relay PTY Synchronization Tests (windows-latest) needs: changes if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.rust_changed == 'true') runs-on: windows-latest timeout-minutes: 10 @@ -68,7 +68,7 @@ jobs: name: Cross-compile check (${{ matrix.target }}) needs: changes if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.rust_changed == 'true') runs-on: ${{ matrix.os }} strategy: @@ -104,7 +104,7 @@ jobs: name: Clippy (${{ matrix.os }}) needs: changes if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.rust_changed == 'true') runs-on: ${{ matrix.os }} strategy: @@ -125,7 +125,7 @@ jobs: name: Format needs: changes if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.rust_changed == 'true') runs-on: ubuntu-latest steps: @@ -139,7 +139,7 @@ jobs: name: SDK TypeScript Check needs: changes if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.node_changed == 'true') runs-on: ubuntu-latest steps: diff --git a/.github/workflows/rust-fmt-fix.yml b/.github/workflows/rust-fmt-fix.yml index 1df1a6ab7..614ecc44a 100644 --- a/.github/workflows/rust-fmt-fix.yml +++ b/.github/workflows/rust-fmt-fix.yml @@ -21,7 +21,7 @@ jobs: runs-on: ubuntu-latest # Only run on same-repo PRs where the bot can push back formatting commits. if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository) permissions: contents: write diff --git a/.github/workflows/security.yml b/.github/workflows/security.yml index 89d55e1c7..083b90b5c 100644 --- a/.github/workflows/security.yml +++ b/.github/workflows/security.yml @@ -21,7 +21,7 @@ concurrency: jobs: changes: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' uses: ./.github/workflows/detect-changes.yml # NPM audit for known vulnerabilities @@ -29,7 +29,7 @@ jobs: name: NPM Audit needs: changes if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.node_changed == 'true') runs-on: ubuntu-latest @@ -61,7 +61,7 @@ jobs: name: Dependency Review runs-on: ubuntu-latest if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (github.event_name == 'pull_request') # This job requires the dependency graph to be enabled in repo settings # Make it non-blocking until that's configured @@ -83,7 +83,7 @@ jobs: name: CodeQL Analysis needs: changes if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.node_changed == 'true') runs-on: ubuntu-latest permissions: @@ -111,7 +111,7 @@ jobs: # Secret scanning (check for accidentally committed secrets) secrets-scan: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Secret Scanning runs-on: ubuntu-latest # Make non-blocking - review findings manually @@ -144,7 +144,7 @@ jobs: name: License Compliance needs: changes if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.node_changed == 'true') runs-on: ubuntu-latest diff --git a/.github/workflows/stress-tests.yml b/.github/workflows/stress-tests.yml index ccaaa351d..035fe1c11 100644 --- a/.github/workflows/stress-tests.yml +++ b/.github/workflows/stress-tests.yml @@ -29,7 +29,7 @@ concurrency: jobs: stress-test-orchestrator: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Orchestrator Stress Tests runs-on: ${{ matrix.os }} strategy: @@ -79,7 +79,7 @@ jobs: fi stress-test-orchestrator-integration: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Orchestrator Integration Stress Tests runs-on: ${{ matrix.os }} strategy: @@ -164,7 +164,7 @@ jobs: needs: [stress-test-orchestrator, stress-test-orchestrator-integration] runs-on: ubuntu-latest if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (always()) steps: diff --git a/.github/workflows/targeted-feature-verification.yml b/.github/workflows/targeted-feature-verification.yml index 429a8d848..52aa4f102 100644 --- a/.github/workflows/targeted-feature-verification.yml +++ b/.github/workflows/targeted-feature-verification.yml @@ -35,7 +35,7 @@ concurrency: jobs: select: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Select Flows v2 shards runs-on: ubuntu-24.04 timeout-minutes: 15 @@ -94,7 +94,7 @@ jobs: retention-days: 14 targeted: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Flows v2 shard ${{ matrix.shard }} needs: select runs-on: ubuntu-24.04 diff --git a/.github/workflows/test-build.yml b/.github/workflows/test-build.yml index 4a15760e0..19ea0e52a 100644 --- a/.github/workflows/test-build.yml +++ b/.github/workflows/test-build.yml @@ -21,7 +21,7 @@ env: jobs: test-build-script: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Test build-bun.sh runs-on: ${{ matrix.os }} strategy: @@ -109,7 +109,7 @@ jobs: done test-build-edge-cases: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Test edge cases runs-on: ubuntu-latest @@ -158,7 +158,7 @@ jobs: needs: [test-build-script, test-build-edge-cases] runs-on: ubuntu-latest if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (always()) steps: diff --git a/.github/workflows/test-install.yml b/.github/workflows/test-install.yml index 47ab6edb4..de387c77d 100644 --- a/.github/workflows/test-install.yml +++ b/.github/workflows/test-install.yml @@ -26,7 +26,7 @@ env: jobs: test-install: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Test on ${{ matrix.os }} (${{ matrix.node }}) runs-on: ${{ matrix.os }} strategy: @@ -183,7 +183,7 @@ jobs: # Test fallback when gunzip is not available test-no-gunzip: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Test without gunzip runs-on: ubuntu-latest @@ -228,7 +228,7 @@ jobs: ' test-install-alpine: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Test on Alpine Linux runs-on: ubuntu-latest @@ -268,7 +268,7 @@ jobs: ' test-install-fedora: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Test on Fedora runs-on: ubuntu-latest @@ -304,7 +304,7 @@ jobs: # Test script syntax and edge cases test-script-quality: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Script Quality Checks runs-on: ubuntu-latest @@ -352,7 +352,7 @@ jobs: needs: [test-install, test-install-alpine, test-install-fedora, test-no-gunzip, test-script-quality] runs-on: ubuntu-latest if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (always()) steps: diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 785c27a46..3ec1dda0c 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -21,13 +21,13 @@ env: jobs: changes: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main') + if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' uses: ./.github/workflows/detect-changes.yml test: needs: changes if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.node_changed == 'true') runs-on: ${{ matrix.os }} strategy: @@ -65,7 +65,7 @@ jobs: name: Windows credential ACL validation needs: changes if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.node_changed == 'true') runs-on: windows-latest steps: @@ -98,7 +98,7 @@ jobs: name: Coverage (upload) needs: changes if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.node_changed == 'true') runs-on: ubuntu-latest steps: @@ -128,7 +128,7 @@ jobs: lint: needs: changes if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.node_changed == 'true') runs-on: ubuntu-latest steps: @@ -157,7 +157,7 @@ jobs: name: Swift SDK Tests needs: changes if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || (github.head_ref == 'trunk' && github.event.pull_request.head.repo.full_name == github.repository && github.base_ref == 'main')) && + ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && (needs.changes.outputs.sdk_swift_changed == 'true') runs-on: macos-latest defaults: diff --git a/.github/workflows/trunk-guard.yml b/.github/workflows/trunk-guard.yml index fbe379e4b..2ddb49758 100644 --- a/.github/workflows/trunk-guard.yml +++ b/.github/workflows/trunk-guard.yml @@ -1,22 +1,17 @@ name: Trunk guard -# Only this repository's `trunk` branch may merge into main. Feature branches -# merge into `trunk`; CI runs only on the trunk -> main PR. -# pull_request_target loads this workflow from the base branch, so a PR cannot -# edit the guard to pass itself. It never checks out PR code. +# Only the `trunk` branch may merge into main. Feature branches merge into +# `trunk` (via the merge train); CI runs only on the trunk -> main PR. on: - pull_request_target: + pull_request: branches: [main] - types: [opened, synchronize, reopened, edited] - -permissions: {} jobs: only-trunk-merges-to-main: - if: github.head_ref != 'trunk' || github.event.pull_request.head.repo.full_name != github.repository + if: github.head_ref != 'trunk' runs-on: ubuntu-latest timeout-minutes: 1 steps: - run: | - echo "::error::Only this repository's 'trunk' branch may merge into main. Retarget this PR to 'trunk'." + echo "::error::Only the 'trunk' branch may merge into main. Retarget this PR to 'trunk'." exit 1 From b2547e25a9db4dbe72574f4f3c8930aaeb62301e Mon Sep 17 00:00:00 2001 From: khaliqgant Date: Fri, 9 Oct 2026 17:34:14 -0700 Subject: [PATCH 3/4] Revert "ci: run CI only for trunk PRs and main pushes" This reverts commit 649a3c91004e4f3056fc10c7ad51d99f5ba5c8fd. --- .github/workflows/cancel-on-merge.yml | 4 +-- .github/workflows/codegen-models.yml | 4 +-- .github/workflows/e2e-tests.yml | 1 - .github/workflows/fleet-e2e.yml | 3 +-- .github/workflows/large-files.yml | 1 - .github/workflows/node-compat.yml | 9 ++----- .github/workflows/package-validation.yml | 14 ++++------- .github/workflows/prettier-fmt-fix.yml | 4 +-- .github/workflows/relay-evals.yml | 1 - .../workflows/relayflow-pr-proof-broker.yml | 4 +-- .github/workflows/rust-ci.yml | 25 +++++-------------- .github/workflows/rust-fmt-fix.yml | 4 +-- .github/workflows/security.yml | 18 +++---------- .github/workflows/stress-tests.yml | 6 +---- .../targeted-feature-verification.yml | 2 -- .github/workflows/test-build.yml | 7 +----- .github/workflows/test-install.yml | 10 +------- .github/workflows/test.yml | 21 ++++------------ .github/workflows/trunk-guard.yml | 17 ------------- 19 files changed, 33 insertions(+), 122 deletions(-) delete mode 100644 .github/workflows/trunk-guard.yml diff --git a/.github/workflows/cancel-on-merge.yml b/.github/workflows/cancel-on-merge.yml index 592ab6cad..a18c65315 100644 --- a/.github/workflows/cancel-on-merge.yml +++ b/.github/workflows/cancel-on-merge.yml @@ -6,9 +6,7 @@ on: jobs: cancel: - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (github.event.pull_request.merged == true) + if: github.event.pull_request.merged == true runs-on: ubuntu-latest steps: - name: Cancel running workflows for this PR diff --git a/.github/workflows/codegen-models.yml b/.github/workflows/codegen-models.yml index 4261bb366..b8c27c467 100644 --- a/.github/workflows/codegen-models.yml +++ b/.github/workflows/codegen-models.yml @@ -5,14 +5,14 @@ on: paths: - 'packages/utils/cli-registry.yaml' branches: - [main] + - main + - 'feature/**' pull_request: paths: - 'packages/utils/cli-registry.yaml' jobs: codegen: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' runs-on: ubuntu-latest permissions: contents: write diff --git a/.github/workflows/e2e-tests.yml b/.github/workflows/e2e-tests.yml index a6b684258..188accab9 100644 --- a/.github/workflows/e2e-tests.yml +++ b/.github/workflows/e2e-tests.yml @@ -35,7 +35,6 @@ env: jobs: e2e-test: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: E2E Integration Test runs-on: ${{ matrix.os }} strategy: diff --git a/.github/workflows/fleet-e2e.yml b/.github/workflows/fleet-e2e.yml index df4f591cd..21e7221a3 100644 --- a/.github/workflows/fleet-e2e.yml +++ b/.github/workflows/fleet-e2e.yml @@ -15,7 +15,7 @@ name: Fleet E2E # only run after something reaches main). Update to `[main]` once the stack lands. on: push: - branches: [main] + branches: [main, 'feat/fleet-**'] paths: - 'packages/fleet/**' - 'packages/cli/**' @@ -54,7 +54,6 @@ env: jobs: fleet-e2e: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Two-node fleet matrix runs-on: ubuntu-latest timeout-minutes: 20 diff --git a/.github/workflows/large-files.yml b/.github/workflows/large-files.yml index b4e35abb9..ee6e2afb0 100644 --- a/.github/workflows/large-files.yml +++ b/.github/workflows/large-files.yml @@ -2,7 +2,6 @@ name: Large File Check on: [pull_request] jobs: check: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' runs-on: ubuntu-latest steps: - uses: actions/checkout@v4 diff --git a/.github/workflows/node-compat.yml b/.github/workflows/node-compat.yml index b49a24880..4bc54e16c 100644 --- a/.github/workflows/node-compat.yml +++ b/.github/workflows/node-compat.yml @@ -18,14 +18,11 @@ concurrency: jobs: changes: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' uses: ./.github/workflows/detect-changes.yml install-test: name: Install Test (Node ${{ matrix.node-version }}) needs: changes - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.node_changed == 'true') + if: needs.changes.outputs.node_changed == 'true' runs-on: ubuntu-latest strategy: matrix: @@ -86,9 +83,7 @@ jobs: fresh-install: name: Fresh Install (Node ${{ matrix.node-version }}) needs: changes - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.node_changed == 'true') + if: needs.changes.outputs.node_changed == 'true' runs-on: ubuntu-latest strategy: matrix: diff --git a/.github/workflows/package-validation.yml b/.github/workflows/package-validation.yml index 1e7a9adf1..71fb73175 100644 --- a/.github/workflows/package-validation.yml +++ b/.github/workflows/package-validation.yml @@ -21,14 +21,11 @@ env: jobs: changes: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' uses: ./.github/workflows/detect-changes.yml validate: name: Build & Validate needs: changes - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.node_changed == 'true') + if: needs.changes.outputs.node_changed == 'true' runs-on: ubuntu-latest env: NPM_CONFIG_FUND: false @@ -202,9 +199,7 @@ jobs: publish-fresh-install-build: name: Publish Fresh Install Build needs: changes - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.node_changed == 'true') + if: needs.changes.outputs.node_changed == 'true' runs-on: ubuntu-latest env: NPM_CONFIG_FUND: false @@ -253,8 +248,9 @@ jobs: # provisioning an undeletable workspace; other same-repository PRs and # main pushes must reuse the CI workspace. if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.node_changed == 'true' && github.actor != 'dependabot[bot]' && (github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name == github.repository)) + needs.changes.outputs.node_changed == 'true' && + github.actor != 'dependabot[bot]' && + (github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name == github.repository) runs-on: macos-latest env: NPM_CONFIG_FUND: false diff --git a/.github/workflows/prettier-fmt-fix.yml b/.github/workflows/prettier-fmt-fix.yml index 763587d18..4f85827b3 100644 --- a/.github/workflows/prettier-fmt-fix.yml +++ b/.github/workflows/prettier-fmt-fix.yml @@ -38,9 +38,7 @@ jobs: name: Auto-format Prettier files runs-on: ubuntu-latest # Only run on same-repo PRs where the bot can push back formatting commits. - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository) + if: github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository permissions: contents: write steps: diff --git a/.github/workflows/relay-evals.yml b/.github/workflows/relay-evals.yml index 48066368a..dc3ea0f59 100644 --- a/.github/workflows/relay-evals.yml +++ b/.github/workflows/relay-evals.yml @@ -38,7 +38,6 @@ env: jobs: evals: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Offline evals runs-on: ubuntu-latest diff --git a/.github/workflows/relayflow-pr-proof-broker.yml b/.github/workflows/relayflow-pr-proof-broker.yml index d90146aa9..6c152d9ba 100644 --- a/.github/workflows/relayflow-pr-proof-broker.yml +++ b/.github/workflows/relayflow-pr-proof-broker.yml @@ -31,8 +31,8 @@ jobs: build: name: ${{ github.event_name == 'pull_request' && 'Validate PR-head Linux broker' || 'Build exact Linux broker' }} if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (github.event_name != 'pull_request_target' || github.event.pull_request.head.repo.full_name == github.repository) + github.event_name != 'pull_request_target' || + github.event.pull_request.head.repo.full_name == github.repository runs-on: ubuntu-latest timeout-minutes: 30 steps: diff --git a/.github/workflows/rust-ci.yml b/.github/workflows/rust-ci.yml index 662b275a7..bf8833d3e 100644 --- a/.github/workflows/rust-ci.yml +++ b/.github/workflows/rust-ci.yml @@ -17,15 +17,12 @@ env: jobs: changes: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' uses: ./.github/workflows/detect-changes.yml rust-test: name: Rust Tests (${{ matrix.os }}) needs: changes - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.rust_changed == 'true') + if: needs.changes.outputs.rust_changed == 'true' runs-on: ${{ matrix.os }} strategy: fail-fast: false @@ -45,9 +42,7 @@ jobs: relay-pty-windows-test: name: Relay PTY Synchronization Tests (windows-latest) needs: changes - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.rust_changed == 'true') + if: needs.changes.outputs.rust_changed == 'true' runs-on: windows-latest timeout-minutes: 10 steps: @@ -67,9 +62,7 @@ jobs: rust-cross-compile: name: Cross-compile check (${{ matrix.target }}) needs: changes - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.rust_changed == 'true') + if: needs.changes.outputs.rust_changed == 'true' runs-on: ${{ matrix.os }} strategy: fail-fast: false @@ -103,9 +96,7 @@ jobs: rust-clippy: name: Clippy (${{ matrix.os }}) needs: changes - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.rust_changed == 'true') + if: needs.changes.outputs.rust_changed == 'true' runs-on: ${{ matrix.os }} strategy: fail-fast: false @@ -124,9 +115,7 @@ jobs: rust-fmt: name: Format needs: changes - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.rust_changed == 'true') + if: needs.changes.outputs.rust_changed == 'true' runs-on: ubuntu-latest steps: - uses: actions/checkout@v4 @@ -138,9 +127,7 @@ jobs: sdk-check: name: SDK TypeScript Check needs: changes - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.node_changed == 'true') + if: needs.changes.outputs.node_changed == 'true' runs-on: ubuntu-latest steps: - uses: actions/checkout@v4 diff --git a/.github/workflows/rust-fmt-fix.yml b/.github/workflows/rust-fmt-fix.yml index 614ecc44a..70472298f 100644 --- a/.github/workflows/rust-fmt-fix.yml +++ b/.github/workflows/rust-fmt-fix.yml @@ -20,9 +20,7 @@ jobs: name: Auto-format Rust code runs-on: ubuntu-latest # Only run on same-repo PRs where the bot can push back formatting commits. - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository) + if: github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository permissions: contents: write steps: diff --git a/.github/workflows/security.yml b/.github/workflows/security.yml index 083b90b5c..13e4df279 100644 --- a/.github/workflows/security.yml +++ b/.github/workflows/security.yml @@ -21,16 +21,13 @@ concurrency: jobs: changes: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' uses: ./.github/workflows/detect-changes.yml # NPM audit for known vulnerabilities npm-audit: name: NPM Audit needs: changes - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.node_changed == 'true') + if: needs.changes.outputs.node_changed == 'true' runs-on: ubuntu-latest steps: @@ -60,9 +57,7 @@ jobs: dependency-review: name: Dependency Review runs-on: ubuntu-latest - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (github.event_name == 'pull_request') + if: github.event_name == 'pull_request' # This job requires the dependency graph to be enabled in repo settings # Make it non-blocking until that's configured continue-on-error: true @@ -82,9 +77,7 @@ jobs: codeql: name: CodeQL Analysis needs: changes - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.node_changed == 'true') + if: needs.changes.outputs.node_changed == 'true' runs-on: ubuntu-latest permissions: security-events: write @@ -111,7 +104,6 @@ jobs: # Secret scanning (check for accidentally committed secrets) secrets-scan: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Secret Scanning runs-on: ubuntu-latest # Make non-blocking - review findings manually @@ -143,9 +135,7 @@ jobs: license-check: name: License Compliance needs: changes - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.node_changed == 'true') + if: needs.changes.outputs.node_changed == 'true' runs-on: ubuntu-latest steps: diff --git a/.github/workflows/stress-tests.yml b/.github/workflows/stress-tests.yml index 035fe1c11..323dbbf0a 100644 --- a/.github/workflows/stress-tests.yml +++ b/.github/workflows/stress-tests.yml @@ -29,7 +29,6 @@ concurrency: jobs: stress-test-orchestrator: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Orchestrator Stress Tests runs-on: ${{ matrix.os }} strategy: @@ -79,7 +78,6 @@ jobs: fi stress-test-orchestrator-integration: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Orchestrator Integration Stress Tests runs-on: ${{ matrix.os }} strategy: @@ -163,9 +161,7 @@ jobs: name: Stress Test Summary needs: [stress-test-orchestrator, stress-test-orchestrator-integration] runs-on: ubuntu-latest - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (always()) + if: always() steps: - name: Download all artifacts diff --git a/.github/workflows/targeted-feature-verification.yml b/.github/workflows/targeted-feature-verification.yml index 52aa4f102..3e9fc7f72 100644 --- a/.github/workflows/targeted-feature-verification.yml +++ b/.github/workflows/targeted-feature-verification.yml @@ -35,7 +35,6 @@ concurrency: jobs: select: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Select Flows v2 shards runs-on: ubuntu-24.04 timeout-minutes: 15 @@ -94,7 +93,6 @@ jobs: retention-days: 14 targeted: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Flows v2 shard ${{ matrix.shard }} needs: select runs-on: ubuntu-24.04 diff --git a/.github/workflows/test-build.yml b/.github/workflows/test-build.yml index 19ea0e52a..abc1a1918 100644 --- a/.github/workflows/test-build.yml +++ b/.github/workflows/test-build.yml @@ -2,7 +2,6 @@ name: Test Build Scripts on: push: - branches: [main] paths: - 'scripts/build-bun.sh' - 'scripts/sign-macos-binary.sh' @@ -21,7 +20,6 @@ env: jobs: test-build-script: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Test build-bun.sh runs-on: ${{ matrix.os }} strategy: @@ -109,7 +107,6 @@ jobs: done test-build-edge-cases: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Test edge cases runs-on: ubuntu-latest @@ -157,9 +154,7 @@ jobs: name: Summary needs: [test-build-script, test-build-edge-cases] runs-on: ubuntu-latest - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (always()) + if: always() steps: - name: Summary diff --git a/.github/workflows/test-install.yml b/.github/workflows/test-install.yml index de387c77d..0e5a2b065 100644 --- a/.github/workflows/test-install.yml +++ b/.github/workflows/test-install.yml @@ -2,7 +2,6 @@ name: Test Install Script on: push: - branches: [main] paths: - 'install.sh' - 'install.test.sh' @@ -26,7 +25,6 @@ env: jobs: test-install: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Test on ${{ matrix.os }} (${{ matrix.node }}) runs-on: ${{ matrix.os }} strategy: @@ -183,7 +181,6 @@ jobs: # Test fallback when gunzip is not available test-no-gunzip: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Test without gunzip runs-on: ubuntu-latest @@ -228,7 +225,6 @@ jobs: ' test-install-alpine: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Test on Alpine Linux runs-on: ubuntu-latest @@ -268,7 +264,6 @@ jobs: ' test-install-fedora: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Test on Fedora runs-on: ubuntu-latest @@ -304,7 +299,6 @@ jobs: # Test script syntax and edge cases test-script-quality: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' name: Script Quality Checks runs-on: ubuntu-latest @@ -351,9 +345,7 @@ jobs: name: Test Summary needs: [test-install, test-install-alpine, test-install-fedora, test-no-gunzip, test-script-quality] runs-on: ubuntu-latest - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (always()) + if: always() steps: - name: Summary diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 3ec1dda0c..88db5acb9 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -21,14 +21,11 @@ env: jobs: changes: - if: (github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk' uses: ./.github/workflows/detect-changes.yml test: needs: changes - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.node_changed == 'true') + if: needs.changes.outputs.node_changed == 'true' runs-on: ${{ matrix.os }} strategy: matrix: @@ -64,9 +61,7 @@ jobs: windows-credential-acl: name: Windows credential ACL validation needs: changes - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.node_changed == 'true') + if: needs.changes.outputs.node_changed == 'true' runs-on: windows-latest steps: - name: Checkout repository @@ -97,9 +92,7 @@ jobs: coverage: name: Coverage (upload) needs: changes - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.node_changed == 'true') + if: needs.changes.outputs.node_changed == 'true' runs-on: ubuntu-latest steps: - name: Checkout repository @@ -127,9 +120,7 @@ jobs: lint: needs: changes - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.node_changed == 'true') + if: needs.changes.outputs.node_changed == 'true' runs-on: ubuntu-latest steps: - name: Checkout repository @@ -156,9 +147,7 @@ jobs: swift-test: name: Swift SDK Tests needs: changes - if: >- - ((github.event_name != 'pull_request' && github.event_name != 'pull_request_target') || github.head_ref == 'trunk') && - (needs.changes.outputs.sdk_swift_changed == 'true') + if: needs.changes.outputs.sdk_swift_changed == 'true' runs-on: macos-latest defaults: run: diff --git a/.github/workflows/trunk-guard.yml b/.github/workflows/trunk-guard.yml deleted file mode 100644 index 2ddb49758..000000000 --- a/.github/workflows/trunk-guard.yml +++ /dev/null @@ -1,17 +0,0 @@ -name: Trunk guard - -# Only the `trunk` branch may merge into main. Feature branches merge into -# `trunk` (via the merge train); CI runs only on the trunk -> main PR. -on: - pull_request: - branches: [main] - -jobs: - only-trunk-merges-to-main: - if: github.head_ref != 'trunk' - runs-on: ubuntu-latest - timeout-minutes: 1 - steps: - - run: | - echo "::error::Only the 'trunk' branch may merge into main. Retarget this PR to 'trunk'." - exit 1 From 6be62f51928a9eff13876e266f8b0c8e4864906e Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" Date: Sat, 10 Oct 2026 00:39:20 +0000 Subject: [PATCH 4/4] style: auto-format with Prettier --- AGENTS.md | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/AGENTS.md b/AGENTS.md index b9bbd8cac..9459ebd04 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -315,6 +315,7 @@ Future agents can query past trajectories to learn from your decisions. + ## Merging: `trunk` + the `mergeable` label CI suites do **not** run automatically on feature branches. They run only on @@ -326,21 +327,24 @@ run on any branch. (Repos whose default branch is not agent batches ready PRs into `trunk`, gets that one PR green, and merges it. **When you open a PR** + 1. Branch from `trunk` and open the PR with **base `trunk`**, not `main`. A PR into `main` from any other branch fails the `Trunk guard` check. 2. No CI runs on your PR, so verify locally before calling it ready: run the typecheck, tests and lint this repo uses, and list the exact commands and results in the PR body. -**When the PR is ready** -3. Add the label **`mergeable`** once all of these are true: - - The change is complete and the local checks above pass. - - Review feedback (human and bot) is addressed or answered. - - It is not a draft and does not depend on an unmerged PR. +**When the PR is ready** 3. Add the label **`mergeable`** once all of these are true: + +- The change is complete and the local checks above pass. +- Review feedback (human and bot) is addressed or answered. +- It is not a draft and does not depend on an unmerged PR. + 4. Remove `mergeable` if the PR stops being ready (new work, a failing check, a blocking question). The label is read live from GitHub on every sweep. **What you must not do** + - Do not merge your own PR, and never merge into or push to `trunk` or `main` directly. - Do not re-enable CI for feature branches or edit the `trunk` gates in @@ -355,4 +359,5 @@ to rebase on `trunk`; do so and keep the label. > Interim: the sweep worker is not deployed yet. Until it is, a human or a > designated agent performs the merge-agent steps manually. Labelling is unchanged. +