JSON/YAML tag uses "omitempty" as the field name

[Vasco-jofra]

What happened?

Hi! While doing a broad search for this pattern, I noticed that you use "omitempty" as the name of the Replacements field in the ReplacementTransformerPlugin structure.

kustomize/plugin/builtin/replacementtransformer/ReplacementTransformer.go

Line 20 in bb7a280

Replacements []types.Replacement `json:"omitempty" yaml:"omitempty"`

Instead, I believe you wanted to write ",omitempty" to keep the same name but omit when empty, although, in this structure, it might conflict with the name of the ReplacementList field.

What did you expect to happen?

How can we reproduce it (as minimally and precisely as possible)?

You can test this behavior with this simple Go program:

package main

import (
	"encoding/json"
	"fmt"
)

type ReplacementTransformerPlugin struct {
	ReplacementList []string `json:"replacements,omitempty" yaml:"replacements,omitempty"`
	Replacements    []string `json:"omitempty" yaml:"omitempty"`
}

func main() {
	u := ReplacementTransformerPlugin{}
	_ = json.Unmarshal([]byte(`{"replacements": ["replacement_list_1", "replacement_list_2"], "omitempty": ["replacement_1", "replacement_2"]}`), &u)
	fmt.Printf("Result: %#v\n", u)
	// Result: main.ReplacementTransformerPlugin{ReplacementList:[]string{"replacement_list_1", "replacement_list_2"}, Replacements:[]string{"replacement_1", "replacement_2"}}
}

As you can see by the result, the omitempty key in the JSON string gets unmarshaled to the Replacements field.

Expected output

No response

Actual output

No response

Kustomize version

latest

Operating system

None

[koba1t]

I can agree that it is a bug.
This missing tag was introduced at #3737.

/triage accepted