fix: narrow type of BaseHandler (#308)

Pimm · Skn0tt · web-flow · commit d5bb602c7906 · 2022-02-15T09:06:18.000+01:00
Removed the synchronous version from the union that is the return type of BaseHandler.

Co-authored-by: Simon Knott &lt;info@simonknott.de&gt;
diff --git a/src/function/handler.ts b/src/function/handler.ts
@@ -8,7 +8,7 @@ export interface HandlerCallback<ResponseType extends Response = Response> {
 }
 
 export interface BaseHandler<ResponseType extends Response = Response, C extends Context = Context> {
-  (event: Event, context: C, callback?: HandlerCallback<ResponseType>): void | ResponseType | Promise<ResponseType>
+  (event: Event, context: C, callback?: HandlerCallback<ResponseType>): void | Promise<ResponseType>
 }
 
 export type Handler = BaseHandler<Response, Context>

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@ export interface HandlerCallback<ResponseType extends Response = Response> {`
`8`	`8`	`}`
`9`	`9`
`10`	`10`	`export interface BaseHandler<ResponseType extends Response = Response, C extends Context = Context> {`
`11`		`- (event: Event, context: C, callback?: HandlerCallback<ResponseType>): void \| ResponseType \| Promise<ResponseType>`
	`11`	`+ (event: Event, context: C, callback?: HandlerCallback<ResponseType>): void \| Promise<ResponseType>`
`12`	`12`	`}`
`13`	`13`
`14`	`14`	`export type Handler = BaseHandler<Response, Context>`