From 741d7367153aa2dd23982457887e72b95b290e70 Mon Sep 17 00:00:00 2001 From: "Rodrigo Rodriguez (Pragmatismo)" Date: Wed, 5 Nov 2025 14:28:14 -0300 Subject: [PATCH] feat: update deepseek_r3 regex and add test case Updated the regex pattern in DeepseekR3Handler to use (?s) flag for dot-matches-newline behavior when removing tags. Added comprehensive test case that verifies the handler correctly processes content with multiline think tags. Also made styling changes to the web interface, though the full diff was truncated. --- src/llm_models/deepseek_r3.rs | 2 +- src/llm_models/llm_models.test.rs | 19 +- web/html/index.html | 1363 +++++++++++++++++++++++++---- 3 files changed, 1211 insertions(+), 173 deletions(-) diff --git a/src/llm_models/deepseek_r3.rs b/src/llm_models/deepseek_r3.rs index cce8a624..daf715d0 100644 --- a/src/llm_models/deepseek_r3.rs +++ b/src/llm_models/deepseek_r3.rs @@ -9,7 +9,7 @@ impl ModelHandler for DeepseekR3Handler { } fn process_content(&self, content: &str) -> String { - let re = regex::Regex::new(r".*?").unwrap(); + let re = regex::Regex::new(r"(?s).*?").unwrap(); re.replace_all(content, "").to_string() } diff --git a/src/llm_models/llm_models.test.rs b/src/llm_models/llm_models.test.rs index a1cadaca..7a35d37d 100644 --- a/src/llm_models/llm_models.test.rs +++ b/src/llm_models/llm_models.test.rs @@ -12,9 +12,24 @@ mod tests { } #[test] - fn test_deepseek_r3() { + fn test_deepseek_r3_process_content() { test_util::setup(); - assert!(true, "Deepseek R3 placeholder test"); + let handler = DeepseekR3Handler; + let input = r#" +Alright, I need to help the user revise their resume entry. Let me read what they provided first. + +The original message says: " Auxiliom has been updated last week! New release!" They want it in a few words. Hmm, so maybe instead of saying "has been updated," we can use more concise language because resumes usually don't require too much detail unless there's specific information to include. + +I notice that the user wants it for their resume, which often requires bullet points or short sentences without being verbose. So perhaps combining these two thoughts into a single sentence would make sense. Also, using an exclamation mark might help convey enthusiasm about the new release. + +Let me put it together: "Auxiliom has been updated last week! New release." That's concise and fits well for a resume. It effectively communicates both that something was updated recently and introduces them as having a new release without adding unnecessary details. + + +" Auxiliom has been updated last week! New release.""#; + + let expected = r#"" Auxiliom has been updated last week! New release.""#; + let result = handler.process_content(input); + assert_eq!(result, expected); } #[test] diff --git a/web/html/index.html b/web/html/index.html index 2837a0e3..335ad374 100644 --- a/web/html/index.html +++ b/web/html/index.html @@ -4,201 +4,1224 @@ General Bots -
- - - - -