Use of the patch decorator

[the-sean-c]

Would it be worth discussing the use (or maybe just overuse) of the patch decorator?

I can see that it might have uses in some cases, but I can't help but feel like it adds a disorganized dimension to some code that makes it harder to grok. For example, fasthtml/oauth.py:

• WebApplicationClient is subclassed as _AppClient, which is then inherited by the other app clients. Instead of adding methods with patch, why not just put the methods directly into _AppClient?
• GoogleAppClient has a method creds patched in at the bottom of the script. Why not just add it directly in the GoogleAppClient class?

Particularly, it feels to me like patching classes that are defined within the same file just a few lines above might be overuse of a patch decorator.