Fmt

Author: KevinFairise2

src/dda/cli/validate/__init__.py

@@ -13,4 +13,3 @@ def cmd() -> None:
     """
     Validate tools and utilities for development workflow.
     """
-    pass

src/dda/cli/validate/ai_rules/__init__.py

@@ -3,13 +3,12 @@
 # SPDX-License-Identifier: MIT
 from __future__ import annotations
 
-import os
 from typing import TYPE_CHECKING
 
 import click
-from dda.utils.diff import pretty_diff
 
 from dda.cli.base import dynamic_command, pass_app
+from dda.utils.diff import pretty_diff
 from dda.utils.fs import Path
 
 if TYPE_CHECKING:
@@ -18,6 +17,7 @@
 CURSOR_RULES_DIR = Path(".cursor/rules")
 TARGETS_FILES = [Path("CLAUDE.md")]
 
+
 @dynamic_command(short_help="Validate AI rules are coherent between all the coding agent config files")
 @click.option(
     "--fix",
@@ -46,7 +46,7 @@ def cmd(app: Application, *, should_fix: bool) -> None:
         new_content = generate_content(rule_files, target_file)
         old_content = ""
         if target_file.exists() and target_file.is_file():
-            with open(target_file, "r", encoding="utf-8") as f:
+            with open(target_file, encoding="utf-8") as f:
                 old_content = f.read()
         diff = pretty_diff(old_content, new_content)
         if not diff:
@@ -58,15 +58,16 @@ def cmd(app: Application, *, should_fix: bool) -> None:
                 f.write(new_content)
             app.display_success(f"Successfully fixed {target_file}")
         else:
-                unsynced_targets.append(str(target_file))
+            unsynced_targets.append(str(target_file))
     if unsynced_targets:
         app.display_error(f"The following targets are not in sync: {', '.join(unsynced_targets)}")
         app.abort()
     app.display_success("All targets are in sync")
 
+
 def get_rule_files(cursor_rules_dir: Path) -> list[Path]:
     """Find all rule files in cursor rules directory (recursively), excluding personal rules."""
-    return sorted(rule for rule in cursor_rules_dir.glob('**/*.mdc') if "personal" not in rule.parts)
+    return sorted(rule for rule in cursor_rules_dir.glob("**/*.mdc") if "personal" not in rule.parts)
 
 
 def generate_content(rule_files: list[Path], target_file: Path) -> str:

src/dda/utils/diff.py

@@ -1,7 +1,7 @@
 import difflib
 
 
-def pretty_diff(string1, string2) -> str:
+def pretty_diff(string1: str, string2: str) -> str:
     lines1 = string1.splitlines()
     lines2 = string2.splitlines()
 
@@ -15,4 +15,4 @@ def pretty_diff(string1, string2) -> str:
             result.append(f"\033[32m{line}\033[0m")  # Green for additions
         else:
             result.append(line)
-    return '\n'.join(result)
+    return "\n".join(result)

tests/cli/validate/__init__.py

@@ -1,3 +1,3 @@
 # SPDX-FileCopyrightText: 2025-present Datadog, Inc. <dev@datadoghq.com>
 #
-# SPDX-License-Identifier: MIT
+# SPDX-License-Identifier: MIT

tests/cli/validate/test_ai-rules.py renamed to tests/cli/validate/test_ai_rules.py

@@ -3,25 +3,25 @@
 # SPDX-License-Identifier: MIT
 from __future__ import annotations
 
-import os
 import shutil
 from typing import TYPE_CHECKING
-from dda.utils.fs import Path
 
 import pytest
 
+from dda.utils.fs import Path
+
 if TYPE_CHECKING:
     from collections.abc import Callable
 
     from tests.conftest import CliRunner
 
 
-
 @pytest.fixture(name="use_temp_fixture_folder")
-def fixt_use_temp_folder(temp_dir: Path):
-    def _use_temp_folder(folder_name: str):
-        shutil.copytree(Path((__file__)).parent / "fixtures" / "ai_rules" / folder_name, temp_dir / folder_name)
+def fixt_use_temp_folder(temp_dir: Path) -> Callable[[str], Path]:
+    def _use_temp_folder(folder_name: str) -> Path:
+        shutil.copytree(Path(__file__).parent / "fixtures" / "ai_rules" / folder_name, temp_dir / folder_name)
         return Path(temp_dir) / folder_name
+
     return _use_temp_folder
 
 
@@ -50,13 +50,13 @@ def test_validate_with_fix_flag(
     result.check_exit_code(exit_code=0)
     assert (path / "CLAUDE.md").exists()
     content = (path / "CLAUDE.md").read_text(encoding="utf-8")
-    assert f"@.cursor/rules/coding-standards.mdc" in content
-    assert f"@.cursor/rules/security.mdc" in content
-    assert f"@.cursor/rules/testing.mdc" in content
-    assert f"imhere.txt" not in content
-    assert f"@.cursor/rules/personal/my-rule.mdc" not in content
-    assert f"@.cursor/rules/nested/my-nested-rule.mdc" in content
-    assert f"@CLAUDE_PERSONAL.md" in content
+    assert "@.cursor/rules/coding-standards.mdc" in content
+    assert "@.cursor/rules/security.mdc" in content
+    assert "@.cursor/rules/testing.mdc" in content
+    assert "imhere.txt" not in content
+    assert "@.cursor/rules/personal/my-rule.mdc" not in content
+    assert "@.cursor/rules/nested/my-nested-rule.mdc" in content
+    assert "@CLAUDE_PERSONAL.md" in content
 
 
 def test_validate_no_cursor_rules_directory(