Skip to content

Conversation

Copy link

Copilot AI commented Nov 21, 2025

Refactoring to use only new class pattern (OpenAI, Claude, etc.)

New Requirement:

  • Remove all legacy Wrapper class aliases
  • Refactor client.py to not depend on Wrapper classes
  • Update examples to use only the new pattern
  • Use operation_mode="online" to avoid embedding model downloads

Plan:

  • Fix: Create alias classes (OpenAIWrapper = OpenAI, etc.) in each wrapper file
  • Test: Verify all imports work correctly
  • NEW: Remove all Wrapper class aliases
  • NEW: Refactor client.py Memory class
  • NEW: Update wrappers/init.py to remove Wrapper references
  • NEW: Update examples to use new pattern with operation_mode="online"
  • Test: Verify all imports still work
  • Test: Run updated examples
  • Consider: Add LM Studio support as requested by user (separate issue)
Original prompt

This section details on the original issue you should resolve

<issue_title>It won´t work. There is a bug in the code.</issue_title>
<issue_description>I tried to install using pip. after installation i just made a basic test with running your basic usage example in python. It returns an error that openaiapiwrapper can not be found or is not defined. Since i am no coder, just a simple user i am sorry i can´t provide any more details. P.S. i would love to use it locally but i don´t like ollama. instead i love to use LM Studio. Is there a way to include LM Studio Server in your wrappers collection?</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI self-assigned this Nov 21, 2025
@divagr18
Copy link
Owner

@copilot clean up and commit

@divagr18 divagr18 marked this pull request as ready for review November 21, 2025 06:51
@divagr18 divagr18 closed this Nov 21, 2025
@divagr18 divagr18 deleted the copilot/fix-openaiapiwrapper-bug branch November 21, 2025 07:02
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.

It won´t work. There is a bug in the code.

2 participants