chore: regenerate API client from latest OpenAPI spec - #97
Conversation
andreakiro
left a comment
There was a problem hiding this comment.
Auto-approved: scheduled API client regeneration.
|
| Filename | Overview |
|---|---|
| internal/api/client.gen.go | Adds a generated proxy-credential endpoint whose transport integration is consistent with existing operations, but which lacks focused request and response tests. |
Prompt To Fix All With AI
### Issue 1
internal/api/client.gen.go:14965
**Missing endpoint tests**
This PR adds a generated credential-minting endpoint without unit or integration coverage. That violates the repository directive requiring tests to be called out when none are added, so the requirement must be satisfied before merging. Add focused tests for the POST path and JSON body as well as the 200 and 422 response mappings, so future schema or generator regressions cannot silently break this API surface.
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.Reviews (1): Last reviewed commit: "chore: regenerate API client from latest..." | Re-trigger Greptile
63eb00d to
207e54a
Compare
andreakiro
left a comment
There was a problem hiding this comment.
Auto-approved: scheduled API client regeneration.
207e54a to
f3346de
Compare
andreakiro
left a comment
There was a problem hiding this comment.
Auto-approved: scheduled API client regeneration.
Regenerate the generated API client (
internal/api,internal/cmd/*_flags.gen.go) from the latest staging OpenAPI spec.The new
POST /proxies/gateway/credentialsoperation is intentionally recorded as infrastructure-only: function runners call it to mint short-lived proxy-gateway credentials, so it is not exposed as a CLI command.Verification:
make checkmake check-endpointsmake buildgo test -race -short ./...