From 53db5ef6997484e89d94cf911cb9ca4b334959d8 Mon Sep 17 00:00:00 2001 From: bkellam Date: Wed, 11 Feb 2026 13:54:45 -0800 Subject: [PATCH 1/5] 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 --- packages/backend/src/repoCompileUtils.ts | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/packages/backend/src/repoCompileUtils.ts b/packages/backend/src/repoCompileUtils.ts index 12b2a6caf..7ff12c622 100644 --- 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) => { From 1b1ae2258049d02e115e17a195f5d007d2c335e1 Mon Sep 17 00:00:00 2001 From: bkellam Date: Wed, 11 Feb 2026 13:55:50 -0800 Subject: [PATCH 2/5] docs: add changelog entry for Bitbucket Server URL fix Co-Authored-By: Claude Opus 4.5 --- CHANGELOG.md | 3 +++ 1 file changed, 3 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index e28ac8b48..438911ffe 100644 --- a/CHANGELOG.md +++ b/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) From 6c435530b025865d54d3c7c8cfc66563230d476e Mon Sep 17 00:00:00 2001 From: bkellam Date: Wed, 11 Feb 2026 14:01:10 -0800 Subject: [PATCH 3/5] docs: add branch naming and PR conventions to CLAUDE.md Co-Authored-By: Claude Opus 4.5 --- CLAUDE.md | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/CLAUDE.md b/CLAUDE.md index 2ff8c9615..5836074fb 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -29,3 +29,17 @@ 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: `/-` +- Bug fixes: `/fix-` +- If no Linear issue ID is available, omit it from the branch name + +PR description: +- If a GitHub issue number was provided, include `Fixes #` in the PR description From ac2b623f5f972bb21ad341b4754150918f941fd2 Mon Sep 17 00:00:00 2001 From: bkellam Date: Wed, 11 Feb 2026 14:02:06 -0800 Subject: [PATCH 4/5] docs: add changelog update step to PR conventions Co-Authored-By: Claude Opus 4.5 --- CLAUDE.md | 3 +++ 1 file changed, 3 insertions(+) diff --git a/CLAUDE.md b/CLAUDE.md index 5836074fb..d0b888853 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -43,3 +43,6 @@ Branch naming convention: PR description: - If a GitHub issue number was provided, include `Fixes #` in the PR description + +After the PR is created: +- Update CHANGELOG.md with an entry under `[Unreleased]` linking to the new PR From 8f5ad1ca87575c0d31cd4b1742e5bf8e74e17b0a Mon Sep 17 00:00:00 2001 From: bkellam Date: Wed, 11 Feb 2026 14:03:03 -0800 Subject: [PATCH 5/5] docs: clarify MCP changelog location Co-Authored-By: Claude Opus 4.5 --- CLAUDE.md | 1 + 1 file changed, 1 insertion(+) diff --git a/CLAUDE.md b/CLAUDE.md index d0b888853..85e88aaad 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -46,3 +46,4 @@ 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