Skip to content

feat: enable forward-compatible enums - #22

Open
atonks2 wants to merge 1 commit into
versions/2026.10from
forward-compatible-enums-2026.10
Open

feat: enable forward-compatible enums#22
atonks2 wants to merge 1 commit into
versions/2026.10from
forward-compatible-enums-2026.10

Conversation

@atonks2

@atonks2 atonks2 commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Description

Sets forwardCompatibleEnumsByDefault: true in .speakeasy/gen.yaml so this SDK treats unrecognized enum values in API responses as open/unknown instead of raising a validation error. Today, when the API starts returning a new enum value, older pinned SDKs throw an exception on the response with no way for the caller to know whether their request actually succeeded.

Per Speakeasy's SDK resilience docs.

Config-only change — the next scheduled/triggered SDK generation run will regenerate against this setting.

Sets forwardCompatibleEnumsByDefault: true so the generated SDK treats
unrecognized enum values as open/unknown instead of raising a validation
error, per Speakeasy's SDK resilience guidance.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the Speakeasy generator configuration in .speakeasy/gen.yaml to enable forward-compatible enums by default for Java generation. There are no review comments, and I have no additional feedback to provide.

@moov-review-bot moov-review-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Severity Count
🚫 Blocking 0
⚠️ Important 1
💡 Suggestion 0

Config-only PR enabling forward-compatible enums; the PR title is missing a required component per Moov Conventional Commits standards.

Trigger another review: /moov-review

Comment thread .speakeasy/gen.yaml
explicitDocImports: false
flattenGlobalSecurity: true
forwardCompatibleEnumsByDefault: false
forwardCompatibleEnumsByDefault: true

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ PR title missing required component for feat type

Per Conventional Commits convention, feat requires a component segment. Correct the title to feat(sdk): enable forward-compatible enums or feat(java): enable forward-compatible enums.

(95% confidence — Component choice (sdk vs java) could be confirmed by checking how other SDK-config PRs are titled in this repo.)

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