Skip to content

bug fix propertyNames schema is not removed as unreferenced by SpecFilter #5017#5020

Merged
ewaostrowska merged 1 commit intoswagger-api:masterfrom
OllieKosh:bugfix/5017-property-names-schema-wrongfully-errased
Feb 19, 2026
Merged

bug fix propertyNames schema is not removed as unreferenced by SpecFilter #5017#5020
ewaostrowska merged 1 commit intoswagger-api:masterfrom
OllieKosh:bugfix/5017-property-names-schema-wrongfully-errased

Conversation

@OllieKosh
Copy link
Contributor

@OllieKosh OllieKosh commented Nov 18, 2025

Pull Request

Description

Fixes: #5017

Adding a check in specfilter to validate Schema#propertyNames is not empty and adding it to referenced schemas.

Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • ♻️ Refactor (non-breaking change)
  • 🧪 Tests
  • 📝 Documentation
  • 🧹 Chore (build or tooling)

Checklist

  • I have added/updated tests as needed
  • I have added/updated documentation where applicable
  • The PR title is descriptive
  • The code builds and passes tests locally
  • I have linked related issues (if any)

@OllieKosh OllieKosh force-pushed the bugfix/5017-property-names-schema-wrongfully-errased branch from 3c62cb5 to 7cd4ae7 Compare November 27, 2025 19:05
@ewaostrowska ewaostrowska force-pushed the bugfix/5017-property-names-schema-wrongfully-errased branch from 1e6e72b to 40416df Compare February 19, 2026 13:14
@ewaostrowska ewaostrowska merged commit 0c6fe0d into swagger-api:master Feb 19, 2026
7 checks passed
@ewaostrowska
Copy link
Contributor

Thank you @OllieKosh for taking care of this issue!

@OllieKosh
Copy link
Contributor Author

@ewaostrowska thank you for the review!

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: OpenAPI 3.1: Enum used in @Schema#propertyNames removed when filtering unreferenced schemas

3 participants