NEC-108: The pnpm store cache has never restored anything #180

Merged

nectenda-agent merged 2 commits from worktree-nec-108-the-pnpm-store-cache-has-never-restored into main 2026-09-26 07:15:27 +01:00

Conversation 0 Commits 2 Files changed 8 +217 -29

nectenda-agent commented 2026-09-26 06:57:40 +01:00

Collaborator

Copy link

Task: NEC-108 — https://projectron.nerchure.com/tasks/108

CI's pnpm cache has restored a directory pnpm never reads since pnpm 12: on the runner the workspace is its own volume, so pnpm puts its store in node_modules/.pnpm-store (pnpm CHANGELOG, 12.0.0) while the cache restored ~/.local/share/pnpm/store — a reported hit, reused 0, 726 packages downloaded every run. A new composite action, .forgejo/actions/pnpm-install, asks 'pnpm store path' where the store is, caches that under a new key (prefix pnpm-store-, plus the store layout dir), installs, and then fails if node_modules/.modules.yaml records a different store than the one cached (scripts/pnpm-store-check.mjs, unit-tested and mutation-checked). It replaces the three copies in ci.yml, obsidian-e2e and checkpoint-soak (the card said four; there were three). Proof that the cache now restores needs a second run: build and e2e run in parallel, so the first run can only save.

Spec: docs/changes/NEC-108-pnpm-store-cache/spec.md

Changelog

NONE

Task: NEC-108 — https://projectron.nerchure.com/tasks/108 CI's pnpm cache has restored a directory pnpm never reads since pnpm 12: on the runner the workspace is its own volume, so pnpm puts its store in node_modules/.pnpm-store (pnpm CHANGELOG, 12.0.0) while the cache restored ~/.local/share/pnpm/store — a reported hit, reused 0, 726 packages downloaded every run. A new composite action, .forgejo/actions/pnpm-install, asks 'pnpm store path' where the store is, caches that under a new key (prefix pnpm-store-, plus the store layout dir), installs, and then fails if node_modules/.modules.yaml records a different store than the one cached (scripts/pnpm-store-check.mjs, unit-tested and mutation-checked). It replaces the three copies in ci.yml, obsidian-e2e and checkpoint-soak (the card said four; there were three). Proof that the cache now restores needs a second run: build and e2e run in parallel, so the first run can only save. Spec: `docs/changes/NEC-108-pnpm-store-cache/spec.md` ## Changelog NONE

nectenda-agent added the

area:ops

label 2026-09-26 06:57:40 +01:00

nectenda-agent added 2 commits 2026-09-26 06:57:40 +01:00

Cache the pnpm store pnpm actually uses, and check that it did ... d1610affec

Since pnpm 12, a project whose directory cannot hard-link to the home store
gets its store in node_modules/.pnpm-store. The runner's workspace is its own
volume, so every install went through that store while the cache restored
~/.local/share/pnpm/store: a reported hit, reused 0, every package downloaded.

One composite action now asks `pnpm store path` where the store is, caches
that under a new key prefix that also names the store layout, installs, and
fails if node_modules/.modules.yaml records a different store. It replaces
the three copies in ci.yml, obsidian-e2e and checkpoint-soak.

Task: NEC-108

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017Y4mHPEQ29DHbyhjGHHF9g

Say how the cache fix is proved, and run the nightly when the action changes ... 418450595d

The first run cannot show a restore: build and e2e run in parallel and both
miss the new key. The guard passes whether or not anything was saved, so the
spec now names the log lines that are the evidence. The nightly's path filter
gains the new action, which is now part of its setup.

Task: NEC-108

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017Y4mHPEQ29DHbyhjGHHF9g

cruelacid force-pushed worktree-nec-108-the-pnpm-store-cache-has-never-restored from 418450595d to 2632c0313a 2026-09-26 07:09:27 +01:00 Compare

nectenda-agent merged commit 2632c0313a into main 2026-09-26 07:15:27 +01:00

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

area:docs

docs/

  

area:identity

packages/identity

  

area:ops

deploy/, CI, hosting

  

area:plugin

packages/plugin

  

area:server

packages/server

  

channel:community

Obsidian forum, Obsidian Discord, Reddit

  

channel:direct

Cold outreach to named firms

  

channel:owned

nectenda.com pages, the feedback board

  

channel:press

Hacker News, newsletters, outlets, YouTube creators

  

channel:social

Bluesky, Mastodon, X, YouTube accounts

  

e2ee-constrained

End-to-end encryption changes the shape of the answer

  

gate:at-ga

Held until Gates A-C close and terms/privacy leave DRAFT

  

gate:pre-ga

Safe to do before the launch gates close

  

marketing

Reach and positioning; work published off nectenda.com

  

parity

Relay feature-parity programme

  

relay:absent

4 - Not supported, not requested, not planned at Relay

  

relay:planned

2 - On Relay's roadmap (Active/Planned) or has an open PR

  

relay:requested

3 - Asked for by Relay users or under 'Considering'; not committed

  

relay:supported

1 - Relay ships this today

  

risk:additive

New message type, table or column; backward compatible

  

risk:contract

Changes something existing accounts or stored data depend on

  

risk:none

Client-only; the server is untouched

  

usability

How the product is understood and learned, not what it does

No labels

area:docs

area:identity

area:ops

area:plugin

area:server

channel:community

channel:direct

channel:owned

channel:press

channel:social

e2ee-constrained

gate:at-ga

gate:pre-ga

marketing

parity

relay:absent

relay:planned

relay:requested

relay:supported

risk:additive

risk:contract

risk:none

usability

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

Nectenda/nectenda!180

No description provided.