fix: close the config watcher so that Node can exit safely

netlify · Feb 14, 2022 · 63c8a12 · 63c8a12
1 parent fcb5ec5
commit 63c8a12
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/src/commands/base-command.js b/src/commands/base-command.js
@@ -121,6 +121,9 @@ class BaseCommand extends Command {
           await this.init(actionCommand)
           debug(`${name}:preAction`)('end')
         })
+        .hook('postAction', async () => {
+          await this.configWatcherHandle.close()
+        })
     )
   }
 
@@ -431,9 +434,9 @@ class BaseCommand extends Command {
     const apiOpts = { ...apiUrlOpts, agent }
     const globalConfig = await getGlobalConfig()
     const { NetlifyAPI } = await jsClient
-    const configWatcher = new events.EventEmitter()
 
-    watchDebounced(configPath, {
+    const configWatcher = new events.EventEmitter()
+    const configWatcherHandle = await watchDebounced(configPath, {
       depth: 1,
       onChange: async () => {
         const { config: newConfig } = await actionCommand.getConfig({ cwd, state, token, ...apiUrlOpts })
@@ -443,6 +446,9 @@ class BaseCommand extends Command {
       },
     })
 
+    // chokidar handler
+    this.configWatcherHandle = configWatcherHandle
+
     actionCommand.netlify = {
       // api methods
       api: new NetlifyAPI(token || '', apiOpts),