From 2fedcf7e2f84debec244fdee7d12dd37fbdc772e Mon Sep 17 00:00:00 2001 From: Ragnor Comerford Date: Fri, 17 Apr 2026 16:06:54 +0200 Subject: [PATCH] Revert "Add pre-commit hook for openapi.json drift" This reverts commit 7c6d89d194ab2c682ca8267e709b7cd6dafb5813. --- .pre-commit-config.yaml | 10 ---------- CONTRIBUTING.md | 18 ------------------ 2 files changed, 28 deletions(-) delete mode 100644 .pre-commit-config.yaml diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml deleted file mode 100644 index cfe2e89..0000000 --- a/.pre-commit-config.yaml +++ /dev/null @@ -1,10 +0,0 @@ -repos: - - repo: local - hooks: - - id: openapi-sync - name: Sync openapi.json - description: Regenerate openapi.json when the server API surface changes. - entry: bash -c 'OMNIGRAPH_UPDATE_OPENAPI=1 cargo test -p omnigraph-server --test openapi openapi_spec_is_up_to_date --quiet' - language: system - pass_filenames: false - files: ^(crates/omnigraph-server/src/.*\.rs|crates/omnigraph-server/Cargo\.toml|openapi\.json)$ diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index eb6ce7c..65d1e24 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -16,24 +16,6 @@ cargo test --workspace If you touch S3-backed flows, the CI model uses a local RustFS instance for integration tests. -### OpenAPI spec - -`openapi.json` is a committed artifact generated from the Utoipa annotations in -`crates/omnigraph-server`. CI fails if it drifts from the source. To regenerate -manually: - -```bash -OMNIGRAPH_UPDATE_OPENAPI=1 cargo test -p omnigraph-server --test openapi openapi_spec_is_up_to_date -``` - -Optional: install [pre-commit](https://pre-commit.com) to run this automatically -before each commit that touches the server: - -```bash -pip install pre-commit -pre-commit install -``` - ## Pull Requests - keep changes focused