build: bump prettier to 3 and typescript to 5.6

The format target was broken on `packages/core/rollup.config.ts` because
prettier 2 cannot parse the modern `with { type: "json" }` import
attribute syntax. Bump prettier to ^3.8.3 (and prettier-plugin-organize-
imports to ^4.3.0 for compatibility), and align the other rollup
configs to `with` so the repo uses one syntax.

The organize-imports plugin parses through the workspace's installed
TypeScript, so bump TypeScript to ^5.6.3 too — 4.9.4 also rejects the
`with` syntax. Drop the explicit `Promise<typeof import("tinybench")>`
return type on WalltimeRunner.importTinybench(); TS 5.x's stricter type
equality flags it as incompatible with the inferred type on vitest's
NodeBenchmarkRunner base class.

Reformat ~30 source files where prettier 3's new `trailingComma: "all"`
default added trailing commas after multi-line arguments.

Refs COD-2721
Co-Authored-By: Claude <noreply@anthropic.com>

Author: GuillaumeLagrange

package.json

@@ -31,15 +31,15 @@
     "jest": "^29.5.0",
     "jest-config": "^29.5.0",
     "lerna": "^6.6.1",
-    "prettier": "^2.8.7",
-    "prettier-plugin-organize-imports": "^3.2.2",
+    "prettier": "^3.8.3",
+    "prettier-plugin-organize-imports": "^4.3.0",
     "rollup": "^4.47.1",
     "rollup-plugin-dts": "^6.1.0",
     "rollup-plugin-esbuild": "^6.1.0",
     "ts-jest": "^29.1.0",
     "tslib": "^2.5.0",
     "turbo": "^2.5.6",
-    "typescript": "4.9.4"
+    "typescript": "^5.6.3"
   },
   "packageManager": "pnpm@10.12.4",
   "engines": {

packages/benchmark.js-plugin/benches/parsePr.ts

@@ -13,7 +13,7 @@ function sendEvent(numberOfOperations: number): void {
 
 function logMetrics(
   numberOfOperations: number,
-  numberOfDeepOperations: number
+  numberOfDeepOperations: number,
 ): void {
   for (let i = 0; i < numberOfOperations; i++) {
     for (let i = 0; i < numberOfOperations; i++) {

packages/benchmark.js-plugin/benches/sample.ts

@@ -5,7 +5,7 @@ import parsePr from "./parsePr";
 const LONG_BODY =
   new Array(1_000)
     .fill(
-      "Lorem ipsum dolor sit amet consectetur adipisicing elit. Sunt, earum. Atque architecto vero veniam est tempora fugiat sint quo praesentium quia. Autem, veritatis omnis beatae iste delectus recusandae animi non."
+      "Lorem ipsum dolor sit amet consectetur adipisicing elit. Sunt, earum. Atque architecto vero veniam est tempora fugiat sint quo praesentium quia. Autem, veritatis omnis beatae iste delectus recusandae animi non.",
     )
     .join("\n") + "fixes #123";
 

packages/benchmark.js-plugin/rollup.config.ts

@@ -1,6 +1,6 @@
 import { defineConfig } from "rollup";
 import { declarationsPlugin, jsPlugins } from "../../rollup.options";
-import pkg from "./package.json" assert { type: "json" };
+import pkg from "./package.json" with { type: "json" };
 
 const entrypoint = "src/index.ts";
 

packages/benchmark.js-plugin/src/__tests__/buildSuiteAdd.test.ts

@@ -18,8 +18,8 @@ describe("buildSuiteAdd", () => {
     suite.add({ name: "test", fn: emptyBench });
     suite.forEach((bench: CodSpeedBenchmark) =>
       expect(bench.uri).toBe(
-        "packages/benchmark.js-plugin/src/__tests__/buildSuiteAdd.test.ts::test"
-      )
+        "packages/benchmark.js-plugin/src/__tests__/buildSuiteAdd.test.ts::test",
+      ),
     );
   });
 
@@ -28,8 +28,8 @@ describe("buildSuiteAdd", () => {
     suite.add(emptyBench, { name: "test" });
     suite.forEach((bench: CodSpeedBenchmark) =>
       expect(bench.uri).toBe(
-        "packages/benchmark.js-plugin/src/__tests__/buildSuiteAdd.test.ts::test"
-      )
+        "packages/benchmark.js-plugin/src/__tests__/buildSuiteAdd.test.ts::test",
+      ),
     );
   });
 
@@ -38,8 +38,8 @@ describe("buildSuiteAdd", () => {
     suite.add("test", { fn: emptyBench });
     suite.forEach((bench: CodSpeedBenchmark) =>
       expect(bench.uri).toBe(
-        "packages/benchmark.js-plugin/src/__tests__/buildSuiteAdd.test.ts::test"
-      )
+        "packages/benchmark.js-plugin/src/__tests__/buildSuiteAdd.test.ts::test",
+      ),
     );
   });
 
@@ -48,8 +48,8 @@ describe("buildSuiteAdd", () => {
     suite.add("test", emptyBench);
     suite.forEach((bench: CodSpeedBenchmark) =>
       expect(bench.uri).toBe(
-        "packages/benchmark.js-plugin/src/__tests__/buildSuiteAdd.test.ts::test"
-      )
+        "packages/benchmark.js-plugin/src/__tests__/buildSuiteAdd.test.ts::test",
+      ),
     );
   });
 
@@ -59,8 +59,8 @@ describe("buildSuiteAdd", () => {
     suite.add("test", emptyBench);
     suite.forEach((bench: CodSpeedBenchmark) =>
       expect(bench.uri).toBe(
-        "packages/benchmark.js-plugin/src/__tests__/buildSuiteAdd.test.ts::suite::test"
-      )
+        "packages/benchmark.js-plugin/src/__tests__/buildSuiteAdd.test.ts::suite::test",
+      ),
     );
   });
 

