Skip to content

[REF] LLM: update models, errors & code cleanup #6

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
Merged
merged 1 commit into from
Nov 21, 2025
Merged

[REF] LLM: update models, errors & code cleanup #6

merged 1 commit into from
Nov 21, 2025

Conversation

@sea-odoo
Copy link
Contributor

@sea-odoo sea-odoo commented Nov 21, 2025

…

Description

  • Modernize type hints
  • Update LLM model list
  • Enhance LLM error handling
  • Use defusedxml

Compliance

  • I have read the contribution guide
  • I made sure the documentation is up-to-date both in doctrings and the docs directory
  • I have added or modified unit tests where necessary
  • I have added new libraries to the requirements.txt file, if any
  • I have incremented the version number according the versioning guide
  • The PR contains my changes only and no other external commit

@sea-odoo sea-odoo requested a review from brinkflew November 21, 2025 18:53
brinkflew
brinkflew approved these changes Nov 21, 2025
View reviewed changes
Copy link
Contributor

@brinkflew brinkflew left a comment

Choose a reason for hiding this comment

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

Niiice!

@sea-odoo sea-odoo merged commit 37a5606 into main Nov 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@brinkflew brinkflew brinkflew approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@sea-odoo @brinkflew