Skip to content

Detect components.pathItems in OpenAPI 3.0 documents (PathItemsIn30 rule) - #196

Merged
geemus merged 4 commits into
ota42y:masterfrom
takayamaki:pr7-pathitems-30-warning
Jul 16, 2026
Merged

Detect components.pathItems in OpenAPI 3.0 documents (PathItemsIn30 rule)#196
geemus merged 4 commits into
ota42y:masterfrom
takayamaki:pr7-pathitems-30-warning

Conversation

@takayamaki

Copy link
Copy Markdown
Contributor

Continuing the OpenAPI 3.1 work from #195 / #152.

components.pathItems was added in OpenAPI 3.1 and is not part of the 3.0 spec.
The parse layer already accepts it permissively (#191), but until now there was no way to detect the version mismatch.

This PR adds a PathItemsIn30 rule to SpecValidator that reports a violation when a 3.0 document declares components.pathItems.

How it works

The rule checks root.components.raw_schema for a pathItems key.
It only fires on 3.0 documents.

OpenAPIParser.load(
  'spec.yaml',
  strict_specification_version: :warn,
)
# [PathItemsIn30] #/components/pathItems — `components.pathItems` is a 3.1 addition; 3.0 documents should not declare it

Housekeeping

This PR also backfills the CHANGELOG entries for NullableDeprecation (#193) and ExampleSingularDeprecation (#195), which I forgot to include in those PRs.

… PR7)

The parse layer keeps accepting Components.pathItems in any version
(geemus suggested permissive parsing); this rule reports the 3.0
mismatch so callers on :warn / :raise can surface it. Detection
inspects components.raw_schema directly to catch an explicitly empty
`pathItems: {}` literal too.
`components.pathItems` on a 3.0 document warns and raises (3.1 addition);
the same field on a 3.1 document stays clean.
@takayamaki
takayamaki marked this pull request as ready for review July 4, 2026 02:38

@geemus geemus left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, thanks!

@geemus
geemus merged commit 648e188 into ota42y:master Jul 16, 2026
14 checks 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.

2 participants