Skip to content

Conversation

@seborama
Copy link
Contributor

@seborama seborama commented Jun 5, 2025

This allows support for openapi specs that use circular references (such as forms-api's Fields property).

redocly does not have the ability to control circular references when bundling the spec in JSON format.

api-docs's CI using a transformation to resolve this problem.

However, repos that reference the published components, that are generated by api-docs, will still be faced with the circular reference (e.g. https://typeform.design/api-docs/schema/Field.yaml).

api-docs's CI is solely responsible for generating the "official" openapi.json spec bundle. It downloads all the openapi.yaml files from the other API repos and bundles them with the common models defined in api-docs.

The validate-openapi step of the ci-standard-checks action exists as an early detection of potential spec bundling issues that api-docs's CI could face. It is solely a check. While this PR changes the output format to YAML will not affect the CI executed by api-docs, not will it change its JSON format.

this allows support for openapi specs that use circular references (such
as forms-api's Fields property)
@gitstream-cm
Copy link

gitstream-cm bot commented Jun 5, 2025

🥷 Code experts: no user matched threshold 10

See details

scripts/validate-openapi/run.sh

Knowledge based on git-blame:

To learn more about /:\ gitStream - Visit our Docs

@seborama seborama changed the title feat(openapi-yaml-bundle): switch redocly bundle to yaml format feat(TU-26042): switch redocly bundle to yaml format Jun 5, 2025
@seborama seborama requested a review from rdumont June 5, 2025 09:40
@seborama seborama merged commit 2c75b37 into main Jun 5, 2025
10 checks passed
@seborama seborama deleted the feat/openapi-yaml-bundle branch June 5, 2025 10:21
@typeform-ops-gha
Copy link

🎉 This PR is included in version 1.31.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants