feat(generated): Agents (batch 13ebc8c1) - #611
Conversation
Greptile SummaryThe PR regenerates the Agents SDK surface to make blueprint session settings optional and add agent-token validation.
Confidence Score: 4/5The PR appears safe to merge, with a non-blocking test gap around serialization of the required agent access token. The generated method and response model follow existing SDK conventions, but the new test uses an empty request and would not catch malformed or missing token serialization. Files Needing Attention: agents_test.go Important Files Changed
|
| defer server.Close() | ||
|
|
||
| client := workos.NewClient("sk_test", workos.WithBaseURL(server.URL)) | ||
| result, err := client.Agents().ValidateBlueprintToken(context.Background(), "test_agent_blueprint_id", &workos.AgentsValidateBlueprintTokenParams{}) |
There was a problem hiding this comment.
Required token remains untested
The test sends an empty AgentsValidateBlueprintTokenParams and never asserts the decoded request body, so it continues passing when agent_access_token is empty or serialized under the wrong name, leaving real API rejections uncovered.
Prompt To Fix With AI
This is a comment left during a code review.
Path: agents_test.go
Line: 187
Comment:
**Required token remains untested**
The test sends an empty `AgentsValidateBlueprintTokenParams` and never asserts the decoded request body, so it continues passing when `agent_access_token` is empty or serialized under the wrong name, leaving real API rejections uncovered.
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.
Summary
Regenerated SDK from spec changes.
Triggered by workos/openapi-spec@b6350ef