fix: fix navigation to search for provider

Author: alexdln

app/components/Header/SearchBox.vue

@@ -15,12 +15,13 @@ const emit = defineEmits(['blur', 'focus'])
 
 const router = useRouter()
 const route = useRoute()
+const { searchProvider } = useSearchProvider()
 // The actual search provider (from URL, used for API calls)
 const searchProviderParam = computed(() => {
   const p = normalizeSearchParam(route.query.p)
+  if (!p && searchProvider.value === 'npm') return 'npm'
   return p === 'npm' ? 'npm' : 'algolia'
 })
-const { searchProvider } = useSearchProvider()
 const searchProviderValue = computed(() => searchProviderParam.value || searchProvider.value)
 
 const isSearchFocused = shallowRef(false)
@@ -35,15 +36,13 @@ const searchQuery = shallowRef(normalizeSearchParam(route.query.q))
 // Pages that have their own local filter using ?q
 const pagesWithLocalFilter = new Set(['~username', 'org'])
 
-function updateUrlQueryImpl(value: string) {
+function updateUrlQueryImpl(value: string, provider: 'npm' | 'algolia') {
   // Don't navigate away from pages that use ?q for local filtering
   if (pagesWithLocalFilter.has(route.name as string)) {
     return
   }
   if (route.name === 'search') {
-    router.replace({
-      query: { q: value || undefined, p: searchProviderValue.value === 'npm' ? 'npm' : undefined },
-    })
+    router.replace({ query: { q: value || undefined, p: provider === 'npm' ? 'npm' : undefined } })
     return
   }
   if (!value) {
@@ -54,7 +53,7 @@ function updateUrlQueryImpl(value: string) {
     name: 'search',
     query: {
       q: value,
-      p: searchProviderValue.value === 'npm' ? 'npm' : undefined,
+      p: provider === 'npm' ? 'npm' : undefined,
     },
   })
 }
@@ -64,7 +63,10 @@ const updateUrlQueryAlgolia = debounce(updateUrlQueryImpl, 80)
 
 const updateUrlQuery = Object.assign(
   (value: string) =>
-    (searchProviderValue.value === 'algolia' ? updateUrlQueryAlgolia : updateUrlQueryNpm)(value),
+    (searchProviderValue.value === 'algolia' ? updateUrlQueryAlgolia : updateUrlQueryNpm)(
+      value,
+      searchProviderValue.value,
+    ),
   {
     flush: () =>
       (searchProviderValue.value === 'algolia' ? updateUrlQueryAlgolia : updateUrlQueryNpm).flush(),
@@ -96,7 +98,7 @@ function handleSubmit() {
       name: 'search',
       query: {
         q: searchQuery.value,
-        p: searchProviderValue.value,
+        p: searchProviderValue.value === 'npm' ? 'npm' : undefined,
       },
     })
   } else {