fix(web): health endpoint and remove deploy maintenance mode

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: hakalb

apps/web/fly.toml

@@ -14,9 +14,9 @@ primary_region = 'arn'
 
 [[http_service.checks]]
   method = 'GET'
-  path = '/'
+  path = '/api/health'
   grace_period = '10s'
-  interval = '20s'
+  interval = '30s'
   timeout = '5s'
 
 [[vm]]

apps/web/server.ts

@@ -24,6 +24,8 @@ const build = (await import(
 )) as unknown as RemixMiddlewareOptions['build'];
 
 const app = new Hono()
+  // Lightweight health check — must not trigger Remix/Payload data fetching
+  .get('/api/health', (c) => c.text('ok'))
   // Serve static files from Remix client build
   .use('*', serveStatic({ root: './build/client' }))
   // Let Remix handle all requests

packages/fly-deployment-action/src/lib/utils/run-deploy-apps.ts

@@ -148,13 +148,6 @@ export const runDeployApps = async (options: {
 
       core.info(`Deploy app '${appName}' to '${environment}'${tenantLabel}...`);
 
-      // Enable maintenance mode on existing apps before deploying
-      const appExists = (await fly.status({ app: appName })) !== null;
-      if (appExists) {
-        core.info(`Enabling maintenance mode for '${appName}'`);
-        await fly.secrets.set({ MAINTENANCE_MODE: 'true' }, { app: appName });
-      }
-
       // Deploy app
       try {
         const result = await fly.deploy({
@@ -192,66 +185,6 @@ export const runDeployApps = async (options: {
         if (isHostDeployment) {
           failedHostApps.add(configAppName);
         }
-      } finally {
-        if (appExists) {
-          core.info(`Disabling maintenance mode for '${appName}'`);
-          // Fly.io machines may still be stabilizing after deploy — retry if the unset
-          // fails with a concurrent-update error.
-          let unsetSucceeded = false;
-          const maxAttempts = 5;
-          const retryDelayMs = 15_000;
-          for (let attempt = 1; attempt <= maxAttempts; attempt++) {
-            try {
-              await fly.secrets.unset('MAINTENANCE_MODE', { app: appName });
-              unsetSucceeded = true;
-              break;
-            } catch (cleanupError) {
-              const msg =
-                cleanupError instanceof Error
-                  ? cleanupError.message
-                  : String(cleanupError);
-              if (attempt === maxAttempts) {
-                core.warning(
-                  `Failed to disable maintenance mode for '${appName}' after ${maxAttempts} attempts: ${msg}`
-                );
-              } else {
-                core.info(
-                  `Retry ${attempt}/${maxAttempts - 1} disabling maintenance mode for '${appName}' in ${retryDelayMs / 1000}s: ${msg}`
-                );
-                await new Promise((r) => setTimeout(r, retryDelayMs));
-              }
-            }
-          }
-
-          // MAINTENANCE_MODE is baked into the machine's per-machine config.env
-          // from when it was a Fly secret during deployment. fly secrets unset
-          // removes it from the secrets store but NOT from the machine-level config.
-          // fly machine update creates a new machine version with the env var cleared,
-          // forcing a cold restart — not a resume from a suspended memory snapshot.
-          if (unsetSucceeded) {
-            const statusAfterUnset = await fly.status({ app: appName });
-            if (statusAfterUnset?.machines) {
-              for (const machine of statusAfterUnset.machines) {
-                try {
-                  core.info(
-                    `Clearing MAINTENANCE_MODE env for machine '${machine.id}' of '${appName}'`
-                  );
-                  await fly.machines.update(appName, machine.id, {
-                    env: { MAINTENANCE_MODE: '' }
-                  });
-                } catch (updateErr) {
-                  const msg =
-                    updateErr instanceof Error
-                      ? updateErr.message
-                      : String(updateErr);
-                  core.warning(
-                    `Failed to clear MAINTENANCE_MODE for machine '${machine.id}' of '${appName}': ${msg}`
-                  );
-                }
-              }
-            }
-          }
-        }
       }
     }
 

packages/fly-node/src/lib/fly.class.ts

@@ -482,9 +482,7 @@ export class Fly {
       try {
         await this.ensureInitialized();
         await this.updateMachine(app, machineId, options);
-        this.logger.info(
-          `Machine '${machineId}' in app '${app}' was updated`
-        );
+        this.logger.info(`Machine '${machineId}' in app '${app}' was updated`);
       } catch (error) {
         throw new Error(`[update machine] something broke\n${error}`);
       }