kata-python-review

Review Python pull requests for correctness, safety, and maintainability.

Updated Sep 3, 2025
One-click install
npx skills add https://github.com/guardiatechnology/design-system --skill kata-python-review
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: kata-python-review
Source: https://github.com/guardiatechnology/design-system/tree/main/.claude/skills/kata-python-review
Command: npx skills add https://github.com/guardiatechnology/design-system --skill kata-python-review

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

It helps teams catch bugs, security issues, missing tests, and architectural violations early by enforcing a repeatable checklist for Python pull requests.

Core Features & Use Cases

  • Correctness & edge cases: validates logic against the PR intent and ensures edge paths are handled.
  • Type safety & boundary validation: checks typing discipline and whether validation happens at inputs/outputs.
  • Quality gates for testing & security: assesses test coverage quality and ensures secrets, injection, and sensitive data exposure are not introduced.
  • Error handling & architecture compliance: reviews exception specificity, logging/context, and dependency direction within the codebase.
  • Use Case: When a backend PR changes core business logic or request/DB boundaries, use this skill to produce a structured review that clarifies must-fix critical issues and optional improvements.

Quick Start

Ask the AI to perform a Python PR code review using the checklist from "Kata: Python Code Review" and return a structured result with summary, critical issues, suggestions, and positive notes.

Frequently Asked Questions about kata-python-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I systematically review Python pull requests for code quality and security?

To systematically review Python pull requests, apply a repeatable checklist that validates correctness, edge cases, type safety, test adequacy, and secure handling of inputs. This approach catches bugs and architectural violations early while producing a structured approval or changes request.

What should a Python PR code review checklist cover for backend engineering?

A Python PR code review checklist for backend engineering should cover domain logic correctness, validation boundaries, typing discipline, test coverage quality, security-sensitive inputs, error handling paths, exception specificity, and dependency direction for architecture compliance.

How can I check type safety and boundary validation during a Python code review?

Check type safety and boundary validation by verifying typing discipline is enforced and confirming that validation happens exactly at input and output boundaries. This ensures data integrity before it reaches core domain logic or database layers.

Does automated Python code review work for finding missing tests and security issues?

Automated Python code review works for finding missing tests and security issues by assessing test coverage quality and ensuring secrets, injection vulnerabilities, and sensitive data exposure are not introduced across security-sensitive inputs and error handling paths.

How do I ensure error handling and architecture compliance in a Python pull request?

Ensure error handling and architecture compliance by reviewing exception specificity, verifying logging and context are appropriate, and checking that dependency direction stays correct within the codebase to prevent architectural violations in backend changes.

What is the best way to structure feedback when reviewing Python PRs?

The best way to structure feedback when reviewing Python PRs is to produce a structured result containing a summary, critical must-fix issues, optional improvement suggestions, and positive notes, clarifying exactly what must change before approval.