fix(perf): remove pulsing from terminal icon by t3dotgg · Pull Request #3975 · pingdotgg/t3code

t3dotgg · 2026-07-14T21:28:48Z

Summary

remove continuous pulse animations from running terminal icons
retain the existing terminal status color and tooltip
avoid keeping the browser compositor active while terminals are running

Validation

vp check
vp run typecheck

Note

Low Risk
Pure UI class change on status icons; no logic, auth, or data paths affected.

Overview
Stops applying animate-pulse on the running-terminal TerminalIcon in the sidebar thread row and in ThreadRowTrailingStatus (e.g. command palette).

When terminals are active, the icon still uses the teal status color and the same “Terminal process running” tooltip; only the continuous CSS pulse animation is removed so the compositor isn’t kept busy for every visible running terminal.

^{Reviewed by Cursor Bugbot for commit 2451aa3. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Remove pulsing animation from terminal status icon in sidebar

The TerminalIcon in sidebar thread rows previously added an animate-pulse class when terminals were running. This removes that conditional, making the icon always static with a fixed size-3 class. The change applies in both Sidebar.tsx and ThreadStatusIndicators.tsx.

^{Macroscope summarized 2451aa3.}

cursor

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 2451aa3. Configure here.}

cursor · 2026-07-14T21:31:16Z

 }
 >
- <TerminalIcon className={`size-3 ${terminalStatus.pulse ? "animate-pulse" : ""}`} />
+ <TerminalIcon className="size-3" />


Unused terminal pulse property
Low Severity
After removing the animate-pulse class, the pulse property on TerminalStatusIndicator is no longer used. It's still declared and always true from terminalStatusFromRunningIds, creating dead code that could mislead future development.
^{Reviewed by Cursor Bugbot for commit 2451aa3. Configure here.}

macroscopeapp · 2026-07-14T21:31:38Z

Approvability

Verdict: Approved

This PR removes a CSS animation class from terminal icons - a purely visual change with no runtime behavior impact. The unresolved comment about dead code is low-severity cleanup that doesn't affect correctness.

^{You can customize Macroscope's approvability policy. Learn more.}

t3dotgg · 2026-07-14T23:05:36Z

Trumped by #3978

t3dotgg · 2026-07-14T21:28:48Z

Summary

remove continuous pulse animations from running terminal icons
retain the existing terminal status color and tooltip
avoid keeping the browser compositor active while terminals are running

Validation

vp check
vp run typecheck

Note

Low Risk
Pure UI class change on status icons; no logic, auth, or data paths affected.

Overview
Stops applying animate-pulse on the running-terminal TerminalIcon in the sidebar thread row and in ThreadRowTrailingStatus (e.g. command palette).

When terminals are active, the icon still uses the teal status color and the same “Terminal process running” tooltip; only the continuous CSS pulse animation is removed so the compositor isn’t kept busy for every visible running terminal.

^{Reviewed by Cursor Bugbot for commit 2451aa3. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Remove pulsing animation from terminal status icon in sidebar

The TerminalIcon in sidebar thread rows previously added an animate-pulse class when terminals were running. This removes that conditional, making the icon always static with a fixed size-3 class. The change applies in both Sidebar.tsx and ThreadStatusIndicators.tsx.

^{Macroscope summarized 2451aa3.}

cursor

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 2451aa3. Configure here.}

cursor · 2026-07-14T21:31:16Z

 }
 >
- <TerminalIcon className={`size-3 ${terminalStatus.pulse ? "animate-pulse" : ""}`} />
+ <TerminalIcon className="size-3" />


Unused terminal pulse property
Low Severity
After removing the animate-pulse class, the pulse property on TerminalStatusIndicator is no longer used. It's still declared and always true from terminalStatusFromRunningIds, creating dead code that could mislead future development.
^{Reviewed by Cursor Bugbot for commit 2451aa3. Configure here.}

macroscopeapp · 2026-07-14T21:31:38Z

Approvability

Verdict: Approved

This PR removes a CSS animation class from terminal icons - a purely visual change with no runtime behavior impact. The unresolved comment about dead code is low-severity cleanup that doesn't affect correctness.

^{You can customize Macroscope's approvability policy. Learn more.}

t3dotgg · 2026-07-14T23:05:36Z

Trumped by #3978

t3dotgg · 2026-07-14T21:28:48Z

Summary

remove continuous pulse animations from running terminal icons
retain the existing terminal status color and tooltip
avoid keeping the browser compositor active while terminals are running

Validation

vp check
vp run typecheck

Note

Low Risk
Pure UI class change on status icons; no logic, auth, or data paths affected.

Overview
Stops applying animate-pulse on the running-terminal TerminalIcon in the sidebar thread row and in ThreadRowTrailingStatus (e.g. command palette).

When terminals are active, the icon still uses the teal status color and the same “Terminal process running” tooltip; only the continuous CSS pulse animation is removed so the compositor isn’t kept busy for every visible running terminal.

^{Reviewed by Cursor Bugbot for commit 2451aa3. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Remove pulsing animation from terminal status icon in sidebar

The TerminalIcon in sidebar thread rows previously added an animate-pulse class when terminals were running. This removes that conditional, making the icon always static with a fixed size-3 class. The change applies in both Sidebar.tsx and ThreadStatusIndicators.tsx.

^{Macroscope summarized 2451aa3.}

cursor

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 2451aa3. Configure here.}

cursor · 2026-07-14T21:31:16Z

 }
 >
- <TerminalIcon className={`size-3 ${terminalStatus.pulse ? "animate-pulse" : ""}`} />
+ <TerminalIcon className="size-3" />


Unused terminal pulse property
Low Severity
After removing the animate-pulse class, the pulse property on TerminalStatusIndicator is no longer used. It's still declared and always true from terminalStatusFromRunningIds, creating dead code that could mislead future development.
^{Reviewed by Cursor Bugbot for commit 2451aa3. Configure here.}

macroscopeapp · 2026-07-14T21:31:38Z

Approvability

Verdict: Approved

This PR removes a CSS animation class from terminal icons - a purely visual change with no runtime behavior impact. The unresolved comment about dead code is low-severity cleanup that doesn't affect correctness.

^{You can customize Macroscope's approvability policy. Learn more.}

t3dotgg · 2026-07-14T23:05:36Z

Trumped by #3978

t3dotgg · 2026-07-14T21:28:48Z

Summary

remove continuous pulse animations from running terminal icons
retain the existing terminal status color and tooltip
avoid keeping the browser compositor active while terminals are running

Validation

vp check
vp run typecheck

Note

Low Risk
Pure UI class change on status icons; no logic, auth, or data paths affected.

Overview
Stops applying animate-pulse on the running-terminal TerminalIcon in the sidebar thread row and in ThreadRowTrailingStatus (e.g. command palette).

When terminals are active, the icon still uses the teal status color and the same “Terminal process running” tooltip; only the continuous CSS pulse animation is removed so the compositor isn’t kept busy for every visible running terminal.

^{Reviewed by Cursor Bugbot for commit 2451aa3. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Remove pulsing animation from terminal status icon in sidebar

The TerminalIcon in sidebar thread rows previously added an animate-pulse class when terminals were running. This removes that conditional, making the icon always static with a fixed size-3 class. The change applies in both Sidebar.tsx and ThreadStatusIndicators.tsx.

^{Macroscope summarized 2451aa3.}

cursor

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 2451aa3. Configure here.}

cursor · 2026-07-14T21:31:16Z

 }
 >
- <TerminalIcon className={`size-3 ${terminalStatus.pulse ? "animate-pulse" : ""}`} />
+ <TerminalIcon className="size-3" />


Unused terminal pulse property
Low Severity
After removing the animate-pulse class, the pulse property on TerminalStatusIndicator is no longer used. It's still declared and always true from terminalStatusFromRunningIds, creating dead code that could mislead future development.
^{Reviewed by Cursor Bugbot for commit 2451aa3. Configure here.}

macroscopeapp · 2026-07-14T21:31:38Z

Approvability

Verdict: Approved

This PR removes a CSS animation class from terminal icons - a purely visual change with no runtime behavior impact. The unresolved comment about dead code is low-severity cleanup that doesn't affect correctness.

