Skip to content

chore(deps): bump nixpkgs from d3b13ad to 885ba4d in the nix group across 1 directory - #502

Merged
kixelated merged 2 commits into
mainfrom
dependabot/nix/nix-b58e14d64f
Nov 7, 2026
Merged

kixelated merged 2 commits into
mainfrom
dependabot/nix/nix-b58e14d64f

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

Bumps the nix group with 1 update in the / directory: nixpkgs.

Updates nixpkgs from d3b13ad to 885ba4d

Commits

Changes

  • @opencode/client moves from 2.1.1 to 2.2.1 in package.json and bun.lock. The new nixpkgs revision supplies OpenCode 2.2.1 (the previous one supplied 2.1.1), and flake.nix's opencode-client check requires the two to match exactly.
  • Verified: @opencode/client@2.2.1 is on npm, with @opencode/protocol and @opencode/schema at 2.2.1 and the same peer dependencies as 2.1.1. The bun.lock diff touches only these three entries.

Checks

  • bun run check, bun run test (29 files, 1088 passed, 5 skipped), and nix flake check pass locally on x86_64-linux. The opencode-client check passes.

Review

  • Reviewer: codex, gpt-6.1-sol, low effort; round 1; session 01a1b4d8-bdbf-7a31-9dc1-d8c931a42895
  • Reviewed head: 76d492ce0db8f331bfc0865aac424e2e1b442292 (base origin/main at 4b07dd37)
  • Findings: 0

(Written by Claude Haiku 5.5)

@dependabot dependabot Bot added dependencies Pull requests that update a dependency file nix Pull requests that update nix code labels Nov 4, 2026

@kixelated kixelated left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Comment thread flake.lock Resolved
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>
@dependabot dependabot Bot changed the title chore(deps): bump nixpkgs from d3b13ad to 58deea6 in the nix group chore(deps): bump nixpkgs from d3b13ad to 885ba4d in the nix group across 1 directory Nov 7, 2026
@dependabot
dependabot Bot force-pushed the dependabot/nix/nix-b58e14d64f branch from 6e899b6 to 02c8411 Compare November 7, 2026 05:13

@kixelated kixelated left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 kixelated left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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))

@kixelated
kixelated added this pull request to the merge queue Nov 7, 2026
Merged via the queue into main with commit f7c2ccf Nov 7, 2026
17 checks passed
@kixelated
kixelated deleted the dependabot/nix/nix-b58e14d64f branch November 7, 2026 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file nix Pull requests that update nix code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant