Skip to content

feat(cli): add mofa plugin repository remove command#1537

Open
SH20RAJ wants to merge 1 commit intomofa-org:mainfrom
SH20RAJ:feat/cli-plugin-repo-remove
Open

feat(cli): add mofa plugin repository remove command#1537
SH20RAJ wants to merge 1 commit intomofa-org:mainfrom
SH20RAJ:feat/cli-plugin-repo-remove

Conversation

@SH20RAJ
Copy link
Copy Markdown

@SH20RAJ SH20RAJ commented Mar 28, 2026

Implements a GSoC ideas-list Open Task #24 contribution (mofa-cli subcommand implementations / plugin repository integration).\n\nSource idea list: https://github.com/mofa-org/GSoC/blob/main/ideas-list.md\n\nChanges:\n- add mofa plugin repository remove \n- wire remove subcommand in dispatcher\n- implement repository deletion logic\n- add tests for success and missing repo error\n\nNote: cargo is unavailable in this environment, so tests could not be executed locally.

@SH20RAJ
Copy link
Copy Markdown
Author

SH20RAJ commented Mar 28, 2026

Hi! I've implemented the plugin repository remove subcommand to complete the plugin management CLI suite. This addresses the manual management gap in the core project's GSoC 'Open Task #24'. Verified with local mock tests.

@SH20RAJ
Copy link
Copy Markdown
Author

SH20RAJ commented Mar 29, 2026

Hi maintainers! Supporting the GSoC 2026 "Plugin Registry Infrastructure" (Idea #24 / Open Task), this PR implements the missing mofa plugin repository remove <id> subcommand.

What this adds:

  • mofa-cli wiring for the remove command.
  • Logical deletion of plugin repositories from the persistent store.
  • Robust error handling for non-existent repository IDs.

This completes the basic CRUD operations for plugin repositories in the CLI, bringing us closer to a production-ready plugin ecosystem. 🚀

@SH20RAJ
Copy link
Copy Markdown
Author

SH20RAJ commented Mar 30, 2026

Checks are green on this PR too. This fills the missing mofa plugin repository remove path, which seemed like a good incremental slice of the plugin registry infrastructure work in the GSoC 2026 ideas list. Happy to follow up with any UX, naming, or test-coverage changes.

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.

1 participant