From a7cdf2f6ab44f342cc14d3d9d313843a86f377b9 Mon Sep 17 00:00:00 2001
From: Thomas Schmelzer <thomas.schmelzer@gmail.com>
Date: Fri, 25 Sep 2026 16:44:01 +0400
Subject: [PATCH] chore: add a pre-commit config adapted from jebel-quant/rhiza

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 .pre-commit-config.yaml    | 101 +++++++++++++++++++++++++++++++++++++
 scripts/check-dashboard.py |   2 -
 2 files changed, 101 insertions(+), 2 deletions(-)
 create mode 100644 .pre-commit-config.yaml

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
new file mode 100644
index 0000000..2afc8a3
--- /dev/null
+++ b/.pre-commit-config.yaml
@@ -0,0 +1,101 @@
+# Adapted from the jebel-quant/rhiza repository
+# (https://github.com/jebel-quant/rhiza), for a repo that is not rhiza-managed
+# and keeps its Python project in collector/ rather than at the root. What was
+# dropped or rescoped, and why:
+#
+# - rhiza-hooks (check-rhiza-config and friends): they validate .rhiza/, which
+#   this repo does not have.
+# - check-jsonschema for template-bundles.yml and check-renovate: neither file
+#   exists here.
+# - interrogate: rhiza's hook reads [tool.interrogate], and without that table
+#   interrogate falls back to its own defaults - a standard nobody here chose.
+#   Add the table first if docstring coverage should be gated.
+# - bandit: rhiza passes --ini .bandit; here it is scoped to the package by path
+#   instead, since the tests assert freely and that is not a finding.
+# - ruff is pinned to the version in collector/uv.lock, which is what CI runs,
+#   so the hook and the Lint step cannot disagree. Bump both together.
+# - uv-lock points at collector/, where the lock lives.
+#
+# Pin node so pre-commit provisions a compatible runtime instead of using the
+# system node. Some npm-based hooks (markdownlint-cli) pull transitive deps that
+# reject odd-numbered current node releases (e.g. v25), failing on EBADENGINE.
+default_language_version:
+  node: "24.12.0"
+
+repos:
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v6.0.0
+    hooks:
+      - id: check-toml
+      - id: check-yaml
+        args: ['--unsafe']
+
+  - repo: local
+    hooks:
+      - id: no-python-cache-files
+        name: Prevent committing Python cache files
+        language: fail
+        entry: Python cache files (__pycache__, .pyc, .pyo, .pyd) must not be committed.
+        files: '(/__pycache__/|\.py[cod]$)'
+
+      - id: no-rej-files
+        name: Reject .rej files
+        entry: "bash -c 'find . -type f -name \"*.rej\" | grep -q . && { echo \"ERROR: .rej files detected\"; find . -type f -name \"*.rej\"; exit 1; } || exit 0'"
+        language: system
+        pass_filenames: false
+
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: 'v0.16.9'
+    hooks:
+      - id: ruff
+        args: [ --fix, --exit-non-zero-on-fix, --unsafe-fixes ]
+
+      # Run the formatter
+      - id: ruff-format
+
+  - repo: https://github.com/igorshubovych/markdownlint-cli
+    rev: v0.49.1
+    hooks:
+      - id: markdownlint
+        args: ["--disable", "MD013"]
+
+  - repo: https://github.com/python-jsonschema/check-jsonschema
+    rev: 0.38.0
+    hooks:
+      - id: check-github-workflows
+        args: ["--verbose"]
+
+  - repo: https://github.com/rhysd/actionlint
+    rev: v1.7.12
+    hooks:
+      - id: actionlint
+
+  - repo: https://github.com/abravalheri/validate-pyproject
+    rev: "0.26"
+    hooks:
+      - id: validate-pyproject
+        files: ^collector/pyproject\.toml$
+
+  - repo: https://github.com/PyCQA/bandit
+    rev: 1.9.4
+    hooks:
+      - id: bandit
+        files: ^collector/jq_collector/.*\.py$
+
+  - repo: https://github.com/shellcheck-py/shellcheck-py
+    rev: v0.11.0.1
+    hooks:
+      - id: shellcheck
+        files: '\.sh$'
+
+  - repo: https://github.com/betterleaks/betterleaks
+    rev: v1.8.1
+    hooks:
+      - id: betterleaks
+
+  - repo: https://github.com/astral-sh/uv-pre-commit
+    rev: 0.12.10
+    hooks:
+      - id: uv-lock
+        args: [--project, collector]
+        files: ^collector/(pyproject\.toml|uv\.lock)$
diff --git a/scripts/check-dashboard.py b/scripts/check-dashboard.py
index 14c44cb..60bfafc 100755
--- a/scripts/check-dashboard.py
+++ b/scripts/check-dashboard.py
@@ -44,7 +44,6 @@ def walk(panels):
     # claiming the same cells makes Grafana shuffle them unpredictably on load.
     placed = [p for p in dash["panels"]]
 
-
     def overlaps(a, b):
         return not (
             a["x"] + a["w"] <= b["x"]
@@ -53,7 +52,6 @@ def overlaps(a, b):
             or b["y"] + b["h"] <= a["y"]
         )
 
-
     for i, first in enumerate(placed):
         for second in placed[i + 1 :]:
             if overlaps(first["gridPos"], second["gridPos"]):