mainsequence 8.1.20.dev4__tar.gz → 8.1.20.dev7__tar.gz

{mainsequence-8.1.20.dev4/mainsequence.egg-info → mainsequence-8.1.20.dev7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mainsequence
-Version: 8.1.20.dev4
+Version: 8.1.20.dev7
 Summary: Main Sequence SDK 
 Author-email: Main Sequence GmbH <dev@main-sequence.io>
 License-Expression: LicenseRef-MainSequence-Proprietary

{mainsequence-8.1.20.dev4 → mainsequence-8.1.20.dev7}/mainsequence/client/base.py

@@ -58,7 +58,9 @@ class HtmlSaveException(Exception):
 
 
 class BasePydanticModel(BaseModel):
-    model_config = ConfigDict(extra="forbid")  # Forbid extra fields in v2
+    # Tolerant reading (ADR 0032): a backend response that carries a field this
+    # SDK release does not declare parses, and the undeclared field is dropped.
+    model_config = ConfigDict(extra="ignore")
     orm_class: str = None  # This will be set to the class that inherits
 
     @classmethod
@@ -780,10 +782,21 @@ class BaseObjectOrm:
 
             return raw_key
 
+        def _declares_field(obj, field_name: str) -> bool:
+            if not isinstance(obj, BaseModel):
+                return True
+            if field_name in type(obj).model_fields:
+                return True
+            # Tolerant reading (ADR 0032): a model that keeps undeclared keys can
+            # take them; every other model drops them, as parsing does.
+            return type(obj).model_config.get("extra") == "allow"
+
         def recursive_update(obj, update_dict, path=()):
             for k, v in update_dict.items():
                 current_path = (*path, k)
                 field_name = _resolve_model_field_name(obj, k)
+                if not _declares_field(obj, field_name):
+                    continue
                 # Get the existing nested object, defaulting to None if it doesn't exist
                 nested_obj = getattr(obj, field_name, None)
 

{mainsequence-8.1.20.dev4 → mainsequence-8.1.20.dev7}/mainsequence/client/metatables/core.py

@@ -1945,54 +1945,6 @@ class MetaTable(
         raise_for_response(response, payload=error_payload)
         return response.json()
 
-    @classmethod
-    def description_search(
-        cls,
-        q: str,
-        *,
-        q_embedding: Sequence[float] | None = None,
-        trigram_k: int = 200,
-        embed_k: int = 200,
-        w_trgm: float = 0.65,
-        w_emb: float = 0.35,
-        embedding_model: str = "default",
-        **filters,
-    ):
-        q = (q or "").strip()
-        if not q:
-            raise ValueError("q is required")
-        operation = f"{cls.__name__}.description_search"
-        cls._reject_sdk_context_overrides(filters, operation=operation)
-
-        body = {
-            "q": q,
-            "trigram_k": trigram_k,
-            "embed_k": embed_k,
-            "w_trgm": w_trgm,
-            "w_emb": w_emb,
-            "embedding_model": embedding_model,
-        }
-
-        if q_embedding is not None:
-            body["q_embedding"] = [float(x) for x in q_embedding]
-
-        if filters:
-            body.update(filters)
-        body.update(cls._sdk_owned_query_context(operation))
-
-        payload = {"params": serialize_to_json(body)}
-        response = make_request(
-            s=cls.build_session(),
-            loaders=cls.LOADERS,
-            r_type="GET",
-            url=f"{cls.get_object_url().rstrip('/')}/description-search/",
-            payload=payload,
-        )
-        if response.status_code != 200:
-            raise_for_response(response, payload=payload)
-
-        return cls._deserialize_search_response(response.json())
-
     @classmethod
     def column_search(cls, q: str, **filters):
         q = (q or "").strip()
@@ -2768,23 +2720,6 @@ class TimeIndexTableUpdate(TableUpdateNode, BaseObjectOrm):
             return None
         return r.json()
 
-    def verify_if_direct_dependencies_are_updated(self) -> dict:
-        """
-        Response({
-            "error_on_update_dependencies": False,
-            "updated": all_success,
-        })
-        """
-        s = self.build_session()
-        url = (
-            self.get_object_url()
-            + f"/{self._public_uid()}/verify-if-direct-dependencies-are-updated/"
-        )
-        r = make_request(s=s, loaders=None, r_type="GET", url=url)
-        if r.status_code != 200:
-            raise Exception(f"Error in request: {r.text}")
-        return r.json()
-
     def get_data_between_dates_from_api(self, *args, **kwargs):
         return self.output_table.get_data_between_dates_from_api(*args, **kwargs)
 
@@ -3259,6 +3194,54 @@ class TimeIndexMetaTable(MetaTable):
             )
         return self
 
+    @classmethod
+    def description_search(
+        cls,
+        q: str,
+        *,
+        q_embedding: Sequence[float] | None = None,
+        trigram_k: int = 200,
+        embed_k: int = 200,
+        w_trgm: float = 0.65,
+        w_emb: float = 0.35,
+        embedding_model: str = "default",
+        **filters,
+    ):
+        q = (q or "").strip()
+        if not q:
+            raise ValueError("q is required")
+        operation = f"{cls.__name__}.description_search"
+        cls._reject_sdk_context_overrides(filters, operation=operation)
+
+        body = {
+            "q": q,
+            "trigram_k": trigram_k,
+            "embed_k": embed_k,
+            "w_trgm": w_trgm,
+            "w_emb": w_emb,
+            "embedding_model": embedding_model,
+        }
+
+        if q_embedding is not None:
+            body["q_embedding"] = [float(x) for x in q_embedding]
+
+        if filters:
+            body.update(filters)
+        body.update(cls._sdk_owned_query_context(operation))
+
+        payload = {"params": serialize_to_json(body)}
+        response = make_request(
+            s=cls.build_session(),
+            loaders=cls.LOADERS,
+            r_type="GET",
+            url=f"{cls.get_object_url().rstrip('/')}/description-search/",
+            payload=payload,
+        )
+        if response.status_code != 200:
+            raise_for_response(response, payload=payload)
+
+        return cls._deserialize_search_response(response.json())
+
     def _time_indexed_storage_layout(self) -> dict[str, Any]:
         profile = self.time_indexed_profile
         if profile is not None and isinstance(profile.storage_layout, Mapping):

{mainsequence-8.1.20.dev4 → mainsequence-8.1.20.dev7}/mainsequence/client/models_foundry.py

@@ -697,13 +697,6 @@ class GitHubRepositoryBranchList(BasePydanticModel):
     results: list[GitHubRepositoryBranch]
 
 
-class GitHubRepositoryBranchImportResult(BasePydanticModel):
-    repository_uid: str
-    code_repository_branch_uid: str
-    repository_branch: str
-    is_initialized: bool
-
-
 class GitHubRepositoryBinding(BasePydanticModel, BaseObjectOrm):
     FILTERSET_FIELDS: ClassVar[dict[str, list[str]]] = {
         "uid": ["in", "exact"],
@@ -731,28 +724,6 @@ class GitHubRepositoryBinding(BasePydanticModel, BaseObjectOrm):
         raise_for_response(r)
         return GitHubRepositoryBranchList.model_validate(r.json())
 
-    def import_branch(
-        self,
-        *,
-        repository_branch: str,
-        metatables_data_source_uid: str | None = None,
-        timeout=None,
-    ) -> GitHubRepositoryBranchImportResult:
-        body: dict[str, Any] = {"repository_branch": repository_branch}
-        if metatables_data_source_uid is not None:
-            body["metatables_data_source_uid"] = str(metatables_data_source_uid)
-        url = f"{type(self).get_object_url()}/{self._public_detail_reference()}/import-branch/"
-        r = make_request(
-            s=type(self).build_session(),
-            loaders=type(self).LOADERS,
-            r_type="POST",
-            url=url,
-            payload={"json": body},
-            time_out=timeout,
-        )
-        raise_for_response(r)
-        return GitHubRepositoryBranchImportResult.model_validate(r.json())
-
 
 class CodeRepositoryImageSourceProvenance(BasePydanticModel):
     verification_state: Literal["verified", "unverified", "building", "failed"]

{mainsequence-8.1.20.dev4 → mainsequence-8.1.20.dev7/mainsequence.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mainsequence
-Version: 8.1.20.dev4
+Version: 8.1.20.dev7
 Summary: Main Sequence SDK 
 Author-email: Main Sequence GmbH <dev@main-sequence.io>
 License-Expression: LicenseRef-MainSequence-Proprietary

{mainsequence-8.1.20.dev4 → mainsequence-8.1.20.dev7}/mainsequence.egg-info/SOURCES.txt

@@ -133,6 +133,7 @@ tests/test_time_index_table_update_configuration.py
 tests/test_time_index_table_update_configuration_client.py
 tests/test_time_index_table_update_flow.py
 tests/test_time_index_table_update_hard_cut.py
+tests/test_tolerant_response_reading.py
 tests/test_update_runner_uid_runtime.py
 tests/test_update_statistics.py
 tests/test_update_uid_guards.py

{mainsequence-8.1.20.dev4 → mainsequence-8.1.20.dev7}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "mainsequence"
-version = "8.1.20.dev4"
+version = "8.1.20.dev7"
 description = "Main Sequence SDK "
 readme = "README.md"
 requires-python = ">=3.13"

{mainsequence-8.1.20.dev4 → mainsequence-8.1.20.dev7}/tests/test_canonical_api_route_contract.py

@@ -129,11 +129,18 @@ def test_client_auth_provider_routes_match_backend_contract(monkeypatch):
 
 def test_removed_backend_actions_are_not_exposed_by_client_models():
     from mainsequence.client.metatables import DataSource, TimeIndexTableUpdate
+    from mainsequence.client.models_foundry import GitHubRepositoryBinding
     from mainsequence.client.models_helpers import Job
 
     removed_actions = {
         DataSource: ("get_or_create_sqlite", "create_sqlite"),
-        TimeIndexTableUpdate: ("add_tags", "filter_by_hash_id", "get_upstream_nodes"),
+        TimeIndexTableUpdate: (
+            "add_tags",
+            "filter_by_hash_id",
+            "get_upstream_nodes",
+            "verify_if_direct_dependencies_are_updated",
+        ),
+        GitHubRepositoryBinding: ("import_branch",),
         Job: ("bulk_get_or_create", "create_from_configuration", "sync_jobs"),
     }
     violations = [
@@ -146,6 +153,16 @@ def test_removed_backend_actions_are_not_exposed_by_client_models():
     assert violations == []
 
 
+def test_description_search_is_defined_only_where_the_backend_serves_it():
+    """`description-search/` is served for time-index-meta-tables only."""
+    from mainsequence.client.metatables import MetaTable, TimeIndexMetaTable
+
+    assert "description_search" not in vars(MetaTable)
+    assert not hasattr(MetaTable, "description_search")
+    assert "description_search" in vars(TimeIndexMetaTable)
+    assert TimeIndexMetaTable.get_object_url().endswith("/time-index-meta-tables")
+
+
 def test_cli_does_not_reintroduce_legacy_numeric_detail_routes_or_agent_runs():
     from mainsequence.cli import api
     from mainsequence.client import agent_runtime_models

{mainsequence-8.1.20.dev4 → mainsequence-8.1.20.dev7}/tests/test_code_repository_branch_contract.py

@@ -82,11 +82,15 @@ def test_code_repository_is_only_the_logical_aggregate_contract():
     assert not hasattr(code_repository.branches[0], "code_repository_type")
     assert not hasattr(code_repository, "repository_branch")
 
-    with pytest.raises(ValidationError):
+    # Tolerant reading (ADR 0032): the retired field is dropped, not declared.
+    assert "repository_branch" not in models_foundry.CodeRepository.model_fields
+    assert not hasattr(
         models_foundry.CodeRepository(
             **logical_code_repository_payload(),
             repository_branch="main",
-        )
+        ),
+        "repository_branch",
+    )
 
 
 def test_code_repository_requires_canonical_type_and_derived_technology():

{mainsequence-8.1.20.dev4 → mainsequence-8.1.20.dev7}/tests/test_code_repository_runtime_context.py

@@ -722,7 +722,7 @@ def test_metatable_request_always_submits_git_resolved_context(monkeypatch):
     }
 
 
-def test_code_repository_model_rejects_removed_default_data_source_fields():
+def test_code_repository_model_drops_removed_default_data_source_fields():
     payload = {
         "uid": CODE_REPOSITORY_UID,
         "code_repository_name": "Example CodeRepository",
@@ -737,5 +737,9 @@ def test_code_repository_model_rejects_removed_default_data_source_fields():
         "default_metatables_data_source_uid": DATA_SOURCE_UID,
     }
 
-    with pytest.raises(ValueError, match="default_metatables_data_source_uid"):
-        models_foundry.CodeRepository.model_validate(payload)
+    # Tolerant reading (ADR 0032): the retired field is dropped, not declared.
+    assert "default_metatables_data_source_uid" not in models_foundry.CodeRepository.model_fields
+    assert not hasattr(
+        models_foundry.CodeRepository.model_validate(payload),
+        "default_metatables_data_source_uid",
+    )

{mainsequence-8.1.20.dev4 → mainsequence-8.1.20.dev7}/tests/test_environment_response_contracts.py

@@ -87,13 +87,13 @@ RUNTIME_UPDATE_FIELD = "runtime_update"
         ),
     ],
 )
