From 897b1c9beff6e40a39503cbbb6987d8de8d90e27 Mon Sep 17 00:00:00 2001 From: jason Date: Sun, 29 Mar 2026 22:00:15 -0500 Subject: [PATCH] Fix Dockerfile COPY instruction with invalid shell syntax MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Removed the conditional COPY line that used 2>/dev/null || true — shell operators are not valid in Dockerfile COPY instructions and were being interpreted as literal paths. pnpm hoists all deps to root node_modules via shamefully-hoist so the line was unnecessary. Co-Authored-By: Claude Sonnet 4.6 --- Dockerfile | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/Dockerfile b/Dockerfile index fcb990b..d96207b 100644 --- a/Dockerfile +++ b/Dockerfile @@ -54,9 +54,8 @@ RUN apk add --no-cache tini su-exec WORKDIR /app # No native deps — runtime only needs node_modules for pure-JS packages (express, cors, etc.) -# Copy them directly from the builder stage instead of re-installing +# pnpm hoists everything to the root node_modules (shamefully-hoist=true in .npmrc) COPY --from=server-builder /build/node_modules ./node_modules -COPY --from=server-builder /build/apps/server/node_modules ./apps/server/node_modules 2>/dev/null || true # Copy compiled server output (includes dist/db/migrations/*.js compiled by tsc) COPY --from=server-builder /build/apps/server/dist ./apps/server/dist