I reject the extension if I see the optional restore on disable, because that can fall under selective disable which can bypass the GNOME Shell re-basing. (we have a rule for that).
That’s why this line won’t get approved:
// Only disable the extension fully if it is not just being rebased
if (state === Utils.ExtensionState.DISABLING) {
For extensions overriding the same method or property:
IMO, conflict will be inevitable and that issue should be solved between two extensions rather than on review.