YamlNodeSerializer corrupts data due to implicit types conversion

[eugene-krivobokov]

Describe the bug

Hi! YamlNode loses data and types due to implicit automatic conversion to numbers.

"+44123456" -> 44123456

Reproduction repo

N/A

Steps to reproduce

@Test
fun YamlNode_reproducer() {
    // https://yaml.org/spec/1.2.2/#10113-generic-string
    val originalYaml = """|"user":
                      |    "phone_with_prefix_double_quote": "+44123456"
                      |    "phone_with_prefix_single_quote": '+44123456'
                      |    "phone_with_prefix_str_tag": !!str "+44123456"
                      |    "phone_without_prefix": "44123456"
                      |""".trimMargin()

    val yaml = Yaml(
        configuration = YamlConfiguration(
            singleLineStringStyle = SingleLineStringStyle.DoubleQuoted,
            ambiguousQuoteStyle = AmbiguousQuoteStyle.DoubleQuoted,
        )
    )
    val yamlNode = yaml.parseToYamlNode(originalYaml)

    val encodedYaml = yaml.encodeToString(yamlNode)

    // "phone_with_prefix_double_quote": 44123456
    // "phone_with_prefix_single_quote": 44123456
    // "phone_with_prefix_str_tag": !!str '44123456'
    // "phone_without_prefix": 44123456
    expect(encodedYaml).toEqual(originalYaml)
}

Expected behaviour

Quoted strings kept as is, not losing content,
or !!str tag allows you to enforce it.

Actual behaviour

1. Type is lost (quoted string becomes a number).
2. String's content is modified

Version information

0.92.0

Any other information

No response

[charleskorn]

Thanks for the bug report. I don't have time to fix this any time soon, so PRs are welcome 🙂

[Tomas-Jakubec]

@charleskorn I have prepared a PR with a fix, can you look at it, please?