Skip to content

Let getFixVersion count only advisories at least a severity - #123

Merged
irskep merged 1 commit into
spec-due-date-floor-and-sibling-versionsfrom
fix-version-by-severity
Oct 14, 2026
Merged

irskep merged 1 commit into
spec-due-date-floor-and-sibling-versionsfrom
fix-version-by-severity

Conversation

@irskep

@irskep irskep commented Oct 14, 2026 •

Copy link
Copy Markdown
Collaborator

getFixVersion returns the lowest release that fixes every advisory on a version. A spec that only files issues for high and critical advisories wants the release that fixes those. If a medium advisory's fix shipped later, the issue asked for that later release and counted its due date from it, so the high finding came due late.

getFixVersion(store, name, version, { minSeverity: 'high' }) counts only the advisories at least that severe. An advisory with no severity of its own takes the severity of its finding. A finding without per-advisory detail is counted or left out as a whole. Without the option, nothing changes.

🤖 Generated with Claude Code

@irskep
irskep force-pushed the fix-version-by-severity branch from cd740f0 to 4477546 Compare October 14, 2026 23:23
A policy that files tickets for high and critical advisories wants the
release that fixes those, due four weeks after it shipped. The fix
version on a finding is the lowest release that fixes every advisory,
so a medium advisory fixed later moved the ask and the deadline to its
release. getFixVersion(store, name, version, { minSeverity: 'high' })
leaves the medium out.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@irskep
irskep force-pushed the fix-version-by-severity branch from 4477546 to 44d2bab Compare October 14, 2026 23:23
@irskep

irskep commented Oct 14, 2026

Copy link
Copy Markdown
Collaborator Author

Folded into #121. 🤖

@irskep
irskep merged commit 44d2bab into spec-due-date-floor-and-sibling-versions Oct 14, 2026
@irskep
irskep deleted the fix-version-by-severity branch October 14, 2026 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants