committed 11:27PM - 23 Jun 26 UTC
* Chore: Add CodeRabbit configuration
The CodeRabbit GitHub app is installed bu…t the repository has no
`.coderabbit.yaml`, so every PR gets a "Review skipped, auto reviews are
disabled" status instead of an actual review.
This adds a committed config so review behaviour is config-as-code for all
contributors:
- `auto_review.enabled: true` re-enables automated PR review.
- `review_status: false` silences the recurring "review skipped" status
comment.
- `tools.ruff.enabled: true` reuses the linter already run in `poe lint`.
- `path_instructions` encode two intentional conventions so they are not
flagged as defects: lazy heavy/optional imports (to keep the CLI fast) and
the deliberately unbounded line length (ruff E501 disabled). They also note
that type checking is `ty`, not mypy, and that tests are co-located as
`*_test.py`.
- `profile: chill` keeps review noise low; it can be raised to `assertive`.
* Align auto_review with org-wide setting (disabled, on-demand)
amotl intentionally disables CodeRabbit auto-review org-wide to conserve
limited OSS review cycles, preferring on-demand reviews triggered via
`@coderabbitai review` or the "Please review now!" checkbox (available
to org members). Set auto_review.enabled: false and drafts: true to match,
and drop base_branches (moot while auto-review is off). The rest of the
config (chill profile, path_instructions, ruff) still applies to triggered
reviews.
* Chore: Update CodeRabbit test path instruction for new tests/ layout
PR #79 moved the co-located `*_test.py` files into a dedicated `tests/`
tree (`tests/main/` for core/unit, `tests/saas/` for SaaS integration,
named `test_*.py`). The existing `omniload/**/*_test.py` path
instruction no longer matched any files, and its text described the old
co-located layout. Repoint the glob at `tests/**/test_*.py` and update
the guidance to describe the new structure.