fix: 修复代码格式以符合PEP规范

- 移除未使用的变量和导入
- 修复f-string格式问题
- 移除重复函数定义
- 修正异常处理中的冗余类型
- 修复未使用的循环变量命名

🤖 Generated with [Claude Code]

Co-Authored-By: Claude <[email protected]>

Author: SepineTam

src/stata_mcp/webui/__init__.py

@@ -110,18 +110,19 @@ def config():
 
         if all_valid:
             # Create backup before saving
-            backup_path = _create_backup()
+            _create_backup()
 
             try:
                 # Save all configuration sections
-                for section, fields in new_config.items():
-                    for field, value in fields.items():
+                for section_name, fields in new_config.items():
+                    for field_name, value in fields.items():
                         if isinstance(value, dict):
-                            for sub_field, sub_value in value.items():
+                            for sub_field_name, sub_value in value.items():
                                 config_mgr.set(
-                                    f'{section}.{field}.{sub_field}', str(sub_value))
+                                    f'{section_name}.{field_name}.{sub_field_name}', str(sub_value))
                         else:
-                            config_mgr.set(f'{section}.{field}', str(value))
+                            config_mgr.set(
+                                f'{section_name}.{field_name}', str(value))
 
                 return redirect(url_for('config', saved='1'))
             except Exception as e:
@@ -200,10 +201,10 @@ def import_config():
 
         # Check if all validations pass
         all_valid = True
-        for section, fields in validation_results.items():
-            for field, result in fields.items():
+        for _section_name, fields in validation_results.items():
+            for _field_name, result in fields.items():
                 if isinstance(result, dict):
-                    for sub_field, sub_result in result.items():
+                    for _sub_field_name, sub_result in result.items():
                         if not sub_result['valid']:
                             all_valid = False
                             break

src/stata_mcp/webui/utils/config_validator.py

@@ -5,7 +5,6 @@
 from the complete example.toml structure.
 """
 
-import time  # For backup timestamp
 import os
 import platform
 import re
@@ -72,7 +71,7 @@ def validate_output_base_path(path: str) -> Tuple[bool, str, Optional[str]]:
 
             # Check write permissions
             if not os.access(path, os.W_OK):
-                return False, f"Directory is not writable: {path}", f"Check directory permissions"
+                return False, f"Directory is not writable: {path}", "Check directory permissions"
         else:
             # Try to create the directory
             try:
@@ -82,7 +81,7 @@ def validate_output_base_path(path: str) -> Tuple[bool, str, Optional[str]]:
                 with open(test_file, "w") as f:
                     f.write("test")
                 os.remove(test_file)
-            except (OSError, IOError) as e:
+            except OSError as e:
                 return False, f"Cannot create or write to directory: {path}", str(
                     e)
 
@@ -292,22 +291,3 @@ def create_configuration_backup(config_path: str) -> str:
     backup_path = f"{config_path}.backup.{timestamp}"
     shutil.copy2(config_path, backup_path)
     return backup_path
-
-
-def create_configuration_backup(config_path: str) -> str:
-    """
-    Create a backup of the current configuration file.
-
-    Args:
-        config_path: Path to the configuration file
-
-    Returns:
-        Path to the backup file
-    """
-    if not os.path.exists(config_path):
-        return ""
-
-    timestamp = int(time.time())
-    backup_path = f"{config_path}.backup.{timestamp}"
-    shutil.copy2(config_path, backup_path)
-    return backup_path