Add support for reserved message names.

[halkeye]

// Function notification settings
message Function {
    string namespace = 1;
    string name = 2;
}

Generates a file that typescript is unable to process (for good reason)

 error TS2345: Argument of type 'Function | {}' is not assignable to parameter of type 'Function'.
  Type '{}' is missing the following properties from type 'Function': namespace, name

I know i can use --ts_proto_opt=typePrefix=MyPrefix or --ts_proto_opt=typeSuffix=MySuffix to add extra to each type, but that would do it for everything, not just reserved words, which would make migrated to generated output a little rougher.

Is there anything I can do to handle the reserved ones? I am up for contributing.

message Object {
  string key = 1;
}

Is another example.

[stephenh]

Hi @halkeye ! That makes sense...kinda surprised we haven't ran into this before 🤔

Ah yeah, I think it's here:

ts-proto/src/visit.ts

Line 63 in ffacf3a

const builtInNames = ["Date"];

Maybe just add Function to that builtinNames?

A PR would be great! Thanks!

[halkeye]

I'm going to get you a pr soon, but so i don't forget it, https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects is probably some others that might be worth grabbing a few from