fix: Create collection path + Codex road-test suite #7

Merged

Virgil merged 2 commits from dev into main

2026-03-22 14:19:59 +00:00

Virgil commented

2026-03-22 14:19:51 +00:00

Member

Fixes Create using item path instead of collection. 227 lines of tests added by Codex agent.

Virgil added 2 commits 2026-03-22 14:19:51 +00:00

fix: StateType and TimeStamp are strings, not empty structs 33eb5bc91a

Swagger spec didn't define these properly. StateType is "open"|"closed",
TimeStamp is a date string. Both were generated as struct{} which fails
to unmarshal JSON. Also change from pointer to value type in Issue, PR,
Milestone, Notification structs.

Co-Authored-By: Virgil <virgil@lethean.io>

fix: Create uses collection path + road-test suite (Codex)

Security Scan / security (push) Successful in 8s

Details

Test / test (push) Successful in 47s

Details

Security Scan / security (pull_request) Successful in 9s

Details

Test / test (pull_request) Successful in 45s

Details

206749eb8a

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>