@ripple-ts/prettier-plugin 0.2.172 → 0.2.174

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ripple-ts/prettier-plugin",
-  "version": "0.2.172",
+  "version": "0.2.174",
   "description": "Ripple plugin for Prettier",
   "type": "module",
   "module": "src/index.js",
@@ -25,7 +25,7 @@
   },
   "devDependencies": {
     "prettier": "^3.6.2",
-    "ripple": "0.2.172"
+    "ripple": "0.2.174"
   },
   "dependencies": {},
   "files": [

package/src/index.js

@@ -1400,7 +1400,15 @@ function printRippleNode(node, path, options, print, args) {
 
 		case 'SpreadElement': {
 			const argumentDoc = path.call(print, 'argument');
-			nodeContent = concat(['...', argumentDoc]);
+			// Wrap argument in parens if it's a low-precedence logical expression (e.g., nullish coalescing)
+			// that needs them for correct parsing
+			const needsParens =
+				node.argument.type === 'LogicalExpression' && node.argument.operator === '??';
+			if (needsParens) {
+				nodeContent = concat(['...(', argumentDoc, ')']);
+			} else {
+				nodeContent = concat(['...', argumentDoc]);
+			}
 			break;
 		}
 		case 'RestElement': {
@@ -3003,17 +3011,6 @@ function printObjectExpression(node, path, options, print, args) {
 				// 1-property objects: force inline with spaces
 				return concat([open_brace, ' ', properties[0], ' ', '}']);
 			}
-			// 2-property objects: let normal formatting handle it (will be multiline)
-			// Fall through to default multiline formatting below
-		} else {
-			// For attributes, force inline without spaces
-			const parts = [open_brace];
-			for (let i = 0; i < properties.length; i++) {
-				if (i > 0) parts.push(', ');
-				parts.push(properties[i]);
-			}
-			parts.push('}');
-			return concat(parts);
 		}
 	}
 

package/src/index.test.js

@@ -878,6 +878,22 @@ export component Test({ a, b }: Props) {}`;
 			expect(result).toBeWithNewline(expected);
 		});
 
+		it('should format object in attribute with spaces at each side', async () => {
+			const input = `component App() {
+  <button
+  class="test another"
+  onClick={{handleEvent: handler}}>{'Click Me'}</button>
+}`;
+			const expected = `component App() {
+  <button class="test another" onClick={{ handleEvent: handler }}>
+    {'Click Me'}
+  </button>
+}`;
+
+			const result = await format(input, { singleQuote: true });
+			expect(result).toBeWithNewline(expected);
+		});
+
 		it('should not format function parameter spread', async () => {
 			const expected = `component Two({ arg1, ...rest }) {}`;
 
@@ -1691,6 +1707,16 @@ const program =
 			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];`;
+			const expected = `files = [...(files ?? []), ...dt.files];
+files = [...(files ?? []), ...dt.files];`;
+
+			const result = await format(input, { singleQuote: true, printWidth: 100 });
+			expect(result).toBeWithNewline(expected);
+		});
 	});
 
 	describe('edge cases', () => {
@@ -2787,9 +2813,9 @@ component RowList({ rows, Row }) {
 
 			const expected = `export component App() {
   <div>
-    <RowList rows={#[{id: 'a'}, {id: 'b'}, {id: 'c'}]}>
+    <RowList rows={#[{ id: 'a' }, { id: 'b' }, { id: 'c' }]}>
       component Row({ id, index, isHighlighted = (index) => index % 2 === 0 }) {
-        <div class={{highlighted: isHighlighted(index)}}>
+        <div class={{ highlighted: isHighlighted(index) }}>
           {index}
           {' - '}
           {id}