brawer / osmdiffs / 43011732062

94%

Build Type

push

github

Committed by web-flow

Commit Message

fix(edits): Compare tag values as the matchers do (#925)

The edit suggester proposed an edit whenever the ATP and OSM strings
differed, so it asked mappers to rewrite `+1-812-414-2137` as
`+1 812-414-2137` or `http://example.ch` as `https://www.example.ch/`.
Now phone, fax, email and website values are compared as
`AtpContacts` compares them, and `opening_hours` without regard to
spaces, as `match.evidence.agreeing` does. Unlike `agreeing`, this
compares one key at a time, so a phone that agrees can't hide a fax
that doesn't.

On the 2026-11-02 planet file, `agreeing` says about 212,000 of the
2,155,000 suggested tag edits differ only in notation, among them
92,094 of 125,729 phone differences.

Part of #924.


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

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

Coverage Stats

75 of 77 new or added lines in 2 files covered. (97.4%)

1 existing line in 1 file now uncovered.

16788 of 17920 relevant lines covered (93.68%)

3677.79 hits per line