refactor!: remove legacy timing plugin

Author: pi0

docs/3.config/0.index.md

@@ -128,15 +128,6 @@ Uses built-in SWR functionality (using caching layer and storage) for Netlify an
 
 Storage configuration, read more in the [Storage Layer](/guide/storage) section.
 
-### `timing`
-
-- Default: `false`{lang=ts}
-
-Enable timing information:
-
-- Nitro startup time log
-- `Server-Timing` header on HTTP responses
-
 ### `renderer`
 
 Path to main render (file should export an event handler as default)

src/build/rollup/base.ts

@@ -10,7 +10,7 @@ import { isAbsolute, join, dirname, resolve } from "pathe";
 import { hash } from "ohash";
 import { defu } from "defu";
 import { resolveModulePath } from "exsolve";
-import { isTest, isWindows } from "std-env";
+import { isWindows } from "std-env";
 import { defineEnv } from "unenv";
 import { runtimeDir, runtimeDependencies } from "nitro/runtime/meta";
 import unimportPlugin from "unimport/unplugin";
@@ -23,7 +23,6 @@ import { publicAssets } from "./plugins/public-assets";
 import { raw } from "./plugins/raw";
 import { serverAssets } from "./plugins/server-assets";
 import { storage } from "./plugins/storage";
-import { timing } from "./plugins/timing";
 import { virtual } from "./plugins/virtual";
 import { errorHandler } from "./plugins/error-handler";
 import { externals } from "./plugins/externals";
@@ -34,15 +33,6 @@ export function baseRollupPlugins(
 ) {
   const plugins: Plugin[] = [];
 
-  // Server timing
-  if (nitro.options.timing) {
-    plugins.push(
-      timing({
-        silent: isTest,
-      })
-    );
-  }
-
   // Auto imports
   if (nitro.options.imports) {
     plugins.push(unimportPlugin.rollup(nitro.options.imports) as Plugin);

src/build/rollup/plugins/timing.ts

@@ -6,7 +6,6 @@ import { isDebug, isTest } from "std-env";
 export const NitroDefaults: NitroConfig = {
   // General
   debug: isDebug,
-  timing: isDebug,
   logLevel: isTest ? 1 : 3,
   runtimeConfig: { app: {}, nitro: {} },
 

src/config/defaults.ts

@@ -47,14 +47,11 @@ export async function createNitro(
     await nitro.storage.dispose();
   });
 
-  // Debug and timing
+  // Debug
   if (nitro.options.debug) {
     createDebugger(nitro.hooks, { tag: "nitro" });
     nitro.options.plugins.push(join(runtimeDir, "internal/debug"));
   }
-  if (nitro.options.timing) {
-    nitro.options.plugins.push(join(runtimeDir, "internal/timing"));
-  }
 
   // Logger
   if (nitro.options.logLevel !== undefined) {

src/nitro.ts

@@ -72,7 +72,6 @@ export interface NitroOptions extends PresetOptions {
   database: DatabaseConnectionConfigs;
   devDatabase: DatabaseConnectionConfigs;
   bundledStorage: string[];
-  timing: boolean;
   renderer?: string;
   serveStatic: boolean | "node" | "deno" | "inline";
   noPublicDir: boolean;

src/runtime/internal/timing.ts

@@ -132,7 +132,6 @@ export async function setupTest(
     output: {
       dir: ctx.outDir,
     },
-    timing: !ctx.isWorker,
   });
   const nitro = (ctx.nitro = await createNitro(config, {
     compatibilityDate: opts.compatibilityDate || formatDate(new Date()),
@@ -569,16 +568,6 @@ export function testNitro(
     });
   });
 
-  if (ctx.nitro!.options.timing) {
-    it("set server timing header", async () => {
-      const { status, headers } = await callHandler({
-        url: "/api/hello",
-      });
-      expect(status).toBe(200);
-      expect(headers["server-timing"]).toMatch(/-;dur=\d+;desc="Generate"/);
-    });
-  }
-
   it("static build flags", async () => {
     const { data } = await callHandler({ url: "/static-flags" });
     expect(data).toMatchObject({