mostlyright-data 0.28.0__tar.gz → 0.28.1__tar.gz

{mostlyright_data-0.28.0 → mostlyright_data-0.28.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mostlyright-data
-Version: 0.28.0
+Version: 0.28.1
 Summary: Mostly Right hosted CLI for reviewed datasets
 Project-URL: Homepage, https://mostlyright.md/
 Project-URL: Documentation, https://mostlyright.md/docs/guides/cli/

{mostlyright_data-0.28.0 → mostlyright_data-0.28.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "mostlyright-data"
-version = "0.28.0"
+version = "0.28.1"
 description = "Mostly Right hosted CLI for reviewed datasets"
 readme = "README.md"
 requires-python = ">=3.11"

{mostlyright_data-0.28.0 → mostlyright_data-0.28.1}/src/mostlyright/data_harness/table_manifest.py

@@ -56,6 +56,7 @@ __all__ = [
     "COMPRESSION",
     "DROPPED_ALL",
     "MANIFEST_MERGE_KINDS",
+    "MANIFEST_PARTITION_ENCODINGS",
     "MAX_MANIFEST_BYTES",
     "MAX_PARTS_PER_RUN",
     "MAX_PARTS_PER_VERSION",
@@ -94,6 +95,15 @@ TABLE_MANIFEST_SCHEMA = "mostlyright-table-manifest.v1"
 TABLE_MANIFEST_SCHEMA_V2 = "mostlyright-table-manifest.v2"
 _SUPPORTED_MANIFEST_SCHEMAS = frozenset((TABLE_MANIFEST_SCHEMA, TABLE_MANIFEST_SCHEMA_V2))
 
+#: The closed spellings a v2 manifest may bind a partition field to: exactly the ones Studio's
+#: table sink writes and Keel's seal binds, and exactly the recipe's ``table.partition.encoding``
+#: vocabulary. A reader narrower than the writers refuses a table the service sealed, which is how
+#: ``iso_month_prefix`` tables read as TABLE_MANIFEST_INVALID on 0.28.0 (planning#807).
+#: ``tests/test_thin_table_manifest_encodings.py`` holds this equal to the vendored recipe schema.
+MANIFEST_PARTITION_ENCODINGS: frozenset[str] = frozenset(
+    {"value", "iso_date_prefix", "iso_date_value", "iso_month_prefix"}
+)
+
 #: The two media types the format adds to the closed artifact vocabulary.
 TABLE_MANIFEST_MEDIA_TYPE = "application/vnd.mostlyright.table-manifest+json"
 TABLE_PART_MEDIA_TYPE = "application/vnd.apache.parquet"
@@ -744,8 +754,10 @@ class TableManifest:
                 raise _refuse(
                     "TABLE_MANIFEST_INVALID", "a v2 manifest binds every partition encoding"
                 )
+            # ``isinstance`` first: a JSON array or object is unhashable, and a set lookup on one
+            # would escape as a TypeError rather than as this refusal.
             if any(
-                value not in {"value", "iso_date_prefix", "iso_date_value"}
+                not isinstance(value, str) or value not in MANIFEST_PARTITION_ENCODINGS
                 for value in self.partition_encodings.values()
             ):
                 raise _refuse("TABLE_MANIFEST_INVALID", "a v2 partition encoding is not closed")