Skip to content

fix(web): показывать ошибки сетевых UI-действий - #732

Merged
konard merged 3 commits into
mainfrom
issue-709-a86119ebe7ec
Jul 14, 2026
Merged

fix(web): показывать ошибки сетевых UI-действий#732
konard merged 3 commits into
mainfrom
issue-709-a86119ebe7ec

Conversation

@konard

@konard konard commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Что изменено

  • добавлена общая безопасная обёртка для асинхронных UI-действий: она преобразует отклонение promise в текст для существующего error banner и не оставляет unhandledrejection;
  • обработка подключена к изменению trust level, блокировке/разблокировке и удалению агента на странице Network;
  • фильтрация по типу события на странице Events теперь также показывает ошибку API;
  • добавлены unit-тесты для Error и произвольных значений rejection.

Как воспроизвести

  1. Открыть Network или Events.
  2. Инициировать одно из затронутых действий при недоступном API либо ответе 500.
  3. До исправления действие молча не выполнялось, а rejection оставался необработанным.
  4. После исправления существующий баннер показывает сообщение API, а promise завершается без необработанного rejection.

Проверки

  • npm test -- --run web/src/lib/__tests__/handleUiAction.test.ts — 2/2 успешно;
  • npm run build:sdk && npm run typecheck — успешно;
  • npm run typecheck --prefix web — успешно;
  • npm run lint — успешно;
  • npm run check:i18n --prefix web — успешно;
  • npm run build:web — успешно;
  • полный npm test: 3813 тестов успешно; два существующих несвязанных теста src/agents/__tests__/service.test.ts падают также при отдельном повторе (ожидание лога дочернего процесса и существующий timeout 10 с).

Fixes #709

Adding .gitkeep for PR creation (default mode).
This file will be removed when the task is complete.

Issue: #709
@konard konard self-assigned this Jul 14, 2026
@konard konard changed the title [WIP] [AUDIT/V7] Network/Events UI mutations are fired via void/.then with no catch, silently swallowing failures fix(web): показывать ошибки сетевых UI-действий Jul 14, 2026
@konard
konard marked this pull request as ready for review July 14, 2026 08:03
@konard

konard commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator Author

Working session summary

Issue #709 исправлен, PR готов к ревью: #732

Что сделано:

  • ошибки trust/block/remove на Network теперь отображаются в существующем баннере;
  • ошибки фильтра Events больше не создают необработанные rejection;
  • добавлена общая безопасная async-обёртка;
  • добавлены два regression-теста;
  • изменения разбиты на два атомарных коммита и отправлены в нужную ветку.

Все свежие CI-проверки на SHA c14e26ff прошли, merge state — CLEAN, рабочее дерево чистое. PR обновлён и переведён в Ready for review.


This summary was automatically extracted from the AI working session output.

@konard

konard commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator Author

🤖 Solution Draft Log

This log file contains the complete execution trace of the AI solution draft process.

💰 Cost estimation:

  • Model: GPT-5.6 Sol
  • Provider: OpenAI
  • Public pricing estimate: $10.061604

📊 Context and tokens usage:

  • 175.8K / 200K (88%) input tokens, 16.9K / 128K (13%) output tokens

Total: (175.8K + 7.5M cached) input tokens, 16.9K output tokens, $10.061604 cost

🤖 Models used:

  • Tool: OpenAI Codex
  • Requested: gpt-5.6-sol
  • Thinking level: off (disabled)
  • Model: GPT 5.6 Sol (gpt-5.6-sol)

📎 Log file uploaded as Gist (2720KB)


Now working session is ended, feel free to review and add any feedback on the solution draft.

@konard
konard merged commit d07cc4f into main Jul 14, 2026
29 checks passed
@konard

konard commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator Author

🎉 Auto-merged

This pull request has been automatically merged by hive-mind.

  • All CI checks have passed

Auto-merged by hive-mind with --auto-merge flag

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.

[AUDIT/V7] Network/Events UI mutations are fired via void/.then with no catch, silently swallowing failures

1 participant