Skip to content

Commit

Permalink
move MaybePromise from RequestHandlerOutput to RequestHandler return …
Browse files Browse the repository at this point in the history
…value (#4519)

* move MaybePromise from RequestHandlerOutput to RequestHandler return value

* prefer interface

Co-authored-by: Ignatius Bagus <ignatius.mbs@gmail.com>
  • Loading branch information
Rich-Harris and ignatiusmb committed Apr 5, 2022
1 parent 2ce93d6 commit 1dfbe54
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 3 deletions.
5 changes: 5 additions & 0 deletions .changeset/strong-apples-walk.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@sveltejs/kit': patch
---

Move MaybePromise from RequestHandlerOutput to RequestHandler return value
6 changes: 3 additions & 3 deletions packages/kit/types/index.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -223,14 +223,14 @@ export interface RequestHandler<
Params extends Record<string, string> = Record<string, string>,
Output extends ResponseBody = ResponseBody
> {
(event: RequestEvent<Params>): RequestHandlerOutput<Output>;
(event: RequestEvent<Params>): MaybePromise<RequestHandlerOutput<Output>>;
}

export type RequestHandlerOutput<Output extends ResponseBody = ResponseBody> = MaybePromise<{
export interface RequestHandlerOutput<Output extends ResponseBody = ResponseBody> {
status?: number;
headers?: Headers | Partial<ResponseHeaders>;
body?: Output;
}>;
}

export type ResponseBody = JSONValue | Uint8Array | ReadableStream | import('stream').Readable;

Expand Down

0 comments on commit 1dfbe54

Please sign in to comment.