Skip to content

Plugin Stuff#649

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

Plugin Stuff#649
lbedner merged 1 commit intomainfrom
plugin-stuff

Conversation

@lbedner
Copy link
Copy Markdown
Owner

@lbedner lbedner commented Apr 29, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 29, 2026 01:31
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

Introduces a declarative “file manifest” mechanism to drive post-generation cleanup based on selected components/services, as a first step (R1) in a plugin-system refactor.

Changes:

  • Added aegis.core.file_manifest with FileManifest, plus helpers to iterate cleanup paths and apply file/dir removal.
  • Updated component/service registries to declare files: FileManifest ownership (primary paths) for Pattern A cleanup.
  • Refactored cleanup_components() to remove per-spec file/dir blocks in favor of iterating manifests; added contract/sanity tests for the manifest behavior and registry shape.

Reviewed changes

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

Show a summary per file
File Description
aegis/core/file_manifest.py New manifest model + cleanup helpers (R1 scope).
aegis/core/post_gen_tasks.py Uses manifests for per-spec cleanup (Pattern A) and keeps option-driven cleanups inline.
aegis/core/components.py Adds files: FileManifest to component specs and populates primary cleanup paths.
aegis/core/services.py Adds files: FileManifest to service specs and populates primary cleanup paths.
tests/core/test_file_manifest.py New tests locking in manifest contract and validating registries declare manifests.

Comment thread aegis/core/components.py
Comment thread aegis/core/components.py Outdated
Comment thread aegis/core/services.py Outdated
Comment thread aegis/core/file_manifest.py Outdated
Comment thread aegis/core/post_gen_tasks.py
@lbedner lbedner merged commit ce81827 into main Apr 29, 2026
8 checks passed
@lbedner lbedner deleted the plugin-stuff branch April 29, 2026 02: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