MergeYaml can not parse valid yaml

[dpozinen]

When using MergeYaml with @:@ in the value open rewrite cannot parse the yaml, even though it is valid,
it appears this is because of changes introduced in 8b705c3
specifically this regex:

rewrite/rewrite-yaml/src/main/java/org/openrewrite/yaml/YamlParser.java

Line 55 in 5f2d11c

private static final Pattern VARIABLE_PATTERN = Pattern.compile(":\\s*(@[^\n\r@]+@)");

which causes UUID replacements

Here's a test to reproduce:

rewriteRun(
    spec -> spec.recipe(new MergeYaml(
      "$",
      """
          container:
              docker-image-name: "${@project.groupId@/@service.name@:@project.version@}"
              """,
      null, null,null)
    ),
    yaml(
      """
          container:
              a: b
      """,
      """
          container:
              a: b
              docker-image-name: "${@project.groupId@/@service.name@:@project.version@}"
      """
    )
  );

results in:

java.lang.ClassCastException: class org.openrewrite.tree.ParseError cannot be cast to class org.openrewrite.yaml.tree.Yaml$Documents (org.openrewrite.tree.ParseError and org.openrewrite.yaml.tree.Yaml$Documents are in unnamed module of loader 'app')

[timtebeek]

Thanks for providing that context link and a runnable example; did you already explore any possible fix?

[dpozinen]

not really, i messed around a bit with the regex, I might have a look more today