Skip to content
This repository was archived by the owner on Aug 5, 2026. It is now read-only.

fix: pnpm pinnen — der 11.11-Bump hat den Produktions-Build zerlegt - #5

Merged
dev0zzz merged 1 commit into
mainfrom
fix/pin-pnpm-for-docker-build
Jul 23, 2026
Merged

fix: pnpm pinnen — der 11.11-Bump hat den Produktions-Build zerlegt#5
dev0zzz merged 1 commit into
mainfrom
fix/pin-pnpm-for-docker-build

Conversation

@dev0zzz

@dev0zzz dev0zzz commented Jul 23, 2026

Copy link
Copy Markdown
Owner

Der Deploy von PR #4 ist fehlgeschlagen, und mein eigener Commit war schuld.

Der Bump von packageManager auf pnpm 11.11.0 hat den allowBuilds-Matcher verschärft: 11.11 will den vollen Spezifizierer eines Pakets (Tarball-URL plus Commit-Hash), wo 11.5 den Paketnamen akzeptierte. Damit greift der Eintrag tessera-sdk: true in pnpm-workspace.yaml nicht mehr und pnpm install --frozen-lockfile stirbt im Docker-Build.

(Coolifys Log maskierte den Commit-Hash als <REDACTED>, was wie ein geleaktes Secret aussah. Ist keins — nur eine lange Hex-Zeichenkette.)

Den vollen Spezifizierer in die Allowlist zu schreiben würde bis zum nächsten SDK-Update halten, dann ändert sich der Hash und es bricht erneut. Zurück auf 11.5.2 ist der ehrliche Weg: Der Bump hat nichts gebracht und einen funktionierenden Deploy gekostet.

Zusätzlich stand im Dockerfile pnpm@11 — ein gleitender Major, der beim Bauen einfach das neueste 11.x zieht. Derselbe Fehler wäre also irgendwann von allein gekommen. Beide Stellen sind jetzt auf 11.5.2 gepinnt und kommentiert.

Verifiziert durch einen lokalen docker build, nicht durch Hoffnung.

Lehre für später

Die CI blieb die ganze Zeit grün, weil pnpm test das Image nicht baut. Der Deploy ist aber der Image-Build — ein grüner Haken war hier irreführend. Ein docker build-Schritt in der CI hätte das vor dem Merge gefangen. Das braucht eine Änderung an .github/workflows/, die mit dem aktuellen Token nicht pushbar ist; sag Bescheid, dann mache ich es separat.

🤖 Generated with Claude Code

https://claude.ai/code/session_01SU6PBvGFbeHyKhBgnAnz1H

My own chore commit caused this. Bumping packageManager to 11.11.0 changed the
allowBuilds matcher: 11.11 wants a package's FULL specifier (tarball URL plus
commit hash) where 11.5 accepted its name, so the `tessera-sdk: true` entry in
pnpm-workspace.yaml stopped matching and `pnpm install --frozen-lockfile` died
inside the Docker build. Coolify's log masked the commit hash as <REDACTED>,
which made it look like a leaked secret — it is not, just a long hex string.

Writing the full specifier into allowBuilds would "fix" it until the next SDK
version, when the hash changes and it breaks again. Reverting to 11.5.2 is the
honest move: the bump bought nothing and cost a working deploy.

The Dockerfile also asked for `pnpm@11`, a floating major that silently picks
up whatever 11.x is newest at build time — the same failure was going to
arrive on its own eventually. Both are pinned to 11.5.2 now and annotated to
stay in lockstep.

Verified by building the image locally, not by hoping.

Note for later: CI stayed green through all of this because `pnpm test` does
not build the image. The deploy IS the image build, so a green check was
misleading. Adding a docker build step to CI would have caught it pre-merge.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01SU6PBvGFbeHyKhBgnAnz1H
@dev0zzz
dev0zzz merged commit 027ef08 into main Jul 23, 2026
2 checks passed
@dev0zzz
dev0zzz deleted the fix/pin-pnpm-for-docker-build branch July 23, 2026 10:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant