fix(backend): strip /browse suffix from Bitbucket Server webUrl

brendan-kellam · claude · brendan-kellam · commit 53db5ef69974 · 2026-02-11T13:54:45.000-08:00
Bitbucket Server's API returns a self link that includes `/browse` at the end. This caused duplicate `/browse` segments when constructing file URLs and incorrect paths for commit URLs. Fixes #876 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
diff --git a/packages/backend/src/repoCompileUtils.ts b/packages/backend/src/repoCompileUtils.ts
@@ -443,7 +443,12 @@ export const compileBitbucketConfig = async (
             throw new Error(`No ${isServer ? 'self' : 'html'} link found for ${isServer ? 'server' : 'cloud'} repo ${repoName}`);
         }
 
-        return link.href;
+        // @note: Bitbucket Server's self link includes `/browse` at the end.
+        // Strip it so that we can simply append either `/browse`, `/commits`, etc. to
+        // the base URL. 
+        const href = isServer ? link.href.replace(/\/browse\/?$/, '') : link.href;
+
+        return href;
     }
 
     const repos = bitbucketRepos.map((repo) => {

Original file line number	Diff line number	Diff line change
`@@ -443,7 +443,12 @@ export const compileBitbucketConfig = async (`
`443`	`443`	throw new Error(`No ${isServer ? 'self' : 'html'} link found for ${isServer ? 'server' : 'cloud'} repo ${repoName}`);
`444`	`444`	`}`
`445`	`445`
`446`		`- return link.href;`
	`446`	+ // @note: Bitbucket Server's self link includes `/browse` at the end.
	`447`	+ // Strip it so that we can simply append either `/browse`, `/commits`, etc. to
	`448`	`+ // the base URL.`
	`449`	`+ const href = isServer ? link.href.replace(/\/browse\/?$/, '') : link.href;`
	`450`	`+`
	`451`	`+ return href;`
`447`	`452`	`}`
`448`	`453`
`449`	`454`	`const repos = bitbucketRepos.map((repo) => {`