Skip to content

ci: accept existing org read token for feedback sweeps#60

Merged
haasonsaas merged 1 commit into
mainfrom
codex/feedback-existing-token-20260510
May 10, 2026
Merged

ci: accept existing org read token for feedback sweeps#60
haasonsaas merged 1 commit into
mainfrom
codex/feedback-existing-token-20260510

Conversation

@haasonsaas
Copy link
Copy Markdown
Contributor

Summary

  • let feedback sentinel/backfill use the existing EVALOPS_ORG_READ_TOKEN secret as the org-wide PR read token
  • keep EVALOPS_REVIEW_GUARD_TOKEN as the preferred name when present
  • preserve the no-github.token fallback guard so org-wide sweeps cannot silently publish false-empty reports

Verification

  • actionlint .github/workflows/review-feedback-backfill.yml .github/workflows/review-feedback-sentinel.yml
  • YAML parse check for both workflows
  • confirmed gh secret list --repo evalops/.github currently exposes EVALOPS_ORG_READ_TOKEN

@cursor
Copy link
Copy Markdown

cursor Bot commented May 10, 2026

PR Summary

Low Risk
Low risk: updates GitHub Actions workflow secret selection only, with no application logic changes. Main risk is misconfigured secrets causing the sentinel/backfill jobs to fail earlier rather than run.

Overview
Updates the review feedback sentinel and 30-day backfill workflows to accept secrets.EVALOPS_ORG_READ_TOKEN as a fallback when secrets.EVALOPS_REVIEW_GUARD_TOKEN is not set.

Also adjusts the guard step error messaging to reflect either secret, preserving the explicit failure when no org-wide token is provided (avoiding silent runs with github.token).

Reviewed by Cursor Bugbot for commit 5d09fc4. Bugbot is set up for automated code reviews on this repo. Configure here.

@haasonsaas haasonsaas merged commit 90b8653 into main May 10, 2026
4 checks passed
@haasonsaas haasonsaas deleted the codex/feedback-existing-token-20260510 branch May 10, 2026 06:32
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.

1 participant