Skip to content

deps: batch update 3 dependencies#237

Merged
wesm merged 5 commits intomainfrom
deps/batch-update
Mar 31, 2026
Merged

deps: batch update 3 dependencies#237
wesm merged 5 commits intomainfrom
deps/batch-update

Conversation

@wesm
Copy link
Copy Markdown
Owner

@wesm wesm commented Mar 31, 2026

Consolidates the following dependabot updates:

Also fixes all 343 golangci-lint v2 errcheck/staticcheck warnings that were blocking the pre-commit hook, and updates the nix vendorHash for the new go.sum.

Closes #233, #234, #235

🤖 Generated with Claude Code

dependabot bot and others added 5 commits March 30, 2026 22:16
Bumps [DeterminateSystems/nix-installer-action](https://github.com/determinatesystems/nix-installer-action) from 21 to 22.
- [Release notes](https://github.com/determinatesystems/nix-installer-action/releases)
- [Commits](DeterminateSystems/nix-installer-action@c5a866b...ef8a148)

---
updated-dependencies:
- dependency-name: DeterminateSystems/nix-installer-action
  dependency-version: '22'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [actions/setup-go](https://github.com/actions/setup-go) from 6.3.0 to 6.4.0.
- [Release notes](https://github.com/actions/setup-go/releases)
- [Commits](actions/setup-go@4b73464...4a36011)

---
updated-dependencies:
- dependency-name: actions/setup-go
  dependency-version: 6.4.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps the minor-and-patch group with 2 updates: [github.com/mark3labs/mcp-go](https://github.com/mark3labs/mcp-go) and [github.com/mattn/go-sqlite3](https://github.com/mattn/go-sqlite3).


Updates `github.com/mark3labs/mcp-go` from 0.45.0 to 0.46.0
- [Release notes](https://github.com/mark3labs/mcp-go/releases)
- [Commits](mark3labs/mcp-go@v0.45.0...v0.46.0)

Updates `github.com/mattn/go-sqlite3` from 1.14.37 to 1.14.38
- [Release notes](https://github.com/mattn/go-sqlite3/releases)
- [Commits](mattn/go-sqlite3@v1.14.37...v1.14.38)

---
updated-dependencies:
- dependency-name: github.com/mark3labs/mcp-go
  dependency-version: 0.46.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: minor-and-patch
- dependency-name: github.com/mattn/go-sqlite3
  dependency-version: 1.14.38
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: minor-and-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
golangci-lint v2 enables errcheck by default, surfacing 343
unchecked error return values and style issues across the codebase.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@roborev-ci
Copy link
Copy Markdown

roborev-ci bot commented Mar 31, 2026

roborev: Combined Review (ef08fe2)

Verdict: Changes are not clean; there is one medium-severity regression that should be fixed before merge.

Medium

  • internal/store/inspect.go:79, internal/store/inspect.go:103
    Replacing defer rows.Close() with defer func() { _ = rows.Close() }() changes closure binding semantics. Because InspectMessage reuses the same rows variable for a later query, both deferred closures can end up closing the second result set, leaking the first. This can leave cursors/statements open on each call.
    Fix: restore defer rows.Close(), or bind the current value explicitly with defer func(r *sql.Rows) { _ = r.Close() }(rows).

Synthesized from 3 reviews (agents: codex, gemini | types: default, security)

@wesm wesm merged commit 3c9ae7c into main Mar 31, 2026
4 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.

1 participant