From 419c68c024cf52ab431de0c37c88f15fbd43f603 Mon Sep 17 00:00:00 2001
From: WrBug <iwrbug@gmail.com>
Date: Wed, 21 Jan 2026 21:04:25 +0800
Subject: [PATCH] =?UTF-8?q?fix:=20=E4=BF=AE=E5=A4=8D=E6=9C=AC=E5=9C=B0=20D?=
 =?UTF-8?q?ocker=20=E6=9E=84=E5=BB=BA=E6=97=B6=20frontend/dist=20=E5=92=8C?=
 =?UTF-8?q?=20backend/build=20=E4=B8=8D=E5=AD=98=E5=9C=A8=E7=9A=84?=
 =?UTF-8?q?=E9=97=AE=E9=A2=98?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- 移除 Dockerfile 中无效的 COPY frontend/dist 指令
- 改进前端构建逻辑，添加外部产物检查
- 改进后端构建逻辑，支持复制外部构建的 JAR 文件
- 在 deploy.sh 中添加创建占位符目录步骤，避免 COPY 失败
- 保持与 GitHub Actions 构建流程的兼容性（BUILD_IN_DOCKER=false）

修复了本地使用 ./deploy.sh 部署时的构建错误：
"failed to calculate checksum: /frontend/dist: not found"
---
 Dockerfile | 44 ++++++++++++++++++++++++++++++++------------
 deploy.sh  |  4 ++++
 2 files changed, 36 insertions(+), 12 deletions(-)

diff --git a/Dockerfile b/Dockerfile
index 952b9f9..bd408f3 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -31,20 +31,22 @@ RUN if [ "$BUILD_IN_DOCKER" = "true" ]; then \
 COPY frontend/ ./
 
 # 条件：仅在 Docker 内部编译时执行构建
-# 如果 BUILD_IN_DOCKER=false，需要从构建上下文复制外部编译的 dist
+# 如果 BUILD_IN_DOCKER=false，需要确保构建上下文中存在 frontend/dist
+# 注意：COPY frontend/ ./ 已经复制了整个 frontend 目录（包括 dist，如果存在）
 RUN if [ "$BUILD_IN_DOCKER" = "true" ]; then \
       echo "🔨 Docker 内部编译前端..."; \
       npm run build; \
     else \
-      echo "⏭️  使用外部产物，将在下一步复制"; \
-      mkdir -p dist; \
+      echo "⏭️  使用外部产物..."; \
+      if [ ! -d "dist" ] || [ -z "$(ls -A dist 2>/dev/null)" ]; then \
+        echo "❌ 错误：BUILD_IN_DOCKER=false 但找不到外部产物 frontend/dist"; \
+        echo "   请先执行: cd frontend && npm install && npm run build"; \
+        exit 1; \
+      else \
+        echo "✅ 找到外部构建的前端产物"; \
+      fi; \
     fi
 
-# 仅在使用外部产物时复制 dist 目录到构建上下文
-# 在 BUILD_IN_DOCKER=true 时，上面的 npm run build 已经创建了 dist
-# 在 BUILD_IN_DOCKER=false 时，需要从构建上下文复制外部编译的 dist
-COPY frontend/dist ./dist
-
 # ==================== 阶段2：构建后端 ====================
 FROM gradle:8.5-jdk17 AS backend-build
 ARG BUILD_IN_DOCKER
@@ -63,9 +65,25 @@ RUN if [ "$BUILD_IN_DOCKER" = "true" ]; then \
 # 复制源代码
 COPY backend/src ./src
 
-# 如果使用外部产物，先从构建上下文复制外部编译的 JAR
-# 注意：如果 BUILD_IN_DOCKER=true 且本地没有 JAR，这个 COPY 会失败，但会在下面编译生成
-COPY backend/build/libs/*.jar build/libs/
+# 尝试复制外部构建的 JAR（如果存在）
+# 注意：COPY 指令如果源不存在会失败
+# GitHub Actions 使用 BUILD_IN_DOCKER=false，会先构建产物，所以 backend/build 应该存在
+# 本地开发使用 BUILD_IN_DOCKER=true，会在 Docker 内编译，所以 backend/build 可能不存在
+# 解决方案：先复制整个 backend 目录（包括 build，如果存在），然后只使用需要的部分
+# 使用 .dockerignore 确保不会复制不需要的文件（如 .gradle、out、bin 等）
+COPY backend/build ./build-external
+
+# 处理外部构建的 JAR（如果存在）
+RUN if [ -d "build-external/libs" ] && [ -n "$(ls -A build-external/libs/*.jar 2>/dev/null)" ]; then \
+      echo "📦 找到外部构建的后端产物，复制到 build/libs..."; \
+      mkdir -p build/libs; \
+      cp build-external/libs/*.jar build/libs/; \
+      rm -rf build-external; \
+    else \
+      echo "⏭️  未找到外部构建的 JAR，将在 Docker 内编译"; \
+      rm -rf build-external; \
+      mkdir -p build/libs; \
+    fi
 
 # 条件：仅在 Docker 内部编译时执行构建（会覆盖外部产物）
 RUN if [ "$BUILD_IN_DOCKER" = "true" ]; then \
@@ -73,10 +91,12 @@ RUN if [ "$BUILD_IN_DOCKER" = "true" ]; then \
       gradle bootJar --no-daemon; \
     else \
       echo "⏭️  使用外部产物"; \
-      mkdir -p build/libs; \
       if [ -z "$(ls -A build/libs/*.jar 2>/dev/null)" ]; then \
         echo "❌ 错误：BUILD_IN_DOCKER=false 但找不到外部产物 backend/build/libs/*.jar"; \
+        echo "   请先执行: cd backend && ./gradlew bootJar"; \
         exit 1; \
+      else \
+        echo "✅ 使用外部构建的后端产物"; \
       fi; \
     fi
 
diff --git a/deploy.sh b/deploy.sh
index 89dce2f..e79eace 100755
--- a/deploy.sh
+++ b/deploy.sh
@@ -168,6 +168,10 @@ deploy() {
         
         info "构建 Docker 镜像（本地构建，版本号: ${DOCKER_VERSION}）..."
         
+        # 创建占位符目录（如果不存在），避免 Dockerfile COPY 失败
+        # 当 BUILD_IN_DOCKER=true 时，backend/build 可能不存在
+        mkdir -p backend/build/libs
+        
         # 设置构建参数（通过环境变量传递给 docker-compose.yml）
         export VERSION=${DOCKER_VERSION}
         export GIT_TAG=${DOCKER_VERSION}