Skip to content

test: harmless edit to trigger the external benchmark gate#3

Open
Joshdb-Amazon wants to merge 3 commits into
mainfrom
test-external-gate
Open

test: harmless edit to trigger the external benchmark gate#3
Joshdb-Amazon wants to merge 3 commits into
mainfrom
test-external-gate

Conversation

@Joshdb-Amazon

Copy link
Copy Markdown
Owner

No description provided.

Adds .github/workflows/benchmark-external.yml so a maintainer can run the gate on
an EXTERNAL (fork) PR. The default pull_request gate can't -- fork PRs get no
secrets -- so this uses pull_request_target to access KIRO_API_KEY, guarded so it
only runs when a maintainer opts a PR in.

Control model:
- Triggers ONLY on the "labeled" event; the job runs ONLY if the label is
  "run-benchmark". A fork contributor cannot self-add the label, so nothing runs
  until a maintainer reviews the PR and applies it.
- A later push does NOT re-run it -- the maintainer must re-label (re-review).
- permissions: contents: read (drops the default write GITHUB_TOKEN).

The maintainer who labels a PR is taking responsibility for it: the job runs the
PR's contributed skill while holding KIRO_API_KEY, so only label PRs you've
reviewed. Use a service-account key so it can be rotated independently.

On its own branch (not main) so it can be adopted separately from the internal gate.
Add label-triggered external-PR benchmark gate (security-sensitive; n…
@Joshdb-Amazon Joshdb-Amazon added the run-benchmark Maintainer opt-in: run the NAD benchmark gate on this (external) PR label Jul 13, 2026
@Joshdb-Amazon Joshdb-Amazon reopened this Jul 14, 2026
@Joshdb-Amazon Joshdb-Amazon added run-benchmark Maintainer opt-in: run the NAD benchmark gate on this (external) PR and removed run-benchmark Maintainer opt-in: run the NAD benchmark gate on this (external) PR labels Jul 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-benchmark Maintainer opt-in: run the NAD benchmark gate on this (external) PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant