fix: dont throw on missing performance marks

hey-api · Mar 4, 2025 · 57fcec8 · 57fcec8
1 parent 9eccf4b
commit 57fcec8
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 9 deletions.
diff --git a/.changeset/chilled-tips-lie.md b/.changeset/chilled-tips-lie.md
@@ -0,0 +1,5 @@
+---
+'@hey-api/openapi-ts': patch
+---
+
+fix: don't throw on missing performance marks
diff --git a/packages/openapi-ts/bin/index.cjs b/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);

diff --git a/packages/openapi-ts/src/utils/performance.ts b/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
+      }
     });
   }
 }