fix: Error: Limit must be between 1 and 100 (#8010)#8012
fix: Error: Limit must be between 1 and 100 (#8010)#8012erxes-dev-agent[bot] wants to merge 1 commit into
Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideAdds server-side normalization of the Sequence diagram for normalized limit in team member user querysequenceDiagram
participant Client
participant userQueries
participant cursorPaginate
participant UsersModel
Client->>userQueries: userQueries.users(args)
userQueries->>userQueries: Math.max(1, args.limit || 20)
userQueries->>userQueries: Math.min(100, computedLimit)
userQueries->>cursorPaginate: cursorPaginate({ model: UsersModel, params: { ...args, limit }, query: selector })
cursorPaginate->>UsersModel: findWithCursor(params, query)
cursorPaginate-->>userQueries: { list, totalCount, pageInfo }
userQueries-->>Client: { list, totalCount, pageInfo }
File-Level Changes
Assessment against linked issues
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| JavaScript | Jun 12, 2026 5:55p.m. | Review ↗ | |
| Docker | Jun 12, 2026 5:55p.m. | Review ↗ |
Important
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
|
🤖 autofix: CI is green ✅ (converged in 1 round(s)). |
|
|
@erxes-dev-agent what are you doing? |
|
I'm reviewing PR #8012 ( This is a read-only bug fix for issue #8010: I'm ready to answer questions about this specific PR — the diff, the fix approach, potential edge cases, or how it relates to the reported issue. |
Automated fix for #8010 generated by OpenCode (kimi) via the erxes-sentry-fixer agent.
Closes #8010
Summary by Sourcery
Bug Fixes: