Add Projects rate limit UI change #1603
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Owner
logancyang
commented
Jul 3, 2025
logancyang
commented
- Add rate limit info to tooltip
- Add a notice when rate limit is reached
- Added checks for rate limit errors in ProjectManager and ChatControls, re-throwing errors when detected. - Introduced a method in Docs4LLMParser to show user-friendly notices for rate limit exceedances, limiting notifications to once per minute. - Updated error logging to use a centralized logging function for consistency.
|
bugbot run |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
- Updated error checks in ProjectManager and ChatControls to include status code 429 for better rate limit detection. - Ensured that errors are re-thrown when rate limits are exceeded, maintaining consistent error handling across components.
|
bugbot run |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
- Introduced a centralized utility function `isRateLimitError` to streamline rate limit error detection in ProjectManager, ChatControls, and Docs4LLMParser. - Removed redundant error checking logic from multiple locations, enhancing code maintainability. - Added a new utility function `extractRetryTime` for better handling of retry time extraction from error messages.
|
bugbot run |
Merged
wenzhengjiang
pushed a commit
that referenced
this pull request
Jul 9, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.