fix(60887): Incorrect Formatting on if (a) try {} finally {} (#60898)

microsoft · Feb 24, 2025 · 2c3be44 · 2c3be44
1 parent 8ae98d0
commit 2c3be44
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/src/services/formatting/smartIndenter.ts b/src/services/formatting/smartIndenter.ts
@@ -729,6 +729,11 @@ export namespace SmartIndenter {
                     return false;
                 }
                 break;
+            case SyntaxKind.TryStatement:
+                if (childKind === SyntaxKind.Block) {
+                    return false;
+                }
+                break;
         }
         // No explicit rule for given nodes so the result will follow the default value argument
         return indentByDefault;

diff --git a/tests/cases/fourslash/formatTryFinally.ts b/tests/cases/fourslash/formatTryFinally.ts
@@ -0,0 +1,15 @@
+/// <reference path="fourslash.ts"/>
+
+////if (true) try  {
+////    // ...
+////}   finally    {
+////    // ...
+////}
+
+format.document();
+verify.currentFileContentIs(
+`if (true) try {
+    // ...
+} finally {
+    // ...
+}`);