Commit graph

4 commits

Author SHA1 Message Date
Snider
107c78dc07 test(services): road-test resource CRUD paths
All checks were successful
Security Scan / security (pull_request) Successful in 8s
Test / test (pull_request) Successful in 49s
Co-Authored-By: Virgil <virgil@lethean.io>
2026-03-22 14:13:40 +00:00
Snider
c7c279fc17 fix: Resource.List/ListAll/Iter use collection path, not item path
All checks were successful
Security Scan / security (push) Successful in 8s
Test / test (push) Successful in 44s
Security Scan / security (pull_request) Successful in 8s
Test / test (pull_request) Successful in 46s
The Resource path like /repos/{owner}/{repo}/issues/{index} is the
item path. List operations need the collection path (without the
trailing /{placeholder}). Fixed by deriving the collection path at
construction time — only strips the last segment if it's a pure
placeholder like /{index}, not mixed segments like /repos.

Co-Authored-By: Virgil <virgil@lethean.io>
2026-03-22 14:01:14 +00:00
Snider
57d8af13ad feat: modernise to Go 1.26 iterators and stdlib helpers
All checks were successful
Security Scan / security (push) Successful in 14s
Test / test (push) Successful in 2m1s
Add ListIter in pagination + generic Resource.Iter for streaming
paginated results as iter.Seq2[T, error]. Add Iter* methods across
all service files (actions, admin, branches, issues, labels, notifs,
orgs, packages, pulls, releases, repos, teams, users, webhooks).
Modernise cmd/forgegen with slices.Sort, maps.Keys, strings.FieldsFuncSeq.

Co-Authored-By: Gemini <noreply@google.com>
Co-Authored-By: Virgil <virgil@lethean.io>
2026-02-23 05:39:13 +00:00
Snider
286d91383f feat: generic Resource[T,C,U] for CRUD operations
Co-Authored-By: Virgil <virgil@lethean.io>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-21 15:27:01 +00:00