-def test_backend_environment_response_contract_is_declared_by_strict_sdk_model(
+def test_backend_environment_response_contract_is_declared_by_sdk_model(
     serializer_names,
     model,
     expected_fields,
 ):
     """Keep SDK response models aligned with ADR-0036/ADR-0037 projections."""
-    assert model.model_config.get("extra") == "forbid", serializer_names
+    assert model.model_config.get("extra") == "ignore", serializer_names
     assert expected_fields <= set(model.model_fields), serializer_names
 
 
@@ -106,13 +106,13 @@ def test_backend_environment_response_contract_is_declared_by_strict_sdk_model(
         helper_models.ResourceRelease,
     ],
 )
-def test_owner_observability_code_repositoryion_is_declared_by_strict_sdk_model(model):
-    assert model.model_config.get("extra") == "forbid"
+def test_owner_observability_code_repositoryion_is_declared_by_sdk_model(model):
+    assert model.model_config.get("extra") == "ignore"
     assert "observability" in model.model_fields
 
 
 def test_agent_session_runtime_capabilities_code_repositoryion_is_declared_and_typed():
-    assert agent_models.AgentSession.model_config.get("extra") == "forbid"
+    assert agent_models.AgentSession.model_config.get("extra") == "ignore"
     assert (
         agent_models.AgentSession.model_fields["runtime_capabilities"].annotation == dict[str, str]
     )

