Skip to content

refactor: propagate login errors instead of returning false #25

Description

@dapi

Context

При ревью PR #23 (retry history в JSON output) выявлена проблема в telegram-client.js: login recovery paths возвращают false вместо проброса ошибки.

Problem

  • telegram-client.js:593-596 и 608-611 — session-reset recovery ловит resetError и возвращает false
  • telegram-client.js:612 — финальный catch возвращает false, скрывая все неожиданные ошибки
  • Caller (runAuthLogin в cli.js) получает только false и выбрасывает generic "Failed to login to Telegram." — реальная причина ошибки теряется
  • В --json режиме пользователь не получает никакой диагностической информации

Proposed Solution

  • Выбрасывать resetError (или обёртку) вместо return false
  • Обеспечить что writeError в cli.js получает реальное сообщение об ошибке
  • Обновить listDialogs session-reset recovery (telegram-client.js:631-637) аналогично

Related

🤖 Generated with Claude Code

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions