From a1266378fc612fe85d47d3a94309be7910f4bd4f Mon Sep 17 00:00:00 2001
From: Bernard Siebens <bernard@siebens.org>
Date: Wed, 15 Jul 2026 13:10:09 +0200
Subject: [PATCH] Fix deploy script: compose run was eating the rest of the
 heredoc
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The whole remote script is fed to `ssh bash -s` as stdin (a heredoc). `docker
compose run` without -T attaches that stdin to the container, so it consumed every
line after the migrate — web was never restarted and no health check ran, yet the
script exited 0 and printed "Done". A deploy that half-ran and reported success.

`-T` plus `</dev/null` on the migrate command stops it reading the heredoc.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
---
 deploy/deploy-dev.sh | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/deploy/deploy-dev.sh b/deploy/deploy-dev.sh
index d56baf7..28ba504 100755
--- a/deploy/deploy-dev.sh
+++ b/deploy/deploy-dev.sh
@@ -75,7 +75,10 @@ step "Starting db + redis"
 dc up -d db redis
 
 step "Running migrations"
-dc run --rm web python manage.py migrate --noinput
+# -T and </dev/null are load-bearing: this whole script IS ssh's stdin (a heredoc), and
+# `compose run` without them attaches that stdin to the container — swallowing every command
+# below it, so web never restarts and the script exits 0 having done half the job.
+dc run --rm -T web python manage.py migrate --noinput </dev/null
 
 # Recreate only web, with the freshly built image. db and redis keep running untouched.
 step "Restarting web"