diff --git a/.github/workflows/code-quality.yml b/.github/workflows/code-quality.yml index 7822e89a0..98a247460 100644 --- a/.github/workflows/code-quality.yml +++ b/.github/workflows/code-quality.yml @@ -188,13 +188,20 @@ jobs: run: | docker run -d --name homarr-mem-test -p 17575:7575 homarr-memory-test # Wait for container to be healthy (up to 60s) + healthy=false for i in $(seq 1 30); do if curl -sf http://localhost:17575/api/health 2>/dev/null; then echo "Container healthy after $((i * 2))s" + healthy=true break fi sleep 2 done + if [ "$healthy" != "true" ]; then + echo "ERROR: Container never became healthy" + docker logs homarr-mem-test --tail 50 + exit 1 + fi # Warm up: hit the dashboard page a few times for i in $(seq 1 5); do curl -sf http://localhost:17575/ > /dev/null 2>&1 || true diff --git a/Dockerfile b/Dockerfile index 288dee4f8..9c4d41de8 100644 --- a/Dockerfile +++ b/Dockerfile @@ -20,19 +20,6 @@ ARG DISABLE_REDIS_LOGS='true' RUN corepack enable pnpm && pnpm build -# Copy server chunk source maps into standalone output when enabled. -# Off by default to avoid the significant memory overhead of --enable-source-maps in Node.js. -# See https://github.com/homarr-labs/homarr/issues/6123 and https://github.com/nodejs/node/issues/46140 -ARG ENABLE_SOURCE_MAPS=false -RUN if [ "$ENABLE_SOURCE_MAPS" = "true" ]; then \ - cd apps/nextjs/.next/server && \ - find . -name '*.map' | while IFS= read -r map; do \ - dest="../standalone/apps/nextjs/.next/server/$map"; \ - mkdir -p "$(dirname "$dest")"; \ - cp "$map" "$dest"; \ - done; \ - fi - FROM base AS runner WORKDIR /app @@ -77,9 +64,6 @@ ENV DB_DRIVER='better-sqlite3' ENV AUTH_PROVIDERS='credentials' ENV REDIS_IS_EXTERNAL='false' ENV NODE_ENV='production' -# Pass through the source maps toggle so run.sh can conditionally set NODE_OPTIONS. -ARG ENABLE_SOURCE_MAPS=false -ENV HOMARR_ENABLE_SOURCE_MAPS=$ENABLE_SOURCE_MAPS ENTRYPOINT [ "/app/entrypoint.sh" ] CMD ["sh", "run.sh"] diff --git a/apps/nextjs/next.config.ts b/apps/nextjs/next.config.ts index a050ecb80..636df3f52 100644 --- a/apps/nextjs/next.config.ts +++ b/apps/nextjs/next.config.ts @@ -31,7 +31,6 @@ const nextConfig: NextConfig = { }, output: "standalone", reactStrictMode: true, - productionBrowserSourceMaps: process.env.HOMARR_ENABLE_SOURCE_MAPS === "true", // react compiler breaks mantine-react-table, so disabled for now //reactCompiler: true, /** We already do typechecking as separate tasks in CI */ @@ -42,7 +41,6 @@ const nextConfig: NextConfig = { */ serverExternalPackages: ["dockerode", "isomorphic-dompurify", "jsdom", "better-sqlite3"], experimental: { - serverSourceMaps: process.env.HOMARR_ENABLE_SOURCE_MAPS === "true", optimizePackageImports: ["@mantine/core", "@mantine/hooks", "@tabler/icons-react"], turbopackFileSystemCacheForDev: true, preloadEntriesOnStart: false, diff --git a/scripts/run.sh b/scripts/run.sh index 817c86018..83eff588e 100644 --- a/scripts/run.sh +++ b/scripts/run.sh @@ -58,12 +58,6 @@ terminate() { trap terminate TERM INT -# Enable source maps in Node.js when explicitly opted in. -# Default off — see https://github.com/homarr-labs/homarr/issues/6123 -if [ "${HOMARR_ENABLE_SOURCE_MAPS}" = "true" ]; then - export NODE_OPTIONS="--enable-source-maps${NODE_OPTIONS:+ $NODE_OPTIONS}" -fi - node apps/nextjs/server.js & NEXTJS_PID=$!