fix(nuxt): ignore errors when treeshaking composables within other composables

danielroe · danielroe · commit f99eac516aa5 · 2025-09-07T21:48:56.000+01:00
resolves #33151
diff --git a/packages/nuxt/src/core/plugins/tree-shake.ts b/packages/nuxt/src/core/plugins/tree-shake.ts
@@ -82,8 +82,12 @@ export const TreeShakeComposablesPlugin = (options: TreeShakeComposablesPluginOp
               return
             }
 
-            // TODO: validate function name against actual auto-imports registry
-            s.overwrite(node.start, node.end, ` false && /*@__PURE__*/ ${functionName}${code.slice(node.callee.end, node.end)}`)
+            try {
+              // TODO: validate function name against actual auto-imports registry
+              s.overwrite(node.start, node.end, ` false && /*@__PURE__*/ ${functionName}${code.slice(node.callee.end, node.end)}`)
+            } catch {
+              // TODO: fix when we can skip subsequent nodes even with scope tracker
+            }
           },
         })
 
diff --git a/packages/nuxt/test/tree-shake.test.ts b/packages/nuxt/test/tree-shake.test.ts
@@ -34,6 +34,24 @@ describe('tree-shake', () => {
     `)
   })
 
+  it('should not error when tree-shaking composables within other tree-shaken composables', () => {
+    const code = `
+      import { onMounted as _onMounted } from 'vue'
+      _onMounted(() => {
+        onMounted(() => {})
+      })
+      console.log('Hello World')
+    `
+    const { code: result } = transformPlugin.transform.handler(code, 'test.js')
+    expect(clean(result)).toMatchInlineSnapshot(`
+      "import { onMounted as _onMounted } from 'vue'
+       false && /*@__PURE__*/ _onMounted(() => {
+        onMounted(() => {})
+      })
+      console.log('Hello World')"
+    `)
+  })
+
   it('should tree-shake explicitly-imported composables from #imports', () => {
     const code = `
       import { onMounted } from '#imports'
