build: update dependencies, switch to prek and go-overlay#90
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR prepares the sysdig-mcp-server for release 2.0.2 by updating Go/Nix dependencies and evolving the development toolchain: it switches the Go toolchain sourcing to go-overlay and replaces pre-commit usage with prek while keeping the existing .pre-commit-config.yaml workflow.
Changes:
- Bump package version to
2.0.2and refreshvendorHash. - Update Go module dependencies (
mcp-go,ginkgo/gomega,x/net,x/tools) and refreshgo.sum. - Update Nix flake inputs/lock to use
go-overlay, and replacepre-commitcommands withprekin dev tooling + docs.
Reviewed changes
Copilot reviewed 8 out of 10 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| package.nix | Switch build function to buildGoLatestModule, bump version to 2.0.2, update vendorHash. |
| flake.nix | Add go-overlay, introduce go_latest + override buildGoLatestModule, and switch dev shell hook/tooling to prek. |
| flake.lock | Update lockfile for new inputs (notably go-overlay) and refreshed nixpkgs pins. |
| go.mod | Bump Go dependencies to newer versions. |
| go.sum | Sync checksums with the updated module graph. |
| justfile | Replace pre-commit autoupdate with prek autoupdate. |
| docs/TROUBLESHOOTING.md | Update troubleshooting instructions from pre-commit to prek. |
| AGENTS.md | Update developer handbook section to refer to prek instead of pre-commit. |
| docker-base-amd64.nix | Refresh pinned base image digest/hash for amd64. |
| docker-base-aarch64.nix | Refresh pinned base image digest/hash for aarch64. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
mateobur
previously approved these changes
Jun 19, 2026
f8e8e4a to
43201b3
Compare
alecron
approved these changes
Jun 22, 2026
mateobur
approved these changes
Jun 22, 2026
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.
Updates all dependencies, bumps the version to
2.0.2for release, and migrates the Go toolchain source togo-overlay.Why drop
nixos-25.11for Go?In #84 we pinned
go_1_26tonixos-25.11because there it wasn't the latest stable Go (1.25 was), so it had few reverse dependencies and got updated within hours of an upstream release.That no longer works:
nixos-25.11is now a frozen stable release channel, so its Go toolchain won't move forward anymore — staying on it means getting stuck on an increasingly outdated Go.nixpkgs-unstableisn't a fix either: there the latest-stable Go is a dependency of many packages, so a bump has to clear the whole test/rebuild pipeline before landing — typically ~3 weeks between a Go release and it being usable.go-overlayremoves both problems: it tracks upstream Go releases and updates within hours, independent of the nixpkgs rebuild cycle, so we always build against the current toolchain.Why
prekinstead ofpre-commit?prekis a drop-inpre-commitreimplementation in Rust: same.pre-commit-config.yaml, no Python/virtualenv bootstrap, faster install and parallel hook execution, and a smaller on-disk footprint.