Merge pull request #4 from stolenzc/fix/parser

fix: some bugs

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,25 @@ impl AppleReminders {
 
     /// 检查列表是否存在
     pub fn list_exists(list_name: &str) -> Result<bool> {
-        let script = format!(
-            r#"tell application "Reminders"
+        let script = 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")
+    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,7 @@
-use crate::{cors::Recurrence, parser::ParsedReminder};
+use crate::{
+    cors::{Priority, Recurrence},
+    parser::ParsedReminder,
+};
 use clap::{Arg, Command};
 use std::io::{self, Write};
 
@@ -166,7 +169,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]