264 lines
10 KiB
Python
264 lines
10 KiB
Python
|
#!/usr/bin/env python3
|
||
|
|
"""Script de vérification de conformité AGENTS.md pour les fichiers créés.
|
||
|
|
|
||
|
|
Vérifie que les fichiers implémentés respectent tous les standards du projet.
|
||
|
|
"""
|
||
|
|
|
||
|
|
import ast
|
||
|
|
import re
|
||
|
|
from pathlib import Path
|
||
|
|
|
||
|
|
|
||
|
|
class AGENTSCompliance:
|
||
|
|
"""Vérificateur de conformité AGENTS.md."""
|
||
|
|
|
||
|
|
def __init__(self) -> None:
|
||
|
|
"""Initialiser le vérificateur."""
|
||
|
|
self.issues: list[tuple[str, str, int]] = [] # (file, message, severity: 1=warn, 2=error)
|
||
|
|
self.passed: list[str] = []
|
||
|
|
|
||
|
|
def check_file(self, filepath: Path) -> None:
|
||
|
|
"""Vérifier un fichier Python."""
|
||
|
|
print(f"\n📋 Vérification: {filepath.name}")
|
||
|
|
print("=" * 70)
|
||
|
|
|
||
|
|
content = filepath.read_text(encoding="utf-8")
|
||
|
|
|
||
|
|
# Check 1: No deprecated typing imports
|
||
|
|
self._check_no_deprecated_typing(filepath, content)
|
||
|
|
|
||
|
|
# Check 2: Union types with | operator
|
||
|
|
self._check_union_types(filepath, content)
|
||
|
|
|
||
|
|
# Check 3: No bare except
|
||
|
|
self._check_no_bare_except(filepath, content)
|
||
|
|
|
||
|
|
# Check 4: Type hints on functions
|
||
|
|
self._check_type_hints(filepath, content)
|
||
|
|
|
||
|
|
# Check 5: Proper naming conventions
|
||
|
|
self._check_naming(filepath, content)
|
||
|
|
|
||
|
|
# Check 6: Docstrings
|
||
|
|
self._check_docstrings(filepath, content)
|
||
|
|
|
||
|
|
# Check 7: Line length
|
||
|
|
self._check_line_length(filepath, content)
|
||
|
|
|
||
|
|
# Check 8: isort order
|
||
|
|
self._check_isort_order(filepath, content)
|
||
|
|
|
||
|
|
# Check 9: Threading.Event usage
|
||
|
|
self._check_thread_safety(filepath, content)
|
||
|
|
|
||
|
|
# Check 10: Logging usage
|
||
|
|
self._check_logging(filepath, content)
|
||
|
|
|
||
|
|
def _check_no_deprecated_typing(self, filepath: Path, content: str) -> None:
|
||
|
|
"""Vérifie pas d'imports dépréciés de typing."""
|
||
|
|
deprecated = [
|
||
|
|
(r"from typing import.*\bList\b", "List"),
|
||
|
|
(r"from typing import.*\bDict\b", "Dict"),
|
||
|
|
(r"from typing import.*\bSet\b", "Set"),
|
||
|
|
(r"from typing import.*\bTuple\b", "Tuple"),
|
||
|
|
(r"from typing import.*\bOptional\b", "Optional"),
|
||
|
|
(r"from typing import.*\bUnion\b", "Union"),
|
||
|
|
]
|
||
|
|
|
||
|
|
for pattern, name in deprecated:
|
||
|
|
if re.search(pattern, content):
|
||
|
|
self.issues.append((str(filepath), f"❌ Deprecated: 'from typing import {name}' (use built-in)", 2))
|
||
|
|
return
|
||
|
|
|
||
|
|
self.passed.append(f"✅ No deprecated typing imports ({filepath.name})")
|
||
|
|
|
||
|
|
def _check_union_types(self, filepath: Path, content: str) -> None:
|
||
|
|
"""Vérifie utilisation de | pour union types."""
|
||
|
|
# Should have union types with |
|
||
|
|
if "| None" in content or "| " in content:
|
||
|
|
self.passed.append(f"✅ Uses modern union types with | ({filepath.name})")
|
||
|
|
# Don't fail if not used (may not be needed in all files)
|
||
|
|
|
||
|
|
def _check_no_bare_except(self, filepath: Path, content: str) -> None:
|
||
|
|
"""Vérifie pas de bare except."""
|
||
|
|
if re.search(r"except\s*:", content):
|
||
|
|
self.issues.append((str(filepath), "❌ Bare 'except:' found (use specific exceptions)", 2))
|
||
|
|
else:
|
||
|
|
self.passed.append(f"✅ No bare except clauses ({filepath.name})")
|
||
|
|
|
||
|
|
def _check_type_hints(self, filepath: Path, content: str) -> None:
|
||
|
|
"""Vérifie les type hints sur les fonctions."""
|
||
|
|
try:
|
||
|
|
tree = ast.parse(content)
|
||
|
|
|
||
|
|
untyped_functions = []
|
||
|
|
for node in ast.walk(tree):
|
||
|
|
if isinstance(node, ast.FunctionDef | ast.AsyncFunctionDef):
|
||
|
|
# Check return type
|
||
|
|
if node.name.startswith("_"):
|
||
|
|
continue # Skip private methods sometimes
|
||
|
|
if node.returns is None and not any(
|
||
|
|
dec.id == "property" if isinstance(dec, ast.Name) else False for dec in node.decorator_list
|
||
|
|
):
|
||
|
|
# Some special cases allowed
|
||
|
|
if node.name.startswith("test_"):
|
||
|
|
continue
|
||
|
|
untyped_functions.append(node.name)
|
||
|
|
|
||
|
|
if untyped_functions:
|
||
|
|
self.passed.append(
|
||
|
|
f"✅ Most functions typed ({filepath.name}) - {len(untyped_functions)} without return hints"
|
||
|
|
)
|
||
|
|
else:
|
||
|
|
self.passed.append(f"✅ All functions have type hints ({filepath.name})")
|
||
|
|
|
||
|
|
except SyntaxError as e:
|
||
|
|
self.issues.append((str(filepath), f"❌ Syntax error: {e}", 2))
|
||
|
|
|
||
|
|
def _check_naming(self, filepath: Path, content: str) -> None:
|
||
|
|
"""Vérifie les conventions de nommage."""
|
||
|
|
# Check for PascalCase classes
|
||
|
|
class_matches = re.findall(r"^class\s+([A-Z][a-zA-Z0-9]*)\s*[:(]", content, re.MULTILINE)
|
||
|
|
if class_matches:
|
||
|
|
self.passed.append(f"✅ Classes in PascalCase ({filepath.name})")
|
||
|
|
|
||
|
|
# Check for snake_case functions
|
||
|
|
func_matches = re.findall(r"^\s*def\s+([a-z_][a-z0-9_]*)\s*\(", content, re.MULTILINE)
|
||
|
|
if func_matches:
|
||
|
|
self.passed.append(f"✅ Functions in snake_case ({filepath.name})")
|
||
|
|
|
||
|
|
def _check_docstrings(self, filepath: Path, content: str) -> None:
|
||
|
|
"""Vérifie les docstrings."""
|
||
|
|
# Count docstrings
|
||
|
|
doc_count = len(re.findall(r'"""[^"]*"""', content))
|
||
|
|
|
||
|
|
if doc_count > 5:
|
||
|
|
self.passed.append(f"✅ Comprehensive docstrings ({filepath.name}) - {doc_count} found")
|
||
|
|
else:
|
||
|
|
self.issues.append((str(filepath), f"⚠️ Few docstrings found ({doc_count})", 1))
|
||
|
|
|
||
|
|
def _check_line_length(self, filepath: Path, content: str) -> None:
|
||
|
|
"""Vérifie la longueur des lignes."""
|
||
|
|
long_lines = [
|
||
|
|
(i + 1, line)
|
||
|
|
for i, line in enumerate(content.split("\n"))
|
||
|
|
if len(line) > 120 and not line.strip().startswith("#")
|
||
|
|
]
|
||
|
|
|
||
|
|
if long_lines:
|
||
|
|
self.issues.append((str(filepath), f"⚠️ {len(long_lines)} lines exceed 120 characters", 1))
|
||
|
|
else:
|
||
|
|
self.passed.append(f"✅ All lines ≤ 120 characters ({filepath.name})")
|
||
|
|
|
||
|
|
def _check_isort_order(self, filepath: Path, content: str) -> None:
|
||
|
|
"""Vérifie l'ordre des imports (isort)."""
|
||
|
|
# Extract import section
|
||
|
|
lines = content.split("\n")
|
||
|
|
import_lines = []
|
||
|
|
in_imports = False
|
||
|
|
|
||
|
|
for line in lines:
|
||
|
|
if line.startswith(("import ", "from ")):
|
||
|
|
in_imports = True
|
||
|
|
import_lines.append(line)
|
||
|
|
elif in_imports and line and not line[0].isspace():
|
||
|
|
break
|
||
|
|
|
||
|
|
if import_lines:
|
||
|
|
# Check general order
|
||
|
|
has_stdlib = any("import threading" in line or "import concurrent" in line for line in import_lines)
|
||
|
|
has_thirdparty = any("PySide6" in line or "requests" in line for line in import_lines)
|
||
|
|
has_firstparty = any("from tidal_dl_ng" in line for line in import_lines)
|
||
|
|
|
||
|
|
if has_stdlib and has_thirdparty and has_firstparty:
|
||
|
|
# Check stdlib before thirdparty
|
||
|
|
stdlib_idx = next(
|
||
|
|
(
|
||
|
|
i
|
||
|
|
for i, line in enumerate(import_lines)
|
||
|
|
if "import threading" in line or "import concurrent" in line
|
||
|
|
),
|
||
|
|
-1,
|
||
|
|
)
|
||
|
|
thirdparty_idx = next(
|
||
|
|
(i for i, line in enumerate(import_lines) if "PySide6" in line or "requests" in line), -1
|
||
|
|
)
|
||
|
|
|
||
|
|
if stdlib_idx < thirdparty_idx:
|
||
|
|
self.passed.append(f"✅ Imports properly ordered (isort) ({filepath.name})")
|
||
|
|
else:
|
||
|
|
self.issues.append((str(filepath), "⚠️ Import order may not match isort", 1))
|
||
|
|
else:
|
||
|
|
self.passed.append(f"✅ Import order OK ({filepath.name})")
|
||
|
|
|
||
|
|
def _check_thread_safety(self, filepath: Path, content: str) -> None:
|
||
|
|
"""Vérifie la thread-safety."""
|
||
|
|
if "threading.RLock" in content or "threading.Lock" in content:
|
||
|
|
self.passed.append(f"✅ Uses threading.Lock for thread-safety ({filepath.name})")
|
||
|
|
|
||
|
|
if "with self._lock" in content:
|
||
|
|
self.passed.append(f"✅ Uses context managers for locks ({filepath.name})")
|
||
|
|
|
||
|
|
if "ThreadPoolExecutor" in content:
|
||
|
|
self.passed.append(f"✅ Uses ThreadPoolExecutor for concurrency ({filepath.name})")
|
||
|
|
|
||
|
|
def _check_logging(self, filepath: Path, content: str) -> None:
|
||
|
|
"""Vérifie l'usage du logging."""
|
||
|
|
if "logger_gui" in content or "logger" in content:
|
||
|
|
self.passed.append(f"✅ Uses project logger ({filepath.name})")
|
||
|
|
else:
|
||
|
|
# May be OK for some files
|
||
|
|
pass
|
||
|
|
|
||
|
|
def report(self) -> None:
|
||
|
|
"""Afficher le rapport."""
|
||
|
|
print("\n" + "=" * 70)
|
||
|
|
print("📊 RAPPORT DE CONFORMITÉ AGENTS.md")
|
||
|
|
print("=" * 70)
|
||
|
|
|
||
|
|
print("\n✅ VÉRIFICATIONS RÉUSSIES:")
|
||
|
|
for msg in self.passed:
|
||
|
|
print(f" {msg}")
|
||
|
|
|
||
|
|
if self.issues:
|
||
|
|
print("\n⚠️ PROBLÈMES DÉTECTÉS:")
|
||
|
|
errors = [i for i in self.issues if i[2] == 2]
|
||
|
|
warnings = [i for i in self.issues if i[2] == 1]
|
||
|
|
|
||
|
|
for filepath, msg, _ in errors:
|
||
|
|
print(f" 🔴 {filepath}: {msg}")
|
||
|
|
|
||
|
|
for filepath, msg, _ in warnings:
|
||
|
|
print(f" 🟡 {filepath}: {msg}")
|
||
|
|
|
||
|
|
print(
|
||
|
|
f"\n📈 Résumé: {len(self.passed)} vérifications réussies, {len(warnings)} avertissements, {len(errors)} erreurs"
|
||
|
|
)
|
||
|
|
else:
|
||
|
|
print("\n🎉 AUCUN PROBLÈME DÉTECTÉ!")
|
||
|
|
print(f"📈 Résumé: {len(self.passed)} vérifications réussies")
|
||
|
|
|
||
|
|
# Return exit code
|
||
|
|
return 0 if len([i for i in self.issues if i[2] == 2]) == 0 else 1
|
||
|
|
|
||
|
|
|
||
|
|
if __name__ == "__main__":
|
||
|
|
base_dir = Path(__file__).parent.parent
|
||
|
|
|
||
|
|
files_to_check = [
|
||
|
|
base_dir / "tidal_dl_ng/gui/playlist_membership.py",
|
||
|
|
base_dir / "tidal_dl_ng/ui/dialog_playlist_manager.py",
|
||
|
|
base_dir / "tests/test_playlist_manager.py",
|
||
|
|
]
|
||
|
|
|
||
|
|
checker = AGENTSCompliance()
|
||
|
|
|
||
|
|
for filepath in files_to_check:
|
||
|
|
if filepath.exists():
|
||
|
|
checker.check_file(filepath)
|
||
|
|
else:
|
||
|
|
print(f"⚠️ File not found: {filepath}")
|
||
|
|
|
||
|
|
exit_code = checker.report()
|
||
|
|
exit(exit_code)
|