Repository navigation
chore(deps): bump the go_modules group across 1 directory with 3 updates - #87
Merged
Merged
Conversation
Bumps the go_modules group with 3 updates in the / directory: [github.com/go-git/go-git/v5](https://github.com/go-git/go-git), [github.com/cloudflare/circl](https://github.com/cloudflare/circl) and [github.com/go-git/go-billy/v5](https://github.com/go-git/go-billy). Updates `github.com/go-git/go-git/v5` from 5.16.2 to 5.19.2 - [Release notes](https://github.com/go-git/go-git/releases) - [Changelog](https://github.com/go-git/go-git/blob/main/HISTORY.md) - [Commits](go-git/go-git@v5.16.2...v5.19.2) Updates `github.com/cloudflare/circl` from 1.6.1 to 1.6.3 - [Release notes](https://github.com/cloudflare/circl/releases) - [Commits](cloudflare/circl@v1.6.1...v1.6.3) Updates `github.com/go-git/go-billy/v5` from 5.6.2 to 5.9.0 - [Release notes](https://github.com/go-git/go-billy/releases) - [Commits](go-git/go-billy@v5.6.2...v5.9.0) --- updated-dependencies: - dependency-name: github.com/go-git/go-git/v5 dependency-version: 5.19.2 dependency-type: direct:production dependency-group: go_modules - dependency-name: github.com/cloudflare/circl dependency-version: 1.6.3 dependency-type: indirect dependency-group: go_modules - dependency-name: github.com/go-git/go-billy/v5 dependency-version: 5.9.0 dependency-type: indirect dependency-group: go_modules ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
Bot
added
dependencies
tjamet
reviewed
Oct 31, 2026
tjamet
left a comment
tjamet
left a comment
There was a problem hiding this comment.
Verdict: Static review only (author is dependabot[bot], so per policy I did not build or run the PR's code). I found nothing that blocks merging: it is a go.mod/go.sum-only dependency bump from the usual bot, with no code, CI or workflow changes. Merge once the build and build-darwin checks, still in progress when I reviewed, are green. Those checks are the only functional evidence, because I ran none myself.
Findings
Blocking: none.
Should fix: none.
Nit
go.mod:7: go-git jumps from v5.16.2 to v5.19.2 (three minors). The release notes list worktree symlink-safe wrapper changes (#2100, #2277), shell-quoting of the SSH transport path (#2068), stricter reference-name and delta validation, and a newfollowRedirectspolicy (#2004). maiao uses go-git forplumbing/transport/ssh(pkg/credentials/auth_method.go) and for repo, config and worktree operations (pkg/git/repo.go,pkg/cmd/review.go). If the CI tests don't exercise SSH auth and worktree operations, smoke-testmaiao reviewagainst an SSH remote before releasing.- Commit subject
chore(deps): bump the go_modules group across 1 directory with 3 updates: the subject says "3 updates" but 7 modules change. The transitive ones arefilepath-securejoin0.4.1→0.6.1,sha1cd0.3.2→0.6.0, newklauspost/cpuid/v2v2.3.0, andx/exp(go.sum only). The body has no Problem/Goal/Solution sections. This is bot-generated and acceptable. If you squash-merge, consider:chore(deps): bump go-git to v5.19.2, circl to v1.6.3 and billy to v5.9.0.
Dimensions
- Functionality: Only go.mod and go.sum change, and the go.mod diff is consistent with the PR description. Behaviour changes are limited to upstream hardening (see the first nit). Tests were not run (see below).
- Duplication / libraries: no issues, since there is no new code.
- Maintenance: no issues. There is one new indirect dependency,
klauspost/cpuid/v2, pulled in bysha1cdv0.6.0. - Malicious insertion: no issues. All modules are the expected, well-known upstreams (go-git, cloudflare/circl, go-billy, cyphar/filepath-securejoin, pjbgf/sha1cd, klauspost/cpuid). Every changed go.mod requirement has matching
h1:and/go.modlines in go.sum, with no replace directives and no workflow or CI files touched.x/expappears only in go.sum, which is normal for the module graph. I did not independently verify the hashes against sum.golang.org. - Security: this is a net improvement.
cloudflare/circl1.6.3 fixes a P-384 scalar-multiplication bug (point doubling in addition). The go-git releases include several[SECURITY]dependency updates, path-traversal rejection in reference names, and a symlink-safe worktree boundary. No scanners ran:govulncheck/gosecneed a build of untrusted code, which I skipped under policy. Dependabot's own advisory data is the basis for the security claim. - Architecture: no issues. The public APIs maiao imports are unchanged by this bump, as far as I can tell without compiling. CI will confirm.
- Commits/history: a single commit, no fixup/wip/merge commits. See the nit above.
What I ran
git/diff inspection only, plus a grep of go-git usage in the repo.- Not run:
go build,go test,govulncheck,gosec. The author is a bot rather than tjamet or an org member. - CI at review time:
buildandbuild-darwinin progress.
Generated by Claude Code
Give fake commits a tree hash and return a fresh reader per Reader() call. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BwpSVFu7pFXrkciiC19He3
dependabot
Bot
deleted the
dependabot/go_modules/go_modules-533ba9854e
branch
October 31, 2026 19:29
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps the go_modules group with 3 updates in the / directory: github.com/go-git/go-git/v5, github.com/cloudflare/circl and github.com/go-git/go-billy/v5.
Updates
github.com/go-git/go-git/v5from 5.16.2 to 5.19.2Release notes
Sourced from github.com/go-git/go-git/v5's releases.
... (truncated)
Commits
3eeb238Merge pull request #2277 from go-git/checkout-v5008a78fgit: worktree, make the filesystem wrapper a symlink-safe boundary2263fb5Merge pull request #2268 from go-git/renovate/releases/v5.x-go-golang.org-x-t...77b7625build: Update module golang.org/x/text to v0.39.0 [SECURITY]85ea767Merge pull request #2267 from go-git/renovate/releases/v5.x-go-golang.org-x-n...198675abuild: Update module golang.org/x/net to v0.56.0 [SECURITY]4a0e66dMerge pull request #2254 from pjbgf/v5-dotgit-ref-name-containment3b306efstorage: dotgit, align reference-name safety with refname_is_safef3d0cc1storage: dotgit, reject path traversal in reference names979cfe9Merge pull request #2262 from joshblum/joshblum/to-slash-v5Updates
github.com/cloudflare/circlfrom 1.6.1 to 1.6.3Release notes
Sourced from github.com/cloudflare/circl's releases.
Commits
24ae53cRelease CIRCL v1.6.3581020bRename method to oddMultiplesProjective.12209a4Removing unused cmov for jacobian points.fcba359ecc/p384: use of complete projective formulas for scalar multiplication.5e1bae8ecc/p384: handle point doubling in point addition with Jacobian coordinates.3416046Check opts for nil value.a763d47Release CIRCL v1.6.23c70bf9Bump x/crypto x/sys dependencies.3f0f15bRevert to using package-declared HPKE errors for shortkem instead of standard...23491bdAdding generic Power2Round method.Updates
github.com/go-git/go-billy/v5from 5.6.2 to 5.9.0Release notes
Sourced from github.com/go-git/go-billy/v5's releases.
Commits
237e529Merge pull request #206 from pjbgf/v5-improvements04edb39build: Add go-git integration testd8efefdosfs: preserve empty ChrootOS base07f2a0bMerge pull request #205 from pjbgf/v5-improvements25207c8build: Bump Go versions in workflows2fda229osfs: ChrootOS eval baseDir on creation427b27fMerge pull request #203 from pjbgf/v5-improvements7d5a23echroot: Reject symlink loops2c2287autil: avoid following symlinks in RemoveAll fallbackcbd88e9Fix mount path handlingDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditionsYou can disable automated security fix PRs for this repo from the Security Alerts page.