And some improvements:
Remove unused code.
We cannot rely on python-yaml being available, I don't want users to install python modules just to run include-fixer. Can we restrict ourselves to the JSON subset of YAML here?
Get rid of yaml dependency, using json module.
Show error message when clang-include-fixer died with a fatal error.
Can you just llvm::yaml::escape the string?
Use llvm::yaml::escape to escape double quote.
I think we should do the escaping always and not just when the first character is a quote.
Always escape headers.