Follow-up from "Process usage quota error from Duo Workflow Service"

The following discussion from !214713 (merged) should be addressed:

  • @halilcoban started a discussion:

    issue: Linter is now failing with internal/ai_assist/duoworkflow/handler.go:60:1: cognitive complexity 23 of func (*Handler).Build is high (> 20) (gocognit)

    we could extract some of the logic to helpers.

    However, due to urgency of this MR, I'm fine with handling it in a follow-up too. If so, I believe you need to run

    make golangci > _support/lint_last_known_acceptable.txt

    in order to merge this one.

  • @igor.drozdov

Igor Drozdov @igor.drozdov 5 Dec 2025 at 16:41 Maintainer

nitpick: the message is duplicated here: "handleAgentMessages: failed to read a gRPC message: %w" + "handleWebSocketMessages: usage quota exceeded"

As a result, we receive:

handleAgentMessages: failed to read a gRPC message: handleWebSocketMessages: usage quota exceeded But as long as it's properly detected that the returned error is errUsageQuotaExceededError, then we can fix it in a follow up. I tested locally and I receive Insufficient credits: quota exceeded on close 👍

Edited by 🤖 GitLab Bot 🤖