factory: parenthesize for-of expression when necessary (#34229)

Fixes: #33856

Author: ajafff

src/compiler/factory.ts

@@ -1811,7 +1811,7 @@ namespace ts {
         const node = <ForOfStatement>createSynthesizedNode(SyntaxKind.ForOfStatement);
         node.awaitModifier = awaitModifier;
         node.initializer = initializer;
-        node.expression = expression;
+        node.expression = isCommaSequence(expression) ? createParen(expression) : expression;
         node.statement = asEmbeddedStatement(statement);
         return node;
     }

tests/baselines/reference/nullishCoalescingOperator12.js

@@ -0,0 +1,10 @@
+//// [nullishCoalescingOperator12.ts]
+const obj: { arr: any[] } = { arr: [] };
+for (const i of obj?.arr ?? []) { }
+
+
+//// [nullishCoalescingOperator12.js]
+"use strict";
+var _a, _b;
+const obj = { arr: [] };
+for (const i of (_b = (_a = obj) === null || _a === void 0 ? void 0 : _a.arr, (_b !== null && _b !== void 0 ? _b : []))) { }

tests/baselines/reference/nullishCoalescingOperator12.symbols

@@ -0,0 +1,12 @@
+=== tests/cases/conformance/expressions/nullishCoalescingOperator/nullishCoalescingOperator12.ts ===
+const obj: { arr: any[] } = { arr: [] };
+>obj : Symbol(obj, Decl(nullishCoalescingOperator12.ts, 0, 5))
+>arr : Symbol(arr, Decl(nullishCoalescingOperator12.ts, 0, 12))
+>arr : Symbol(arr, Decl(nullishCoalescingOperator12.ts, 0, 29))
+
+for (const i of obj?.arr ?? []) { }
+>i : Symbol(i, Decl(nullishCoalescingOperator12.ts, 1, 10))
+>obj?.arr : Symbol(arr, Decl(nullishCoalescingOperator12.ts, 0, 12))
+>obj : Symbol(obj, Decl(nullishCoalescingOperator12.ts, 0, 5))
+>arr : Symbol(arr, Decl(nullishCoalescingOperator12.ts, 0, 12))
+

tests/baselines/reference/nullishCoalescingOperator12.types

@@ -0,0 +1,16 @@
+=== tests/cases/conformance/expressions/nullishCoalescingOperator/nullishCoalescingOperator12.ts ===
+const obj: { arr: any[] } = { arr: [] };
+>obj : { arr: any[]; }
+>arr : any[]
+>{ arr: [] } : { arr: never[]; }
+>arr : never[]
+>[] : never[]
+
+for (const i of obj?.arr ?? []) { }
+>i : any
+>obj?.arr ?? [] : any[]
+>obj?.arr : any[]
+>obj : { arr: any[]; }
+>arr : any[]
+>[] : never[]
+

tests/cases/conformance/expressions/nullishCoalescingOperator/nullishCoalescingOperator12.ts

@@ -0,0 +1,5 @@
+// @strict: true
+// @target: ES2015
+
+const obj: { arr: any[] } = { arr: [] };
+for (const i of obj?.arr ?? []) { }