Skip to content

refactor: streamline tag filtering logic in SpecFilter - #5256

Merged
daniel-kmiecik merged 3 commits into
swagger-api:masterfrom
Mattias-Sehlstedt:refactor-tag-filtering
Aug 17, 2026
Merged

daniel-kmiecik merged 3 commits into
swagger-api:masterfrom
Mattias-Sehlstedt:refactor-tag-filtering

Conversation

@Mattias-Sehlstedt

Copy link
Copy Markdown
Contributor

Pull Request

Thank you for contributing to swagger-core!

Please fill out the following information to help us review your PR efficiently.


Description

Introduced an object (TagFilter) to couple the Tag filtering fields to each other. it also breaks out the filtering logic into a private support method. This is part of making the behavior more understandable with the intent of making the necessary changes needed for the OAS 3.2 Tag functionality easier to integrate. Mainly the pruning of filtered Tags that are involved with the new parent field (i.e., if the parent is removed as part of the regular filtering, then the children should no longer refer to it).

This change is extracted from the Tag implementation present on branch.

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)

Screenshots / Additional Context

@ewaostrowska ewaostrowska added the backlog label Aug 6, 2026 — with Claude
@daniel-kmiecik
daniel-kmiecik merged commit c56ca3b into swagger-api:master Aug 17, 2026
7 checks passed
@Mattias-Sehlstedt
Mattias-Sehlstedt deleted the refactor-tag-filtering branch August 17, 2026 19:35
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.

4 participants