fix(web): remove Home status strip from active chats - #2677
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe global assistant no longer displays ChangesHome guidance layout
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The reviewed change has no identified merge-blocking risk. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
Summary
Verification
./node_modules/.bin/vitest run apps/web/src/components/layout/middle-content/page-views/dashboard/__tests__/GlobalAssistantView.test.tsx apps/web/src/components/layout/middle-content/page-views/dashboard/__tests__/global-assistant-home-guidance.guard.test.ts --pool=forks --maxWorkers=1 --minWorkers=1bun run --filter web lint(passes; existing warnings only)NODE_OPTIONS='--max-old-space-size=6144' bun run --filter web typecheck(timed out in sandbox after 120 seconds)TDD: added the guard test before removing the active-conversation strip; the first runnable attempt was blocked by missing dependencies, then the expectation initially failed against the pre-fix source.
Summary by CodeRabbit
User Experience
Tests