public-share: share-ai-widget показывает generic «Something went wrong», игнорируя error.message (нарушает AGENTS.md и паттерн describeChatError) #87

New Issue

Ghost · 2026-06-21T02:33:13+03:00

Ghost commented

2026-06-21 02:33:13 +03:00

Найдено в multi-aspect code review всех изменений с коммита 053a9c0d (ветка develop).

Грань: conventions · Severity: warning
Где: apps/client/src/features/share/components/share-ai-widget.tsx:185-195

Проблема
AGENTS.md:259 запрещает показывать «bare generic string like "Something went wrong"» и требует включать реальную причину. Паттерн репо для этой поверхности — describeChatError(error.message ?? "", t) (chat-thread.tsx:174), различающий 503 (провайдер не настроен) и форвардящий сообщения провайдера (402/429). Share-виджет получает тот же useChat().error, но хардкодит title/тело и не читает error.message. Когда funnel вернёт 429 («temporarily busy») или 503, читателю покажут только «try again», скрыв причину.

Предлагаемый фикс
Импортировать describeChatError из @/features/ai-chat/utils/error-message.ts и отрендерить {describeChatError(error.message ?? "", t)} в теле Alert. Новый хелпер не нужен — форма ошибки уже совпадает.

Найдено в multi-aspect code review всех изменений с коммита `053a9c0d` (ветка develop). **Грань:** conventions · **Severity:** warning **Где:** `apps/client/src/features/share/components/share-ai-widget.tsx:185-195` **Проблема** AGENTS.md:259 запрещает показывать «bare generic string like "Something went wrong"» и требует включать реальную причину. Паттерн репо для этой поверхности — describeChatError(error.message ?? "", t) (chat-thread.tsx:174), различающий 503 (провайдер не настроен) и форвардящий сообщения провайдера (402/429). Share-виджет получает тот же useChat().error, но хардкодит title/тело и не читает error.message. Когда funnel вернёт 429 («temporarily busy») или 503, читателю покажут только «try again», скрыв причину. **Предлагаемый фикс** Импортировать describeChatError из @/features/ai-chat/utils/error-message.ts и отрендерить {describeChatError(error.message ?? "", t)} в теле Alert. Новый хелпер не нужен — форма ошибки уже совпадает.

Ghost referenced this issue from a commit

2026-06-21 03:18:09 +03:00

fix(share): surface the real error on the share AI widget (#87)

Ghost referenced this issue

2026-06-21 04:19:31 +03:00

fix: review/red-team batch 2 — 30 issues (security, ws, page-templates, html-embed, mcp, tests, docs) #101

Ghost closed this issue

2026-06-21 14:10:33 +03:00

Sign in to join this conversation.