Eu concordo, e descobri que esta é uma das complicações técnicas mais impactantes do “empacotamento de plugins com o núcleo” também. Tínhamos alguns plugins ramificados (forked) e era muito complicado fazê-los funcionar sem remover o plugin empacotado.
Eu não acho que Jay esteja sugerindo isso. Um plugin também pode sobrescrever partes muito específicas de outro plugin.
A melhor abordagem seria convencer a equipe de que seu código vale a pena ser mesclado (merged) no plugin oficial. Isso funcionará se sua modificação for genérica ou flexível o suficiente. Vejo que você já criou um fork e suas alterações/adições estão bem limpas. Talvez a string “Flagged” codificada pudesse estar em um arquivo de tradução e, se você fizer com que :flagged seja false por padrão, você não precisará modificar o manipulador de eventos original com um parâmetro extra, mas fora isso, parece válido. Se eu fosse você, eu o atualizaria, abriria um PR e discutiria isso no tópico do plugin.
Se esse caminho falhar, você pode simplesmente criar um plugin que sobrescreva as três funções que você alterou e adicione o manipulador on(:reviewable_created).