[BUILD] Remove redundant npm install step - reflex export handles it (fix) | ApprovalToken: AT-202512310829
Some checks failed
ci/woodpecker/push/woodpecker Pipeline failed

This commit is contained in:
Ehsan.Asadi
2025-12-31 08:29:59 +03:30
parent 5813e6458a
commit a19f73d826

View File

@@ -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)