CLI Audit: Опасные действия без защиты реакций и фильтра

[axisrow]

Из аудита #561, раздел «Опасные действия без защиты».

Поставить реакцию. В строке работает без подтверждения. У всех других опасных действий (переслать, удалить, закрепить, выгнать, отправить) подтверждение есть. В вебе и у агента команды нет вообще.

Очистка фильтра. В строке только «снести всё». В вебе есть «очистить только выбранные». Выборочного варианта в строке нет — приходится сносить всё.

---

Parent: #561

[axisrow]

Аудит претензий

Проверил код по обоим пунктам.

1. Реакция — претензия подтверждается полностью

• В строке dialogs react отрабатывает сразу, без вопроса «точно делаем?» — у соседей (send, forward, delete-message, pin-message, kick, edit-message) такой вопрос есть.
• В вебе нет кнопки реакции вообще.
• У агента нет инструмента «поставить реакцию».
• Контракт действий (telegram_action_inventory) у send_reaction для веба и агента честно стоит «пусто» — то есть пропуск зафиксирован, но никем не закрыт.

2. «Очистка фильтра» — претензия неточная, но рядом есть три реальных дыры

В разделе фильтров три похожие команды, и ни одна не совпадает дословно с текстом из issue:

Команда	Что делает	Выборочно в CLI	Выборочно в вебе	Подтверждение
filter reset	снимает галочку «отфильтрован», ничего не удаляет	нет	нет	нет
filter purge	удаляет накопленные сообщения, каналы остаются	есть (--pks)	есть (галочки)	нет нигде
filter hard-delete	сносит сами каналы из БД (только dev-mode)	есть (--pks) и «все сразу»	только выборочно	есть

Что реально не так:

• purge — массово удаляет сообщения без подтверждения ни в строке, ни нигде.
• reset — выборочного режима нет нигде, а в вебе нет никакой видимой разницы между сбросом одного и всех.
• hard-delete — наоборот: в строке можно «снести всех отфильтрованных», а в вебе такой кнопки нет, только по галочкам.

---

План починки (простыми словами)

Реакция

1. В команде dialogs react спрашивать «уверен? [y/N]» и сделать флаг --yes, чтобы можно было пропустить вопрос.
2. В вебе добавить кнопку «поставить реакцию» рядом с уже существующими «закрепить»/«переслать» в карточке сообщения.
3. У агента добавить инструмент «поставить реакцию» — он будет спрашивать подтверждение через тот же механизм, что и send_message/pin_message.
4. В контракте действий проставить, что у реакции теперь есть и веб-команда, и инструмент агента — чтобы пропуск не вернулся.

Фильтры

5. filter purge (массовое удаление сообщений) — добавить подтверждение в строке. В вебе уже есть наглядный список выбранного перед нажатием кнопки, отдельного подтверждения не нужно.
6. filter reset — добавить выборочный режим:
   • в строке: filter reset --pks 1,2,3,
   • в вебе: отдельная кнопка «сбросить выбранные» рядом с «сбросить все».
7. filter hard-delete — добавить в веб кнопку «удалить все отфильтрованные» с двойным подтверждением (через JS confirm), под флагом dev-mode — для симметрии со строкой.

Что не трогаем

• Бэкенд-методы (TelegramActionService.send_reaction, FilterDeletionService.purge_channels_by_pks/hard_delete_channels_by_pks) уже есть — переиспользуем как есть, новые абстракции не вводим.
• Текущая логика «снести всё» в filter reset и filter purge остаётся как поведение по умолчанию — изменения только дополняющие.

Как проверять

• В строке: dialogs react @канал 123 👍 должна спросить подтверждение; filter purge — тоже; filter reset --pks 1,2 — сбросить только указанные.
• В вебе на странице фильтров должны появиться кнопки «сбросить выбранные», «сбросить все», и под dev-mode «удалить все отфильтрованные».
• В карточке сообщения в вебе должна появиться кнопка реакции.
• У агента должен появиться инструмент send_reaction, требующий confirm=true.
• Должны пройти контрактные тесты tests/test_telegram_action_contract.py.