Merge pull request #21940 from Amit5601/fix-vscode-snippets

ChayimFriedman2 · web-flow · commit 6bd50efa46c9 · 2026-04-05T19:15:45.000Z
fix: support multiple snippet placeholders in VS Code extension
diff --git a/editors/code/src/snippets.ts b/editors/code/src/snippets.ts
@@ -53,7 +53,7 @@ export async function applySnippetTextEdits(editor: vscode.TextEditor, edits: vs
 }
 
 function hasSnippet(snip: string): boolean {
-    const m = snip.match(/\$\d+|\{\d+:[^}]*\}/);
+    const m = snip.match(/\$\d+|\$\{\d+:[^}]*\}/);
     return m != null;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ export async function applySnippetTextEdits(editor: vscode.TextEditor, edits: vs`
`53`	`53`	`}`
`54`	`54`
`55`	`55`	`function hasSnippet(snip: string): boolean {`
`56`		`- const m = snip.match(/\$\d+\|\{\d+:[^}]*\}/);`
	`56`	`+ const m = snip.match(/\$\d+\|\$\{\d+:[^}]*\}/);`
`57`	`57`	`return m != null;`
`58`	`58`	`}`
`59`	`59`