Skip to content

actions/checkout version annotation # v7.0.1 is factually incorrect for the pinned SHA #131

Closed as not planned

Description

@twistedmelonman

Non-Blocking Review Concern: actions/checkout version annotation # v7.0.1 is factually incorrect for the pinned SHA

Source: pre-push whole-codebase review
Location: .github/workflows/claude-assistant.yml:41
Date: 2026-08-16

What was flagged

The SHA 3d3c42e5aac5ba805825da76410c181273ba90b1 belongs to the actions/checkout v4.x release line, not v7.0.1 (which does not exist for this action). The old annotation # v4 was closer to correct. The pinned SHA is what actually executes, so there is no security or correctness impact — but the misleading comment defeats the purpose of human-readable annotations, which is to let maintainers quickly verify that the SHA is current. The same issue exists at .github/workflows/claude-blocking-review.yml:132. Correct annotation would be something like # v4.2.2 (verify the exact tag against the SHA at https://github.com/actions/checkout/releases).

Context

This issue was automatically created from a non-blocking concern identified
during pre-push whole-codebase review. It was flagged for tracking.


Created by lib-review-issues.sh

Activity

  1. twistedmelonman commented on Aug 18, 2026

    @twistedmelonman
    MemberAuthor

    Closing — the annotation is correct, so there is nothing to fix.

    The claim is that # v7.0.1 on actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 is factually wrong. Resolving the tag directly:

    $ gh api repos/actions/checkout/git/ref/tags/v7.0.1 --jq '{type:.object.type, sha:.object.sha}'
    {"sha":"3d3c42e5aac5ba805825da76410c181273ba90b1","type":"commit"}

    v7.0.1 is a lightweight tag pointing directly at that commit — the annotation matches.

    Worth recording how this looked wrong at first: querying repos/actions/checkout/tags and filtering for the SHA returns nothing unless paginated far enough, which reads as "no tag matches this commit." The git/ref/tags/<tag> lookup is authoritative and cheap; the /tags listing is not, for this purpose.

    Tracked the broader pattern in smartwatermelon/claude-config#328.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    tech-debtTechnical debt to address

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions