refactor(server): improve version link parsing

Author: MickaelCa

src/server/server_config.py

@@ -42,14 +42,24 @@ def get_version_info() -> dict[str, str]:
     version = VERSION
     repo_url = REPOSITORY_URL.rstrip("/")
 
-    # Check if version looks like a tag (doesn't contain branch-commit pattern)
-    if version != "unknown" and "-" in version and len(version.split("-")) >= MIN_BRANCH_COMMIT_PARTS:
-        # This looks like branch-commit format (e.g., "main-abc1234")
+    # Check if version contains PR number (e.g., "pr-123", "pull-456")
+    if version != "unknown" and ("-" in version):
         parts = version.split("-")
         if len(parts) >= MIN_BRANCH_COMMIT_PARTS:
-            # Take the last part as commit hash
-            commit_hash = parts[-1]
-            version_link = f"{repo_url}/commit/{commit_hash}"
+            # Check if first part indicates a PR
+            if parts[0].lower() in ("pr", "pull"):
+                # Extract PR number from the second part
+                try:
+                    pr_number = int(parts[1])
+                    version_link = f"{repo_url}/pull/{pr_number}"
+                except (ValueError, IndexError):
+                    # If PR number is invalid, fallback to main branch
+                    version_link = f"{repo_url}/tree/main"
+            else:
+                # This looks like branch-commit format (e.g., "main-abc1234")
+                # Take the last part as commit hash
+                commit_hash = parts[-1]
+                version_link = f"{repo_url}/commit/{commit_hash}"
         else:
             # Fallback to main branch
             version_link = f"{repo_url}/tree/main"

src/server/templates/components/footer.jinja

@@ -18,13 +18,5 @@
                                 'Discord') }}
             </div>
         </div>
-        {# Version information row #}
-        <div class="mt-2 text-center text-xs text-gray-600">
-            <span>Version:</span>
-            <a href="{{ version_link }}"
-               target="_blank"
-               rel="noopener noreferrer"
-               class="text-blue-600 hover:text-blue-800 underline">{{ version }}</a>
-        </div>
     </div>
 </footer>

src/server/templates/components/navbar.jinja

@@ -3,11 +3,17 @@
         <div class="flex justify-between items-center h-16">
             {# Logo #}
             <div class="flex items-center gap-4">
-                <h1 class="text-2xl font-bold tracking-tight">
-                    <a href="/" class="hover:opacity-80 transition-opacity">
-                        <span class="text-gray-900">Git</span><span class="text-[#FE4A60]">ingest</span>
-                    </a>
-                </h1>
+                <div class="flex flex-col gap-px">
+                    <h1 class="text-2xl font-bold tracking-tight mb-0">
+                        <a href="/" class="hover:opacity-80 transition-opacity">
+                            <span class="text-gray-900">Git</span><span class="text-[#FE4A60]">ingest</span>
+                        </a>
+                    </h1>
+                    <a href="{{ version_link }}"
+                       target="_blank"
+                       rel="noopener noreferrer"
+                       class="text-blue-600 hover:text-blue-800 text-sm self-end no-underline">{{ version }}</a>
+                </div>
             </div>
             {# Navigation with updated styling #}
             <nav class="flex items-center space-x-6">
@@ -36,5 +42,4 @@
         </div>
     </div>
 </header>
-{# Load GitHub stars script #}
 <script defer src="/static/js/navbar.js"></script>