fix(s1-rtc): consolidate every orbit group, not just the last-ingested one

[lhoupert]

Why

S1 RTC cubes end up with only one orbit consolidated on disk — staging shows 30TWM asc✓/desc✗, 32TNN & 30UVU asc✗/desc✓.

Root cause (traced + reproduced): each acquisition is ingested by a separate pod that strips all consolidated metadata (so time can resize), ingests its single orbit, then calls consolidate_s1_store(store, orbit_direction) — which only re-consolidated the one orbit passed in. So whichever orbit is ingested last is the only one left with on-disk consolidated metadata.

What

consolidate_s1_store now consolidates every orbit group present (root.groups()), then the root. Signature unchanged — orbit_direction is kept for logging / caller compatibility (no changes to the CLI or the data-pipeline callers).

root = zarr.open_group(str(store_path), mode="r", zarr_format=3)
for orbit_name, _ in root.groups():
    zarr.consolidate_metadata(str(store_path), path=orbit_name, zarr_format=3)
zarr.consolidate_metadata(str(store_path), zarr_format=3)

Safety: the pipeline's minimal append-fetch already pulls all zarr.json (ingest_v1_s1_rtc.py:336-337); only bulk chunks are skipped. So both orbits' metadata is local when this runs — the same reason the root consolidation already produces a correct both-orbit view.

Impact — cosmetic, not a correctness/deploy blocker

The root consolidated metadata is already complete for both orbits, and a consolidated root synthesizes a child orbit's view — so titiler (opens at root, selects via /orbit:vv) renders the unconsolidated orbit fine (verified: 30UVU ascending = HTTP 200 despite asc✗). The fix matters for clients that open a single orbit group standalone (e.g. following the STAC <cube>.zarr/<orbit> asset hrefs) with consolidated=True, which otherwise fall back to a listing. Existing cubes self-heal on their next re-ingest.

Tests (TDD, RED→GREEN)

tests/test_s1_rtc_ingest.py::TestConsolidation::test_consolidate_all_orbits_present — ingests both orbits, calls consolidate_s1_store(store, "descending") (one orbit passed), asserts both orbit groups are consolidated standalone.

⚠️ The assertion opens each orbit group standalone — asserting via root["ascending"].metadata.consolidated_metadata is a false-green (a consolidated root synthesizes the child view, so it's non-None even on the buggy code; verified empirically).

RED on ac3167f, GREEN after the fix. Full data-model suite green; ruff/mypy add zero new findings vs baseline.

Stacks on #202; targets #180.

🤖 Generated with Claude Code