refactor: make DependencyManagers objects of base class

[sdn4z]

No description provided.

[sdn4z]

/lgtm review

[github-actions]

🦉 lgtm Review

Score: Nitpicks 🤓

🔍 Summary

This pull request is a great refactoring of the dependency manager logic. Moving from classes as data containers to instances of a single DependencyManager class is a solid design improvement that simplifies the code and makes it more object-oriented. Consolidating the related logic into a single managers.py file also improves cohesion. The changes have been correctly propagated throughout the codebase, including tests.

I have one main suggestion for improvement regarding encapsulation:

• The src/twyn/dependency_managers/__init__.py file has been emptied. I recommend re-introducing it to export the public symbols of the dependency_managers package. This will hide the internal module structure and provide a more stable API for other parts of the application.

I've also left a minor nitpick about a potential simplification.

Overall, this is a high-quality PR.

More information

• Id: 278d745a471346a4b540d1a1ae4ad0ea
• Model: gemini-2.5-pro
• Created at: 2025-10-29T16:34:05.438853+00:00

Usage summary

• Request count: 2
• Request tokens: 47,680
• Response tokens: 14,704
• Total tokens: 62,384

See the 📚 lgtm-ai repository for more information about lgtm.