[chore] Path changes, fixes, and improvements

Delete unnecessary comments from code.

Wherever the level was specified as a string, the LogLevel structure is used.

Author: Qu1nel

src/code_validator/components/ast_utils.py

@@ -15,7 +15,6 @@ def enrich_ast_with_parents(tree: ast.Module) -> None:
     """
     for node in ast.walk(tree):
         for child in ast.iter_child_nodes(node):
-            # Dynamically add a reference to the parent node.
             child.parent = node
 
 
@@ -37,4 +36,4 @@ def get_full_name(node: ast.AST) -> str | None:
     if isinstance(node, ast.Attribute):
         base = get_full_name(node.value)
         return f"{base}.{node.attr}" if base else node.attr
-    return None
+    return None

src/code_validator/core.py

@@ -45,14 +45,12 @@ def _parse_ast_tree(self) -> bool:
             enrich_ast_with_parents(self._ast_tree)
             return True
         except SyntaxError as e:
-            # Ищем правило check_syntax, чтобы вывести его кастомное сообщение
             for rule in self._validation_rules:
                 if getattr(rule.config, "type", None) == "check_syntax":
-                    self._console.print(rule.config.message, level="ERROR")
+                    self._console.print(rule.config.message, level=LogLevel.ERROR)
                     self._failed_rules.append(rule.config.rule_id)
                     return False
-            # Если такого правила нет, выводим стандартное сообщение
-            self._console.print(f"Syntax Error found: {e}", level="ERROR")
+            self._console.print(f"Syntax Error found: {e}", level=LogLevel.ERROR)
             return False
 
     def _load_and_parse_rules(self) -> None:
@@ -75,7 +73,7 @@ def run(self) -> bool:
         """Runs the entire validation process."""
         try:
             self._load_source_code()
-            self._load_and_parse_rules()  # Загружаем правила до парсинга AST
+            self._load_and_parse_rules()
 
             if not self._parse_ast_tree():
                 return False
@@ -84,17 +82,16 @@ def run(self) -> bool:
             raise
 
         for rule in self._validation_rules:
-            # check_syntax уже обработан в _parse_ast_tree, пропускаем его
             if getattr(rule.config, "type", None) == "check_syntax":
                 continue
 
             self._console.print(f"Executing rule: {rule.config.rule_id}", level=LogLevel.DEBUG)
             is_passed = rule.execute(self._ast_tree, self._source_code)
             if not is_passed:
-                self._console.print(rule.config.message, level="ERROR")
+                self._console.print(rule.config.message, level=LogLevel.ERROR)
                 self._failed_rules.append(rule.config.rule_id)
                 if getattr(rule.config, "is_critical", False) or self._config.stop_on_first_fail:
-                    self._console.print("Critical rule failed. Halting validation.", level="WARNING")
+                    self._console.print("Critical rule failed. Halting validation.", level=LogLevel.WARNING)
                     break
 
-        return not self._failed_rules
+        return not self._failed_rules

src/code_validator/rules_library/basic_rules.py

@@ -1,5 +1,3 @@
-# src/code_validator/rules_library/basic_rules.py
-
 """Contains concrete implementations of executable validation rules.
 
 This module defines the handler classes for both "short" (pre-defined) and
@@ -77,10 +75,10 @@ def execute(self, tree: ast.Module | None, source_code: str | None = None) -> bo
             True if no PEP8 violations are found, False otherwise.
         """
         if not source_code:
-            self._console.print("Source code is empty, skipping PEP8 check.", level="WARNING")
+            self._console.print("Source code is empty, skipping PEP8 check.", level=LogLevel.WARNING)
             return True
 
-        self._console.print(f"Rule {self.config.rule_id}: Running flake8 linter...", level="DEBUG")
+        self._console.print(f"Rule {self.config.rule_id}: Running flake8 linter...", level=LogLevel.DEBUG)
 
         params = self.config.params
         args = [sys.executable, "-m", "flake8", "-"]
@@ -102,19 +100,20 @@ def execute(self, tree: ast.Module | None, source_code: str | None = None) -> bo
 
             if process.returncode != 0 and process.stdout:
                 linter_output = process.stdout.strip()
-                self._console.print(f"Flake8 found issues:\n{linter_output}", level="DEBUG")
+                self._console.print(f"Flake8 found issues:\n{linter_output}", level=LogLevel.DEBUG)
                 return False
             elif process.returncode != 0:
-                self._console.print(f"Flake8 exited with code {process.returncode}:\n{process.stderr}", level="ERROR")
+                self._console.print(f"Flake8 exited with code {process.returncode}:\n{process.stderr}",
+                                    level=LogLevel.ERROR)
                 return False
 
-            self._console.print("PEP8 check passed.", level="DEBUG")
+            self._console.print("PEP8 check passed.", level=LogLevel.ERROR)
             return True
         except FileNotFoundError:
-            self._console.print("flake8 not found. Is it installed in the venv?", level="CRITICAL")
+            self._console.print("flake8 not found. Is it installed in the venv?", level=LogLevel.CRITICAL)
             return False
         except Exception as e:
-            self._console.print(f"An unexpected error occurred while running flake8: {e}", level="CRITICAL")
+            self._console.print(f"An unexpected error occurred while running flake8: {e}", level=LogLevel.CRITICAL)
             return False
 
 
@@ -157,11 +156,11 @@ def execute(self, tree: ast.Module | None, source_code: str | None = None) -> bo
             The boolean result of applying the constraint to the selected nodes.
         """
         if not tree:
-            self._console.print("AST not available, skipping rule.", level="WARNING")
+            self._console.print("AST not available, skipping rule.", level=LogLevel.WARNING)
             return True
 
-        self._console.print(f"Applying selector: {self._selector.__class__.__name__}", level="DEBUG")
+        self._console.print(f"Applying selector: {self._selector.__class__.__name__}", level=LogLevel.DEBUG)
         selected_nodes = self._selector.select(tree)
 
-        self._console.print(f"Applying constraint: {self._constraint.__class__.__name__}", level="DEBUG")
-        return self._constraint.check(selected_nodes)
+        self._console.print(f"Applying constraint: {self._constraint.__class__.__name__}", level=LogLevel.DEBUG)
+        return self._constraint.check(selected_nodes)