Added an optional generic type to API routes to enforce the return type

Author: csansoon

apps/web/src/middlewares/adminHandler.ts

@@ -1,8 +1,9 @@
 import { getCurrentUserOrRedirect } from '$/services/auth/getCurrentUser'
 import { notFound } from 'next/navigation'
 import { NextRequest } from 'next/server'
+import { RouteHandler } from './types'
 
-export function adminHandler(handler: any) {
+export function adminHandler<T>(handler: RouteHandler<T>) {
   return async (
     req: NextRequest,
     { params, ...rest }: { params?: Promise<Record<string, string>> } = {},

apps/web/src/middlewares/authHandler.ts

@@ -1,7 +1,8 @@
 import { getDataFromSession } from '$/data-access'
 import { NextRequest, NextResponse } from 'next/server'
+import { RouteHandler } from './types'
 
-export function authHandler(handler: any) {
+export function authHandler<T>(handler: RouteHandler<T>) {
   return async (
     req: NextRequest,
     {

apps/web/src/middlewares/errorHandler.ts

@@ -3,6 +3,7 @@ import { env } from '@latitude-data/env'
 import { captureException } from '$/helpers/captureException'
 import { NextRequest, NextResponse } from 'next/server'
 import debug from '@latitude-data/core/lib/debug'
+import { RouteHandler } from './types'
 
 interface AbortError extends DOMException {
   name: 'AbortError'
@@ -18,7 +19,7 @@ export function isAbortError(error: unknown): error is AbortError {
   )
 }
 
-export function errorHandler(handler: any) {
+export function errorHandler<T>(handler: RouteHandler<T>) {
   const isDev = env.NODE_ENV === 'development'
   return async (req: NextRequest, res: any) => {
     try {

apps/web/src/middlewares/types.ts

@@ -0,0 +1,10 @@
+import { NextRequest, NextResponse } from 'next/server'
+
+type ApiErrorResponse = {
+  message: string
+}
+
+export type RouteHandler<T> = (
+  req: NextRequest,
+  res: any,
+) => Promise<NextResponse<T> | NextResponse<ApiErrorResponse>>