{mainsequence-8.1.20.dev4 → mainsequence-8.1.20.dev7}/tests/test_filter_normalization.py

@@ -67,7 +67,7 @@ def test_agent_runtime_update_parses_every_backend_state(
     ) == remediation_tool
 
 
-def test_agent_runtime_update_contract_is_complete_and_closed():
+def test_agent_runtime_update_contract_is_complete_and_read_tolerantly():
     payload = _agent_runtime_update_contract("unknown")
     for field_name in ("state", "needs_redeploy", "remediation"):
         incomplete = dict(payload)
@@ -75,8 +75,12 @@ def test_agent_runtime_update_contract_is_complete_and_closed():
         with pytest.raises(ValidationError, match=field_name):
             agent_models_mod.AgentRuntimeUpdate.model_validate(incomplete)
 
-    with pytest.raises(ValidationError, match="extra_forbidden"):
-        agent_models_mod.AgentRuntimeUpdate.model_validate(payload | {"future_state": "value"})
+    # Tolerant reading (ADR 0032): a field this release does not declare is dropped.
+    tolerated = agent_models_mod.AgentRuntimeUpdate.model_validate(
+        payload | {"future_state": "value"}
+    )
+    assert tolerated.state == payload["state"]
+    assert not hasattr(tolerated, "future_state")
 
     invalid_remediation = _agent_runtime_update_contract("update_required")
     invalid_remediation["remediation"] = {"tool": "agent.update_session"}
