From 58a5eac884077272a4ae7624e5e883a30d78f3b3 Mon Sep 17 00:00:00 2001
From: Markus Hofbauer <markus.hofbauer@flyzipline.com>
Date: Fri, 1 May 2026 18:41:28 +0200
Subject: [PATCH] Drop pre-commit for yaml parsing

---
 .pre-commit-hooks.yaml                        |  4 +-
 .../check_useless_exclude_paths_hooks.py      | 40 ++++---------------
 2 files changed, 9 insertions(+), 35 deletions(-)

diff --git a/.pre-commit-hooks.yaml b/.pre-commit-hooks.yaml
index 094b413..cb3cf1d 100644
--- a/.pre-commit-hooks.yaml
+++ b/.pre-commit-hooks.yaml
@@ -190,7 +190,7 @@
   always_run: true
   language: python
   additional_dependencies:
-    - pre-commit==4.6.0
+    - ruamel-yaml==0.19.1
 - id: print-pre-commit-metrics
   name: Print pre-commit metrics such as number of excluded files
   description: |-
@@ -203,7 +203,7 @@
   pass_filenames: false
   language: python
   additional_dependencies:
-    - pre-commit==4.6.0
+    - ruamel-yaml==0.19.1
 - id: sync-vscode-config
   name: Sync VSCode Config
   description: |-
diff --git a/dev_tools/check_useless_exclude_paths_hooks.py b/dev_tools/check_useless_exclude_paths_hooks.py
index 9cb5639..3701cae 100644
--- a/dev_tools/check_useless_exclude_paths_hooks.py
+++ b/dev_tools/check_useless_exclude_paths_hooks.py
@@ -6,20 +6,12 @@
 import itertools
 import sys
 from collections import Counter
-from functools import partial
 from pathlib import Path
-from typing import TYPE_CHECKING, Any
+from typing import Any
 
-from pre_commit.clientlib import load_config as pre_commit_load_config
-from pre_commit.constants import CONFIG_FILE
-from pre_commit.yaml import yaml_load
+from ruamel.yaml import YAML
 
-if TYPE_CHECKING:
-    from collections.abc import Callable
-
-IGNORED_HOOK_ATTRIBUTES = {
-    "pass_filenames"  # https://prek.j178.dev/configuration/#pass_filenames supports also int instead of boolean in pre-commit
-}
+CONFIG_FILE = ".pre-commit-config.yaml"
 
 
 class Hook:
@@ -87,29 +79,11 @@ def has_excludes(hook_config: dict[str, str]) -> bool:
     return bool(hook_config.get("exclude")) and hook_config.get("exclude") != "^$"
 
 
-def _load_config_with_compatibility_overrides(contents: str) -> object:
-    config = yaml_load(contents)
-
-    if isinstance(config, dict):
-        for repo in config.get("repos", []):
-            if isinstance(repo, dict):
-                # Set a version for builtin since pre-commit does not know this as local repo
-                # prek only: https://prek.j178.dev/builtin/?h=buil#2-explicit-builtin-repository
-                if repo.get("repo") == "builtin":
-                    repo.setdefault("rev", "")
-
-                for hook in repo.get("hooks", []):
-                    if isinstance(hook, dict):
-                        for attribute in IGNORED_HOOK_ATTRIBUTES:
-                            hook.pop(attribute, None)
-
-    return config
-
+def load_config(config_file: Path) -> dict[str, Any]:
+    yaml = YAML(typ="safe")
+    config = yaml.load(config_file.read_text(encoding="utf-8"))
 
-load_config: Callable[[Path], dict[str, Any]] = partial(
-    pre_commit_load_config,
-    load_strategy=_load_config_with_compatibility_overrides,
-)
+    return config if isinstance(config, dict) else {}
 
 
 def load_hooks(root_directory: Path, config_file: Path) -> list[Hook]: