Skip to content

Plugin Stuff Round 10#664

Merged
lbedner merged 1 commit intomainfrom
plugin-stuff-round10
May 2, 2026
Merged

Plugin Stuff Round 10#664
lbedner merged 1 commit intomainfrom
plugin-stuff-round10

Conversation

@lbedner
Copy link
Copy Markdown
Owner

@lbedner lbedner commented May 2, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 2, 2026 21:05
@lbedner lbedner force-pushed the plugin-stuff-round10 branch from 13414d6 to 1391209 Compare May 2, 2026 21:06
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

Adds a new aegis plugins update command to re-apply an installed plugin’s templates when the pip-installed version differs from the version recorded in .copier-answers.yml, plus a focused CLI test suite for its dispatch/summary behavior.

Changes:

  • Implement aegis plugins update command flow (target selection, version comparison, re-add via ManualUpdater.add_plugin, and summary/exit code handling).
  • Add CLI tests covering argument validation, no-op paths, success/failure summaries, and “pip-uninstalled plugin” messaging.

Reviewed changes

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

File Description
aegis/commands/plugins.py Introduces plugins update command and helper functions for resolving installed plugin specs and filtering _plugins entries.
tests/cli/test_plugins_update_command.py Adds CLI-level tests for update command behavior with mocked spec resolution / updater.

Comment thread aegis/commands/plugins.py Outdated
Comment thread aegis/commands/plugins.py
Comment thread aegis/commands/plugins.py
@lbedner lbedner force-pushed the plugin-stuff-round10 branch from 1391209 to 7fe39f6 Compare May 2, 2026 21:30
@lbedner lbedner merged commit 24a9d4c into main May 2, 2026
8 checks passed
@lbedner lbedner deleted the plugin-stuff-round10 branch May 2, 2026 21:41
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