coderabbit.yaml

reviews:
  auto_review:
      enabled: true
      ignore_title_keywords:
        - "bump"
        - "WIP"
        - "DO NOT MERGE"
      drafts: false

  review_simple_changes: false

  # Exclude files that contribute noise to docstring / lint coverage
  # metrics. The thesis artifact lives under src/ and tests/; everything
  # else is generated, locked, or non-Python data we don't want CodeRabbit
  # gating PRs on.
  path_filters:
    - "!**/*.lock"
    - "!**/*.min.js"
    - "!**/*.min.css"
    - "!**/dist/**"
    - "!**/build/**"
    - "!**/.venv/**"
    - "!**/__pycache__/**"
    - "!**/*.db"
    - "!data/**"          # benchmark inputs + ground truth, not code
    - "!**/*.MMD"
    - "!**/*.JSON"

  # Global instructions
  instructions: |
    - Suggest using 'f-strings' over older '.format()' or '%' formatting.
    - Watch for mutable default arguments in functions (e.g., def func(a=[])).
    - LLM provider classes (subclasses of LLMProvider in src/maestro/providers/) must
      use the 'Provider' suffix (e.g. AnthropicProvider, MistralProvider). This avoids
      name collisions with vendor SDK client classes (anthropic.Anthropic,
      openai.OpenAI, mistralai.Mistral).
    - Strategy classes (subclasses of base.Strategy in src/maestro/strategies/) must
      use the 'Strategy' suffix (e.g. SingleAgentStrategy, SOPStrategy).
    - Docstring scope for this codebase: substantive prose on classes,
      public functions, and any function whose behaviour is not obvious
      from name + type signature. Do NOT flag missing docstrings on:
      __init__ methods that only call super() and assign attributes;
      single-line private helpers (_name) when the parent class has
      module/class-level prose explaining the pattern; trivial @property
      accessors. Consistency with existing codebase style takes
      precedence over universal coverage: this is a thesis artifact,
      not a library API.