feat(generated): Agents (batch 13ebc8c1) - #437
Conversation
Greptile SummaryThis generated SDK update makes blueprint session settings optional and adds agent-token validation support.
Confidence Score: 4/5The public parameter reorder should be fixed before merging because it causes existing positional blueprint-creation calls to fail at runtime. Making sessionSettings optional moves it from the second argument to the fifth, so previously valid positional calls bind the settings object to the string description parameter and throw a TypeError. Files Needing Attention: lib/Service/Agents.php and lib/Resource/AgentBlueprintsCreateRequest.php Important Files Changed
|
| public function createBlueprint( | ||
| string $name, | ||
| \WorkOS\Resource\AgentBlueprintsCreateRequestSessionSetting $sessionSettings, | ||
| ?string $description = null, | ||
| ?array $permissions = null, | ||
| ?\WorkOS\Resource\AgentBlueprintsCreateRequestInvocableBy $invocableBy = null, | ||
| ?\WorkOS\Resource\AgentBlueprintsCreateRequestSessionSetting $sessionSettings = null, | ||
| ?\WorkOS\RequestOptions $options = null, |
There was a problem hiding this comment.
Session settings change position
When existing consumers call createBlueprint($name, $sessionSettings) positionally, the settings object now binds to the ?string $description parameter, causing a TypeError. Making this parameter optional should preserve its existing positional slot; the corresponding resource constructor has the same compatibility break.
Prompt To Fix With AI
This is a comment left during a code review.
Path: lib/Service/Agents.php
Line: 70-76
Comment:
**Session settings change position**
When existing consumers call `createBlueprint($name, $sessionSettings)` positionally, the settings object now binds to the `?string $description` parameter, causing a `TypeError`. Making this parameter optional should preserve its existing positional slot; the corresponding resource constructor has the same compatibility break.
---
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