Skip to content

Commit

Permalink
fix: small bug fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
pi0 committed Apr 6, 2022
1 parent 613a559 commit 26046ee
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 14 deletions.
18 changes: 7 additions & 11 deletions src/build.ts
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
import { promises as fsp } from 'fs'
import { relative, resolve, join } from 'pathe'
import * as rollup from 'rollup'
import fse from 'fs-extra'
Expand All @@ -13,18 +14,13 @@ import type { Nitro } from './types'
import { runtimeDir } from './dirs'

export async function prepare (nitro: Nitro) {
await cleanupDir(nitro.options.output.dir)

if (!nitro.options.output.publicDir.startsWith(nitro.options.output.dir)) {
await cleanupDir(nitro.options.output.publicDir)
}

if (!nitro.options.output.serverDir.startsWith(nitro.options.output.dir)) {
await cleanupDir(nitro.options.output.serverDir)
}
await prepareDir(nitro.options.output.dir)
await prepareDir(nitro.options.output.publicDir)
await prepareDir(nitro.options.output.serverDir)
}

async function cleanupDir (dir: string) {
async function prepareDir (dir: string) {
await fsp.mkdir(dir, { recursive: true })
await fse.emptyDir(dir)
}

Expand All @@ -42,7 +38,7 @@ export async function build (nitro: Nitro) {
const htmlSrc = resolve(nitro.options.buildDir, 'views/app.template.html')
const htmlTemplate = { src: htmlSrc, contents: '', dst: '' }
htmlTemplate.dst = htmlTemplate.src.replace(/.html$/, '.mjs').replace('app.template.mjs', 'document.template.mjs')
htmlTemplate.contents = nitro.vfs[htmlTemplate.src] || await fse.readFile(htmlTemplate.src, 'utf-8').catch(() => '')
htmlTemplate.contents = nitro.vfs[htmlTemplate.src] || await fsp.readFile(htmlTemplate.src, 'utf-8').catch(() => '')
if (htmlTemplate.contents) {
await nitro.hooks.callHook('nitro:document', htmlTemplate)
const compiled = 'export default ' + serializeTemplate(htmlTemplate.contents)
Expand Down
4 changes: 2 additions & 2 deletions src/prerender.ts
Original file line number Diff line number Diff line change
Expand Up @@ -26,12 +26,12 @@ export async function prerender (nitro: Nitro) {
await build(nitroRenderer)

// Import renderer entry
const app = await import(resolve(nitroRenderer.options.output.serverDir, 'index.mjs'))
const { nitroApp } = await import(resolve(nitroRenderer.options.output.serverDir, 'index.mjs'))

// Start prerendering
const generatedRoutes = new Set()
const generateRoute = async (route: string) => {
const res = await app.localFetch(route)
const res = await nitroApp.localFetch(route)
const contents = await res.text()

const additionalExtension = getExtension(route) ? '' : guessExt(res.headers.get('content-type'))
Expand Down
2 changes: 1 addition & 1 deletion src/presets/node-server.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import { defineNitroPreset } from '../preset'

export const nodeServer = defineNitroPreset({
extends: 'node',
entry: '#nitro/entries/server',
entry: '#nitro/entries/node-server',
serveStatic: true,
commands: {
preview: 'node ./server/index.mjs'
Expand Down
1 change: 1 addition & 0 deletions src/runtime/entries/node.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
import '#polyfill'
import * as nodeFetch from 'node-fetch'
import { nitroApp } from '../app'
export { nitroApp } from '../app'

// TODO: Workaround for rollup treeshaking polyfills
// @ts-ignore
Expand Down

0 comments on commit 26046ee

Please sign in to comment.