Remove detail panel tab separator

Simplify detail panel refinements

Author: AkiKurisu

desktop/src/renderer/components/conversation/PlanApprovalComposer.tsx

@@ -10,7 +10,6 @@ import {
 import { useT } from '../../contexts/LocaleContext'
 import { useConversationStore } from '../../stores/conversationStore'
 import { useUIStore } from '../../stores/uiStore'
-import { PLAN_IMPLEMENTATION_MESSAGE } from '../../utils/planImplementation'
 import { startTurnWithOptimisticUI } from '../../utils/startTurn'
 import { ComposerShell, DECISION_MASCOT } from './ComposerShell'
 import { ConversationColumn } from './ConversationColumn'
@@ -74,7 +73,7 @@ export function PlanApprovalComposer({
     await startTurnWithOptimisticUI({
       threadId,
       workspacePath,
-      text: PLAN_IMPLEMENTATION_MESSAGE,
+      text: 'Implement the plan.',
       fallbackThreadName: t('toast.imageMessage'),
       fileFallbackThreadName: t('toast.fileReferenceMessage'),
       attachmentFallbackThreadName: t('toast.attachmentMessage'),

desktop/src/renderer/components/detail/styles/automation-viewer-tab-keyframes.css

@@ -6,14 +6,3 @@
     transform: translate(-60%);
   }
 }
-
-@keyframes dotcraft-automation-tab-dot {
-  0%, 100% {
-    opacity: 0.35;
-    transform: scale(0.8);
-  }
-  50% {
-    opacity: 1;
-    transform: scale(1);
-  }
-}

desktop/src/renderer/components/detail/styles/automation-viewer-tab.css

@@ -43,33 +43,3 @@
   position: relative;
   z-index: 1;
 }
-
-.dotcraft-automation-viewer-tab__glyph {
-  position: relative;
-  display: inline-flex;
-  align-items: center;
-  justify-content: center;
-  color: var(--accent);
-}
-
-.dotcraft-automation-viewer-tab__dot {
-  position: absolute;
-  top: -1px;
-  right: -2px;
-  width: 4px;
-  height: 4px;
-  border-radius: 999px;
-  background: var(--accent);
-  animation: dotcraft-automation-tab-dot 1.4s ease-in-out infinite;
-}
-
-@media (prefers-reduced-motion: reduce) {
-  .dotcraft-automation-viewer-tab::before {
-    animation: none;
-    transform: translate(0);
-  }
-
-  .dotcraft-automation-viewer-tab__dot {
-    animation: none;
-  }
-}

desktop/src/renderer/components/layout/DetailPanel.tsx

@@ -265,31 +265,12 @@ export function DetailPanel({
           )
         })}
 
