Skip to content

fix: allow null in verified yields enums - #24

Merged
ReynardoEW merged 1 commit into
mainfrom
alpaca/fix-nullable-enums-1789715406
Sep 18, 2026
Merged

ReynardoEW merged 1 commit into
mainfrom
alpaca/fix-nullable-enums-1789715406

Conversation

@alpacai

@alpacai alpacai Bot commented Sep 18, 2026

Copy link
Copy Markdown

Summary

Fix the verified-yields OpenAPI enums that rejected legitimate null values during production validation. This follows the schema mismatch reproduced in the investigation record and does not change API response behavior.

Changes

  • defillama-openapi-pro.json: include null in every nullable enum under the verified earn list, detail, and history schemas.

Verification

  • jq empty defillama-openapi-pro.json -> JSON parsed successfully.
  • uv run --with jsonschema .tmp_validate_verified.py -> list, detail, full-history, and 90-day-history responses returned HTTP 200 with 0 schema errors.
  • Nullable-enum audit for /yields/v2/earn/verified* -> 0 nullable enums remain without a null enum member.
  • git diff --check -> passed.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying api-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 03245bb
Status: ✅  Deploy successful!
Preview URL: https://b7214e64.api-docs-59g.pages.dev
Branch Preview URL: https://alpaca-fix-nullable-enums-17.api-docs-59g.pages.dev

View logs

@ReynardoEW
ReynardoEW marked this pull request as ready for review September 18, 2026 07:31
@ReynardoEW
ReynardoEW merged commit f1d43fc into main Sep 18, 2026
1 check passed
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