From e4f95a60ff5035a9163346466b51ad25f7c4ac44 Mon Sep 17 00:00:00 2001
From: ila <ila2002@qq.com>
Date: Thu, 27 Aug 2026 10:28:36 +0800
Subject: [PATCH] =?UTF-8?q?fix:=20=E4=BF=AE=E5=A4=8D=E7=94=9F=E6=88=90?=
 =?UTF-8?q?=E8=AF=A6=E6=83=85=E5=AF=B9=E8=B1=A1=E8=A7=A3=E6=9E=90=20(#72)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 admin-ui/src/views/chorus/generations/index.vue | 5 +++--
 admin-ui/src/views/chorus/shared.js             | 5 +++++
 admin-ui/tests/unit/chorus/shared.spec.js       | 8 +++++++-
 3 files changed, 15 insertions(+), 3 deletions(-)

diff --git a/admin-ui/src/views/chorus/generations/index.vue b/admin-ui/src/views/chorus/generations/index.vue
index b155b6d..9c58d2b 100644
--- a/admin-ui/src/views/chorus/generations/index.vue
+++ b/admin-ui/src/views/chorus/generations/index.vue
@@ -43,7 +43,7 @@
 </template>
 <script>
 import { getGeneration, getGenerationMedia, listGenerations } from '@/api/chorus'
-import { contains, dataOf, formatTime } from '../shared'
+import { contains, dataOf, detailOf, formatTime } from '../shared'
 import { attemptPresentation, generationDetailSize, generationTiming, kindText, reconcileGenerationDetail } from './helpers'
 export default {
   name: 'ChorusGenerations',
@@ -60,7 +60,8 @@ export default {
       this.revokeObjectURLs(); this.inputMedia = []; this.imageOutputs = []; this.textOutputs = []; this.detailError = ''
       this.selected = row; this.detailOpen = true; this.detailLoading = true
       try {
-        const detail = dataOf(await getGeneration(row.id))
+        const detail = detailOf(await getGeneration(row.id))
+        if (!detail) throw new Error('详情响应格式无效')
         if (version !== this.detailVersion) return
         this.selected = detail
         this.inputMedia = (detail.inputs || []).map(item => ({ ...item, loading: true, error: '', preview_url: '' }))
diff --git a/admin-ui/src/views/chorus/shared.js b/admin-ui/src/views/chorus/shared.js
index 3e572f2..fe86279 100644
--- a/admin-ui/src/views/chorus/shared.js
+++ b/admin-ui/src/views/chorus/shared.js
@@ -2,6 +2,11 @@ export function dataOf(response) {
   return Array.isArray(response && response.data) ? response.data : []
 }
 
+export function detailOf(response) {
+  const data = response && response.data
+  return data && typeof data === 'object' && !Array.isArray(data) ? data : null
+}
+
 export function formatTime(value) {
   if (!value) return '-'
   const date = new Date(value)
diff --git a/admin-ui/tests/unit/chorus/shared.spec.js b/admin-ui/tests/unit/chorus/shared.spec.js
index 5de7f1e..0c91a47 100644
--- a/admin-ui/tests/unit/chorus/shared.spec.js
+++ b/admin-ui/tests/unit/chorus/shared.spec.js
@@ -1,4 +1,4 @@
-import { contains, dataOf, formatTime } from '@/views/chorus/shared'
+import { contains, dataOf, detailOf, formatTime } from '@/views/chorus/shared'
 
 describe('Chorus admin shared helpers', () => {
   test('unwraps only list responses', () => {
@@ -6,6 +6,12 @@ describe('Chorus admin shared helpers', () => {
     expect(dataOf({ data: { id: 1 } })).toEqual([])
   })
 
+  test('unwraps only detail object responses', () => {
+    expect(detailOf({ data: { id: 1, outputs: [] }})).toEqual({ id: 1, outputs: [] })
+    expect(detailOf({ data: [{ id: 1 }] })).toBeNull()
+    expect(detailOf({ data: null })).toBeNull()
+  })
+
   test('filters case-insensitively and handles empty values', () => {
     expect(contains('Provider Alpha', 'alpha')).toBe(true)
     expect(contains(null, 'alpha')).toBe(false)