From 8f6d47ee86c0ea730bc35212b1a2dd4000ab3fc2 Mon Sep 17 00:00:00 2001 From: Matt Miller Date: Tue, 21 Jul 2026 11:13:42 -0700 Subject: [PATCH] ci(bump-cursor-review): trigger on .github/cursor-review/** asset changes (BE-3916) The bump-cursor-review-callers fleet only triggered on changes to .github/workflows/cursor-review.yml, but that workflow's runtime behavior is not self-contained: consumers check out this repo's .github/cursor-review/* prompts + scripts (e.g. extract-findings.py) at the pinned workflows_ref SHA. A behavior-changing edit under .github/cursor-review/** therefore changes what every pinned caller runs without opening a SHA-bump PR, so consumers silently stayed on the old revision. Add .github/cursor-review/** to the push path filter, mirroring the agents-md (.github/agents-md-integrity/**) and pr-size (scripts/check-pr-size/**) fleets, and update the bump-callers README fleet table row for consistency. --- .github/bump-callers/README.md | 2 +- .github/workflows/bump-cursor-review-callers.yml | 6 ++++++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/.github/bump-callers/README.md b/.github/bump-callers/README.md index b93488d..75e2075 100644 --- a/.github/bump-callers/README.md +++ b/.github/bump-callers/README.md @@ -23,7 +23,7 @@ forward automatically instead of silently drifting commits behind. | Entrypoint | Triggers on a change to | Caller variable | Seeded | |---|---|---|---| -| [`bump-cursor-review-callers.yml`](../workflows/bump-cursor-review-callers.yml) | `cursor-review.yml` | `CURSOR_REVIEW_CALLERS` | non-empty (hard-fails if empty) | +| [`bump-cursor-review-callers.yml`](../workflows/bump-cursor-review-callers.yml) | `cursor-review.yml` or `cursor-review/**` | `CURSOR_REVIEW_CALLERS` | non-empty (hard-fails if empty) | | [`bump-agents-md-callers.yml`](../workflows/bump-agents-md-callers.yml) | `agents-md-integrity.yml` or `agents-md-integrity/**` | `AGENTS_MD_CALLERS` | empty `[]` (grows as callers land) | | [`bump-pr-size-callers.yml`](../workflows/bump-pr-size-callers.yml) | `pr-size.yml` or `scripts/check-pr-size/**` | `PR_SIZE_CALLERS` | empty `[]` (grows as callers land) | diff --git a/.github/workflows/bump-cursor-review-callers.yml b/.github/workflows/bump-cursor-review-callers.yml index 395fb92..9dbc3e5 100644 --- a/.github/workflows/bump-cursor-review-callers.yml +++ b/.github/workflows/bump-cursor-review-callers.yml @@ -45,6 +45,12 @@ on: branches: [main] paths: - .github/workflows/cursor-review.yml + # The prompts + scripts consumers actually run live here, loaded from the + # `workflows_ref` SHA a caller pins (see cursor-review.yml's header) — a + # behavior change here (e.g. extract-findings.py parsing) needs the same + # caller bump as a workflow-file change, or pinned callers silently drift + # onto stale review behavior. + - .github/cursor-review/** permissions: contents: read