refactor: merge app into runtimeConfig.nitro

README.md

@@ -139,20 +139,13 @@ Built-in presets: `azure_functions`, `azure`, `browser`, `cli`, `cloudflare`, `f
 
 Log verbosity level. See [unjs/consola#level](https://github.com/unjs/consola/#level) for more information.
 
-### App
-
 #### `runtimeConfig`
 
-- Default: `{}`
+- Default: `{ nitro: { ... }, ...yourOptions }`
 
 Server runtime configuration.
 
-#### `app`
-
-- Default: `{ baseURL: '/', cdnURL: undefined, buildAssetsDir: 'dist' }`
-
-Runtime app configuration.
-
+**Note:**: `nitro` namespace is reserved.
 
 ### Directories
 

src/dev/server.ts

@@ -107,7 +107,7 @@ export function createDevServer (nitro: Nitro): NitroDevServer {
 
   // Serve asset dirs
   for (const asset of nitro.options.publicAssets) {
-    const url = joinURL(nitro.options.app.baseURL, asset.baseURL)
+    const url = joinURL(nitro.options.runtimeConfig.nitro.baseURL, asset.baseURL)
     app.use(url, serveStatic(asset.dir))
     if (!asset.fallthrough) {
       app.use(url, servePlaceholder())
@@ -116,7 +116,7 @@ export function createDevServer (nitro: Nitro): NitroDevServer {
 
   // Serve placeholder 404 assets instead of hitting SSR
   // TODO: Option to opt-out
-  app.use(nitro.options.app.baseURL, servePlaceholder({ skipUnknown: true }))
+  app.use(nitro.options.runtimeConfig.nitro.baseURL, servePlaceholder({ skipUnknown: true }))
 
   // Worker proxy
   const proxy = httpProxy.createProxy()

src/options.ts

@@ -13,13 +13,11 @@ const NitroDefaults: NitroConfig = {
   preset: undefined,
   logLevel: 3,
   runtimeConfig: {
-    public: {},
-    private: {}
-  },
-  app: {
-    baseURL: '/',
-    cdnURL: undefined,
-    buildAssetsDir: 'dist'
+    nitro: {
+      baseURL: '/',
+      cdnURL: undefined,
+      buildAssetsDir: 'dist'
+    }
   },
 
   // Dirs
@@ -112,10 +110,9 @@ export async function loadOptions (userConfig: NitroConfig = {}): Promise<NitroO
     options.scanDirs = [options.srcDir]
   }
 
-  options.app.routes = options.routes
   options.runtimeConfig = defu(options.runtimeConfig, {
-    public: {
-      app: options.app
+    app: {
+      routes: options.routes
     }
   })
 

src/runtime/app.ts

@@ -18,7 +18,7 @@ app.use(timingMiddleware)
 
 const router = createRouter()
 
-const routerOptions = createMatcher({ routes: config.app.routes })
+const routerOptions = createMatcher({ routes: config.nitro.routes })
 
 for (const h of handlers) {
   let handler = h.lazy ? lazyEventHandler(h.handler as any) : h.handler

src/runtime/error.ts

@@ -57,7 +57,7 @@ function renderHTMLError (error: ParsedError): string {
     <ul>${'\n' + (error.stack || []).map(i => `      <li>${i}</li>`).join('\n')}
     </ul>
     <hr>
-    Generated by nitropack at ${new Date().toISOString()}
+    Generated by nitro at ${new Date().toISOString()}
   </body>
 </html>`
 }

src/runtime/paths.ts

@@ -2,18 +2,18 @@ import { joinURL } from 'ufo'
 import config from '#nitro/config'
 
 export function baseURL (): string {
-  return config.app.baseURL
+  return config.nitro.baseURL
 }
 
 export function buildAssetsDir (): string {
-  return config.app.buildAssetsDir
+  return config.nitro.buildAssetsDir
 }
 
 export function buildAssetsURL (...path: string[]): string {
-  return joinURL(publicAssetsURL(), config.app.buildAssetsDir, ...path)
+  return joinURL(publicAssetsURL(), config.nitro.buildAssetsDir, ...path)
 }
 
 export function publicAssetsURL (...path: string[]): string {
-  const publicBase = config.app.cdnURL || config.app.baseURL
+  const publicBase = config.nitro.cdnURL || config.nitro.baseURL
   return path.length ? joinURL(publicBase, ...path) : publicBase
 }

src/types/nitro.ts

@@ -70,16 +70,14 @@ export interface NitroOptions {
   // General
   preset: string
   logLevel: LogLevel
-
-  // App
-  runtimeConfig: { public: any, private: any }
-  routes: NitroRoutesOptions
-  app: {
-    /** @deprecated Use top-level routes option! */
-    routes: NitroRoutesOptions
-    baseURL: string,
-    cdnURL: string,
-    buildAssetsDir: string
+  runtimeConfig: {
+    nitro: {
+      /** @deprecated Use top-level routes option! */
+      routes: NitroRoutesOptions
+      baseURL: string,
+      cdnURL: string,
+      buildAssetsDir: string
+    }
     [key: string]: any
   }
 
@@ -104,6 +102,7 @@ export interface NitroOptions {
   }
   serverAssets: ServerAssetDir[]
   publicAssets: PublicAssetDir[]
+  autoImport: UnimportOptions
 
   // Dev
   dev: boolean
@@ -112,23 +111,23 @@ export interface NitroOptions {
 
   // Routing
   handlers: NitroEventHandler[]
+  routes: NitroRoutesOptions
   devHandlers: NitroDevEventHandler[]
   prerender: {
     crawlLinks: boolean
     routes: string[]
   }
 
   // Rollup
+  rollupConfig?: RollupConfig
   entry: string
   unenv: UnenvPreset
   alias: Record<string, string>
   minify: boolean
   inlineDynamicImports: boolean
   sourceMap: boolean
   node: boolean
-  rollupConfig?: RollupConfig
   moduleSideEffects: string[]
-  autoImport: UnimportOptions
   esbuild?: {
     options?: EsbuildOptions
   }