fix: Do not require quotes for implicit keys with flow indicators (#494)

eemeli · Aug 28, 2023 · 9558e96 · 9558e96
1 parent 07063bc
commit 9558e96
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/stringify/stringifyString.ts b/src/stringify/stringifyString.ts
@@ -280,7 +280,7 @@ function plainString(
   const { type, value } = item
   const { actualString, implicitKey, indent, indentStep, inFlow } = ctx
   if (
-    (implicitKey && /[\n[\]{},]/.test(value)) ||
+    (implicitKey && value.includes('\n')) ||
     (inFlow && /[[\]{},]/.test(value))
   ) {
     return quotedString(value, ctx)

diff --git a/tests/doc/stringify.ts b/tests/doc/stringify.ts
@@ -336,6 +336,11 @@ z:
     expect(String(doc)).toBe('a: 1\n? b: 2\n  c: 3\n')
   })
 
+  test('plain key containing flow indicators', () => {
+    const doc = YAML.parseDocument('foo[{}]: bar')
+    expect(doc.toString()).toBe('foo[{}]: bar\n')
+  })
+
   describe('No extra whitespace for empty values', () => {
     const getDoc = () =>
       new YAML.Document<YAML.YAMLMap<YAML.Scalar, YAML.Scalar>, false>({