feat: Implement smart default onCrash

zenflow · Mar 18, 2021 · 5dd2d20 · 5dd2d20
1 parent a3c8857
commit 5dd2d20
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/interfaces/ServiceConfig.ts b/src/interfaces/ServiceConfig.ts
@@ -69,7 +69,7 @@ export interface ServiceConfig {
 
   /**
    * A function to be executed each time the service crashes.
-   * Defaults to `() => {}`.
+   * Defaults to `ctx => { if (!ctx.isServiceReady) throw new Error('Crashed before becoming ready') }`.
    *
    * This function is called with an {@link OnCrashContext} object as its only argument.
    *

diff --git a/src/validateAndNormalizeConfig.ts b/src/validateAndNormalizeConfig.ts
@@ -80,11 +80,13 @@ function processServiceConfig(
   const merged = {
     dependencies: [],
     cwd: '.',
-    // no default command
+    command: undefined, // no default command
     env: process.env,
     ready: () => Promise.resolve(),
     forceKillTimeout: 5000,
-    onCrash: () => {},
+    onCrash: (ctx: OnCrashContext) => {
+      if (!ctx.isServiceReady) throw new Error('Crashed before becoming ready')
+    },
     logTailLength: 0,
     minimumRestartDelay: 1000,
     ...removeUndefinedProperties(defaults),