reverted changes in the nextfunction type

Author: sdangol

packages/event-handler/src/appsync-graphql/AppSyncGraphQLResolver.ts

@@ -190,7 +190,7 @@ class AppSyncGraphQLResolver extends Router {
    * @param options - Optional resolve options for customizing resolver behavior.
    */
   async #withErrorHandling(
-    fn: () => Promise<unknown> | unknown,
+    fn: () => unknown,
     event: AppSyncResolverEvent<Record<string, unknown>>,
     options?: ResolveOptions
   ) {
@@ -377,7 +377,7 @@ class AppSyncGraphQLResolver extends Router {
     event: AppSyncResolverEvent<Record<string, unknown>>,
     context: Context,
     options?: ResolveOptions
-  ): Promise<unknown> | unknown {
+  ): unknown {
     const { fieldName, parentTypeName: typeName } = event.info;
 
     const resolverHandlerOptions = this.resolverRegistry.resolve(

packages/event-handler/src/types/rest.ts

@@ -54,7 +54,7 @@ interface CompiledRoute {
 
 type DynamicRoute = Route & CompiledRoute;
 
-type HandlerResponse = Response | JSONObject | undefined;
+type HandlerResponse = Response | JSONObject;
 
 type RouteHandler<TReturn = HandlerResponse> = (
   reqCtx: RequestContext
@@ -79,16 +79,13 @@ type RestRouteOptions = {
   middleware?: Middleware[];
 };
 
-type NextFunction = () =>
-  | Promise<HandlerResponse>
-  | HandlerResponse
-  | Promise<void>
-  | void;
+type NextFunction = () => Promise<HandlerResponse | void>; // biome-ignore lint/suspicious/noConfusingVoidType: To ensure next function is awaited
 
 type Middleware = (args: {
   reqCtx: RequestContext;
   next: NextFunction;
-}) => NextFunction;
+  // biome-ignore lint/suspicious/noConfusingVoidType: To ensure next function is awaited
+}) => Promise<HandlerResponse | void>;
 
 type RouteRegistryOptions = {
   /**