Я вас понимаю, и я тоже считаю, что это одна из самых серьезных технических проблем, связанных с «упаковкой плагинов в ядро». У нас было несколько форков плагинов, и их было очень сложно заставить работать без удаления встроенного плагина.
Я не думаю, что Джей это предлагает. Плагин также может переопределять очень конкретные части другого плагина.
Лучший подход — убедить команду, что ваш код стоит включить в официальный плагин. Это сработает, если ваше изменение достаточно универсально или гибко. Я вижу, что вы уже сделали форк, и ваши изменения/добавления выглядят довольно чисто. Возможно, закодированную строку “Flagged” можно вынести в файл переводов, и если вы установите :flagged по умолчанию в false, то вам не придется модифицировать оригинальный обработчик событий с дополнительным параметром, но помимо этого, всё выглядит достойно. Если бы я был на вашем месте, я бы привёл код в актуальное состояние, открыл PR и обсудил это в теме плагина.
Если этот путь не сработает, вы можете просто создать плагин, который переопределяет эти три изменённые вами функции и добавляет обработчик on(:reviewable_created).