^{You can customize Macroscope's approvability policy. Learn more.}

t3dotgg · 2026-07-14T23:05:36Z

Trumped by #3978

t3dotgg · 2026-07-14T21:28:48Z

Summary

remove continuous pulse animations from running terminal icons
retain the existing terminal status color and tooltip
avoid keeping the browser compositor active while terminals are running

Validation

vp check
vp run typecheck

Note

Low Risk
Pure UI class change on status icons; no logic, auth, or data paths affected.

Overview
Stops applying animate-pulse on the running-terminal TerminalIcon in the sidebar thread row and in ThreadRowTrailingStatus (e.g. command palette).

When terminals are active, the icon still uses the teal status color and the same “Terminal process running” tooltip; only the continuous CSS pulse animation is removed so the compositor isn’t kept busy for every visible running terminal.

^{Reviewed by Cursor Bugbot for commit 2451aa3. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Remove pulsing animation from terminal status icon in sidebar

The TerminalIcon in sidebar thread rows previously added an animate-pulse class when terminals were running. This removes that conditional, making the icon always static with a fixed size-3 class. The change applies in both Sidebar.tsx and ThreadStatusIndicators.tsx.

^{Macroscope summarized 2451aa3.}

cursor

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 2451aa3. Configure here.}

cursor · 2026-07-14T21:31:16Z

 }
 >
- <TerminalIcon className={`size-3 ${terminalStatus.pulse ? "animate-pulse" : ""}`} />
+ <TerminalIcon className="size-3" />


Unused terminal pulse property
Low Severity
After removing the animate-pulse class, the pulse property on TerminalStatusIndicator is no longer used. It's still declared and always true from terminalStatusFromRunningIds, creating dead code that could mislead future development.
^{Reviewed by Cursor Bugbot for commit 2451aa3. Configure here.}

macroscopeapp · 2026-07-14T21:31:38Z

Approvability

Verdict: Approved

This PR removes a CSS animation class from terminal icons - a purely visual change with no runtime behavior impact. The unresolved comment about dead code is low-severity cleanup that doesn't affect correctness.

^{You can customize Macroscope's approvability policy. Learn more.}

t3dotgg · 2026-07-14T23:05:36Z

Trumped by #3978

t3dotgg · 2026-07-14T21:28:48Z

Summary

remove continuous pulse animations from running terminal icons
retain the existing terminal status color and tooltip
avoid keeping the browser compositor active while terminals are running

Validation

vp check
vp run typecheck

Note

Low Risk
Pure UI class change on status icons; no logic, auth, or data paths affected.

Overview
Stops applying animate-pulse on the running-terminal TerminalIcon in the sidebar thread row and in ThreadRowTrailingStatus (e.g. command palette).

When terminals are active, the icon still uses the teal status color and the same “Terminal process running” tooltip; only the continuous CSS pulse animation is removed so the compositor isn’t kept busy for every visible running terminal.

^{Reviewed by Cursor Bugbot for commit 2451aa3. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Remove pulsing animation from terminal status icon in sidebar

The TerminalIcon in sidebar thread rows previously added an animate-pulse class when terminals were running. This removes that conditional, making the icon always static with a fixed size-3 class. The change applies in both Sidebar.tsx and ThreadStatusIndicators.tsx.

^{Macroscope summarized 2451aa3.}

cursor

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 2451aa3. Configure here.}

cursor · 2026-07-14T21:31:16Z

 }
 >
- <TerminalIcon className={`size-3 ${terminalStatus.pulse ? "animate-pulse" : ""}`} />
+ <TerminalIcon className="size-3" />


Unused terminal pulse property
Low Severity
After removing the animate-pulse class, the pulse property on TerminalStatusIndicator is no longer used. It's still declared and always true from terminalStatusFromRunningIds, creating dead code that could mislead future development.
^{Reviewed by Cursor Bugbot for commit 2451aa3. Configure here.}

macroscopeapp · 2026-07-14T21:31:38Z

Approvability

Verdict: Approved

This PR removes a CSS animation class from terminal icons - a purely visual change with no runtime behavior impact. The unresolved comment about dead code is low-severity cleanup that doesn't affect correctness.

^{You can customize Macroscope's approvability policy. Learn more.}

t3dotgg · 2026-07-14T23:05:36Z

Trumped by #3978

t3dotgg · 2026-07-14T21:28:48Z

Summary

remove continuous pulse animations from running terminal icons
retain the existing terminal status color and tooltip
avoid keeping the browser compositor active while terminals are running

Validation

vp check
vp run typecheck

Note

Low Risk
Pure UI class change on status icons; no logic, auth, or data paths affected.

Overview
Stops applying animate-pulse on the running-terminal TerminalIcon in the sidebar thread row and in ThreadRowTrailingStatus (e.g. command palette).

When terminals are active, the icon still uses the teal status color and the same “Terminal process running” tooltip; only the continuous CSS pulse animation is removed so the compositor isn’t kept busy for every visible running terminal.

^{Reviewed by Cursor Bugbot for commit 2451aa3. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Remove pulsing animation from terminal status icon in sidebar

The TerminalIcon in sidebar thread rows previously added an animate-pulse class when terminals were running. This removes that conditional, making the icon always static with a fixed size-3 class. The change applies in both Sidebar.tsx and ThreadStatusIndicators.tsx.

^{Macroscope summarized 2451aa3.}

cursor

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 2451aa3. Configure here.}

cursor · 2026-07-14T21:31:16Z

 }
 >
- <TerminalIcon className={`size-3 ${terminalStatus.pulse ? "animate-pulse" : ""}`} />
+ <TerminalIcon className="size-3" />


Unused terminal pulse property
Low Severity
After removing the animate-pulse class, the pulse property on TerminalStatusIndicator is no longer used. It's still declared and always true from terminalStatusFromRunningIds, creating dead code that could mislead future development.
^{Reviewed by Cursor Bugbot for commit 2451aa3. Configure here.}

macroscopeapp · 2026-07-14T21:31:38Z

Approvability

Verdict: Approved

This PR removes a CSS animation class from terminal icons - a purely visual change with no runtime behavior impact. The unresolved comment about dead code is low-severity cleanup that doesn't affect correctness.

^{You can customize Macroscope's approvability policy. Learn more.}

t3dotgg · 2026-07-14T23:05:36Z

Trumped by #3978

t3dotgg · 2026-07-14T21:28:48Z

Summary

remove continuous pulse animations from running terminal icons
retain the existing terminal status color and tooltip
avoid keeping the browser compositor active while terminals are running

Validation

vp check
vp run typecheck

Note

Low Risk
Pure UI class change on status icons; no logic, auth, or data paths affected.

Overview
Stops applying animate-pulse on the running-terminal TerminalIcon in the sidebar thread row and in ThreadRowTrailingStatus (e.g. command palette).

When terminals are active, the icon still uses the teal status color and the same “Terminal process running” tooltip; only the continuous CSS pulse animation is removed so the compositor isn’t kept busy for every visible running terminal.

^{Reviewed by Cursor Bugbot for commit 2451aa3. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Remove pulsing animation from terminal status icon in sidebar

The TerminalIcon in sidebar thread rows previously added an animate-pulse class when terminals were running. This removes that conditional, making the icon always static with a fixed size-3 class. The change applies in both Sidebar.tsx and ThreadStatusIndicators.tsx.

^{Macroscope summarized 2451aa3.}

cursor

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 2451aa3. Configure here.}

cursor · 2026-07-14T21:31:16Z

 }
 >
- <TerminalIcon className={`size-3 ${terminalStatus.pulse ? "animate-pulse" : ""}`} />
+ <TerminalIcon className="size-3" />


Unused terminal pulse property
Low Severity
After removing the animate-pulse class, the pulse property on TerminalStatusIndicator is no longer used. It's still declared and always true from terminalStatusFromRunningIds, creating dead code that could mislead future development.
^{Reviewed by Cursor Bugbot for commit 2451aa3. Configure here.}

macroscopeapp · 2026-07-14T21:31:38Z

Approvability

Verdict: Approved

This PR removes a CSS animation class from terminal icons - a purely visual change with no runtime behavior impact. The unresolved comment about dead code is low-severity cleanup that doesn't affect correctness.

^{You can customize Macroscope's approvability policy. Learn more.}

t3dotgg · 2026-07-14T23:05:36Z

Trumped by #3978

fix(perf): remove pulsing from terminal icon
2451aa3

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Jul 14, 2026

cursorBot reviewed Jul 14, 2026
View reviewed changes

macroscopeappBot approved these changes Jul 14, 2026
View reviewed changes

t3dotgg closed this Jul 14, 2026

fix(perf): remove pulsing from terminal icon
2451aa3

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Jul 14, 2026

cursorBot reviewed Jul 14, 2026
View reviewed changes

macroscopeappBot approved these changes Jul 14, 2026
View reviewed changes

t3dotgg closed this Jul 14, 2026

fix(perf): remove pulsing from terminal icon
2451aa3

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Jul 14, 2026

cursorBot reviewed Jul 14, 2026
View reviewed changes

macroscopeappBot approved these changes Jul 14, 2026
View reviewed changes

t3dotgg closed this Jul 14, 2026

fix(perf): remove pulsing from terminal icon
2451aa3

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Jul 14, 2026

cursorBot reviewed Jul 14, 2026
View reviewed changes

macroscopeappBot approved these changes Jul 14, 2026
View reviewed changes

t3dotgg closed this Jul 14, 2026

fix(perf): remove pulsing from terminal icon
2451aa3

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Jul 14, 2026

cursorBot reviewed Jul 14, 2026
View reviewed changes

macroscopeappBot approved these changes Jul 14, 2026
View reviewed changes

t3dotgg closed this Jul 14, 2026

fix(perf): remove pulsing from terminal icon
2451aa3

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Jul 14, 2026

cursorBot reviewed Jul 14, 2026
View reviewed changes

macroscopeappBot approved these changes Jul 14, 2026
View reviewed changes

t3dotgg closed this Jul 14, 2026

fix(perf): remove pulsing from terminal icon
2451aa3

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Jul 14, 2026

cursorBot reviewed Jul 14, 2026
View reviewed changes

macroscopeappBot approved these changes Jul 14, 2026
View reviewed changes

t3dotgg closed this Jul 14, 2026

fix(perf): remove pulsing from terminal icon
2451aa3

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Jul 14, 2026

cursorBot reviewed Jul 14, 2026
View reviewed changes

macroscopeappBot approved these changes Jul 14, 2026
View reviewed changes

t3dotgg closed this Jul 14, 2026

Uh oh!

Conversation

t3dotgg commented Jul 14, 2026• edited by macroscopeapp BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Validation

Remove pulsing animation from terminal status icon in sidebar

Uh oh!

cursorBot left a comment

Choose a reason for hiding this comment

Uh oh!

cursorBotJul 14, 2026

Choose a reason for hiding this comment

Unused terminal pulse property

Uh oh!

macroscopeappBot commented Jul 14, 2026

Approvability

Uh oh!

t3dotgg commented Jul 14, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

t3dotgg commented Jul 14, 2026• edited by macroscopeapp BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Validation

Remove pulsing animation from terminal status icon in sidebar

Uh oh!

cursorBot left a comment

Choose a reason for hiding this comment

Uh oh!

cursorBotJul 14, 2026

Choose a reason for hiding this comment

Unused terminal pulse property

Uh oh!

macroscopeappBot commented Jul 14, 2026

Approvability

Uh oh!

t3dotgg commented Jul 14, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

t3dotgg commented Jul 14, 2026• edited by macroscopeapp BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Validation

Remove pulsing animation from terminal status icon in sidebar

Uh oh!

cursorBot left a comment

Choose a reason for hiding this comment

Uh oh!

cursorBotJul 14, 2026

Choose a reason for hiding this comment

Unused terminal pulse property

Uh oh!

macroscopeappBot commented Jul 14, 2026

Approvability

Uh oh!

t3dotgg commented Jul 14, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

t3dotgg commented Jul 14, 2026•
edited by macroscopeapp Bot
Loading

t3dotgg commented Jul 14, 2026•
edited by macroscopeapp Bot
Loading

t3dotgg commented Jul 14, 2026•
edited by macroscopeapp Bot
Loading

t3dotgg commented Jul 14, 2026•
edited by macroscopeapp Bot
Loading

t3dotgg commented Jul 14, 2026•
edited by macroscopeapp Bot
Loading