This is neat, though some of it (mutable defaults for example) are already covered by existing linters. I’d also look into pre-commit support as well.

How did you decide on the patterns to check?

why