fix: conform merged main changes to refactored code

[reachraza]

Adapt recent main branch changes (orphaned file preview fix, status indicator styles) to work with the refactored store/hook architecture.

Summary by CodeRabbit

• Bug Fixes

  • Prevented file preview from displaying in terminal mode to avoid orphaned UI elements.

• Refactor

  • Simplified GroupChatPanel API by removing unused groups property.
  • Improved session mode switching logic with better cleanup when transitioning between AI and terminal modes.
  • Refined session list styling structure for better maintainability.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4b01abc and 0dc11e5.

📒 Files selected for processing (5)

• src/renderer/App.tsx
• src/renderer/components/GroupChatPanel.tsx
• src/renderer/components/MainPanel.tsx
• src/renderer/components/SessionList.tsx
• src/renderer/hooks/remote/useRemoteIntegration.ts

💤 Files with no reviewable changes (2)

• src/renderer/components/GroupChatPanel.tsx
• src/renderer/App.tsx

---

📝 Walkthrough

Walkthrough

Five files are updated across components and hooks. The primary change removes the groups prop from GroupChatPanel and its usage in App.tsx. Additionally, file preview rendering is guarded by AI mode in MainPanel.tsx, session item styling is refactored in SessionList.tsx, and a helper function is extracted in useRemoteIntegration.ts to centralize input mode synchronization logic.

Changes

Cohort / File(s)	Summary
GroupChatPanel Prop Removal src/renderer/App.tsx, src/renderer/components/GroupChatPanel.tsx	Removes the groups prop from the GroupChatPanelProps interface and its corresponding usage in MaestroConsoleInner, eliminating prop-based group data passing to the panel.
File Preview Rendering Guard src/renderer/components/MainPanel.tsx	Introduces canShowFilePreview boolean to gate file preview rendering based on AI mode (activeSession.inputMode === 'ai'), replacing inline mode checks in render branches.
Session Item Styling Refactor src/renderer/components/SessionList.tsx	Refactors session item circle styling by moving opacity control to outer style and restructuring conditional border/background logic, preserving visual behavior with simplified structure.
Input Mode Synchronization Helper src/renderer/hooks/remote/useRemoteIntegration.ts	Extracts new applyInputModeToSession() helper function to centralize inputMode updates and activeFileTabId clearing when switching to terminal mode, replacing inline synchronization logic in two code paths.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• fix: prevent orphaned file preview when tab bar is hidden #456: Directly modifies the GroupChatPanel groups prop API in both its declaration and usage, opposing the removal in this PR.
• refactor: eliminate prop-drilling in core panels via direct Zustand store subscriptions #452: Refactors group-chat UI prop-drilling by migrating group data to store-driven access, addressing the same coupling concern as the groups prop removal.

Suggested reviewers

• pedramamini

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 33.33% 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 describes the main change: adapting recent main branch changes (orphaned file preview fix and status indicator styling) to work with the refactored code architecture.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch code-refactor

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[greptile-apps]

Greptile Summary

Conforms orphaned file preview fix (#456) and status indicator polish from main to the refactored store architecture. Removes unused groups prop from GroupChatPanel, adopts canShowFilePreview guard in MainPanel, extracts applyInputModeToSession() helper in useRemoteIntegration, and refactors status dot opacity logic in SessionList.

Changes are surgical and follow the established refactor patterns — no logic bugs introduced.

Confidence Score: 5/5

• Safe to merge — clean refactor conformance with no logic changes
• All changes are straightforward adaptations of main branch fixes to the refactored codebase. The groups prop removal is correct (eliminated by Zustand migration), file preview guards match the original fix pattern, and the extracted helper improves code organization without changing behavior.
• No files require special attention

Important Files Changed

Filename	Overview
src/renderer/App.tsx	Removed unused groups prop from GroupChatPanel — correct cleanup after refactor eliminated prop-drilling
src/renderer/components/GroupChatPanel.tsx	Removed unused groups prop and import — aligns with refactored store architecture
src/renderer/components/MainPanel.tsx	Applied orphaned file preview fix pattern by introducing canShowFilePreview guard variable
src/renderer/components/SessionList.tsx	Refactored status indicator styles to use spread syntax for cleaner opacity handling
src/renderer/hooks/remote/useRemoteIntegration.ts	Extracted applyInputModeToSession helper to centralize terminal mode file preview cleanup logic

_{Last reviewed commit: 0dc11e5}