chore: minor cleanups (#1286)

Author: OrbisK

app/composables/npm/useSearch.ts

@@ -38,7 +38,7 @@ export function useSearch(
   } | null>(null)
 
   const isLoadingMore = shallowRef(false)
-  const isRateLimited = ref(false)
+  const isRateLimited = shallowRef(false)
 
   const suggestions = shallowRef<SearchSuggestion[]>([])
   const suggestionsLoading = shallowRef(false)

app/composables/useActiveTocItem.ts

@@ -10,11 +10,10 @@ import type { Ref } from 'vue'
  * @public
  */
 export function useActiveTocItem(toc: Ref<TocItem[]>) {
-  const activeId = ref<string | null>(null)
+  const activeId = shallowRef<string | null>(null)
 
   // Only run observer logic on client
   if (import.meta.server) {
-    // eslint-disable-next-line @typescript-eslint/no-empty-function
     return { activeId }
   }
 

app/composables/useCanGoBack.ts

@@ -1,5 +1,5 @@
 export function useCanGoBack() {
-  const canGoBack = ref(false)
+  const canGoBack = shallowRef(false)
 
   const router = useRouter()
 

app/composables/useCharts.ts

@@ -312,11 +312,11 @@ export function useCharts() {
     )
 
     const endDateOnly = toDateOnly(downloadEvolutionOptions.endDate)
-    const end = endDateOnly ? new Date(`${endDateOnly}T00:00:00.000Z`) : yesterday
+    const end = endDateOnly ? parseIsoDateOnly(endDateOnly) : yesterday
 
     const startDateOnly = toDateOnly(downloadEvolutionOptions.startDate)
     if (startDateOnly) {
-      const start = new Date(`${startDateOnly}T00:00:00.000Z`)
+      const start = parseIsoDateOnly(startDateOnly)
       return { start, end }
     }
 

app/composables/useFileTreeState.ts

@@ -1,7 +1,5 @@
 export function useFileTreeState(baseUrl: string) {
-  const stateKey = computed(() => `npmx-file-tree${baseUrl}`)
-
-  const expanded = useState<Set<string>>(stateKey.value, () => new Set<string>())
+  const expanded = useState<Set<string>>(`npmx-file-tree${baseUrl}`, () => new Set<string>())
 
   function toggleDir(path: string) {
     if (expanded.value.has(path)) {

app/composables/usePreferencesProvider.ts

@@ -1,3 +1,5 @@
+import { defu } from 'defu'
+
 /**
  * Abstraction for preferences storage
  * Currently uses localStorage, designed for future user prefs API
@@ -58,17 +60,17 @@ function createLocalStorageProvider<T>(key: string): StorageProvider<T> {
  * Abstracts the storage mechanism to allow future migration to API-based storage
  *
  */
-export function usePreferencesProvider<T>(defaultValue: T) {
+export function usePreferencesProvider<T extends object>(defaultValue: T) {
   const provider = createLocalStorageProvider<T>(STORAGE_KEY)
-  const data = ref<T>(defaultValue) as Ref<T>
+  const data = ref<T>(defaultValue)
   const isHydrated = shallowRef(false)
 
   // Load from storage on client
   onMounted(() => {
     const stored = provider.get()
     if (stored) {
       // Merge stored values with defaults to handle schema evolution
-      data.value = { ...defaultValue, ...stored }
+      data.value = defu(stored, defaultValue)
     }
     isHydrated.value = true
   })

app/pages/package/[[org]]/[name].vue

@@ -453,7 +453,7 @@ const isLoadingLikeData = computed(
   () => likeStatus.value !== 'error' && likeStatus.value !== 'success',
 )
 
-const isLikeActionPending = ref(false)
+const isLikeActionPending = shallowRef(false)
 
 const likeAction = async () => {
   if (user.value?.handle == null) {