Skip to content

feat(generated): Agents (batch 89943d4b) - #610

Closed
workos-sdk-automation[bot] wants to merge 2 commits into
mainfrom
oagen/batch-89943d4b
Closed

feat(generated): Agents (batch 89943d4b)#610
workos-sdk-automation[bot] wants to merge 2 commits into
mainfrom
oagen/batch-89943d4b

Conversation

@workos-sdk-automation

Copy link
Copy Markdown
Contributor

Summary

Regenerated SDK from spec changes.

Triggered by workos/openapi-spec@b6350ef

@workos-sdk-automation workos-sdk-automation Bot added the autogenerated Autogenerated code or content label Sep 1, 2026
@workos-sdk-automation
workos-sdk-automation Bot requested review from a team as code owners September 1, 2026 21:39
@workos-sdk-automation workos-sdk-automation Bot added the autogenerated Autogenerated code or content label Sep 1, 2026
@greptile-apps

greptile-apps Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR regenerates the Agents SDK surface to add blueprint-token validation and make blueprint session settings optional.

  • Adds ValidateBlueprintToken and its request/response models.
  • Updates generated manifests, fixtures, and synchronization metadata.
  • Adds endpoint coverage, though the test does not verify the required token payload.

Confidence Score: 4/5

The PR appears safe to merge, with a non-blocking test gap around verification of the required token request payload.

The generated endpoint and models are internally consistent, but the new test would not detect regressions in agent_access_token serialization.

Files Needing Attention: agents_test.go

Important Files Changed

Filename Overview
agents.go Adds the blueprint-token validation method and makes session settings optional; no blocking implementation defect was established.
models.go Adds the token-validation response model using field representations consistent with nearby generated models.
agents_test.go Covers the new endpoint path and response decoding but does not exercise or assert serialization of the required access token.
.oagen-manifest.json Registers the new endpoint and generated fixtures consistently.
testdata/agent_token_validation.json Supplies a representative successful token-validation response fixture.
Prompt To Fix All With AI
### Issue 1
agents_test.go:188
**Required token payload untested**

The test passes an empty parameter object and never asserts the decoded request body, so a regression that omits, renames, or incorrectly tags the required `agent_access_token` field will continue to pass while callers send malformed validation requests.

---

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

Reviews (1): Last reviewed commit: "chore(generated): add release notes frag..." | Re-trigger Greptile

Comment thread agents_test.go

client := workos.NewClient("sk_test", workos.WithBaseURL(server.URL))
result, err := client.Agents().ValidateBlueprintToken(context.Background(), "test_agent_blueprint_id", &workos.AgentsValidateBlueprintTokenParams{})
require.NoError(t, err)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P2 Required token payload untested

The test passes an empty parameter object and never asserts the decoded request body, so a regression that omits, renames, or incorrectly tags the required agent_access_token field will continue to pass while callers send malformed validation requests.

Prompt To Fix With AI
This is a comment left during a code review.
Path: agents_test.go
Line: 188

Comment:
**Required token payload untested**

The test passes an empty parameter object and never asserts the decoded request body, so a regression that omits, renames, or incorrectly tags the required `agent_access_token` field will continue to pass while callers send malformed validation requests.

---

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

@gjtorikian gjtorikian closed this Sep 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autogenerated Autogenerated code or content

Development

Successfully merging this pull request may close these issues.

1 participant