Skip to content

Conversation

@qnixsynapse
Copy link
Contributor

@qnixsynapse qnixsynapse commented Nov 17, 2025

Describe Your Changes

Migrate the legacy LocalOAIEngine abstraction to a new, generic LocalAIEngine base.

  • Move all OpenAI type definitions to LocalAIEngineTypes and expose them via the engine index.
  • Remove the old LocalOAIEngine files (and its test) and replace them with LocalAIEngine that defines the abstract API for local inference providers.
  • Update the llamacpp-extension to extend LocalAIEngine instead of AIEngine.
  • Adjust exports to include the new engine and types, and clean up unused imports.
  • Add a new test suite for LocalAIEngine ensuring the abstract contract is satisfied.

Fixes Issues

  • Closes #
  • Closes #

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed
  • Adjust FE code

@qnixsynapse qnixsynapse marked this pull request as draft November 17, 2025 08:51
@github-actions
Copy link
Contributor

Barecheck - Code coverage report

Total: 30.03%

Your code coverage diff: -0.02% ▾

✅ All code changes are covered

@louis-jan louis-jan closed this Nov 25, 2025
@louis-jan louis-jan force-pushed the chore/extend_LocalAIEngine branch from 0637715 to 8ce7ef1 Compare November 25, 2025 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants