Skip to content

fix: recompute tool checksums on Renovate's pull requests - #94

Merged
Stensel8 merged 1 commit into
mainfrom
claude/hugo-cicd-cleanup-if8vl9
Aug 14, 2026
Merged

fix: recompute tool checksums on Renovate's pull requests#94
Stensel8 merged 1 commit into
mainfrom
claude/hugo-cicd-cleanup-if8vl9

Conversation

@Stensel8

Copy link
Copy Markdown
Collaborator

Summary

Renovate bumpt de gepinde toolversies maar kan geen checksum berekenen. Elke bump landt dus met de hash van de vórige release en de build stopt op computed checksum did NOT match.

In THectic.nl is dat al echt gebeurd op een Hugo-bump. actionlint en lychee zijn hier op dezelfde manier gepind en zouden op dezelfde manier omvallen.

Type of change

  • fix — bug fix (broken link, incorrect command, layout issue)

Waar dit speelt

Bestand Tool
.github/workflows/config-validation.yml actionlint
.github/workflows/pr-checks.yml lychee

De aanpak

Overgenomen uit Stensel8/scripts, waar de nginx-installer dit al zo doet: een script dat de hashes herberekent, plus een workflow die dat op Renovate's eigen branches draait en het resultaat terugcommit.

Eén detail uit dat voorbeeld dat makkelijk te missen is:

"gitIgnoredAuthors": [
  "github-actions[bot]@users.noreply.github.com",
  "41898282+github-actions[bot]@users.noreply.github.com"
]

Zonder die regel ziet Renovate de bot-commit als "de branch is door iemand anders aangeraakt" en stopt met het onderhouden van de PR.

De pin wordt niet verzwakt

Het script schrijft niet klakkeloos op wat de download opleverde. Het haalt eerst het door upstream gepubliceerde checksum-bestand op, verifieert de download daartegen, en weigert te schrijven bij een mismatch. Beide tools publiceren er één (actionlint _checksums.txt, lychee .tar.gz.sha256).

Checklist

  • PR title follows the commit convention
  • Both EN and NL versions updated (if applicable) — n.v.t., geen contentwijziging
  • Media is in AVIF format (not PNG/JPG) — n.v.t.
  • No broken image references — n.v.t.
  • Tested locally with hugo server — n.v.t.; in plaats daarvan is het script hier gedraaid (beide berekende hashes identiek aan wat er gepind staat), en zijn actionlint, renovate-config-validator --strict en shellcheck groen

🤖 Generated with Claude Code

https://claude.ai/code/session_01SwrLVfDhkTVHC945s1kZ2s


Generated by Claude Code

Renovate bumps the pinned tool versions but cannot compute a checksum, so
every bump arrives with the previous release's hash still in place and the
build stops at "computed checksum did NOT match". It has already happened in
THectic.nl on a Hugo bump; actionlint and lychee are pinned the same way here
and would fail the same way.

This follows the pattern already in use in Stensel8/scripts for the nginx
installer: a script that recalculates the hashes, and a workflow that runs it
on Renovate's own branches and commits the result back.

One detail carried over from there that is easy to miss: gitIgnoredAuthors.
Without it Renovate sees the bot's commit as the branch having been modified
by someone else, and stops maintaining the pull request.

The hash is not simply whatever the download returned. Both projects publish
a checksum file next to their release; the script verifies the download
against that first and refuses to write anything on a mismatch, so a hash
only lands here if upstream vouches for it too.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01SwrLVfDhkTVHC945s1kZ2s
Copilot AI lite review requested due to automatic review settings August 14, 2026 16:02

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Stensel8 Stensel8 self-assigned this Aug 14, 2026
@Stensel8
Stensel8 merged commit 58ebaf2 into main Aug 14, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants