From 52610c546462415cd54a1751fbcc4af1211ef45b Mon Sep 17 00:00:00 2001
From: Conor Schaefer <conorsch@users.noreply.github.com>
Date: Mon, 3 Feb 2025 12:26:08 -0800
Subject: [PATCH] fix(build): ditch corepack in containerfile (#197)

The deploy pipeline was failing on container builds (across the PL
frontend repos) due to upstream breakage in corepack [0].
This changes matches changes in e.g. [1], [2].

[0] https://github.com/nodejs/corepack/issues/612
[1] https://github.com/penumbra-zone/penumbers/pull/19
[2] https://github.com/penumbra-zone/dex-explorer/pull/341
---
 apps/web/Containerfile | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/apps/web/Containerfile b/apps/web/Containerfile
index 0eca07d..95f781e 100644
--- a/apps/web/Containerfile
+++ b/apps/web/Containerfile
@@ -1,13 +1,14 @@
 # slimmest image. only for running the container.
-FROM docker.io/library/node:20-alpine AS alpine
+ARG NODE_MAJOR_VERSION=22
+FROM docker.io/library/node:${NODE_MAJOR_VERSION}-alpine AS base
 LABEL maintainer="team@penumbralabs.xyz"
 
-# provide pnpm globally for dep installing and building
-FROM alpine AS base
 ENV NEXT_TELEMETRY_DISABLED=1
 ENV PNPM_HOME="/pnpm"
 ENV PATH="$PNPM_HOME:$PATH"
-RUN corepack enable pnpm
+# We no longer use `corepack enable pnpm` due to breakage documented in
+# https://github.com/nodejs/corepack/issues/612
+RUN npm install -g pnpm@${PNPM_VERSION}
 RUN pnpm install turbo --global
 
 # prune package structure + code into out/
@@ -44,7 +45,7 @@ COPY turbo.json turbo.json
 RUN turbo run build --filter=cuiloa-app
 
 # Run container
-FROM alpine AS runner
+FROM base AS runner
 WORKDIR /app
 
 # Disable telemetry.