refactor: rename things

Author: eduardoboucas

src/function/handler.ts

@@ -1,23 +1,26 @@
-import type { Context } from './context.js'
-import type { Event } from './event.js'
-import type { Response, BuilderResponse, StreamingResponse } from './response.js'
+import type { HandlerContext } from './handler_context.js'
+import type { HandlerEvent } from './handler_event.js'
+import type { HandlerResponse, BuilderResponse, StreamingResponse } from './handler_response.js'
 
-export interface HandlerCallback<ResponseType extends Response = Response> {
+export interface HandlerCallback<ResponseType extends HandlerResponse = HandlerResponse> {
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   (error: any, response: ResponseType): void
 }
 
-export interface BaseHandler<ResponseType extends Response = Response, C extends Context = Context> {
-  (event: Event, context: C, callback?: HandlerCallback<ResponseType>): void | Promise<ResponseType>
+export interface BaseHandler<
+  ResponseType extends HandlerResponse = HandlerResponse,
+  C extends HandlerContext = HandlerContext,
+> {
+  (event: HandlerEvent, context: C, callback?: HandlerCallback<ResponseType>): void | Promise<ResponseType>
 }
 
-export interface BackgroundHandler<C extends Context = Context> {
-  (event: Event, context: C): void | Promise<void>
+export interface BackgroundHandler<C extends HandlerContext = HandlerContext> {
+  (event: HandlerEvent, context: C): void | Promise<void>
 }
 
-export type Handler = BaseHandler<Response, Context>
-export type BuilderHandler = BaseHandler<BuilderResponse, Context>
+export type Handler = BaseHandler<HandlerResponse, HandlerContext>
+export type BuilderHandler = BaseHandler<BuilderResponse, HandlerContext>
 
 export interface StreamingHandler {
-  (event: Event, context: Context): Promise<StreamingResponse>
+  (event: HandlerEvent, context: HandlerContext): Promise<StreamingResponse>
 }

src/function/context.ts renamed to src/function/handler_context.ts

@@ -2,7 +2,7 @@
 /* eslint-disable @typescript-eslint/no-explicit-any */
 
 // From https://docs.aws.amazon.com/lambda/latest/dg/nodejs-prog-model-context.html.
-export interface Context {
+export interface HandlerContext {
   callbackWaitsForEmptyEventLoop: boolean
   functionName: string
   functionVersion: string

src/function/event.ts renamed to src/function/handler_event.ts

@@ -14,7 +14,7 @@ interface EventMultiValueQueryStringParameters {
   [name: string]: string[] | undefined
 }
 
-export interface Event {
+export interface HandlerEvent {
   rawUrl: string
   rawQuery: string
   path: string

src/function/response.ts renamed to src/function/handler_response.ts

@@ -1,6 +1,6 @@
 import type { PipelineSource } from 'node:stream'
 
-export interface Response {
+export interface HandlerResponse {
   statusCode: number
   headers?: {
     [header: string]: boolean | number | string
@@ -11,11 +11,11 @@ export interface Response {
   body?: string
   isBase64Encoded?: boolean
 }
-export interface BuilderResponse extends Response {
+export interface BuilderResponse extends HandlerResponse {
   ttl?: number
 }
 
-export interface StreamingResponse extends Omit<Response, 'body'> {
+export interface StreamingResponse extends Omit<HandlerResponse, 'body'> {
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   body?: string | PipelineSource<any>
 }

src/function/index.ts

@@ -1,5 +1,5 @@
-export { Context as HandlerContext } from './context.js'
-export { Event as HandlerEvent } from './event.js'
+export { HandlerContext } from './handler_context.js'
+export { HandlerEvent } from './handler_event.js'
 export { BuilderHandler, Handler, BackgroundHandler, HandlerCallback, StreamingHandler } from './handler.js'
-export { BuilderResponse, Response as HandlerResponse, StreamingResponse } from './response.js'
+export { BuilderResponse, HandlerResponse, StreamingResponse } from './handler_response.js'
 export { Context } from './v2.js'

src/lib/builder.ts

@@ -1,8 +1,8 @@
 import isPromise from 'is-promise'
 
 import { BuilderHandler, Handler, HandlerCallback } from '../function/handler.js'
+import { HandlerResponse, BuilderResponse } from '../function/handler_response.js'
 import { HandlerContext, HandlerEvent } from '../function/index.js'
-import { Response, BuilderResponse } from '../function/response.js'
 
 import { BUILDER_FUNCTIONS_FLAG, HTTP_STATUS_METHOD_NOT_ALLOWED, METADATA_VERSION } from './consts.js'
 
@@ -21,7 +21,7 @@ const augmentResponse = (response: BuilderResponse) => {
 const wrapHandler =
   (handler: BuilderHandler): Handler =>
   // eslint-disable-next-line promise/prefer-await-to-callbacks
-  (event: HandlerEvent, context: HandlerContext, callback?: HandlerCallback<Response>) => {
+  (event: HandlerEvent, context: HandlerContext, callback?: HandlerCallback<HandlerResponse>) => {
     if (event.httpMethod !== 'GET' && event.httpMethod !== 'HEAD') {
       return Promise.resolve({
         body: 'Method Not Allowed',