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

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

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>

* docs: add changelog entry for Bitbucket Server URL fix

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* docs: add branch naming and PR conventions to CLAUDE.md

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* docs: add changelog update step to PR conventions

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* docs: clarify MCP changelog location

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

Author: brendan-kellam

CHANGELOG.md

@@ -11,6 +11,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Changed the me-control to render the user's avatar in the top-bar. [#874](https://github.com/sourcebot-dev/sourcebot/pull/874)
 - Moved the "current version" indicator into the "what's new" dropdown. [#874](https://github.com/sourcebot-dev/sourcebot/pull/874)
 
+### Fixed
+- Fixed issue where "Open in Bitbucket" and commit links for Bitbucket Server repos had an extra `/browse` in the URL. [#877](https://github.com/sourcebot-dev/sourcebot/pull/877)
+
 ### Removed
 - Removed the Discord and GitHub buttons from the top corner. [#874](https://github.com/sourcebot-dev/sourcebot/pull/874)
 

CLAUDE.md

@@ -29,3 +29,21 @@ className="border-border bg-card text-foreground text-muted-foreground bg-muted
 // Incorrect
 className="border-[var(--border)] bg-[var(--card)] text-[var(--foreground)]"
 ```
+
+## Branches and Pull Requests
+
+When creating a branch or opening a PR, ask the user for:
+1. The Linear issue ID (if available)
+2. The GitHub issue number (if available)
+
+Branch naming convention:
+- General: `<username>/<branch_name>-<linear_issue_id>`
+- Bug fixes: `<username>/fix-<linear_issue_id>`
+- If no Linear issue ID is available, omit it from the branch name
+
+PR description:
+- If a GitHub issue number was provided, include `Fixes #<github_issue_number>` in the PR description
+
+After the PR is created:
+- Update CHANGELOG.md with an entry under `[Unreleased]` linking to the new PR
+- If the change touches `packages/mcp`, update `packages/mcp/CHANGELOG.md` instead

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) => {