# Verifying Python changes Files: `.py`, `.pyi`. Derived from 64 scanned source repositories and 707 review prompts on https://awesomereviewers.com. Page: https://awesomereviewers.com/verify/python/ How to use this: run every check under *Run on every change*. Then, for each section under *Add when the change touches* that describes your diff, run its checks too. Checks catch what a tool can find; then apply the listed review prompts for what tools miss (fetch each prompt's raw URL). Usage is the share of scanned repositories in this language that run the tool. A missing tool means no scanned repository runs one the scanner recognises: fall back to the prompts. ## Run on every change - **Formatting**: `ruff-format` (50% of repos). Command seen: `ruff format .`. Autofixes. Exit codes: `0` already formatted; `1` files would be reformatted (--check); `2` error. Alternatives: black (14%), isort (12%). Docs: https://docs.astral.sh/ruff/formatter/#exit-codes - **Linting**: `ruff-check` (72% of repos). Command seen: `ruff check .`. Autofixes. Exit codes: `0` no violations; `1` violations found; `2` invalid configuration, CLI options or internal error. Alternatives: flake8 (20%), pylint (12%). Docs: https://docs.astral.sh/ruff/linter/#exit-codes - **Type checking**: `mypy` (42% of repos). Command seen: `mypy`. Exit codes: `0` no type errors; `1` type errors found; `2` crash or invalid invocation. Alternatives: pyright (17%). Docs: https://mypy.readthedocs.io/en/stable/command_line.html - **Unit and integration tests**: `pytest` (86% of repos). Command seen: `pytest`. Exit codes: `0` all tests passed; `1` some tests failed; `2` interrupted by the user; `3` internal error; `4` command line usage error; `5` no tests were collected. Alternatives: unittest (12%). Docs: https://docs.pytest.org/en/stable/reference/exit-codes.html Review prompts (111 in total, most discussed first): - [Use descriptive names](https://awesomereviewers.com/raw/sdk-python-use-descriptive-names.md) — strands-agents/sdk-python, Naming Conventions - [Consistent module organization](https://awesomereviewers.com/raw/heretic-consistent-module-organization.md) — p-e-w/heretic, Code Style - [Avoid code duplication](https://awesomereviewers.com/raw/litellm-avoid-code-duplication.md) — BerriAI/litellm, Code Style - [eliminate unnecessary code](https://awesomereviewers.com/raw/unstructured-eliminate-unnecessary-code.md) — Unstructured-IO/unstructured, Code Style - [Readable formatting and idioms](https://awesomereviewers.com/raw/celery-readable-formatting-and-idioms.md) — celery/celery, Code Style - [eliminate code duplication](https://awesomereviewers.com/raw/sglang-eliminate-code-duplication.md) — sgl-project/sglang, Code Style - [Use semantically clear names](https://awesomereviewers.com/raw/unstructured-use-semantically-clear-names.md) — Unstructured-IO/unstructured, Naming Conventions - [Eliminate redundant constructs](https://awesomereviewers.com/raw/mxnet-eliminate-redundant-constructs.md) — apache/mxnet, Code Style - [Centralize shared utilities](https://awesomereviewers.com/raw/mcp-centralize-shared-utilities.md) — awslabs/mcp, Code Style - [Collision-Safe Identifier Mapping](https://awesomereviewers.com/raw/hermes-agent-collision-safe-identifier-mapping.md) — nousresearch/hermes-agent, Naming Conventions - [Idiomatic, minimal Python constructs](https://awesomereviewers.com/raw/flask-idiomatic-minimal-python-constructs.md) — pallets/flask, Code Style - [Use clear names](https://awesomereviewers.com/raw/heretic-use-clear-names.md) — p-e-w/heretic, Naming Conventions ## Add when the change touches ### Authentication, input handling, secrets, permissions - **Security scanning**: `codeql` (28% of repos). Runs via GitHub Action github/codeql-action/analyze. Alternatives: bandit (9%), semgrep (8%). Docs: https://github.com/github/codeql - **Secret scanning**: `gitleaks` (2% of repos). Runs via pre-commit hook gitleaks. Docs: https://github.com/gitleaks/gitleaks - **Dependency audit**: `dependency-review` (8% of repos). Runs via GitHub Action actions/dependency-review-action. Alternatives: pip-audit (3%). Docs: https://github.com/actions/dependency-review-action Review prompts (48 in total, most discussed first): - [Prevent sensitive data exposure](https://awesomereviewers.com/raw/checkov-prevent-sensitive-data-exposure.md) — bridgecrewio/checkov, Security - [Protect sensitive data](https://awesomereviewers.com/raw/checkov-protect-sensitive-data.md) — bridgecrewio/checkov, Security - [Boundary Hardening Rules](https://awesomereviewers.com/raw/hermes-agent-boundary-hardening-rules.md) — nousresearch/hermes-agent, Security - [Fail-Closed Input Sanitization](https://awesomereviewers.com/raw/mcp-fail-closed-input-sanitization.md) — awslabs/mcp, Security - [Secure Untrusted Boundaries](https://awesomereviewers.com/raw/azure-cli-secure-untrusted-boundaries.md) — Azure/azure-cli, Security - [Trust Boundaries And Safe Parsing](https://awesomereviewers.com/raw/pr-agent-trust-boundaries-and-safe-parsing.md) — the-pr-agent/pr-agent, Security - [Secure Auth and Logging](https://awesomereviewers.com/raw/openai-python-secure-auth-and-logging.md) — openai/openai-python, Security - [Sanitize Untrusted Boundaries](https://awesomereviewers.com/raw/last30days-skill-sanitize-untrusted-boundaries.md) — mvanhorn/last30days-skill, Security - [Centralized input guards](https://awesomereviewers.com/raw/ragflow-centralized-input-guards.md) — infiniflow/ragflow, Security - [Validate user-controlled paths](https://awesomereviewers.com/raw/airflow-validate-user-controlled-paths.md) — apache/airflow, Security - [Least privilege principle](https://awesomereviewers.com/raw/prowler-least-privilege-principle.md) — prowler-cloud/prowler, Security - [Defensive Security Validation](https://awesomereviewers.com/raw/codex-security-defensive-security-validation.md) — openai/codex-security, Security ### Public APIs, signatures, error paths, nullability - **Type checking**: `mypy` (42% of repos). Command seen: `mypy`. Exit codes: `0` no type errors; `1` type errors found; `2` crash or invalid invocation. Alternatives: pyright (17%). Docs: https://mypy.readthedocs.io/en/stable/command_line.html - **Unit and integration tests**: `pytest` (86% of repos). Command seen: `pytest`. Exit codes: `0` all tests passed; `1` some tests failed; `2` interrupted by the user; `3` internal error; `4` command line usage error; `5` no tests were collected. Alternatives: unittest (12%). Docs: https://docs.pytest.org/en/stable/reference/exit-codes.html - **Property-based tests**: `hypothesis` (14% of repos). Docs: https://github.com/HypothesisWorks/hypothesis Review prompts (167 in total, most discussed first): - [Actionable Error Propagation](https://awesomereviewers.com/raw/azure-cli-actionable-error-propagation.md) — Azure/azure-cli, Error Handling - [Avoid silent error masking](https://awesomereviewers.com/raw/mcp-avoid-silent-error-masking.md) — awslabs/mcp, Error Handling - [Exception Handling Policy](https://awesomereviewers.com/raw/pr-agent-exception-handling-policy.md) — the-pr-agent/pr-agent, Error Handling - [Validate Inputs Explicitly](https://awesomereviewers.com/raw/python-validate-inputs-explicitly.md) — TheAlgorithms/Python, Error Handling - [Surface errors clearly](https://awesomereviewers.com/raw/heretic-surface-errors-clearly.md) — p-e-w/heretic, Error Handling - [explicit None handling](https://awesomereviewers.com/raw/unstructured-explicit-none-handling.md) — Unstructured-IO/unstructured, Null Handling - [Deterministic exception handling](https://awesomereviewers.com/raw/celery-deterministic-exception-handling.md) — celery/celery, Error Handling - [Safe null handling](https://awesomereviewers.com/raw/dify-safe-null-handling.md) — langgenius/dify, Null Handling - [Null-safe Defaults](https://awesomereviewers.com/raw/pr-agent-null-safe-defaults.md) — the-pr-agent/pr-agent, Null Handling - [Use Null-Safe Config Reads](https://awesomereviewers.com/raw/hermes-agent-use-null-safe-config-reads.md) — nousresearch/hermes-agent, Null Handling - [Accurate API Contracts](https://awesomereviewers.com/raw/mcp-accurate-api-contracts.md) — awslabs/mcp, API - [Standardize Typed Errors](https://awesomereviewers.com/raw/openai-python-standardize-typed-errors.md) — openai/openai-python, Error Handling ### Service and framework code - **Unit and integration tests**: `pytest` (86% of repos). Command seen: `pytest`. Exit codes: `0` all tests passed; `1` some tests failed; `2` interrupted by the user; `3` internal error; `4` command line usage error; `5` no tests were collected. Alternatives: unittest (12%). Docs: https://docs.pytest.org/en/stable/reference/exit-codes.html - **Version and platform matrix**: `ci-matrix` (56% of repos). Command seen: `analyze (include: 2)`. Docs: https://docs.github.com/en/actions/how-tos/write-workflows/choose-what-workflows-do/run-job-variations Review prompts (6 in total, most discussed first): - [Test Canvas Edge Cases](https://awesomereviewers.com/raw/celery-test-canvas-edge-cases.md) — celery/celery, Celery - [Ensure deterministic resume behavior](https://awesomereviewers.com/raw/sdk-python-ensure-deterministic-resume-behavior.md) — strands-agents/sdk-python, Temporal - [Offload heavy operations](https://awesomereviewers.com/raw/dify-offload-heavy-operations.md) — langgenius/dify, Celery - [leverage framework capabilities](https://awesomereviewers.com/raw/posthog-leverage-framework-capabilities.md) — PostHog/posthog, Temporal - [Task signature methods](https://awesomereviewers.com/raw/prowler-task-signature-methods.md) — prowler-cloud/prowler, Celery - [Use acks_late for reliability](https://awesomereviewers.com/raw/sentry-use-acks-late-for-reliability.md) — getsentry/sentry, Celery ### Algorithms, parsers, data transforms - **Property-based tests**: `hypothesis` (14% of repos). Docs: https://github.com/HypothesisWorks/hypothesis - **Fuzzing** — no tool detected in the sample. Catches: Crashes, panics and hangs on malformed input, especially in parsers and decoders. - **Mutation testing**: `mutmut` (2% of repos). Command seen: `mutmut run 2>&1`. Docs: https://github.com/boxed/mutmut - **Proofs and specifications**: `lean` (2% of repos). Docs: https://github.com/leanprover/lean4 Review prompts (47 in total, most discussed first): - [Choose optimal algorithms](https://awesomereviewers.com/raw/checkov-choose-optimal-algorithms.md) — bridgecrewio/checkov, Algorithms - [Deterministic Algorithm Correctness](https://awesomereviewers.com/raw/langchain-deterministic-algorithm-correctness.md) — langchain-ai/langchain, Algorithms - [optimize algorithmic efficiency](https://awesomereviewers.com/raw/poetry-optimize-algorithmic-efficiency.md) — python-poetry/poetry, Algorithms - [Prevent algorithmic pitfalls](https://awesomereviewers.com/raw/boto3-prevent-algorithmic-pitfalls.md) — boto/boto3, Algorithms - [Precise Matching Rules](https://awesomereviewers.com/raw/hermes-agent-precise-matching-rules.md) — nousresearch/hermes-agent, Algorithms - [Algorithm Correctness Standards](https://awesomereviewers.com/raw/python-algorithm-correctness-standards.md) — TheAlgorithms/Python, Algorithms - [implement algorithms correctly](https://awesomereviewers.com/raw/dspy-implement-algorithms-correctly.md) — stanfordnlp/dspy, Algorithms - [Optimize iteration patterns](https://awesomereviewers.com/raw/mxnet-optimize-iteration-patterns.md) — apache/mxnet, Algorithms - [Verify algorithmic equivalence](https://awesomereviewers.com/raw/cpython-verify-algorithmic-equivalence.md) — python/cpython, Algorithms - [Consistent, Guarded Ranking](https://awesomereviewers.com/raw/last30days-skill-consistent-guarded-ranking.md) — mvanhorn/last30days-skill, Algorithms - [Deterministic Mapping Pipeline](https://awesomereviewers.com/raw/ragflow-deterministic-mapping-pipeline.md) — infiniflow/ragflow, Algorithms - [Optimization algorithm compatibility](https://awesomereviewers.com/raw/heretic-optimization-algorithm-compatibility.md) — p-e-w/heretic, Algorithms ### Threads, async code, locks, shared state - **Concurrency model checking** — no tool detected in the sample. Catches: Races and orderings that fail only under specific interleavings. - **Undefined-behaviour detection** — no tool detected in the sample. Catches: Memory-safety and aliasing violations in unsafe code. - **Property-based tests**: `hypothesis` (14% of repos). Docs: https://github.com/HypothesisWorks/hypothesis Review prompts (30 in total, most discussed first): - [Nonblocking Concurrency Rules](https://awesomereviewers.com/raw/hermes-agent-nonblocking-concurrency-rules.md) — nousresearch/hermes-agent, Concurrency - [Avoid shared mutable state](https://awesomereviewers.com/raw/langchain-avoid-shared-mutable-state.md) — langchain-ai/langchain, Concurrency - [Thread-safe lifecycle isolation](https://awesomereviewers.com/raw/celery-thread-safe-lifecycle-isolation.md) — celery/celery, Concurrency - [Safe Async Resource Handling](https://awesomereviewers.com/raw/mcp-safe-async-resource-handling.md) — awslabs/mcp, Concurrency - [Avoid event-loop blocking](https://awesomereviewers.com/raw/openai-python-avoid-event-loop-blocking.md) — openai/openai-python, Concurrency - [Choose appropriate synchronization](https://awesomereviewers.com/raw/sdk-python-choose-appropriate-synchronization.md) — strands-agents/sdk-python, Concurrency - [Ensure operation completion safety](https://awesomereviewers.com/raw/lmcache-ensure-operation-completion-safety.md) — LMCache/LMCache, Concurrency - [Non-Blocking Async Recovery](https://awesomereviewers.com/raw/ragflow-non-blocking-async-recovery.md) — infiniflow/ragflow, Concurrency - [Concurrent resource management](https://awesomereviewers.com/raw/dify-concurrent-resource-management.md) — langgenius/dify, Concurrency - [Deterministic concurrency tests](https://awesomereviewers.com/raw/cpython-deterministic-concurrency-tests.md) — python/cpython, Concurrency - [use context managers concurrency](https://awesomereviewers.com/raw/openpilot-use-context-managers-concurrency.md) — commaai/openpilot, Concurrency - [background task coordination](https://awesomereviewers.com/raw/litellm-background-task-coordination.md) — BerriAI/litellm, Concurrency ### Network calls, retries, timeouts, protocols - **Unit and integration tests**: `pytest` (86% of repos). Command seen: `pytest`. Exit codes: `0` all tests passed; `1` some tests failed; `2` interrupted by the user; `3` internal error; `4` command line usage error; `5` no tests were collected. Alternatives: unittest (12%). Docs: https://docs.pytest.org/en/stable/reference/exit-codes.html - **Fuzzing** — no tool detected in the sample. Catches: Crashes, panics and hangs on malformed input, especially in parsers and decoders. Review prompts (10 in total, most discussed first): - [Restrict public access](https://awesomereviewers.com/raw/checkov-restrict-public-access.md) — bridgecrewio/checkov, Networking - [Restrict public network](https://awesomereviewers.com/raw/checkov-restrict-public-network.md) — bridgecrewio/checkov, Networking - [Configure HTTP requests properly](https://awesomereviewers.com/raw/poetry-configure-http-requests-properly.md) — python-poetry/poetry, Networking - [Preserve protocol semantics](https://awesomereviewers.com/raw/cpython-preserve-protocol-semantics.md) — python/cpython, Networking - [Robust aiohttp Networking](https://awesomereviewers.com/raw/openai-python-robust-aiohttp-networking.md) — openai/openai-python, Networking - [HTTP protocol correctness](https://awesomereviewers.com/raw/flask-http-protocol-correctness.md) — pallets/flask, Networking - [Manage network resources](https://awesomereviewers.com/raw/anthropic-sdk-python-manage-network-resources.md) — anthropics/anthropic-sdk-python, Networking - [network service identification](https://awesomereviewers.com/raw/lmcache-network-service-identification.md) — LMCache/LMCache, Networking - [validate connection timeouts](https://awesomereviewers.com/raw/browser-use-validate-connection-timeouts.md) — browser-use/browser-use, Networking - [Platform-specific socket configuration](https://awesomereviewers.com/raw/openpilot-platform-specific-socket-configuration.md) — commaai/openpilot, Networking ### UI components and pages - **End-to-end browser tests** — no tool detected in the sample. Catches: User flows broken across components, routing and the network. - **Linting**: `ruff-check` (72% of repos). Command seen: `ruff check .`. Autofixes. Exit codes: `0` no violations; `1` violations found; `2` invalid configuration, CLI options or internal error. Alternatives: flake8 (20%), pylint (12%). Docs: https://docs.astral.sh/ruff/linter/#exit-codes ### Databases, migrations, caching - **Unit and integration tests**: `pytest` (86% of repos). Command seen: `pytest`. Exit codes: `0` all tests passed; `1` some tests failed; `2` interrupted by the user; `3` internal error; `4` command line usage error; `5` no tests were collected. Alternatives: unittest (12%). Docs: https://docs.pytest.org/en/stable/reference/exit-codes.html - **Version and platform matrix**: `ci-matrix` (56% of repos). Command seen: `analyze (include: 2)`. Docs: https://docs.github.com/en/actions/how-tos/write-workflows/choose-what-workflows-do/run-job-variations Review prompts (34 in total, most discussed first): - [SQLAlchemy 2.0 patterns](https://awesomereviewers.com/raw/dify-sqlalchemy-20-patterns.md) — langgenius/dify, Database - [Cache Scope and Keys](https://awesomereviewers.com/raw/hermes-agent-cache-scope-and-keys.md) — nousresearch/hermes-agent, Caching - [Safe caching practices](https://awesomereviewers.com/raw/langchain-safe-caching-practices.md) — langchain-ai/langchain, Caching - [Credential-Aware Caching](https://awesomereviewers.com/raw/mcp-credential-aware-caching.md) — awslabs/mcp, Caching - [Cache invalidation consistency](https://awesomereviewers.com/raw/posthog-cache-invalidation-consistency.md) — PostHog/posthog, Caching - [Use connection by alias](https://awesomereviewers.com/raw/django-use-connection-by-alias.md) — django/django, Database - [Cache validity and bounds](https://awesomereviewers.com/raw/openai-python-cache-validity-and-bounds.md) — openai/openai-python, Caching - [Cache state consistency](https://awesomereviewers.com/raw/lmcache-cache-state-consistency.md) — LMCache/LMCache, Caching - [Prevent N+1 database queries](https://awesomereviewers.com/raw/sentry-prevent-n1-database-queries.md) — getsentry/sentry, Database - [Cache Only After Inputs](https://awesomereviewers.com/raw/celery-cache-only-after-inputs.md) — celery/celery, Caching - [Optimize cache performance](https://awesomereviewers.com/raw/sglang-optimize-cache-performance.md) — sgl-project/sglang, Caching - [Split complex migrations incrementally](https://awesomereviewers.com/raw/posthog-split-complex-migrations-incrementally.md) — PostHog/posthog, Migrations ### Hot paths and performance - No tool-based check applies; rely on the prompts. Review prompts (36 in total, most discussed first): - [Avoid Unnecessary Work](https://awesomereviewers.com/raw/celery-avoid-unnecessary-work.md) — celery/celery, Performance Optimization - [Bounded Hot-Path Work](https://awesomereviewers.com/raw/hermes-agent-bounded-hot-path-work.md) — nousresearch/hermes-agent, Performance Optimization - [Bound Work and Queries](https://awesomereviewers.com/raw/ragflow-bound-work-and-queries.md) — infiniflow/ragflow, Performance Optimization - [Manage tensors proactively](https://awesomereviewers.com/raw/heretic-manage-tensors-proactively.md) — p-e-w/heretic, Performance Optimization - [Eliminate redundant operations](https://awesomereviewers.com/raw/sglang-eliminate-redundant-operations.md) — sgl-project/sglang, Performance Optimization - [Vectorize over Python loops](https://awesomereviewers.com/raw/vllm-vectorize-over-python-loops.md) — vllm-project/vllm, Performance Optimization - [Budget-Aware Performance](https://awesomereviewers.com/raw/pr-agent-budget-aware-performance.md) — the-pr-agent/pr-agent, Performance Optimization - [Enforce Resource Caps](https://awesomereviewers.com/raw/mcp-enforce-resource-caps.md) — awslabs/mcp, Performance Optimization - [Performance-Sensitive Efficiency](https://awesomereviewers.com/raw/python-performance-sensitive-efficiency.md) — TheAlgorithms/Python, Performance Optimization - [Optimize hot-path code](https://awesomereviewers.com/raw/cpython-optimize-hot-path-code.md) — python/cpython, Performance Optimization - [Avoid Hidden Performance Traps](https://awesomereviewers.com/raw/langchain-avoid-hidden-performance-traps.md) — langchain-ai/langchain, Performance Optimization - [Cache expensive operations](https://awesomereviewers.com/raw/posthog-cache-expensive-operations.md) — PostHog/posthog, Performance Optimization ### Model, prompt and LLM code - **Unit and integration tests**: `pytest` (86% of repos). Command seen: `pytest`. Exit codes: `0` all tests passed; `1` some tests failed; `2` interrupted by the user; `3` internal error; `4` command line usage error; `5` no tests were collected. Alternatives: unittest (12%). Docs: https://docs.pytest.org/en/stable/reference/exit-codes.html Review prompts (40 in total, most discussed first): - [Model integration rules](https://awesomereviewers.com/raw/heretic-model-integration-rules.md) — p-e-w/heretic, AI - [prefer OpenAI compatibility](https://awesomereviewers.com/raw/litellm-prefer-openai-compatibility.md) — BerriAI/litellm, AI - [Improve code organization](https://awesomereviewers.com/raw/sglang-improve-code-organization.md) — sgl-project/sglang, AI - [Device-agnostic acceleration code](https://awesomereviewers.com/raw/pytorch-device-agnostic-acceleration-code.md) — pytorch/pytorch, AI - [Avoid magic numbers](https://awesomereviewers.com/raw/vllm-avoid-magic-numbers.md) — vllm-project/vllm, AI - [Fail-closed evaluation metrics](https://awesomereviewers.com/raw/promptfoo-fail-closed-evaluation-metrics.md) — promptfoo/promptfoo, AI - [Model-Safe Parameter Validation](https://awesomereviewers.com/raw/pr-agent-model-safe-parameter-validation.md) — the-pr-agent/pr-agent, AI - [Preserve AI Structure Invariants](https://awesomereviewers.com/raw/langchain-preserve-ai-structure-invariants.md) — langchain-ai/langchain, AI - [Handle optional dependencies gracefully](https://awesomereviewers.com/raw/dspy-handle-optional-dependencies-gracefully.md) — stanfordnlp/dspy, AI - [leverage existing framework functionality](https://awesomereviewers.com/raw/llama.cpp-leverage-existing-framework-functionality.md) — ggml-org/llama.cpp, AI - [LLM Interface Contracts](https://awesomereviewers.com/raw/mcp-llm-interface-contracts.md) — awslabs/mcp, AI - [preserve dtype and shapes](https://awesomereviewers.com/raw/heretic-preserve-dtype-and-shapes.md) — p-e-w/heretic, Pytorch ### Logging, metrics, tracing - No tool-based check applies; rely on the prompts. Review prompts (26 in total, most discussed first): - [Scoped, Accurate Warning Logs](https://awesomereviewers.com/raw/azure-cli-scoped-accurate-warning-logs.md) — Azure/azure-cli, Logging - [Logging Correctness Rules](https://awesomereviewers.com/raw/celery-logging-correctness-rules.md) — celery/celery, Logging - [Follow logging best practices](https://awesomereviewers.com/raw/vllm-follow-logging-best-practices.md) — vllm-project/vllm, Logging - [Logging consistency and safety](https://awesomereviewers.com/raw/pr-agent-logging-consistency-and-safety.md) — the-pr-agent/pr-agent, Logging - [Use appropriate log levels](https://awesomereviewers.com/raw/dify-use-appropriate-log-levels.md) — langgenius/dify, Logging - [Use appropriate logging levels](https://awesomereviewers.com/raw/core-use-appropriate-logging-levels.md) — home-assistant/core, Logging - [Use proper logging mechanisms](https://awesomereviewers.com/raw/litellm-use-proper-logging-mechanisms.md) — BerriAI/litellm, Logging - [Structure logs with context](https://awesomereviewers.com/raw/sentry-structure-logs-with-context.md) — getsentry/sentry, Logging - [Sanitize observability data exports](https://awesomereviewers.com/raw/opentelemetry-python-sanitize-observability-data-exports.md) — open-telemetry/opentelemetry-python, Observability - [Add logging without duplication](https://awesomereviewers.com/raw/compose-add-logging-without-duplication.md) — docker/compose, Logging - [Structure logs effectively](https://awesomereviewers.com/raw/django-structure-logs-effectively.md) — django/django, Logging - [Use appropriate logging levels](https://awesomereviewers.com/raw/checkov-use-appropriate-logging-levels.md) — bridgecrewio/checkov, Logging ### CI workflows and configuration files - **CI workflow linting** — no tool detected in the sample. Catches: Broken GitHub Actions syntax, script injection, over-broad token permissions. - **File hygiene**: `pre-commit-hooks` (39% of repos). Runs via pre-commit hook end-of-file-fixer. Autofixes. Docs: https://github.com/pre-commit/pre-commit-hooks Review prompts (68 in total, most discussed first): - [Configuration ownership and determinism](https://awesomereviewers.com/raw/heretic-configuration-ownership-and-determinism.md) — p-e-w/heretic, Configurations - [Centralized Config Standards](https://awesomereviewers.com/raw/db-gpt-centralized-config-standards.md) — eosphoros-ai/DB-GPT, Configurations - [Centralize environment variables](https://awesomereviewers.com/raw/checkov-centralize-environment-variables.md) — bridgecrewio/checkov, Configurations - [validate configuration defaults](https://awesomereviewers.com/raw/unstructured-validate-configuration-defaults.md) — Unstructured-IO/unstructured, Configurations - [Use configurable default values](https://awesomereviewers.com/raw/prowler-use-configurable-default-values.md) — prowler-cloud/prowler, Configurations - [Explicit config behavior](https://awesomereviewers.com/raw/celery-explicit-config-behavior.md) — celery/celery, Configurations - [Robust environment config handling](https://awesomereviewers.com/raw/flask-robust-environment-config-handling.md) — pallets/flask, Configurations - [Language-agnostic configuration design](https://awesomereviewers.com/raw/serena-language-agnostic-configuration-design.md) — oraios/serena, Configurations - [Config.yaml First Policy](https://awesomereviewers.com/raw/hermes-agent-configyaml-first-policy.md) — nousresearch/hermes-agent, Configurations - [Region configuration consistency](https://awesomereviewers.com/raw/mcp-region-configuration-consistency.md) — awslabs/mcp, Configurations - [Optional configuration parameters](https://awesomereviewers.com/raw/dspy-optional-configuration-parameters.md) — stanfordnlp/dspy, Configurations - [Feature flag rollouts](https://awesomereviewers.com/raw/sentry-feature-flag-rollouts.md) — getsentry/sentry, Configurations ### Dependencies and lockfiles - **Dependency audit**: `dependency-review` (8% of repos). Runs via GitHub Action actions/dependency-review-action. Alternatives: pip-audit (3%). Docs: https://github.com/actions/dependency-review-action - **Lockfile consistency**: `uv-lock` (20% of repos). Command seen: `uv lock --check`. Autofixes. Docs: https://github.com/astral-sh/uv - **Version and platform matrix**: `ci-matrix` (56% of repos). Command seen: `analyze (include: 2)`. Docs: https://docs.github.com/en/actions/how-tos/write-workflows/choose-what-workflows-do/run-job-variations ### The tests themselves - **Mutation testing**: `mutmut` (2% of repos). Command seen: `mutmut run 2>&1`. Docs: https://github.com/boxed/mutmut - **Version and platform matrix**: `ci-matrix` (56% of repos). Command seen: `analyze (include: 2)`. Docs: https://docs.github.com/en/actions/how-tos/write-workflows/choose-what-workflows-do/run-job-variations Review prompts (52 in total, most discussed first): - [Thorough edge-case verification](https://awesomereviewers.com/raw/cpython-thorough-edge-case-verification.md) — python/cpython, Testing - [Mock tests in tests/litellm](https://awesomereviewers.com/raw/litellm-mock-tests-in-testslitellm.md) — BerriAI/litellm, Testing - [Parameterize similar tests](https://awesomereviewers.com/raw/poetry-parameterize-similar-tests.md) — python-poetry/poetry, Testing - [Mock external dependencies only](https://awesomereviewers.com/raw/core-mock-external-dependencies-only.md) — home-assistant/core, Testing - [Automate All Behavior Tests](https://awesomereviewers.com/raw/python-automate-all-behavior-tests.md) — TheAlgorithms/Python, Testing - [Behavioral Regression Testing](https://awesomereviewers.com/raw/hermes-agent-behavioral-regression-testing.md) — nousresearch/hermes-agent, Testing - [Prefer Robust Testing Assertions](https://awesomereviewers.com/raw/langchain-prefer-robust-testing-assertions.md) — langchain-ai/langchain, Testing - [Write clear test cases](https://awesomereviewers.com/raw/unstructured-write-clear-test-cases.md) — Unstructured-IO/unstructured, Testing - [Use idiomatic pytest features](https://awesomereviewers.com/raw/flask-use-idiomatic-pytest-features.md) — pallets/flask, Testing - [Deterministic recorded testing](https://awesomereviewers.com/raw/azure-cli-deterministic-recorded-testing.md) — Azure/azure-cli, Testing - [Meaningful Tests For Guards](https://awesomereviewers.com/raw/celery-meaningful-tests-for-guards.md) — celery/celery, Testing - [Write self-documenting tests](https://awesomereviewers.com/raw/boto3-write-self-documenting-tests.md) — boto/boto3, Testing ### Documentation and comments - **Docs build**: `sphinx` (12% of repos). Command seen: `sphinx-build -j2 -b configcheck -d {envtmpdir}/doctrees docs docs/_build/configcheck`. Alternatives: mkdocs (6%). Docs: https://github.com/sphinx-doc/sphinx - **Spelling**: `codespell` (25% of repos). Command seen: `codespell --toml pyproject.toml --write`. Docs: https://github.com/codespell-project/codespell Review prompts (32 in total, most discussed first): - [Docstring Standards](https://awesomereviewers.com/raw/python-docstring-standards.md) — TheAlgorithms/Python, Documentation - [Docstrings With Intent](https://awesomereviewers.com/raw/langchain-docstrings-with-intent.md) — langchain-ai/langchain, Documentation - [PEP 257 docstring compliance](https://awesomereviewers.com/raw/unstructured-pep-257-docstring-compliance.md) — Unstructured-IO/unstructured, Documentation - [Document code thoroughly](https://awesomereviewers.com/raw/vllm-document-code-thoroughly.md) — vllm-project/vllm, Documentation - [Concise, Correct CLI Help](https://awesomereviewers.com/raw/azure-cli-concise-correct-cli-help.md) — Azure/azure-cli, Documentation - [Document Behavior Changes](https://awesomereviewers.com/raw/celery-document-behavior-changes.md) — celery/celery, Documentation - [Document Behavior Accurately](https://awesomereviewers.com/raw/mcp-document-behavior-accurately.md) — awslabs/mcp, Documentation - [Complete parameter documentation](https://awesomereviewers.com/raw/dspy-complete-parameter-documentation.md) — stanfordnlp/dspy, Documentation - [Document code rationale](https://awesomereviewers.com/raw/pydantic-document-code-rationale.md) — pydantic/pydantic, Documentation - [Write purposeful comments](https://awesomereviewers.com/raw/opentelemetry-python-write-purposeful-comments.md) — open-telemetry/opentelemetry-python, Documentation - [Comprehensive API documentation](https://awesomereviewers.com/raw/boto3-comprehensive-api-documentation.md) — boto/boto3, Documentation - [Accurate, synced documentation](https://awesomereviewers.com/raw/cpython-accurate-synced-documentation.md) — python/cpython, Documentation ## Source repositories [nousresearch/hermes-agent](https://awesomereviewers.com/sources/nousresearch-hermes-agent/), [TheAlgorithms/Python](https://awesomereviewers.com/sources/thealgorithms-python/), [tensorflow/tensorflow](https://awesomereviewers.com/sources/tensorflow-tensorflow/), [langchain-ai/langchain](https://awesomereviewers.com/sources/langchain-ai-langchain/), [langgenius/dify](https://awesomereviewers.com/sources/langgenius-dify/), [langflow-ai/langflow](https://awesomereviewers.com/sources/langflow-ai-langflow/), [pytorch/pytorch](https://awesomereviewers.com/sources/pytorch-pytorch/), [fastapi/fastapi](https://awesomereviewers.com/sources/fastapi-fastapi/), [django/django](https://awesomereviewers.com/sources/django-django/), [comfyanonymous/ComfyUI](https://awesomereviewers.com/sources/comfyanonymous-comfyui/), [home-assistant/core](https://awesomereviewers.com/sources/home-assistant-core/), [infiniflow/ragflow](https://awesomereviewers.com/sources/infiniflow-ragflow/), [python/cpython](https://awesomereviewers.com/sources/python-cpython/), [microsoft/markitdown](https://awesomereviewers.com/sources/microsoft-markitdown/), [pallets/flask](https://awesomereviewers.com/sources/pallets-flask/), [TauricResearch/TradingAgents](https://awesomereviewers.com/sources/tauricresearch-tradingagents/), [browser-use/browser-use](https://awesomereviewers.com/sources/browser-use-browser-use/), [commaai/openpilot](https://awesomereviewers.com/sources/commaai-openpilot/), [mvanhorn/last30days-skill](https://awesomereviewers.com/sources/mvanhorn-last30days-skill/), [karpathy/nanochat](https://awesomereviewers.com/sources/karpathy-nanochat/), [vllm-project/vllm](https://awesomereviewers.com/sources/vllm-project-vllm/), [calesthio/OpenMontage](https://awesomereviewers.com/sources/calesthio-openmontage/), [getsentry/sentry](https://awesomereviewers.com/sources/getsentry-sentry/), [apache/airflow](https://awesomereviewers.com/sources/apache-airflow/), [docker/compose](https://awesomereviewers.com/sources/docker-compose/), [Aider-AI/aider](https://awesomereviewers.com/sources/aider-ai-aider/), [pola-rs/polars](https://awesomereviewers.com/sources/pola-rs-polars/), [crewaiinc/crewai](https://awesomereviewers.com/sources/crewaiinc-crewai/), [python-poetry/poetry](https://awesomereviewers.com/sources/python-poetry-poetry/), [openai/openai-python](https://awesomereviewers.com/sources/openai-openai-python/), [celery/celery](https://awesomereviewers.com/sources/celery-celery/), [PostHog/posthog](https://awesomereviewers.com/sources/posthog-posthog/), [BerriAI/litellm](https://awesomereviewers.com/sources/berriai-litellm/), [stanfordnlp/dspy](https://awesomereviewers.com/sources/stanfordnlp-dspy/), [pydantic/pydantic](https://awesomereviewers.com/sources/pydantic-pydantic/), [apache/mxnet](https://awesomereviewers.com/sources/apache-mxnet/), [eosphoros-ai/DB-GPT](https://awesomereviewers.com/sources/eosphoros-ai-db-gpt/), [agent0ai/agent-zero](https://awesomereviewers.com/sources/agent0ai-agent-zero/), [sgl-project/sglang](https://awesomereviewers.com/sources/sgl-project-sglang/), [SWE-agent/SWE-agent](https://awesomereviewers.com/sources/swe-agent-swe-agent/), [bytedance/deer-flow](https://awesomereviewers.com/sources/bytedance-deer-flow/), [oraios/serena](https://awesomereviewers.com/sources/oraios-serena/), [semgrep/semgrep](https://awesomereviewers.com/sources/semgrep-semgrep/), [the-pr-agent/pr-agent](https://awesomereviewers.com/sources/the-pr-agent-pr-agent/), [emcie-co/parlant](https://awesomereviewers.com/sources/emcie-co-parlant/), [Unstructured-IO/unstructured](https://awesomereviewers.com/sources/unstructured-io-unstructured/), [prowler-cloud/prowler](https://awesomereviewers.com/sources/prowler-cloud-prowler/), [huggingface/tokenizers](https://awesomereviewers.com/sources/huggingface-tokenizers/), [awslabs/mcp](https://awesomereviewers.com/sources/awslabs-mcp/), [boto/boto3](https://awesomereviewers.com/sources/boto-boto3/), [openai/codex-security](https://awesomereviewers.com/sources/openai-codex-security/), [bytedance/trae-agent](https://awesomereviewers.com/sources/bytedance-trae-agent/), [openai/skills](https://awesomereviewers.com/sources/openai-skills/), [bridgecrewio/checkov](https://awesomereviewers.com/sources/bridgecrewio-checkov/), [p-e-w/heretic](https://awesomereviewers.com/sources/p-e-w-heretic/), [Azure/azure-cli](https://awesomereviewers.com/sources/azure-azure-cli/), [strands-agents/sdk-python](https://awesomereviewers.com/sources/strands-agents-sdk-python/), [LMCache/LMCache](https://awesomereviewers.com/sources/lmcache-lmcache/), [anthropics/anthropic-sdk-python](https://awesomereviewers.com/sources/anthropics-anthropic-sdk-python/), [awslabs/agentcore-samples](https://awesomereviewers.com/sources/awslabs-agentcore-samples/), [smallcloudai/refact](https://awesomereviewers.com/sources/smallcloudai-refact/), [anthropics/claude-agent-sdk-python](https://awesomereviewers.com/sources/anthropics-claude-agent-sdk-python/), [salesforce/cloudsplaining](https://awesomereviewers.com/sources/salesforce-cloudsplaining/), [open-telemetry/opentelemetry-python](https://awesomereviewers.com/sources/open-telemetry-opentelemetry-python/)