refactor(ci): unify on selfhosted profile name (#1798)

* fix(ci): default self-host health checks to selfhost profile

* refactor(ci): unify on selfhosted profile name

Simplify by using "selfhosted" consistently everywhere instead of
having two terms ("selfhost" for Docker Compose, "selfhosted" for
scripts). This removes the need for translation logic in the health
check script.

- Rename Docker Compose profile: selfhost → selfhosted
- Remove resolved_compose_profiles() and remote_compose_env_prefix()
- Simplify remote_compose_prefix() to check PROFILE directly
- Update all scripts, workflows, and docs

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* refactor: export COMPOSE_PROFILES once instead of per-command

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* revert: remove unrelated import change from PR

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

---------

Co-authored-by: Tyler Dane <tyler@switchback.tech>
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

Author: 3 people

.github/scripts/deploy-health-check.sh

@@ -367,14 +367,11 @@ ssh_remote() {
 }
 
 remote_compose_prefix() {
-  local profiles=${COMPOSE_PROFILES:-}
-
-  if [ -n "$profiles" ]; then
-    printf "cd ~/compass && COMPOSE_PROFILES=%q docker compose --project-name compass -f compose.yaml" "$profiles"
-    return
+  if [ "${PROFILE:-}" = "selfhosted" ]; then
+    printf 'cd ~/compass && COMPOSE_PROFILES=selfhosted docker compose --project-name compass -f compose.yaml'
+  else
+    printf 'cd ~/compass && docker compose --project-name compass -f compose.yaml'
   fi
-
-  printf 'cd ~/compass && docker compose --project-name compass -f compose.yaml'
 }
 
 remote_check_stack() {
@@ -500,6 +497,7 @@ remote_check_selfhosted_data() {
   ssh_remote "bash -se" <<REMOTE
 set -euo pipefail
 cd ~/compass
+export COMPOSE_PROFILES=selfhosted
 
 # Verify MongoDB is reachable and the replica set is healthy.
 docker compose --project-name compass -f compose.yaml exec -T mongo mongosh --quiet \
@@ -568,18 +566,24 @@ run_all_checks() {
   finish
 }
 
-case "${1:-run}" in
-  run)
-    run_all_checks
-    ;;
-  validate-frontend)
-    validate_frontend
-    ;;
-  validate-frontend-version)
-    validate_frontend_version
-    ;;
-  *)
-    printf 'usage: %s [run|validate-frontend|validate-frontend-version]\n' "$0" >&2
-    exit 2
-    ;;
-esac
+main() {
+  case "${1:-run}" in
+    run)
+      run_all_checks
+      ;;
+    validate-frontend)
+      validate_frontend
+      ;;
+    validate-frontend-version)
+      validate_frontend_version
+      ;;
+    *)
+      printf 'usage: %s [run|validate-frontend|validate-frontend-version]\n' "$0" >&2
+      exit 2
+      ;;
+  esac
+}
+
+if [ "${BASH_SOURCE[0]}" = "$0" ]; then
+  main "$@"
+fi

.github/workflows/_deploy-environment.yml

@@ -128,7 +128,7 @@ jobs:
           fi
           ssh -i ~/.ssh/staging_key "$SSH_USER@$SSH_HOST" "curl -fsSL https://raw.githubusercontent.com/SwitchbackTech/compass/${COMPOSE_GIT_REF}/self-host/compose.yaml -o ~/compass/compose.yaml"
           ssh -i ~/.ssh/staging_key "$SSH_USER@$SSH_HOST" "curl -fsSL https://raw.githubusercontent.com/SwitchbackTech/compass/${COMPOSE_GIT_REF}/self-host/compass -o ~/compass/compass && chmod +x ~/compass/compass"
-          ssh -i ~/.ssh/staging_key "$SSH_USER@$SSH_HOST" "cd ~/compass && COMPOSE_PROFILES=selfhost docker compose --project-name compass -f compose.yaml down 2>/dev/null || true"
+          ssh -i ~/.ssh/staging_key "$SSH_USER@$SSH_HOST" "cd ~/compass && COMPOSE_PROFILES=selfhosted docker compose --project-name compass -f compose.yaml down 2>/dev/null || true"
           if [ -n "$COMPOSE_PROFILES" ]; then
             ssh -i ~/.ssh/staging_key "$SSH_USER@$SSH_HOST" "cd ~/compass && COMPOSE_PROFILES='${COMPOSE_PROFILES}' ./compass update"
           else

docs/self-hosting/backup-and-restore.md

