Manager-Database/.coderabbit.yaml at main · stranske/Manager-Database · GitHub

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
# CodeRabbit configuration — https://docs.coderabbit.ai/getting-started/configure-coderabbit
# Fleet default, synced from stranske/Workflows (templates/consumer-repo/.coderabbit.yaml).
# Tuned for the stranske autonomous-agent fleet:
#   - assertive review of substantial agent-authored PRs
#   - advisory / non-gating, matching the fleet rule that review never blocks a merge
#   - maintenance-bot, generated sync/release, and draft PRs do not spend review budget
#   - path guidance concentrates review depth on core source, migrations, and workflow YAML
language: en-US
reviews:
  profile: assertive
  request_changes_workflow: false      # advisory: never auto-block a merge (fleet rule)
  high_level_summary: true
  poem: false
  auto_review:
    enabled: true
    drafts: false                      # opener creates drafts; review when marked ready
    # Skip automatic-review budget on maintenance-bot PRs (dependency bumps, CI/version
    # syncs) and generated sync/release PRs. agents-workflows-bot is intentionally NOT listed
    # — it authors substantial agent code PRs (e.g. "Agent belt for #N") that should still be
    # reviewed; its release-please PRs are caught by the "chore(main): release" title filter.
    ignore_usernames:
      - "renovate[bot]"
      - "dependabot[bot]"
      - "github-actions[bot]"
      - "stranske-keepalive[bot]"
    ignore_title_keywords:
      - "chore(main): release"
      - "chore: sync workflow templates"
      - "sync workflow templates"
    labels:
      - "!sync"
      - "!workflow:source-sync"
      - "!workflow:source-maintenance"
      - "!consumer-sync"
      - "!integration-sync"
      - "!workflows-sync"
      - "!template-sync"
  path_filters:
    - "!**/*.lock"
    - "!**/vendor/**"
    - "!**/.workflows-lib/**"
  path_instructions:
    - path: ".github/workflows/**"
      instructions: >-
        Flag template-injection, unpinned third-party actions, and spoofable bot-actor
        checks — this workflow YAML is synced across consumer repos, so one bug
        replicates fleet-wide.
    - path: "**/*.py"
      instructions: >-
        Prioritize correctness, error handling, and test coverage. Flag new or changed
        behavior with no accompanying test, silently swallowed exceptions, and unguarded
        NaN/None propagation in numeric or scoring code.
    - path: "**/alembic/**"
      instructions: >-
        Verify migrations are reversible and portable across SQLite (CI) and PostgreSQL
        (prod): keep constraint/index identifiers within Postgres' 63-char limit, use
        op.f() for auto-named constraints, and guard dialect-specific SQL. Flag a
        downgrade() that does not invert upgrade().
    - path: "**/migrations/**"
      instructions: >-
        Verify migrations are reversible and portable across SQLite (CI) and PostgreSQL
        (prod): keep constraint/index identifiers within Postgres' 63-char limit, use
        op.f() for auto-named constraints, and guard dialect-specific SQL. Flag a
        downgrade() that does not invert upgrade().
chat:
  auto_reply: true