python-code-review

Run a structured review checking line length, indentation, imports, naming conventions, and whitespace rules against PEP 8. This Skill walks through each category systematically and reports violations with line numbers and corrected code examples.

PEP 8 is Python's official style guide covering formatting and naming basics. The Google Python Style Guide builds on it with stricter rules for docstrings, type hints, imports, and practices like avoiding mutable default arguments and staticmethods.

Look for SQL injection via string-formatted queries, hardcoded credentials, unsafe calls like eval() and exec(), and unsafe deserialization with pickle or yaml.load(). The review flags each issue with severity and a parameterized or safer alternative.

pylint with Google's pylintrc configuration provides comprehensive linting, while Black or Pyink handle auto-formatting. Complement them with pytype or mypy for type checking and isort for import ordering.

Consistency within a project takes priority over rigid rule adherence. Violations are acceptable when following the rule reduces readability, when matching surrounding legacy code, or when compliance would break backwards compatibility.