proposed fix: Merging operations causes duplicate key/value pairs in "map" fields

[sloretz]

Does this look like a change you'd accept?

I'd like to override the content-type annotation of a streaming API to text/event-stream for use with server-sent events. However, when I add an openapi.v3.operation annotation and generate an OpenAPI spec I get a schema that lists two response types for "200".

I'm assuming the problem is the proto file lists fields as repeated with Named... message types rather than map, and this proto.Merge appends those messages as if they were a list no matter what.

https://github.com/google/gnostic/blob/e0e09f70628157dc0db87ed60a109465ae62723b/cmd/protoc-gen-openapi/generator/generator.go#L756-L760

I think the fix would be to write a custom merge function that resolves conflicts. My preference would be to keep the last annotation. For now I've only updated the tests to show the behavior I want to change. How does it look to you?