Missing top-level required OpenAPI alternatives

[nolannbiron]

No description provided.

[linear]

RND-6827 Using allof merge schema doesn't reflect required properties correctly

[changeset-bot]

🦋 Changeset detected

Latest commit: 82bef3e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages

Name	Type
@gitbook/react-openapi	Patch
gitbook	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[github-actions]

Summary of the deployments:

Version 1

Version	URL	Status
Latest commit	https://a0a88329.gitbook-open.pages.dev	✅
PR	https://pr3207.gitbook-open.pages.dev	✅

Version 2

Version	URL	Status
Vercel	https://gitbook-v2-d0sobbklj-gitbook.vercel.app	✅
Cloudflare	https://c1a39b70-gitbook-open-v2-preview.gitbook.workers.dev	✅

Test content

Site	v1	2v	2c
GitBook	https://a0a88329.gitbook-open.pages.dev/docs.gitbook.com	https://gitbook-v2-d0sobbklj-gitbook.vercel.app/url/docs.gitbook.com	https://c1a39b70-gitbook-open-v2-preview.gitbook.workers.dev/url/docs.gitbook.com
E2E	https://a0a88329.gitbook-open.pages.dev/gitbook.gitbook.io/test-gitbook-open	https://gitbook-v2-d0sobbklj-gitbook.vercel.app/url/gitbook.gitbook.io/test-gitbook-open	https://c1a39b70-gitbook-open-v2-preview.gitbook.workers.dev/url/gitbook.gitbook.io/test-gitbook-open

[argos-ci]

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build	Status	Details	Updated (UTC)
customers-v1 (Inspect)	👍 Changes approved	2 changed	May 2, 2025, 9:42 AM
customers-v2 (Inspect)	👍 Changes approved	79 changed	May 2, 2025, 9:44 AM
default (Inspect)	👍 Changes approved	10 changed	May 2, 2025, 9:46 AM
v2-vercel (Inspect)	👍 Changes approved	6 changed	May 2, 2025, 9:45 AM

[gregberge]

I don't think it's good. A required parent does not mean children are required. Also we have ancestors to know that.

[nolannbiron]

@gregberge You can require children in parent like this :

xxSchema:
      required:
        - name
        - type
      allOf:
        - $ref: '#/components/schemas/...'
        - ...

I'll try to get this from ancestors 👍🏻

[gregberge]

Hmm ok so it's valid. Yeah let's use the latest ancestor and we are good to go.