Skip to content

Commit 7365ff2

Browse files
committed
fix(diff): update package URLs to use /package/ prefix
- SidebarPanel.vue: Replace bare `/${dep.name}` dependency links with `packageRoute(dep.name)` which resolves to /package/:org?/:name - ViewerPanel.vue: Update code browser URLs from /code/ alias to canonical /package-code/ path - diff/[...path].vue: Remove duplicated local packageRoute() function and use the shared utility from ~/utils/router instead
1 parent 1820691 commit 7365ff2

File tree

3 files changed

+7
-15
lines changed

3 files changed

+7
-15
lines changed

app/components/diff/SidebarPanel.vue

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
<script setup lang="ts">
22
import type { CompareResponse, FileChange } from '#shared/types'
3+
import { packageRoute } from '~/utils/router'
34
45
const props = defineProps<{
56
compare: CompareResponse
@@ -141,7 +142,7 @@ function handleFileSelect(file: FileChange) {
141142
/>
142143

143144
<NuxtLink
144-
:to="`/${dep.name}`"
145+
:to="packageRoute(dep.name)"
145146
class="font-mono hover:text-fg transition-colors truncate min-w-0"
146147
>
147148
{{ dep.name }}

app/components/diff/ViewerPanel.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -208,7 +208,7 @@ function formatBytes(bytes: number | undefined): string {
208208
209209
// Build code browser URL
210210
function getCodeUrl(version: string): string {
211-
return `/code/${props.packageName}/v/${version}/${props.file.path}`
211+
return `/package-code/${props.packageName}/v/${version}/${props.file.path}`
212212
}
213213
</script>
214214

app/pages/diff/[...path].vue

Lines changed: 4 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
<script setup lang="ts">
22
import type { CompareResponse, FileChange } from '#shared/types'
3+
import { packageRoute } from '~/utils/router'
34
45
definePageMeta({
56
name: 'diff',
@@ -109,16 +110,6 @@ const toVersionUrlPattern = computed(() => {
109110
return `/diff/${packageName.value}/v/${fromVersion.value}...{version}`
110111
})
111112
112-
function packageRoute(ver?: string | null) {
113-
const segments = packageName.value.split('/')
114-
const org = segments.length > 1 ? segments[0] : undefined
115-
const name = segments.length > 1 ? segments[1]! : segments[0]!
116-
if (ver) {
117-
return { name: 'package-version' as const, params: { org, name, version: ver } }
118-
}
119-
return { name: 'package' as const, params: { org, name } }
120-
}
121-
122113
useSeoMeta({
123114
title: () => {
124115
if (fromVersion.value && toVersion.value) {
@@ -139,7 +130,7 @@ useSeoMeta({
139130
<!-- Package info -->
140131
<div class="flex items-center gap-2 mb-3 flex-wrap min-w-0">
141132
<NuxtLink
142-
:to="packageRoute()"
133+
:to="packageRoute(packageName)"
143134
class="font-mono text-lg font-medium hover:text-fg transition-colors min-w-0 truncate"
144135
>
145136
{{ packageName }}
@@ -186,7 +177,7 @@ useSeoMeta({
186177
<p class="text-fg-muted mb-4">
187178
Invalid comparison URL. Use format: /diff/package/v/from...to
188179
</p>
189-
<NuxtLink :to="packageRoute()" class="btn">Go to package</NuxtLink>
180+
<NuxtLink :to="packageRoute(packageName)" class="btn">Go to package</NuxtLink>
190181
</div>
191182

192183
<!-- Loading state -->
@@ -198,7 +189,7 @@ useSeoMeta({
198189
<!-- Error state -->
199190
<div v-else-if="compareStatus === 'error'" class="container py-20 text-center" role="alert">
200191
<p class="text-fg-muted mb-4">Failed to compare versions</p>
201-
<NuxtLink :to="packageRoute()" class="btn">Back to package</NuxtLink>
192+
<NuxtLink :to="packageRoute(packageName)" class="btn">Back to package</NuxtLink>
202193
</div>
203194

204195
<!-- Comparison content -->

0 commit comments

Comments
 (0)