Skip to content

evals: gate paid tiers on PRs behind the paid-evals label - #137

Merged
JRichlen merged 1 commit into
mainfrom
ci/gate-paid-evals-on-label
Sep 23, 2026
Merged

JRichlen merged 1 commit into
mainfrom
ci/gate-paid-evals-on-label

Conversation

@JRichlen

Copy link
Copy Markdown
Owner

Every push to a PR re-ran the full paid suite (12 promptfoo packs x repeat:3 with the Anthropic grader, plus routing and deep tiers). PR #131 was pushed 6 times on 2026-09-22 and bought the whole suite each time.

Paid legs now run on PRs only when the PR has the paid-evals label; the aggregates already report green on a skipped leg. Pushes to main and workflow_dispatch are unchanged.

To get a paid pass on a PR: add the paid-evals label (this triggers a run). Remove it to stop paying for later pushes.

🤖 Generated with Claude Code

https://claude.ai/code/session_0169LhXkWbFbcHjngKSCLJBV

Every push to a PR re-ran the full paid suite (12 promptfoo packs x repeat:3 with the Anthropic grader, plus routing and deep tiers). PR #131 was pushed 6 times on 2026-09-22 and bought the whole suite each time.

Paid legs now run on PRs only when the PR has the paid-evals label; the aggregates already report green on a skipped leg. Pushes to main and workflow_dispatch are unchanged.
Copilot AI lite review requested due to automatic review settings September 22, 2026 23:07
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-22T23:11:07.988638Z 97c9296 PR opened
🔒 Security Review ✅ Completed 2026-09-22T23:27:48.922627Z 97c9296 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

The label trigger can rerun paid evaluations for unrelated labels, and the testing documentation is outdated.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 Medium severity · 1 Low severity

Open (2)
What changed in this PR

Gates paid PR evaluation tiers behind the paid-evals label while preserving main-branch and manual runs.

Changes:

  • Adds label-based PR triggering.
  • Gates paid behavioral, routing, and deep evaluations.
  • Preserves aggregate reporting for skipped jobs.
File Summary Findings
.github/​workflows/​evals.yml Implements label-based paid-tier gating. Restrict label-triggered runs to additions of paid-evals; update docs/testing.md to document the new gate.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines 458 to +460
github.event_name != 'pull_request' ||
github.event.pull_request.head.repo.full_name == github.repository
(github.event.pull_request.head.repo.full_name == github.repository &&
contains(github.event.pull_request.labels.*.name, 'paid-evals'))
push:
branches: [main]
pull_request:
types: [opened, synchronize, reopened, labeled]

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 97c92960c3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +276 to +277
(github.event.pull_request.head.repo.full_name == github.repository &&
contains(github.event.pull_request.labels.*.name, 'paid-evals')))

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Update the testing inventory for the new label gate

This predicate re-scopes when the behavioral tier runs, and equivalent predicates now re-scope the routing and deep tiers, but docs/testing.md remains unchanged: its “Fires” entries at lines 152-154, 205-207, and 358-360 still describe these jobs as path-gated without mentioning that PRs also require the paid-evals label. This leaves the canonical inventory claiming qualifying PRs are evaluated when their required aggregates can now report green without running the paid legs; update those entries in this change.

AGENTS.md reference: AGENTS.md:L129-L139

Useful? React with 👍 / 👎.

@JRichlen
JRichlen merged commit 31df431 into main Sep 23, 2026
43 checks passed
@JRichlen
JRichlen deleted the ci/gate-paid-evals-on-label branch September 23, 2026 02:16
JRichlen pushed a commit that referenced this pull request Sep 23, 2026
Brings in #137 (paid tiers gated behind the paid-evals label).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DghMFX2SuCASz3fjHoYwjb
JRichlen pushed a commit that referenced this pull request Sep 23, 2026
Brings in #137 (paid tiers gated behind the paid-evals label).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DghMFX2SuCASz3fjHoYwjb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants