Skip to content

Commit daebed5

Browse files
committed
fix(issues): skip closed issues already fixed before installed version
Previously, closed issue fetching only set an upper bound (releasedAt + 6mo) with no lower bound, pulling in issues resolved before the user's version. Now adds closed:>=releasedAt as a lower bound so only post-release fixes are included. Keeps the +6mo upper cap for old versions to bound result size.
1 parent 34b9cfc commit daebed5

1 file changed

Lines changed: 8 additions & 4 deletions

File tree

src/sources/issues.ts

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -255,10 +255,14 @@ function fetchIssuesByState(
255255
}
256256
else if (state === 'closed') {
257257
if (releasedAt) {
258-
// For older versions, include issues closed up to 6 months after release
259-
const date = new Date(releasedAt)
260-
date.setMonth(date.getMonth() + 6)
261-
datePart = `+closed:<=${isoDate(date.toISOString())}`
258+
// Lower bound: skip issues already fixed before this release
259+
datePart = `+closed:>=${isoDate(releasedAt)}`
260+
// Upper cap for old versions so we don't pull hundreds of issues
261+
const cap = new Date(releasedAt)
262+
cap.setMonth(cap.getMonth() + 6)
263+
if (cap < new Date()) {
264+
datePart += `+closed:<=${isoDate(cap.toISOString())}`
265+
}
262266
}
263267
else {
264268
datePart = `+closed:>${oneYearAgo()}`

0 commit comments

Comments
 (0)