fix: disable pre-commit checks running before first user message, misc fixes

Author: ErikBjare

gptme/chat.py

@@ -327,19 +327,26 @@ def prompt_input(prompt: str, value=None) -> str:  # pragma: no cover
 def check_for_modifications(log: Log) -> bool:
     """Check if there are any file modifications in last 3 messages or since last user message."""
     messages_since_user = []
+    found_user_message = False
+
     for m in reversed(log):
         if m.role == "user":
+            found_user_message = True
             break
         messages_since_user.append(m)
 
+    # If no user message found, skip the check (only system messages so far)
+    if not found_user_message:
+        return False
+
     # FIXME: this is hacky and unreliable
     has_modifications = any(
         tu.tool in ["save", "patch", "append"]
         for m in messages_since_user[:3]
         for tu in ToolUse.iter_from_content(m.content)
     )
     # logger.debug(
-    #     f"Found {len(messages_since_user)} messages since user ({has_modifications=})"
+    #     f"Found {len(messages_since_user)} messages since user ({found_user_message=}, {has_modifications=})"
     # )
     return has_modifications
 

gptme/tools/browser.py

@@ -118,7 +118,6 @@ def init() -> ToolSpec:
     return tool
 
 
-# TODO: remove the unnecessary new .is_available property and the .available Callable type union (bool only for simplicity)
 @lru_cache
 def has_browser_tool():
     return browser is not None

gptme/util/cli.py

@@ -174,7 +174,7 @@ def tools_list(available: bool, langtags: bool):
             status = "✓" if tool.is_available else "✗"
             print(
                 f"""
-{status} {tool.name}
+ {status} {tool.name}
    {tool.desc}"""
             )