Repository navigation
Separate mobile host card metadata - #8819
Conversation
Co-authored-by: Orca <help@stably.ai>
📝 WalkthroughWalkthrough
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
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.
🧹 Nitpick comments (1)
mobile/src/components/MobileHostCard.tsx (1)
22-24: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low valueEnhance readability of the worktree summary string.
The inline template literal logic is functionally correct but quite dense. Consider extracting the logic into intermediate variables to improve readability and maintainability.
♻️ Proposed refactor
- const worktreeSummary = props.worktreeCounts - ? `${props.worktreeCounts.total} worktree${props.worktreeCounts.total === 1 ? '' : 's'}${props.worktreeCounts.active > 0 ? ` · ${props.worktreeCounts.active} active` : ''}` - : null + let worktreeSummary = null + if (props.worktreeCounts) { + const { total, active } = props.worktreeCounts + const suffix = active > 0 ? ` · ${active} active` : '' + worktreeSummary = `${total} worktree${total === 1 ? '' : 's'}${suffix}` + }
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: b2c90c6b-ed4a-4165-9c07-ef4d71830983
📒 Files selected for processing (1)
mobile/src/components/MobileHostCard.tsx
Co-authored-by: Orca <help@stably.ai>
What changed
Why
Adding Orca Relay to the connection label made the single metadata string wrap unpredictably when desktop worktree counts were large. Separating connection state from desktop inventory restores a clear hierarchy without shrinking text or hiding the Relay path.
Validation
Made with Orca 🐋