Skip to content

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

Merged
giordano-lucas merged 2 commits into
mainfrom
chore/regenerate-api-client
Oct 22, 2026
Merged

giordano-lucas merged 2 commits into
mainfrom
chore/regenerate-api-client

Conversation

@giordano-lucas

@giordano-lucas giordano-lucas commented Oct 20, 2026 •

Copy link
Copy Markdown
Member

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

Last regenerated: 2026-10-22 10:37 UTC

@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 Oct 20, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 4/5

[Medium impact] The changes affect the generated API client and session-start CLI workflow.

No concrete behavior failure was established, but the repository’s testing requirement should be satisfied before merging.

Fix All in Claude CodeFindings

  1. P2 JSON changes lack tests ▶
Fix with agent prompt
### Issue 1
internal/api/client.gen.go:2795
This PR changes the JSON the client sends but adds no unit or integration tests, triggering the repository directive to flag missing tests. Add tests in `internal/api/client_gen_test.go` for omitted and populated `FunctionRunUpdateRequest.Result` values and string `Model` values on the three changed scrape requests. The repository’s testing requirement must be satisfied before merging.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Summary

Regenerates the API client from the latest API definition.

  • Scraping requests use plain model names and updated image fields.
  • Function run updates can leave out a result when none is set.
  • Screenshot help and response fields reflect the refreshed API description.

Reviews (1) · Last reviewed commit: "chore: regenerate API client from latest..." · Reviewed by Greptile

Comment thread internal/api/client.gen.go Outdated

// Result The result of the workflow run
Result interface{} `json:"result"`
Result interface{} `json:"result,omitempty"`

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 JSON changes lack tests

This PR changes the JSON the client sends but adds no unit or integration tests, triggering the repository directive to flag missing tests. Add tests in internal/api/client_gen_test.go for omitted and populated FunctionRunUpdateRequest.Result values and string Model values on the three changed scrape requests. The repository’s testing requirement must be satisfied before merging.

Context Used: # Test guidelines - Add a comment if the PR does not add any unit/integration tests - At least one integration should be added on every bug fix PR and major feature update in backend - If the repo contains a mobile app, any new product feature from w... (source)

Prompt To Fix With AI
This is a comment left during a code review.
Path: internal/api/client.gen.go
Line: 2795

Comment:
**JSON changes lack tests**

This PR changes the JSON the client sends but adds no unit or integration tests, triggering the repository directive to flag missing tests. Add tests in `internal/api/client_gen_test.go` for omitted and populated `FunctionRunUpdateRequest.Result` values and string `Model` values on the three changed scrape requests. The repository’s testing requirement must be satisfied before merging.

**Context Used:** # Test guidelines - Add a comment if the PR does not add any unit/integration tests - At least one integration should be added on every bug fix PR and major feature update in backend - If the repo contains a mobile app, any new product feature from w... ([source](https://app.greptile.com/review/custom-context?memory=instruction-0))

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Fix in Claude Code

@giordano-lucas
giordano-lucas force-pushed the chore/regenerate-api-client branch from ea262e5 to 6a3c0cd Compare October 20, 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.

@giordano-lucas
giordano-lucas force-pushed the chore/regenerate-api-client branch from 6a3c0cd to be24a87 Compare October 20, 2026 14:02

@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 be24a87 to da9034c Compare October 21, 2026 06:57

@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 da9034c to dde24b1 Compare October 21, 2026 10:38

@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 dde24b1 to 42784fc Compare October 21, 2026 14:03

@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 42784fc to 70ebbd8 Compare October 22, 2026 06:56

@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 70ebbd8 to 1522338 Compare October 22, 2026 10:37

@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.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@giordano-lucas
giordano-lucas merged commit 292dc90 into main Oct 22, 2026
7 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.

2 participants