Fix sharp missing error in logs (SSWConsulting#2570)

pnpm - update to use version specified in package.json
Specifies version of pnpm in package.json to avoid version mismatches between dev laptops/CI runners

Author: wicksipedia

.dockerignore

@@ -3,7 +3,6 @@ _docs
 .devcontainer
 .git
 .github
-# .next
 .vscode
 .vs
 Dockerfile

.github/workflows/main-build-and-deploy.yml

@@ -27,7 +27,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: pnpm/action-setup@v3
         with:
-          version: 8
+          package_json_file: package.json
       - name: Install dependencies
         run: pnpm install
 

.github/workflows/pr-lint-code.yml

@@ -33,7 +33,7 @@ jobs:
 
       - uses: pnpm/action-setup@v3
         with:
-          version: 8
+          package_json_file: package.json
 
       - name: Setup node
         uses: actions/setup-node@v4

.github/workflows/pr-push-deploy.yml

@@ -25,7 +25,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: pnpm/action-setup@v3
         with:
-          version: 8
+          package_json_file: package.json
       - name: Install dependencies
         run: pnpm install
 

.github/workflows/template-ui-tests.yml

@@ -34,7 +34,7 @@ jobs:
 
       - uses: pnpm/action-setup@v3
         with:
-          version: 8
+          package_json_file: package.json
 
       - name: Setup Node
         uses: actions/setup-node@v4

Dockerfile

@@ -1,6 +1,7 @@
+FROM node:lts-alpine AS base
+
 # Install dependencies only when needed
-FROM node:lts-alpine AS deps
-RUN corepack enable
+FROM base AS deps
 
 # Check https://github.com/nodejs/docker-node/tree/b4117f9333da4138b03a546ec926ef50a31506c3#nodealpine to understand why libc6-compat might be needed.
 RUN apk add --no-cache libc6-compat
@@ -16,12 +17,11 @@ RUN \
   fi
 
 # Rebuild the source code only when needed
-FROM node:lts-alpine AS builder
-RUN corepack enable
+FROM base AS builder
 WORKDIR /app
 
-COPY . .
 COPY --from=deps /app/node_modules ./node_modules
+COPY . .
 
 # Add env for production
 # COPY .docker/production/.env.local .env.local
@@ -89,32 +89,42 @@ ENV TINA_SEARCH_TOKEN $TINA_SEARCH_TOKEN
 ARG NEXT_PUBLIC_SLOT_URL
 ENV NEXT_PUBLIC_SLOT_URL $NEXT_PUBLIC_SLOT_URL
 
-RUN pnpm run build
+RUN \
+  if [ -f yarn.lock ]; then yarn run build; \
+  elif [ -f package-lock.json ]; then npm run build; \
+  elif [ -f pnpm-lock.yaml ]; then corepack enable pnpm && pnpm run build; \
+  else echo "Lockfile not found." && exit 1; \
+  fi
 
 # Production image, copy all the files and run next
-FROM node:lts-alpine AS runner
-RUN corepack enable
+FROM base AS runner
 WORKDIR /app
 
+ENV NODE_ENV production
 # Uncomment the following line in case you want to disable telemetry during runtime.
 # ENV NEXT_TELEMETRY_DISABLED 1
 
 RUN addgroup --system --gid 1001 nodejs
 RUN adduser --system --uid 1001 nextjs
 
+COPY --from=builder /app/public ./public
+
+# Set the correct permission for prerender cache
+RUN mkdir .next
+RUN chown nextjs:nodejs .next
+
 # Automatically leverage output traces to reduce image size
 # https://nextjs.org/docs/advanced-features/output-file-tracing
+COPY --from=builder --chown=nextjs:nodejs /app/.next/standalone ./
+COPY --from=builder --chown=nextjs:nodejs /app/.next/static ./.next/static
+COPY --from=builder --chown=nextjs:nodejs /app/appInsight-api.js ./
 
 USER nextjs
 
 EXPOSE 3000
 
-ENV NODE_ENV production
 ENV PORT 3000
 
-COPY --from=builder --chown=nextjs:nodejs /app/public ./public
-COPY --from=builder --chown=nextjs:nodejs /app/.next/standalone ./
-COPY --from=builder --chown=nextjs:nodejs /app/.next/static ./.next/static
-COPY --from=builder --chown=nextjs:nodejs /app/appInsight-api.js ./
-
-CMD ["node", "--require", "./appInsight-api.js", "server.js"]
+# server.js is created by next build from the standalone output
+# https://nextjs.org/docs/pages/api-reference/next-config-js/output
+CMD HOSTNAME="0.0.0.0" node --require ./appInsight-api.js server.js

package.json

@@ -98,5 +98,6 @@
   },
   "resolutions": {
     "final-form": "4.20.1"
-  }
+  },
+  "packageManager": "[email protected]"
 }