-        {/* Separator — only visible when both system and viewer tabs exist */}
-        {openSystemTabs.length > 0 && viewerTabs.length > 0 && (
-          <div
-            aria-hidden
-            style={{
-              alignSelf: 'center',
-              width: '1px',
-              height: '16px',
-              backgroundColor: 'var(--glass-border)',
-              flexShrink: 0
-            }}
-          />
-        )}
-
         {/* Viewer tabs — label + leading icon slot that becomes the close button on hover. */}
         {viewerTabs.map((tab) => {
           const automationActive = tab.kind === 'browser' && tab.automationActive === true
           const icon = tab.kind === 'browser'
             ? (automationActive
-                ? (
-                    <span className="dotcraft-automation-viewer-tab__glyph" aria-hidden="true">
-                      <MousePointer2 size={14} strokeWidth={2} />
-                      <span className="dotcraft-automation-viewer-tab__dot" />
-                    </span>
-                  )
+                ? <MousePointer2 size={14} strokeWidth={2} aria-hidden style={{ display: 'block', color: 'var(--accent)' }} />
                 : browserTabIcon(tab.faviconDataUrl))
             : tab.kind === 'terminal'
               ? <SquareTerminal size={14} strokeWidth={2} aria-hidden style={{ display: 'block' }} />

desktop/src/renderer/tests/MessageStream.test.tsx

@@ -5,7 +5,6 @@ import { LocaleProvider } from '../contexts/LocaleContext'
 import { MessageStream } from '../components/conversation/MessageStream'
 import { useConversationStore } from '../stores/conversationStore'
 import { useThreadStore } from '../stores/threadStore'
-import { PLAN_IMPLEMENTATION_MESSAGE } from '../utils/planImplementation'
 import type { ThreadGoal } from '../types/thread'
 import type { FileDiff } from '../types/toolCall'
 import { installDesktopApiMock } from './desktopApiMock'
@@ -165,7 +164,7 @@ describe('MessageStream', () => {
             id: 'u1',
             type: 'userMessage',
             status: 'completed',
-            text: PLAN_IMPLEMENTATION_MESSAGE,
+            text: 'Implement the plan.',
             createdAt: new Date().toISOString()
           },
           {
@@ -181,7 +180,7 @@ describe('MessageStream', () => {
 
     renderWithLocale(<MessageStream />)
 
-    expect(screen.getByText(PLAN_IMPLEMENTATION_MESSAGE)).toBeInTheDocument()
+    expect(screen.getByText('Implement the plan.')).toBeInTheDocument()
     expect(screen.getByText('Executing accepted plan now.')).toBeInTheDocument()
   })
 

desktop/src/renderer/tests/PlanApprovalComposer.test.tsx

@@ -7,7 +7,6 @@ import { useConnectionStore } from '../stores/connectionStore'
 import { useConversationStore } from '../stores/conversationStore'
 import { useThreadStore } from '../stores/threadStore'
 import { useUIStore } from '../stores/uiStore'
-import { PLAN_IMPLEMENTATION_MESSAGE } from '../utils/planImplementation'
 import { installDesktopApiMock } from './desktopApiMock'
 
 const appServerSendRequest = vi.fn()
@@ -148,7 +147,7 @@ describe('PlanApprovalComposer', () => {
         'turn/start',
         expect.objectContaining({
           threadId: 'thread-1',
-          input: [{ type: 'text', text: PLAN_IMPLEMENTATION_MESSAGE }]
+          input: [{ type: 'text', text: 'Implement the plan.' }]
         })
       )
     })
@@ -157,7 +156,7 @@ describe('PlanApprovalComposer', () => {
     expect(useConversationStore.getState().turns).toEqual([
       expect.objectContaining({
         id: 'turn-server-1',
-        items: [expect.objectContaining({ type: 'userMessage', text: PLAN_IMPLEMENTATION_MESSAGE })]
+        items: [expect.objectContaining({ type: 'userMessage', text: 'Implement the plan.' })]
       })
     ])
   })
@@ -180,7 +179,7 @@ describe('PlanApprovalComposer', () => {
         'turn/start',
         expect.objectContaining({
           threadId: 'thread-1',
-          input: [{ type: 'text', text: PLAN_IMPLEMENTATION_MESSAGE }]
+          input: [{ type: 'text', text: 'Implement the plan.' }]
         })
       )
     })

desktop/src/renderer/utils/formatCompactCount.test.ts

@@ -15,11 +15,4 @@ describe('formatCompactCount', () => {
   ])('formats %s as %s', (value, expected) => {
     expect(formatCompactCount(value)).toBe(expected)
   })
-
-  it.each([-1, Number.NaN, Number.POSITIVE_INFINITY, Number.NEGATIVE_INFINITY])(
-    'treats invalid count %s as zero',
-    (value) => {
-      expect(formatCompactCount(value)).toBe('0')
-    }
-  )
 })

desktop/src/renderer/utils/formatCompactCount.ts

@@ -2,8 +2,6 @@ const COMPACT_COUNT_UNITS = ['', 'k', 'M', 'B'] as const
 
 /** Formats a non-negative count with decimal compact units and at most one decimal place. */
 export function formatCompactCount(value: number): string {
-  if (!Number.isFinite(value) || value < 0) return '0'
-
   const count = Math.round(value)
   if (count < 1_000) return String(count)