Skip to content

Comments

Adding Scrollbar and Improving vite config#154

Merged
leonvanzyl merged 1 commit intoAutoForgeAI:masterfrom
paperlinguist:master
Feb 4, 2026
Merged

Adding Scrollbar and Improving vite config#154
leonvanzyl merged 1 commit intoAutoForgeAI:masterfrom
paperlinguist:master

Conversation

@paperlinguist
Copy link
Contributor

@paperlinguist paperlinguist commented Feb 3, 2026

Add scrollbar to Mission Control panel
Improve code-splitting to reduce bundle size (removes chunk size warning)

Summary by CodeRabbit

  • Style

    • Refined scrolling behavior in Agent Mission Control by improving overflow handling for expanded and collapsed content states.
  • Chores

    • Optimized build configuration by reorganizing vendor code chunks, separating UI components and utilities for improved code splitting and performance.

 Add scrollbar to Mission Control panel
Improve code-splitting to reduce bundle size
@coderabbitai
Copy link

coderabbitai bot commented Feb 3, 2026

📝 Walkthrough

Walkthrough

Two minor updates were made: CSS overflow behavior adjusted in AgentMissionControl to conditionally apply scrolling based on expand/collapse state, and vendor chunk configuration in vite.config refactored to split UI packages into separate bundles for optimized code splitting.

Changes

Cohort / File(s) Summary
CSS Overflow Adjustment
ui/src/components/AgentMissionControl.tsx
Modified overflow handling in content container to use overflow-y-auto when expanded and overflow-hidden when collapsed, enabling scrolling behavior based on expand state.
Vendor Chunk Refactoring
ui/vite.config.ts
Split monolithic "vendor-ui" chunk into two separate bundles: "vendor-radix" for Radix UI packages and "vendor-utils" for utilities (lucide-react, canvas-confetti, class-variance-authority, clsx, tailwind-merge).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A scroll here, a bundle there,
The code now flows with graceful care,
Split the chunks in ordered ways,
Let the UI dance through all its days!

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately captures both main changes: adding scrollbar functionality to AgentMissionControl and reorganizing the Vite config for better chunk management.

✏️ 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

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Reviews will stop working after February 8, 2026 if the new IP is not added to your allowlist.


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.

@leonvanzyl leonvanzyl merged commit f6510b4 into AutoForgeAI:master Feb 4, 2026
3 checks passed
@leonvanzyl
Copy link
Collaborator

Thank you!

CoreAspectStu added a commit to CoreAspectStu/autocoder-custom that referenced this pull request Feb 9, 2026
Created comprehensive E2E test suite to verify ChatTab handles 404 errors
and missing conversations gracefully, as implemented in Features AutoForgeAI#146, AutoForgeAI#147,
and AutoForgeAI#148.

Test File: ui/e2e/chattab-errors.spec.ts (599 lines, 10 test scenarios)

Test Scenarios:
1. ChatTab loads without crashes when project is selected
2. No 404 errors when conversation list is empty
3. No TypeError when conversation is missing or deleted
4. Console shows warnings (not errors) for 404s
5. ChatTab recovers gracefully from API errors
6. Dev Mode toggle resets ChatTab state correctly
7. No console errors when opening browser DevTools
8. No errors on rapid project switching
9. Empty state message displayed when no conversations
10. Validate network requests for conversation endpoints

Key Features:
- Helper functions for project selection, ChatTab switching, console tracking
- Comprehensive error detection (404, TypeError, crashes)
- Screenshot assertions for visual verification (13 screenshots)
- Network request validation
- Console error tracking
- Graceful degradation testing

Related Features:
- Feature AutoForgeAI#146: ChatTab 404 Error Handling
- Feature AutoForgeAI#147: Defensive Programming for ChatTab
- Feature AutoForgeAI#148: ChatTab State Reset on Mode Switch

Verification:
- All 10 required test scenarios implemented
- Follows existing E2E test patterns (conversation-history.spec.ts)
- Uses @playwright/test framework
- Can run headless or with UI
- Added to E2E test suite

How to Run:
  cd /home/stu/projects/autocoder/ui
  npm run test:e2e -- chattab-errors.spec.ts
  npm run test:e2e:ui  (interactive mode)

Co-Authored-By: Claude <noreply@anthropic.com>
CoreAspectStu pushed a commit to CoreAspectStu/autocoder-custom that referenced this pull request Feb 9, 2026
Adding Scrollbar and Improving vite config
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.

2 participants