support cors output without experimental flag

Signed-off-by: kkk777-7 <kota.kimura0725@gmail.com>

Author: kkk777-7

cmd/print.go

@@ -85,7 +85,7 @@ type PrintRunner struct {
 	// Defaults to "standard".
 	emitter string
 
-	// allowExperimentalGatewayAPI indicates whether Experimental Gateway API features (like CORS, URLRewrite) should be included in the output.
+	// allowExperimentalGatewayAPI indicates whether Experimental Gateway API features (like URLRewrite) should be included in the output.
 	allowExperimentalGatewayAPI bool
 }
 
@@ -421,7 +421,7 @@ if specified with --namespace.`)
 	cmd.Flags().StringSliceVar(&pr.providers, "providers", []string{},
 		fmt.Sprintf("If present, the tool will try to convert only resources related to the specified providers, supported values are %v.", i2gw.GetSupportedProviders()))
 
-	cmd.Flags().BoolVar(&pr.allowExperimentalGatewayAPI, "allow-experimental-gw-api", false, "If present, the tool will include Experimental Gateway API fields (e.g. CORS, URLRewrite) in the output. Default is false.")
+	cmd.Flags().BoolVar(&pr.allowExperimentalGatewayAPI, "allow-experimental-gw-api", false, "If present, the tool will include Experimental Gateway API fields (e.g. URLRewrite) in the output. Default is false.")
 
 	pr.providerSpecificFlags = make(map[string]*string)
 	for provider, flags := range i2gw.GetProviderSpecificFlagDefinitions() {

pkg/i2gw/emitter.go

@@ -60,6 +60,6 @@ type EmitterName string
 type EmitterConstructor func(conf *EmitterConf) Emitter
 
 type EmitterConf struct {
-	// AllowExperimentalGatewayAPI indicates whether Experimental Gateway API features (like CORS, URLRewrite) should be included in the output.
+	// AllowExperimentalGatewayAPI indicates whether Experimental Gateway API features (like URLRewrite) should be included in the output.
 	AllowExperimentalGatewayAPI bool
 }

pkg/i2gw/emitters/common_emitter/emitter.go

@@ -72,9 +72,6 @@ func applyPathRewrites(ir *emitterir.EmitterIR) {
 }
 
 func (e *Emitter) applyCorsPolicies(ir *emitterir.EmitterIR) {
-	if e.conf != nil && !e.conf.AllowExperimentalGatewayAPI {
-		return
-	}
 	for key, routeCtx := range ir.HTTPRoutes {
 		for ruleIdx, policy := range routeCtx.CorsPolicyByRuleIdx {
 			if policy == nil {

pkg/i2gw/emitters/common_emitter/emitter_test.go

@@ -164,7 +164,7 @@ func TestEmitCORSFiltering(t *testing.T) {
 			name:                 "experimental denied + cors in sidecar -> cors NOT added",
 			allowExperimental:    false,
 			corsInSidecar:        &gatewayv1.HTTPCORSFilter{},
-			expectedFiltersCount: 0,
+			expectedFiltersCount: 1,
 		},
 		{
 			name:              "other filters preserved regardless of flag",
@@ -173,7 +173,7 @@ func TestEmitCORSFiltering(t *testing.T) {
 				{Type: gatewayv1.HTTPRouteFilterRequestHeaderModifier},
 			},
 			corsInSidecar:        &gatewayv1.HTTPCORSFilter{},
-			expectedFiltersCount: 1, // only header modifier
+			expectedFiltersCount: 2, // only header modifier
 		},
 	}