plugins/applications: make entry_priority not optional

Author: HelloWorld017

plugins/applications/README.md

@@ -33,7 +33,7 @@ Config(
   )),
 
   // Whether to prioritize actions or applications
-  // Could be ActionsFirst, ApplicationsFirst, or NoPriority
-  entry_priority: Some(ActionsFirst),
+  // Could be ActionsFirst (default), ApplicationsFirst, or NoPriority
+  entry_priority: ActionsFirst,
 )
 ```

plugins/applications/src/lib.rs

@@ -11,7 +11,9 @@ pub struct Config {
     max_entries: usize,
     terminal: Option<Terminal>,
     preprocess_exec_script: Option<PathBuf>,
-    entry_priority: Option<EntryPriority>,
+
+    #[serde(default)]
+    entry_priority: EntryPriority,
 }
 
 #[derive(Deserialize)]
@@ -20,8 +22,9 @@ pub struct Terminal {
     args: String,
 }
 
-#[derive(Deserialize)]
+#[derive(Deserialize, Default)]
 pub enum EntryPriority {
+    #[default]
     ActionsFirst,
     ApplicationsFirst,
     NoPriority,
@@ -34,7 +37,7 @@ impl Default for Config {
             max_entries: 5,
             preprocess_exec_script: None,
             terminal: None,
-            entry_priority: None,
+            entry_priority: EntryPriority::default(),
         }
     }
 }
@@ -186,12 +189,6 @@ pub fn init(config_dir: RString) -> State {
 #[get_matches]
 pub fn get_matches(input: RString, state: &State) -> RVec<Match> {
     let matcher = fuzzy_matcher::skim::SkimMatcherV2::default().ignore_case();
-    let entry_priority = state
-        .config
-        .entry_priority
-        .as_ref()
-        .unwrap_or(&EntryPriority::ActionsFirst);
-
     let mut entries = state
         .entries
         .iter()
@@ -216,7 +213,7 @@ pub fn get_matches(input: RString, state: &State) -> RVec<Match> {
             let mut score = (name_score * 10 + desc_score + keyword_score) - entry.offset;
 
             // Apply priority
-            score *= match (entry_priority, entry.is_action) {
+            score *= match (&state.config.entry_priority, entry.is_action) {
                 (EntryPriority::ActionsFirst, true) => 2,
                 (EntryPriority::ApplicationsFirst, false) => 2,
                 _ => 1,