Add pre-commit hook for formatting, type checking, and testing

- Create pre-commit hook script to run formatter, mypy, and pytest
- Add setup script to install Git hooks
- Update DEVELOPMENT.md with documentation for Git hooks
- Ensure formatting changes are automatically added to commits

Author: jpstroop

docs/DEVELOPMENT.md

@@ -300,9 +300,34 @@ The OAuth callback mechanism is implemented using two main classes:
 1. Create a new branch for your feature/fix
 2. Make your changes, following the style guidelines (see also:
    [LINTING](docs/LINTING.md))
-3. Run formatting checks (`pdm format`) and tests (`pdm run pytest`)
+3. Run formatting checks (`pdm run format`) and tests (`pdm run pytest`)
 4. Submit a pull request with a clear description of changes
 
+### Git Hooks
+
+The project includes pre-commit hooks to ensure code quality. To set up the
+hooks:
+
+```bash
+# From the repository root
+./lint/setup-hooks.sh
+```
+
+This will set up a pre-commit hook that automatically runs:
+
+1. `pdm run format` - Runs black, isort, and adds file headers
+2. `pdm run mypy` - Runs type checking
+3. `pdm run pytest` - Runs all tests
+
+If any of these checks fail, the commit will be blocked. Any formatting changes
+made by the hook will be automatically added to the commit.
+
+To bypass the hooks for a specific commit (not recommended for normal usage):
+
+```bash
+git commit --no-verify -m "Your commit message"
+```
+
 ## Release Process
 
 This section will be documented as we near our first release.

lint/pre-commit.sh

@@ -0,0 +1,55 @@
+#!/bin/bash
+
+echo "Running pre-commit hooks..."
+
+# Save current staged changes temporarily
+echo "Stashing unstaged changes..."
+git stash -q --keep-index
+
+# Get the directory of the repository
+REPO_DIR=$(git rev-parse --show-toplevel)
+cd "$REPO_DIR" || exit 1
+
+# Execute linting and tests
+echo "Running formatter..."
+pdm run format
+
+FORMATTING=$?
+
+echo "Running mypy..."
+pdm run mypy
+
+MYPY=$?
+
+echo "Running pytest..."
+pdm run pytest
+
+PYTEST=$?
+
+# Restore unstaged changes
+echo "Restoring unstaged changes..."
+git stash pop -q
+
+# If any command failed, abort the commit
+if [ $FORMATTING -ne 0 ] || [ $MYPY -ne 0 ] || [ $PYTEST -ne 0 ]; then
+    echo "Pre-commit hooks failed!"
+    if [ $FORMATTING -ne 0 ]; then
+        echo "Formatting failed!"
+    fi
+    if [ $MYPY -ne 0 ]; then
+        echo "Type checking failed!"
+    fi
+    if [ $PYTEST -ne 0 ]; then
+        echo "Tests failed!"
+    fi
+    exit 1
+fi
+
+# If any files were changed by formatting, add them to the commit
+if git diff --name-only | grep -q '.'; then
+    echo "Formatting changed files. Adding them to the commit..."
+    git add -u
+fi
+
+echo "Pre-commit hooks passed!"
+exit 0

lint/setup-hooks.sh

@@ -0,0 +1,19 @@
+#!/bin/bash
+
+# Script to set up Git hooks
+
+REPO_ROOT=$(git rev-parse --show-toplevel)
+HOOK_DIR="$REPO_ROOT/.git/hooks"
+SCRIPT_DIR="$REPO_ROOT/lint"
+
+echo "Setting up Git hooks..."
+
+# Create hooks directory if it doesn't exist
+mkdir -p "$HOOK_DIR"
+
+# Create symlink for pre-commit hook
+ln -sf ../../lint/pre-commit.sh "$HOOK_DIR/pre-commit"
+chmod +x "$SCRIPT_DIR/pre-commit.sh"
+
+echo "Git hooks setup complete!"
+echo "Pre-commit hook will run formatter, type checker, and tests before each commit."