Python 代码审查¶

# 型チェック
mypy .

# リンティングとフォーマット
ruff check .
black --check .
isort --check-only .

# セキュリティスキャン
bandit -r .

# 依存関係監査
pip-audit
safety check

# テスト
pytest --cov=app --cov-report=term-missing

User: /python-review

Agent:
# Pythonコードレビューレポート

## レビューされたファイル
- app/routes/user.py (変更)
- app/services/auth.py (変更)

## 静的解析結果
✓ ruff: 問題なし
✓ mypy: エラーなし
WARNING: black: 2ファイルが再フォーマット必要
✓ bandit: セキュリティ問題なし

## 発見された問題

[CRITICAL] SQLインジェクション脆弱性
ファイル: app/routes/user.py:42
問題: ユーザー入力が直接SQLクエリに挿入されている
```python
query = f"SELECT * FROM users WHERE id = {user_id}"  # 悪い

query = "SELECT * FROM users WHERE id = %s"  # 良い
cursor.execute(query, (user_id,))

def process_items(items=[]):  # 悪い
    items.append("new")
    return items

def process_items(items=None):  # 良い
    if items is None:
        items = []
    items.append("new")
    return items

def get_user(user_id):  # 悪い
    return db.find(user_id)

def get_user(user_id: str) -> Optional[User]:  # 良い
    return db.find(user_id)

f = open("config.json")  # 悪い
data = f.read()
f.close()

with open("config.json") as f:  # 良い
    data = f.read()

## 批准标准 {#approval-criteria}

| 状态 | 条件 |
|--------|-----------|
| PASS: 批准 | 没有 CRITICAL 或 HIGH 问题 |
| WARNING: 警告 | 仅有 MEDIUM 问题（谨慎合并） |
| FAIL: 阻止 | 发现 CRITICAL 或 HIGH 问题 |

## 与其他命令的集成 {#integration-with-other-commands}

- 首先使用 `/python-test` 确认测试通过
- 使用 `/code-review` 处理非 Python 特有问题
- 在提交前使用 `/python-review`
- 当静态分析工具失败时使用 `/build-fix`

## 框架特定审查 {#framework-specific-reviews}

### Django 项目 {#django-projects}
审查者会检查以下内容：
- N+1 查询问题（使用 `select_related` 和 `prefetch_related`）
- 模型变更缺少迁移
- 在 ORM 可以处理时使用原生 SQL
- 多步操作缺少 `transaction.atomic()`

### FastAPI 项目 {#fastapi-projects}
审查者会检查以下内容：
- CORS 配置错误
- 用于请求验证的 Pydantic 模型
- 响应模型的准确性
- 正确使用 async/await
- 依赖注入模式

### Flask 项目 {#flask-projects}
审查者会检查以下内容：
- 上下文管理（app 上下文、request 上下文）
- 适当的错误处理
- Blueprint 配置
- 配置管理

## 相关 {#related}

- 代理：`agents/python-reviewer.md`
- 技能：`skills/python-patterns/`, `skills/python-testing/`

## 常见修复 {#common-fixes}

### 添加类型提示 {#adding-type-hints}
```python
# 変更前
def calculate(x, y):
    return x + y

# 変更後
from typing import Union

def calculate(x: Union[int, float], y: Union[int, float]) -> Union[int, float]:
    return x + y

# 変更前
f = open("file.txt")
data = f.read()
f.close()

# 変更後
with open("file.txt") as f:
    data = f.read()

# 変更前
result = []
for item in items:
    if item.active:
        result.append(item.name)

# 変更後
result = [item.name for item in items if item.active]

# 変更前
def append(value, items=[]):
    items.append(value)
    return items

# 変更後
def append(value, items=None):
    if items is None:
        items = []
    items.append(value)
    return items

# 変更前
name = "Alice"
greeting = "Hello, " + name + "!"
greeting2 = "Hello, {}".format(name)

# 変更後
greeting = f"Hello, {name}!"

# 変更前
result = ""
for item in items:
    result += str(item)

# 変更後
result = "".join(str(item) for item in items)