chore: integrate pre-commit hooks (baserow#5392)

* chore: integrate pre-commit hooks for backend and frontend code quality enforcement

* fix: uv.lock

* refactor: consolidate pre-commit hooks to use just recipes and add a pre-commit-uninstall command

* docs: document pre-commit branch linting and update pre-commit configuration to increase file size limits and remove serial execution requirements.

Author: alamin-br

.pre-commit-config.yaml

@@ -0,0 +1,38 @@
+# Pre-commit hooks for Baserow.
+#
+# The lint/format hooks intentionally delegate to the `just` recipes
+# (`just b fix` and `just f fix`) so that pre-commit, CI and manual runs all
+# use the exact same commands and configuration. This avoids drift between
+# the tools and re-running `just fix` after pre-commit will not produce
+# additional changes.
+repos:
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v5.0.0
+    hooks:
+      - id: check-yaml
+        # Helm templates and multi-document GitLab CI files are not valid
+        # standalone YAML and are validated by their own tooling.
+        args: [--allow-multiple-documents]
+        exclude: ^(web-frontend/\.nuxt/|web-frontend/dist/|web-frontend/\.output/|deploy/helm/|\.gitlab/)
+      - id: check-added-large-files
+        # Built-in templates ship as multi-MB .zip fixtures; bump the limit so
+        # those are allowed while still flagging unexpectedly large additions.
+        args: [--maxkb=5000]
+      - id: check-merge-conflict
+
+  - repo: local
+    hooks:
+      - id: backend-fix
+        name: backend lint & format (just b fix)
+        entry: just b fix
+        language: system
+        pass_filenames: false
+        files: \.py$
+
+      - id: frontend-fix
+        name: frontend lint & format (just f fix)
+        entry: just f fix
+        language: system
+        pass_filenames: false
+        files: \.(js|jsx|ts|tsx|vue|mjs|css|scss|json|md|html)$
+        exclude: ^(web-frontend/\.nuxt/|web-frontend/dist/|web-frontend/\.output/)

AGENTS.md

@@ -13,6 +13,7 @@ Use `just` from the repo root; it wraps the backend and frontend workflows consi
 - `just b test -n=auto` runs backend pytest suites in parallel.
 - `just f test` runs frontend Vitest suites.
 - `just lint` runs both backend and frontend linters; `just fix` applies auto-fixes.
+- `just b run pre-commit run --files $(git diff --name-only HEAD)` lints only the files you've touched on your branch (staged + unstaged); use `origin/develop...HEAD` instead of `HEAD` to scope to the whole branch. See `docs/development/code-quality.md` for details.
 - `just b migrate` runs Django migrations.
 
 For direct package-manager use, backend commands run through `uv` and frontend commands through `yarn`.

backend/justfile

@@ -207,11 +207,11 @@ check: _check-dev
     #!/usr/bin/env bash
     set -euo pipefail
     {{ _load_env }}
-   
+
     {{ uv_run }} ruff check --config pyproject.toml {{ backend_source_dirs }} {{ backend_tests_dirs }}
     {{ uv_run }} ruff format --config pyproject.toml --check {{ backend_source_dirs }} {{ backend_tests_dirs }}
 
-alias lint := check 
+alias lint := check
 alias l := check
 
 # Fix code style (sort imports + format)
@@ -340,7 +340,7 @@ check-migrations: _check-dev
     #!/usr/bin/env bash
     set -euo pipefail
     {{ _load_env }}
-    
+
     DJANGO_SETTINGS_MODULE={{ django_settings }} {{ uv_run }} baserow makemigrations --dry-run --check
 
 # =============================================================================

backend/pyproject.toml

@@ -114,6 +114,7 @@ baserow = "baserow.manage:main"
 
 [dependency-groups]
 dev = [
+    "pre-commit>=4.0.0",
     "ruff>=0.8.0",
     "pytest==9.0.3",
     "pytest-django==4.12.0",

backend/uv.lock

@@ -10,22 +10,87 @@ If you have the [development environment](./running-the-dev-env-locally.md) up a
 you can easily run the linters using [just](./justfile.md) commands.
 
 **Backend (from project root or `backend/` directory):**
-* `just b format`: auto format all Python code using black.
-* `just b sort`: sort imports using isort.
-* `just b fix`: run both format and sort.
-* `just b lint`: check Python code with flake8, black, isort, and bandit.
+
+- `just b format`: auto-format all Python code using Ruff formatter.
+- `just b fix`: run Ruff checks with automatic fixes and format Python code.
+- `just b lint`: check Python code with Ruff.
 
 **Frontend (from project root or `web-frontend/` directory):**
-* `just f lint`: check JavaScript with eslint and SCSS with stylelint.
-* `just f fix`: auto-fix code style issues.
+
+- `just f lint`: check JavaScript and SCSS files with ESLint and Stylelint.
+- `just f fix`: auto-fix code style issues.
 
 ## Running tests
 
 There are also commands to easily run the tests.
 
-* `just b test` (backend): run all backend Python tests with pytest.
-* `just b test -n=auto` (backend): run tests in parallel for faster execution.
-* `just f test` (frontend): run all frontend tests with Jest.
+- `just b test` (backend): run all backend Python tests with pytest.
+- `just b test -n=auto` (backend): run tests in parallel for faster execution.
+- `just f test` (frontend): run all frontend tests with Jest.
+
+## Git pre-commit hooks
+
+Baserow uses [`pre-commit`](https://pre-commit.com/) to automatically run linters and
+formatters before commits are created. This ensures your changes comply with repo-wide
+code quality rules without waiting for CI feedback.
+
+The lint/format hooks delegate to the same `just b fix` and `just f fix` recipes used
+by CI and manual runs, so pre-commit will never produce changes that differ from
+running `just fix` yourself.
+
+### Installation
+
+To set up the pre-commit hooks locally in your `.git/` folder, run the following command from the repository root:
+
+```bash
+just pre-commit-install
+```
+
+This registers a few general hygiene hooks (YAML syntax checks, merge-conflict
+markers, large files) plus the backend and frontend lint/format hooks. Trailing
+whitespace and end-of-file fixes are intentionally left to `ruff`/`prettier` to
+avoid touching legacy files.
+
+To remove the hooks again:
+
+```bash
+just pre-commit-uninstall
+```
+
+### Running manually
+
+You can also run pre-commit manually at any time against all files or staged changes:
+
+```bash
+# Run against all files
+just b run pre-commit run --all-files
+
+# Run against specific files
+just b run pre-commit run --files path/to/file1.py path/to/file2.js
+```
+
+#### Tip: lint everything you've touched on this branch
+
+A normal `git commit` runs the hooks on **staged files only**, and `pre-commit run`
+with no arguments does the same. To lint every file you have changed relative to
+`HEAD` (staged *and* unstaged), pass the diff explicitly:
+
+```bash
+just b run pre-commit run --files $(git diff --name-only HEAD)
+```
+
+This is handy before opening a pull request: it scopes the run to your
+work-in-progress without re-linting the entire monorepo the way
+`--all-files` does. Swap `HEAD` for a base ref (for example `origin/develop`) to
+lint everything your branch changes:
+
+```bash
+just b run pre-commit run --files $(git diff --name-only origin/develop...HEAD)
+```
+
+See the [pre-commit documentation](https://pre-commit.com/#usage) for more advanced
+usage (skipping hooks for a commit, running a single hook, updating hook versions,
+etc.).
 
 ## Continuous integration