Use amber for permission status indicator

Change StatusIndicator permission state from red to amber and add a focused render regression test.
This commit is contained in:
Jinwoo Hong 2026-05-28 21:37:56 -04:00 committed by GitHub
parent 41f0bc6394
commit 29916cafe0
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 8 additions and 1 deletions

View File

@ -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')

View File

@ -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