feat: auto-mark app as ready on first inbound HTTP request (#144)

Author: sohankshirsagar

src/core/TuskDrift.ts

@@ -170,12 +170,16 @@ export class TuskDriftCore {
     }
 
     if (!status.enabled) {
-      logger.info(`Rust core path disabled at startup (env=${envDisplay}, reason=${status.reason}).`);
+      logger.info(
+        `Rust core path disabled at startup (env=${envDisplay}, reason=${status.reason}).`,
+      );
       return;
     }
 
     if (status.bindingLoaded) {
-      logger.info(`Rust core path enabled at startup (env=${envDisplay}, reason=${status.reason}).`);
+      logger.info(
+        `Rust core path enabled at startup (env=${envDisplay}, reason=${status.reason}).`,
+      );
       return;
     }
 

src/instrumentation/libraries/http/Instrumentation.ts

@@ -1365,7 +1365,10 @@ export class HttpInstrumentation extends TdInstrumentationBase {
     return (originalEmit: Function) => {
       return function (this: Server, eventName: string, ...args: any[]) {
         if (eventName === "request") {
-          // Sample as soon as we can to avoid additional overhead if this request is not sampled
+          if (!self.tuskDrift.isAppReady()) {
+            self.tuskDrift.markAppAsReady();
+          }
+
           if (self.mode === TuskDriftMode.RECORD) {
             if (
               !shouldSample({