moved from [...path] -> [[org]]/[name] ....

Author: WilcoSp

app/components/Package/Header.vue

@@ -133,7 +133,7 @@ const changelogLink = computed((): RouteLocationRaw | null => {
   if (!changelog.value || props.pkg == null || props.resolvedVersion == null) {
     return null
   }
-  return { name: 'changes', params: { path: [props.pkg.name, 'v', props.resolvedVersion] } }
+  return changelogRoute(props.pkg.name, props.resolvedVersion)
 })
 
 const keyboardShortcuts = useKeyboardShortcuts()

app/pages/package-changes/[...path].vue …pages/package-changes/[[org]]/[name].vueapp/pages/package-changes/[...path].vue renamed to app/pages/package-changes/[[org]]/[name].vue app/pages/package-changes/[...path].vue renamed to app/pages/package-changes/[[org]]/[name].vue

@@ -3,51 +3,41 @@ import { useProviderIcon } from '~/composables/useProviderIcon'
 
 definePageMeta({
   name: 'changes',
-  path: '/package-changes/:path+',
-  alias: ['/package/changes/:path+', '/changes/:path+'],
+  // path: '/package-changes/:path+',
+  // alias: ['/package/changes/:path+', '/changes/:path+'],
   scrollMargin: 190,
 })
 
 /// routing
 
 const route = useRoute('changes')
-const router = useRouter()
 // Parse package name, version, and file path from URL
 // Patterns:
-//   /code/nuxt/v/4.2.0 → packageName: "nuxt", version: "4.2.0", filePath: null (show tree)
-//   /code/nuxt/v/4.2.0/src/index.ts → packageName: "nuxt", version: "4.2.0", filePath: "src/index.ts"
-//   /code/@nuxt/kit/v/1.0.0 → packageName: "@nuxt/kit", version: "1.0.0", filePath: null
+//   /changes/nuxt/v/4.2.0 → packageName: "nuxt", version: "4.2.0", filePath: null (show tree)
+//   /changes/nuxt/v/4.2.0/src/index.ts → packageName: "nuxt", version: "4.2.0", filePath: "src/index.ts"
+//   /changes/@nuxt/kit/v/1.0.0 → packageName: "@nuxt/kit", version: "1.0.0", filePath: null
+
+//   const packageName = computed(() => {
+//   const { org, name } = route.params
+//   return org ? `${org}/${name}` : name
+// })
 const parsedRoute = computed(() => {
-  const segments = route.params.path
-
-  // Find the /v/ separator for version
-  const vIndex = segments.indexOf('v')
-  if (vIndex === -1 || vIndex >= segments.length - 1) {
-    // No version specified - redirect or error
-    return {
-      packageName: segments.join('/'),
-      version: null as string | null,
-      filePath: null as string | null,
-    }
-  }
+  const { org, name } = route.params
+
+  const packageName = org ? `${org}/${name}` : name
 
-  const packageName = segments.slice(0, vIndex).join('/')
-  const afterVersion = segments.slice(vIndex + 1)
-  const version = afterVersion[0] ?? null
-  const filePath = afterVersion.length > 1 ? afterVersion.slice(1).join('/') : null
+  const version = 'version' in route.params ? route.params.version : null
 
-  return { packageName, version, filePath }
+  return { packageName, version }
 })
 
 const packageName = computed(() => parsedRoute.value.packageName)
 const version = computed(() => parsedRoute.value.version)
-const filePath = computed(() => parsedRoute.value.filePath?.replace(/\/$/, ''))
 
 const { data: pkg } = usePackage(packageName, version)
 
 const versionUrlPattern = computed(() => {
-  const base = `/package-changes/${packageName.value}/v/{version}`
-  return filePath.value ? `${base}/${filePath.value}` : base
+  return `/package-changes/${packageName.value}/v/{version}`
 })
 
 const latestVersion = computed(() => {
@@ -57,16 +47,16 @@ const latestVersion = computed(() => {
   return pkg.value.versions[latestTag] ?? null
 })
 
-watch(
-  [version, () => latestVersion.value?.version, packageName],
-  ([v, latest, name]) => {
-    if (!v && latest && name) {
-      const pathSegments = [...name.split('/'), 'v', latest]
-      router.replace({ name: 'changes', params: { path: pathSegments as [string, ...string[]] } })
-    }
-  },
-  { immediate: true },
-)
+// watch(
+//   [version, () => latestVersion.value?.version, packageName],
+//   ([v, latest, name]) => {
+//     if (!v && latest && name) {
+//       const pathSegments = [...name.split('/'), 'v', latest]
+//       navigateTo({ name: 'changes', params: { path: pathSegments as [string, ...string[]] } })
+//     }
+//   },
+//   { immediate: true },
+// )
 
 // getting info
 const { data: changelog, pending } = usePackageChangelog(packageName, version)

app/pages/package-changes/[[org]]/[name]/v/[version].vue

@@ -0,0 +1,10 @@
+<script setup lang="ts">
+// stub
+
+definePageMeta({
+  name: 'changes-version',
+})
+</script>
+<template>
+  <div />
+</template>

app/utils/router.ts

@@ -45,3 +45,32 @@ export function diffRoute(
     },
   }
 }
+
+export function changelogRoute(
+  packageName: string,
+  version?: string | null,
+  hash?: string,
+): RouteLocationRaw {
+  const [org, name = ''] = packageName.startsWith('@') ? packageName.split('/') : ['', packageName]
+
+  if (version) {
+    return {
+      name: 'changes-version',
+      params: {
+        org,
+        name,
+        // remove spaces to be correctly resolved by router
+        version: version.replace(/\s+/g, ''),
+      },
+      hash,
+    }
+  }
+
+  return {
+    name: 'changes',
+    params: {
+      org,
+      name,
+    },
+  }
+}