Skip to content

fix(monitor): exclude business-limited requests from SLA - #125

Merged
JayHome137 merged 2 commits into
sub2api-customfrom
codex/fix-monitor-business-limited
Nov 7, 2026
Merged

JayHome137 merged 2 commits into
sub2api-customfrom
codex/fix-monitor-business-limited

Conversation

@JayHome137

Copy link
Copy Markdown
Owner

Problem

Local business-limited requests such as Insufficient account balance were written into Channel Monitor V2 error_requests, so the UI reported low upstream availability even when the request never reached an upstream account.

Change

  • Carry is_business_limited into the error aggregation CTE.
  • Exclude business-limited rows from SLA request/error and per-user metrics while retaining them in the error breakdown.
  • Classify insufficient account balance as quota_or_balance in Go and SQL taxonomies.
  • Add regression coverage for the SQL filter and taxonomy parity.

Validation

  • go test ./internal/service -count=1
  • go test ./internal/repository -count=1
  • Targeted Channel Monitor V2 tests
  • git diff --check

Target branch: sub2api-custom.

@JayHome137
JayHome137 merged commit f317404 into sub2api-custom Nov 7, 2026
5 checks passed
@JayHome137
JayHome137 deleted the codex/fix-monitor-business-limited branch November 7, 2026 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant