Кнопка 'Взять диалог' не работает + UX проблемы на странице чатов

[dapi]

Описание проблемы

На странице чатов (/chats) обнаружено три UX-проблемы:

1. Кнопка "Взять диалог" не работает 🔴 Critical

Ожидаемое поведение: При клике на кнопку "Взять диалог" чат переходит в режим менеджера.

Фактическое поведение: Ничего не происходит, кнопка остаётся без изменений.

Причина: Контроллер Tenants::Chats::ManagerController#takeover всегда рендерит JSON ответ (строки 69-77), игнорируя turbo_stream запросы:

# app/controllers/tenants/chats/manager_controller.rb:60-78
def takeover
  result = Manager::TakeoverService.call(...)
  
  if result.success?
    render json: { ... }  # <-- всегда JSON!
  else
    render json: { ... }
  end
end

При этом turbo_stream шаблон app/views/tenants/chats/takeover.turbo_stream.slim существует, но не используется!

Решение: Добавить respond_to блок:

def takeover
  result = Manager::TakeoverService.call(...)

  respond_to do |format|
    format.turbo_stream if result.success?
    format.json do
      if result.success?
        render json: { success: true, chat: chat_json(result.chat), ... }
      else
        render json: { success: false, error: result.error }, status: :unprocessable_entity
      end
    end
  end
end

2. Курсор над кнопкой не pointer 🟡 Minor

Ожидаемое поведение: При наведении на кнопку "Взять диалог" курсор должен меняться на pointer.

Фактическое поведение: Курсор остаётся стандартным.

Файл: app/views/tenants/chats/_chat_controls.html.slim:8

Решение: Добавить класс cursor-pointer к кнопке.

3. URL не обновляется при клике на чат 🟡 Medium

Ожидаемое поведение: При клике на чат в списке URL должен меняться на /chats/123.

Фактическое поведение: URL остаётся /chats или /chats?page=2, пользователь не может поделиться ссылкой на конкретный чат.

Причина: В link_to отсутствует data-turbo-action="advance":

# app/views/tenants/chats/_chat_list.html.slim:4
= link_to tenant_chat_path(chat), class: "...", id: "chat_list_item_#{chat.id}"

Решение: Добавить data: { turbo_action: "advance" }:

= link_to tenant_chat_path(chat), class: "...", id: "chat_list_item_#{chat.id}", data: { turbo_action: "advance" }

Файлы для изменения

• app/controllers/tenants/chats/manager_controller.rb — добавить respond_to в takeover, release, create_message
• app/views/tenants/chats/_chat_controls.html.slim — добавить cursor-pointer
• app/views/tenants/chats/_chat_list.html.slim — добавить data-turbo-action="advance"

Шаги для воспроизведения

1. Авторизоваться в tenant dashboard
2. Перейти на /chats
3. Кликнуть на чат в списке — URL не меняется
4. Навести курсор на кнопку "Взять диалог" — курсор не pointer
5. Кликнуть на кнопку "Взять диалог" — ничего не происходит

Среда

• URL: https://demo.3012.brandymint.ru/chats
• Браузер: любой

Связанные документы

• US-010: Tenant Chat Takeover
• TSD-011: Tenant Chat Takeover Technical Specification

[dapi]

Дополнение: кнопка "Вернуть боту" тоже не обновляет UI

Такая же проблема с методом release:

• Действие происходит (чат возвращается боту на бэкенде)
• Но UI не обновляется — кнопка остаётся прежней

Причина: Метод release также рендерит только JSON:

# app/controllers/tenants/chats/manager_controller.rb:118-134
def release
  result = Manager::ReleaseService.call(...)

  if result.success?
    render json: { ... }  # <-- только JSON, turbo_stream игнорируется
  else
    render json: { ... }
  end
end

Шаблон release.turbo_stream.slim существует, но не используется.

Итого нужно исправить все три метода:

• takeover — кнопка "Взять диалог"
• release — кнопка "Вернуть боту"
• create_message — отправка сообщения от менеджера

[dapi]

План имплементации (для обсуждения)

Подход: JSON → Turbo Streams

Вместо JSON ответов использовать Turbo Streams для обновления UI. Шаблоны уже есть:

• takeover.turbo_stream.slim
• release.turbo_stream.slim
• send_message.turbo_stream.slim

Файлы для изменения

1. app/controllers/tenants/chats/manager_controller.rb

Заменить render json: {...} на render turbo_stream: с массивом turbo_stream actions:

def takeover
  result = Manager::TakeoverService.call(...)
  
  if result.success?
    @chat.reload
    render turbo_stream: [
      turbo_stream.replace("chat_#{@chat.id}_header", partial: 'chat_header', locals: { chat: @chat }),
      turbo_stream.replace("chat_#{@chat.id}_controls", partial: 'chat_controls', locals: { chat: @chat }),
      turbo_stream.replace("chat_#{@chat.id}_status", partial: 'status', locals: { chat: @chat })
    ]
  else
    render_error(result.error)
  end
end

2. app/views/tenants/chats/_chat_controls.html.slim

Добавить cursor-pointer к кнопкам:

= button_to ..., class: "... cursor-pointer"

3. app/views/tenants/chats/_chat_list.html.slim

Добавить data-turbo-action="advance" для обновления URL:

= link_to tenant_chat_path(chat), ..., data: { turbo_action: "advance" }

4. Обработка ошибок

Варианты:

• A) Flash message через turbo_stream.prepend в контейнер #flash
• B) Inline error рядом с кнопкой
• C) Toast notification

Какой вариант предпочтительнее?

5. Дополнительно

• Нужно добавить контейнер #flash в layout (если нет)
• Добавить partial _flash_message.html.slim для отображения ошибок
• Добавить переводы в config/locales/ru.yml

Вопросы

1. Удалять ли turbo_stream шаблоны-файлы (takeover.turbo_stream.slim и т.д.) если будем рендерить inline в контроллере?
2. Нужен ли fallback на JSON для API-клиентов или только Turbo?
3. Как показывать ошибки пользователю — flash, toast, inline?

---

Жду фидбек перед началом работы.