Use stdlib importlib.metadata instead of importlib_metadata + pkg_resources fallback

[andlo]

What

Replaces the try: from importlib_metadata import entry_points / except ImportError: import pkg_resources fallback in _iter_plugins() with a direct from importlib.metadata import entry_points (stdlib).

Why

This project already requires python_requires >= 3.10 (pyproject.toml), well past the Python 3.8 floor for stdlib importlib.metadata. Both the external importlib_metadata backport and the pkg_resources fallback are unnecessary now.

Confirmed as a real, reproducible bug on real hardware, not just a style cleanup: in a fresh virtualenv where importlib_metadata isn't installed (it's not declared as a dependency of this package), _iter_plugins() falls through to the pkg_resources branch -- which itself raises ModuleNotFoundError, since recent setuptools releases no longer bundle pkg_resources by default (and are removing it entirely per the upstream deprecation notice). This broke skill loading end-to-end for us -- reproduced consistently via skill-ovos-stop: ModuleNotFoundError: No module named 'pkg_resources'.

Using importlib.metadata directly removes both failure modes at once, with no new dependency to declare.

Testing

All 27 existing tests in test/unittests/test_utils.py pass unchanged against this change.

Context

Found while building a venv-per-skill isolation layer for a Home Assistant integration project -- happy to share more reproduction detail if useful.

Summary by CodeRabbit

• Refactor
  • Updated plugin discovery to use Python’s standard library.
  • Removed legacy compatibility paths for plugin discovery.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 1ee70951-5873-4b61-b3b2-85450918d679

📥 Commits

Reviewing files that changed from the base of the PR and between d0aeae5 and d5df7d9.

📒 Files selected for processing (1)

• ovos_plugin_manager/utils/__init__.py

---

📝 Walkthrough

Walkthrough

Entry-point discovery now uses Python’s standard-library importlib.metadata API. The importlib_metadata and pkg_resources fallbacks were removed. _iter_plugins yields entries for the requested group.

Changes

Entry-point discovery

Layer / File(s)	Summary
Standard-library entry-point lookup ovos_plugin_manager/utils/__init__.py	The compatibility imports and pkg_resources fallback were removed. _iter_plugins now uses importlib.metadata.entry_points for the requested group.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: jarbasal

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly identifies the main change: replacing compatibility fallbacks with the standard-library importlib.metadata API.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}