Skip to content

fix: use correct error message when user-installed plugin is outdated#518

Merged
datlechin merged 1 commit intomainfrom
fix/plugin-outdated-error-message
Mar 31, 2026
Merged

fix: use correct error message when user-installed plugin is outdated#518
datlechin merged 1 commit intomainfrom
fix/plugin-outdated-error-message

Conversation

@datlechin
Copy link
Copy Markdown
Collaborator

Summary

  • When a user-installed plugin has a lower PluginKit version than the app requires, the error message was misleading — it said "Plugin requires PluginKit version 3, but app provides version 2" when the reality was the opposite (app requires v3, plugin was built with v2)
  • Added a dedicated pluginOutdated error case with a clear message: "Plugin was built with PluginKit version X, but version Y is required. Please update the plugin."
  • Updated 3 call sites in PluginManager (2 throws + 1 log message) to use the new error case

Test plan

  • Install a plugin built with an older PluginKit version and verify the new error message appears correctly
  • Verify plugins with matching PluginKit versions still load normally
  • Verify plugins with a newer PluginKit version than the app still show the original incompatibleVersion message

@datlechin datlechin merged commit 6187af4 into main Mar 31, 2026
2 checks passed
@datlechin datlechin deleted the fix/plugin-outdated-error-message branch March 31, 2026 03:06
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