Merge 785a077eff into c3fae1aaab

packages/web/src/app/[domain]/browse/[...path]/page.tsx

@@ -7,22 +7,23 @@ import { Metadata } from "next";
 import { parsePathForTitle} from "@/lib/utils";
 
 type Props = {
-  params: {
+  params: Promise<{
     domain: string;
     path: string[];
-  };
+  }>;
 };
 
-export async function generateMetadata({ params }: Props): Promise<Metadata> {
+export async function generateMetadata({ params: paramsPromise }: Props): Promise<Metadata> {
   let title = 'Browse'; // Default Fallback
 
   try {
+    const params = await paramsPromise;
     title = parsePathForTitle(params.path);
 
   } catch (error) {
     // TODO: Maybe I need to look into a better way of handling this error.
     // for now, it is just a log, fallback tab title and prevents the app from crashing.
-    console.error("Failed to generate metadata title from path:", params.path, error);
+    console.error("Failed to generate metadata title from path:", error);
   }
 
   return {
@@ -44,7 +45,6 @@ export default async function BrowsePage(props: BrowsePageProps) {
     } = params;
 
     const rawPath = _rawPath.join('/');
-    console.log("rawPath:", rawPath);
     const { repoName, revisionName, path, pathType } = getBrowseParamsFromPathParam(rawPath);
 
     return (

packages/web/src/lib/utils.ts

@@ -17,6 +17,7 @@ import { ErrorCode } from "./errorCodes";
 import { NextRequest } from "next/server";
 import { Org } from "@sourcebot/db";
 import { OrgMetadata, orgMetadataSchema } from "@/types";
+import { getBrowseParamsFromPathParam } from "@/app/[domain]/browse/hooks/utils";
 
 export function cn(...inputs: ClassValue[]) {
     return twMerge(clsx(inputs))
@@ -499,50 +500,31 @@ export const isHttpError = (error: unknown, status: number): boolean => {
  * @returns A formatted title string.
  */
 export const parsePathForTitle = (path: string[]): string => {
-  const delimiterIndex = path.indexOf('-');
-  if (delimiterIndex === -1 || delimiterIndex === 0) {
-    return 'Browse';
-  }
+  const pathParam = path.join('/');
 
-  const repoParts = path.slice(1, delimiterIndex);
-  if (repoParts.length === 0) return 'Browse';
+  const { repoName, revisionName, path: filePath, pathType } = getBrowseParamsFromPathParam(pathParam);
 
-  const lastPart = decodeURIComponent(repoParts.pop()!);
-  const [repoNamePart, revision = ''] = lastPart.split('@');
-  const ownerParts = repoParts;
-  const fullRepoName = [...ownerParts, repoNamePart].join('/');
-  const repoAndRevision = `${fullRepoName}${revision ? ` @ ${revision}` : ''}`;
-
-  // Check for file (`blob`) or directory (`tree`) view
-  const blobIndex = path.indexOf('blob');
-  const treeIndex = path.indexOf('tree');
-
-  // Case 1: Viewing a file
-  if (blobIndex !== -1 && path.length > blobIndex + 1) {
-    const encodedFilePath = path[blobIndex + 1];
-    const filePath = decodeURIComponent(encodedFilePath);
+  // Build the base repository and revision string.
+  const cleanRepoName = repoName.split('/').slice(1).join('/'); // Remove the version control system prefix
+  const repoAndRevision = `${cleanRepoName}${revisionName ? ` @ ${revisionName}` : ''}`;
 
+  switch (pathType) {
+    case 'blob': {
+      // For blobs, get the filename from the end of the path.
       const fileName = filePath.split('/').pop() || filePath;
-    
-    // Return a title like: "agents.ts - sourcebot-dev/sourcebot @ HEAD"
       return `${fileName} - ${repoAndRevision}`;
     }
-
-  // Case 2: Viewing a directory
-  if (treeIndex !== -1 && path.length > treeIndex + 1) {
-    const encodedDirPath = path[treeIndex + 1];
-    const dirPath = decodeURIComponent(encodedDirPath);
-    
-    // If we're at the root of the tree, just show the repo name
-    if (dirPath === '/' || dirPath === '') {
+    case 'tree': {
+      // If the path is empty, it's the repo root.
+      if (filePath === '' || filePath === '/') {
         return repoAndRevision;
       }
-
-    // Otherwise, show the directory path
-    // Return a title like: "client/src/store/ - sourcebot-dev/sourcebot @ HEAD"
-    return `${dirPath.endsWith('/') ? dirPath : dirPath + '/'} - ${repoAndRevision}`;
+      // Otherwise, show the directory path.
+      const directoryPath = filePath.endsWith('/') ? filePath : `${filePath}/`;
+      return `${directoryPath} - ${repoAndRevision}`;
     }
-
-  // Case 3: Fallback to the repository root
+    default:
+      // Fallback to just the repository name.
       return repoAndRevision;
   }
+}