Skip to content

F-028 (P3) β€” Year-prefixed updates across year boundaries are downgraded to NeedsReviewΒ #82

Description

@LarsLaskowski

Severity: 🟑 Β· Criterion: K1 (Correctness) Β· Phase: P3 Β· Status (review): πŸ†•

File(s): src/DockerUpdateGuard/Images/UpdateDetectionService.cs:211-229 (esp. :217)

Finding

GetHigherYearPrefixedCandidates filters tagYear == currentYear β€” a 2025-*
successor to 2024-* is thus never recognized as UpdateAvailable; the case falls into the
digest/NeedsReview path. The method name ("higher year-prefixed candidates") suggests the
opposite.

Impact

Signal downgrade across year boundaries (real updates appear only as "manual
review"). Recovers as a review, no data loss.

Recommendation

Clarify intent; for the desired cross-year update, allow tagYear >= currentYear
(or comparison via CompareYearPrefixedTags) and test it.


Auto-generated from the repository code review. Source of truth: docs/review/findings.md β†’ F-028 (P3), branch review/review-analysis-plan.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions