fix: Improve Windows file deletion handling

- Refactor rmtree function to handle Windows deletion errors
- Remove unused shutil imports from multiple files
- Add error handling and retry logic for file removal

Author: bobo yang

devchat/_cli/run.py

@@ -1,6 +1,7 @@
 from typing import List, Optional, Tuple
 
 import click
+
 from devchat.utils import rmtree
 
 

devchat/_cli/utils.py

@@ -1,5 +1,4 @@
 import os
-import shutil
 import sys
 import zipfile
 from contextlib import contextmanager

devchat/_service/route/workflows.py

@@ -1,4 +1,3 @@
-import shutil
 from pathlib import Path
 from typing import List
 

devchat/utils.py

@@ -244,6 +244,7 @@ def openai_response_tokens(message: dict, model: str) -> int:
     """Returns the number of tokens used by a response."""
     return openai_message_tokens(message, model)
 
+
 def rmtree(path: str) -> None:
     import shutil
 
@@ -268,4 +269,4 @@ def __onerror(func, path, _1):
             # Retry the function that failed
             func(path)
 
-    shutil.rmtree(path, onerror=__onerror)
+    shutil.rmtree(path, onerror=__onerror)