fix(nuxthub): register sql_dump route even if the database is disabled

[onmax]

Closes #3667
resolves #3666

Summary

• sql_dump.txt routes fail prerendering with crawlLinks: true because handlers need cloudflare bindings (ASSETS) unavailable at build time
• Fix: add prerender.ignore pattern for these routes in cloudflare and node presets

StackBlitz

	Link	Expected
Bug	nuxthub-754	❌ Build fails
Fix	nuxthub-754-fixed	✅ Build succeeds

CLI Reproduction

git clone --depth 1 --filter=blob:none --sparse https://github.com/onmax/repros.git
cd repros && git sparse-checkout set nuxthub-754
cd nuxthub-754 && pnpm i && pnpm build

Verify fix

git sparse-checkout add nuxthub-754-fixed
cd ../nuxthub-754-fixed && pnpm i && pnpm build

The -fixed folder uses pnpm patch to apply the fix locally.

Related

• Failed to Prerender sql_dump.txt from Nuxt Content nuxt-hub/core#754

[vercel]

@onmax is attempting to deploy a commit to the Nuxt Team on Vercel.

A member of the Team first needs to authorize it.

[pkg-pr-new]

npm i https://pkg.pr.new/@nuxt/content@3668

commit: bb3627b

[farnabaz]

Disabling pre-render is not the solution for the problem. When database is disabled, NuxtHub Preset does not call parent preset and therefore sql_dump route do not register.

content/src/presets/nuxthub.ts

Lines 15 to 18 in 7c07081

	if (!nuxtOptions.hub?.db && !nuxtOptions.hub?.database) {
	logger.warn('NuxtHub dedected but the database is not enabled. Using local SQLite as default database instead.')
	return
	}

Do you mind updating your PR with:

// file: src/presets/nuxthub.ts
    if (nuxtOptions.hub?.db || nuxtOptions.hub?.database) {
    const runtimeConfig = nuxt.options.runtimeConfig as unknown as { hub: { db?: boolean | { driver: string, connection: { url?: string } }, database?: boolean | { driver: string, connection: { url?: string } } } }
    // Read from the final hub database configuration
    const hubDb = runtimeConfig.hub.db || runtimeConfig.hub.database
    // NuxtHub <= 0.9
    if (nuxtOptions.hub?.database === true) {
      options.database ||= { type: 'd1', bindingName: 'DB' }
    }
    else if (typeof nuxtOptions.hub?.db === 'string' && typeof hubDb === 'object') {
      if (hubDb.driver === 'd1') {
        options.database ||= { type: 'd1', bindingName: 'DB' }
      }
      else if (hubDb.driver === 'node-postgres') {
        options.database ||= { type: 'postgresql', url: hubDb.connection.url as string }
      }
      else {
        options.database ||= { type: hubDb.driver as 'sqlite' | 'postgresql' | 'postgres' | 'libsql' | 'pglite', ...hubDb.connection } as unknown as SqliteDatabaseConfig | LibSQLDatabaseConfig | PGliteDatabaseConfig
      }
    }
    } else {
      logger.warn('NuxtHub dedected but the database is not enabled. Using local SQLite as default database instead.')
      return
    }

[farnabaz]

@luminous8 @onmax Do you mind testing with PR release npm i https://pkg.pr.new/@nuxt/content@bb3627b and confirm the fix?

[onmax]

Tested with npm i https://pkg.pr.new/@nuxt/content@bb3627b - build succeeds ✅

[nitro] ├─ /__nuxt_content/content/sql_dump.txt (56ms)
└  ✨ Build complete!

Repro: https://github.com/onmax/repros/tree/main/nuxthub-754-fixed