[sebxt]

With condenseFlow: true, stringify() wrapped every flow mapping key in double quotes, including keys that were already serialized as quoted scalars. { "1": 1 } became {"'1'":1}, which parses back as { "'1'": 1 }, and keys such as a"b or a\nb produced output that parse() rejects. Plain keys were wrapped without escaping, so a plain key containing a backslash changed meaning, and the ? marker for keys over 1024 characters ended up inside the quotes.

This change leaves keys that are already quoted as they are, escapes plain keys before wrapping them in double quotes, and keeps the ? marker outside the key.

Added a test covering the affected keys and a round trip through parse(). deno task test for yaml and deno fmt --check / deno lint pass.

Fixes #7362

[CLAassistant]

All committers have signed the CLA.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.06%. Comparing base (f834d02) to head (9cece57).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7363      +/-   ##
==========================================
+ Coverage   95.04%   95.06%   +0.02%     
==========================================
  Files         619      618       -1     
  Lines       52012    51763     -249     
  Branches     9450     9407      -43     
==========================================
- Hits        49434    49208     -226     
+ Misses       2031     2014      -17     
+ Partials      547      541       -6     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.