Skip to content

chore: drop the Error.errCode enum - #209

Open
anonvt wants to merge 1 commit into
mainfrom
chore/drop-errcode-enum
Open

anonvt wants to merge 1 commit into
mainfrom
chore/drop-errcode-enum

Conversation

@anonvt

@anonvt anonvt commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Removes the closed enum from Error.errCode, keeping it a free string with examples. Stacked on #208. Ticket: INTE-1541.

Why

Sentry flagged on #208 that new error codes were missing from the enum. Checking the list showed it has drifted well past that: around 44 codes auction-engine returns today are not listed, and about 19 listed codes no longer appear in auction-engine. Pato's call is that the enum will not be maintained, so a closed list only puts strict generated clients at risk of failing to parse a 400 whenever a new code appears. The message field already says clients must not dispatch on its value; errCode stays the stable identifier, just not as a closed set.

Validation

prettier --check, yamllint and vacuum lint (0 errors) pass. redocly lint --extends recommended-strict reports the same 4 pre-existing TravelAuction* oneOf errors as main.

🤖 Generated with Claude Code

@anonvt
anonvt requested a review from a team as a code owner October 13, 2026 18:55
@anonvt
anonvt force-pushed the chore/drop-errcode-enum branch from caa6a72 to feadcca Compare October 13, 2026 19:24
@anonvt
anonvt force-pushed the feat/video-events branch from 75ce850 to 730c81b Compare October 13, 2026 21:57
@anonvt
anonvt force-pushed the chore/drop-errcode-enum branch from feadcca to 459bd5d Compare October 13, 2026 21:57
@anonvt
anonvt changed the base branch from feat/video-events to main October 13, 2026 23:23
The enum had drifted from auction-engine (dozens of returned codes
unlisted) and will not be kept in sync. Keep errCode as a free string
with examples so generated clients never fail to parse a 400.

INTE-1541

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@anonvt
anonvt force-pushed the chore/drop-errcode-enum branch from 459bd5d to 38f9097 Compare October 13, 2026 23:24
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.

1 participant