Автоматическая проверка кода

# Проверка изменений относительно базового коммита
python -m src.cli review --base-ref HEAD~3

# Проверка только проиндексированных изменений
python -m src.cli review --staged

# Проверка конкретных файлов
python -m src.cli review --files src/api/main.py src/api/auth.py

# Вывод в формате SARIF для CI-интеграции
python -m src.cli review --base-ref HEAD~5 --format sarif --output-file report.sarif

# JSON-вывод с отдельным SARIF-файлом
python -m src.cli review --base-ref HEAD~3 --format json --sarif-file findings.sarif

# Пропуск анализа безопасности
python -m src.cli review --base-ref HEAD~3 --no-security

# Указание базы данных
python -m src.cli review --db data/projects/postgres.duckdb --base-ref origin/main

import duckdb
from src.patch_review import ReviewWorkflow

# Подключение к базе данных CPG
from src.project_manager import ProjectManager
conn = duckdb.connect(ProjectManager.get_active_db_path())

# Настройка параметров DoD
dod_config = {
    'auto_generate': True,
    'interactive': False,
    'extraction': {
        'sources': ['pr_body', 'jira', 'commit_message'],
        'formats': ['checklist', 'yaml', 'markdown'],
    },
}

# Создание рабочего процесса
workflow = ReviewWorkflow(conn, dod_config=dod_config)

# Запуск проверки
verdict = workflow.run(
    patch_source='git_diff',
    patch_data={'diff': diff_content},
    pr_body=pr_description,
    task_description=task_desc,
    interactive_mode=False,
)

# Доступ к результатам
print(f"Оценка: {verdict.overall_score}/100")
print(f"Рекомендация: {verdict.recommendation.value}")

# Результаты проверки DoD
if verdict.dod_validation:
    print(f"Соответствие DoD: {verdict.dod_validation.compliance_score}%")

┌─────────────────────────────────────────────────────────┐
│ Уровень 1: CLI Review Pipeline (src/review/)            │
│   ReviewPipeline → ReviewAggregator → ReviewReport      │
│   Точка входа: python -m src.cli review                 │
│   Фокус: CPG-качество + безопасность, SARIF, scope      │
├─────────────────────────────────────────────────────────┤
│ Уровень 2: Patch Review Workflow (src/patch_review/)     │
│   ReviewWorkflow (LangGraph) → Analyzers → Verdicts     │
│   Точка входа: ReviewWorkflow.run() / REST API / демо   │
│   Фокус: DoD, дельта CPG, интеграция с PR/MR            │
├─────────────────────────────────────────────────────────┤
│ Уровень 3: Workflow Scenario 9 (src/workflow/scenarios/) │
│   PRAnalyzer → ContextAggregator → ReviewReporter       │
│   Точка входа: MultiScenarioCopilot (code_review)       │
│   Фокус: LLM-проверка, оптимизация обработчиками        │
└─────────────────────────────────────────────────────────┘

python -m src.cli review [OPTIONS]

pipeline = ReviewPipeline(
    db_path: Optional[str] = None,
    base_ref: Optional[str] = None,
    files: Optional[List[str]] = None,
    staged: bool = False,
    include_security: bool = True,
)
report = pipeline.run()  # -> ReviewReport

@dataclass
class ReviewReport:
    findings: List[ReviewFinding]          # Все отфильтрованные находки
    cpg_status: Optional[CPGStatus]        # Информация о свежести CPG
    parse_scope: Optional[ParseScope]      # Границы области анализа
    project_name: str                      # Название проекта
    base_ref: str                          # Базовый коммит
    duration_ms: float                     # Время выполнения конвейера
    suppressed_count: int                  # Подавленные находки (по scope)
    metadata: Dict[str, Any]              # Дополнительные данные

@dataclass
class ReviewFinding:
    file: str                              # Путь к файлу
    line: Optional[int]                    # Номер строки
    type: str                              # Тип (dead_code, high_complexity и т.д.)
    severity: str                          # critical|high|medium|low|info
    message: str                           # Описание
    suggestion: Optional[str]              # Рекомендация по исправлению
    metric_value: Optional[float]          # Числовая метрика (CC, LOC, fan_in)
    scope_limited: bool                    # True при частичной области видимости
    source: str                            # "quality" или "security"

