Sync namespace to folder reformats the entire document

[davkean]

Version Used:
2019 RTM

Steps to Reproduce:

1. At $ run Sync namespace to folder

using System;

namespace Namespace
{
    class Program
    {
        //          Column1               Column2
        [InlineData("FooBar",             "Bar")]
        [InlineData("FooBarFooBar",       "Bar")]
        [InlineData("FooBarFooBarFooBar", "Bar")]
        static void Main(string[] args)
        {
            Console.WriteLine("Hello World!");
        }
    }
}

Expected Behavior:

using System;

namespace Namespace.NewFolder
{
    class Program
    {
        //          Column1               Column2
        [InlineData("FooBar",             "Bar")]
        [InlineData("FooBarFooBar",       "Bar")]
        [InlineData("FooBarFooBarFooBar", "Bar")]
        static void Main(string[] args)
        {
            Console.WriteLine("Hello World!");
        }
    }
}

Actual Behavior:

using System;

namespace Namespace.NewFolder
{
    class Program
    {
        //          Column1               Column2
        [InlineData("FooBar", "Bar")]
        [InlineData("FooBarFooBar", "Bar")]
        [InlineData("FooBarFooBarFooBar", "Bar")]
        static void Main(string[] args)
        {
            Console.WriteLine("Hello World!");
        }
    }
}

I cannot use this feature in any test project as it breaks all tables. Why does it need to reformat the document to fix the namespace?

[felixhirt]

I noticed that (at least for me), it even formats based on rules which are marked as silent in .editorconfig. To be precise, i have: csharp_prefer_braces = false:silent in my editorconfig, and fixing the namespace removes all braces for single line statements. This breaks the feature for me

[sharwell]

To be precise, i have: csharp_prefer_braces = false:silent in my editorconfig, and fixing the namespace removes all braces for single line statements.

Roslyn does not have a feature to remove braces (see #29606 for a request to add it). I have no idea how/what/why you observed this.

[felixhirt]

Your right, i guess its a VisualStudio feature. Im talking about this:

And its respective .editorconfig setting. The reason why i mention it here (and not in the VisualStudio uservoice) is because it happens for me on the namespace refactoring

Im sorry if this is the wrong place for this issue, im happy to move this wherever it is supposed to go

[felixhirt]

Also, i dont think i have any code cleanup or formatting options defined which do this for me automatically (on save or by hotkey or whatever), which is why it confuses me even more. The only way to do this normally in my setup is to move the cursor to the braces and use the "remove braces" refactoring it offers there.

[CyrusNajmabadi]

Roslyn does not have a feature to remove braces (see #29606 for a request to add it). I have no idea how/what/why you observed this.

@sharwell if someone puts ont he simplifaction annotation on code, this is a simplification that can happen. That happens so that someone can use the normal SyntaxGenerator codepaths but end up with blocks that match their "allow preference" here:

If its the case that sync-namespace is adding teh simplification annotation (for example, to shorten type names post-move) then this might fall out because of it.

[felixhirt]

Ah, i understand. Thanks for the info! Sorry to bother again, but could you give me any hints at how i can disable this feature for this specific option? The option is allready set to "silent", which seems to be the lowest severity? I also would like to keep all the other code style options on and also apply them on code generation if possible
edit: pretty sure the original issue creator would prefer to completely disable these simplifications for this specific refactoring. I would be fine with that as well

[felixhirt]

I just realized that if i set the option to "csharp_prefer_braces = when_multiline:silent" it is not automatically applied, even if it would be the case. No idea why, but that works for me...

[felixhirt]

I have to revoke my last comment. The brace removal started again. I cannot figure out why it happens sometimes and sometimes not.