Ich kann das nachvollziehen und habe dies auch als eine der wirkungsvollsten technischen Komplikationen des „Bündelns von Plugins mit dem Kern“ empfunden. Wir hatten einige geforkte Plugins, und es war sehr kompliziert, sie zum Laufen zu bringen, ohne das gebündelte Plugin zu entfernen.
Ich glaube nicht, dass Jay das vorschlägt. Ein Plugin kann auch sehr spezifische Teile eines anderen Plugins überschreiben.
Der beste Ansatz wäre, das Team davon zu überzeugen, dass Ihr Code es wert ist, in das offizielle Plugin integriert zu werden. Das funktioniert, wenn Ihre Änderung generisch oder flexibel genug ist. Ich sehe, Sie haben bereits einen Fork erstellt, und Ihre Änderungen/Ergänzungen sind ziemlich sauber. Vielleicht könnte der fest codierte String „Flagged“ in einer Übersetzungsdatei stehen, und wenn Sie :flagged standardmäßig auf false setzen, müssen Sie den ursprünglichen Event-Handler nicht mit einem zusätzlichen Parameter ändern, aber abgesehen davon sieht es lohnenswert aus. Wenn ich Sie wäre, würde ich es auf den neuesten Stand bringen, einen PR öffnen und dies im Plugin-Thema besprechen.
Wenn dieser Weg fehlschlägt, könnten Sie einfach ein Plugin erstellen, das diese drei Funktionen, die Sie geändert haben, überschreibt und den on(:reviewable_created)-Handler hinzufügt.