Commit graph

9 commits

Author SHA1 Message Date
Virgil
edbf3f7088 fix(forge): make stringers nil-safe
Some checks failed
Security Scan / security (push) Successful in 15s
Test / test (push) Has been cancelled
Co-Authored-By: Virgil <virgil@lethean.io>
2026-04-02 07:54:28 +00:00
Virgil
7e8340a3d4 feat(forge): add safe resource stringers
Some checks failed
Security Scan / security (push) Successful in 12s
Test / test (push) Has been cancelled
Co-Authored-By: Virgil <virgil@lethean.io>
2026-04-02 07:52:00 +00:00
Virgil
8473c5676f docs(forge): add AX usage examples to core API
Some checks failed
Security Scan / security (push) Successful in 14s
Test / test (push) Has been cancelled
Co-Authored-By: Virgil <virgil@lethean.io>
2026-04-02 07:19:20 +00:00
Virgil
551a964fdb refactor(ax): enforce v0.8.0 polish rules
All checks were successful
Security Scan / security (push) Successful in 9s
Test / test (push) Successful in 1m36s
Co-Authored-By: Virgil <virgil@lethean.io>
2026-03-29 23:39:24 +00:00
Claude
1ffb4bee5a
feat: upgrade to core v0.8.0-alpha.1, replace banned stdlib imports
All checks were successful
Security Scan / security (push) Successful in 8s
Test / test (push) Successful in 1m20s
Replace fmt, errors, strings, path/filepath, encoding/json with Core
primitives across 23 files. Keep encoding/json for streaming
NewDecoder/NewEncoder, strings for Fields/FieldsFuncSeq.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-26 13:27:06 +00:00
Snider
206749eb8a fix: Create uses collection path + road-test suite (Codex)
All checks were successful
Security Scan / security (push) Successful in 8s
Test / test (push) Successful in 47s
Security Scan / security (pull_request) Successful in 9s
Test / test (pull_request) Successful in 45s
Bug: Resource.Create was POSTing to item path (/issues/{index})
instead of collection path (/issues). Same class as the List fix.

Tests: path validation on all service methods, Update tests for
issues/repos, CreateComment test, ListComments test, PR merge
error case (conflict handling).

227 lines of test coverage added by Codex agent.

Co-Authored-By: Virgil <virgil@lethean.io>
2026-03-22 14:19:28 +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