From 29916cafe01a0fa7ba1171c7adf4cd3b86d68aa5 Mon Sep 17 00:00:00 2001
From: Jinwoo Hong <73622457+Jinwoo-H@users.noreply.github.com>
Date: Thu, 28 May 2026 21:37:56 -0400
Subject: [PATCH] Use amber for permission status indicator

Change StatusIndicator permission state from red to amber and add a focused render regression test.
---
 .../src/components/sidebar/StatusIndicator.test.ts         | 7 +++++++
 src/renderer/src/components/sidebar/StatusIndicator.tsx    | 2 +-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/renderer/src/components/sidebar/StatusIndicator.test.ts b/src/renderer/src/components/sidebar/StatusIndicator.test.ts
index aa198eb81..7ca742800 100644
--- a/src/renderer/src/components/sidebar/StatusIndicator.test.ts
+++ b/src/renderer/src/components/sidebar/StatusIndicator.test.ts
@@ -25,6 +25,13 @@ describe('StatusIndicator', () => {
     expect(classNames).toContain('animate-spin')
   })
 
+  it('renders permission as an amber attention dot', () => {
+    const classNames = renderDotClassNames('permission')
+
+    expect(classNames).toContain('bg-amber-500')
+    expect(classNames).not.toContain('bg-red-500')
+  })
+
   it('renders active as full emerald dot', () => {
     const classNames = renderDotClassNames('active')
 
diff --git a/src/renderer/src/components/sidebar/StatusIndicator.tsx b/src/renderer/src/components/sidebar/StatusIndicator.tsx
index e49fee21e..154b1b343 100644
--- a/src/renderer/src/components/sidebar/StatusIndicator.tsx
+++ b/src/renderer/src/components/sidebar/StatusIndicator.tsx
@@ -48,7 +48,7 @@ const StatusIndicator = React.memo(function StatusIndicator({
         className={cn(
           'block size-2 rounded-full',
           status === 'permission'
-            ? 'bg-red-500'
+            ? 'bg-amber-500'
             : status === 'done' || status === 'active'
               ? // Green dot for both hook-reported 'done' and the heuristic
                 // 'active' (terminal open, quiet). Working uses a yellow