Also indent top-level scalars with lines starting with % directives (#…

…162)
eemeli · May 9, 2020 · 3900895 · 3900895
1 parent 0426e4a
commit 3900895
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/stringify/stringifyString.js b/src/stringify/stringifyString.js
@@ -14,7 +14,9 @@ const getFoldOptions = ({ indentAtStart }) =>
     ? Object.assign({ indentAtStart }, strOptions.fold)
     : strOptions.fold
 
-const containsDocumentMarker = str => /^(---|\.\.\.)/m.test(str)
+// Also checks for lines starting with %, as parsing the output as YAML 1.1 will
+// presume that's starting a new document.
+const containsDocumentMarker = str => /^(%|---|\.\.\.)/m.test(str)
 
 function lineLengthOverLimit(str, limit) {
   const strLen = str.length

diff --git a/tests/doc/stringify.js b/tests/doc/stringify.js
@@ -671,4 +671,10 @@ describe('Document markers in top-level scalars', () => {
     expect(str).toBe('"foo\n\n  ..."\n')
     expect(YAML.parse(str)).toBe('foo\n...')
   })
+
+  test('foo\\n%bar\\n', () => {
+    const str = YAML.stringify('foo\n%bar\n')
+    expect(str).toBe('|\n  foo\n  %bar\n')
+    expect(YAML.parse(str)).toBe('foo\n%bar\n')
+  })
 })