fix(channel): restore thread reply fetch debounce#5040
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (6)
📝 WalkthroughSummary by CodeRabbit
WalkthroughChannel threads now receive the channel find-bar state and use a dedicated reactive gate for reply fetching. The gate immediately enables targeted replies while the find bar is open and debounces expanded, targeted, or high-count thread fetches. The deferred gate is exported and cancels pending timers on scope disposal. Tests cover debounce timing, immediate find-bar targeting, and cleanup on unmount. 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
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 |
What changed
Why
PR #3305 unintentionally removed the existing reply-fetch debounce while optimizing in-channel search. That allowed transiently mounted virtualized threads to enable reply queries immediately. Normal expansion and non-search deep-link navigation are now debounced; Cmd+F retains immediate targeted-reply navigation.
Validation
bunx vitest run src/features/channel/Thread/tests— 27 tests passedbun run check— type-check and Biome passed