fix(zarr-metadata): a field whose configuration is not an object is still claimed by its name

`resolve` read a field whose configuration was not an object as claimed
by nothing, although its name names a definition, so the codec order was
judged without it: `[{"name": "gzip", "configuration": 5}, "bytes"]` drew
no problem for the gzip before the bytes codec. The field keeps the
definition its name names, unread.

Assisted-by: ClaudeCode:claude-opus-5-5

Author: d-v-b

packages/zarr-metadata/src/zarr_metadata/v3/_definition.py

@@ -985,9 +985,13 @@ def _read(
     data: JSONValue, kind: type[Definition[Any]], context: Context, loc: Loc
 ) -> tuple[Resolved[Definition[Any]], Problems]:
     name, given, malformed = named_configuration(data)
-    if name is None or len(malformed) != 0:
+    if name is None:
         return Resolved(data, "invalid", None, None), ()
     definition = context.claimant(kind, name)
+    if len(malformed) != 0:
+        # A configuration that is not an object, which the envelope's
+        # problems say; the name still says what claims the field.
+        return Resolved(data, "invalid", definition, None), ()
     if definition is None:
         return Resolved(data, "out_of_scope", None, None), ()
     _, carried = spelled(kind, name)

packages/zarr-metadata/tests/v3/test_definitions.py

@@ -394,8 +394,9 @@ def test_error_a_required_configuration_is_missing() -> None:
 
 
 def test_error_the_configuration_is_not_an_object() -> None:
+    # Unread, and still claimed by the definition its name names.
     resolved, found = resolve({"name": "gzip", "configuration": 5}, CodecDefinition, SCOPE)
-    assert resolved.resolution == "invalid"
+    assert (resolved.resolution, resolved.definition) == ("invalid", GZIP_CODEC)
     assert [found.loc for found in found] == [("configuration",)]
 
 

packages/zarr-metadata/tests/v3/test_pipelines.py

@@ -206,6 +206,14 @@ def test_error_a_codec_out_of_order(codecs: list[JSONValue], index: int) -> None
     assert _problems(codecs) == [((index,), "invalid_value")]
 
 
+def test_error_a_codec_out_of_order_whose_configuration_is_not_an_object() -> None:
+    # Its name still says what it is.
+    assert _problems([{"name": "gzip", "configuration": 5}, "bytes"]) == [
+        ((0, "configuration"), "invalid_type"),
+        ((1,), "invalid_value"),
+    ]
+
+
 def test_error_a_second_array_to_bytes_codec() -> None:
     assert _problems(["bytes", "bytes"]) == [((1,), "invalid_value")]