Add next as a devDependency for types

Author: hsubox76

common/api-review/telemetry.api.md

@@ -18,29 +18,16 @@ export function flush(telemetry: Telemetry): Promise<void>;
 export function getTelemetry(app?: FirebaseApp): Telemetry;
 
 // @public (undocumented)
-export type InstrumentationOnRequestError = (error: unknown, errorRequest: Readonly<{
-    path: string;
-    method: string;
-    headers: NodeJS.Dict<string | string[]>;
-}>, errorContext: Readonly<RequestErrorContext>) => void | Promise<void>;
-
-// @public
-export const nextOnRequestError: InstrumentationOnRequestError;
-
-// @public
-export interface RequestErrorContext {
-    // (undocumented)
-    renderSource?: 'react-server-components' | 'react-server-components-payload' | 'server-rendering';
+export namespace Instrumentation {
+    // Warning: (ae-forgotten-export) The symbol "InstrumentationOnRequestError" needs to be exported by the entry point index.d.ts
+    //
     // (undocumented)
-    revalidateReason: 'on-demand' | 'stale' | undefined;
-    // (undocumented)
-    routePath: string;
-    // (undocumented)
-    routerKind: 'Pages Router' | 'App Router';
-    // (undocumented)
-    routeType: 'render' | 'route' | 'action' | 'middleware';
+    export type onRequestError = InstrumentationOnRequestError;
 }
 
+// @public
+export const nextOnRequestError: Instrumentation.onRequestError;
+
 // @public
 export interface Telemetry {
     app: FirebaseApp;

packages/telemetry/api-extractor.json

@@ -6,5 +6,6 @@
     "enabled": true,
     "untrimmedFilePath": "<projectFolder>/dist/<unscopedPackageName>.d.ts",
     "betaTrimmedFilePath": "<projectFolder>/dist/<unscopedPackageName>-public.d.ts"
-  }
+  },
+  "bundledPackages": ["next"]
 }

packages/telemetry/package.json

@@ -58,6 +58,7 @@
     "@firebase/app": "0.14.2",
     "@opentelemetry/sdk-trace-web": "2.1.0",
     "@rollup/plugin-json": "6.1.0",
+    "next": "15.5.2",
     "rollup": "2.79.2",
     "rollup-plugin-replace": "2.2.0",
     "rollup-plugin-typescript2": "0.36.0",

packages/telemetry/src/next.ts

@@ -17,27 +17,10 @@
 
 import { getApp } from '@firebase/app';
 import { captureError, getTelemetry } from './api';
+// eslint-disable-next-line import/no-extraneous-dependencies
+import { Instrumentation } from 'next';
 
-// The following types mirror those from Next at v15.5
-export interface RequestErrorContext {
-  routerKind: 'Pages Router' | 'App Router';
-  routePath: string;
-  routeType: 'render' | 'route' | 'action' | 'middleware';
-  renderSource?:
-    | 'react-server-components'
-    | 'react-server-components-payload'
-    | 'server-rendering';
-  revalidateReason: 'on-demand' | 'stale' | undefined;
-}
-export type InstrumentationOnRequestError = (
-  error: unknown,
-  errorRequest: Readonly<{
-    path: string;
-    method: string;
-    headers: NodeJS.Dict<string | string[]>;
-  }>,
-  errorContext: Readonly<RequestErrorContext>
-) => void | Promise<void>;
+export { Instrumentation };
 
 /**
  * Automatically report uncaught errors from server routes to Firebase Telemetry.
@@ -50,7 +33,7 @@ export type InstrumentationOnRequestError = (
  *
  * @public
  */
-export const nextOnRequestError: InstrumentationOnRequestError = async (
+export const nextOnRequestError: Instrumentation.onRequestError = async (
   error,
   errorRequest,
   errorContext