Skip to content

chore(deps): routine updates — protobuf, cobra, gopher-lua-libs - #127

Open
arun-k-bharathan wants to merge 1 commit into
freshworks-oss:mainfrom
arun-k-bharathan:deps/routine-updates
Open

chore(deps): routine updates — protobuf, cobra, gopher-lua-libs#127
arun-k-bharathan wants to merge 1 commit into
freshworks-oss:mainfrom
arun-k-bharathan:deps/routine-updates

Conversation

@arun-k-bharathan

@arun-k-bharathan arun-k-bharathan commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Consolidated replacement for three stale Dependabot PRs, none of which carries a security fix:

  • google.golang.org/protobuf 1.36.9 → 1.36.11
  • github.com/spf13/cobra 1.10.1 → 1.10.2
  • github.com/vadv/gopher-lua-libs 0.7.0 → 0.8.0

The diff is three requirement lines in go.mod and six in go.sum. No source or test files changed.

govulncheck ./... reports the same 14 reachable vulnerabilities here as on main, which is what you would expect: nothing in this PR is a security bump. #125 and #126 cover those.

gopher-lua-libs is the largest jump of the three, a minor version against two patches. It reaches this codebase through a single call, gopher_lua_libs.Preload(ls) at internal/lua/lua.go:257, which registers the library's modules into each worker's LState. The Docker-backed internal/lua tests exercise that path and pass.

Relationship to the other dependency PRs

#125 bumps grpc to 1.82.1, and grpc@v1.82.1/go.mod requires protobuf v1.36.11, the same version this PR sets directly. If #125 merges first, the protobuf line here becomes a no-op. If this merges first, it stands on its own. Either way MVS lands on 1.36.11, since both are the same module path.

Merge order across the cleanup: #125, then #126, then this one, then #128. Nothing here constrains the order, so it can go whenever convenient.

Supersedes

All three are still open. Please close them after merging this one, since I'm working from a fork and can't close them myself.

Test plan

  • go build ./...
  • make test (go test -count=1 -race ./...), all 13 test packages pass, including the Docker-backed ones for cql, lua, psql, mongo and kafka
  • make lint matches the main baseline at 238 findings, none new
  • CI on this PR

@arun-k-bharathan
arun-k-bharathan marked this pull request as ready for review August 17, 2026 04:53
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.

1 participant