prettycsharp 1.1.1__tar.gz → 1.1.3__tar.gz

{prettycsharp-1.1.1 → prettycsharp-1.1.3}/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 Each release's section becomes the body of its GitHub release.
 
+## [1.1.3] - 2026-10-17
+
+- A property's accessor list or an enum that holds only a comment no longer
+  gains a blank line each time the file is formatted. Found by fuzzing.
+
+## [1.1.2] - 2026-10-17
+
+- Nothing is padded into columns by default any more: `align_assignments`,
+  `align_enum_values` and `align_switch_arms` now default to `false`, so each
+  `=` and `=>` has one space before it. Padded columns shift whenever one
+  line changes, and the diff then touches every line of the run. Set any of
+  the three to `true` to align as before.
+
 ## [1.1.1] - 2026-10-17
 
 - `syntax_errors = "format_around"` now works inside each view of a file

{prettycsharp-1.1.1 → prettycsharp-1.1.3}/DESIGN.md

@@ -233,12 +233,14 @@ int[] codes = [200, 404];
 
 | Setting | Default | Alternatives |
 | --- | --- | --- |
-| `align_switch_arms` | `true` | `false` |
-| `align_assignments` | `true`: consecutive local assignments and field initializers | `false` |
-| `align_enum_values` | `true` | `false` |
-
-Only the `=` or `=>` aligns. Types and names are not put into columns. A run ends at a blank
-line or at any statement of a different kind.
+| `align_switch_arms` | `false`: one space before each arm's `=>` | `true` |
+| `align_assignments` | `false`: one space before each `=` | `true`: consecutive local assignments and field initializers |
+| `align_enum_values` | `false`: one space before each `=` | `true` |
+
+By default nothing is padded into columns: alignment shifts whenever one line
+changes, and the diff touches every line of the run. Turned on, only the `=`
+or `=>` aligns; types and names are not put into columns, and a run ends at a
+blank line or at any statement of a different kind:
 
 ```csharp
 var id       = order.Id;

{prettycsharp-1.1.1 → prettycsharp-1.1.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: prettycsharp
-Version: 1.1.1
+Version: 1.1.3
 Summary: An opinionated formatter for C#, in the spirit of Prettier
 Project-URL: Homepage, https://github.com/WilliamSmithEdward/prettyCSharp
 Project-URL: Repository, https://github.com/WilliamSmithEdward/prettyCSharp
@@ -47,7 +47,7 @@ Description-Content-Type: text/markdown
 An opinionated formatter for C#, written in Python, in the spirit of Prettier.
 It parses each file with [tree-sitter](https://tree-sitter.github.io/) and
 reprints the whole file in one consistent layout: braces, indentation, line
-breaks, blank lines, alignment and a few safe rewrites such as file-scoped
+breaks, blank lines and a few safe rewrites such as file-scoped
 namespaces, sorted usings and expression-bodied members. It needs no .NET
 SDK.
 
@@ -175,7 +175,7 @@ Settings come from, highest precedence first:
 max_line_length = 100
 brace_style = "kr"
 continuation_keywords = "same_line"
-align_assignments = false
+align_assignments = true
 ```
 
 Every setting, its default, its alternatives and an example of each layout

{prettycsharp-1.1.1 → prettycsharp-1.1.3}/README.md

