feat(nuxt): simplify getCurrentUser

Author: posva

Diff for: packages/nuxt/playground/middleware/vuefire-auth.ts

@@ -1,8 +1,7 @@
-import { getCurrentUser } from 'vuefire'
 export default defineNuxtRouteMiddleware(async (to, from) => {
-  const app = useNuxtApp().$firebaseApp
-  console.log('app name', app.name)
-  const user = await getCurrentUser(app.name)
+  const user = await getCurrentUser()
+
+  console.log('got user in middleware', user?.uid)
 
   if (!user) {
     return navigateTo('/authentication')

Diff for: packages/nuxt/src/module.ts

@@ -140,7 +140,17 @@ const VueFire: NuxtModule<VueFireNuxtModuleOptions> =
         addVueFireImports([
           // auth
           { from: 'vuefire', name: 'useFirebaseAuth' },
-          { from: 'vuefire', name: 'useCurrentUser' },
+        ])
+        // these are improved for nuxt to avoid the need to pass the app name
+        addImports([
+          {
+            from: resolve(runtimeDir, 'auth/composables'),
+            name: 'getCurrentUser',
+          },
+          {
+            from: resolve(runtimeDir, 'auth/composables'),
+            name: 'useCurrentUser',
+          },
         ])
       }
 

Diff for: packages/nuxt/src/runtime/auth/composables.ts

@@ -0,0 +1,10 @@
+import { getCurrentUser as _getCurrentUser } from 'vuefire'
+import { useNuxtApp } from '#app'
+
+/**
+ * @inheritDoc {getCurrentUser}
+ */
+export function getCurrentUser(name?: string) {
+  // This makes the `getCurrentUser()` function work by default in more places when using the Nuxt module
+  return _getCurrentUser(name ?? useNuxtApp().$firebaseApp.name)
+}