fix: Add onDestroyHandlers to meta so that they can be invoked

core/garment/src/garment.ts

@@ -67,6 +67,15 @@ export interface CacheEntry {
   output: File[];
 }
 
+interface ActionMeta {
+  isWatchActive: boolean;
+  subscriptions: Set<Subscription>;
+  logs: LogEntry[];
+  inputHandler?: MetaInputHandler;
+  onDestroyHandler?: () => void;
+  outputPath?: string;
+}
+
 export type MetaInputHandler =
   | { type: 'single'; fn: InputFnCallBack }
   | { type: 'multi'; fn: InputFnCallBack<File[]> };
@@ -189,17 +198,7 @@ async function garmentFromWorkspace(
   const getSnapshotId = (hash: string) => `action-${hash}`;
 
   // The map to store an information related to each action, so it can be reused between action executions
-  const metaByAction = new Map<
-    Action,
-    {
-      isWatchActive: boolean;
-      subscriptions: Set<Subscription>;
-      logs: LogEntry[];
-      inputHandler?: MetaInputHandler;
-      onDestroyHandler?: () => void;
-      outputPath?: string;
-    }
-  >();
+  const metaByAction = new Map<Action, ActionMeta>();
 
   let allSubscriptionsCached: Subscription[] = [];
 
@@ -1013,13 +1012,16 @@ async function garmentFromWorkspace(
       {
         runner: RunnerMeta;
         batch: { project: Project; options: any }[];
+        batchMeta: ActionMeta;
       }
     >();
     for (const action of actions) {
       if (!batchesByRunner.has(action.runner.handlerPath)) {
         batchesByRunner.set(action.runner.handlerPath, {
           runner: action.runner,
-          batch: []
+          batch: [],
+          // We don't want meta per action, since longRunning should be invoked once for all projects
+          batchMeta: metaByAction.get(action)
         });
       }
       const { batch } = batchesByRunner.get(action.runner.handlerPath)!;
@@ -1034,7 +1036,7 @@ async function garmentFromWorkspace(
         })
       });
     }
-    for (const { runner, batch } of batchesByRunner.values()) {
+    for (const { runner, batch, batchMeta } of batchesByRunner.values()) {
       onUpdate({ type: 'before-batch', runner, batch });
 
       const options = {
@@ -1073,7 +1075,12 @@ async function garmentFromWorkspace(
         dependsOnFile() {},
         cacheProvider,
         batch,
-        logger
+        logger,
+        longRunning(onDestroy) {
+          if (batchMeta) {
+            batchMeta.onDestroyHandler = onDestroy;
+          }
+        }
       });
 
       const handler = watch ? getWatcher(runner) : getHandler(runner);