picard icon indicating copy to clipboard operation
picard copied to clipboard

Add provision for adding/removing plugin related tags

Open rdswift opened this issue 7 months ago • 5 comments

Summary

  • This is a…
    • [ ] Bug fix
    • [x] Feature addition
    • [ ] Refactoring
    • [ ] Minor / simple change (like a typo)
    • [ ] Other
  • Describe this change in 1-2 sentences: Adds provision to register plugin created tags & variables

Problem

Tags and variables created by plugins do not appear in the autocomplete list or scripting documentation.

  • JIRA ticket (optional): PICARD-XXX

Solution

This PR provides the ability to register plugin-related tags and variables (when a plugin is enabled), and automatically removes (de-registers) the tags & variables when a plugin is no longer enabled.

Action

Additional actions required:

  • [ ] Update Picard documentation (please include a reference to this PR)
  • [ ] Other (please specify below)

rdswift avatar May 04 '25 20:05 rdswift

Need some input as to when to register the tags & variables (when a plugin is loaded or enabled), and whether they should be removed from ALL_TAGS when the plugin is disabled or when the plugin is removed. If not removed from ALL_TAGS until the plugin is removed, then there will need to be some filtering so that tags from installed but not enabled plugins do not appear in the autocomplete list or documentation.

I didn't want to go any further until we are sure that this conceptually fits in with the pending changes to the plugin management system.

rdswift avatar May 04 '25 20:05 rdswift

I also envision adding the name (or at least the ID) of the plugin to the full documentation display, and possibly to the tooltip display.

rdswift avatar May 04 '25 21:05 rdswift

I like it. Not sure if it should be merged yet though (we can merge and modify). New plugin system is the thing we need to work on now, the idea of having a proper way to register variables is definitively the way to go.

zas avatar May 04 '25 21:05 zas

Yes, we should integrate this once the new plugin system is in place. I'll get the current PR into a shape were we can continue from there.

phw avatar May 05 '25 15:05 phw