Entiendo, y he encontrado que esta es una de las complicaciones técnicas más impactantes de “incluir complementos con el núcleo” también. Teníamos algunos complementos bifurcados y era muy complicado hacer que funcionaran sin eliminar el complemento incluido.
No creo que Jay esté sugiriendo eso. Un complemento también puede anular partes muy específicas de otro complemento.
El mejor enfoque sería convencer al equipo de que su código merece ser fusionado con el complemento oficial. Eso funcionará si su modificación es lo suficientemente genérica o flexible. Veo que ya ha hecho una bifurcación y sus cambios/adiciones son bastante limpios. Quizás la cadena “Flagged” codificada podría estar en un archivo de traducción y si hace que :flagged sea falso por defecto, no tendrá que modificar el controlador de eventos original con un parámetro adicional, pero aparte de eso, parece valioso. Si yo fuera usted, lo pondría al día, abriría un PR y discutiría esto en el tema del complemento.
Si esa vía falla, podría simplemente crear un complemento que anule esas tres funciones que cambió y agregue el manejador on(:reviewable_created).