[SOU-301] Fix branch filtering to use short names instead of refs/heads by msukkari · Pull Request #806 · sourcebot-dev/sourcebot · GitHub

Problem

Searching on filtered branches was returning no results because branch names were being stored with the refs/heads/ prefix when they should have been stored as short names only.

Solution

Remove the unnecessary mapping that prefixed branch names with refs/heads/ in the branch filtering logic. The zoekt index stores branch references as short names (e.g., main, develop), so the filtering logic should match this format.

Changes

Removed .map((branch) => \refs/heads/${branch}`)from branch filtering inrepoIndexManager.ts`
Branch names are now correctly passed as short names to match indexed branch references

This fixes the issue where the rev: query syntax was not finding any results when filtering on indexed branches.

Fixes #800

View Niteshift Task

Summary by CodeRabbit

Bug Fixes
- Improved repository branch selection processing for indexing operations.

✏️ Tip: You can customize this high-level summary in your review settings.

…ering

Walkthrough

Removed a mapping step in the branch selection flow for indexing in repoIndexManager.ts. Matching branches are now appended to revisions as raw branch names instead of being converted to full refs/heads/ format, altering the composition of the revisions array.

Changes

Cohort / File(s)	Summary
Branch revision formatting packages/backend/src/repoIndexManager.ts	Removed mapping operation that converted matching branch names to full refs/heads/ refs; branches now appended as raw names

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

Possibly related PRs

fix(web): Fixed issue where Sourcebot would not index the new default branch when changed #789: Also modifies repoIndexManager.ts revisions list construction; changes default branch selection logic alongside this branch name formatting change.

🚥 Pre-merge checks | ✅ 3 ✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the main change: removing refs/heads prefixes and using short branch names instead, which directly matches the core modification in repoIndexManager.ts.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches

📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@msukkari your pull request is missing a changelog!

I don't think this is a correct change. #800 is referring to the filter on the /repos page. We need to have refs/heads/ and refs/tags/ to allow the user to distinguish between filtering a branch and a tag (e.g., in the event where a tag and branch are named the same thing). This is documented here: https://docs.sourcebot.dev/docs/features/search/multi-branch-indexing#search-syntax

The current suggested change would still be a welcomed change.

Making sure rev: maps to the short name of the default branch makes it easier for non technical people to understand which branch they are looking at. They might not know what refs/heads stands for, but they would know what main, feature/, etc are as that is generally what is displayed in all code hosting platforms
If you specify, which branches to index in the SB configs, like this

    "revisions": {
        "branches":
          "main"
          "release/*"
        ]

it indexes main and ref/head separately (iirc)
3. I am working on adding the feature to be able to switch revisions directly from the code tree/preview panels. and this would be useful in my efforts. (and goes hand in hand with 1. above)

Making sure rev: maps to the short name of the default branch makes it easier for non technical people to understand which branch they are looking at.

I believe zoekt does a substring match. For example:

I would like to keep it such that a user can explicitly specify a refs/heads or refs/tags like this. In the UI, I think we can just strip the refs/heads & refs/tags

2.If you specify, which branches to index in the SB configs, like this it indexes main and ref/head separately (iirc)

This actually appears to be a bug. Tracked in #808

I am working on adding the feature to be able to switch revisions directly from the code tree/preview panels. and this would be useful in my efforts. (and goes hand in hand with 1. above)

Cool! I assume you mean something like a dropdown selector that lets users switch between branches? Created #810 to track this if you want to discuss over there.

Fixed in #851

refactor: remove unnecessary refs/heads prefix mapping in branch filt…

a44dba4

…ering

msukkari marked this pull request as ready for review January 28, 2026 18:31

brendan-kellam reviewed Jan 28, 2026

View reviewed changes

brendan-kellam closed this Feb 4, 2026

                               allBranches
-                                  .filter((branch) => micromatch.isMatch(branch, branchGlobs))
-                                  .map((branch) => `refs/heads/${branch}`);
+                                  .filter((branch) => micromatch.isMatch(branch, branchGlobs));

[SOU-301] Fix branch filtering to use short names instead of refs/heads by msukkari · Pull Request #806 · sourcebot-dev/sourcebot · GitHub

Uh oh!

[SOU-301] Fix branch filtering to use short names instead of refs/heads - #806

[SOU-301] Fix branch filtering to use short names instead of refs/heads#806
msukkari wants to merge 1 commit into
mainfrom
msukkari/fix-filtered-branch-search-6i2p4p

msukkari commented Jan 28, 2026 •
edited by brendan-kellam
Loading

Uh oh!

Uh oh!

coderabbitai Bot commented Jan 28, 2026 •
edited
Loading

Uh oh!

Uh oh!

github-actions Bot commented Jan 28, 2026

Uh oh!

brendan-kellam Jan 28, 2026

Uh oh!

CyberT17 Jan 28, 2026

Uh oh!

brendan-kellam Jan 28, 2026

Uh oh!

brendan-kellam commented Feb 4, 2026 •
edited
Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

[SOU-301] Fix branch filtering to use short names instead of refs/heads by msukkari · Pull Request #806 · sourcebot-dev/sourcebot · GitHub

Uh oh!

[SOU-301] Fix branch filtering to use short names instead of refs/heads - #806

[SOU-301] Fix branch filtering to use short names instead of refs/heads#806msukkari wants to merge 1 commit intomainsourcebot-dev/sourcebot:mainfrom msukkari/fix-filtered-branch-search-6i2p4psourcebot-dev/sourcebot:msukkari/fix-filtered-branch-search-6i2p4pCopy head branch name to clipboard

Conversation

msukkari commented Jan 28, 2026 • edited by brendan-kellam Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Problem

Solution

Changes

Summary by CodeRabbit

Uh oh!

coderabbitai Bot commented Jan 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Uh oh!

github-actions Bot commented Jan 28, 2026

Uh oh!

brendan-kellam Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

CyberT17 Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

brendan-kellam Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

brendan-kellam commented Feb 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

[SOU-301] Fix branch filtering to use short names instead of refs/heads#806
msukkari wants to merge 1 commit into
mainfrom
msukkari/fix-filtered-branch-search-6i2p4p

msukkari commented Jan 28, 2026 •
edited by brendan-kellam
Loading

coderabbitai Bot commented Jan 28, 2026 •
edited
Loading

brendan-kellam commented Feb 4, 2026 •
edited
Loading