fix(nuxt): revert async transform of setup within defineComponent (#24784)

packages/nuxt/src/app/composables/asyncContext.ts

@@ -1,11 +1,9 @@
 // @ts-expect-error withAsyncContext is internal API
 import { getCurrentInstance, withAsyncContext as withVueAsyncContext } from 'vue'
 
-export function withNuxtContext <T>(fn: () => T) {
+export function withAsyncContext (fn: () => PromiseLike<unknown>) {
+  return withVueAsyncContext(() => {
     const nuxtApp = getCurrentInstance()?.appContext.app.$nuxt
     return nuxtApp ? nuxtApp.runWithContext(fn) : fn()
-}
+  })
-
-export function withAsyncContext (fn: () => PromiseLike<unknown>) {
-  return withVueAsyncContext(() => withNuxtContext(fn))
 }

packages/nuxt/src/app/composables/component.ts

@@ -1,9 +1,8 @@
-import { defineComponent as _defineComponent, getCurrentInstance, reactive, toRefs } from 'vue'
+import { getCurrentInstance, reactive, toRefs } from 'vue'
-import type { ComponentOptions, DefineComponent } from 'vue'
+import type { DefineComponent, defineComponent } from 'vue'
 import { useHead } from '@unhead/vue'
 import type { NuxtApp } from '../nuxt'
 import { useNuxtApp } from '../nuxt'
-import { withNuxtContext } from './asyncContext'
 import { useAsyncData } from './asyncData'
 import { useRoute } from './router'
 import { createError } from './error'
@@ -29,7 +28,7 @@ async function runLegacyAsyncData (res: Record<string, any> | Promise<Record<str
 }
 
 /*@__NO_SIDE_EFFECTS__*/
-export const defineNuxtComponent: typeof _defineComponent =
+export const defineNuxtComponent: typeof defineComponent =
   function defineNuxtComponent (...args: any[]): any {
     const [options, key] = args
     const { setup } = options
@@ -69,18 +68,3 @@ export const defineNuxtComponent: typeof _defineComponent =
       }
     } as DefineComponent
   }
-
-export const defineComponent: typeof _defineComponent = (arg1: Function | Record<string, any>, arg2?: Partial<ComponentOptions>) => {
-  if (typeof arg1 === 'function') {
-    return _defineComponent((...args) => withNuxtContext(() => arg1(...args)), arg2)
-  }
-
-  if (arg1.setup) {
-    return _defineComponent({
-      ...arg1,
-      setup: (...args: any[]) => withNuxtContext(() => arg1.setup(...args))
-    }) as any
-  }
-
-  return _defineComponent(arg1)
-}

packages/nuxt/src/imports/presets.ts

@@ -30,7 +30,7 @@ const granularAppPresets: InlinePreset[] = [
     from: '#app/config'
   },
   {
-    imports: ['defineComponent', 'defineNuxtComponent'],
+    imports: ['defineNuxtComponent'],
     from: '#app/composables/component'
   },
   {
@@ -156,6 +156,7 @@ const vuePreset = defineUnimportPreset({
     'onScopeDispose',
 
     // Component
+    'defineComponent',
     'defineAsyncComponent',
     'resolveComponent',
     'getCurrentInstance',

packages/nuxt/test/auto-imports.test.ts

@@ -81,8 +81,6 @@ describe('imports:nuxt', () => {
 })
 
 const excludedVueHelpers = [
-  // Nuxt stub for this helper
-  'defineComponent',
   // Already globally registered
   'defineEmits',
   'defineExpose',

packages/schema/src/config/build.ts

@@ -188,7 +188,6 @@ export default defineUntypedSchema({
     asyncTransforms: {
       asyncFunctions: ['defineNuxtPlugin', 'defineNuxtRouteMiddleware'],
       objectDefinitions: {
-        defineComponent: ['setup'],
         defineNuxtComponent: ['asyncData', 'setup'],
         defineNuxtPlugin: ['setup'],
         definePageMeta: ['middleware', 'validate']

test/basic.test.ts

@@ -1951,14 +1951,10 @@ describe.skipIf(isDev() || isWindows || !isRenderingJson)('payload rendering', (
   })
 })
 
-describe('Async context', () => {
+describe.skipIf(process.env.TEST_CONTEXT !== 'async')('Async context', () => {
-  it.skipIf(process.env.TEST_CONTEXT !== 'async')('should be available', async () => {
+  it('should be available', async () => {
     expect(await $fetch('/async-context')).toContain('"hasApp": true')
   })
-  it('should transform `setup` within defineComponent', async () => {
-    const html = await $fetch('/async-transform-component')
-    expect(html).toContain('using automatic `setup` async transform')
-  })
 })
 
 describe.skipIf(isWindows)('useAsyncData', () => {

test/fixtures/basic/pages/async-transform-component.vue

@@ -1,14 +0,0 @@
-<script>
-export default defineComponent({
-  async setup () {
-    await nextTick()
-    useRuntimeConfig()
-  }
-})
-</script>
-
-<template>
-  <div>
-    should not error out when using automatic `setup` async transform
-  </div>
-</template>