Skip to content

fix(renovate): run go mod tidy on dependency updates - #161

Merged
prskr merged 1 commit into
mainfrom
fix-renovate-gomod-tidy
Sep 25, 2026
Merged

prskr merged 1 commit into
mainfrom
fix-renovate-gomod-tidy

Conversation

@prskr

@prskr prskr commented Sep 25, 2026

Copy link
Copy Markdown
Owner

Summary

Three open Renovate PRs (#136 gofumpt, #138 x/crypto, #139 x/tools) are failing CI with go tool gotestsum: missing go.sum entry errors that are unrelated to their actual dependency bumps.

Root cause: renovate.json doesn't set postUpdateOptions: ["gomodTidy"], so Renovate only patches go.sum entries for the module it bumped rather than running a full go mod tidy. When a bump shifts transitive versions used solely by the go.mod tool(...) block (gotestsum, gofumpt, goimports) — which aren't in the main dependency graph go mod download walks — go.sum ends up missing entries for their own indirect dependencies (golang.org/x/text, x/sync, x/term, etc.), and go tool gotestsum fails at CI time.

Adding gomodTidy makes Renovate run go mod tidy after every update, keeping go.sum fully consistent (matching what a contributor running go mod tidy locally would produce).

Test plan

  • Verified locally: go mod tidy on the affected branches produces a complete, consistent go.sum and go tool gotestsum then succeeds.
  • This is a Renovate bot configuration change; it takes effect the next time Renovate processes the repo (rebase of an existing PR, or a new PR).

🤖 Generated with Claude Code

https://claude.ai/code/session_01KgKYpETCoqe5rCiiGaUrx5


Generated by Claude Code

Without postUpdateOptions.gomodTidy, Renovate only patches the
go.sum entries for the module it bumped. When that shifts transitive
versions used solely by the go.mod `tool(...)` block (gotestsum,
gofumpt, goimports), go.sum ends up missing entries for their own
dependencies, and `go tool gotestsum` fails in CI with "missing
go.sum entry" errors unrelated to the PR's actual change. This has
been silently breaking several open Renovate PRs (#136, #138, #139).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KgKYpETCoqe5rCiiGaUrx5
@coderabbitai

coderabbitai Bot commented Sep 25, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 33 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: fe27dde1-93d9-49a5-b519-3896506f04e1

📥 Commits

Reviewing files that changed from the base of the PR and between 77b446a and 7f9015f.

📒 Files selected for processing (1)
  • renovate.json

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@prskr
prskr merged commit e244720 into main Sep 25, 2026
5 checks passed

This branch was successfully deployed

1 active deployment
github — 7f9015f6 Deployed Sep 25, 2026 by prskr via deploy #397
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.

2 participants