fix: cross-platform docker compose wrapper

Replace the bash-only `$([ -f .env ] && echo --env-file .env)` substitution
in the docker scripts with `scripts/docker.mjs`, a small Node wrapper that
conditionally passes `--env-file <repo-root>/.env` when the file exists.
The substitution form failed on Windows (`cmd.exe` doesn't parse $(...)),
which pnpm uses by default when running scripts on native Windows.

Also routes `internal-packages/clickhouse/db:migrate` through the same
helper so the migrator picks up `COMPOSE_PROJECT_NAME` from `.env` instead
of the hardcoded `-p triggerdotdev-docker`, matching the multi-instance
workflow.

Author: ericallam

internal-packages/clickhouse/package.json

@@ -22,7 +22,7 @@
     "typecheck": "tsc --noEmit -p tsconfig.build.json",
     "build": "pnpm run clean && tsc -p tsconfig.build.json",
     "dev": "tsc --watch  -p tsconfig.build.json",
-    "db:migrate": "docker compose -p triggerdotdev-docker -f ../../docker/docker-compose.yml up clickhouse_migrator --build",
+    "db:migrate": "node ../../scripts/docker.mjs -f docker/docker-compose.yml up clickhouse_migrator --build",
     "db:migrate:down": "GOOSE_COMMAND=down pnpm run db:migrate",
     "test": "vitest --sequence.concurrent=false --no-file-parallelism",
     "test:coverage": "vitest --sequence.concurrent=false --no-file-parallelism --coverage.enabled"

package.json

@@ -19,10 +19,10 @@
     "i:dev": "infisical run -- turbo run dev",
     "generate": "turbo run generate",
     "lint": "turbo run lint",
-    "docker": "docker compose $([ -f .env ] && echo '--env-file .env') -f docker/docker-compose.yml up -d --build --remove-orphans",
-    "docker:stop": "docker compose $([ -f .env ] && echo '--env-file .env') -f docker/docker-compose.yml stop",
-    "docker:full": "docker compose $([ -f .env ] && echo '--env-file .env') -f docker/docker-compose.yml -f docker/docker-compose.extras.yml up -d --build --remove-orphans",
-    "docker:full:stop": "docker compose $([ -f .env ] && echo '--env-file .env') -f docker/docker-compose.yml -f docker/docker-compose.extras.yml stop",
+    "docker": "node scripts/docker.mjs -f docker/docker-compose.yml up -d --build --remove-orphans",
+    "docker:stop": "node scripts/docker.mjs -f docker/docker-compose.yml stop",
+    "docker:full": "node scripts/docker.mjs -f docker/docker-compose.yml -f docker/docker-compose.extras.yml up -d --build --remove-orphans",
+    "docker:full:stop": "node scripts/docker.mjs -f docker/docker-compose.yml -f docker/docker-compose.extras.yml stop",
     "dev:docker": "docker compose -p triggerdotdev-dev-docker -f docker/dev-compose.yml up -d --build --remove-orphans",
     "dev:docker:build": "docker compose -p triggerdotdev-dev-docker -f docker/dev-compose.yml up -d --build",
     "dev:docker:stop": "docker compose -p triggerdotdev-dev-docker -f docker/dev-compose.yml stop",

scripts/docker.mjs

@@ -0,0 +1,27 @@
+#!/usr/bin/env node
+// Cross-platform wrapper for `docker compose` that conditionally passes
+// `--env-file <repo-root>/.env` when the file exists. Replaces an earlier
+// inline `$([ -f .env ] && echo --env-file .env)` shell substitution that
+// only worked in POSIX shells, breaking native Windows `cmd.exe` runs.
+//
+// Used by the root `pnpm run docker` / `docker:full` scripts and by the
+// clickhouse package's `db:migrate` script. Always runs compose with cwd
+// set to the repo root, so callers can pass `-f docker/docker-compose.yml`
+// from anywhere in the workspace.
+import { existsSync } from "node:fs";
+import { execFileSync } from "node:child_process";
+import { resolve, dirname } from "node:path";
+import { fileURLToPath } from "node:url";
+
+const repoRoot = resolve(dirname(fileURLToPath(import.meta.url)), "..");
+const envPath = resolve(repoRoot, ".env");
+const envArgs = existsSync(envPath) ? ["--env-file", envPath] : [];
+
+try {
+  execFileSync("docker", ["compose", ...envArgs, ...process.argv.slice(2)], {
+    stdio: "inherit",
+    cwd: repoRoot,
+  });
+} catch (err) {
+  process.exit(err.status ?? 1);
+}