fix: url should be optional in dev

unjs · Feb 7, 2024 · 9eb9b76 · 9eb9b76
1 parent d6ae4da
commit 9eb9b76
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/layers/core/cli/cli.ts b/layers/core/cli/cli.ts
@@ -25,7 +25,7 @@ export function createCLI(opts: DocsCLIOptions = {}) {
     },
     args: { ...sharedArgs },
     async setup({ args }) {
-      const { appDir, nuxtConfig } = await setupDocs(args.dir, opts.setup)
+      const { appDir, nuxtConfig } = await setupDocs(args.dir, { ...opts.setup, dev: true })
       process.chdir(appDir)
       await import('nuxi').then((nuxi) =>
         nuxi.runCommand('dev', [appDir, '--no-fork', '--port', '4000'], { overrides: nuxtConfig }),

diff --git a/layers/core/cli/setup.ts b/layers/core/cli/setup.ts
@@ -10,6 +10,7 @@ const pkgDir = fileURLToPath(new URL('../../..', import.meta.url))
 
 export interface SetupDocsOptions {
   defaults?: DocsConfig
+  dev?: boolean
   extends?: string[]
 }
 
@@ -21,7 +22,7 @@ export async function setupDocs(docsDir: string, opts: SetupDocsOptions = {}) {
   docsconfig.dir = docsDir = resolve(docsconfig.dir || docsDir)
 
   // URL is required for production build (SEO)
-  if (!docsconfig.url) {
+  if (!docsconfig.url && !opts.dev) {
     throw new Error('`url` config is required for production build!')
   }