Good catch!

The emit_doc optional argument looks good, but the with_doc argument is not necessary: the doc property should always be cloned (copied) from the original node. This will simplify code a lot, and doesn't involve any performance or memory penalty.

@asterite Thanks! I'll fix it.

@makenowjust Great, thank you!! :-)

Now that I think about it, the comments are probably missing too when you declare a type inside record, or well you use record inside record with comment:

record Foo do
  # Bar comment
  record Bar

  # Baz comment
  class Baz
  end
end

This is because ClassDef, ModuleDef, EnumDef and Call also have a doc property, so this logic should probably be applied to those too. I can merge this PR and we (either me, you or somebody else) can do it later, or you can do it all in this PR? What do you prefer?

@asterite I think it is no problem. Because doc property is copied by ASTNode#clone, not Def#clone_without_location, so we can copy doc every AST nodes.

@makenowjust ありがとう！

@asterite Muchas gracias