Entendi que essa ferramenta, que não conheço, gera código ruim, e por isso mesmo deveria procurar outra solução.
Deve existir um motivo para serem geradas no build e não deixar disponível no projeto e é justamente não deixar mexer nele. Mais ainda, é provável que a mudança ocorra com frequência, então mexer no código não faz sentido, vai perder tudo ou no próximo build, ou quando precisar uma mudança. Esses códigos gerados são feitos para não mexer.
Subverter isso parece bastante falta de bom senso. Se não confia na ferramenta, não a use. Sou favorável ao uso de scaffolding, mas precisa ser a ferramenta certa e usada da forma correta.
Inverta a situação. Tudo o que pode gerar um esforço deve ser justificado. Peça para eles argumentarem o porquê de fazer isso. Se for fazer por fazer não é um bom argumento. Se for para melhorar ou personalizar o código, já tem os argumentos contrários ditos acima. Se for outra coisa precisa procurar um motivo para contra-argumentar, mas eu acho que fica nesses dois que eu disse.
Boa ou má prática é argumento de quem não tem argumento. Não precisa existir uma má prática, precisa existir um bom motivo para fazer algo e ele não pode trazer prejuízos indesejáveis. Tem que analisar tecnicamente. Má prática é evitar análise e assumir o que outra pessoa disse sem conhecer o seu contexto.
C# tem até classes parciais para que o código gerado antes do build faça parte do projeto e ao mesmo tempo fique protegido de mudanças, se tudo estiver bem estruturado.