Skip to content

Plugin Stuff Round 5#654

Merged
lbedner merged 1 commit intomainfrom
plugin-stuff-round5
Apr 30, 2026
Merged

Plugin Stuff Round 5#654
lbedner merged 1 commit intomainfrom
plugin-stuff-round5

Conversation

@lbedner
Copy link
Copy Markdown
Owner

@lbedner lbedner commented Apr 30, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 30, 2026 00:41
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR introduces runtime plugin discovery via Python entry points and wires plugin-provided Typer sub-apps into the main aegis CLI, enabling third-party plugins to extend both the plugin-spec layer and CLI surface.

Changes:

  • Added aegis/core/plugin_discovery.py to discover PluginSpec and plugin CLI typer.Typer apps via the aegis.plugins / aegis.plugins.cli entry-point groups (with caching and error tolerance).
  • Mounted discovered plugin Typer sub-apps into the main CLI in aegis/__main__.py.
  • Added tests to pin discovery behavior and documented the entry-point groups in pyproject.toml.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
aegis/core/plugin_discovery.py Implements entry-point based discovery for plugin specs and CLI sub-apps, including caching and stderr warnings.
aegis/__main__.py Mounts discovered plugin Typer apps under aegis <plugin> ... at CLI initialization time.
tests/core/test_plugin_discovery.py Adds unit tests covering discovery happy paths, error tolerance, collisions, and caching behavior.
pyproject.toml Documents the plugin entry-point group contract via empty entry-point group tables.

Comment thread aegis/core/plugin_discovery.py Outdated
Comment thread aegis/core/plugin_discovery.py
Comment thread aegis/__main__.py
@lbedner lbedner force-pushed the plugin-stuff-round5 branch from 5b980cd to 906eb29 Compare April 30, 2026 00:48
@lbedner lbedner force-pushed the plugin-stuff-round5 branch from 906eb29 to f54d2ea Compare April 30, 2026 00:53
@lbedner lbedner merged commit 4c8bac8 into main Apr 30, 2026
8 checks passed
@lbedner lbedner deleted the plugin-stuff-round5 branch April 30, 2026 01:04
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.

2 participants