feat(mobile): show lightweight running terminal indicators - #7793
feat(mobile): show lightweight running terminal indicators#7793Adamulek123 wants to merge 4 commits into
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
ApprovabilityVerdict: Not approved Macroscope's review found this PR not approvable — This PR introduces a new user-facing feature with new animation components, hooks, and integrations across multiple screens (thread lists, headers). New capabilities with cross-cutting UI changes warrant human review. You can add or adjust custom eligibility rules. Learn more. |
91ed2af to
c6dbec7
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit c6dbec7. Configure here.

What Changed
Why
Running terminal sessions were easy to miss on mobile, while a conventional continuously interpolated animation could keep repainting at the display refresh rate and waste GPU time. This keeps the requested always-on motion but reduces animation updates to eight opacity steps per roughly 1.9-second cycle and shares the animated value across indicators.
Before:
After:
Validation
Checklist
Generated with GPT-5.6 Sol in the Codex harness through T3 Code.
Note
Add pulsing terminal running indicators to mobile thread list and headers
TerminalRunningIndicatorandStatusPulseto show a pulsing terminal icon when terminal sessions are running, respecting the system reduced-motion setting.countRunningTerminalSessions,terminalRunningSessionLabel, anduseThreadRunningTerminalCountto provide reactive per-thread running session counts and accessibility labels.AppHeaderIconButtonandNativeHeaderToolbarwith custom and pulsing capabilities, replacing the old menu-based terminal item withTerminalHeaderMenuButton.--color-terminal-activetheme token to light (#0d9488) and dark (rgba(94, 234, 212, 0.9)) variants.ThreadGitControlsreplacesNativeHeaderToolbar.MenuwithNativeHeaderToolbar.Customfor the terminal control; existing menu actions remain the same.Macroscope summarized 03fe4f3.
Note
Medium Risk
Touches native header item types and a shared Reanimated pulse used across lists and toolbars, so header chrome and animation behavior can regress, but it does not change auth or data handling.
Overview
Shows when a thread has a running terminal process on mobile: a teal pulsing terminal glyph on list rows and the thread header, with matching accessibility labels.
Adds a shared
StatusPulse(stepped opacity, one loop for all indicators, reduced-motion off) plus--color-terminal-active. The terminal glyph always uses Tabler so iOS and Android match.The header terminal control is now a custom pulsing button wrapping the existing scripts/sessions menu (
NativeHeaderToolbar.Custom). Running counts come from terminal metadata viauseThreadRunningTerminalCount.Reviewed by Cursor Bugbot for commit 03fe4f3. Bugbot is set up for automated code reviews on this repo. Configure here.