Skip to content

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

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

chore: regenerate API client from latest OpenAPI spec#94
giordano-lucas merged 2 commits into
mainfrom
chore/regenerate-api-client

Conversation

@giordano-lucas

@giordano-lucas giordano-lucas commented Sep 3, 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-09-04 05:14 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 Sep 3, 2026

Copy link
Copy Markdown

Greptile Summary

This PR regenerates the Go API client and session-start flags from the latest OpenAPI specification.

  • Replaces legacy storage endpoints and models with session-scoped file APIs.
  • Removes the file-storage field and flag from session creation.
  • Adds session-file models, pagination, upload/download/delete operations, and the persona internal email field.

Confidence Score: 3/5

This PR is not safe to merge until the hand-written session-start and file-command consumers are migrated to the regenerated API contract.

The regenerated types remove a field and several legacy operations that unchanged production code still references, preventing the CLI command package from compiling.

Files Needing Attention: internal/api/client.gen.go, internal/cmd/sessionstart_flags.gen.go, internal/cmd/sessionstart_optout.go, internal/cmd/files.go

Important Files Changed

Filename Overview
internal/api/client.gen.go Regenerates the API contract around session files but removes symbols still required by hand-written commands, causing compile failures.
internal/cmd/sessionstart_flags.gen.go Removes the use-file-storage flag while the hand-written opt-out implementation and existing CLI contract still depend on it.

Fix all with Greploop Fix All in Claude Code

Prompt To Fix All With AI
### Issue 1
internal/api/client.gen.go:721-724
**Session storage field breaks build**

When the command package is built, `sessionstart_optout.go` still assigns `ApiSessionStartRequest.UseFileStorage`, but this regeneration removes that field, causing the CLI build to fail. It also removes the still-documented `--use-file-storage` flag.

### Issue 2
internal/api/client.gen.go:9689-9699
**Removed file APIs break commands**

When the CLI is built, `internal/cmd/files.go` still references the removed `FileListUploads`, `FileListDownloads`, `FileUpload`, and `FileDownload` types and methods, causing undefined-symbol compilation failures. The command callers need to be migrated with the new session-scoped file API.

### Issue 3
internal/api/client.gen.go:9689-9699
**Session file changes lack tests**

This PR changes generated request fields, CLI flags, and file endpoint names without adding unit or integration coverage. Add regression coverage for the new session-file contract so incompatibilities with hand-written command code are caught.

---

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

Comment on lines 721 to 724
// SolveCaptchas Whether to try to automatically solve captchas
SolveCaptchas *bool `json:"solve_captchas,omitempty"`

// UseFileStorage Whether FileStorage should be attached to the session.
UseFileStorage *bool `json:"use_file_storage,omitempty"`

// UserAgent The user agent to use for the session

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Session storage field breaks build

When the command package is built, sessionstart_optout.go still assigns ApiSessionStartRequest.UseFileStorage, but this regeneration removes that field, causing the CLI build to fail. It also removes the still-documented --use-file-storage flag.

Knowledge Base Used:

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

Comment:
**Session storage field breaks build**

When the command package is built, `sessionstart_optout.go` still assigns `ApiSessionStartRequest.UseFileStorage`, but this regeneration removes that field, causing the CLI build to fail. It also removes the still-documented `--use-file-storage` flag.

**Knowledge Base Used:**
- [Browser automation workflows](https://app.greptile.com/nottelabs/-/custom-context/knowledge-base/nottelabs/notte-cli/-/docs/browser-automation.md)
- [Session lifecycle and execution](https://app.greptile.com/nottelabs/-/custom-context/knowledge-base/nottelabs/notte-cli/-/docs/session-lifecycle.md)
- [Command bootstrap and shared behavior](https://app.greptile.com/nottelabs/-/custom-context/knowledge-base/nottelabs/notte-cli/-/docs/command-bootstrap.md)

---

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

Fix in Claude Code

Comment thread internal/api/client.gen.go
@giordano-lucas giordano-lucas changed the title chore: regenerate API client from latest OpenAPI spec feat(personas): expose alias and internal email Sep 3, 2026
@giordano-lucas
giordano-lucas force-pushed the chore/regenerate-api-client branch from 03f31da to fe8fbf8 Compare September 3, 2026 14:21
@giordano-lucas giordano-lucas changed the title feat(personas): expose alias and internal email chore: regenerate API client from latest OpenAPI spec Sep 3, 2026

@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 fe8fbf8 to 5f5fc04 Compare September 4, 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 merged commit abda492 into main Sep 4, 2026
3 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