@@ -311,7 +311,7 @@ When the rehearsal is done, remove only the temporary project:
 ```bash
 cd "$CHECK_DIR"
 ./compass stop
-COMPOSE_PROFILES=selfhost docker compose \
+COMPOSE_PROFILES=selfhosted docker compose \
   --project-name "$CHECK_PROJECT" \
   -f compose.yaml \
   down -v

self-host/compass

@@ -132,7 +132,7 @@ load_env_if_present() {
 set_compose_env() {
   load_env
   export COMPASS_CONFIG_FILE="$CONFIG_FILE"
-  export COMPOSE_PROFILES="${COMPOSE_PROFILES-selfhost}"
+  export COMPOSE_PROFILES="${COMPOSE_PROFILES-selfhosted}"
   export COMPASS_VERSION="$(strip_quotes "$(read_config_value runtime.version)")"
   export COMPASS_WEB_IMAGE="$(strip_quotes "$(read_config_value web.image)")"
   export WEB_PORT="$(strip_quotes "$(read_config_value web.port)")"

self-host/compose.yaml

@@ -48,7 +48,7 @@ services:
 
   mongo:
     image: switchbacktech/compass-mongo:${COMPASS_VERSION:-latest}
-    profiles: [selfhost]
+    profiles: [selfhosted]
     environment:
       MONGO_INITDB_ROOT_USERNAME: ${MONGO_INITDB_ROOT_USERNAME:-compass}
       MONGO_INITDB_ROOT_PASSWORD: ${MONGO_INITDB_ROOT_PASSWORD:-change-me-mongo-password-32chars}
@@ -70,7 +70,7 @@ services:
 
   supertokens:
     image: supertokens/supertokens-postgresql:11.0.2
-    profiles: [selfhost]
+    profiles: [selfhosted]
     depends_on:
       supertokens-db:
         condition: service_healthy
@@ -91,7 +91,7 @@ services:
 
   supertokens-db:
     image: postgres:16-alpine
-    profiles: [selfhost]
+    profiles: [selfhosted]
     environment:
       POSTGRES_USER: ${SUPERTOKENS_POSTGRES_USER:-supertokens}
       POSTGRES_PASSWORD: ${SUPERTOKENS_POSTGRES_PASSWORD:-change-me-supertokens-postgres-pass-32}

self-host/docker-compose.test.ts

@@ -37,6 +37,32 @@ async function runHealthScript(
   return { exitCode, stderr, stdout };
 }
 
+async function runHealthScriptFunction(
+  command: string,
+  env: Record<string, string> = {},
+) {
+  const proc = Bun.spawn(
+    ["bash", "-c", `. .github/scripts/deploy-health-check.sh; ${command}`],
+    {
+      cwd: repoRoot,
+      env: {
+        ...process.env,
+        ...env,
+      },
+      stderr: "pipe",
+      stdout: "pipe",
+    },
+  );
+
+  const [stdout, stderr, exitCode] = await Promise.all([
+    new Response(proc.stdout).text(),
+    new Response(proc.stderr).text(),
+    proc.exited,
+  ]);
+
+  return { exitCode, stderr, stdout };
+}
+
 function makeTempDir() {
   const dir = mkdtempSync(join(tmpdir(), "compass-health-check-"));
   tempDirs.push(dir);
@@ -100,7 +126,7 @@ describe("self-host helper", () => {
     const helper = readRepoFile("self-host/compass");
 
     expect(helper).toContain(
-      'COMPOSE_PROFILES="' + "$" + '{COMPOSE_PROFILES-selfhost}"',
+      'COMPOSE_PROFILES="' + "$" + '{COMPOSE_PROFILES-selfhosted}"',
     );
   });
 
@@ -267,4 +293,14 @@ describe("deploy health check script", () => {
     expect(result.stderr).toContain("frontend-version");
     expect(result.stderr).toContain("expected 0.5.27, got 0.5.26");
   });
+
+  it("uses selfhosted profile for selfhosted deployments", async () => {
+    const result = await runHealthScriptFunction("remote_compose_prefix", {
+      PROFILE: "selfhosted",
+    });
+
+    expect(result.exitCode).toBe(0);
+    expect(result.stdout).toContain("COMPOSE_PROFILES=selfhosted");
+    expect(result.stdout).toContain("docker compose --project-name compass");
+  });
 });

self-host/install.sh

@@ -482,7 +482,7 @@ EOF
 set_compose_env() {
   [ -f "$CONFIG_FILE" ] || fail "Missing config file: $CONFIG_FILE."
   export COMPASS_CONFIG_FILE="$CONFIG_FILE"
-  export COMPOSE_PROFILES="${COMPOSE_PROFILES-selfhost}"
+  export COMPOSE_PROFILES="${COMPOSE_PROFILES-selfhosted}"
   export COMPASS_VERSION="$(strip_quotes "$(read_config_value runtime.version)")"
   export WEB_PORT="$(strip_quotes "$(read_config_value web.port)")"
   export PORT="$(strip_quotes "$(read_config_value backend.port)")"