Repeated annotations and broken formatting when adding suppressions

[lazaroclapp]

Describe the bug
When running the auto-annotator on suppression (and @NullUnmarked) adding mode, I am observing two issues on fields that otherwise already contain annotations:

• Duplicated instances of @Nullable added
• Lack of spaces between the last added annotation and the field's modifiers

For example, this is the initial code:

@JsonProperty
@JsonInclude(JsonInclude.Include.NON_NULL)
public String foo;

And this the auto-annotator's result (before Google Java Format, but that's irrelevant for now because this code won't build):

@JsonProperty @JsonInclude(JsonInclude.Include.NON_NULL) @Nullable @SuppressWarnings("NullAway.Init")
@Nullablepublic   String foo;

Note: a) @Nullable is repeated, despite it already having @SuppressWarnings("NullAway.Init") (which makes no sense combined with @Nullable); b) @Nullablepublic as opposed to @Nullable public

To Reproduce
See code-example above. Configuration includes --chain --activate-downstream-dependencies-analysis --force-resolve.

Expected behavior
I would expect that field to either be annotated:

@JsonProperty
@JsonInclude(JsonInclude.Include.NON_NULL)
@SuppressWarnings("NullAway.Init")
public String foo;

Or:

@JsonProperty
@JsonInclude(JsonInclude.Include.NON_NULL)
@Nullable
public String foo;

Stack trace
N/A

OS (please complete the following information):

• OS: Linux
• Annotator Version : this commit, currently the head of Fix #75 by updating TOP method node id and returned overridden methods #76

[nimakarimipour]

#90 is landed and resolves this issue.