brawer / osmdiffs / 43372364083

94%

push

github

fix(pipeline): rebuild later steps' outputs when their inputs are newer (#943)

On a kept workdir with a newer OSM planet, several memoized steps
reused their outputs just because they existed, so a run kept serving
results built from the old planet:

- import_osm's index build kept osm-features.index even after the
  assembly (and its string pool) had been built again, pairing a stale
  index with new tag ids;
- extract_conflated_layers and suggest_edits kept their jsonl layers
  after conflated.parquet was rebuilt;
- render_tiles and join_tiles kept their pmtiles after their layers
  were rebuilt.

They now reuse their outputs only when none is older than its inputs,
like the other steps already do, via a shared `is_current` helper.

An integration test runs the pipeline on the fixtures, checks that an
unchanged rerun builds nothing again, and that a rerun with a newer
planet rebuilds every output, from osm-features.index and
alltheplaces.parquet through to the pmtiles.

Fixes #935


Claude-Session: https://claude.ai/code/session_01Ttfb2MU7QCuGDjefU8WWe5

Co-authored-by: Claude <noreply@anthropic.com>

57 of 57 new or added lines in 7 files covered. (100.0%)

19340 of 20472 relevant lines covered (94.47%)

3486.91 hits per line