[feat] Allow transformPage to return a promise (#4036)

* allow transformPage to return a promise * add changeset
sveltejs · Feb 21, 2022 · 1264c46 · 1264c46
1 parent 74d197d
commit 1264c46
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/.changeset/forty-seahorses-rest.md b/.changeset/forty-seahorses-rest.md
@@ -0,0 +1,5 @@
+---
+'@sveltejs/kit': patch
+---
+
+Allow the `transformPage` resolve option to return a promise
diff --git a/packages/kit/src/runtime/server/page/render.js b/packages/kit/src/runtime/server/page/render.js
@@ -299,7 +299,7 @@ export async function render_response({
 	const assets =
 		options.paths.assets || (segments.length > 0 ? segments.map(() => '..').join('/') : '.');
 
-	const html = resolve_opts.transformPage({
+	const html = await resolve_opts.transformPage({
 		html: options.template({ head, body, assets, nonce: /** @type {string} */ (csp.nonce) })
 	});
 

diff --git a/packages/kit/types/internal.d.ts b/packages/kit/types/internal.d.ts
@@ -206,7 +206,7 @@ export type RecursiveRequired<T> = {
 
 export interface RequiredResolveOptions {
 	ssr: boolean;
-	transformPage: ({ html }: { html: string }) => string;
+	transformPage: ({ html }: { html: string }) => MaybePromise<string>;
 }
 
 export interface Respond {