ci: split P2 gate from legacy full suite

Co-Authored-By: Codex <noreply@openai.com>

Author: Pigbibi

.github/workflows/ci.yml

@@ -131,7 +131,100 @@ jobs:
       - name: Ensure uv.lock matches pyproject.toml
         run: uv lock --check
 
-      - name: Run unit tests
+      - name: Run P2 scoped unit tests
+        run: |
+          set -euo pipefail
+          # P2 scoped gate; this is not a repository-wide full-suite result.
+          PYTHONPATH=. PYTEST_DISABLE_PLUGIN_AUTOLOAD=1 uv run --no-sync python -m pytest -q tests/test_broker_reconciliation.py tests/test_execution_service.py tests/test_rebalance_service.py tests/test_ci_unit_test_contract.py
+
+      - name: Summarize P2 scoped gate
+        run: |
+          echo "P2 scoped gate passed; this is not a repository-wide full-suite result." >> "$GITHUB_STEP_SUMMARY"
+
+  legacy-full-suite:
+    runs-on: ubuntu-latest
+    timeout-minutes: 20
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v6
+
+      - name: Resolve QuantPlatformKit ref
+        id: quant-platform-kit-ref
+        run: |
+          set -euo pipefail
+          ref="main"
+          for candidate in "${GITHUB_HEAD_REF:-}" "${GITHUB_BASE_REF:-}"; do
+            if [ -n "$candidate" ] && git ls-remote --exit-code --heads https://github.com/QuantStrategyLab/QuantPlatformKit.git "$candidate" >/dev/null 2>&1; then
+              ref="$candidate"
+              break
+            fi
+          done
+          echo "ref=${ref}" >> "$GITHUB_OUTPUT"
+
+      - name: Resolve UsEquityStrategies ref
+        id: us-equity-strategies-ref
+        run: |
+          set -euo pipefail
+          ref="main"
+          for candidate in "${GITHUB_HEAD_REF:-}" "${GITHUB_BASE_REF:-}"; do
+            if [ -n "$candidate" ] && git ls-remote --exit-code --heads https://github.com/QuantStrategyLab/UsEquityStrategies.git "$candidate" >/dev/null 2>&1; then
+              ref="$candidate"
+              break
+            fi
+          done
+          echo "ref=${ref}" >> "$GITHUB_OUTPUT"
+
+      - name: Resolve UsEquitySnapshotPipelines ref
+        id: us-equity-snapshot-pipelines-ref
+        run: |
+          set -euo pipefail
+          ref="main"
+          for candidate in "${GITHUB_HEAD_REF:-}" "${GITHUB_BASE_REF:-}"; do
+            if [ -n "$candidate" ] && git ls-remote --exit-code --heads https://github.com/QuantStrategyLab/UsEquitySnapshotPipelines.git "$candidate" >/dev/null 2>&1; then
+              ref="$candidate"
+              break
+            fi
+          done
+          echo "ref=${ref}" >> "$GITHUB_OUTPUT"
+
+      - name: Checkout QuantPlatformKit
+        uses: actions/checkout@v6
+        with:
+          repository: QuantStrategyLab/QuantPlatformKit
+          ref: ${{ steps.quant-platform-kit-ref.outputs.ref }}
+          path: external/QuantPlatformKit
+
+      - name: Checkout UsEquityStrategies
+        uses: actions/checkout@v6
+        with:
+          repository: QuantStrategyLab/UsEquityStrategies
+          ref: ${{ steps.us-equity-strategies-ref.outputs.ref }}
+          path: external/UsEquityStrategies
+
+      - name: Checkout UsEquitySnapshotPipelines
+        uses: actions/checkout@v6
+        with:
+          repository: QuantStrategyLab/UsEquitySnapshotPipelines
+          ref: ${{ steps.us-equity-snapshot-pipelines-ref.outputs.ref }}
+          path: external/UsEquitySnapshotPipelines
+
+      - name: Setup Python
+        uses: actions/setup-python@v6
+        with:
+          python-version: "3.12"
+
+      - name: Install dependencies
+        run: |
+          set -euo pipefail
+          python -m pip install --upgrade pip uv
+          uv sync --frozen --extra test
+
+      - name: Install editable shared repositories
+        run: |
+          set -euo pipefail
+          uv pip install --no-deps -e external/QuantPlatformKit -e external/UsEquityStrategies -e external/UsEquitySnapshotPipelines
+
+      - name: Run legacy full suite
         run: |
           set -euo pipefail
           PYTHONPATH=. PYTEST_DISABLE_PLUGIN_AUTOLOAD=1 uv run --no-sync python -m pytest -q tests --ignore=tests/test_request_handling.py --ignore=tests/test_event_loop.py --ignore=tests/test_monitor_dispatcher.py --ignore=tests/test_notifications.py --ignore=tests/test_connect_timeout_alert.py

tests/test_ci_unit_test_contract.py

@@ -1,9 +1,43 @@
 from pathlib import Path
+import re
 
 
-def test_ci_runs_unit_tests_fail_closed() -> None:
+_P2_TEST_COMMAND = (
+    "python -m pytest -q "
+    "tests/test_broker_reconciliation.py "
+    "tests/test_execution_service.py "
+    "tests/test_rebalance_service.py "
+    "tests/test_ci_unit_test_contract.py"
+)
+_LEGACY_FULL_SUITE_COMMAND = (
+    "python -m pytest -q tests "
+    "--ignore=tests/test_request_handling.py "
+    "--ignore=tests/test_event_loop.py "
+    "--ignore=tests/test_monitor_dispatcher.py "
+    "--ignore=tests/test_notifications.py "
+    "--ignore=tests/test_connect_timeout_alert.py"
+)
+
+
+def _job_block(workflow: str, job_name: str) -> str:
+    match = re.search(
+        rf"^  {re.escape(job_name)}:\n(?P<block>.*?)(?=^  [A-Za-z][A-Za-z0-9-]*:\n|\Z)",
+        workflow,
+        flags=re.MULTILINE | re.DOTALL,
+    )
+    assert match is not None
+    return match.group("block")
+
+
+def test_ci_runs_p2_scoped_required_gate_and_unsuppressed_legacy_suite() -> None:
     workflow = Path(".github/workflows/ci.yml").read_text(encoding="utf-8")
-    unit_test_step = workflow.split("      - name: Run unit tests\n", maxsplit=1)[1]
+    required_job = _job_block(workflow, "test")
+    legacy_job = _job_block(workflow, "legacy-full-suite")
 
-    assert "python -m pytest -q tests" in unit_test_step
-    assert "|| true" not in unit_test_step
+    assert _P2_TEST_COMMAND in required_job
+    assert _LEGACY_FULL_SUITE_COMMAND in legacy_job
+    assert "P2 scoped gate" in required_job
+    assert "GITHUB_STEP_SUMMARY" in required_job
+    assert "|| true" not in required_job
+    assert "|| true" not in legacy_job
+    assert "continue-on-error" not in workflow