Skip to content

feat(telegram): обязательный alias для бриджинга - #34

Merged
Flexlug merged 9 commits into
mainfrom
worktree-config-example-review
Jul 4, 2026
Merged

feat(telegram): обязательный alias для бриджинга#34
Flexlug merged 9 commits into
mainfrom
worktree-config-example-review

Conversation

@Flexlug

@Flexlug Flexlug commented Jul 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • Сообщения Telegram-пользователей без alias больше не публикуются в LoRa — гейт в on_message, сразу после проверки is_disabled.
  • Новый флаг require_alias: bool = True в TelegramCommandsConfig; действует для всех ролей без исключений (OWNER включительно).
  • UX: реакция 🪪 + самоудаляющееся через 5с напоминание про /set_alias (личка с ботом, т.к. командный роутер private_only=True).

Дизайн и план: docs/superpowers/specs/2026-07-04-require-alias-design.md, docs/superpowers/plans/2026-07-04-require-alias.md.

Test plan

  • pytest -q — 361 passed
  • ruff check — clean
  • mypy lora_bridge — clean
  • Новые тесты: tests/test_telegram_require_alias.py, доп. кейсы в test_telegram_reactions.py и test_config_schema.py

Flexlug and others added 7 commits July 4, 2026 16:10
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…nFeedback

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…-нарровингу

mypy не сужает объявленный Optional[UserSettings] после присваивания
не-Optional значения (в отличие от pyright); без аннотации сужение
работает по ветке if/else как положено.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@Flexlug
Flexlug force-pushed the worktree-config-example-review branch from 5ea0ff3 to 4ebc8e8 Compare July 4, 2026 13:11
Flexlug and others added 2 commits July 4, 2026 16:17
_delete_after в commands/framework.py и новый _delete_reply_after в
reactions.py делали одно и то же — единый примитив для самоудаляющихся
служебных сообщений бота.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@Flexlug
Flexlug merged commit efe1d88 into main Jul 4, 2026
6 checks passed
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