fix: some bugs

1. read config always failed
2. update default reminder minutes from 15 to 0(not before alert)
3. fix always create new reminder list
4. update default priority from Medium to None
5. Configuration parameters allow default value

Author: stolenzc

README.md

@@ -49,7 +49,7 @@ reminders add "明天下午3点开会"
     "model": "gpt-4-turbo-preview"
   },
   "default_list": "提醒事项",
-  "default_reminder_minutes": [15, 30]
+  "default_reminder_minutes": [0]
 }
 ```
 

examples/config.example.json

@@ -5,6 +5,6 @@
     "model": "gpt-4-turbo-preview"
   },
   "default_list": "reminders",
-  "default_reminder_minutes": [15, 30],
+  "default_reminder_minutes": [0],
   "use_ai": true
 }

src/apple_reminders.rs

@@ -2,7 +2,7 @@ use crate::cors::{Priority, Recurrence};
 use crate::reminder::Reminder;
 use anyhow::{Result, anyhow};
 use chrono::{DateTime, Datelike, Local, Timelike};
-use std::process::Command;
+use std::process::{Command, Output};
 
 /// 苹果提醒事项集成（通过 AppleScript 与 Reminders.app 通信）
 pub struct AppleReminders;
@@ -29,20 +29,22 @@ impl AppleReminders {
 
     /// 检查列表是否存在
     pub fn list_exists(list_name: &str) -> Result<bool> {
-        let script = format!(
-            r#"tell application "Reminders"
-    set listNames to name of every list
-    repeat with n in listNames
-        if n = "{}" then return "yes"
-    end repeat
-    return "no"
-end tell"#,
-            escape_string(list_name)
-        );
 
-        let output = Command::new("osascript").arg("-e").arg(&script).output()?;
-        let stdout = String::from_utf8_lossy(&output.stdout);
-        Ok(stdout.trim() == "yes")
+        let script = r#"tell application "Reminders"
+    set listNames to name of every list
+    return listNames
+end tell"#;
+
+        // 获取全部列表名称, 不使用AppleScript检测
+        let output: Output = Command::new("osascript").arg("-e").arg(&script).output()?;
+        let list_str = String::from_utf8_lossy(&output.stdout);
+        let list_vec: Vec<String> = list_str.replace("\n", ",").split(",").map(|s| s.to_string()).collect();
+        for exist_name in list_vec {
+            if exist_name == list_name {
+                return Ok(true);
+            }
+        }
+        Ok(false)
     }
 
     /// 创建新列表（如果不存在）

src/cli.rs

@@ -1,4 +1,4 @@
-use crate::{cors::Recurrence, parser::ParsedReminder};
+use crate::{cors::{Priority, Recurrence}, parser::ParsedReminder};
 use clap::{Arg, Command};
 use std::io::{self, Write};
 
@@ -166,7 +166,9 @@ pub fn show_parsed_summary(parsed: &ParsedReminder) {
         println!("  │ 截止时间: {}", due_date.format("%Y-%m-%d %H:%M"));
     }
 
-    println!("  │ 优先级: {}", parsed.priority);
+    if parsed.priority != Priority::None {
+        println!("  │ 优先级: {}", parsed.priority);
+    }
 
     if parsed.is_urgent {
         println!("  │ 是否紧急: 是");

src/config.rs

@@ -1,4 +1,4 @@
-use anyhow::{Result, anyhow};
+use anyhow::Result;
 use serde::{Deserialize, Serialize};
 use std::fs;
 use std::path::Path;
@@ -16,37 +16,39 @@ pub struct AIConfig {
 
 /// 应用配置
 #[derive(Debug, Clone, Serialize, Deserialize)]
+#[serde(default)]
 pub struct AppConfig {
     /// AI 配置
     pub ai: AIConfig,
     /// 默认列表
     pub default_list: String,
     /// 默认提醒时间（分钟）
     pub default_reminder_minutes: Vec<i32>,
-    /// 是否使用 AI 解析（默认 true）
-    #[serde(default = "default_true")]
+    /// 是否使用 AI 解析
     pub use_ai: bool,
 }
 
-fn default_true() -> bool {
-    true
-}
-
 impl Default for AppConfig {
     fn default() -> Self {
         Self {
-            ai: AIConfig {
-                api_url: "https://api.openai.com/v1/chat/completions".to_string(),
-                api_key: "".to_string(),
-                model: "gpt-4-turbo-preview".to_string(),
-            },
+            ai: AIConfig::default(),
             default_list: "提醒事项".to_string(),
-            default_reminder_minutes: vec![15],
+            default_reminder_minutes: vec![0],
             use_ai: false,
         }
     }
 }
 
+impl Default for AIConfig {
+    fn default() -> Self {
+        Self {
+            api_url: "".to_string(),
+            api_key: "".to_string(),
+            model: "".to_string()
+        }
+    }
+}
+
 /// 配置管理器
 #[derive(Clone)]
 pub struct ConfigManager {
@@ -56,9 +58,7 @@ pub struct ConfigManager {
 impl ConfigManager {
     /// 创建新的配置管理器
     pub fn new() -> Result<Self> {
-        let config_dir = dirs::config_dir()
-            .ok_or_else(|| anyhow!("无法获取配置目录"))?
-            .join("reminders");
+        let config_dir = dirs::home_dir().unwrap().join(".config/reminders");
 
         if !config_dir.exists() {
             fs::create_dir_all(&config_dir)?;
@@ -119,7 +119,7 @@ mod tests {
     fn test_default_config() {
         let config = AppConfig::default();
         assert_eq!(config.default_list, "提醒事项");
-        assert_eq!(config.default_reminder_minutes, vec![15]);
+        assert_eq!(config.default_reminder_minutes, vec![0]);
         assert!(!config.use_ai);
     }
 }

src/parser/parser_ai.rs

@@ -145,7 +145,7 @@ impl AIParser {
         let current_time = chrono::Local::now().format("%Y-%m-%d %H:%M:%S").to_string();
         let default_list = self.config_manager.get_default_list();
 
-        let template = "你是一个智能提醒事项解析器。请将用户输入的自然语言解析为结构化的提醒事项信息。\n\n输入：{input}\n\n请按照以下 JSON 格式返回解析结果：\n\n{{\n  \"title\": \"提醒事项标题\",\n  \"description\": \"可选描述\",\n  \"due_date\": \"截止日期时间，格式：YYYY-MM-DD HH:MM:SS，如果无法确定则为 null\",\n  \"start_date\": \"开始日期时间，格式：YYYY-MM-DD HH:MM:SS，可选\",\n  \"priority\": \"优先级，可选值：none, low, medium, high\",\n  \"is_urgent\": \"是否紧急，布尔值\",\n  \"recurrence\": \"重复模式，可选：none, daily, weekly, monthly, yearly, weekdays, weekends\",\n  \"location\": {{\n    \"name\": \"位置名称\",\n    \"address\": \"详细地址，可选\"\n  }},\n  \"reminder_minutes\": [15, 30],\n  \"tags\": [\"标签1\", \"标签2\"],\n  \"list\": \"列表名称\"\n}}\n\n注意：\n1. 当前时间：{current_time}\n2. 如果用户没有指定时间，请根据上下文推断合理的时间\n3. 标题应该简洁明了\n4. 优先使用中文标签\n5. 如果用户指定了列表，使用用户指定的列表，否则使用 \"{default_list}\"\n6. 日期时间请使用 24 小时制\n7. 如果无法确定某些字段，请使用合理的默认值\n8. 请确保返回的是有效的 JSON 格式";
+        let template = "你是一个智能提醒事项解析器。请将用户输入的自然语言解析为结构化的提醒事项信息。\n\n输入：{input}\n\n请按照以下 JSON 格式返回解析结果：\n\n{{\n  \"title\": \"提醒事项标题\",\n  \"description\": \"可选描述\",\n  \"due_date\": \"截止日期时间，格式：YYYY-MM-DD HH:MM:SS，如果无法确定则为 null\",\n  \"start_date\": \"开始日期时间，格式：YYYY-MM-DD HH:MM:SS，可选\",\n  \"priority\": \"优先级，可选值：none, low, medium, high\",\n  \"is_urgent\": \"是否紧急，布尔值\",\n  \"recurrence\": \"重复模式，可选：none, daily, weekly, monthly, yearly, weekdays, weekends\",\n  \"location\": {{\n    \"name\": \"位置名称\",\n    \"address\": \"详细地址，可选\"\n  }},\n  \"reminder_minutes\": [0],\n  \"tags\": [\"标签1\", \"标签2\"],\n  \"list\": \"列表名称\"\n}}\n\n注意：\n1. 当前时间：{current_time}\n2. 如果用户没有指定时间，请根据上下文推断合理的时间\n3. 标题应该简洁明了\n4. 优先使用中文标签\n5. 如果用户指定了列表，使用用户指定的列表，否则使用 \"{default_list}\"\n6. 日期时间请使用 24 小时制\n7. 如果无法确定某些字段，请使用合理的默认值\n8. 请确保返回的是有效的 JSON 格式";
         template
             .replace("{input}", input)
             .replace("{current_time}", &current_time)

src/parser/parser_regexp.rs

@@ -236,11 +236,11 @@ pub fn parse_input(input: &str, default_list: &str) -> Result<ParsedReminder> {
         title: String::new(),
         due_date: None,
         start_date: None,
-        priority: Priority::Medium,
+        priority: Priority::None,
         is_urgent: false,
         recurrence: Recurrence::None,
         location: None,
-        reminder_minutes: vec![15],
+        reminder_minutes: vec![0],
         tags: Vec::new(),
         list: default_list.to_string(),
     };
@@ -306,7 +306,7 @@ pub fn parse_input(input: &str, default_list: &str) -> Result<ParsedReminder> {
         if let Some(caps) = pattern.captures(&text)
             && let Some(m) = caps.get(1)
         {
-            let value: i32 = m.as_str().parse().unwrap_or(15);
+            let value: i32 = m.as_str().parse().unwrap_or(0);
             let minutes = match *kind {
                 "hours" => value * 60,
                 "days" => value * 24 * 60,

src/reminder.rs

@@ -42,7 +42,7 @@ impl Reminder {
             is_urgent: false,
             recurrence: Recurrence::None,
             location: None,
-            reminder_minutes: vec![15, 5], // 默认提前15分钟和5分钟提醒
+            reminder_minutes: vec![0],
             tags: Vec::new(),
             list,
         }
@@ -157,7 +157,7 @@ mod tests {
         assert_eq!(reminder.list, "默认列表");
         assert_eq!(reminder.priority, Priority::Medium);
         assert!(!reminder.is_urgent);
-        assert_eq!(reminder.reminder_minutes, vec![15, 5]);
+        assert_eq!(reminder.reminder_minutes, vec![0]);
     }
 
     #[test]