Skip to content

[OMEGA-481] Fix: Plugin module identity - #389

Open
timur-ashkenov wants to merge 3 commits into
singnet:mainfrom
timur-ashkenov:fix/plugin-module-identity
Open

timur-ashkenov wants to merge 3 commits into
singnet:mainfrom
timur-ashkenov:fix/plugin-module-identity

Conversation

@timur-ashkenov

Copy link
Copy Markdown
Collaborator

Description

Fixes #360: register location-loaded plugins in sys.modules to prevent duplicate instances and state. Rename the OpenAI plugin to omega_openai to avoid SDK conflicts.

How Has This Been Tested?

Verified shared module state, repeated loading, failure cleanup, name collisions, and Telegram queue sharing.

  • tests/: 138 passed, 4 skipped.
  • Autotests/unit/: 176 passed.

Checklist

  • PR contains autogenerated code
  • Self-review completed
  • Test scenarios above are passed with the version of the code from PR

This branch has not been deployed

No deployments
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.

Plugins loaded by location are not registered in sys.modules, so importing one elsewhere creates a second instance

1 participant