Skip to content

Commit 69ab40f

Browse files
devm33Copilot
andcommitted
Default ExpConfigEntry.Id to "" in Java
The Id field is required by the wire contract, but defaulting to null let class-level NON_NULL drop the key for a zero-value entry. Default it to "" so the required key is always emitted, matching the Go and .NET defaults. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 43d7a02b-08ff-4e7d-a8c0-afa6dfbe94b0
1 parent 562a187 commit 69ab40f

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

java/src/main/java/com/github/copilot/rpc/ExpConfigEntry.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,15 +21,15 @@
2121
public class ExpConfigEntry {
2222

2323
@JsonProperty("Id")
24-
private String id;
24+
private String id = "";
2525

2626
@JsonProperty("Parameters")
2727
private Map<String, Object> parameters = new LinkedHashMap<>();
2828

2929
/**
3030
* Gets the identifier of this configuration entry.
3131
*
32-
* @return the entry identifier, or {@code null} if not set
32+
* @return the entry identifier (empty string when unset)
3333
*/
3434
public String getId() {
3535
return id;

0 commit comments

Comments
 (0)