admin/goodgo-platform
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(web): call next binary directly instead of npx in Dockerfile
Some checks failed
CI / Lint → Typecheck → Test → Build (22) (push) Failing after 8s
Details
CI / E2E Tests (push) Has been skipped
Details
CI / AI Services (Python) — Smoke (push) Failing after 5s
Details
CodeQL Analysis / CodeQL (javascript-typescript) (push) Failing after 1m4s
Details
Deploy / Build API Image (push) Failing after 15s
Details
Deploy / Build Web Image (push) Failing after 6s
Details
Deploy / Build AI Services Image (push) Failing after 7s
Details
E2E Tests / Playwright E2E (push) Failing after 11s
Details
Security Scanning / Dependency Audit (pnpm) (push) Failing after 4s
Details
Security Scanning / Trivy Scan — API Image (push) Failing after 48s
Details
Security Scanning / Trivy Scan — Web Image (push) Failing after 37s
Details
Security Scanning / Trivy Scan — AI Services Image (push) Failing after 1m40s
Details
Security Scanning / Trivy Filesystem Scan (push) Failing after 1m48s
Details
Deploy / Deploy to Staging (push) Has been skipped
Details
Deploy / Smoke Test Staging (push) Has been skipped
Details
Deploy / Deploy to Production (push) Has been skipped
Details
Deploy / Smoke Test Production (push) Has been skipped
Details
Security Scanning / Security Gate (push) Failing after 1s
Details
Deploy / Rollback Staging (push) Has been skipped
Details
Deploy / Rollback Production (push) Has been skipped
Details

Browse Source 
The hoisted node_modules layout (set in deps stage `.npmrc` with
`node-linker=hoisted`) puts `next` at `/app/node_modules/next` only —
`apps/web/node_modules/next` doesn't exist. The previous
`cd apps/web && npx next build` failed with:

  Cannot find module '/app/apps/web/node_modules/next/dist/bin/next'

because `npx` resolves binaries against the cwd subtree and doesn't
walk up. Switching to the explicit binary path
`/app/node_modules/.bin/next build` makes the build reproducible
in the goodgo Docker image.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
Ho Ngoc Hai
2026-04-30 13:49:51 +07:00
parent a38c797846
commit 416d1a5959
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
apps/web/Dockerfile
View File

@@ -23,7 +23,10 @@ COPY --from=deps /app/.npmrc ./
COPY --from=deps /app/node_modules ./node_modules
COPY tsconfig.base.json ./
COPY apps/web/ apps/web/
RUN cd apps/web && npx next build
# Hoisted layout: `next` binary is in /app/node_modules/.bin/next.
# Neither `npx next` nor `pnpm run build` resolve it from apps/web cwd
# because that subtree has no node_modules. Call the binary directly.
RUN cd apps/web && /app/node_modules/.bin/next build
# ---- Production ----
FROM node:22-slim AS production