From be9ae9979324b95ebbaafb3a0a601ba5606d6f21 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E7=A8=8B=E5=BA=8F=E5=91=98=E9=98=BF=E6=B1=9F=28Relakkes?=
 =?UTF-8?q?=29?= <relakkes@gmail.com>
Date: Mon, 6 Apr 2026 21:41:48 +0800
Subject: [PATCH] =?UTF-8?q?fix:=20address=20code=20review=20findings=20?=
 =?UTF-8?q?=E2=80=94=20consistent=20px-3,=20rounded-lg,=20leading-[1.3]?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 desktop/src/components/chat/CodeViewer.tsx       | 2 +-
 desktop/src/components/chat/PermissionDialog.tsx | 4 ++--
 desktop/src/components/chat/ToolCallBlock.tsx    | 6 +++---
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/desktop/src/components/chat/CodeViewer.tsx b/desktop/src/components/chat/CodeViewer.tsx
index f75721e5..4098915e 100644
--- a/desktop/src/components/chat/CodeViewer.tsx
+++ b/desktop/src/components/chat/CodeViewer.tsx
@@ -67,7 +67,7 @@ export function CodeViewer({ code, language, maxLines = 20, showLineNumbers = tr
                 <span className="w-0" />
               )}
               <span
-                className="overflow-hidden bg-white px-4 py-px whitespace-pre-wrap break-words text-[#24292f]"
+                className="overflow-hidden bg-white px-3 py-px whitespace-pre-wrap break-words text-[#24292f]"
                 dangerouslySetInnerHTML={{ __html: highlightedLines[index] ?? escapeHtml(line) }}
               />
             </div>
diff --git a/desktop/src/components/chat/PermissionDialog.tsx b/desktop/src/components/chat/PermissionDialog.tsx
index 7446dfd3..2097a6ef 100644
--- a/desktop/src/components/chat/PermissionDialog.tsx
+++ b/desktop/src/components/chat/PermissionDialog.tsx
@@ -98,7 +98,7 @@ function renderPermissionPreview(toolName: string, input: unknown) {
   if (toolName === 'Bash' && typeof obj.command === 'string') {
     return (
       <div className="overflow-x-auto rounded-[var(--radius-md)] bg-[#1e1e1e] px-3 py-2.5">
-        <pre className="font-[var(--font-mono)] text-[11px] leading-[1.45] text-[#d4d4d4] whitespace-pre-wrap break-words">
+        <pre className="font-[var(--font-mono)] text-[11px] leading-[1.3] text-[#d4d4d4] whitespace-pre-wrap break-words">
           <span className="text-[#28c840] select-none">$ </span>{obj.command}
         </pre>
       </div>
@@ -209,7 +209,7 @@ export function PermissionDialog({ requestId, toolName, input, description }: Pr
         )}
 
         {allowRawToggle && showRaw && (
-          <pre className="mt-2 max-h-[220px] overflow-y-auto overflow-x-auto rounded-[var(--radius-md)] bg-[#1e1e1e] px-3 py-2.5 font-[var(--font-mono)] text-[11px] leading-[1.45] text-[#d4d4d4] whitespace-pre-wrap break-words">
+          <pre className="mt-2 max-h-[220px] overflow-y-auto overflow-x-auto rounded-[var(--radius-md)] bg-[#1e1e1e] px-3 py-2.5 font-[var(--font-mono)] text-[11px] leading-[1.3] text-[#d4d4d4] whitespace-pre-wrap break-words">
             {rawInput}
           </pre>
         )}
diff --git a/desktop/src/components/chat/ToolCallBlock.tsx b/desktop/src/components/chat/ToolCallBlock.tsx
index 63516c60..c826b2a0 100644
--- a/desktop/src/components/chat/ToolCallBlock.tsx
+++ b/desktop/src/components/chat/ToolCallBlock.tsx
@@ -110,7 +110,7 @@ function renderPreview(
   if (toolName === 'Bash' && typeof obj.command === 'string') {
     return (
       <TerminalChrome title={typeof obj.description === 'string' ? obj.description : filePath}>
-        <div className="px-3 py-2.5 font-[var(--font-mono)] text-[11px] leading-[1.45] text-[#d8d8d8]">
+        <div className="px-3 py-2.5 font-[var(--font-mono)] text-[11px] leading-[1.3] text-[#d8d8d8]">
           <span className="text-[#28c840]">$</span> {obj.command}
         </div>
       </TerminalChrome>
@@ -125,7 +125,7 @@ function renderPreview(
     const text = extractTextContent(result.content)
     if (text) {
       return (
-        <div className={`overflow-hidden rounded-2xl border ${
+        <div className={`overflow-hidden rounded-lg border ${
           result.isError
             ? 'border-[var(--color-error)]/20 bg-[var(--color-error-container)]/60'
             : 'border-[var(--color-border)] bg-[var(--color-surface)]'
@@ -153,7 +153,7 @@ function renderDetails(toolName: string, obj: Record<string, unknown>) {
 
   const text = JSON.stringify(obj, null, 2)
   return (
-    <div className="overflow-hidden rounded-2xl border border-[var(--color-border)] bg-[var(--color-surface)]">
+    <div className="overflow-hidden rounded-lg border border-[var(--color-border)] bg-[var(--color-surface)]">
       <div className="flex items-center justify-between border-b border-[var(--color-border)] px-3 py-2 text-[10px] uppercase tracking-[0.18em] text-[var(--color-outline)]">
         <span>Tool Input</span>
         <CopyButton