Skip to content

fix: count ALL open issues for spam multiplier instead of lookback-bounded subset (#929)#1161

Closed
alpurkan17 wants to merge 6 commits into
entrius:testfrom
alpurkan17:fix/929-open-issue-spam-count
Closed

fix: count ALL open issues for spam multiplier instead of lookback-bounded subset (#929)#1161
alpurkan17 wants to merge 6 commits into
entrius:testfrom
alpurkan17:fix/929-open-issue-spam-count

Conversation

@alpurkan17
Copy link
Copy Markdown
Contributor

Summary

run_mirror_issue_discovery derives open_issue_count from the same lookback-bounded /issues response used for scoring. Open issues created before PR_LOOKBACK_DAYS (35 days) are invisible, so the calculate_open_issue_spam_multiplier gate never penalizes miners with many old open issues.

This fix makes a second mirror API call without the since filter to count ALL open issues across mirror-enabled repos for the spam multiplier, while keeping the lookback-bounded call for scoring.

Validation

  • ruff check --fix and ruff format pass
  • Miners with open issues only older than the lookback window are now correctly penalized
  • Falls back to lookback-bounded count if the unfiltered call fails
  • Scoring remains lookback-bounded (unchanged)

Closes #929

@xiao-xiao-mao xiao-xiao-mao Bot added the bug Something isn't working label May 11, 2026
@anderdc
Copy link
Copy Markdown
Collaborator

anderdc commented May 11, 2026

Duplicate of #930 and contribution quality is poor

@anderdc anderdc closed this May 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Mirror issue-discovery spam count ignores open issues older than lookback window

2 participants