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.

Sign up for GitHub

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

Jump to bottom

fix: add length check for toolRequestParts in `fromOpenAiChoice` #138

Open

pedi wants to merge 2 commits into TheFireCo:main from pedi:fix/openai-streaming-response

pedi commented Sep 3, 2024

This pull request is related to:

A bug
A new feature
Documentation
Other (please specify)

I have checked the following:

I have read and understood the contribution guidelines and the code of conduct;
I have added new tests (for bug fixes/features);
I have added/updated the documentation (for bug fixes / features).

Description:
Please check description in the original issue

Related issues:
Fix #137


          fix: add length check for toolRequestParts in fromOpenAiChoice

aaf43c6

pedi requested a review from Dabolus as a code owner

September 3, 2024 04:26


          add built lib

949e3f0

pedi requested review from retzd-tech, davidoort and EPMatt as code owners

September 4, 2024 01:19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Dabolus Awaiting requested review from Dabolus Dabolus is a code owner

retzd-tech Awaiting requested review from retzd-tech retzd-tech is a code owner

davidoort Awaiting requested review from davidoort davidoort is a code owner

EPMatt Awaiting requested review from EPMatt EPMatt is a code owner

At least 1 approving review is required to merge this pull request.

Labels

None yet