Skip to content

Commit 6e4294e

Browse files
committed
fix(ws-hook): prefer data.error over data.message in error events
- Error message now uses: data.error || data.message || 'Unknown error' - Server-provided error code is now visible to UI - Same error message passed to setError, setIsRecoverable, and onErrorRef
1 parent 12148a2 commit 6e4294e

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

frontend/src/hooks/useRepoIndexingWebSocket.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -137,9 +137,11 @@ export function useRepoIndexingWebSocket(
137137

138138
case 'error':
139139
setPhase('error')
140-
setError(data.message || 'Unknown error')
140+
// Prefer data.error (server error code) over data.message (human-readable)
141+
const errorMessage = data.error || data.message || 'Unknown error'
142+
setError(errorMessage)
141143
setIsRecoverable(data.recoverable || false)
142-
onErrorRef.current?.(data.message || 'Unknown error', data.recoverable || false)
144+
onErrorRef.current?.(errorMessage, data.recoverable || false)
143145
break
144146
}
145147
} catch (err) {

0 commit comments

Comments
 (0)