From 19054718d5d4aa2c545c98a27067b837a3b49611 Mon Sep 17 00:00:00 2001
From: yuetsh <517252939@qq.com>
Date: Wed, 26 Aug 2026 08:25:17 -0600
Subject: [PATCH] =?UTF-8?q?fix(=E9=83=A8=E7=BD=B2):=20=E8=BF=81=E7=A7=BB?=
 =?UTF-8?q?=E9=82=A3=E6=AD=A5=E5=8E=BB=E6=8E=89=20--no-deps=EF=BC=8C?=
 =?UTF-8?q?=E5=90=A6=E5=88=99=E8=87=AA=E5=B8=A6=E6=95=B0=E6=8D=AE=E5=BD=A2?=
 =?UTF-8?q?=E6=80=81=E4=B8=8B=E8=B5=B7=E4=B8=8D=E6=9D=A5=E5=BA=93?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

上一个提交让 deploy.sh 支持「自带数据」形态，但迁移那步还留着 --no-deps。
`run --rm --no-deps api oj2-api migrate` 不会拉起 oj-postgres，于是这一步
直接连不上库 —— 而它正好排在 `up -d` 之前，postgres 这时候还没起。

oj-api 的依赖恰好就是 oj-postgres / oj-redis，两种形态都是对的：

- 自带数据：靠 run 把 postgres 起来，depends_on 的 condition: service_healthy
  还顺带保证库真的就绪了才开始迁移
- 外接：那两个服务不在启用的 profile 里，depends_on 上的 required: false 让
  compose 跳过，不会多起东西

worker / web 不是 api 的依赖，两种形态下都不会被带起来，去掉 --no-deps
不会有副作用。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
---
 docker/deploy.sh | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/docker/deploy.sh b/docker/deploy.sh
index bc25b70..822c493 100755
--- a/docker/deploy.sh
+++ b/docker/deploy.sh
@@ -174,14 +174,19 @@ fi
 # ---------------------------------------------------------------- 迁移
 # 在起栈**之前**跑：schema 先就位，新代码再启动。
 #
-# --no-deps 是必须的：这一步只要 api 镜像里的二进制，不该顺带把 worker/web 拉起来。
-# 库连的是 env 里的 DB_HOST，和起栈用的是同一份配置。
+# **不要加 --no-deps。** `run` 只会拉起被点名服务的依赖，而 oj-api 的依赖恰好就是
+# oj-postgres / oj-redis：
+#   · 自带数据形态 —— 必须靠它把 postgres 起来，否则这一步连不上库（加过 --no-deps，
+#     踩过这个坑）。depends_on 的 condition: service_healthy 还顺带保证了库真的就绪。
+#   · 外接形态 —— 那两个服务不在启用的 profile 里，depends_on 上的 required: false
+#     让 compose 直接跳过，不会多起任何东西。
+# 两种形态下 worker / web 都不会被带起来，它们不是 api 的依赖。
 #
 # 含 DROP TABLE 这类破坏性语句的迁移会被拦下并退出 4，需要显式放行：
 #   OJ2_ALLOW_DESTRUCTIVE=1 docker/deploy.sh
 # 放行前先备份。这道闸门在 apps/api/src/db/migrate.ts。
 say "数据库迁移"
-"${COMPOSE[@]}" run --rm --no-deps \
+"${COMPOSE[@]}" run --rm \
   -e OJ2_ALLOW_DESTRUCTIVE="${OJ2_ALLOW_DESTRUCTIVE:-}" \
   api oj2-api migrate \
   || die "迁移没通过，已中止部署（旧容器还在跑，没动过）。