Revert "Add output filtering (oapi-codegen#2201)"

This reverts commit db8abb8.

Author: jamietanna

experimental/Configuration.md

@@ -37,26 +37,6 @@ generation:
     path: github.com/org/project/models
     alias: models  # optional, defaults to last segment of path
 
-# Output options: control which operations and schemas are included.
-output-options:
-  # Only include operations tagged with one of these tags. Ignored when empty.
-  include-tags:
-    - public
-    - beta
-  # Exclude operations tagged with one of these tags. Ignored when empty.
-  exclude-tags:
-    - internal
-  # Only include operations with one of these operation IDs. Ignored when empty.
-  include-operation-ids:
-    - listPets
-    - createPet
-  # Exclude operations with one of these operation IDs. Ignored when empty.
-  exclude-operation-ids:
-    - deprecatedEndpoint
-  # Exclude schemas with the given names from generation. Ignored when empty.
-  exclude-schemas:
-    - InternalConfig
-
 # Type mappings: OpenAPI type/format to Go type.
 # User values are merged on top of defaults — you only need to specify overrides.
 type-mapping:

experimental/cmd/oapi-codegen/main.go

@@ -3,9 +3,6 @@ package main
 import (
 	"flag"
 	"fmt"
-	"io"
-	"net/http"
-	"net/url"
 	"os"
 	"path/filepath"
 	"strings"
@@ -23,9 +20,9 @@ func main() {
 	flagPackage := flag.String("package", "", "Go package name for generated code")
 	flagOutput := flag.String("output", "", "output file path (default: <spec-basename>.gen.go)")
 	flag.Usage = func() {
-		fmt.Fprintf(os.Stderr, "Usage: %s [options] <spec-path-or-url>\n\n", os.Args[0])
+		fmt.Fprintf(os.Stderr, "Usage: %s [options] <spec-path>\n\n", os.Args[0])
 		fmt.Fprintf(os.Stderr, "Arguments:\n")
-		fmt.Fprintf(os.Stderr, "  spec-path-or-url    path or URL to OpenAPI spec file\n\n")
+		fmt.Fprintf(os.Stderr, "  spec-path    path to OpenAPI spec file\n\n")
 		fmt.Fprintf(os.Stderr, "Options:\n")
 		flag.PrintDefaults()
 	}
@@ -38,8 +35,8 @@ func main() {
 
 	specPath := flag.Arg(0)
 
-	// Load the OpenAPI spec from file or URL
-	specData, err := loadSpec(specPath)
+	// Parse the OpenAPI spec
+	specData, err := os.ReadFile(specPath)
 	if err != nil {
 		fmt.Fprintf(os.Stderr, "error reading spec: %v\n", err)
 		os.Exit(1)
@@ -81,12 +78,7 @@ func main() {
 
 	// Default output to <spec-basename>.gen.go
 	if cfg.Output == "" {
-		// For URLs, extract the filename from the URL path
-		baseName := specPath
-		if u, err := url.Parse(specPath); err == nil && u.Scheme != "" && u.Host != "" {
-			baseName = u.Path
-		}
-		base := filepath.Base(baseName)
+		base := filepath.Base(specPath)
 		ext := filepath.Ext(base)
 		cfg.Output = strings.TrimSuffix(base, ext) + ".gen.go"
 	}
@@ -111,31 +103,3 @@ func main() {
 
 	fmt.Printf("Generated %s\n", cfg.Output)
 }
-
-// loadSpec loads an OpenAPI spec from a file path or URL.
-func loadSpec(specPath string) ([]byte, error) {
-	u, err := url.Parse(specPath)
-	if err == nil && u.Scheme != "" && u.Host != "" {
-		return loadSpecFromURL(u.String())
-	}
-	return os.ReadFile(specPath)
-}
-
-// loadSpecFromURL fetches an OpenAPI spec from an HTTP(S) URL.
-func loadSpecFromURL(specURL string) ([]byte, error) {
-	resp, err := http.Get(specURL) //nolint:gosec // URL comes from user-provided spec path
-	if err != nil {
-		return nil, fmt.Errorf("fetching spec from URL: %w", err)
-	}
-	defer func() { _ = resp.Body.Close() }()
-
-	if resp.StatusCode != http.StatusOK {
-		return nil, fmt.Errorf("fetching spec from URL: HTTP %d %s", resp.StatusCode, resp.Status)
-	}
-
-	data, err := io.ReadAll(resp.Body)
-	if err != nil {
-		return nil, fmt.Errorf("reading spec from URL: %w", err)
-	}
-	return data, nil
-}

experimental/internal/codegen/codegen.go

@@ -31,9 +31,6 @@ func Generate(doc libopenapi.Document, specData []byte, cfg Configuration) (stri
 		return "", fmt.Errorf("gathering schemas: %w", err)
 	}
 
-	// Filter excluded schemas
-	schemas = FilterSchemasByName(schemas, cfg.OutputOptions.ExcludeSchemas)
-
 	// Pass 2: Compute names for all schemas
 	converter := NewNameConverter(cfg.NameMangling, cfg.NameSubstitutions)
 	ComputeSchemaNames(schemas, converter, contentTypeNamer)
@@ -105,9 +102,6 @@ func Generate(doc libopenapi.Document, specData []byte, cfg Configuration) (stri
 			return "", fmt.Errorf("gathering operations: %w", err)
 		}
 
-		// Apply operation filters
-		ops = FilterOperations(ops, cfg.OutputOptions)
-
 		// Generate client
 		clientGen, err := NewClientGenerator(schemaIndex, cfg.Generation.SimpleClient, cfg.Generation.ModelsPackage)
 		if err != nil {
@@ -164,9 +158,6 @@ func Generate(doc libopenapi.Document, specData []byte, cfg Configuration) (stri
 			return "", fmt.Errorf("gathering operations: %w", err)
 		}
 
-		// Apply operation filters
-		ops = FilterOperations(ops, cfg.OutputOptions)
-
 		if len(ops) > 0 {
 			// Generate server
 			serverGen, err := NewServerGenerator(cfg.Generation.Server)

experimental/internal/codegen/configuration.go

@@ -15,8 +15,6 @@ type Configuration struct {
 	Output string `yaml:"output"`
 	// Generation controls which parts of the code are generated
 	Generation GenerationOptions `yaml:"generation,omitempty"`
-	// OutputOptions controls filtering of operations and schemas
-	OutputOptions OutputOptions `yaml:"output-options,omitempty"`
 	// TypeMapping allows customizing OpenAPI type/format to Go type mappings
 	TypeMapping TypeMapping `yaml:"type-mapping,omitempty"`
 	// NameMangling configures how OpenAPI names are converted to Go identifiers
@@ -40,20 +38,6 @@ type Configuration struct {
 	StructTags StructTagsConfig `yaml:"struct-tags,omitempty"`
 }
 
-// OutputOptions controls filtering of which operations and schemas are included in generation.
-type OutputOptions struct {
-	// IncludeTags only includes operations tagged with one of these tags. Ignored when empty.
-	IncludeTags []string `yaml:"include-tags,omitempty"`
-	// ExcludeTags excludes operations tagged with one of these tags. Ignored when empty.
-	ExcludeTags []string `yaml:"exclude-tags,omitempty"`
-	// IncludeOperationIDs only includes operations with one of these operation IDs. Ignored when empty.
-	IncludeOperationIDs []string `yaml:"include-operation-ids,omitempty"`
-	// ExcludeOperationIDs excludes operations with one of these operation IDs. Ignored when empty.
-	ExcludeOperationIDs []string `yaml:"exclude-operation-ids,omitempty"`
-	// ExcludeSchemas excludes schemas with the given names from generation. Ignored when empty.
-	ExcludeSchemas []string `yaml:"exclude-schemas,omitempty"`
-}
-
 // ModelsPackage specifies an external package containing the model types.
 type ModelsPackage struct {
 	// Path is the import path for the models package (e.g., "github.com/org/project/models")