packages/benchmark.js-plugin/src/index.ts

@@ -18,17 +18,16 @@ declare const __VERSION__: string;
 
 tryIntrospect();
 
-interface WithCodSpeedBenchmark
-  extends Omit<
-    Benchmark,
-    "run" | "abort" | "clone" | "compare" | "emit" | "off" | "on" | "reset"
-  > {
+interface WithCodSpeedBenchmark extends Omit<
+  Benchmark,
+  "run" | "abort" | "clone" | "compare" | "emit" | "off" | "on" | "reset"
+> {
   abort(): WithCodSpeedBenchmark;
   clone(options: Benchmark.Options): WithCodSpeedBenchmark;
   compare(benchmark: Benchmark): number;
   off(
     type?: string,
-    listener?: CallableFunction
+    listener?: CallableFunction,
   ): Benchmark | Promise<Benchmark>;
   off(types: string[]): WithCodSpeedBenchmark;
   on(type?: string, listener?: CallableFunction): WithCodSpeedBenchmark;
@@ -38,31 +37,30 @@ interface WithCodSpeedBenchmark
   run(options?: Benchmark.Options): Benchmark | Promise<Benchmark>;
 }
 
-export interface WithCodSpeedSuite
-  extends Omit<
-    Benchmark.Suite,
-    | "run"
-    | "abort"
-    | "clone"
-    | "compare"
-    | "emit"
-    | "off"
-    | "on"
-    | "reset"
-    | "add"
-    | "filter"
-    | "each"
-    | "forEach"
-  > {
+export interface WithCodSpeedSuite extends Omit<
+  Benchmark.Suite,
+  | "run"
+  | "abort"
+  | "clone"
+  | "compare"
+  | "emit"
+  | "off"
+  | "on"
+  | "reset"
+  | "add"
+  | "filter"
+  | "each"
+  | "forEach"
+> {
   abort(): WithCodSpeedSuite;
   add(
     name: string,
     fn: CallableFunction | string,
-    options?: Benchmark.Options
+    options?: Benchmark.Options,
   ): WithCodSpeedSuite;
   add(
     fn: CallableFunction | string,
-    options?: Benchmark.Options
+    options?: Benchmark.Options,
   ): WithCodSpeedSuite;
   add(name: string, options?: Benchmark.Options): WithCodSpeedSuite;
   add(options: Benchmark.Options): WithCodSpeedSuite;
@@ -94,7 +92,7 @@ function withCodSpeedBenchmark(bench: Benchmark): WithCodSpeedBenchmark {
     const rawRun = bench.run;
     bench.run = (options?: Benchmark.Options) => {
       console.warn(
-        `[CodSpeed] bench detected but no instrumentation found, falling back to benchmark.js`
+        `[CodSpeed] bench detected but no instrumentation found, falling back to benchmark.js`,
       );
       return rawRun.bind(bench)(options);
     };
@@ -124,7 +122,7 @@ function withCodSpeedSuite(suite: Benchmark.Suite): WithCodSpeedSuite {
     const rawRun = suite.run;
     suite.run = (options?: Benchmark.Options) => {
       console.warn(
-        `[CodSpeed] ${suite.length} benches detected but no instrumentation found, falling back to benchmark.js`
+        `[CodSpeed] ${suite.length} benches detected but no instrumentation found, falling back to benchmark.js`,
       );
       return rawRun.bind(suite)(options);
     };
@@ -135,7 +133,7 @@ function withCodSpeedSuite(suite: Benchmark.Suite): WithCodSpeedSuite {
   // eslint-disable-next-line @typescript-eslint/no-unused-vars, @typescript-eslint/ban-ts-comment
   // @ts-ignore
   suite.run = async function (
-    options?: Benchmark.Options
+    options?: Benchmark.Options,
   ): Promise<Benchmark.Suite> {
     const suiteName = suite.name;
     const benches = this as unknown as BenchmarkWithOptions[];
@@ -231,7 +229,7 @@ async function runBenchmarks({
  * Dynamically setup the CodSpeed instruments.
  */
 export async function setupInstruments(
-  body: SetupInstrumentsRequestBody
+  body: SetupInstrumentsRequestBody,
 ): Promise<SetupInstrumentsResponse> {
   if (!InstrumentHooks.isInstrumented()) {
     console.warn("[CodSpeed] No instrumentation found, using default mongoUrl");