discovery: ensure that missingheads are always heads of everything we tried This only alter the third block executed when: * repo have secret changeset * onlyheads is not None In other case findcommonoutgoing already had this behavior

phase: extracts heads computation logics from analyzeremotephases

phase: fix warning text from invalid remote phase add missing end of line and substitution.