Adding indentations in csi

[ManishJayaswal]

@amcasey

[genlu]

Here's how Notepad++ works, I will follow the same behavior if you think it is reasonable:

• Caret following any typed newline will indent with previous line, i.e. first non-space char.
• However, if a typed newline character is proceeded by an opening bracket '{', then the caret at next line will start with 4 additional spaces from the column where the first non-space char of previous line located (for now, but this should be adjustable, maybe using script global object in the future)
• If a typed character is a closing bracket '}' which is the first non-space character of that line, then it will indent with the first non-space char of the line where its matching opening bracket is located. If there's no matching opening bracket (an error), then it will be treated as any other regular char.

@dotnet/roslyn-interactive

[mattwar]

Shouldn't you be using the formatter or auto-indenter?

[genlu]

@mattwar This task is about csi, the command-line REPL. My understanding is all the IDE features are irrelevant here. @amcasey showed me something related to this in compiler layer (e.g. SyntaxNormalizer, NormalizeWhitespace method on SyntaxToken, etc.), but we can't use them since they are manipulating syntax tree, which will not reflect back to existing text in console. I will look at their implementation and follow the same behavior.

[ManishJayaswal]

Sounds good to me!

[mattwar]

The auto-indenter mostly defers to the formatter, which is in Workspace layer.

[mattwar]

Since this is CSI, you should probably do something simple. If we wanted to spend the time though, being Console is no impediment. You can do crazy stuff in console.