fix: use the lineWidth option for line breaking in flow collection

This should address issues eemeli#288 and eemeli#507.
no92 · Feb 12, 2024 · 9ce4e91 · 9ce4e91
1 parent b7696fc
commit 9ce4e91
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/stringify/stringifyCollection.ts b/src/stringify/stringifyCollection.ts
@@ -148,7 +148,7 @@ function stringifyFlowCollection(
   } else {
     if (!reqNewline) {
       const len = lines.reduce((sum, line) => sum + line.length + 2, 2)
-      reqNewline = len > Collection.maxFlowStringSingleLineLength
+      reqNewline = ctx.options.lineWidth !== 0 && len > ctx.options.lineWidth
     }
     if (reqNewline) {
       str = start

diff --git a/tests/doc/stringify.ts b/tests/doc/stringify.ts
@@ -616,7 +616,7 @@ describe('scalar styles', () => {
   y,
   n
 ]\n`
-    expect(String(doc)).toBe(str)
+    expect(doc.toString({ lineWidth: 1 })).toBe(str)
     expect(YAML.parse(str)).toEqual([
       true,
       false,