Follow-up from "Check Duo Chat access via GraphQL API"
The following discussion from !141389 (merged) should be addressed:
-
@jessieay started a discussion: (+1 comment) Question (non-blocking): why are we checking this feature flag in so many places rather than adding it to one place? I see it being checked here, and in the GraphQL mutation for askinga chat question and also in
ee/app/services/llm/internal/completion_service.rb
and inee/app/services/llm/chat_service.rb
and inee/lib/gitlab/llm/tanuki_bot.rb
. Can we consolidate?