Improved Docker workflow

ref https://linear.app/ghost/issue/AP-550/overhaul-docker-testing-setup

- made `activitypub` rely on `migrate` so we run migrations in dev
- fixed `yarn fix` actually removing the images
- `yarn dev` will now try and recreate the containers, which will often
  no-op, but they'll be updated if the package.json changes

Author: daniellockyer

docker-compose.yml

@@ -15,6 +15,8 @@ services:
       - SKIP_SIGNATURE_VERIFICATION=true
     command: yarn build:watch
     depends_on:
+      migrate:
+        condition: service_started
       mysql:
         condition: service_healthy
       pubsub:

package.json

@@ -9,10 +9,10 @@
   "scripts": {
     "build": "esbuild src/app.ts --platform=neutral --packages=external --bundle --outfile=dist/app.js",
     "build:watch": "concurrently \"yarn build --watch\" \"node --watch dist/app.js\"",
-    "dev": "docker compose up activitypub nginx -d",
+    "dev": "docker compose up activitypub nginx -d --build",
     "stop": "docker compose stop",
     "db": "docker compose exec mysql mysql -uroot -proot activitypub",
-    "fix": "docker compose rm activitypub nginx -sf && docker compose build activitypub nginx",
+    "fix": "docker compose down --rmi all activitypub activitypub-testing cucumber-tests nginx",
     "logs": "docker compose logs activitypub -f",
     "test:cucumber": "docker compose run --rm migrate-testing up && docker compose up cucumber-tests --exit-code-from cucumber-tests",
     "test": "docker compose run --rm migrate-testing up && docker compose run --rm activitypub-testing yarn test:all && yarn test:cucumber",