mainsequence 4.2.37__tar.gz → 4.2.38__tar.gz

{mainsequence-4.2.37/mainsequence.egg-info → mainsequence-4.2.38}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mainsequence
-Version: 4.2.37
+Version: 4.2.38
 Summary: Main Sequence SDK 
 Author-email: Main Sequence GmbH <dev@main-sequence.io>
 License: MainSequence GmbH SDK License Agreement

{mainsequence-4.2.37 → mainsequence-4.2.38}/agent_scaffold/skills/data_publishing/meta_tables/SKILL.md

@@ -292,6 +292,12 @@ SQLAlchemy class and calling normal registration again. Shape-addressed
 foreign keys, or constraints change, so new code cannot reliably recover the
 previous shape-derived table.
 
+Do not modify Alembic revision files that have already been implemented/applied.
+MetaTable migrations are database-backed history: once a revision may exist in a
+database `alembic_version` table, changing that file corrupts the relationship
+between source history and deployed state. For any follow-up schema change,
+create a new Alembic revision on top of the current head.
+
 For contract evolution, define or update one selected
 `AlembicMetaTableMigration` provider:
 

{mainsequence-4.2.37 → mainsequence-4.2.38}/mainsequence/meta_tables/migrations.py

@@ -534,9 +534,15 @@ class AlembicMetaTableMigration:
                 table_name = target_table_names[model]
                 existing_meta_table = existing_by_table_name.get(table_name)
                 if existing_meta_table is not None:
-                    _bind_model_to_existing_metatable(model, existing_meta_table)
-                    reserved_by_model[model] = existing_meta_table
-                    continue
+                    if _meta_table_provisioning_status(existing_meta_table) == "active":
+                        _bind_model_to_existing_metatable(model, existing_meta_table)
+                        reserved_by_model[model] = existing_meta_table
+                        continue
+                    if on_metatable_reservation_status is not None:
+                        on_metatable_reservation_status(
+                            "Restaging existing reserved MetaTable "
+                            f"table_name={table_name} with current provider contract."
+                        )
                 else:
                     bound_meta_table = _bound_meta_table_for_model(model)
                     if bound_meta_table is not None:
@@ -1055,6 +1061,13 @@ def _meta_table_physical_table_name(meta_table: MetaTable) -> str | None:
     return str(physical_table_name)
 
 
+def _meta_table_provisioning_status(meta_table: MetaTable) -> str | None:
+    provisioning_status = meta_table.provisioning_status
+    if provisioning_status in (None, ""):
+        return None
+    return str(provisioning_status)
+
+
 def _meta_table_uid(meta_table: MetaTable) -> str | None:
     uid = meta_table.uid
     if uid in (None, ""):
@@ -1066,7 +1079,6 @@ def _finalize_table_failed(item: ManagedMetaTableFinalizeTableResult) -> bool:
     return (
         item.provisioning_status != "active"
         or item.physical_table_exists is False
-        or item.finalized is False
         or item.error not in (None, "", {})
     )
 

{mainsequence-4.2.37 → mainsequence-4.2.38/mainsequence.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mainsequence
-Version: 4.2.37
+Version: 4.2.38
 Summary: Main Sequence SDK 
 Author-email: Main Sequence GmbH <dev@main-sequence.io>
 License: MainSequence GmbH SDK License Agreement

{mainsequence-4.2.37 → mainsequence-4.2.38}/pyproject.toml

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

{mainsequence-4.2.37 → mainsequence-4.2.38}/tests/test_meta_table_migrations.py

@@ -1069,6 +1069,7 @@ def test_prepare_for_alembic_reserves_existing_table_name_with_provider_identity
     def fake_bulk_create(rows, *, timeout=None, on_status=None):
         reserved_payloads.extend(rows)
         assert [table["identifier"] for table in rows] == [
+            "example_assets__account",
             "example_assets__asset",
         ]
         assert all(table["migration_package"] == "sample" for table in rows)
@@ -1079,11 +1080,17 @@ def test_prepare_for_alembic_reserves_existing_table_name_with_provider_identity
         assert not hasattr(rows[0]["table_contract"], "foreign_keys")
         assert all("schema_management" not in table for table in rows)
         return [
+            _reserved_metatable(
+                uid="aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
+                identifier="example_assets__account",
+                physical_table_name="example_assets__account",
+                storage_hash=rows[0]["storage_hash"],
+            ),
             _reserved_metatable(
                 uid="bbbbbbbb-bbbb-4bbb-8bbb-bbbbbbbbbbbb",
                 identifier="example_assets__asset",
                 physical_table_name="example_assets__asset",
-                storage_hash=rows[0]["storage_hash"],
+                storage_hash=rows[1]["storage_hash"],
             ),
         ]
 
@@ -1106,6 +1113,7 @@ def test_prepare_for_alembic_reserves_existing_table_name_with_provider_identity
 
     assert len(filter_calls) == 1
     assert [payload["identifier"] for payload in reserved_payloads] == [
+        "example_assets__account",
         "example_assets__asset",
     ]
     assert "schema_management" not in reserved_payloads[0]
@@ -1115,6 +1123,10 @@ def test_prepare_for_alembic_reserves_existing_table_name_with_provider_identity
     ]
     assert Account.__table__.name == "example_assets__account"
     assert Asset.__table__.name == "example_assets__asset"
+    assert (
+        "Restaging existing reserved MetaTable table_name=example_assets__account "
+        "with current provider contract."
+    ) in reservation_statuses
 
 
 def test_prepare_for_alembic_reserves_already_staged_existing_rows(monkeypatch):