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

2 commits

Author SHA1 Message Date
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
33eb5bc91a fix: StateType and TimeStamp are strings, not empty structs
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>
2026-03-22 14:07:10 +00:00