Alexander Sklar

we should try this out! thanks for the tip @wjk !

Is any of this documented / stable enough? I couldn't find any docs on this on my cursory search so I worry about relying on implementation details that could change....

@copilot note that we have to support all valid MIDL syntax, not just a handful of keywords. Also make sure we're not mixing up C# syntax into MIDL (e.g. I...

@copilot the build pipeline is hitting lots of errors, excerpt: > [email protected] build > tsc -b Error: client/node_modules/@types/node/buffer.d.ts(128,19): error TS1005: ',' expected. Error: client/node_modules/@types/node/buffer.d.ts(128,51): error TS1005: ',' expected. Error: client/node_modules/@types/node/http.d.ts(241,47):...

@copilot the formatter needs a lot more work (and tests), e.g. this file: ``` namespace foo{ runtimeclass bar{ }} ``` when formatted, ends up not changing (the formatting isn't doing...

@copilot we need a lot more unit tests. check out this case, which breaks: namespace foo{ runtimeclass bar{ Int32 g{get;set;}; }} the formatted text removes the g property altogether!

@copilot let's fix the rest of the tests, spacing normalization is important too

@copilot attributes are being left on the same line as the thing they apply to: `namespace foo{ [foo(a, "b")]runtimeclass bar{ x a;} }`

@copilot if there's an empty line, we should leave it there. Multiple empty lines we should collapse to a single empty line. We shoudl ensure there's a single empty line...

For MCPB, you can specify a `tools` array + a `tools_generated` bool in the manifest. On Windows, we'll also have an additional property under the new `_meta` in MCPB, where...