From 4cbd2a703a01cc78ae3d78b89af9f026c1a22901 Mon Sep 17 00:00:00 2001 From: Harvey Tseng Date: Fri, 11 Sep 2026 20:41:31 -0400 Subject: [PATCH] ci: stop gating generated specs and SDKs at PR time - spec check reconciles routes against handlers and is the step with teeth; spec gen --check compares generator output to itself - the bot lands openapi/asyncapi/mcp-tools/sdks after merge --- .github/workflows/soothfast.yml | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/.github/workflows/soothfast.yml b/.github/workflows/soothfast.yml index 1cc67d2c..87543566 100644 --- a/.github/workflows/soothfast.yml +++ b/.github/workflows/soothfast.yml @@ -242,10 +242,8 @@ jobs: changelog: false - name: Reconcile OpenAPI/AsyncAPI/GraphQL routes against handlers run: cargo soothfast spec check -p finance-query-server --target soothfast-routes - - name: Generated openapi.yaml/asyncapi.yaml are fresh - run: cargo soothfast spec gen -p finance-query-server --target soothfast-routes --check - - name: Generated Python SDK is fresh - run: cargo soothfast sdk gen -p finance-query-server --target soothfast-routes --check + # --from-committed reads the base's committed spec, not a fresh rebuild. + # A branch older than the bot's spec commit reports a BREAKING; rebase. - name: Spec compat gate vs merge-base if: github.event_name == 'pull_request' env: @@ -295,8 +293,6 @@ jobs: changelog: false - name: Reconcile MCP tool routes against mcp-tools.json run: cargo soothfast spec check -p finance-query-mcp --target soothfast-routes --features backtesting - - name: Generated mcp-tools.json is fresh - run: cargo soothfast spec gen -p finance-query-mcp --target soothfast-routes --check --features backtesting - name: Regenerate MCP route pages run: make docs-pages PAGES=mcp - name: Upload MCP route pages