fix(mf2): Escape \ { | } when stringifying messages

XM5jDcsHTyGJtQqlCi · XM5jDcsHTyGJtQqlCi · commit dfe54382c212 · 2024-08-20T20:34:19.000+03:00
diff --git a/packages/mf2-messageformat/src/data-model/stringify.ts b/packages/mf2-messageformat/src/data-model/stringify.ts
@@ -109,13 +109,18 @@ function stringifyPattern(pattern: Pattern, quoted: boolean) {
     quoted = true;
   }
   for (const el of pattern) {
-    if (typeof el === 'string') res += el;
+    if (typeof el === 'string') res += stringifyString(el, quoted);
     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, annotation, attributes }: Expression) {
   let res: string;
   switch (arg?.type) {

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