Repository navigation
chore: regenerate API client from latest OpenAPI spec - #154
Conversation
andreakiro
left a comment
There was a problem hiding this comment.
Auto-approved: scheduled API client regeneration.
[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.
|
|
|
||
| // Result The result of the workflow run | ||
| Result interface{} `json:"result"` | ||
| Result interface{} `json:"result,omitempty"` |
There was a problem hiding this comment.
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!
ea262e5 to
6a3c0cd
Compare
andreakiro
left a comment
There was a problem hiding this comment.
Auto-approved: scheduled API client regeneration.
6a3c0cd to
be24a87
Compare
andreakiro
left a comment
There was a problem hiding this comment.
Auto-approved: scheduled API client regeneration.
be24a87 to
da9034c
Compare
andreakiro
left a comment
There was a problem hiding this comment.
Auto-approved: scheduled API client regeneration.
da9034c to
dde24b1
Compare
andreakiro
left a comment
There was a problem hiding this comment.
Auto-approved: scheduled API client regeneration.
dde24b1 to
42784fc
Compare
andreakiro
left a comment
There was a problem hiding this comment.
Auto-approved: scheduled API client regeneration.
42784fc to
70ebbd8
Compare
andreakiro
left a comment
There was a problem hiding this comment.
Auto-approved: scheduled API client regeneration.
70ebbd8 to
1522338
Compare
andreakiro
left a comment
There was a problem hiding this comment.
Auto-approved: scheduled API client regeneration.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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