fix(app): hide child sessions from sidebar root window#478
Conversation
|
Warning Rate limit exceeded
To continue reviewing without waiting, purchase usage credits in the billing tab. ⌛ 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. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
There was a problem hiding this comment.
Code Review
This pull request updates the session window logic to ensure that only root sessions (those without a parentID) are displayed in the top-level sidebar. It introduces a rootSessions helper function and updates buildPawworkSessionWindow to filter normal, pinned, and active sessions. Corresponding tests have been added to verify this behavior. The review feedback suggests improving the rootSessions helper with defensive checks to handle potentially null or undefined items in the session array.
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
9b19dfa to
07fc60d
Compare
Summary
Filter child/subagent sessions out of the PawWork sidebar root session window. Current child sessions still appear under their parent session through the existing
showChildpath.Why
When the active session is a subagent child session,
buildPawworkSessionWindowwas merging that active child into the top-level sidebar rows. The parent row also renders the current child below it, so the same subagent session title appeared twice.Related Issue
No GitHub issue yet. Reported from v2026.5.5 sidebar behavior.
Human Review Status
Pending. A human should make the final merge decision after reviewing the final diff and verification evidence.
Review Focus
Check that root sessions still stay visible when pinned or active, while child sessions from normal, pinned, or active inputs do not become top-level sidebar rows.
Risk Notes
Low. This changes only the sidebar window helper. It does not change session storage, subagent creation, titles, routing, or the child-row rendering path.
How To Verify
Screenshots or Recordings
Not captured. The changed behavior is covered at the sidebar window data-helper level; Electron manual verification was not run.
Checklist
dev, and my PR title and commit messages use Conventional Commits in English