WealthLedger/backend
5
0
Fork 0
Code Issues 6 Pull Requests Actions 1 Packages Projects Releases Wiki Activity

[Feature] Структурированное логирование ошибок и dev-вывода #120

New Issue
Closed
opened 2026-10-20 21:33:25 +03:00 by codex-bot · 0 comments
codex-bot commented 2026-10-20 21:33:25 +03:00
Owner
Copy Link
Copy Source

Контекст

Неожиданные 5xx не логируются: HttpExceptionFilter (@Catch()) превращает любое исключение в ответ 500 и не передаёт ошибку в логгер, поэтому pino-http видит только статус. В dev вывод — сырой JSON без цвета (pino-pretty не подключён). Полные тела запросов и ответов не логируются намеренно (ADR 008, backend/.ai/conventions/observability.md) и вне scope этой задачи.

Acceptance Criteria

  1. Неожиданная ошибка (не HttpException) логируется через StructuredLogger.event('error', ...) с полями code, name и stack. Поле message не пишется. Публичный ответ problem+json не меняется.
  2. Для 4xx validation-ошибок в лог попадают имена невалидных полей без значений. Тело ответа не меняется.
  3. В completion-строке HTTP указывается userId, если он есть в res.locals.
  4. В NODE_ENV=development вывод форматируется pino-pretty с цветом уровней. В production пакет не загружается, production-зависимостей не добавляется.
  5. Route для 404 остаётся <unmatched> (защита от кардинальности).
  6. Тесты: HTTP-уровень для фильтра (500 пишет error с полями, ответ не изменён), отсутствие значений в полях 4xx, unit для completion/userId.

Вне scope

  • Полные тела запросов и ответов (требует нового ADR).
  • Сбор, хранение и alerting логов.
## Контекст Неожиданные 5xx не логируются: `HttpExceptionFilter` (`@Catch()`) превращает любое исключение в ответ 500 и не передаёт ошибку в логгер, поэтому pino-http видит только статус. В dev вывод — сырой JSON без цвета (`pino-pretty` не подключён). Полные тела запросов и ответов не логируются намеренно (ADR 008, `backend/.ai/conventions/observability.md`) и вне scope этой задачи. ## Acceptance Criteria 1. Неожиданная ошибка (не `HttpException`) логируется через `StructuredLogger.event('error', ...)` с полями `code`, `name` и stack. Поле `message` не пишется. Публичный ответ problem+json не меняется. 2. Для 4xx validation-ошибок в лог попадают имена невалидных полей без значений. Тело ответа не меняется. 3. В completion-строке HTTP указывается `userId`, если он есть в `res.locals`. 4. В `NODE_ENV=development` вывод форматируется `pino-pretty` с цветом уровней. В production пакет не загружается, production-зависимостей не добавляется. 5. Route для 404 остаётся `<unmatched>` (защита от кардинальности). 6. Тесты: HTTP-уровень для фильтра (500 пишет error с полями, ответ не изменён), отсутствие значений в полях 4xx, unit для completion/`userId`. ## Вне scope - Полные тела запросов и ответов (требует нового ADR). - Сбор, хранение и alerting логов.
codex-bot added the Kind/Enhancement
Priority
Medium
3
 labels 2026-10-20 21:33:29 +03:00
hermes-agent was assigned by codex-bot 2026-10-20 21:33:29 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Kind/Dependencies
Dependency update or dependency-management work
 Kind/Investigation
Research or decision required before implementation
 Kind/Process
Developer workflow, CI/CD, tooling, or process improvement
 Kind/Refactor
Internal code or configuration restructuring without new product behavior
 audit/2026-10-11
Technical Audit Remediation — 2026-10-11
 kind/investigation
Нужна проверка evidence до решения; runtime-дефект не утверждается
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Medium
3
The priority is medium
Priority
Low
4
The priority is low
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
Status
Abandoned
3
Somebody has started to work on this but abandoned work
No labels Kind/Enhancement
Priority
Medium
3
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
codex-bot hermes-agent ksv741 publisher-bot renovate-bot
No Assignees hermes-agent
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: WealthLedger/backend#120