@@ -1178,7 +1182,7 @@ def test_get_by_uid_normalizes_read_query_params(monkeypatch):
     }
 
 
-def test_patch_by_uid_raises_with_context_for_unmapped_response_fields(monkeypatch):
+def test_patch_by_uid_ignores_unmapped_response_fields(monkeypatch):
     captured = {}
 
     class FakeResponse:
@@ -1201,14 +1205,15 @@ def test_patch_by_uid_raises_with_context_for_unmapped_response_fields(monkeypat
 
     monkeypatch.setattr(base_mod, "make_request", _fake_make_request)
 
-    with pytest.raises(ValueError) as exc_info:
-        DemoPatchModel.patch_by_uid("uid-9", _into=DemoPatchModel(id=9), label="patched")
+    instance = DemoPatchModel(id=9)
+    patched = DemoPatchModel.patch_by_uid("uid-9", _into=instance, label="patched")
 
-    assert str(exc_info.value) == (
-        "Failed to apply PATCH response to DemoPatchModel at field 'schema'. "
-        "Response fragment: {'schema': {'name': 'customers'}}. "
-        'Original error: "DemoPatchModel" object has no field "schema"'
-    )
+    # Tolerant reading (ADR 0032): a response key the model does not declare is
+    # dropped, exactly as parsing drops it.
+    assert patched is instance
+    assert patched.id == 9
+    assert "schema" not in DemoPatchModel.model_fields
+    assert "schema" not in patched.__dict__
     assert captured == {
         "r_type": "PATCH",
         "url": "https://backend.test/demo-patch/uid-9/",
@@ -1449,7 +1454,7 @@ def test_shareable_can_edit_parses_permission_state(monkeypatch):
     }
 
 
-def test_shareable_access_state_rejects_removed_internal_identity():
+def test_shareable_access_state_drops_removed_internal_identity():
     removed_field = "_".join(("object", "id"))
     payload = {
         "object_uid": "33333333-3333-4333-8333-333333333333",
@@ -1460,8 +1465,11 @@ def test_shareable_access_state_rejects_removed_internal_identity():
         removed_field: 17,
     }
 
-    with pytest.raises(ValidationError, match=removed_field):
-        models_user_mod.ShareableAccessState.model_validate(payload)
+    state = models_user_mod.ShareableAccessState.model_validate(payload)
+
+    # Tolerant reading (ADR 0032): the retired field is dropped, not declared.
+    assert removed_field not in models_user_mod.ShareableAccessState.model_fields
+    assert not hasattr(state, removed_field)
 
 
 @pytest.mark.parametrize(
@@ -1974,13 +1982,22 @@ def test_agent_client_contract_matches_backend_agent_serializer():
         {**payload, "agent_card": None},
         {**payload, "agent_card": {}},
         {**payload, "name": "Unrelated label"},
-        {**payload, "agent_type": "custom"},
-        {**payload, "metadata": {}},
-        {**payload, "has_agent_service": False},
     ):
         with pytest.raises(ValidationError):
             agent_models_mod.Agent.model_validate(invalid_payload)
 
+    # Tolerant reading (ADR 0032): retired keys are dropped, never declared.
+    for retired_field, retired_value in (
+        ("agent_type", "custom"),
+        ("metadata", {}),
+        ("has_agent_service", False),
+    ):
+        assert retired_field not in agent_models_mod.Agent.model_fields
+        assert not hasattr(
+            agent_models_mod.Agent.model_validate({**payload, retired_field: retired_value}),
+            retired_field,
+        )
+
     with pytest.raises(agent_models_mod.ApiError, match="harness_agent workflow"):
         agent_models_mod.Agent.create(name="Research Copilot")
     with pytest.raises(ValueError, match="Unsupported Agent filter"):
@@ -2206,10 +2223,13 @@ def test_agent_get_parses_runtime_update_projection(monkeypatch):
         is None
     )
 
