From a19f73d826d140d441b630b8515a949bdea71795 Mon Sep 17 00:00:00 2001 From: "Ehsan.Asadi" Date: Wed, 31 Dec 2025 08:29:59 +0330 Subject: [PATCH] [BUILD] Remove redundant npm install step - reflex export handles it (fix) | ApprovalToken: AT-202512310829 --- docker/Dockerfile | 30 +++--------------------------- 1 file changed, 3 insertions(+), 27 deletions(-) diff --git a/docker/Dockerfile b/docker/Dockerfile index aee1f06..66b9659 100644 --- a/docker/Dockerfile +++ b/docker/Dockerfile @@ -66,33 +66,9 @@ RUN cd /build/peikarband && \ reflex init --loglevel debug || true && \ reflex export --frontend-only --no-zip --loglevel debug || echo "Export completed with warnings" -# Install npm dependencies if .web directory exists -# Note: reflex export already builds the frontend, we just need to install deps -RUN if [ -d "/build/peikarband/.web" ] && [ -f "/build/peikarband/.web/package.json" ]; then \ - echo "Found .web directory with package.json, installing dependencies..." && \ - cd /build/peikarband/.web && \ - # Remove any existing .npmrc that might override registry - rm -f .npmrc && \ - # Set npm registry to official registry - npm config set registry https://registry.npmjs.org/ && \ - npm config set fetch-retry-mintimeout 20000 && \ - npm config set fetch-retry-maxtimeout 120000 && \ - npm config set fetch-retries 5 && \ - npm config set fetch-timeout 300000 && \ - # Verify registry is set correctly - echo "Using npm registry: $(npm config get registry)" && \ - # Install dependencies (reflex export already built the frontend) - if [ -f "package-lock.json" ]; then \ - npm ci --prefer-offline --no-audit --loglevel verbose || \ - (echo "npm ci failed, retrying with npm install..." && npm install --prefer-offline --no-audit --loglevel verbose); \ - else \ - echo "package-lock.json not found, using npm install..." && \ - npm install --prefer-offline --no-audit --loglevel verbose; \ - fi && \ - echo "Dependencies installed successfully"; \ - else \ - echo "Warning: .web directory or package.json not found, skipping npm install"; \ - fi +# Note: reflex export already builds and installs everything needed +# No additional npm install is required +RUN echo "Frontend built by reflex export" # ============================================ # Stage 2: Runtime (using base image for Node.js)