@@ -12,7 +12,7 @@
 An opinionated formatter for C#, written in Python, in the spirit of Prettier.
 It parses each file with [tree-sitter](https://tree-sitter.github.io/) and
 reprints the whole file in one consistent layout: braces, indentation, line
-breaks, blank lines, alignment and a few safe rewrites such as file-scoped
+breaks, blank lines and a few safe rewrites such as file-scoped
 namespaces, sorted usings and expression-bodied members. It needs no .NET
 SDK.
 
@@ -140,7 +140,7 @@ Settings come from, highest precedence first:
 max_line_length = 100
 brace_style = "kr"
 continuation_keywords = "same_line"
-align_assignments = false
+align_assignments = true
 ```
 
 Every setting, its default, its alternatives and an example of each layout

{prettycsharp-1.1.1 → prettycsharp-1.1.3}/src/prettycsharp/__init__.py

@@ -4,7 +4,7 @@ from prettycsharp.formatter import FormatResult, FormattingError, format_details
 from prettycsharp.parsing import CSharpSyntaxError
 from prettycsharp.settings import Settings, SettingsError
 
-__version__ = "1.1.1"
+__version__ = "1.1.3"
 
 __all__ = [
     "CSharpSyntaxError",

{prettycsharp-1.1.1 → prettycsharp-1.1.3}/src/prettycsharp/printer/declarations.py

@@ -399,7 +399,7 @@ class Declarations(Statements):
         return [
             self.open_brace("type"),
             self.tok(open_tok),
-            indent([HARDLINE, lines]),
+            indent([HARDLINE, lines]) if lines else "",
             self.close_comments(close_tok),
             HARDLINE,
             self.tok(close_tok, leading=False),
@@ -522,10 +522,13 @@ class Declarations(Statements):
             return [" ", self.tok(open_tok), " ", join(" ", [self.print(a) for a in accessors]), " ", self.tok(close_tok)]
         if not accessors and not close_tok.leading:
             return [" ", self.tok(open_tok), " ", self.tok(close_tok)]
+        body: Doc = ""
+        if accessors:
+            body = indent([HARDLINE, self.print_list(accessors, lambda a, b: self.has_blank_before(b))])
         return [
             self.open_brace("accessor"),
             self.tok(open_tok),
-            indent([HARDLINE, self.print_list(accessors, lambda a, b: self.has_blank_before(b))]),
+            body,
             self.close_comments(close_tok),
             HARDLINE,
             self.tok(close_tok, leading=False),

{prettycsharp-1.1.1 → prettycsharp-1.1.3}/src/prettycsharp/settings.py

@@ -70,9 +70,9 @@ class Settings:
     simple_lists_inline: bool = True
     trailing_commas: str = "remove"
 
-    align_switch_arms: bool = True
-    align_assignments: bool = True
-    align_enum_values: bool = True
+    align_switch_arms: bool = False
+    align_assignments: bool = False
+    align_enum_values: bool = False
 
     switch_case_indent: bool = True
     switch_case_blank_lines: bool = True

{prettycsharp-1.1.1 → prettycsharp-1.1.3}/tests/test_layout.py

@@ -131,7 +131,6 @@ def test_single_call_and_property_chains_stay_on_one_line() -> None:
         var city = order.Customer.Address.City;
         var zip = order.Customer.Address.GetPostalCode();
         """,
-        align_assignments=False,
     )
 
 
@@ -230,7 +229,7 @@ def test_fields_grouped_tight_and_blank_lines_around_members_with_bodies() -> No
 
             public Order(int id, string name)
             {
-                _id   = id;
+                _id = id;
                 _name = name;
             }
 
@@ -429,7 +428,6 @@ def test_simple_value_lists_stay_inline_and_complex_ones_break() -> None:
             new Point(1, 1)
         };
         """,
-        align_assignments=False,
     )
 
 
@@ -513,7 +511,35 @@ def test_switch_sections_are_indented_with_blank_lines_between() -> None:
     )
 
 
-def test_switch_expression_arms_align_their_arrows() -> None:
+def test_nothing_is_padded_into_columns() -> None:
+    check(
+        """
+        var label = status switch { Status.Paid => "Paid", Status.Pending => "Pending", _ => "Unknown" };
+        var id   = order.Id;
+        var customer = order.Customer;
+        public enum Status { Pending   = 0, Paid = 1, Cancelled = 2 }
+        """,
+        """
+        var label = status switch
+        {
+            Status.Paid => "Paid",
+            Status.Pending => "Pending",
+            _ => "Unknown"
+        };
+        var id = order.Id;
+        var customer = order.Customer;
+
+        public enum Status
+        {
+            Pending = 0,
+            Paid = 1,
+            Cancelled = 2
+        }
+        """,
+    )
+
+
+def test_switch_expression_arms_align_when_turned_on() -> None:
     check(
         """
         var label = status switch { Status.Paid => "Paid", Status.Pending => "Pending", _ => "Unknown" };
@@ -526,10 +552,11 @@ def test_switch_expression_arms_align_their_arrows() -> None:
             _              => "Unknown"
         };
         """,
+        align_switch_arms=True,
     )
 
 
-def test_consecutive_assignments_and_enum_values_align() -> None:
+def test_assignments_and_enum_values_align_when_turned_on() -> None:
     check(
         """
         var id = order.Id;
@@ -551,6 +578,8 @@ def test_consecutive_assignments_and_enum_values_align() -> None:
             Cancelled = 2
         }
         """,
+        align_assignments=True,
+        align_enum_values=True,
     )
 
 
@@ -678,6 +707,26 @@ def test_obvious_parentheses_are_removed_and_clarifying_ones_kept() -> None:
     )
 
 
+def test_accessor_list_and_enum_holding_only_a_comment() -> None:
+    check(
+        "class C { int P {\n// c\n} }\nenum E {\n// c\n}\n",
+        """
+        class C
+        {
+            int P
+            {
+                // c
+            }
+        }
+
+        enum E
+        {
+            // c
+        }
+        """,
+    )
+
+
 def test_auto_properties_and_empty_bodies_stay_on_one_line() -> None:
     check(
         """
@@ -797,7 +846,6 @@ def test_raw_and_verbatim_strings_are_untouched() -> None:
             """;
         var path = @"C:\\orders\\  ";
         ''',
-        align_assignments=False,
     )
 
 

{prettycsharp-1.1.1 → prettycsharp-1.1.3}/tests/test_rewrites.py

@@ -262,7 +262,6 @@ def test_parentheses_remove_redundant_follows_precedence() -> None:
         var m = n - (o - p);
         """,
         parentheses="remove_redundant",
-        align_assignments=False,
     )
 
 

{prettycsharp-1.1.1 → prettycsharp-1.1.3}/tests/test_settings.py

@@ -238,13 +238,15 @@ def test_initializer_break_threshold_zero_and_simple_lists() -> None:
     assert fmt("var a = new[] { 1, 2 };\n", simple_lists_inline=False) == "var a = new[]\n{\n    1,\n    2\n};\n"
 
 
-def test_alignment_off() -> None:
+def test_alignment_is_off_by_default_and_each_setting_turns_it_on() -> None:
     source = "var id = order.Id;\nvar customer = order.Customer;\n"
-    assert fmt(source, align_assignments=False) == source
-    assert fmt(
-        "var l = s switch { 1 => \"a\", 22 => \"b\" };\n", align_switch_arms=False
-    ) == 'var l = s switch\n{\n    1 => "a",\n    22 => "b"\n};\n'
-    assert fmt("enum E { A = 1, Bb = 2 }\n", align_enum_values=False) == "enum E\n{\n    A = 1,\n    Bb = 2\n}\n"
+    assert fmt(source) == source
+    assert fmt(source, align_assignments=True) == "var id       = order.Id;\nvar customer = order.Customer;\n"
+    switch = 'var l = s switch { 1 => "a", 22 => "b" };\n'
+    assert fmt(switch) == 'var l = s switch\n{\n    1 => "a",\n    22 => "b"\n};\n'
+    assert fmt(switch, align_switch_arms=True) == 'var l = s switch\n{\n    1  => "a",\n    22 => "b"\n};\n'
+    assert fmt("enum E { A = 1, Bb = 2 }\n") == "enum E\n{\n    A = 1,\n    Bb = 2\n}\n"
+    assert fmt("enum E { A = 1, Bb = 2 }\n", align_enum_values=True) == "enum E\n{\n    A  = 1,\n    Bb = 2\n}\n"
 
 
 def test_switch_case_indent_and_blank_lines_off() -> None: