fix: branch listing shows all active branches

[dkrizan]

Summary

• Remove client-side filter in listBranches that was excluding branches with a mergedAt date — all branches returned by the endpoint are now shown
• Remove activeOnly query parameter from fetchBranches so the endpoint returns all branches

Summary by CodeRabbit

• Bug Fixes
  • Updated branch listing to display all branches, including previously merged branches.
  • Modified branch fetching to retrieve all available branches without exclusions.

[coderabbitai]

Walkthrough

Removed activeOnly: true filter from branch fetching query parameters and eliminated the in-function filter that excluded merged branches, allowing both functions to now include all branches in their results instead of filtering out completed ones.

Changes

Cohort / File(s)	Summary
Branch filtering behavior src/utils/branch.ts	Removed activeOnly: true parameter from fetchBranches query and eliminated mergedAt timestamp filter in listBranches, causing both functions to now return all branches without excluding merged ones.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 All branches now welcome, no merges left behind,
The filters have lifted, a cleaner design!
Active or done, each branch gets its say,
No more exclusions—hooray, hooray! 🌿

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check	⚠️ Warning	The title mentions 'all active branches' but the PR actually removes filters to show all branches including merged ones, which contradicts the title's implication of only active branches.	Consider revising the title to 'fix: branch listing shows all branches' or 'fix: remove filters to show all branches including merged ones' to accurately reflect the actual changes.

✅ Passed checks (1 passed)

Check name	Status	Explanation
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

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch dkrizan/branching-list-fix

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

🎉 This PR is included in version 2.16.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