fix: 修复空会话返回 null 导致前端崩溃的问题

## 问题描述
当用户创建新会话但未发送任何消息就切换到其他会话时,前端控制台报错:
"TypeError: Cannot read properties of null (reading 'map')"

根本原因:Go 后端未初始化的切片序列化为 JSON 时会变成 `null` 而非 `[]`,
前端尝试对 `null` 调用 `.map()` 导致崩溃。

## 修复方案
采用多层防御策略,同时修复后端和前端:

### 后端修复(确保 API 契约正确)
1. message_pg.go:93 - 将 `var messages []StoredMessage` 改为
   `messages := make([]StoredMessage, 0)`,确保空结果序列化为 `[]`
2. conversation.go - 在两个响应路径(PG 查询 + 内存回退)添加防御性 nil 检查

### 前端防御(多层保护)
1. useSessionList.ts - 在 loadMessages 和 loadSessions 中添加 null 合并操作
   `(res.data.messages || [])` 确保即使后端退化也不会崩溃

## 影响范围
- 所有空会话(新建后未发送消息的对话)现在可以正常切换
- API 响应符合 JSON 最佳实践(数组字段永远是 `[]` 而非 `null`)
This commit is contained in:
hhs
2026-06-22 12:39:37 +08:00
parent 2180751a9b
commit 4cc0713459
3 changed files with 9 additions and 3 deletions

View File

@@ -273,6 +273,9 @@ func (h *ConversationHandler) GetMessages(c *gin.Context) {
return return
} }
count, _ := h.msgRepo.GetMessageCount(c.Request.Context(), sessionID) count, _ := h.msgRepo.GetMessageCount(c.Request.Context(), sessionID)
if messages == nil {
messages = []store.StoredMessage{}
}
c.JSON(http.StatusOK, gin.H{ c.JSON(http.StatusOK, gin.H{
"messages": messages, "messages": messages,
"total": count, "total": count,
@@ -314,6 +317,9 @@ func (h *ConversationHandler) GetMessages(c *gin.Context) {
} }
messages := allMessages[start:] messages := allMessages[start:]
if messages == nil {
messages = []models.Message{}
}
c.JSON(http.StatusOK, gin.H{ c.JSON(http.StatusOK, gin.H{
"messages": messages, "messages": messages,
"total": total, "total": total,

View File

@@ -90,7 +90,7 @@ func (r *PgMessageRepository) queryMessages(ctx context.Context, query string, a
} }
defer pgxRows.Close() defer pgxRows.Close()
var messages []StoredMessage messages := make([]StoredMessage, 0)
for pgxRows.Next() { for pgxRows.Next() {
var m StoredMessage var m StoredMessage
if err := pgxRows.Scan(&m.ID, &m.SessionID, &m.Role, &m.Content, &m.TokensUsed, &m.CreatedAt); err != nil { if err := pgxRows.Scan(&m.ID, &m.SessionID, &m.Role, &m.Content, &m.TokensUsed, &m.CreatedAt); err != nil {

View File

@@ -80,7 +80,7 @@ export function useSessionList(accessToken?: string | null) {
try { try {
const res = await listConversations(accessToken); const res = await listConversations(accessToken);
if (res.data) { if (res.data) {
const list = res.data.conversations.map(toSessionSummary); const list = (res.data.conversations || []).map(toSessionSummary);
setSessions(list); setSessions(list);
// 恢复上次选中的会话(如果仍然存在) // 恢复上次选中的会话(如果仍然存在)
const lastId = localStorage.getItem(LAST_ACTIVE_KEY); const lastId = localStorage.getItem(LAST_ACTIVE_KEY);
@@ -193,7 +193,7 @@ export function useSessionList(accessToken?: string | null) {
try { try {
const res = await getConversationMessages(accessToken, sessionId); const res = await getConversationMessages(accessToken, sessionId);
if (res.data) { if (res.data) {
return res.data.messages.map(toChatMessage); return (res.data.messages || []).map(toChatMessage);
} }
} catch (err) { } catch (err) {
console.error("[SessionList] 加载消息失败:", err); console.error("[SessionList] 加载消息失败:", err);