@dataclass
class CPGStatus:
    available: bool         # True если БД существует и доступна
    fresh: bool             # True если commits_behind == 0
    commits_behind: int     # Количество коммитов после последнего обновления CPG
    commit_hash: str        # Записанный хеш коммита CPG
    db_path: str            # Путь к файлу базы данных

@dataclass
class ParseScope:
    source_path: str                       # Корневая директория исходного кода
    language: str                          # Основной язык
    include_tests: bool                    # Включены ли тесты в CPG
    include_paths: List[str]               # Пути для выборочного включения
    exclude_paths: List[str]               # Явные исключения
    default_excludes: List[str]            # Шаблоны исключений по умолчанию
    import_mode: str                       # "full" или "selective"

# Прямой SARIF-вывод
python -m src.cli review --base-ref HEAD~5 --format sarif --output-file report.sarif

# Markdown-вывод + отдельный SARIF-файл
python -m src.cli review --base-ref HEAD~5 --format markdown --sarif-file findings.sarif

{
  "$schema": "https://json.schemastore.org/sarif-2.1.0.json",
  "version": "2.1.0",
  "runs": [{
    "tool": {
      "driver": {
        "name": "CodeGraph Review",
        "version": "1.0.0",
        "rules": [
          {"id": "dead_code", "shortDescription": {"text": "Dead Code"}},
          {"id": "high_complexity", "shortDescription": {"text": "High Complexity"}}
        ]
      }
    },
    "results": [
      {
        "ruleId": "high_complexity",
        "level": "warning",
        "message": {"text": "`MyClass.process` has cyclomatic complexity 25"},
        "locations": [{
          "physicalLocation": {
            "artifactLocation": {"uri": "src/service.py"},
            "region": {"startLine": 42}
          }
        }]
      }
    ]
  }]
}

review_pipeline:
  scope_aware_filtering: true
  suppress_tests_outside_scope: true
  demote_dead_code_partial_scope: true

> **Ограниченная область анализа**
>
> CPG построен с исключениями: `vendor/`, `generated/`
>
> Это означает:
> - Находки мёртвого кода могут быть ложноположительными
> - Радиус поражения занижен

parse_patch → extract_dod → [generate_dod] → [confirm_dod] → generate_delta_cpg
    ↓
run_analyzers → generate_verdicts → aggregate_verdict → validate_dod → format_output

## Definition of Done

- [ ] Функциональность работает, как ожидается
- [ ] Не внесено уязвимостей безопасности
- [ ] Добавлены модульные тесты
- [ ] Обновлена документация

dod:
  - description: Функциональность работает, как ожидается
    type: functional
  - description: Модульные тесты проходят
    type: test

{
  "dod": [
    {"description": "Функциональность работает", "type": "functional"},
    {"description": "Тесты проходят", "type": "test"}
  ]
}

============================================================
ОПРЕДЕЛЕНИЕ ЗАВЕРШЁННОСТИ — ПОДТВЕРЖДЕНИЕ
============================================================

Текущее DoD (из pr_body):
----------------------------------------
  1. [FUNC] Функциональность работает, как ожидается
  2. [SECU] Не внесено уязвимостей безопасности
  3. [TEST] Добавлены модульные тесты для новой функциональности
  4. [PERF] Отсутствие регрессии производительности
  5. [CODE] Код соответствует стилю проекта
----------------------------------------
Всего: 5 пунктов

Опции:
  [c] Подтвердить и продолжить
  [e] Редактировать пункты
  [a] Добавить новый пункт
  [r] Удалить пункт
  [s] Пропустить проверку DoD
  [q] Отменить проверку

# Запрос: PatchReviewRequest
{
    "patch_content": "diff --git a/...",       # обязательное
    "task_description": "Добавить авторизацию", # опциональное
    "dod_items": ["Тесты проходят"],            # опциональное
    "output_format": "json"                     # json|markdown|yaml
}
# Заголовок: Authorization (JWT)

# Запрос: GitHubPRReviewRequest
{
    "owner": "myorg",
    "repo": "myrepo",
    "pr_number": 123
}
# Заголовок: X-GitHub-Token (обязательный)

