test(config): set-equality guard for ActionType vs schema actionType.enum

Adding or renaming an ActionType variant in Rust without the matching change
to $defs.actionType.enum in config.v2.schema.json (or vice versa) silently
breaks saving/exporting any profile using it - the same class of bug that once
hit repairClipboard. action_type_set_matches_schema_enum asserts the two list
the same set; an always-on exhaustive match forces a new variant to be handled
here and in the schema. Mirrors control_id_as_str_matches_serde_rename; runs
under ci.yml cargo test and is clean under clippy -D warnings.

Author: danscMax

src-tauri/src/config.rs

@@ -528,8 +528,44 @@ impl ActionType {
             ActionType::RepairClipboard => "repairClipboard",
         }
     }
+
+    /// Every variant, in declaration order. Test-only: the set-equality guard
+    /// `action_type_set_matches_schema_enum` iterates this and asserts it equals
+    /// `$defs.actionType.enum` in `schemas/config.v2.schema.json`. The always-on
+    /// `match` below forces a new variant to be added here and to the schema
+    /// (a missed schema entry is a save-breaker).
+    #[cfg(test)]
+    const ALL: [ActionType; 10] = [
+        ActionType::Shortcut,
+        ActionType::TextSnippet,
+        ActionType::Sequence,
+        ActionType::Launch,
+        ActionType::Menu,
+        ActionType::MouseAction,
+        ActionType::MediaKey,
+        ActionType::ProfileSwitch,
+        ActionType::Disabled,
+        ActionType::RepairClipboard,
+    ];
 }
 
+// Compile-time exhaustiveness: adding an ActionType variant breaks this match
+// (no `_` arm), forcing you to also update `ActionType::ALL` above and
+// `$defs.actionType.enum` in schemas/config.v2.schema.json — the JSON contract a
+// saved config is validated against, where a missed entry is a save-breaker.
+const _: fn(ActionType) = |a| match a {
+    ActionType::Shortcut
+    | ActionType::TextSnippet
+    | ActionType::Sequence
+    | ActionType::Launch
+    | ActionType::Menu
+    | ActionType::MouseAction
+    | ActionType::MediaKey
+    | ActionType::ProfileSwitch
+    | ActionType::Disabled
+    | ActionType::RepairClipboard => {}
+};
+
 #[derive(Clone, Debug, Serialize, Deserialize, PartialEq, Eq)]
 #[serde(tag = "type", rename_all = "camelCase")]
 pub enum ActionCondition {
@@ -4228,6 +4264,29 @@ mod edge_proptests {
         }
     }
 
+    /// ActionType (Rust) and `$defs.actionType.enum` in config.v2.schema.json must
+    /// list the SAME set. The schema is the contract a saved config is validated
+    /// against; a variant in one but not the other is a save-breaker (this is how
+    /// `repairClipboard` once broke saving). The per-variant tests above only cover
+    /// known cases — this closes the whole class against a future added/renamed type.
+    #[test]
+    fn action_type_set_matches_schema_enum() {
+        let schema: Value =
+            serde_json::from_str(CONFIG_SCHEMA_JSON).expect("schema is valid JSON");
+        let schema_enum: std::collections::BTreeSet<&str> = schema["$defs"]["actionType"]["enum"]
+            .as_array()
+            .expect("$defs.actionType.enum is an array")
+            .iter()
+            .map(|v| v.as_str().expect("actionType.enum values are strings"))
+            .collect();
+        let rust: std::collections::BTreeSet<&str> =
+            ActionType::ALL.iter().map(|a| a.as_str()).collect();
+        assert_eq!(
+            rust, schema_enum,
+            "ActionType (Rust) and config.v2.schema.json $defs.actionType.enum drifted apart"
+        );
+    }
+
     /// Layer::as_str() must match its serde JSON representation.
     #[test]
     fn layer_as_str_matches_serde_rename() {