fix(validation): bundle schemas before wheel builds

Fixes #897 by bundling schemas before release and CI wheel builds, and by surfacing SDK-pinned missing schema bundles without warning on explicit unsupported version probes.

Author: bokelley

.github/workflows/ci.yml

@@ -171,6 +171,7 @@ jobs:
       - name: Build and install wheel
         run: |
           python -m pip install --upgrade pip build
+          python scripts/bundle_schemas.py
           python -m build --wheel --outdir dist/
           pip install dist/*.whl
 

.github/workflows/release-please.yml

@@ -95,6 +95,10 @@ jobs:
           python -m pip install --upgrade pip
           pip install build twine
 
+      - name: Bundle schemas
+        if: ${{ steps.release.outputs.release_created == 'true' || (github.event_name == 'workflow_dispatch' && inputs.publish == true) }}
+        run: python scripts/bundle_schemas.py
+
       - name: Build package
         if: ${{ steps.release.outputs.release_created == 'true' || (github.event_name == 'workflow_dispatch' && inputs.publish == true) }}
         run: python -m build

src/adcp/validation/schema_loader.py

@@ -188,7 +188,8 @@ def _ensure_state(version: str | None = None) -> _LoaderState | None:
             return None
         root = _resolve_schema_root(bundle_key)
         if root is None:
-            logger.debug(
+            log_missing = logger.warning if version is None else logger.debug
+            log_missing(
                 "AdCP schemas not found for bundle_key=%s; validation will skip "
                 "all tools for this version",
                 bundle_key,

tests/test_schema_loader_per_version.py

@@ -11,6 +11,7 @@
 from __future__ import annotations
 
 import json
+import logging
 import shutil
 from pathlib import Path
 
@@ -190,6 +191,38 @@ def test_validate_request_unknown_version_skips_safely(
     assert outcome.issues == []
 
 
+def test_missing_sdk_pinned_bundle_warns(
+    monkeypatch: pytest.MonkeyPatch, caplog: pytest.LogCaptureFixture
+) -> None:
+    monkeypatch.setattr(_loader_mod, "_resolve_schema_root", lambda bundle_key=None: None)
+
+    _reset_for_tests()
+    try:
+        with caplog.at_level(logging.WARNING, logger="adcp.validation.schema_loader"):
+            outcome = validate_request("synthetic_tool", {"x": 1})
+    finally:
+        _reset_for_tests()
+
+    assert outcome.valid
+    assert any("AdCP schemas not found" in rec.message for rec in caplog.records)
+
+
+def test_missing_explicit_version_bundle_does_not_warn(
+    monkeypatch: pytest.MonkeyPatch, caplog: pytest.LogCaptureFixture
+) -> None:
+    monkeypatch.setattr(_loader_mod, "_resolve_schema_root", lambda bundle_key=None: None)
+
+    _reset_for_tests()
+    try:
+        with caplog.at_level(logging.WARNING, logger="adcp.validation.schema_loader"):
+            outcome = validate_request("synthetic_tool", {"x": 1}, version="9.9.9")
+    finally:
+        _reset_for_tests()
+
+    assert outcome.valid
+    assert not any("AdCP schemas not found" in rec.message for rec in caplog.records)
+
+
 def test_validate_response_threads_version_through(
     synthetic_legacy_bundle: tuple[str, Path],
 ) -> None: