fix(mf2): When serialising, escape literal {} in quoted patterns

XM5jDcsHTyGJtQqlCi · XM5jDcsHTyGJtQqlCi · commit 46331625301a · 2025-05-14T20:09:32.000+03:00
diff --git a/mf2/messageformat/src/data-model/stringify.ts b/mf2/messageformat/src/data-model/stringify.ts
@@ -103,18 +103,13 @@ function stringifyPattern(pattern: Pattern, quoted: boolean) {
     quoted = true;
   }
   for (const el of pattern) {
-    if (typeof el === 'string') res += stringifyString(el, quoted);
+    if (typeof el === 'string') res += el.replace(/[\\{}]/g, '\\$&');
     else if (el.type === 'markup') res += stringifyMarkup(el);
     else res += stringifyExpression(el);
   }
   return quoted ? `{{${res}}}` : res;
 }
 
-function stringifyString(str: string, quoted: boolean) {
-  const esc = quoted ? /[\\|]/g : /[\\{}]/g;
-  return str.replace(esc, '\\$&');
-}
-
 function stringifyExpression({ arg, attributes, functionRef }: Expression) {
   let res: string;
   switch (arg?.type) {

Original file line number	Diff line number	Diff line change
`@@ -103,18 +103,13 @@ function stringifyPattern(pattern: Pattern, quoted: boolean) {`
`103`	`103`	`quoted = true;`
`104`	`104`	`}`
`105`	`105`	`for (const el of pattern) {`
`106`		`- if (typeof el === 'string') res += stringifyString(el, quoted);`
	`106`	`+ if (typeof el === 'string') res += el.replace(/[\\{}]/g, '\\$&');`
`107`	`107`	`else if (el.type === 'markup') res += stringifyMarkup(el);`
`108`	`108`	`else res += stringifyExpression(el);`
`109`	`109`	`}`
`110`	`110`	return quoted ? `{{${res}}}` : res;
`111`	`111`	`}`
`112`	`112`
`113`		`-function stringifyString(str: string, quoted: boolean) {`
`114`		`- const esc = quoted ? /[\\\|]/g : /[\\{}]/g;`
`115`		`- return str.replace(esc, '\\$&');`
`116`		`-}`
`117`		`-`
`118`	`113`	`function stringifyExpression({ arg, attributes, functionRef }: Expression) {`
`119`	`114`	`let res: string;`
`120`	`115`	`switch (arg?.type) {`