Skip to content

feat: enable forward-compatible enums - #13

Open
atonks2 wants to merge 1 commit into
mainfrom
forward-compatible-enums-main
Open

feat: enable forward-compatible enums#13
atonks2 wants to merge 1 commit into
mainfrom
forward-compatible-enums-main

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>

@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 in the Java SDK generator; PR title is missing the required component field per Moov Conventional Commits convention.

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 Moov's Conventional Commits convention, feat titles must include a component in parentheses. Correct this to: feat(java): enable forward-compatible enums

Suggested change
forwardCompatibleEnumsByDefault: true
feat(java): enable forward-compatible enums

(95% confidence — The rule is clear; the only uncertainty is the exact canonical component name used in this repo.)

@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 for Java in .speakeasy/gen.yaml by enabling forwardCompatibleEnumsByDefault (setting it to true). There are no review comments, and I have no feedback to provide.

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