closure-compiler icon indicating copy to clipboard operation
closure-compiler copied to clipboard

Implement OriginalMapping as an autovalue rather than protobuf message

Open niloc132 opened this issue 4 months ago • 2 comments

Enables downstream projects to depend only on closure-compiler (and not protobuf-java) to read and write sourcemaps.

The proto file has shrunk over time, and only has a single message type, used to communicate details discovered when reading a sourcemap from JSON, composed into a message structure, and then back into separate values. The proto-based implementation doesn't serialize or deserialize, and doesn't appear to be involved in CompilerState even indirectly.

The replacement is slightly stricter - rather than marking each field as optional, only the identifier is optional.

niloc132 avatar Feb 14 '24 02:02 niloc132