chore: use pre-commit for clang-format

.github/workflows/Clang-format-checker.yml

@@ -12,10 +12,12 @@ jobs:
     steps:
       - uses: actions/checkout@v4
 
-      - uses: DoozyX/clang-format-lint-action@v0.18
+      - uses: actions/setup-python@v5
         with:
-          source: '.'
-          exclude: './external'
-          extensions: 'h,cpp,cc'
-          clangFormatVersion: 18 # default is 18.1.8
-          style: file
+          python-version: '3.x'
+
+      - name: Install pre-commit
+        run: python -m pip install pre-commit
+
+      - name: Check formatting
+        run: pre-commit run --all-files --show-diff-on-failure

.gitignore

@@ -47,7 +47,6 @@ test-data
 target
 tmp
 compile_commands.json
-.pre-commit-config.yaml
 
 .codex
 

.pre-commit-config.yaml

@@ -0,0 +1,13 @@
+exclude: |
+  (?x)^(
+    external/|
+    rust/eloqstore-sys/vendor/
+  )
+
+repos:
+  - repo: https://github.com/pre-commit/mirrors-clang-format
+    rev: v18.1.8
+    hooks:
+      - id: clang-format
+        files: \.(c|cc|cpp|cxx|h|hh|hpp|hxx)$
+        args: [--style=file, --fallback-style=none]

CLAUDE.md

@@ -55,9 +55,17 @@ SDK tests (also run in CI): `python3 -m build --wheel` + pytest in `python/`; in
 
 ## Formatting
 
-`bash scripts/format.sh` — installs clang-format 18.1.8 on first run and formats the tree. Style is Google-based with Allman braces, 4-space indent, right pointer alignment (see `.clang-format`).
-
-**Always run `bash scripts/format.sh` and commit any changes it makes before opening (or updating) a PR.** The main branch has a format check in CI that will fail the PR otherwise — hand-written code that looks fine often still gets re-wrapped by clang-format (e.g. a call that now fits on one line). Don't rely on writing conforming code by hand; run the script.
+Install the Git hook once with `pre-commit install`. The hook uses an isolated,
+pinned clang-format 18.1.8 environment, so no system clang-format installation
+is needed. Style is Google-based with Allman braces, 4-space indent, and right
+pointer alignment (see `.clang-format`). Third-party sources under `external/`
+and the Rust vendor-link tree are excluded by `.pre-commit-config.yaml`.
+
+**Always run `pre-commit run --all-files` and commit any changes it makes before
+opening (or updating) a PR.** The main branch runs the same command in CI and
+will fail the PR otherwise — hand-written code that looks fine often still gets
+re-wrapped by clang-format (e.g. a call that now fits on one line). Don't rely on
+writing conforming code by hand; run the hook.
 
 ## Architecture
 

README.md

@@ -219,30 +219,26 @@ ctest --test-dir build/tests/
 ./build/benchmark/simple_bench --kvoptions=./benchmark/opts_append.ini --workload=write-read --kv_size=1024 --batch_size=20000 --max_key=10000000 --read_per_part=4 --partitions=1
 ```
 
-### Install git hooks
+### Install pre-commit
 
 ```shell
-git config core.hooksPath .githooks
+python3 -m pip install pre-commit
+pre-commit install
 ```
 
-The pre-commit hook auto-syncs `pyproject.toml` / `Cargo.toml` versions from `VERSION`.
-Run once after cloning. Bypass with `git commit --no-verify`.
+Run these commands once after cloning. The hook downloads and caches its own
+clang-format 18.1.8 environment; it does not install clang-format system-wide.
+Bypass the hook with `git commit --no-verify` only when necessary.
 
-### Install Format Tool
+### Format C/C++ code
 
-```shell 
-bash scripts/format.sh
+```shell
+pre-commit run --all-files
 ```
 
-This script will execute when necessary:
-- Install code format tools
-- Format the code
-
-The first time this script is executed, it will install:
-- libtinfo5
-- clang-format(18.1.8)
-
-**Note**: The main branch has code format checks; please run this script before committing the MR.
+Only tracked C/C++ files are formatted. Third-party code under `external/` and
+the Rust vendor-link tree are excluded. The main branch runs the same command;
+please run it before opening or updating a PR.
 
 ## 🤝 Contributing