AI Analysis

# repos.py:67
# Severity: High | Category: Performance

# ❌ Current:
async def get_all_repos(db: AsyncSession):
    result = await db.execute(select(Installation))
    installations = result.scalars().all()
    for inst in installations:
        inst.user = await db.execute(select(User).where(User.id == inst.user_id))
    return installations

# ✅ Suggested (eliminates N+1 query):
async def get_all_repos(db: AsyncSession):
    result = await db.execute(
        select(Installation).options(selectinload(Installation.user))
    )
    return result.scalars().all()

# Reasoning: Current code executes 1 + N queries (1 for installations,
# N for users). Use SQLAlchemy's selectinload to fetch in 2 queries total.

# Claude's internal reasoning:
# "Let me check how authentication is handled elsewhere in this repo."

result = await tools.search_code(
    repo="owner/repo",
    query="JWT token validation",
    limit=5,
)

# Returns:
[
    {"file": "app/auth/dependencies.py", "line": 23, "snippet": "..."},
    {"file": "app/auth/jwt_utils.py", "line": 45, "snippet": "..."},
]

experts = await tools.get_file_experts(
    repo="owner/repo",
    paths=["app/services/pr_review_service.py"],
)

# Returns:
[
    {"login": "alice", "touch_count": 47},
    {"login": "bob", "touch_count": 23},
]

# Claude includes in review:
# "Consider asking @alice to review this change to pr_review_service.py,
# as she's the primary maintainer (47 prior commits)."

errors = await tools.fetch_sentry_errors(
    project="backend",
    filename="app/services/context_service.py",
)

# Returns:
[
    {
        "title": "AttributeError: 'NoneType' object has no attribute 'search'",
        "count": 142,
        "last_seen": "2026-03-09T14:23:11Z",
    }
]

# Claude includes in review:
# "⚠️ Warning: This file has 142 production errors in the last 7 days
# related to None checks. Ensure memory_adapter is properly initialized
# before calling .search()."