From 6f4421123ede997e613e60b29254982ded1ac927 Mon Sep 17 00:00:00 2001 From: Claude Date: Fri, 17 Apr 2026 04:48:46 +0000 Subject: [PATCH] fix(ci): pin test deps to prevent breaking updates + restore architecture_brief Root cause: pytest/pytest-asyncio/aiosqlite minimum version pins (>=X) allowed pip to install latest versions that may have breaking changes. The first CI run (a319feb) passed because it ran before the update; all subsequent runs failed with exit code 4 (collection error). Fix: Pin upper bounds on test dependencies: - pytest>=8.0.0,<8.4.0 - pytest-asyncio>=0.24.0,<0.26.0 - aiosqlite>=0.20.0,<0.21.0 - httpx>=0.28.1,<0.29.0 Also restore architecture_brief.py to CI pipeline (was removed during debugging, now restored since issue was deps not script). https://claude.ai/code/session_01W1rJthWDkasijTdXCfxVHs --- .github/workflows/dealix-ci.yml | 3 +++ salesflow-saas/backend/requirements-dev.txt | 7 ++++--- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/.github/workflows/dealix-ci.yml b/.github/workflows/dealix-ci.yml index 2418ef8a..cbbe0f06 100644 --- a/.github/workflows/dealix-ci.yml +++ b/.github/workflows/dealix-ci.yml @@ -25,6 +25,9 @@ jobs: - name: Install dependencies run: | pip install -r requirements.txt -r requirements-dev.txt + - name: Architecture Brief (governance validation) + working-directory: salesflow-saas + run: python scripts/architecture_brief.py - name: Pytest (full suite + launch scenarios) env: DATABASE_URL: sqlite+aiosqlite:///./ci_dealix.db diff --git a/salesflow-saas/backend/requirements-dev.txt b/salesflow-saas/backend/requirements-dev.txt index df0d5f18..5fd48ebd 100644 --- a/salesflow-saas/backend/requirements-dev.txt +++ b/salesflow-saas/backend/requirements-dev.txt @@ -1,4 +1,5 @@ # Dev / CI — not required in production images -pytest>=8.0.0 -pytest-asyncio>=0.24.0 -aiosqlite>=0.20.0 +pytest>=8.0.0,<8.4.0 +pytest-asyncio>=0.24.0,<0.26.0 +aiosqlite>=0.20.0,<0.21.0 +httpx>=0.28.1,<0.29.0