# Запрос: GitLabMRReviewRequest
{
    "project_id": "group/project",
    "mr_iid": 456,
    "gitlab_url": "https://gitlab.com"
}
# Заголовок: X-GitLab-Token (обязательный)

# Запрос: SourceCraftMRReviewRequest
{
    "project_id": "project-id",
    "mr_iid": 789,
    "sourcecraft_url": "https://api.sourcecraft.yandex.cloud"
}
# Заголовок: X-SourceCraft-Token (обязательный)

# Запрос: GitVersePRReviewRequest
{
    "project_id": "owner/repo",
    "pr_number": 101,
    "gitverse_url": "https://gitverse.ru/api/v1"
}
# Заголовок: X-GitVerse-Token (обязательный)

# Запрос: SummaryRequest
{"diff_content": "...", "title": "...", "description": "..."}

# Ответ: SummaryResponse
{
    "summary": "Этот MR добавляет авторизацию...",
    "changed_files": ["src/auth.py"],
    "additions": 45,
    "deletions": 12,
    "changed_methods": ["authenticate", "reset_password"],
    "risk_areas": ["SQL-инъекция в auth.py"]
}

# Запрос: CommitMessageRequest
{"diff_content": "..."}

# Ответ: CommitMessageResponse
{"message": "feat(auth): add password reset flow", "type": "feat", "scope": "auth", "files_changed": 3}

class ReviewResponse:
    recommendation: str      # APPROVE | REQUEST_CHANGES | COMMENT | BLOCK
    score: float             # 0-100
    findings: List[Finding]
    dod_validation: Optional[List[DODItem]]
    summary: str
    processing_time_ms: float
    request_id: str
    metadata: Dict[str, Any]

# Конфигурация определения готовности (Definition of Done)
dod:
  sources:
    - pr_body
    - jira
    - commit_message
    - manual
  source_priority:
    - pr_body
    - jira
    - commit_message
  formats:
    - checklist
    - yaml
    - markdown
    - json
  auto_generate: true
  interactive_confirm: false

  # Интеграция с Jira
  jira:
    url: ${JIRA_URL}
    api_key: ${JIRA_API_KEY}
    dod_field: "customfield_10001"

  # Настройки генерации DoD (при auto_generate=true)
  generation:
    use_llm: true
    fallback_to_rules: true
    include_security: true
    include_performance: true
    require_tests: true

  # Настройки валидации
  validation:
    strict_mode: false
    blocking_severities:
      - critical
      - high

# Политика проверки
policy:
  block_on_critical_security: true
  block_on_high_security: false
  block_on_critical_errors: true
  block_on_breaking_changes: false
  min_score_to_approve: 70.0
  min_score_to_comment: 60.0
  max_critical_findings: 0
  max_high_findings: 5
  max_complexity_increase: 20
  require_dod_compliance: false
  min_dod_compliance_score: 60.0
  custom_rules: []

# Конфигурация анализаторов
analyzers:
  call_graph:
    enabled: true
    max_depth: 5
    include_indirect: true
  dataflow:
    enabled: true
    max_path_length: 10
    sanitization_threshold: 0.7
  control_flow:
    enabled: true
    complexity_threshold: 10
    max_nesting_depth: 5
  dependency:
    enabled: true
    check_circular: true
    max_coupling_increase: 0.2

# Веса вердиктов
verdicts:
  weights:
    security: 0.35
    performance: 0.20
    error: 0.25
    architecture: 0.20
  dod_compliance_weight: 0.10
  thresholds:
    block: 40
    request_changes: 60
    comment: 80
    approve: 80

# Конфигурация вывода
output:
  formats:
    - json
    - markdown
    - pr_comment
  include_dod: true
  include_findings: true
  include_snippets: true
  max_findings_per_category: 10
  pr_comment:
    max_length: 10000
    use_emoji: true
    collapsible_sections: true

workflow = ReviewWorkflow(
    conn: duckdb.DuckDBPyConnection,
    config: Optional[AggregationConfig] = None,
    policy: Optional[ReviewPolicy] = None,
    dod_config: Optional[Dict[str, Any]] = None,
)

