Merge pull request #1779 from hey-api/fix/debug-missing-mark

fix: dont throw on missing performance marks

Author: mrlubos

.changeset/chilled-tips-lie.md

@@ -0,0 +1,5 @@
+---
+'@hey-api/openapi-ts': patch
+---
+
+fix: don't throw on missing performance marks

packages/openapi-ts/bin/index.cjs

@@ -117,6 +117,10 @@ async function start() {
       userConfig.watch = Number.parseInt(params.watch, 10);
     }
 
+    if (!Object.keys(userConfig.logs).length) {
+      delete userConfig.logs;
+    }
+
     const context = await createClient(userConfig);
     if (!context[0] || !context[0].config.watch) {
       process.exit(0);

packages/openapi-ts/src/utils/performance.ts

@@ -32,15 +32,19 @@ export class PerformanceReport {
     );
 
     marks.forEach((mark) => {
-      const markMeasure = Performance.measure(mark);
-      const markDuration = Math.ceil(markMeasure.duration * 100) / 100;
-      const percentage =
-        Math.ceil(
-          (markMeasure.duration / this.totalMeasure.duration) * 100 * 100,
-        ) / 100;
-      console.warn(
-        `${mark}: ${markDuration.toFixed(2)}ms (${percentage.toFixed(2)}%)`,
-      );
+      try {
+        const markMeasure = Performance.measure(mark);
+        const markDuration = Math.ceil(markMeasure.duration * 100) / 100;
+        const percentage =
+          Math.ceil(
+            (markMeasure.duration / this.totalMeasure.duration) * 100 * 100,
+          ) / 100;
+        console.warn(
+          `${mark}: ${markDuration.toFixed(2)}ms (${percentage.toFixed(2)}%)`,
+        );
+      } catch {
+        // noop
+      }
     });
   }
 }