Skip to content

Commit

Permalink
refactor: consistant types + more generic failed routes
Browse files Browse the repository at this point in the history
  • Loading branch information
pi0 committed Aug 6, 2023
1 parent 3d43e50 commit 1e638bc
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
10 changes: 5 additions & 5 deletions src/prerender.ts
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ export async function prerender(nitro: Nitro) {

// Start prerendering
const generatedRoutes = new Set();
const erroredRoutes = new Set<PrerenderRoute>();
const failedRoutes = new Set<PrerenderRoute>();
const skippedRoutes = new Set();
const displayedLengthWarns = new Set();
const canPrerender = (route = "/") => {
Expand Down Expand Up @@ -191,7 +191,7 @@ export async function prerender(nitro: Nitro) {
_route.error = new Error(`[${res.status}] ${res.statusText}`) as any;
_route.error.statusCode = res.status;
_route.error.statusMessage = res.statusText;
erroredRoutes.add(_route);
failedRoutes.add(_route);
}

// Write to the file
Expand Down Expand Up @@ -258,13 +258,13 @@ export async function prerender(nitro: Nitro) {
});

await nitro.hooks.callHook("prerender:done", {
erroredRoutes,
prerenderedRoutes: nitro._prerenderedRoutes,
failedRoutes: [...failedRoutes],
});

if (nitro.options.prerender.failOnError && erroredRoutes.size > 0) {
if (nitro.options.prerender.failOnError && failedRoutes.size > 0) {
nitro.logger.log("\nErrors prerendering:");
for (const route of erroredRoutes) {
for (const route of failedRoutes) {

Check warning on line 267 in src/prerender.ts

View check run for this annotation

Codecov / codecov/patch

src/prerender.ts#L267

Added line #L267 was not covered by tests
const parents = linkParents.get(route.route);
const parentsText = parents?.size
? `\n${[...parents.values()]
Expand Down
2 changes: 1 addition & 1 deletion src/types/nitro.ts
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ export interface NitroHooks {
"prerender:route": (route: PrerenderRoute) => HookResult;
"prerender:done": (result: {
prerenderedRoutes: PrerenderRoute[];
erroredRoutes: Set<PrerenderRoute>;
failedRoutes: PrerenderRoute[];
}) => HookResult;
}

Expand Down

0 comments on commit 1e638bc

Please sign in to comment.