claw bf02733507 fix(github): consolidate review.go and identity.go into reviews.go (#116) ...

Remove github/review.go and github/identity.go, replacing them with a
consolidated github/reviews.go that:

- Uses doJSONRequest for PostReview and DismissReview (cleaner than
  manual marshal + doRequestWithBody)
- Adds paginated ListReviews with per_page=100 and max 100 pages
- Consolidates GetAuthenticatedUser and userResponse type (previously
  duplicated in identity.go)
- Preserves all sentinel errors (ErrCannotDeleteSubmittedReview,
  ErrConflictingCommitIDs), state translation, commit ID validation,
  and SupersedeReviews

This prevents the redeclaration errors that occur when both review.go
and reviews.go exist in the same package, as described in issue #116.

Closes #116

2026-05-13 15:35:55 -07:00

..

client_test.go

fix(github): remove double blank line in client_test.go (gofmt)

2026-05-13 08:48:09 -07:00

client.go

style: remove stray blank line in doRequestWithBody

2026-05-13 08:48:09 -07:00

conformance_test.go

fix: address review feedback - eliminate type assertion via ReviewSuperseder interface

2026-05-13 09:03:42 -07:00

files_test.go

fix(test): SetRetryBackoff with correct slice length

2026-05-12 22:47:31 -07:00

files.go

feat(github): implement FileReader interface (#80)

2026-05-13 05:33:30 +00:00

helpers_test.go

fix(github): validate conflicting commit IDs and extract test helper

2026-05-13 13:18:46 +00:00

identity_test.go

feat(github): implement Reviewer and Identity interfaces (#81)

2026-05-13 13:18:46 +00:00

pr_test.go

fix: address review feedback on PR #102

2026-05-13 04:47:15 +00:00

pr.go

fix: address review feedback on PRReader implementation

2026-05-12 22:17:32 -07:00

review_test.go

fix(github): add DismissReview Event comment; use t.Fatalf for routing assertions

2026-05-13 13:18:46 +00:00

reviews.go

fix(github): consolidate review.go and identity.go into reviews.go (#116)

2026-05-13 15:35:55 -07:00