Be defensive against flushAndExit getting called twice (#30992)

There's some evidence that this can get triggered twice (like multiple SIGINTs firing under certain configurations).

To ensure the cleanup functions only run once, I'm grabbing them + clearing them before we run them and hit the await points.

GitOrigin-RevId: b3b7953455dc32c2131133013d8d84ce5df4a013

Author: sshader

src/bundler/context.ts

@@ -84,7 +84,11 @@ class OneoffContextImpl {
   };
   flushAndExit = async (exitCode: number, err?: any) => {
     logVerbose(this, "Flushing and exiting");
-    const fns = Object.values(this._cleanupFns);
+    const cleanupFns = this._cleanupFns;
+    // Clear the cleanup functions so that there's no risk of running them twice
+    // if this somehow gets triggered twice.
+    this._cleanupFns = {};
+    const fns = Object.values(cleanupFns);
     logVerbose(this, `Running ${fns.length} cleanup functions`);
     for (const fn of fns) {
       await fn();