UForm causing memory leak

[ronenteva]

Environment

• Operating System: Windows_NT
• Node Version: v20.10.0
• Nuxt Version: 3.8.2
• CLI Version: 3.10.0
• Nitro Version: 2.8.1
• Package Manager: pnpm@8.12.0
• Builder: -
• User Config: modules, ui, devtools
• Runtime Modules: @nuxt/ui@2.11.0
• Build Modules: -

Version

v2.11.0

Reproduction

app.vue:

<script setup lang="ts">
import type { FormError, FormSubmitEvent } from '#ui/types'

const state = reactive({
  email: undefined,
  password: undefined
})

const validate = (state: any): FormError[] => {
  const errors = []
  if (!state.email) errors.push({ path: 'email', message: 'Required' })
  if (!state.password) errors.push({ path: 'password', message: 'Required' })
  return errors
}

async function onSubmit (event: FormSubmitEvent<any>) {
  // Do something with data
  console.log(event.data)
}
</script>

<template>
  <UForm :validate="validate" :state="state" class="space-y-4" @submit="onSubmit">
    <UFormGroup label="Email" name="email">
      <UInput v-model="state.email" />
    </UFormGroup>

    <UFormGroup label="Password" name="password">
      <UInput v-model="state.password" type="password" />
    </UFormGroup>

    <UButton type="submit">
      Submit
    </UButton>
  </UForm>
</template>

a simple logger for memory usage /server/plugins/memory.js:

export default defineNitroPlugin(nitroApp => {
  let maxUsage = 0;
  setInterval(() => {
    const memoryUsage = process.memoryUsage();
    if (memoryUsage.heapUsed > maxUsage) {
      maxUsage = memoryUsage.heapUsed;
      console.log(`Heap Used: ${Math.round(memoryUsage.heapUsed / 1024 / 1024 * 100) / 100} MB`);
    }
  }, 1000);
});

run autocannon http://localhost:3000 -f to see the memory usage going up and up

Description

Seems that any usage of <UForm/> is causing memory leak.

Additional context

No response

Logs

No response

[benjamincanac]

@romhml Any idea about this?

[romhml]

I managed to find two different cause: one is the ever-growing counter in useUid and the other is the form's event listeners not being cleared. I fixed both in #1185.