fix(webapp): close the snapshot store clients on shutdown

The snapshot store connections and the orphan sweeper now close on SIGTERM and
SIGINT, matching the other Redis-backed singletons.

Author: d-cs

apps/webapp/app/v3/snapshotStoreWiring.server.ts

@@ -1,11 +1,16 @@
 import { SnapshotOrphanSweeper } from "@internal/run-store";
 import { env } from "~/env.server";
 import { logger } from "~/services/logger.server";
+import { signalsEmitter } from "~/services/signals.server";
 import { engine } from "./runEngine.server";
 import { runStoreWithoutSnapshotDecorator } from "./runStore.server";
 import { buildSnapshotSweepRunner } from "./snapshotSweepRunner.server";
 import { setSnapshotRepairEnqueuer, setSnapshotSweepRunner } from "./snapshotStoreBindings.server";
-import { getSnapshotSweepClient, registerSnapshotStoreQuit } from "./snapshotStoreInstance.server";
+import {
+  getSnapshotSweepClient,
+  quitSnapshotStoreClients,
+  registerSnapshotStoreQuit,
+} from "./snapshotStoreInstance.server";
 
 /**
  * The third module: it imports both sides, so neither the run store nor the engine has to import
@@ -43,6 +48,16 @@ export function registerSnapshotStoreWiring(): boolean {
 
   registerSnapshotStoreQuit(() => sweeper.quit());
 
+  // Close the sweeper and all three connections on the way out, the same way the other Redis-backed
+  // singletons do. `quitSnapshotStoreClients` is async and the signals emitter swallows listener
+  // rejections, so discard the promise explicitly rather than handing it a floating one. The caller
+  // wraps this function in `singleton`, so the listeners are registered once per process.
+  const onShutdown = (): void => {
+    void quitSnapshotStoreClients();
+  };
+  signalsEmitter.on("SIGTERM", onShutdown);
+  signalsEmitter.on("SIGINT", onShutdown);
+
   logger.info("snapshot store wiring registered");
   return true;
 }