(fix) properly format nested array patterns

Fixes sveltejs#259
dummdidumm · Nov 14, 2021 · 6ae9053 · 6ae9053
1 parent f5e5c4f
commit 6ae9053
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,7 @@
 
 * (feat) Support `bracketSameLine` option and deprecate `svelteBracketNewLine` ([#251](https://github.com/sveltejs/prettier-plugin-svelte/pull/251))
 * (fix) Prevent script/style contents from being erased in certain cases ([#255](https://github.com/sveltejs/prettier-plugin-svelte/issues/255))
+* (fix) Properly format nested array patterns ([#259](https://github.com/sveltejs/prettier-plugin-svelte/issues/259))
 
 ## 2.4.0
 

diff --git a/src/print/index.ts b/src/print/index.ts
@@ -1066,7 +1066,7 @@ function expandNode(node: any): string {
         case 'ObjectPattern':
             return ' {' + node.properties.map(expandNode).join(',') + ' }';
         case 'Property':
-            if (node.value.type === 'ObjectPattern') {
+            if (node.value.type === 'ObjectPattern' || node.value.type === 'ArrayPattern') {
                 return ' ' + node.key.name + ':' + expandNode(node.value);
             } else if (node.value.type === 'Identifier' && node.key.name !== node.value.name) {
                 return expandNode(node.key) + ':' + expandNode(node.value);

diff --git a/test/printer/samples/nested-destructuring.html b/test/printer/samples/nested-destructuring.html
@@ -0,0 +1,7 @@
+{#await thePromise}
+    <p>loading...</p>
+{:then { data: [theValue] }}
+    <p>the value is {theValue}</p>
+{:catch { data: [{ data: [theError] }] }}
+    <p>oh no! {theError.message}</p>
+{/await}