fix: remove useId() and ClientOnly from DownloadButton

useId() inside ClientOnly causes server/client ID counter desync,
leading to hydration mismatches. Replace with a static ID since
there's only one download menu per page. Remove ClientOnly wrapper
as the component doesn't do any fetching until user interaction.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: Adebesin-Cell

app/components/Package/DownloadButton.vue

@@ -21,7 +21,7 @@ const highlightedIndex = shallowRef(-1)
 const dropdownPosition = shallowRef<{ top: number; right: number } | null>(null)
 
 const { t } = useI18n()
-const menuId = useId()
+const menuId = 'download-menu'
 const menuItems = computed(() => {
   const items = [{ id: 'package', label: t('package.download.package'), icon: 'i-lucide:package' }]
   if (props.installSize) {

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

@@ -773,15 +773,13 @@ const showSkeleton = shallowRef(false)
             </h2>
             <!-- Package manager dropdown + Download button -->
             <div class="flex items-center gap-2">
-              <ClientOnly>
-                <PackageDownloadButton
-                  v-if="displayVersion"
-                  :package-name="pkg.name"
-                  :version="displayVersion"
-                  :install-size="installSize ?? null"
-                  size="small"
-                />
-              </ClientOnly>
+              <PackageDownloadButton
+                v-if="displayVersion"
+                :package-name="pkg.name"
+                :version="displayVersion"
+                :install-size="installSize ?? null"
+                size="small"
+              />
               <PackageManagerSelect />
             </div>
           </div>