@tsrx/prettier-plugin 0.3.94 → 0.3.95

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tsrx/prettier-plugin",
-  "version": "0.3.94",
+  "version": "0.3.95",
   "description": "Ripple plugin for Prettier",
   "type": "module",
   "module": "src/index.js",

package/src/index.js

@@ -403,6 +403,16 @@ function binaryExpressionNeedsParens(node, parent) {
 		if (nodePrecedence === parentPrecedence && node.operator !== parent.operator) {
 			return true;
 		}
+		if (nodePrecedence === parentPrecedence) {
+			// Same precedence, same operator: dropping the parens regroups a
+			// left-associative chain (`a - (b - c)` !== `a - b - c`; even `+` is
+			// non-associative once strings are involved), so the RIGHT operand
+			// keeps its parens. `**` is right-associative — there it's the LEFT
+			// operand that must keep them.
+			if (parent.operator === '**' ? parent.left === node : parent.right === node) {
+				return true;
+			}
+		}
 	}
 
 	return false;

package/src/index.test.js

@@ -2285,6 +2285,42 @@ const program =
 			expect(result).toBeWithNewline(expected);
 		});
 
+		it('should keep parens around the right operand of a same-operator subtraction', async () => {
+			const expected = `const d = a - (b - c);`;
+
+			const result = await format(expected, { singleQuote: true, printWidth: 100 });
+			expect(result).toBeWithNewline(expected);
+		});
+
+		it('should keep parens around the right operand of a same-operator division', async () => {
+			const expected = `const d = a / (b / c);`;
+
+			const result = await format(expected, { singleQuote: true, printWidth: 100 });
+			expect(result).toBeWithNewline(expected);
+		});
+
+		it('should keep parens around a right-side addition under string concatenation', async () => {
+			const expected = `const s = 'x' + (n + 1);`;
+
+			const result = await format(expected, { singleQuote: true, printWidth: 100 });
+			expect(result).toBeWithNewline(expected);
+		});
+
+		it('should drop redundant parens around the left operand of a same-operator addition', async () => {
+			const input = `const s = (a + b) + c;`;
+			const expected = `const s = a + b + c;`;
+
+			const result = await format(input, { singleQuote: true, printWidth: 100 });
+			expect(result).toBeWithNewline(expected);
+		});
+
+		it('should keep parens around the left operand of exponentiation', async () => {
+			const expected = `const p = (a ** b) ** c;`;
+
+			const result = await format(expected, { singleQuote: true, printWidth: 100 });
+			expect(result).toBeWithNewline(expected);
+		});
+
 		it('should have parents around low-precedence logical expression', async () => {
 			const input = `files = [...files ?? [], ...dt.files];
 files = [...(files ?? []), ...dt.files];`;