Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
140 changes: 140 additions & 0 deletions .coderabbit.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,140 @@
language: en-US
tone_instructions: ''
early_access: true
enable_free_tier: true
reviews:
# Review style + summary placement
profile: assertive
high_level_summary_placeholder: "<!-- coderabbit-summary -->"
review_status: false
collapse_walkthrough: true

# Noise reduction (these default to true; we explicitly turn them off)
changed_files_summary: false
sequence_diagrams: false
estimate_code_review_effort: false
related_issues: false
related_prs: false
suggested_labels: false
labeling_instructions: []
auto_apply_labels: false
suggested_reviewers: true
auto_assign_reviewers: false
in_progress_fortune: false
poem: false
enable_prompt_for_ai_agents: true

# Auto-review behavior
auto_review:
enabled: true
drafts: true
auto_incremental_review: true
base_branches: [".*"]

# Don’t auto-generate docstrings/tests (reduce chatter)
finishing_touches:
docstrings:
enabled: false
unit_tests:
enabled: true
Comment on lines +34 to +39
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

This note and setting disagree on whether unit-test generation is on.

Line 34 says this section is reducing chatter by not generating tests, but Line 39 keeps unit_tests.enabled: true. Either flip the setting or rewrite the comment so the file is not self-contradictory.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In @.coderabbit.yaml around lines 34 - 39, The comment and the settings
conflict: update the finishing_touches section so the comment and
unit_tests.enabled align; either set unit_tests.enabled to false to match "Don’t
auto-generate docstrings/tests (reduce chatter)" or change the comment to
reflect that unit tests are enabled. Locate the finishing_touches block and
modify the unit_tests.enabled value or the comment text accordingly, referencing
the keys finishing_touches, docstrings.enabled, and unit_tests.enabled.

pre_merge_checks:
docstrings:
mode: off
title:
mode: off
description:
mode: off
issue_assessment:
mode: off

# Lightweight, high-signal tools for this repo
tools:
# Audited languages: Rust, Shell, TypeScript/JavaScript, Protobuf, Markdown, YAML, TOML, Makefile.
ast-grep:
rule_dirs: []
util_dirs: []
essential_rules: true
packages: []
shellcheck:
enabled: true
markdownlint:
enabled: true
github-checks:
enabled: false
timeout_ms: 90000
languagetool:
enabled: true
enabled_rules: []
disabled_rules: []
enabled_categories: []
disabled_categories: []
enabled_only: false
level: default
biome:
enabled: true
yamllint:
enabled: true
gitleaks:
enabled: true
eslint:
enabled: true
buf:
enabled: true
actionlint:
enabled: true
semgrep:
enabled: true
clippy:
enabled: true
oxc:
enabled: true
dotenvLint:
enabled: true
checkmake:
enabled: true
osvScanner:
enabled: true
path_instructions:
- path: "{dlp-api,src}/**"
instructions: |
Treat any usage of `.unwrap()` or `.expect()` in production Rust code as a MAJOR issue.
These should not be categorized as trivial or nit-level concerns.
Request proper error handling or explicit justification with invariants.
- path: "{tests}/**"
instructions: Usage of `.unwrap()` or `.expect()` in test code is acceptable and may be treated as trivial.
chat:
# We leave emoji/ASCII art, but no auto-replies
art: true
auto_reply: false

knowledge_base:
# Avoid web-search commentary in reviews
web_search:
enabled: false
code_guidelines:
enabled: true
filePatterns: []
learnings:
scope: auto
issues:
scope: auto
jira:
usage: auto
project_keys: []
linear:
usage: auto
team_keys: []
pull_requests:
scope: auto
mcp:
usage: auto
disabled_servers: []
code_generation:
docstrings:
language: en-US
path_instructions: []
unit_tests:
path_instructions: []
issue_enrichment:
planning:
enabled: true
4 changes: 2 additions & 2 deletions .github/workflows/run-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ jobs:
steps:
- uses: actions/checkout@v4
- name: Run fmt
run: cargo fmt -- --check
run: rustup component add --toolchain nightly-x86_64-unknown-linux-gnu rustfmt && cargo +nightly fmt -- --check
Comment thread
snawaz marked this conversation as resolved.
- name: Run clippy
run: cargo clippy -- --deny=warnings
Comment thread
snawaz marked this conversation as resolved.

Expand Down Expand Up @@ -80,7 +80,7 @@ jobs:
run: |
cargo build

- name: run tests
- name: run tests (cargo test-sbf)
run: |
export PATH="/home/runner/.local/share/solana/install/active_release/bin:$PATH"
cargo test-sbf --features unit_test_config
Expand Down
Loading
Loading