-    with pytest.raises(ValidationError, match="unexpected_projection"):
+    # Tolerant reading (ADR 0032): an undeclared projection is dropped.
+    assert not hasattr(
         agent_models_mod.Agent.model_validate(
-            {**FakeResponse.json(), "unexpected_projection": "still forbidden"}
-        )
+            {**FakeResponse.json(), "unexpected_projection": "ignored"}
+        ),
+        "unexpected_projection",
+    )
 
 
 def test_agent_semantic_search_sends_environment_scope_and_parses_code_repositoryion(monkeypatch):
@@ -2411,10 +2431,13 @@ def test_agent_session_list_and_detail_parse_runtime_capabilities(monkeypatch):
     assert detailed.organization_environment_name == "production"
     assert detailed.catalog_digest == "sha256:" + ("b" * 64)
 
-    with pytest.raises(ValidationError, match="unexpected_projection"):
+    # Tolerant reading (ADR 0032): an undeclared projection is dropped.
+    assert not hasattr(
         agent_models_mod.AgentSession.model_validate(
-            {**session_payload, "unexpected_projection": "still forbidden"}
-        )
+            {**session_payload, "unexpected_projection": "ignored"}
+        ),
+        "unexpected_projection",
+    )
 
 
 def test_agent_session_environment_and_catalog_projection_contract_is_strict():
