fixes

Author: andreaangiolillo

tools/cli/internal/openapi/filter/filter.go

@@ -78,6 +78,7 @@ func DefaultFilters(oas *openapi3.T, metadata *Metadata) []Filter {
 		&HiddenEnvsFilter{oas: oas, metadata: metadata},
 		&TagsFilter{oas: oas},
 		&OperationsFilter{oas: oas},
+		&SchemasFilter{oas: oas},
 	}
 }
 
@@ -91,13 +92,6 @@ func FiltersWithoutVersioning(oas *openapi3.T, metadata *Metadata) []Filter {
 	}
 }
 
-func FiltersToRemoveUnusedElements(oas *openapi3.T) []Filter {
-	return []Filter{
-		&TagsFilter{oas: oas},
-		&SchemasFilter{oas: oas},
-	}
-}
-
 // FiltersToGetVersions returns a list of filters to apply to the OpenAPI document to get the versions.
 func FiltersToGetVersions(oas *openapi3.T, metadata *Metadata) []Filter {
 	return []Filter{

tools/cli/internal/openapi/filter/schemas.go

@@ -17,15 +17,21 @@ package filter
 import (
 	"encoding/json"
 	"fmt"
-	"github.com/getkin/kin-openapi/openapi3"
 	"strings"
+
+	"github.com/getkin/kin-openapi/openapi3"
 )
 
-// SchemasFilter removes tags that are not used in the operations.
+// SchemasFilter removes components that are not used in the operations.
 type SchemasFilter struct {
 	oas *openapi3.T
 }
 
+func (f *SchemasFilter) ValidateMetadata() error {
+	//TODO implement me
+	panic("implement me")
+}
+
 func (f *SchemasFilter) Apply() error {
 	if f.oas.Paths == nil {
 		return nil

tools/cli/internal/openapi/openapi.go

@@ -16,7 +16,6 @@ package openapi
 
 //go:generate mockgen -destination=../openapi/mock_openapi.go -package=openapi github.com/mongodb/openapi/tools/cli/internal/openapi Parser,Merger
 import (
-	"github.com/mongodb/openapi/tools/cli/internal/openapi/filter"
 	"log"
 
 	"github.com/getkin/kin-openapi/openapi3"
@@ -64,12 +63,6 @@ func (o *OasDiff) MergeOpenAPISpecs(paths []string) (*Spec, error) {
 		}
 	}
 
-	for _, f := range filter.FiltersToRemoveUnusedElements(o.base.Spec) {
-		if err := f.Apply(); err != nil {
-			return nil, err
-		}
-	}
-
 	return newSpec(o.base.Spec), nil
 }
 

tools/cli/internal/openapi/openapi3.go

@@ -47,7 +47,6 @@ func (o *OpenAPI3) WithExcludedPrivatePaths() *OpenAPI3 {
 
 func (o *OpenAPI3) CreateOpenAPISpecFromPath(path string) (*load.SpecInfo, error) {
 	o.Loader.IsExternalRefsAllowed = o.IsExternalRefsAllowed
-
 	spec, err := load.NewSpecInfo(o.Loader, load.NewSource(path))
 	if err != nil {
 		return nil, err
@@ -57,7 +56,6 @@ func (o *OpenAPI3) CreateOpenAPISpecFromPath(path string) (*load.SpecInfo, error
 	if o.ExcludePrivatePaths {
 		removePrivatePaths(spec.Spec)
 	}
-
 	return spec, nil
 }