Fix timeout openai - Githubissues

f3rnandomoreno / spring-ai-telegram-task-manager

A telegram task manager chatbot using openAI API

0 stars 0 forks source link

Fix timeout openai #3

Closed f3rnandomoreno closed 2 weeks ago

stackblitz[bot] commented 2 weeks ago

Run & review this pull request in StackBlitz Codeflow.

f3rnandomoreno commented 2 weeks ago

This pull request includes several updates to the project, primarily focusing on dependency updates, code refactoring, and configuration changes. The most important changes include updating dependencies in pom.xml, refactoring the ManagerAgent and NL2SQLAgent classes to use constructor injection, and adding a new integration test for the TelegramClientConfig class.

Dependency Updates:

Updated java.version to 17 and spring-ai.version to 1.0.0 in pom.xml.
Added a new dependency for spring-ai-ollama-spring-boot-starter in pom.xml.

Code Refactoring:

Refactored ManagerAgent and NL2SQLAgent to use constructor injection instead of field injection. [1] [2]
Removed unused imports and reordered existing ones for better readability in several files. [1] [2]

Configuration Changes:

Modified ChatClientConfig to remove specific OpenAiChatOptions and use the default configuration.
Updated TelegramBotConfig to use OkHttpTelegramClient directly instead of creating a new instance within the constructor.

New Tests:

Added a new integration test for TelegramClientConfig to verify the handling of incoming messages and the generation of appropriate responses.

Minor Fixes:

Updated application.properties to enable and configure the OpenAI and Ollama chat options.
Fixed a bug in the TODOs list to ensure the chatbot always shows the database IDs.