Consider using a consistent type for diff_id throughout the PhabricatorPatch class

Consider using a consistent type for diff_id throughout the class. Currently it accepts both str and int but the diff_id property always returns int, which could cause confusion.

Originally posted by @Copilot in https://github.com/mozilla/bugbug/pull/5303#discussion_r2400596679