verdict = workflow.run(
    patch_source: str,          # 'git_diff', 'github_pr', 'gitlab_mr'
    patch_data: Dict[str, Any],
    session_id: Optional[str] = None,
    policy: Optional[ReviewPolicy] = None,
    task_description: Optional[str] = None,
    pr_body: Optional[str] = None,
    jira_ticket: Optional[str] = None,
    interactive_mode: bool = False,
)

@dataclass
class ReviewVerdict:
    # Основные
    patch_id: str
    overall_score: float                      # 0-100
    recommendation: Recommendation            # APPROVE, REQUEST_CHANGES, BLOCK, COMMENT
    security: SecurityVerdict
    performance: PerformanceVerdict
    error: ErrorVerdict
    architecture: ArchitectureVerdict

    # Агрегированные находки
    all_findings: List[Finding]
    critical_count: int
    high_count: int
    medium_count: int
    low_count: int

    # Метрики воздействия
    blast_radius_score: float                 # 0-100, выше = меньший радиус
    review_time_seconds: float
    summary: str
    reviewed_at: Optional[datetime]

    # DoD
    dod_validation: Optional[DoDValidationResult]
    dod_compliance_score: float               # 0-100, по умолчанию 100.0

    # Идентификация
    review_id: str                            # автогенерация REVIEW_xxxx
    session_id: str

@dataclass
class DoDValidationResult:
    dod: DefinitionOfDone
    total_items: int
    satisfied_count: int
    failed_count: int
    pending_count: int
    compliance_score: float                   # 0-100
    blocking_failures: List[DoDItem]

    @property
    def is_compliant(self) -> bool:
        """Вычисляемое свойство: True если failed_count == 0 и pending_count == 0."""
        ...

@dataclass
class ReviewPolicy:
    block_on_critical_security: bool = True
    block_on_high_security: bool = False
    block_on_critical_errors: bool = True
    block_on_breaking_changes: bool = False
    min_score_to_approve: float = 70.0
    min_score_to_comment: float = 60.0
    max_critical_findings: int = 0
    max_high_findings: int = 5
    max_complexity_increase: int = 20
    custom_rules: List[PolicyRule] = field(default_factory=list)

{
  "patch_id": "PATCH_abc123",
  "overall_score": 45.5,
  "recommendation": "REQUEST_CHANGES",
  "dod_compliance_score": 60.0,
  "findings": [...],
  "dod_validation": {
    "compliance_score": 60.0,
    "items": [
      {"description": "...", "satisfied": true, "evidence": "..."},
      {"description": "...", "satisfied": false, "evidence": "..."}
    ]
  }
}

## Резюме проверки изменений

**Общий балл:** 45/100
**Рекомендация:** REQUEST_CHANGES

### Критерии завершённости (Definition of Done)

**Уровень соответствия:** 60%

- [+] Функция работает, как ожидается
- [-] Не внесены уязвимости безопасности
  - Доказательство: Обнаружены проблемы: SQL-инъекция
- [?] Добавлены модульные тесты (ожидает ручной проверки)

### Результаты анализа безопасности

1. [HIGH] Уязвимость к SQL-инъекции
   - Расположение: src/auth/login.py:14
   - CWE-89

gocpg parse --input=/path/to/source --output=project.duckdb

python examples/demo_patch_review.py --db data/projects/postgres.duckdb
python examples/demo_patch_review.py --no-dod
python examples/demo_patch_review.py --auto-dod
python examples/demo_patch_review.py --interactive

pr_data = {
    'number': 123,
    'title': 'Добавить аутентификацию',
    'body': pr_body_with_dod,
    'diff_url': 'https://api.github.com/repos/...',
}

verdict = workflow.run(
    patch_source='github_pr',
    patch_data=pr_data,
    pr_body=pr_data['body'],
)

mr_data = {
    'iid': 456,
    'title': 'Добавить аутентификацию',
    'description': mr_description_with_dod,
}

verdict = workflow.run(
    patch_source='gitlab_mr',
    patch_data=mr_data,
    pr_body=mr_data['description'],
)

from src.patch_review.dod import DoDExtractor

extractor = DoDExtractor()
dod = extractor.create_manual_dod(
    items=[
        "Функция работает как ожидается",
        "Отсутствуют уязвимости безопасности",
        "Добавлены модульные тесты",
    ],
    types=["functional", "security", "test"],
)