Repository navigation
chore(deps): bump nixpkgs from d3b13ad to 885ba4d in the nix group across 1 directory - #502
Conversation
kixelated
left a comment
There was a problem hiding this comment.
Automated review by review (OpenAI)
Reviewed commit: 6e899b6
Keeping the runner and client versions matched is the right constraint; this pin needs the corresponding client update.
Static GitHub review only; I did not execute Nix. The observed Release workflow passed but does not run this Nix check.
Bumps the nix group with 1 update in the / directory: [nixpkgs](https://github.com/NixOS/nixpkgs). Updates `nixpkgs` from `d3b13ad` to `885ba4d` - [Commits](NixOS/nixpkgs@d3b13ad...885ba4d) --- updated-dependencies: - dependency-name: nixpkgs dependency-version: 58deea654ec5dd0644f31c840a07c29fa0592528 dependency-type: direct:production dependency-group: nix ... Signed-off-by: dependabot[bot] <support@github.com>
d3b13ad to 58deea6 in the nix groupd3b13ad to 885ba4d in the nix group across 1 directory
6e899b6 to
02c8411
Compare
kixelated
left a comment
There was a problem hiding this comment.
Automated review by review (OpenAI)
Reviewed commit: 02c8411
Base: 4b07dd3
The earlier runner/client alignment finding remains unresolved. The new nixpkgs target now supplies OpenCode 2.2.1 (the previously reviewed target supplied 2.1.4), but package.json still pins @opencode/client to 2.1.1. flake.nix continues to select pkgs.opencode for the runner and explicitly checks exact equality with that client version. Bun remains 1.4.2 and matches the manifest.
Direction: update and validate the client dependency alongside the runner, or explicitly retain the matching runner version rather than merely moving the nixpkgs revision. I have not established an additional runtime protocol incompatibility beyond the repository's existing exact-version constraint; no new independent finding is added.
Verification: GitHub-only static inspection of the old/new pins, nixpkgs OpenCode and Bun definitions, the manifest, and flake.nix. No Nix evaluation, builds, tests or runner sessions were executed.
(Written by review (OpenAI))
The nixpkgs bump supplies opencode 2.2.1, and flake.nix's opencode-client check requires the runner and @opencode/client to match exactly. Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
kixelated
left a comment
There was a problem hiding this comment.
Automated review by review (OpenAI)
Reviewed commit 76d492c against base 4b07dd3, following review 5810291834 at 02c8411.
The prior runner/client alignment finding is addressed. package.json and the root lock entry now pin @opencode/client 2.2.1, matching pkgs.opencode.version at the unchanged nixpkgs target 885ba4d038f10c3d17d7a92e8c3cc185b99c9158. The locked client, protocol and schema move together to 2.2.1, with the existing effect and solid-js peer constraints retained. flake.nix still enforces exact equality; the follow-up changes only the manifest and these lock entries.
No new actionable findings in this focused dependency-alignment fix. Direction: retain the exact-version check and validate the updated client with the runner before release.
Verification limits: GitHub-only static comparison and inspection of flake.nix and the pinned nixpkgs OpenCode definition. No dependency installation, Nix evaluation, builds, tests or live runner sessions were performed. The reported local checks were not independently reproduced, and this does not certify runtime protocol compatibility.
(Written by review (OpenAI))
Bumps the nix group with 1 update in the / directory: nixpkgs.
Updates
nixpkgsfromd3b13adto885ba4dCommits
885ba4dpostgresqlPackages.anonymizer: 3.2.3 -> 3.3.2 (#580282)964de16chatgpt: 26.1023.50603 -> 26.1027.30824 (#580702)1c89083kdePackages.kirigami-addons: 1.15.0 -> 1.16.0 (#580790)6b1684aartalk: fix generation of shell completion, enable structuredAttrs (#580732)4db90f6nixos/hydra: Fix invocations of runuser in hydra-init (#578316)15baaf0fyne: 1.7.3 -> 1.7.4 (#580718)fbab1a9thunderbird: 158.0 -> 159.0 (#580578)ff316b9sonarqube-cli: 1.10.0.16040 -> 1.11.0.16877 (#580683)f188e1dincus: 7.5.1 -> 7.6.0 (#580773)0d72aaelibetebase: use libsodium over vendored version (#571118)Changes
@opencode/clientmoves from 2.1.1 to 2.2.1 inpackage.jsonandbun.lock. The new nixpkgs revision supplies OpenCode 2.2.1 (the previous one supplied 2.1.1), andflake.nix'sopencode-clientcheck requires the two to match exactly.@opencode/client@2.2.1is on npm, with@opencode/protocoland@opencode/schemaat 2.2.1 and the same peer dependencies as 2.1.1. Thebun.lockdiff touches only these three entries.Checks
bun run check,bun run test(29 files, 1088 passed, 5 skipped), andnix flake checkpass locally on x86_64-linux. Theopencode-clientcheck passes.Review
01a1b4d8-bdbf-7a31-9dc1-d8c931a4289576d492ce0db8f331bfc0865aac424e2e1b442292(baseorigin/mainat4b07dd37)(Written by Claude Haiku 5.5)