fix!: don't disable module if no projectId is specified

BREAKING CHANGE: If you were relying on the auto-disabling part of this module

Author: danielroe

package.json

@@ -67,7 +67,7 @@
     "nuxi": "0.10.1",
     "nuxt3": "^3.0.0-27420403.16e2a54",
     "release-it": "14.12.4",
-    "vitest": "0.4.1"
+    "vitest": "0.4.2"
   },
   "publishConfig": {
     "access": "public"

src/module.ts

@@ -36,18 +36,6 @@ export type ModuleOptions = SanityModuleOptions
 
 const logger = useLogger('@nuxtjs/sanity')
 
-function validateConfig ({ projectId, dataset }: SanityModuleOptions) {
-  if (!projectId) {
-    logger.warn(
-      `Make sure you specify a ${chalk.bold('projectId')} in your sanity config.`,
-    )
-    return false
-  } else {
-    logger.info(`Sanity project ${chalk.bold(projectId)} (${chalk.bold(dataset)}).`)
-    return true
-  }
-}
-
 function getDefaultSanityConfig (jsonPath: string) {
   try {
     const { projectId, dataset } = fse.readJSONSync(jsonPath).api
@@ -74,15 +62,12 @@ export default defineNuxtModule<SanityModuleOptions>({
     withCredentials: false,
     additionalClients: {},
     ...getDefaultSanityConfig(resolve(nuxt.options.rootDir, './sanity.json')),
-    ...nuxt.options.publicRuntimeConfig.sanity
   }),
   async setup (options, nuxt) {
     if (!('useCdn' in options)) {
       options.useCdn = process.env.NODE_ENV === 'production' && !options.token
     }
 
-    if (!validateConfig(options)) return
-
     try {
       if (!options.minimal) {
         options.minimal = !requireModule('@sanity/client')
@@ -93,7 +78,7 @@ export default defineNuxtModule<SanityModuleOptions>({
     }
 
     // Final resolved configuration
-    nuxt.options.publicRuntimeConfig.sanity = defu(nuxt.options.publicRuntimeConfig.sanity, {
+    nuxt.options.publicRuntimeConfig.sanity = options = defu(nuxt.options.publicRuntimeConfig.sanity, {
       useCdn: options.useCdn,
       projectId: options.projectId,
       dataset: options.dataset,
@@ -103,6 +88,12 @@ export default defineNuxtModule<SanityModuleOptions>({
       additionalClients: options.additionalClients,
     })
 
+    if (!options.projectId) {
+      logger.warn(`No Sanity project found. Make sure you specify a ${chalk.bold('projectId')} in your Sanity config.`)
+    } else {
+      logger.info(`Running with Sanity project ${chalk.bold(options.projectId)} (${chalk.bold(options.dataset)}).`)
+    }
+
     const runtimeDir = fileURLToPath(new URL('./runtime', import.meta.url))
     nuxt.options.build.transpile.push(runtimeDir, '@nuxtjs/sanity')
 

yarn.lock

@@ -843,7 +843,7 @@ __metadata:
     nuxt3: ^3.0.0-27420403.16e2a54
     pathe: 0.2.0
     release-it: 14.12.4
-    vitest: 0.4.1
+    vitest: 0.4.2
   languageName: unknown
   linkType: soft
 
@@ -11777,9 +11777,9 @@ __metadata:
   languageName: node
   linkType: hard
 
-"vitest@npm:0.4.1":
-  version: 0.4.1
-  resolution: "vitest@npm:0.4.1"
+"vitest@npm:0.4.2":
+  version: 0.4.2
+  resolution: "vitest@npm:0.4.2"
   dependencies:
     "@types/chai": ^4.3.0
     "@types/chai-subset": ^1.3.3
@@ -11804,7 +11804,7 @@ __metadata:
       optional: true
   bin:
     vitest: vitest.mjs
-  checksum: f350464cd674612151ea6e63fd37401156ff9d3fafd3f663595beb1dbe19658fb1f7f345253250f3a9c5dff7febf8fc3284f94d46ed167f37141bd609d424e05
+  checksum: f369a46955fdeaf35665a94a5307c665407d503778c57ddd54cec43793ebfe419bf4e49b44e78773f7f0d3918d0b7ab4f2c3f96d88816291321ef2fe527f93d4
   languageName: node
   linkType: hard