Skip non-Zarr objects silently when listing group members

[mayuriphad]

Fixes #4161.

Problem

While LocalStore writes a key, it first writes to a temporary file named <name>.<uuid>.partial and then renames it into place. A reader that lists the store while another process is writing sees these files, and Group.members() emitted a ZarrUserWarning ("Object at ... is not recognized as a component of a Zarr hierarchy") for each one. Any other object in the store that is not a Zarr node triggered the same warning.

Change

As discussed in review, the store listings are unchanged and keep showing everything on disk. Instead, listing the members of a group now skips objects that are not Zarr nodes without a warning.

• _iter_members no longer warns in the KeyError branch. It skips the object.
• test_group_members now checks that listing members does not warn about extra objects. The expected warning about consolidated metadata with Zarr format 3 is kept.
• test_use_consolidated_for_children_members no longer expects the warning.
• Changelog entry: changes/4161.bugfix.md.

Testing

pytest tests/test_group.py tests/test_metadata/test_consolidated.py: 1010 passed, 209 skipped. The CI checks on this PR pass.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[d-v-b]

i think we want the store to show these keys. we should remove the warning from member listing instead.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.71%. Comparing base (bd0dc84) to head (9703b96).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4471      +/-   ##
==========================================
- Coverage   94.71%   94.71%   -0.01%     
==========================================
  Files          94       94              
  Lines       13663    13662       -1     
==========================================
- Hits        12941    12940       -1     
  Misses        722      722              

Files with missing lines	Coverage Δ
src/zarr/core/group.py	95.67% <100.00%> (-0.01%)	⬇️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[mayuriphad]

Thanks @d-v-b, makes sense. I've reworked this: \LocalStore\ listings are back to reporting everything on disk (including in-progress .partial\ files), and group member iteration now skips objects that aren't Zarr nodes silently instead of emitting a \ZarrUserWarning. Updated \ est_group_members\ accordingly and the changelog entry.

[d-v-b]

Can you update the pr title and description

[mayuriphad]

Thanks @d-v-b. I updated the title and the description to match the new approach.

[mayuriphad]

Updated as suggested: store listings are unchanged, the warning is removed from member listing instead, and the title and description now reflect that. Also merged main into the branch.