fix: lint errors

Some of them weren't solved previously because it was needed to be verified that the lifecycle of the logout process was correct

Signed-off-by: Fernando Fernández <[email protected]>

Author: ferferga

frontend/src/components/Layout/MainView.vue

@@ -28,7 +28,7 @@
 </template>
 
 <script lang="ts">
-import { onErrorCaptured, shallowRef, type Component, watch, computed, provide, useId, ref, type WatchOptions, inject } from 'vue';
+import { onErrorCaptured, shallowRef, type Component, watch, computed, provide, useId, ref, inject } from 'vue';
 import type { RouteLocationNormalizedGeneric, RouteMeta } from 'vue-router';
 import { isNil } from '@jellyfin-vue/shared/validation';
 import { usePausableEffect } from '@jellyfin-vue/ui-toolkit/composables/use-pausable-effect.ts';
@@ -57,15 +57,14 @@ function getLayoutComponent(layout: RouteMeta['layout']['name']): Component {
 
 const defaultTransition = 'slide-x-reverse';
 const defaultTransitionMode = 'out-in';
-const watchOps = { flush: 'sync' } satisfies WatchOptions;
 const apploaded = shallowRef(false);
 const isRouting = shallowRef(false);
 const _resolveStatus = ref<Record<string, boolean>>({});
 const allResolved = computed(() => Object.values(_resolveStatus.value).every(Boolean));
 const mustBePaused = computed(() => !allResolved.value || isRouting.value);
 
-watch(() => router.currentRoute.value.name, () => isRouting.value = true, watchOps);
-watch(allResolved, () => isRouting.value = false, watchOps);
+watch(() => router.currentRoute.value.name, () => isRouting.value = true);
+watch(allResolved, () => isRouting.value = false);
 </script>
 
 <script setup lang="ts">
@@ -93,7 +92,8 @@ const resolved = computed({
 if (root) {
   provide(JView_isRouting, mustBePaused);
   usePausableEffect(mustBePaused);
-  onErrorCaptured(() => {
+  onErrorCaptured((e, instance, info) => {
+    console.error('[Jellyfin] Error captured in MainView:', e, instance, info);
     resolved.value = true;
     isRouting.value = false;
   });

frontend/src/layouts/default.vue

@@ -29,11 +29,11 @@ const navDrawer = ref(!display.mobile.value);
 const { views } = await fetchIndexPage();
 
 const drawerItems = computed<DrawerItem[]>(() => {
-  return (views.value ?? []).map((view: BaseItemDto) => {
+  return views.value.map((view: BaseItemDto) => {
     return {
       icon: getLibraryIcon(view.CollectionType),
       title: view.Name ?? '',
-      to: `/library/${String(view.Id)}`
+      to: `/library/${view.Id}`
     };
   });
 });

frontend/src/pages/playback/music.vue

@@ -9,7 +9,7 @@
       <VSpacer />
       <AppBarButtonLayout @click="isVisualizing = !isVisualizing">
         <template #icon>
-          <JIcon :class="isVisualizing ? 'i-dashicons:album' : 'mdi:chart-bar'" />
+          <JIcon :class="isVisualizing ? 'i-dashicons:album' : 'i-mdi:chart-bar'" />
         </template>
       </AppBarButtonLayout>
     </VAppBar>

frontend/src/pages/search.vue

@@ -121,7 +121,7 @@ const [{ loading: itemLoading, data: itemSearch }, { loading: peopleLoading, dat
 
 const serverSearchIds = computed(() => {
   if (!peopleLoading.value && !itemLoading.value) {
-    return [...(itemSearch.value ?? []), ...(peopleSearch.value ?? [])].map(i => i.Id!);
+    return [...itemSearch.value, ...peopleSearch.value].map(i => i.Id!);
   }
 
   return [];