@@ -3525,7 +3548,7 @@ def test_deployment_run_billing_parses_every_backend_pricing_state(pricing_state
     assert billing.components.image_build == billing.total_cost
 
 
-def test_deployment_run_billing_contract_is_complete_and_closed():
+def test_deployment_run_billing_contract_is_complete_and_read_tolerantly():
     payload = _deployment_run_billing_payload()
     for field_name in (
         "scope",
@@ -3542,15 +3565,20 @@ def test_deployment_run_billing_contract_is_complete_and_closed():
         with pytest.raises(ValidationError, match=field_name):
             models_helpers_mod.DeploymentRunBilling.model_validate(incomplete)
 
-    with pytest.raises(ValidationError, match="extra_forbidden"):
+    # Tolerant reading (ADR 0032): a billing field or component this release does
+    # not declare is dropped instead of failing the whole response.
+    assert not hasattr(
         models_helpers_mod.DeploymentRunBilling.model_validate(
             payload | {"future_billing_field": "value"}
-        )
-
-    with pytest.raises(ValidationError, match="extra_forbidden"):
+        ),
+        "future_billing_field",
+    )
+    assert not hasattr(
         models_helpers_mod.DeploymentRunBilling.model_validate(
             payload | {"components": payload["components"] | {"future_component": "0.000000"}}
-        )
+        ).components,
+        "future_component",
+    )
 
     with pytest.raises(ValidationError, match="greater_than_equal"):
         models_helpers_mod.DeploymentRunBilling.model_validate(payload | {"unpriced_rows": -1})
@@ -3618,7 +3646,7 @@ def test_deployment_run_cost_summary_parses_nullable_decimal(total_cost):
         ),
     ],
 )
