fix: update breadcrumb links and tag links to point to correct locations

Author: kmendell

src/lib/components/cards/RepositoryCard.svelte

@@ -8,11 +8,6 @@
 
 	// Fixed helper function to get repository URL path with proper handling of 'library' and nested repos
 	function getRepoPath(repoName: string): string {
-		// Handle 'library' special case
-		if (repoName === 'library') {
-			return '';
-		}
-
 		// For nested repos like 'ofkm/caddy', we need to handle them correctly
 		// The repository name itself should be included in the path
 		return `${repoName}/`;
@@ -27,19 +22,8 @@
 		return fullName;
 	}
 
-	// Helper to construct correct detail URLs
-	function getDetailUrl(repoName: string, imageName: string, tagName: string): string {
-		if (repoName === 'library') {
-			return `/details/${imageName}/${tagName}`;
-		}
-		return `/details/${repoName}/${imageName}/${tagName}`;
-	}
-
 	// Helper to construct correct image URLs
 	function getImageUrl(repoName: string, imageName: string): string {
-		if (repoName === 'library') {
-			return `/details/${imageName}`;
-		}
 		return `/details/${repoName}/${imageName}`;
 	}
 

src/routes/details/[repo]/+page.svelte

@@ -53,7 +53,7 @@
 						<Slash class="h-4 w-4" />
 					</Breadcrumb.Separator>
 					<Breadcrumb.Item>
-						<Breadcrumb.Link class="text-foreground font-medium">
+						<Breadcrumb.Link href="/details/{repoName}" class="text-foreground font-medium">
 							{repoName}
 						</Breadcrumb.Link>
 					</Breadcrumb.Item>

src/routes/details/[repo]/[image]/[tag]/+page.svelte

@@ -76,6 +76,8 @@
 			});
 		}
 	}
+
+	const imageName = data.imageFullName.includes('/') ? data.imageFullName.split('/').pop() : data.imageFullName;
 </script>
 
 <svelte:head>
@@ -116,26 +118,25 @@
 						<Breadcrumb.Separator>
 							<Slash class="h-4 w-4" />
 						</Breadcrumb.Separator>
-						{#if data.repo !== 'library'}
-							<Breadcrumb.Item>
-								<Breadcrumb.Link href="/" class="text-muted-foreground hover:text-foreground transition-colors">
-									{data.repo}
-								</Breadcrumb.Link>
-							</Breadcrumb.Item>
-							<Breadcrumb.Separator>
-								<Slash class="h-4 w-4" />
-							</Breadcrumb.Separator>
-						{/if}
 						<Breadcrumb.Item>
-							<Breadcrumb.Link href="/" class="text-muted-foreground hover:text-foreground transition-colors">
-								{data.imageFullName.includes('/') ? data.imageFullName.split('/').pop() : data.imageFullName}
+							<Breadcrumb.Link href="/details/{data.repo}" class="text-muted-foreground hover:text-foreground transition-colors">
+								{data.repo}
+							</Breadcrumb.Link>
+						</Breadcrumb.Item>
+						<Breadcrumb.Separator>
+							<Slash class="h-4 w-4" />
+						</Breadcrumb.Separator>
+						<Breadcrumb.Item>
+							<Breadcrumb.Link href="/details/{data.repo}" class="text-muted-foreground hover:text-foreground transition-colors">
+								<!-- {data.imageFullName.includes('/') ? data.imageFullName.split('/').pop() : data.imageFullName} -->
+								{imageName}
 							</Breadcrumb.Link>
 						</Breadcrumb.Item>
 						<Breadcrumb.Separator>
 							<Slash class="h-4 w-4" />
 						</Breadcrumb.Separator>
 						<Breadcrumb.Item>
-							<Breadcrumb.Link class="text-foreground font-medium">
+							<Breadcrumb.Link href="/details/{data.repo}/{imageName}/{currentTag.name}" class="text-foreground font-medium">
 								{currentTag.name}
 							</Breadcrumb.Link>
 						</Breadcrumb.Item>