refactor(zarr-metadata): validators type an unread document as what it is

[d-v-b]

🤖 AI text below 🤖

Five validators cast a document to Mapping[str, object] before any key was checked, then cast it again to Mapping[object, object] to check the keys. It is Mapping[object, object] until its keys are read, and _missing_keys, _check_literal and _validate_dim_sequence now take it so, which removes 4 casts. No behaviour changes, so there is no changelog entry.

This is one of eight independent PRs from a review of the casts in zarr_metadata.model, each based on main and each passing just check alone, with a test that fails without its fix where there is behaviour to test:

• fix(zarr-metadata): bytes are not a JSON array #4423 bytes are not a JSON array
• fix(zarr-metadata): a v2 array's other members are JSON with string keys #4424 a v2 array's other members are JSON with string keys
• refactor(zarr-metadata): validators type an unread document as what it is #4425 validators type an unread document as what it is
• fix(zarr-metadata): a model shares nothing with the document it read #4426 a model shares nothing with the document it read
• refactor(zarr-metadata): model casts that only restated a type are gone #4427 model casts that only restated a type are gone
• fix(zarr-metadata): the store reader checks that it read bytes #4428 the store reader checks that it read bytes
• fix(zarr-metadata)!: guards narrow only when they say yes #4429 guards narrow only when they say yes
• fix(zarr-metadata): the v2 group document types are closed #4430 the v2 group document types are closed

They and #4422 all change model/_validation.py, as #4420 and #4421 did before they merged, so each one merged after the first needs a rebase. Each is rebased onto both, so its tests are type-checked as #4421 now requires. All eight together, merged with #4421's type-checked tests, pass just check.

🤖 Generated with Claude Code

[read-the-docs-community]

Documentation build overview

📚 zarr-metadata | 🛠️ Build #34777264 | 📁 Comparing 726860b against latest (d21a5b4)

  🔍 Preview build  

2 files changed

± api/model/index.html
± api/pydantic/index.html