From b97fe005afb7008c3dc97d503251663ff96a2ef9 Mon Sep 17 00:00:00 2001
From: tbphp <tangb7420@gmail.com>
Date: Wed, 12 Aug 2026 09:16:38 +0800
Subject: [PATCH] =?UTF-8?q?fix(monitor):=20=E6=94=B6=E7=B4=A7=E6=97=A5?=
 =?UTF-8?q?=E5=BF=97=E8=AF=A6=E6=83=85=E9=97=B4=E8=B7=9D?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 web/src/features/monitor/LogDetailDrawer.vue | 130 ++++++++-----------
 web/src/features/monitor/log-format.ts       |  10 +-
 2 files changed, 63 insertions(+), 77 deletions(-)

diff --git a/web/src/features/monitor/LogDetailDrawer.vue b/web/src/features/monitor/LogDetailDrawer.vue
index 66f49ba8..58101d07 100644
--- a/web/src/features/monitor/LogDetailDrawer.vue
+++ b/web/src/features/monitor/LogDetailDrawer.vue
@@ -25,9 +25,8 @@ import { formatCacheHitRate } from '@/lib/cache-rate'
 import {
   formatLogDuration,
   formatLogOutputRate,
-  formatLogReasoningBudget,
   formatLogTokenCount,
-  reasoningBudgetSemantic,
+  formatRequestLogReasoning,
   requestLogCostDisplayState,
   requestLogUsageDisplayState,
 } from './log-format'
@@ -195,26 +194,16 @@ function upstreamAPILabel(upstreamAPI: RequestLogAttemptDto['upstream_api']): st
   return upstreamAPI
 }
 
