fix: detect tooluses with common path characters when cleaning for sp…

…eech (#412)

* fix: detect tooluses with common path characters when cleaning for speech

* fix: improved clean_for_speech and its testing

gptme/tools/tts.py

@@ -43,7 +43,7 @@
 
 
 re_thinking = re.compile(r"<thinking>.*?(\n</thinking>|$)", flags=re.DOTALL)
-re_tool_use = re.compile(r"```[\w\. ]+\n[^`]*(\n```|$)", flags=re.DOTALL)
+re_tool_use = re.compile(r"```[\w\. ~/\-]+\n(.*?)(\n```|$)", flags=re.DOTALL)
 
 
 def set_speed(speed):

tests/test_tools_tts.py

@@ -78,8 +78,18 @@ def test_clean_for_speech():
     assert re_tool_use.search("```tool\ncontents\n```")
 
     # with arg
-    assert re_tool_use.search("```save test.txt\ncontents\n```")
+    assert re_tool_use.search("```save ~/path_to/test-file1.txt\ncontents\n```")
+
+    # with `text` contents
+    assert re_tool_use.search("```file.md\ncontents with `code` string\n```")
 
     # incomplete
     assert re_thinking.search("\n<thinking>thinking")
     assert re_tool_use.search("```savefile.txt\ncontents")
+
+    # make sure spoken content is correct
+    assert (
+        re_tool_use.sub("", "Using tool\n```tool\ncontents\n```").strip()
+        == "Using tool"
+    )
+    assert re_tool_use.sub("", "```tool\ncontents\n```\nRan tool").strip() == "Ran tool"