c2p bypassConversion static method create unecessary promises

[rubenfiszel]

I spent a lot of time on the debugger for #356 and I got to the bottom of it.

bypassConversion:

monaco-languageclient/packages/client/src/monaco-language-client.ts

Line 18 in 5f056e2

static bypassConversion = (result: any, token?: vscode.CancellationToken) => token != null ? Promise.resolve(result || undefined) : (result || undefined);

in conjunction with c2p.asFormattingOptions at:

monaco-languageclient/packages/client/src/monaco-converter.ts

Line 446 in d55ce36

asFormattingOptions(options: monaco.languages.FormattingOptions): FormattingOptions {

results in: https://github.com/microsoft/vscode-languageserver-node/blob/f97bb73dbfb920af4bc8c13ecdcdc16359cdeda6/client/src/common/formatting.ts#L77 returning a Promise instead of the value itself.

The promise does not get serialized which is the cause for #356