Skip to content

Remove 80% merge gate; auto-merge first submission PR regardless of score - #257

Open
tmimmanuel wants to merge 1 commit into
mainfrom
dev-first-submission-auto-merge
Open

Remove 80% merge gate; auto-merge first submission PR regardless of score#257
tmimmanuel wants to merge 1 commit into
mainfrom
dev-first-submission-auto-merge

Conversation

@tmimmanuel

Copy link
Copy Markdown
Collaborator

Changes the merge promotion logic:

  • Removed the hard 80% threshold gate (score >= threshold)
  • First submission: auto-merged regardless of score (detected when king_score still equals the configured default threshold)
  • Subsequent submissions: must beat the current king_score (set to the first PR's score after merge)
  • Updated tests for new function signature and first-submission behavior

This allows the very first miner PR to set the baseline score, which then becomes the bar for all later submissions.

…core

- should_promote_submission now only checks score > king_score (removed
  the score >= threshold requirement)
- If king_score still equals the default threshold, treat as first
  submission (is_first=True) and auto-merge regardless of score
- After merging, king_score is set to that PR's score, so subsequent
  PRs must beat it
@github-actions github-actions Bot added eval Evaluation changes validator Validator backend changes labels Jul 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

eval Evaluation changes validator Validator backend changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant