Skip to content

docs(tests): retarget stale ADR-0012 ref to ADR-0019#116

Merged
variablefate merged 1 commit into
mainfrom
fix/stale-adr-0012-ref
May 11, 2026
Merged

docs(tests): retarget stale ADR-0012 ref to ADR-0019#116
variablefate merged 1 commit into
mainfrom
fix/stale-adr-0012-ref

Conversation

@variablefate
Copy link
Copy Markdown
Owner

One-line cleanup miss from PR #113.

Context

While PR #113 (Universal Links) was in flight, PR #112 renumbered the roadflared scheme ADR from 00120019 (commit `7c08ac8` — `docs(adr): retarget Swift "See ADR-0012" refs to ADR-0019`). The squash merge of #113 onto main correctly auto-resolved existing `ADR-0012` references in HandleIncomingURLTests.swift and AppState.swift via 3-way merge — but couldn't catch a brand-new line added on the #113 branch:

RoadFlare/RoadFlareTests/AppState/HandleIncomingURLTests.swift:196 says `per ADR-0012 + issue #63` — should be `per ADR-0019`.

This isn't just a stale number — `decisions/0012-coordinator-ride-identity-cache.md` is now an unrelated ADR after the renumbering, so the reference points to the wrong subject.

Diff

```diff

One file, one line changed.

Verification

`HandleIncomingURLTests` target: 12 tests, 0 failures (comment-only change, no logic touched).

🤖 Generated with Claude Code

Cleanup miss from PR #113 (`ce0d263`): the parity-regression test added
in that PR referenced `ADR-0012` for the roadflared/Universal-Links
routing pattern, but the ADR was renumbered to `ADR-0019` in PR #112
(`7c08ac8`) while #113 was in flight. The squash merge auto-resolved
existing `ADR-0012` references in this file via 3-way merge, but
couldn't catch a brand-new line added on the branch.

`decisions/0012-coordinator-ride-identity-cache.md` is now an unrelated
ADR, so the stale ref doesn't just point at the wrong number — it
points at the wrong subject.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@variablefate variablefate marked this pull request as ready for review May 11, 2026 21:56
@variablefate variablefate merged commit 07aff33 into main May 11, 2026
@variablefate variablefate deleted the fix/stale-adr-0012-ref branch May 11, 2026 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant