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.68%. Comparing base (df18415) to head (7039c56).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4471      +/-   ##
==========================================
- Coverage   94.69%   94.68%   -0.01%     
==========================================
  Files          94       94              
  Lines       13597    13595       -2     
==========================================
- Hits        12875    12873       -2     
  Misses        722      722              

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

... and 4 files with indirect coverage changes

🚀 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.