-def test_deployment_run_additive_billing_contracts_are_complete_and_closed(
+def test_deployment_run_additive_billing_contracts_are_complete_and_read_tolerantly(
     model,
     payload,
     required_fields,
@@ -3629,8 +3657,11 @@ def test_deployment_run_additive_billing_contracts_are_complete_and_closed(
         with pytest.raises(ValidationError, match=field_name):
             model.model_validate(incomplete)
 
-    with pytest.raises(ValidationError, match="extra_forbidden"):
-        model.model_validate(payload | {"future_billing_field": "value"})
+    # Tolerant reading (ADR 0032): an undeclared billing field is dropped.
+    assert not hasattr(
+        model.model_validate(payload | {"future_billing_field": "value"}),
+        "future_billing_field",
+    )
 
 
 @pytest.mark.parametrize("field_name", ["priced_rows", "unpriced_rows"])

{mainsequence-8.1.20.dev4 → mainsequence-8.1.20.dev7}/tests/test_observability_client.py

@@ -320,11 +320,12 @@ def test_agent_logs_support_optional_session_filter_without_environment_override
     )
 
 
-def test_owner_observability_models_remain_strict_except_enriched_rows():
-    assert observability_models.ObservabilityLinks.model_config["extra"] == "forbid"
-    assert observability_models.OwnerLogPage.model_config["extra"] == "forbid"
-    assert observability_models.EnvironmentLogSearchPage.model_config["extra"] == "forbid"
-    assert observability_models.ResourceUsagePage.model_config["extra"] == "forbid"
+def test_owner_observability_models_read_tolerantly_and_keep_enriched_rows():
+    # Tolerant reading (ADR 0032): pages drop undeclared fields, rows keep them.
+    assert observability_models.ObservabilityLinks.model_config["extra"] == "ignore"
+    assert observability_models.OwnerLogPage.model_config["extra"] == "ignore"
+    assert observability_models.EnvironmentLogSearchPage.model_config["extra"] == "ignore"
+    assert observability_models.ResourceUsagePage.model_config["extra"] == "ignore"
     assert observability_models.OwnerLogRow.model_config["extra"] == "allow"
     assert observability_models.EnvironmentLogSearchRow.model_config["extra"] == "allow"
 

{mainsequence-8.1.20.dev4 → mainsequence-8.1.20.dev7}/tests/test_source_table_configuration.py

@@ -1,7 +1,6 @@
 import datetime
 
 import pytest
-from pydantic import ValidationError
 
 from mainsequence.client import metatables as models_metatables
 
@@ -60,16 +59,20 @@ def test_time_indexed_profile_parses_canonical_response():
     assert config.earliest_index_value == datetime.datetime(2026, 5, 1, tzinfo=datetime.UTC)
 
 
-def test_time_indexed_profile_rejects_removed_legacy_profile_uid():
+def test_time_indexed_profile_drops_removed_legacy_profile_uid():
     payload = _source_config_payload()
     removed_field = "_".join(("dynamic", "table", "uid"))
-    payload[removed_field] = payload.pop("time_index_meta_table_uid")
+    payload[removed_field] = "legacy-uid-44"
 
-    with pytest.raises(ValidationError, match=removed_field):
-        models_metatables.TimeIndexedProfile(**payload)
+    profile = models_metatables.TimeIndexedProfile(**payload)
 
+    # Tolerant reading (ADR 0032): the legacy name is dropped, never honoured.
+    assert removed_field not in models_metatables.TimeIndexedProfile.model_fields
+    assert not hasattr(profile, removed_field)
+    assert profile.time_index_meta_table_uid == "storage-uid-44"
 
-def test_time_indexed_profile_rejects_foreign_keys():
+
+def test_time_indexed_profile_drops_foreign_keys():
     payload = _source_config_payload()
     payload["foreign_keys"] = [
         {
@@ -81,8 +84,11 @@ def test_time_indexed_profile_rejects_foreign_keys():
         }
     ]
 
-    with pytest.raises(ValidationError, match="foreign_keys"):
-        models_metatables.TimeIndexedProfile(**payload)
+    profile = models_metatables.TimeIndexedProfile(**payload)
+
+    # Tolerant reading (ADR 0032): foreign keys stay Alembic-owned and undeclared.
+    assert "foreign_keys" not in models_metatables.TimeIndexedProfile.model_fields
+    assert not hasattr(profile, "foreign_keys")
 
 
 def test_time_index_meta_table_contract_requires_registered_profile_index_names():

{mainsequence-8.1.20.dev4 → mainsequence-8.1.20.dev7}/tests/test_time_index_table_update_configuration_client.py

@@ -354,7 +354,7 @@ def test_output_table_accepts_namespace():
         "table_name",
     ],
 )
-def test_output_table_rejects_removed_backend_fields(removed_field):
+def test_output_table_drops_removed_backend_fields(removed_field):
     payload = {
         "uid": "time-index-table-storage-1",
         "storage_hash": "hash",
@@ -370,8 +370,11 @@ def test_output_table_rejects_removed_backend_fields(removed_field):
         removed_field: "removed",
     }
 
-    with pytest.raises(ValidationError):
-        TimeIndexMetaTable(**payload)
+    table = TimeIndexMetaTable(**payload)
+
+    # Tolerant reading (ADR 0032): a retired field is dropped, not declared.
+    assert removed_field not in TimeIndexMetaTable.model_fields
+    assert not hasattr(table, removed_field)
 
 
 def test_update_manager_requires_output_table_constructor_argument():