-function appliedReasoningLabel(reasoning: RequestLogAttemptDto['reasoning']): string {
-  if (reasoning === null) return t('monitor.logs.drawer.reasoningNotSpecified')
-  const details: string[] = []
-  if (reasoning.mode !== null) details.push(reasoning.mode)
-  if (reasoning.effort !== null) details.push(reasoning.effort)
-  if (reasoning.budget_tokens !== null) {
-    const semantic = reasoningBudgetSemantic(reasoning.budget_tokens)
-    if (semantic === 'dynamic') {
-      details.push(t('monitor.logs.drawer.reasoningBudgetDynamic'))
-    } else if (semantic === 'disabled') {
-      details.push(t('monitor.logs.drawer.reasoningBudgetDisabled'))
-    } else {
-      details.push(
-        t('monitor.logs.drawer.reasoningBudgetValue', {
-          value: formatLogReasoningBudget(reasoning.budget_tokens, locale.value),
-        }),
-      )
-    }
-  }
-  return details.join(' · ')
+function reasoningLabel(reasoning: RequestLogAttemptDto['reasoning']): string {
+  const value = formatRequestLogReasoning(reasoning, locale.value)
+  return value ? `[${value}]` : ''
+}
+
+function upstreamReasoningLabel(
+  reasoning: RequestLogAttemptDto['reasoning'],
+  routeMode: RequestLogAttemptDto['route_mode'],
+): string {
+  return reasoningLabel(routeMode === 'converted' ? reasoning : (log.value?.reasoning ?? null))
 }
 
 function isFinalAttempt(attempt: RequestLogAttemptDto): boolean {
@@ -433,44 +422,13 @@ function toggleAttemptErrorMessage(sequence: number): void {
           </div>
           <div>
             <dt>{{ t('monitor.logs.drawer.clientModel') }}</dt>
-            <dd>
-              <code>{{ log.client_model ?? '—' }}</code>
+            <dd class="log-detail__model-value">
+              <code>{{ log.client_model ?? '—' }}</code
+              ><small v-if="reasoningLabel(log.reasoning)" class="log-detail__reasoning">{{
+                reasoningLabel(log.reasoning)
+              }}</small>
             </dd>
           </div>
-          <template v-if="log.reasoning">
-            <div v-if="log.reasoning.mode">
-              <dt>{{ t('monitor.logs.drawer.reasoningMode') }}</dt>
-              <dd>
-                <code>{{ log.reasoning.mode }}</code>
-              </dd>
-            </div>
-            <div v-if="log.reasoning.effort">
-              <dt>{{ t('monitor.logs.drawer.reasoningEffort') }}</dt>
-              <dd>
-                <code>{{ log.reasoning.effort }}</code>
-              </dd>
-            </div>
-            <div v-if="log.reasoning.budget_tokens !== null">
-              <dt>{{ t('monitor.logs.drawer.reasoningBudget') }}</dt>
-              <dd v-if="reasoningBudgetSemantic(log.reasoning.budget_tokens) === 'dynamic'">
-                <code>auto</code>
-              </dd>
-              <dd v-else-if="reasoningBudgetSemantic(log.reasoning.budget_tokens) === 'disabled'">
-                <code>disabled</code>
-              </dd>
-              <dd v-else>
-                {{
-                  t('monitor.logs.drawer.reasoningBudgetValue', {
-                    value: formatLogReasoningBudget(log.reasoning.budget_tokens, locale),
-                  })
-                }}
-              </dd>
-            </div>
-          </template>
-          <div v-else>
-            <dt>{{ t('monitor.logs.drawer.reasoningConfig') }}</dt>
-            <dd>{{ t('monitor.logs.drawer.reasoningNotSpecified') }}</dd>
-          </div>
         </dl>
       </section>
 
@@ -495,14 +453,25 @@ function toggleAttemptErrorMessage(sequence: number): void {
           </div>
           <div>
             <dt>{{ t('monitor.logs.drawer.upstreamModel') }}</dt>
-            <dd>
-              <code>{{ log.upstream_model ?? '—' }}</code>
+            <dd class="log-detail__model-value">
+              <code>{{ log.upstream_model ?? '—' }}</code
+              ><small
+                v-if="
+                  upstreamReasoningLabel(
+                    finalAttempt?.reasoning ?? null,
+                    finalAttempt?.route_mode ?? null,
+                  )
+                "
+                class="log-detail__reasoning"
+                >{{
+                  upstreamReasoningLabel(
+                    finalAttempt?.reasoning ?? null,
+                    finalAttempt?.route_mode ?? null,
+                  )
+                }}</small
+              >
             </dd>
           </div>
-          <div v-if="finalAttempt?.reasoning">
-            <dt>{{ t('monitor.logs.drawer.appliedReasoning') }}</dt>
-            <dd>{{ appliedReasoningLabel(finalAttempt.reasoning) }}</dd>
-          </div>
         </dl>
         <div
           v-if="log.model_consistency === 'unknown' || log.model_consistency === 'mismatch'"
@@ -602,7 +571,7 @@ function toggleAttemptErrorMessage(sequence: number): void {
         </dl>
       </section>
 
-      <section v-if="!selfScoped" class="log-detail__section">
+      <section v-if="!selfScoped" class="log-detail__section log-detail__attempt-section">
         <details v-if="log.attempts.length > 0" class="log-attempt-chain">
           <summary>
             <ChevronRight class="log-attempt-chain__chevron" :size="15" aria-hidden="true" />
@@ -639,14 +608,15 @@ function toggleAttemptErrorMessage(sequence: number): void {
                 </div>
                 <div>
                   <dt>{{ t('monitor.logs.drawer.upstreamModel') }}</dt>
-                  <dd>
-                    <code>{{ attempt.upstream_model ?? '—' }}</code>
+                  <dd class="log-detail__model-value">
+                    <code>{{ attempt.upstream_model ?? '—' }}</code
+                    ><small
+                      v-if="upstreamReasoningLabel(attempt.reasoning, attempt.route_mode)"
+                      class="log-detail__reasoning"
+                      >{{ upstreamReasoningLabel(attempt.reasoning, attempt.route_mode) }}</small
+                    >
                   </dd>
                 </div>
-                <div v-if="attempt.reasoning">
-                  <dt>{{ t('monitor.logs.drawer.appliedReasoning') }}</dt>
-                  <dd>{{ appliedReasoningLabel(attempt.reasoning) }}</dd>
-                </div>
               </template>
               <div v-if="showAttemptOperation(attempt)">
                 <dt>{{ t('monitor.logs.drawer.operation') }}</dt>
@@ -790,6 +760,10 @@ function toggleAttemptErrorMessage(sequence: number): void {
   padding: 16px 0;
 }
 
+.log-detail__attempt-section {
+  padding: 8px 0;
+}
+
 .log-detail__section h3 {
   margin: 0 0 12px;
   font-size: var(--text-sm);
@@ -819,6 +793,12 @@ function toggleAttemptErrorMessage(sequence: number): void {
   overflow-wrap: anywhere;
 }
 
+.log-detail__reasoning {
+  color: var(--color-text-faint);
+  font-size: var(--text-label-xs);
+  font-weight: 400;
+}
+
 .log-detail__protocol {
   display: flex;
   min-width: 0;
@@ -926,7 +906,7 @@ function toggleAttemptErrorMessage(sequence: number): void {
 
 .log-attempt-chain summary {
   display: flex;
-  min-height: 28px;
+  min-height: 24px;
   align-items: center;
   gap: 6px;
   color: var(--color-text);
@@ -957,7 +937,7 @@ function toggleAttemptErrorMessage(sequence: number): void {
 }
 
 .log-attempt-chain[open] .log-attempt:first-of-type {
-  margin-top: 10px;
+  margin-top: 4px;
 }
 
 .log-attempt + .log-attempt {
@@ -965,7 +945,7 @@ function toggleAttemptErrorMessage(sequence: number): void {
 }
 
 .log-attempt {
-  padding: 13px 0;
+  padding: 8px 0;
 }
 
 .log-error-message--attempt {
@@ -984,7 +964,7 @@ function toggleAttemptErrorMessage(sequence: number): void {
   align-items: center;
   justify-content: space-between;
   gap: 10px;
-  margin-bottom: 12px;
+  margin-bottom: 8px;
   color: var(--color-text);
   font-size: var(--text-sm);
 }
diff --git a/web/src/features/monitor/log-format.ts b/web/src/features/monitor/log-format.ts
index 044ff528..161aca8b 100644
--- a/web/src/features/monitor/log-format.ts
+++ b/web/src/features/monitor/log-format.ts
@@ -1,4 +1,4 @@
-import type { RequestLogItemDto } from '@/app/resources/request-logs'
+import type { RequestLogItemDto, RequestLogReasoningDto } from '@/app/resources/request-logs'
 
 export type RequestLogUsageDisplayState = 'reported' | 'missing' | 'not_applicable'
 export type RequestLogCostDisplayState = 'complete' | 'unpriced' | 'not_applicable'
@@ -42,7 +42,13 @@ export function formatLogTokenCount(value: string, locale: string): string {
 }
 
 export function formatLogReasoning(log: RequestLogItemDto, locale: string): string {
-  const value = log.reasoning
+  return formatRequestLogReasoning(log.reasoning, locale)
+}
+
+export function formatRequestLogReasoning(
+  value: RequestLogReasoningDto | null,
+  locale: string,
+): string {
   if (value === null) return ''
   const details: string[] = []
   if (