Skip to content

Public release: resolve license evidence for three bundled dependencies #11

Closed as not planned

Description

@jongio

Public-release blocker

Obtain authoritative license evidence for three exact bundled dependency versions before public distribution of the Action. This tracks a rights/evidence gap, not a confirmed security vulnerability.

The esbuild input inventory at source commit 0a6cac52468f8a4cf1f65dc6245a7a2280556340 includes:

Package Published metadata Missing evidence
binary@0.3.0 Declares MIT No full license text in its npm package; listed upstream repository returns 404
chainsaw@0.1.0 Declares MIT/X11 No full license text in its npm package; listed upstream repository returns 404
buffers@0.1.1 No license declaration in the published package/registry metadata No license file or full terms; listed upstream repository returns 404

Registry metadata and package contents were inspected on October 23, 2026. A third-party distributor's license classification is not being substituted for authoritative permission evidence, and MIT permission is not being invented for buffers.

Acceptance

  • Obtain authoritative terms and attribution for binary@0.3.0, or approve a separate dependency-remediation change.
  • Obtain authoritative terms and attribution for chainsaw@0.1.0, or approve a separate dependency-remediation change.
  • Establish the applicable license/permission and attribution for buffers@0.1.1, or approve a separate dependency-remediation change.
  • Bind any supplied evidence to the exact shipped package/version and record provenance.
  • Regenerate the bundled dependency inventory and notices, and verify packaged distribution.
  • Record the release owner's disposition before public distribution.

Boundary

The companion packaging work includes all available notices and explicitly marks these entries missing-license-text. Passing build, packaging, CI, or model checks does not resolve this issue or authorize publication.

Do not close this blocker merely because the notice-generation PR merges. This issue does not authorize dependency replacement, releases, tag moves, Marketplace publication, repository visibility changes, or broader model runs.

Activity

  1. jongio commented on Oct 23, 2026

    @jongio
    CollaboratorAuthor

    Implementation and evidence: #12 at c4e672f adds the inventory and available full notices while preserving these three unresolved entries. This issue remains open after merge; CI success is not license clearance.

  2. jongio commented on Oct 23, 2026

    @jongio
    CollaboratorAuthor

    The review correction in 14db1ff limits inventory to emitted code: 168 contributing packages, with the same three unresolved versions still present. Their published JS/README/metadata were also checked for embedded terms; only the previously recorded MIT/MIT-X11 declarations were found. The protobuf runtime's separate BSD terms are available in its source headers and are now included alongside Apache; that does not resolve these three gaps.

  3. jongio commented on Oct 23, 2026

    @jongio
    CollaboratorAuthor

    The notice-generation PR merged as c529047. This blocker remains open: all three unresolved versions still contribute executable code. Current-main notices do not, by themselves, establish clearance for historical bundled distributions.

  4. jongio commented on Oct 23, 2026

    @jongio
    CollaboratorAuthor

    What is needed to close

    • Verified dependency path: @actions/artifact@6.3.1 -> unzip-stream@0.3.4 -> binary@0.3.0 -> buffers@0.1.1 / chainsaw@0.1.0. The currently published latest artifact and unzip-stream versions still contain this path, so an ordinary version bump is not an established fix. Existing upstream tracking: [@actions/artifact] Transitive dependency buffers has no declared license actions/toolkit#2392; no maintainer resolution was recorded there when checked.
    • Next actor: this issue is unassigned. Nominate an Action maintainer and OSS/legal approver, then choose either authoritative version-bound terms/attribution or a separately approved removal/replacement of this dependency chain. A fork's license label or a package alias is not proof of original rights or API compatibility.
    • Closure evidence: approved disposition for current and historical bundled versions; regenerated emitted-code inventory/notices; and qualification of any replacement's artifact upload/download, report identity/digest, and failure behavior. The notice implementation in fix(packaging): ship bundled dependency notices and inventory #12 is complete, but is not this clearance.

    No dependency override, release, tag move, or publication was performed by this assessment.

  5. jongio commented on Oct 30, 2026

    @jongio
    CollaboratorAuthor

    Audit against github/evals-action@14724b731a90910707b40246d0f95b8e9aac30c9 (main):

    • Implemented on main: Emitted-code notice generation (#12, c529047) and contribution terms (#14, 14724b7).
    • What is left: Bundled dist/index.js still includes executable code from binary@0.3.0, buffers@0.1.1, and chainsaw@0.1.0 (via @actions/artifact@6.3.1 -> unzip-stream@0.3.4), which lack full license texts in their published npm packages. Requires dependency replacement or license clearance before public release.
  6. ronniegeraghty commented on Nov 2, 2026

    @ronniegeraghty
    Collaborator

    Closing per release-owner decision: we are not treating this license-evidence concern as a blocker for making github/evals-action public.

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

Metadata

Metadata

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions