From 5ee39c74af1e10bd7ebab4ad3e4ecc611a6fe3b8 Mon Sep 17 00:00:00 2001
From: Thieu Nguyen <79964592+thieung@users.noreply.github.com>
Date: Sat, 18 Apr 2026 09:17:14 +0700
Subject: [PATCH] fix(docker): exclude node deps and dist from build context
 (#949)

Root .dockerignore did not exclude local dependency folders, so
COPY ui/web/ . shipped ui/web's local install into the web-builder
stage. Stale artifacts (e.g. leftover rollup@4.57.1 from older vite@6)
then overrode the result of `pnpm install --frozen-lockfile`,
breaking the musl build on Alpine with:

  Cannot find module @rollup/rollup-linux-arm64-musl

Add explicit ignores for ui/web dependency and dist folders plus a
catch-all for any nested dep folder so Docker builds are
deterministic and match CI regardless of local dev state. Also
speeds up build context transfer.
---
 .dockerignore | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/.dockerignore b/.dockerignore
index 47dddb0f..1a219b61 100644
--- a/.dockerignore
+++ b/.dockerignore
@@ -19,5 +19,8 @@ tmp/
 ui/desktop/
 ui/simple-saas/
 !ui/web/
+ui/web/node_modules/
+ui/web/dist/
 plans/
 skills-store/
+**/node_modules/