Skip to content

fix(signals): include queue-growth risk in medium burden tier - #466

Merged
JSONbored merged 2 commits into
JSONbored:mainfrom
philluiz2323:fix/burden-forecast-medium-growth-risk
Jun 8, 2026
Merged

fix(signals): include queue-growth risk in medium burden tier#466
JSONbored merged 2 commits into
JSONbored:mainfrom
philluiz2323:fix/burden-forecast-medium-growth-risk

Conversation

@philluiz2323

Copy link
Copy Markdown
Contributor

Closes #465.

buildBurdenForecast's level ternary OR's queueGrowthRisk into the critical (>=80) and high (>=55) tiers but the medium tier checked only projectedReviewLoad >= 25. So a small unreviewable/collision-heavy queue with moderate growth risk (queueGrowthRisk in [25, 55)) but low review load was classified low instead of medium — the single tier that broke the OR pattern.

Example: 5 open PRs, no linked issues, updated 10 days ago (not recent, not stale) -> projectedReviewLoad = 15, queueGrowthRisk = 25 -> previously low, now medium.

Change

  • Add || queueGrowthRisk >= 25 to the medium tier, matching the critical/high tiers.
  • Add a regression test asserting the QGR-driven medium case (fails on the old code, which returned low).

Verification

  • Full unit suite green (1255 passed). Existing low/critical forecast assertions unchanged.

buildBurdenForecast OR'd queueGrowthRisk into the critical and high level tiers but omitted it from the medium tier, so a small unreviewable/collision-heavy queue with moderate growth risk (queueGrowthRisk 25-54) and low review load was classified low instead of medium. Add queueGrowthRisk >= 25 to the medium tier to match the pattern.
@philluiz2323
philluiz2323 requested a review from JSONbored as a code owner June 7, 2026 12:48
@dosubot dosubot Bot added the size:XS label Jun 7, 2026
@github-actions github-actions Bot added the gittensor:bug Gittensor-scored bug fix — scores a 0.05x multiplier. label Jun 7, 2026
@ghost

ghost commented Jun 7, 2026

Copy link
Copy Markdown

Note

Gittensory Gate skipped

PR closed before full evaluation. No late first comment was created.

Signal Result Evidence Action
Gate result ⚠️ Skipped #466 is no longer open. No action.

Checked by Gittensory, a quiet PR intelligence layer for OSS maintainers.

@superagent-security superagent-security Bot added the pr:verified PR passed security analysis. label Jun 8, 2026

@JSONbored JSONbored left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@philluiz2323 this is ready.

A few notes:

  • The medium tier now treats moderate queue-growth risk consistently with the high and critical tiers.
  • The added test covers the exact under-classification case from #465.
  • No code changes requested.

@dosubot dosubot Bot added the lgtm label Jun 8, 2026
@JSONbored
JSONbored merged commit 7d0acdc into JSONbored:main Jun 8, 2026
6 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in gittensory - v1 roadmap Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gittensor:bug Gittensor-scored bug fix — scores a 0.05x multiplier. pr:verified PR passed security analysis.

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

Burden-forecast medium tier ignores queueGrowthRisk, under-classifying queue-growth pressure

2 participants