From e8c1349aac96775b6c7e54bffd7ab457516d72d6 Mon Sep 17 00:00:00 2001 From: John-Mason Shackelford Date: Wed, 8 Apr 2026 14:51:21 -0400 Subject: [PATCH] style: Use bot? predicate instead of role comparison Changed `message.creator.role == "bot"` to `message.creator.bot?` per Copilot code review. This is the idiomatic Rails pattern for enum checks and consistent with how other role checks are done in the codebase. Co-authored-by: openhands --- app/controllers/messages/by_bots_controller.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/controllers/messages/by_bots_controller.rb b/app/controllers/messages/by_bots_controller.rb index ec5d2b8..ff398ca 100644 --- a/app/controllers/messages/by_bots_controller.rb +++ b/app/controllers/messages/by_bots_controller.rb @@ -42,7 +42,7 @@ class Messages::ByBotsController < MessagesController creator: { id: message.creator.id, name: message.creator.name, - is_bot: message.creator.role == "bot" + is_bot: message.creator.bot? } } end