-
Notifications
You must be signed in to change notification settings - Fork 904
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Truncate messages #294
Conversation
…unt of tokens is too big
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❌ Changes requested.
- Reviewed the entire pull request up to c0a9076
- Looked at
460
lines of code in8
files - Took 1 minute and 6 seconds to review
More info
- Skipped
1
files when reviewing. - Skipped posting
0
additional comments because they didn't meet confidence threshold of50%
.
Workflow ID: wflow_t4jriLSZ8HMSJ3cW
Want Ellipsis to fix these issues? Tag @ellipsis-dev
in a comment. We'll respond in a few minutes. Learn more here.
c0a9076
to
59471d4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me!
- Performed an incremental review on 59471d4
- Looked at
488
lines of code in7
files - Took 1 minute and 44 seconds to review
More info
- Skipped
1
files when reviewing. - Skipped posting
0
additional comments because they didn't meet confidence threshold of50%
.
Workflow ID: wflow_Zba7eA2dT6Z2WIvg
Not what you expected? You can customize the content of the reviews using rules. Learn more here.
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
Summary:
This PR introduces a message truncation feature in the
BaseSession
class, anInputTooBigError
exception, and corresponding tests.Key points:
truncate
method in theBaseSession
class in/agents-api/agents_api/routers/sessions/session.py
.InputTooBigError
in/agents-api/agents_api/routers/sessions/exceptions.py
.test_messages_truncation.py
.Generated with ❤️ by ellipsis.dev