fixup: stop before sending timestamps

not-matthias · not-matthias · commit 7a06ec36b199 · 2026-06-11T19:41:01.000+02:00
diff --git a/packages/tinybench-plugin/src/shared.ts b/packages/tinybench-plugin/src/shared.ts
@@ -52,9 +52,9 @@ export abstract class BaseBenchRunner {
       return fn();
     } finally {
       const runEnd = InstrumentHooks.currentTimestamp();
-      this.sendBenchmarkMarkers(runStart, runEnd);
       InstrumentHooks.stopBenchmark();
       InstrumentHooks.setExecutedBenchmark(process.pid, uri);
+      this.sendBenchmarkMarkers(runStart, runEnd);
     }
   }
 
@@ -68,9 +68,9 @@ export abstract class BaseBenchRunner {
       return await fn();
     } finally {
       const runEnd = InstrumentHooks.currentTimestamp();
-      this.sendBenchmarkMarkers(runStart, runEnd);
       InstrumentHooks.stopBenchmark();
       InstrumentHooks.setExecutedBenchmark(process.pid, uri);
+      this.sendBenchmarkMarkers(runStart, runEnd);
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -52,9 +52,9 @@ export abstract class BaseBenchRunner {`
`52`	`52`	`return fn();`
`53`	`53`	`} finally {`
`54`	`54`	`const runEnd = InstrumentHooks.currentTimestamp();`
`55`		`- this.sendBenchmarkMarkers(runStart, runEnd);`
`56`	`55`	`InstrumentHooks.stopBenchmark();`
`57`	`56`	`InstrumentHooks.setExecutedBenchmark(process.pid, uri);`
	`57`	`+ this.sendBenchmarkMarkers(runStart, runEnd);`
`58`	`58`	`}`
`59`	`59`	`}`
`60`	`60`
`@@ -68,9 +68,9 @@ export abstract class BaseBenchRunner {`
`68`	`68`	`return await fn();`
`69`	`69`	`} finally {`
`70`	`70`	`const runEnd = InstrumentHooks.currentTimestamp();`
`71`		`- this.sendBenchmarkMarkers(runStart, runEnd);`
`72`	`71`	`InstrumentHooks.stopBenchmark();`
`73`	`72`	`InstrumentHooks.setExecutedBenchmark(process.pid, uri);`
	`73`	`+ this.sendBenchmarkMarkers(runStart, runEnd);`
`74`	`74`	`}`
`75`	`75`	`}`
`76`	`76`