Skip to content

feat: add virtualization for schemas section for OpenAPI 3.1.0 - #11022

Open
glowcloud wants to merge 7 commits into
feat/OSS-1719--schemas-virtualizationfrom
feat/OSS-1720--oas31-schemas-virtualization
Open

feat: add virtualization for schemas section for OpenAPI 3.1.0#11022
glowcloud wants to merge 7 commits into
feat/OSS-1719--schemas-virtualizationfrom
feat/OSS-1720--oas31-schemas-virtualization

Conversation

@glowcloud

Copy link
Copy Markdown
Contributor

This PR implements the second phase of virtualization - adding it for schemas section for OpenAPI 3.1.0. The virtualized path will kick in if there is 100 or more schemas.

Profiler showing how many components are rendered after expanding schemas section:

  • before:
oas31-perf-before
  • after:
oas31-perf-after

@glowcloud
glowcloud marked this pull request as ready for review August 25, 2026 09:19
@glowcloud
glowcloud requested review from cka121 and robert-hebel-sb and a lite review from Copilot and removed request for Copilot August 25, 2026 09:19
const ArrowDownIcon = getComponent("ArrowDownIcon")
const { getTitle } = fn.jsonSchema202012.useFn()

const schemaEntries = Object.entries(schemas)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would it make sense to add useMemo as it was done in json-schema-5/models.jsx?


const parentRef = useRef(null)
const measurementsCache = useRef([])

@robert-hebel-sb robert-hebel-sb Aug 26, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

2 participants