Skip to content

fix(web): показывать реальные 422 ошибки терминала#249

Merged
gently-whitesnow merged 1 commit into
masterfrom
fix/terminal-422-problem-details
Jul 13, 2026
Merged

fix(web): показывать реальные 422 ошибки терминала#249
gently-whitesnow merged 1 commit into
masterfrom
fix/terminal-422-problem-details

Conversation

@gently-whitesnow

Copy link
Copy Markdown
Owner

Цель

Показывать в панели терминала реальную причину 422 из ProblemDetails вместо универсальной tmux-заглушки.

Изменения

  • Добавлен терминальный mapper по ProblemDetails.code для repository.provider_not_authenticated, capability.disabled, terminal.clone_wait_timeout и других терминальных 422-кодов.
  • Unknown 422 теперь падает в detail/title из ответа API, а не в текст про tmux.
  • В shared/lib добавлены helper'ы для чтения detail/title/extension-полей HttpError.
  • Добавлены тесты на GitLab provider_not_authenticated, tmux capability.disabled, clone_wait_timeout и unknown 422 fallback.

Проверка

  • pnpm -C apps/web test -- src/widgets/intent-panels/agent-terminal-panel/model/terminal-error-message.test.ts
  • pnpm -C apps/web typecheck
  • pnpm -C apps/web lint (есть существующее warning в ReviewInlineComposer.tsx про persistDraft, без новых ошибок)

Решения:
- Вынес маппинг ошибок терминальной сессии в отдельный helper, чтобы 422 разбирался по ProblemDetails.code, а не схлопывался в tmux-заглушку.
- Добавил чтение detail/title/extension-полей HttpError через shared/lib, чтобы fallback мог показывать реальную причину API.
- Покрыл маппинг тестами для GitLab provider_not_authenticated, tmux capability.disabled, clone_wait_timeout и неизвестного 422-кода.

Отброшено:
- Не менял бэкенд-гейты и pre-flight проверки: registry уже отдаёт repository.provider_not_authenticated как 422, проблема была в UX-маппинге фронта.

Ограничения:
- DNS/VPN-доступ к gitlab.ati.st не исправлялся; теперь фронт только честно показывает эту причину.
@gently-whitesnow
gently-whitesnow merged commit 7b85c2e into master Jul 13, 2026
1 check passed
@gently-whitesnow
gently-whitesnow deleted the fix/terminal-422-problem-details branch July 13, 2026 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant