fabro/lib/components/fabro-github
Scott Werner 3ab584691d Drop the branch-head compatibility wrapper
`branch_head_sha` existed to keep callers compiling while the branch-head
lookup moved onto the repository reader. Its last caller now opens a
reader directly, so the wrapper only made the typed API worse: it joined
an already-validated owner and repo into a slug so the reader could split
them apart again, invented an "invalid repository coordinate" error for a
value validated upstream, opened a fresh credential session per call, and
flattened `RepositoryReadError` into `anyhow` while keeping one variant —
leaving callers unable to tell a rate limit from a rejected token.

Its integration test pinned the wrapper rather than the behavior. Replace
it with one that asserts the same 404-means-not-observable semantics
through `resolve_commit`, which is where that contract actually lives.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
2026-08-19 15:43:08 -04:00
..
src Drop the branch-head compatibility wrapper 2026-08-19 15:43:08 -04:00
tests Drop the branch-head compatibility wrapper 2026-08-19 15:43:08 -04:00
Cargo.toml Reduce duplication in GitHub repository reads 2026-08-19 15:15:43 -04:00