chore(skills): /be — refresh the pnpm FOD hash the instant the lockfile changes#1552
Draft
srid wants to merge 1 commit into
Draft
chore(skills): /be — refresh the pnpm FOD hash the instant the lockfile changes#1552srid wants to merge 1 commit into
srid wants to merge 1 commit into
Conversation
…le changes When a /be §2 change touches package.json / pnpm-lock.yaml, the recorded fetchPnpmDeps hash in nix/modules/typescript.nix goes stale and every linux nix-build CI lane reds at once — a guaranteed wasted CI cycle if left for §5 to surface. The export-session run hit exactly this: it changed the lockfile, never refreshed the hash, and only discovered the staleness when a full CI cycle came back red, then had to fix + re-run all of CI. §2 already warns "just check green is not proof the artifact builds" for bundler/entrypoint changes; this adds the sibling dependency-change case and cross-links /nix-typescript, which documents the fix recipe and the background-refresh-on-lockfile-change directive. just check never catches this; only a real nix build does. Provenance: mined from session dd538ee1-881b-406a-aca8-f49137976b27. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
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
1 participant
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.
What
A one-sentence addition to
/be§2, exactly where it already warns "just checkgreen is not proof the artifact builds" — extending that lesson to the sibling dependency-change failure mode and cross-linking the/nix-typescriptskill that fixes it.Source edit lands in
.apm/skills/be/SKILL.md; the two generated copies (.claude/,.agents/) and theapm.lock.yamlcontent-hashes ride the same commit (regenerated viajust ai::apm).Why — the intervention this engineers out
This is the per-session
/self-improvepass over sessiondd538ee1, an autonomous/berun for the HTML session-export feature. The run needed only one human turn (the kickoff) — but it burned a full CI cycle on an avoidable red:package.json+pnpm-lock.yaml(removed deps), never refreshed thefetchPnpmDepsFOD hashci::pnpm-hash-fresh,ci::nix,ci::smoke, …) — the classic stale-hash cascadenix/modules/typescript.nix, re-ran all of CI greenThe
/nix-typescriptskill already documents this exact failure and its fix — and says to refresh the hash in the background the instant the lockfile changes, not at end of session./besimply never loaded it. The fix is a cross-link (the lever map's lowest-churn move), not new machinery.Evidence ledger
/be§2 cross-link to/nix-typescript.claude/skills/nix-typescript/SKILL.md§"Dependency hash management" + "Parallelization"ci::pnpm-hash-freshred on both platforms → hash bump innix/modules/typescript.nix→ CI re-runjust checkcannot catch it (onlynix buildcan) — so the warning belongs in §2 next to the existing build-vs-typecheck caveatjust checkgreen is not proof the artifact builds" clause.claude/+.agents/copies +apm.lock.yamlhashes regenerated from sourcejust ai::apm;just checkgreenGates
just ai::apm— regenerated (pre-existing unrelated orphan warning only)just fmt— clean (0 reformatted)just check— green (tsc all packages + biome)Draft for human review — per
/self-improve, these edits go live only when you merge.🤖 Generated with Claude Code