fix: improved bundle: true debug and cache clearing

nuxt · Sep 15, 2024 · d269066 · d269066
1 parent 5411a75
commit d269066
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 5 deletions.
diff --git a/playground/nuxt.config.ts b/playground/nuxt.config.ts
@@ -6,4 +6,8 @@ export default defineNuxtConfig({
 
   devtools: { enabled: true },
   compatibilityDate: '2024-07-14',
+
+  scripts: {
+    debug: true,
+  },
 })
diff --git a/src/module.ts b/src/module.ts
@@ -101,6 +101,7 @@ export default defineNuxtModule<ModuleOptions>({
     nuxt.options.alias['#nuxt-scripts-validator'] = resolve(`./runtime/validation/${(nuxt.options.dev || nuxt.options._prepare) ? 'valibot' : 'mock'}`)
     nuxt.options.alias['#nuxt-scripts'] = resolve('./runtime/types')
     nuxt.options.alias['#nuxt-scripts-utils'] = resolve('./runtime/utils')
+    logger.level = (config.debug || nuxt.options.debug) ? 4 : 3
     if (!config.enabled) {
       // TODO fallback to useHead?
       logger.debug('The module is disabled, skipping setup.')

diff --git a/src/plugins/transform.ts b/src/plugins/transform.ts
@@ -104,14 +104,22 @@ export function NuxtScriptBundleTransformer(options: AssetBundlerTransformerOpti
   // done after all transformation is done
   // copy all scripts to build
   nuxt.hooks.hook('build:done', async () => {
-    logger.log('[nuxt:scripts:bundler-transformer] Bundling scripts...')
-    await fsp.rm(cacheDir, { recursive: true, force: true })
-    await fsp.mkdir(cacheDir, { recursive: true })
-    await Promise.all([...renderedScript].map(async ([url, content]) => {
+    const scripts = [...renderedScript]
+    if (!scripts.length) {
+      logger.debug('[bundle-script-transformer] No scripts to bundle...')
+      return
+    }
+    logger.info('[bundle-script-transformer] Bundling scripts...')
+    // less aggressive cache clearing in dev
+    if (!nuxt.options.dev) {
+      await fsp.rm(cacheDir, { recursive: true, force: true })
+      await fsp.mkdir(cacheDir, { recursive: true })
+    }
+    await Promise.all(scripts.map(async ([url, content]) => {
       if (content instanceof Error || !content.filename)
         return
       await fsp.writeFile(join(nuxt.options.buildDir, 'cache', 'scripts', content.filename), content.content)
-      logger.log(colors.gray(`  ├─ ${url} → ${joinURL(content.src)} (${content.size.toFixed(2)} kB ${content.encoding})`))
+      logger.info(colors.gray(`  ├─ ${url} → ${joinURL(content.src)} (${content.size.toFixed(2)} kB ${content.encoding})`))
     }))
   })