Skip to content

fix(nix): refresh the pnpm-deps hash and guard it in CI - #367

Merged
Slush97 merged 3 commits into
Slush97:mainfrom
gegnep:fix/nix-pnpm-deps-hash
Aug 19, 2026
Merged

Slush97 merged 3 commits into
Slush97:mainfrom
gegnep:fix/nix-pnpm-deps-hash

Conversation

@gegnep

@gegnep gegnep commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Refresh nix/grimoire.nix dep hash, add .github/workflows/nix.yml so the next pnpm lockfile change fails in CI with the replacement hash instead of failing downstream.

gegnep added 3 commits August 19, 2026 01:51
nixpkgs 2026-07-30 -> 2026-08-18, grimoire-social 2026-05-20 ->
2026-08-02. ci.yml checks out social main, so the flake input sitting on
a May revision was the outlier.

Assisted-by: Opus 5 (main session), claude-code v2.1.232
980f692 moved pnpm-lock.yaml. The vendored pnpm store hash was last
refreshed on 2026-08-06 in Slush97#345, so every nix build of main since then
fails with a fixed-output hash mismatch. The comment now carries the
refresh command and points at the CI job that catches the next one.

Assisted-by: Opus 5 (main session), claude-code v2.1.232
The from-source package hashes the whole dependency set, so any
pnpm-lock.yaml change invalidates it, and nothing in CI touched the
flake. Runs nix flake check --no-build plus nix build
.#grimoire.pnpmDeps on pull requests that move pnpm-lock.yaml,
package.json, .npmrc, pnpm-workspace.yaml, flake.* or nix/**. The full
Electron build stays out: it costs far more runner time than the drift
it would catch, which is Electron API drift rather than lockfile drift.

Assisted-by: Opus 5 (main session), gpt-5.6-sol (review), claude-code v2.1.232
@gegnep
gegnep marked this pull request as ready for review August 19, 2026 06:22
@Slush97
Slush97 merged commit abf5b60 into Slush97:main Aug 19, 2026
2 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.

2 participants