Skip to content

Conversation

@felixwidjaja
Copy link
Contributor

New Workflow: Admin Approve Submission (.github/workflows/admin-approve-submission.yml)

When a submission is rejected or needs review (like axiom.co), admins can:

  1. Comment /approve - Approves with original score
  2. Comment /approve 3 - Approves and overrides score to 3/3

The workflow:

  • Checks if commenter has write/maintain/admin permission
  • Re-evaluates the URL with Claude for metadata
  • Creates the PR
  • Updates labels (needs-review → approved)
  • Posts confirmation comment

Updated Evaluation Workflow

  • Low-scoring submissions now get needs-review label (not auto-closed)
  • Adds a comment explaining how admins can approve: /approve or /approve 3
  • Saves submission data as artifact for reference

Introduce an Admin Approve Submission workflow that lets repo admins/maintainers approve a submission by commenting `/approve` (optionally `/approve <score>`). The workflow checks collaborator permission, re-runs the evaluator with ADMIN_APPROVED and optional ADMIN_SCORE_OVERRIDE, creates a PR (via existing script), updates labels, and posts a confirmation comment.

Update evaluate-submission.yml to: set clearer labels (auto-approved / needs-review), post an admin-review comment for low scores, and upload submission_data.json as an artifact.

Update scripts/evaluate_submission.py to honor ADMIN_APPROVED and ADMIN_SCORE_OVERRIDE environment variables: log admin mode, apply score override, treat admin-approved submissions as passing, include an admin_approved flag in output, avoid setting needs_manual_review when admin-approved, and skip writing evaluation_results.md in admin mode to prevent duplicate comments. These changes enable maintainers to manually approve and override automated scoring while preserving normal evaluation behavior for non-admin reviews.
@felixwidjaja felixwidjaja merged commit a7765ba into main Feb 9, 2026
@felixwidjaja felixwidjaja deleted the fix-for-review-rejected branch February 9, 2026 03:30
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