fix: use last_edited_at instead of updated_at for post-merge edit detection#375
Open
ClawdiaHedgehog wants to merge 23 commits intoentrius:testfrom
Open
fix: use last_edited_at instead of updated_at for post-merge edit detection#375ClawdiaHedgehog wants to merge 23 commits intoentrius:testfrom
ClawdiaHedgehog wants to merge 23 commits intoentrius:testfrom
Conversation
Co-authored-by: Ander <61125407+anderdc@users.noreply.github.com>
Co-authored-by: Ander <61125407+anderdc@users.noreply.github.com>
Co-authored-by: Ander <61125407+anderdc@users.noreply.github.com>
Co-authored-by: Zanie Blue <contact@zanie.dev>
…ate scoring (entrius#314) Co-authored-by: root <root@135-181-76-236.ptr> Co-authored-by: Ander <61125407+anderdc@users.noreply.github.com>
Co-authored-by: Ander <61125407+anderdc@users.noreply.github.com>
… for missing issues (entrius#335)
Co-authored-by: anderdc <me@alexanderdc.com> Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
… 502 errors on large PRs (entrius#331)
Co-authored-by: Ander <61125407+anderdc@users.noreply.github.com>
Co-authored-by: Ander <61125407+anderdc@users.noreply.github.com>
…res (entrius#346) Co-authored-by: mkdev11 <MkDev11@users.noreply.github.com>
…ount (entrius#340) Co-authored-by: Ander <61125407+anderdc@users.noreply.github.com>
…s#351) Co-authored-by: Ander <61125407+anderdc@users.noreply.github.com>
Co-authored-by: anderdc <me@alexanderdc.com>
Document the new Issue Discovery feature: - How it works (find issues, earn when solved) - Credibility system (separate from PR credibility) - Qualification gates (7 valid issues, 80% credibility) - Scoring rules (base score = solving PR score, no double dipping) - Emission split (30% for issue discovery) Link to full specification in issue_discovery/issue-discovery-rewards.md Part of SN74 Gittensor contribution work.
…ection Bug: Issue discovery was penalizing nearly all legitimately solved issues because GitHub's updatedAt field is updated by ANY issue activity (including the closing event itself), not just body/title edits. Fix: - Added last_edited_at field to Issue class (populated from GitHub's lastEditedAt) - Added lastEditedAt to _PR_TIMELINE_QUERY GraphQL query - Updated _search_issue_referencing_prs_graphql to capture and return lastEditedAt - Updated find_solver_from_cross_references to return last_edited_at as 3rd value - Updated repo_scan.py to pass last_edited_at to Issue objects - Updated scoring.py anti-gaming check to use last_edited_at (with updated_at fallback) Now the anti-gaming check correctly identifies actual post-merge body/title edits rather than conflating them with the issue closing event. Fixes entrius#372 Fixes entrius#373
This was referenced Apr 11, 2026
Collaborator
|
please fix conflicts |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
11 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixed a bug where issue discovery was incorrectly penalizing legitimate solved issues.
Problem: The anti-gaming check used
issue.updated_at > pr.merged_atto detect post-merge edits. But GitHub'supdatedAtfield reflects ANY activity on the issue — comments, label changes, and critically the closing event itself. This meant nearly all legitimately solved issues were being penalized.Solution: Use
last_edited_at(GitHub'slastEditedAtfield) which specifically tracks body/title edits by the author, not automatic timestamps like closing.Changes
last_edited_atfield toIssueclasslastEditedAtto GraphQL query, updated functions to return itlast_edited_attoPRInfoTypedDictlast_edited_attoIssueobjectslast_edited_atTesting
All 272 tests pass.
Related Issues
Fixes #372
Fixes #373