fix(plugins): keep ui_extensions driver filter when parsing plugin manifests (#424)

UIExtensionEntry was missing the driver field, so serde silently dropped
it while deserializing plugin manifests. The frontend then received
entries without a driver filter and rendered every connection_content
contribution for every driver, letting plugins overwrite each other's
connection state in the new-connection modal.

Author: debba

src-tauri/src/drivers/driver_trait.rs

@@ -149,6 +149,10 @@ pub struct UIExtensionEntry {
     /// Ordering weight (lower = earlier).
     #[serde(skip_serializing_if = "Option::is_none")]
     pub order: Option<u32>,
+    /// If set, the contribution is only rendered when the active driver
+    /// matches this identifier (e.g. `"wordpress"`).
+    #[serde(skip_serializing_if = "Option::is_none")]
+    pub driver: Option<String>,
 }
 
 /// A single user-configurable setting declared in a plugin's manifest.

src-tauri/src/plugins/tests.rs

@@ -3,6 +3,7 @@ use std::fs;
 use tempfile::tempdir;
 
 use super::installer::read_plugin_info_from_dir;
+use super::manager::ConfigManifest;
 
 #[test]
 fn reads_installed_plugin_info_from_manifest() {
@@ -27,6 +28,36 @@ fn reads_installed_plugin_info_from_manifest() {
     assert_eq!(plugin.description, "Query Sheets");
 }
 
+#[test]
+fn preserves_ui_extension_driver_filter_from_manifest() {
+    let manifest: ConfigManifest = serde_json::from_str(
+        r#"{
+  "id": "wordpress",
+  "name": "WordPress",
+  "version": "1.0.0",
+  "description": "WordPress driver",
+  "ui_extensions": [
+    {
+      "slot": "connection-modal.connection_content",
+      "module": "ui/dist/index.js",
+      "driver": "wordpress"
+    },
+    {
+      "slot": "data-grid.toolbar.actions",
+      "module": "ui/dist/index.js",
+      "order": 10
+    }
+  ]
+}"#,
+    )
+    .expect("parse manifest");
+
+    let entries = manifest.ui_extensions.expect("ui_extensions present");
+    assert_eq!(entries[0].driver.as_deref(), Some("wordpress"));
+    assert_eq!(entries[1].driver, None);
+    assert_eq!(entries[1].order, Some(10));
+}
+
 #[test]
 fn returns_error_for_invalid_manifest() {
     let dir = tempdir().expect("temp dir");