String folding of certain folded block scalar values is ugly

[havran]

Describe the bug
Hi. Seems like for some specific multiline strings, output of stringify() is formatted invalid.

To Reproduce

Example:

console.log(
  stringify({
    'nginx.ingress.kubernetes.io/configuration-snippet':
      'location ~* ^/sites/aaaaaaa.aa/files/(.+) {\n  return 302 https://process.aaaaaaa.aa/sites/aaaaaaa.aa/files/$1;\n}\nlocation ~* ^/partner-application/cls/(.+) {\n  return 301 https://process.aaaaaaa.aa/partner-application/cls/$1$is_args$args;\n}\n',
  })
);

Output:

nginx.ingress.kubernetes.io/configuration-snippet: >
  location ~*
  ^/sites/aaaaaaa.aa/files/(.+) {
    return 302 https://process.aaaaaaa.aa/sites/aaaaaaa.aa/files/$1;
  }

  location ~* ^/partner-application/cls/(.+) {
    return 301 https://process.aaaaaaa.aa/partner-application/cls/$1$is_args$args;
  }

Expected behaviour

Expected output:

nginx.ingress.kubernetes.io/configuration-snippet: >
  location ~* ^/sites/aaaaaaa.aa/files/(.+) {
    return 302 https://process.aaaaaaa.aa/sites/aaaaaaa.aa/files/$1;
  }
  location ~* ^/partner-application/cls/(.+) {
    return 301 https://process.aaaaaaa.aa/partner-application/cls/$1$is_args$args;
  }

Versions (please complete the following information):

Node:

$ node --version
v16.18.1

Yaml: 2.1.3

Note: edited and site name was redacted. I try use example.com but in this case there was expected output. Probably problem is related with long lines?

[eemeli]

It does look like the string folding is a bit buggy there. The value represented by the stringified result is valid and does match the source, but there's no good reason why that first line should be folded like that.

[havran]

Sorry, I do not notice, there are option related with lineWidth, and this fix my problem. Closing :-)

console.log(
  stringify(
    {
      'nginx.ingress.kubernetes.io/configuration-snippet':
        'location ~* ^/sites/aaaaaaa.aa/files/(.+) {\n  return 302 https://process.aaaaaaa.aa/sites/aaaaaaa.aa/files/$1;\n}\nlocation ~* ^/partner-application/cls/(.+) {\n  return 301 https://process.aaaaaaa.aa/partner-application/cls/$1$is_args$args;\n}\n',
    },
    { blockQuote: 'literal', lineWidth: 0 }
  )
);

Output is OK now:

nginx.ingress.kubernetes.io/configuration-snippet: |
  location ~* ^/sites/aaaaaaa.aa/files/(.+) {
    return 302 https://process.aaaaaaa.aa/sites/aaaaaaa.aa/files/$1;
  }
  location ~* ^/partner-application/cls/(.+) {
    return 301 https://process.aaaaaaa.aa/partner-application/cls/$1$is_args$args;
  }

Thank you!

[eemeli]

No, this is still a bug that should be fixed. The > style folding of your input is bad, even if it is valid.

[havran]

True, I see it now. Happy to found new bug :-) blockQuote: literal + lineWidth: 0 solve this for me. Thanks again.