Skip to content

chore: regenerate API client from latest OpenAPI spec - #97

Merged
leo-notte merged 2 commits into
mainfrom
chore/regenerate-api-client
Sep 9, 2026
Merged

chore: regenerate API client from latest OpenAPI spec#97
leo-notte merged 2 commits into
mainfrom
chore/regenerate-api-client

Conversation

@giordano-lucas

@giordano-lucas giordano-lucas commented Sep 8, 2026

Copy link
Copy Markdown
Member

Regenerate the generated API client (internal/api, internal/cmd/*_flags.gen.go) from the latest staging OpenAPI spec.

The new POST /proxies/gateway/credentials operation 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 check
  • make check-endpoints
  • make build
  • go test -race -short ./...

@andreakiro andreakiro left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approved: scheduled API client regeneration.

@greptile-apps

greptile-apps Bot commented Sep 8, 2026

Copy link
Copy Markdown

Greptile Summary

This PR regenerates the API client to expose proxy credential minting through typed request and response models.

  • Adds optional city and country request fields.
  • Adds a POST request builder for /proxies/gateway/credentials.
  • Adds typed handling for successful and validation-error responses.
  • Does not add automated coverage for the new API operation.

Confidence Score: 4/5

The generated endpoint appears internally consistent, but the repository’s explicit test requirement must be satisfied before merging.

The endpoint uses the existing authenticated transport and follows established generated-client patterns; the sole accepted concern is the absence of automated coverage for its request construction and response parsing.

Files Needing Attention: internal/api/client.gen.go

Important Files Changed

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.

Fix all with Greploop Fix All in Claude Code

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

@giordano-lucas
giordano-lucas force-pushed the chore/regenerate-api-client branch from 63eb00d to 207e54a Compare September 9, 2026 05:14

@andreakiro andreakiro left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approved: scheduled API client regeneration.

@giordano-lucas
giordano-lucas force-pushed the chore/regenerate-api-client branch from 207e54a to f3346de Compare September 9, 2026 10:36

@andreakiro andreakiro left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approved: scheduled API client regeneration.

@leo-notte
leo-notte merged commit d3af4c3 into main Sep 9, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants