Add tracker labels support to PRD creation workflow#208
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. WalkthroughThis PR adds tracker label support to the PRD creation workflow. The Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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. Comment |
Codecov Report❌ Patch coverage is
❌ Your patch status has failed because the patch coverage (5.26%) is below the target coverage (50.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #208 +/- ##
==========================================
- Coverage 45.17% 44.87% -0.31%
==========================================
Files 84 84
Lines 24299 24466 +167
==========================================
+ Hits 10977 10978 +1
- Misses 13322 13488 +166
🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@src/tui/components/PrdChatApp.tsx`:
- Around line 409-420: The callback handleTrackerSelect captures trackerLabels
but the dependency array omits it, causing stale closures; update the
useCallback dependencies to include trackerLabels (and any other used state) so
the function always sees current labels, and also make the deduplication that
builds allLabels case-insensitive by normalizing entries (e.g., toLowerCase)
before filtering out 'ralph' and removing duplicates so variants like 'Ralph' or
'RALPH' are handled; locate symbols labelsInstruction and handleTrackerSelect to
apply these changes.
The create-prd command loaded tracker config but never passed the trackerOptions.labels to the PrdChatApp component. Issues created via the beads skill were missing the labels configured during ralph-tui setup. This fix loads labels from config and includes them as an instruction in the beads skill prompt, matching the behavior of the convert command. https://claude.ai/code/session_01KdwvcSwmwRCScMm1UpGFtK
…ive dedup The handleTrackerSelect callback captured trackerLabels but the useCallback dependency array omitted it, risking stale closures. Also normalize labels case-insensitively when deduplicating against the always-included 'ralph' entry so that 'Ralph' or 'RALPH' variants don't produce duplicates in the --labels flag.
Extract parseTrackerLabels and buildBeadsLabelsInstruction into tested helpers to cover new PR lines: config label loading (string/array formats, trimming, filtering) and case-insensitive dedup logic that prevents duplicate 'ralph' variants in the --labels flag.
a218ebf to
e0afd67
Compare
…dA43O Add tracker labels support to PRD creation workflow
Summary
This PR adds support for applying tracker labels to issues created during the PRD (Product Requirements Document) creation workflow. Labels are now loaded from the config file's
trackerOptions.labelssetting and automatically applied to all beads issues (epics and child tasks).Key Changes
trackerLabelsfield to pass labels through the command execution pipelinetrackerOptions.labelsfrom stored config, supporting both comma-separated strings and array formatstrackerLabelsprop to receive labels from parent--labelsflag into allbd createandbr createcommands when labels are configuredImplementation Details
--labelsflag is injected into the AI prompt to ensure consistent application across all created issuesSummary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.