Change https://golang.org/cl/382274 mentions this issue: internal/lsp/debug: add go version to gopls version output

At work we print this information as part of a diagnostics file.

type Diagnostics struct {
    BuildInfo *debug.BuildInfo
    [...]
}

func Write(w io.Writer) error {
    d := Diagnostics{ ... }
    buf, err := json.Marshal(d)
    [...]
}

We use JSON because it's easy for both humans and machines to parse. The new format makes the JSON output inscrutable for humans and breaks machine parsing. While there wasn't any guarantee that BuildInfo would always be representable as JSON, it doesn't seem an unreasonable request.

It would be nice if the fix could make it into 1.18.

cc: @jayconrod

Agree - we can't let the MarshalText break MarshalJSON. Probably we should just rename that method something else and let people call it as they need it. Will discuss with @bcmills and @matloob.

It's unfortunate that MarshalText and UnmarshalText break JSON embedding, because semantically they're a perfect fit: these methods really are marshaling and unmarshaling the object into “a textual form”, and the signatures are more appropriate than String / Parse because the marshaling operation can fail (because field values that could collide with separators do not have escaping).

But maybe it would be better to add some form of escaping and switch to func (*BuildInfo) String() and debug.ParseBuildInfo.

BuildInfo could also implement MarshalJSON and UnmarshalJSON to get the json back, something like:

func (b BuildInfo) MarshalJSON() ([]byte, error) {
    type buildInfo BuildInfo // Mask out the methods
    return json.Marshal(buildInfo(b))
}

It could, if it were in an ordinary package. Unfortunately, it is in runtime/debug, which cannot import encoding/json. 😞

So if we provided a MarshalJSON method it would probably have to be a bespoke implementation — and UnmarshalJSON in particular seems more complex than it's worth.

Perhaps a targeted hack in JSON marshaler will help?

Change https://go.dev/cl/384154 mentions this issue: runtime/debug: replace (*BuildInfo).Marshal methods with Parse and String

Change https://go.dev/cl/384161 mentions this issue: runtime/debug: expand fuzz corpus for FuzzParseBuildInfoRoundTrip