Sign Windows exe with golift/codesign@v1 - #656
Conversation
Skip when CODESIGN_URL is unset so unsigned nightlies still build until the signing endpoint and GitHub secrets exist. Co-authored-by: Cursor <cursoragent@cursor.com>
golangci-lint v2.12 cannot typecheck Go 1.27 stdlib (generic methods in math/rand/v2), so lint jobs fail with "method must have no type parameters" and "undefined: rand" when setup-go installs 'stable'. Install the toolchain from go.mod (1.26.4) for lint, matching release.yml. Co-authored-by: Andreas Echavez <oceanplexian@users.noreply.github.com>
There was a problem hiding this comment.
Stale comment
golangci-lint failed on darwin, linux, and freebsd with typecheck errors in Go’s standard library, not in this PR’s codesign changes:
crypto/internal/randutil/randutil.go: could not import math/rand/v2 math/rand/v2/rand.go: method must have no type parameters undefined: randWhy: lint jobs used
go-version: 'stable', which now installs Go 1.27.0. golangci-lint v2.12 is built with Go 1.26 and cannot typecheck generic methods added in 1.27 (math/rand/v2).gotestonstableis unaffected.Fix: point the golangci-lint
setup-gosteps atgo-version-file: 'go.mod'(1.26.4), matchingrelease.yml.Follow-up commit: b10336f
Note:
pull_request_targetstill runs the workflow frommain, so those checks may stay red until this lands. Thepushworkflow on this branch uses the updated file.Sent by Cursor Automation: CI Failure Auto-Fixer
GitHub Actions rejects secrets in steps.if, which made the workflow file invalid. Co-authored-by: Cursor <cursoragent@cursor.com>
There was a problem hiding this comment.
Stale comment
golangci-lint (linux) and golangci-lint (darwin) are still red on this PR, but that is not a new code defect in the codesign work.
What failed
pull_request_targettest-and-linttypecheck in Go’s standard library:crypto/internal/randutil/randutil.go: could not import math/rand/v2 math/rand/v2/rand.go: method must have no type parameters undefined: randWhy
pull_request_targetruns the workflow file frommain, which still hasgo-version: 'stable'and golangci-lint v2.12. After Go 1.27,stableis 1.27.0. v2.12 cannot typecheck generic methods (now used bymath/rand/v2). The job checkouts this branch’s code, butactions/setup-gostill followsmain.What was already applied
The lint jobs on this branch pin Go withgo-version-file: 'go.mod'(1.26.4):
b10336fThat is enough: the push
test-and-lintrun forcdf7e11is green (linux/darwin/freebsd/windows lint all passed).No further commit
Another workflow edit on this branch cannot changepull_request_targetchecks. Those stay red until the pin is onmain. Merging may require bypassing the stale required lint checks frommain’s workflow.Sent by Cursor Automation: CI Failure Auto-Fixer
There was a problem hiding this comment.
CI on c461e6c is green. No failed checks (14 passed; 8 skipped build-and-release jobs, expected on a PR).
The earlier pull_request_target golangci-lint typecheck failures (math/rand/v2 / Go 1.27 vs golangci-lint v2.12) cleared after merging main, which now runs golangci-lint v2.13. No code fix was applied on this run.
Sent by Cursor Automation: CI Failure Auto-Fixer


Summary
unpackerr.amd64.exewithuses: golift/codesign@v1(default Action install is from the action checkout, notgolift.io).make release WINDOWS_ZIP=0leaves the exe for the Action;make windows_zippackages afterward and refresheschecksums.sha256.txtwithout hashing that file into itself.CODESIGN_URLis unset.Operator follow-up: repo/org secrets
CODESIGN_URL,CODESIGN_CLIENT_CERT,CODESIGN_CLIENT_KEY, and allowlistUnpackerr/unpackerron signerd.Test plan
Made with Cursor