It looks it's first step of refactoring. Now it's easy to see:
-- channel_id at Client.fetch_peers uses only in one case but calculates always. Maybe it would be good idea to put its calculation exactly there where it used
-- maybe instead of getattr(..., ..., None) or getattr(..., ..., None) it would be good idea to use getattr(..., ..., getattr(..., ..., None))
-- maybe invert getattr action order
-- maybe extract attr via type checking

I found patchy so I'm solving my issues
@delivrance you may close this PR. It change nothing in behavior

I have some ideas, so I'll implement it, push it and then remove WIP status