Skip to content
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

IHttpLlmFunction.name's maximum length limit. #120

Merged
merged 1 commit into from
Jan 9, 2025
Merged

Conversation

samchon
Copy link
Owner

@samchon samchon commented Jan 9, 2025

As ChatGPT does not allow 64 length over function name, this PR shorten it.


This pull request includes several changes aimed at improving the functionality and robustness of the HttpLlm module. The main changes include adding a new feature to limit the length of function names, updating type definitions, and adding a new test to validate these changes.

Key Changes:

Feature Enhancements:

  • Added a maxLength option to limit the length of function names in HttpLlm applications. If a function name exceeds this length, it will be truncated or replaced with a UUID if necessary. (src/HttpLlm.ts, src/composers/HttpLlmApplicationComposer.ts, src/structures/IHttpLlmApplication.ts, src/structures/IHttpLlmFunction.ts, src/structures/ILlmFunction.ts) [1] [2] [3] [4] [5] [6]

Testing:

  • Added a new test file test_http_llm_application_funtion_name_length.ts to ensure that function names in HttpLlm applications do not exceed the specified maxLength and are unique. (test/features/llm/test_http_llm_application_funtion_name_length.ts)

Version Update:

  • Updated the version of the @samchon/openapi package from 2.3.2 to 2.3.3. (package.json)

@samchon samchon added the enhancement New feature or request label Jan 9, 2025
@samchon samchon self-assigned this Jan 9, 2025
Copy link
Owner Author

@samchon samchon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test OK

@samchon samchon merged commit abbb1e9 into master Jan 9, 2025
3 checks passed
@samchon samchon deleted the feat/length branch January 9, 2025 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant