Skip to content

Add --limit 0 flag to bypass default 50 result limit in beads queries#257

Merged
subsy merged 2 commits intomainfrom
claude/investigate-issue-233-fHTGH
Feb 3, 2026
Merged

Add --limit 0 flag to bypass default 50 result limit in beads queries#257
subsy merged 2 commits intomainfrom
claude/investigate-issue-233-fHTGH

Conversation

@subsy
Copy link
Owner

@subsy subsy commented Feb 3, 2026

Summary

This PR fixes an issue where epics with more than 50 tasks would have their tasks truncated due to the default result limit in the beads tracker tools. By adding the --limit 0 flag to all list commands, we ensure all results are returned regardless of count.

Changes

  • beads-bv plugin: Added --limit 0 to getEpicChildrenIds() query
  • beads-rust plugin: Added --limit 0 to getTasks() and getEpics() queries
  • beads plugin: Added --limit 0 to getTasks(), getEpics(), and epic children queries
  • Tests: Updated all test expectations to verify --limit 0 is included in command arguments, and added new test cases to explicitly verify the flag is used to bypass the default 50 result limit

Implementation Details

The --limit 0 flag is a standard way to disable result limits in the beads tools (both bd and br). Without this flag, queries default to returning only 50 results, which causes data loss for large epics. This fix ensures complete data retrieval across all tracker operations.

Fixes: #233

https://claude.ai/code/session_01BLc8LHLAkhKT32uxQnUYxb

Summary by CodeRabbit

Release Notes

  • Improvements
    • Updated task and epic retrieval to fetch all available results instead of being limited to a default set, ensuring comprehensive data visibility when browsing tasks and epics in tracker plugins.

…imit

The beads CLI tools (bd/br) have a default limit of 50 results when
listing tasks. This caused epics with more than 50 tasks to have their
tasks truncated, making many tasks invisible in the TUI.

This fix adds --limit 0 to all list commands across the three beads
tracker plugins to retrieve all results regardless of the CLI default.

Fixes #233

https://claude.ai/code/session_01BLc8LHLAkhKT32uxQnUYxb
@vercel
Copy link

vercel bot commented Feb 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ralph-tui Ready Ready Preview, Comment Feb 3, 2026 0:50am

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 3, 2026

Walkthrough

These changes add --limit 0 flags to beads CLI command invocations across multiple tracker plugins to retrieve unlimited results instead of respecting the default 50-item limit, enabling full task and epic loading for large datasets.

Changes

Cohort / File(s) Summary
Beads Tracker
src/plugins/trackers/builtin/beads/index.ts, src/plugins/trackers/builtin/beads.test.ts
Production code updated to append --limit 0 to bd list commands in getTasks, getEpics, and getPrdContext. New tests added verifying these commands include the flag for unlimited result retrieval.
Beads Rust Tracker
src/plugins/trackers/builtin/beads-rust/index.ts, src/plugins/trackers/builtin/beads-rust/index.test.ts
Production code updated with --limit 0 in getTasks and getEpics methods. Test expectations across multiple scenarios (parent filtering, label/status filtering, epic retrieval) updated to verify the flag is present in br list invocations. Inline comments added explaining the bypass of default 50-result limit.
Beads BV Tracker
src/plugins/trackers/builtin/beads-bv/index.ts
Added --limit 0 flag to bd command in getEpicChildrenIds to list all epic children without result limit.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely describes the main change: adding --limit 0 flag to beads queries to bypass the default 50 result limit, which is the primary objective of this PR.
Linked Issues check ✅ Passed The PR fully implements the primary solution from issue #233 by adding --limit 0 flag to all beads list queries (getTasks, getEpics, getEpicChildrenIds) across beads, beads-rust, and beads-bv plugins, directly addressing the truncation issue.
Out of Scope Changes check ✅ Passed All changes are directly related to the linked issue #233 objective. The PR consistently adds --limit 0 to beads CLI invocations and updates tests accordingly, with no unrelated modifications.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ 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 claude/investigate-issue-233-fHTGH

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.

@codecov
Copy link

codecov bot commented Feb 3, 2026

Codecov Report

❌ Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 45.70%. Comparing base (7268090) to head (ec13faa).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
src/plugins/trackers/builtin/beads-bv/index.ts 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #257      +/-   ##
==========================================
+ Coverage   45.49%   45.70%   +0.20%     
==========================================
  Files          91       91              
  Lines       27442    27442              
==========================================
+ Hits        12486    12542      +56     
+ Misses      14956    14900      -56     
Files with missing lines Coverage Δ
src/plugins/trackers/builtin/beads-rust/index.ts 70.62% <100.00%> (ø)
src/plugins/trackers/builtin/beads/index.ts 59.04% <100.00%> (+8.58%) ⬆️
src/plugins/trackers/builtin/beads-bv/index.ts 12.57% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@subsy subsy merged commit 3735f7e into main Feb 3, 2026
9 checks passed
@subsy subsy deleted the claude/investigate-issue-233-fHTGH branch February 3, 2026 12:51
sakaman pushed a commit to sakaman/ralph-tui that referenced this pull request Feb 15, 2026
…fHTGH

Add --limit 0 flag to bypass default 50 result limit in beads queries
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Beads tracker only loads 50 tasks due to default limit

2 participants

Comments